noUIslider tooltips option is ignored.

Hello there!
It seems like all slider options are ignored. Can you please check this ?
I am using it the way, described in docs.
{
                    type: "noUiSlider",
                    label: "Volume level",
                    model: "volume",
                    min: 0,
                    max: 140,
                    sliderOptions: {
                      connect: "lower",
                      step: 10,
                      tooltips: [true],
                    },
}

Is anybody alive here ? :)

This is an open source project, maintained by volunteer on their free time, given to you and the rest world for free
Show some respect and patience, we are NOT at your service
This is the weekend
Provide a fiddle that reproduce the problem that you are facing, we are not telepathic wizard able to fix bugs by reading tea leaves

I'm going to assume that you are maybe new to this, and didn't realized your lack of basic respect by showing such impatient comportment.
So, please provide a fiddle that reproduce this bug, so that it is easier to fix and wait for an eventual answer. Again, we are not at your service.
Until then, I'm closing the issue.

Well said. VFG has become the core of my app. It’s a great contribution and I can’t wait to help further  its development.

Well, sorry if you did not understood me correctly. I posted this on friday morning, and on weekend I just updated this with new comment, that's all I did.
I also understand that you are not at my service, that's why I asked you for help not for someting else. And you're right I'm new with VFG. It's kinda strange that you reacted like that. II will provide a fiddle asap.

You don't seem to understand the problem with your attitude.
If you want to update a comment, there is an edit button on every comment box (the little pencil in the upper right).
The thing is, you did not update your comment to add details, show some progress or explain your issue better.
By adding another comment, you spammed every contributor with a notification, just to grab attention. This might be okay if you feel like you issue is being ignored for too long (at least a month of no anwser/tagging/activity), but even then, you should do this reasonably.

You should not be sorry that "I'm offended", you should be sorry for your lack of respect, and apologize correctly for spamming and begging for update. Stop rejecting the fault on others.
You made a mistake, this is not the end of the world.

In the end we both want to make VFG better, so let's do it ! 😄


Fix typo "Received" to "Receiver" in streaming-kafka-integration.md

Removed typo on line 8 in markdown : "Received" -> "Receiver"

Can one of the admins verify this patch?

Thanks - I've merged this.


Improperly interacting with features that use (DAE) Dynamic Active Effects

Features inside a source compendium for Hero Creation Tool don't seem to function properly if said feature has DAE effects. When an actor is created using the Hero Creation Tool, and features that use DAE are chosen; their effects do not work.
Here is a direct example. I create a feature called fireproof; I add a passive DAE effect that makes it so that the actor in possession of fire proof has resistance to fire damage. I add the feature to an actor. The feature functions as expected with no problems. I add the feature to the source compendium that the Hero Creation Tool uses as its race compendium. I then reference the feature in a race that is in the race compendium. I then create an actor using Hero Creation Tool with a race that references fireproof. Fireproof is added to the actor's features as expected. If I click on the fireproof feature inside the actor's feature tab; the DAE passive effect is still there. However, the automatic resistance to fire damage that should be provided by fireproof does not exist. Instead I will need to delete the fireproof feature, and then manually drag it in from the source compendium to make fireproof function as it should.
I've tested a few other DAE effects, and none of them seem to work if I create an actor that has those features using HCT. Features that are reaction based do not proc when the actor has a chance to make a reaction either. Sound implemented using automated animations appears to function as it should though.
The DAE effects still appear to be present if I open up the feature and click on the effects page. They simply dont function at all.
Here are my active modules.

Hope the bug gets fixed asap. Cheers mate.

Sorry for the delay! I'm investigating this now, and I seem to already found the root cause but can't figure out how to fix it just yet, but hopefully this shouldn't take long

got it!
the item list inside the actor data when calling Actor.create(newActorData) was not transfering the active effects. With help from ghost I split the items aside and created them as embedded documents on the already-created Actor, and it seems to have worked like a charm!


test(ThemeType): refactor test

Description

Improve test in making it more readable for further development.
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[x] 🧪 Tests
[ ] 🗑 Chore

cc @drepram

Here I resolved another issue related to themes:

By Lucas's instruction, I've modified the code in this branch too. Please take a look!

🎉


fix: lint error

Description

Eslint is throwing an unwanted, too-strict error since this error is just happening for a file, disabling it for the specific line makes more sense than disabeling it for entire package.

The error for some reason is not visible on local, but can be simulated on local with exact same configuration as mentioned in CI, I did it on node-20:alpine image to check the trace of the issue as it was;t visible on the dev env.

Checklist

[x] My code follows the contributing guidelines of this project.
[x] I am aware that all my commits will be squashed into a single commit, using the PR title as the commit message.
[x] I have performed a self-review of my own code.
[x] I have commented my code in hard-to-understand areas.
[x] I have updated the user-facing documentation to describe any new or changed behavior.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have not reduced the existing code coverage.

Comments

@SalihuDickson @git-anurag-hub @uniqueg Please review and approve this PR, it just ignore some eslint errors, shouldn't impact the functionality.
The error was too strict in forcing use of "this" keyword in this specific case and miscellaneous as it only popped in CI :man_shrugging:

With this you're disabling eslint entirely on a several lines, which is never advisable. This way we could miss other important linting issues and revisiting the issue to fix it later becomes difficult. Its best to figure out the issue, which in this case is a minor issue, so you can just convert it to a warning to make every violation easy to locate and address later
(depending on how important we consider strict modularization).
My recent build scripts PR address this issue, you can still make a separate PR so this issue can be resolved quickly but this approach less than ideal.

Yeah, makes sense the issue is Eslint is behaving weird here, I tried to figure out whats tripping eslint but since there are no trace in local environment, ie all the checks pass in in local but somehow seem to fail only in CI which I don't know why that's the case. Maybe you can take a look as well here.

Yh. I think there is a disparity in the dev and production rules. You can just change the code for "class-methods-use-this" rule to 1 (it currently defaults to 2) in the eslintrc for the lit tes package. That should fix it.
You can use this as a reference

yeah I did try it, it doesn't seem to work that's why I had to disable it this way.

@SalihuDickson maybe we can have it this way?

It works in my build scripts PR and again in this branch (i tried it again to isolate the solution). Perhaps is there something you could be doing wrong?

Yh. I think there is a disparity in the dev and production rules. You can just change the code for "class-methods-use-this" rule to 1 in the eslintrc for the lit tes package. That should fix it.
You can use this as a reference

By I did I mean disabling individually. Having the eslint rule to turn this into a warning globally doesn't seem ideal to me, maybe when this rule will affect more than 3 files (rule of 3), we can think about that. What do you think?

Like I explained above, making it a warning for the entire package will ensure we can see whenever a file is breaking the rule. So it can easily be revisited later. Hiding catch-all one-liners in individual files will make this particular violation and others harder to catch later.

Closing this PR, please read the above discussion and checkout PR #236 :) !!


Multilingual support

Hello,
Is there any intention to support internationalization (i18n) with GeoServer REST API?
For example, with publishing new layer from PostGIS, I can add Title:en and Title:de ..
https://github.com/geoserver/geoserver/wiki/GSIP-203
https://docs.geoserver.org/latest/en/user/services/internationalization/index.html
Thank you!

Hi @Youssef-Harby,
thanks for your question. Currently we (in the meaning of the current core devs) do not concrete plans to implement this. But in case this is a commonly used feature we are surely happy to receive a contribution for this. Maybe having some short discussion before, how this will look like will be helpful.


About 'maxAge'

Does it means the time to destroy session?

No, it just set the expire time of cookie. :)

@Secbone
Thank you!
And I have another issue.
How can I clear the session,I think the session is more and more.

@hezhongfeng You can write your clear code in destroy function in Store, or set a expire time in your storage.


Styling droppable positions

Hello, first of all thank you for your work, it's really useful!
I'd like to know if Is there any way of styling the droppable positions ?
eg: giving a border color to the place where you'll could drop the  item you're dragging
Thank you

I guess this is a duplicate of this one https://github.com/computerjazz/react-native-draggable-flatlist/issues/185


permission denied during file sync

What happened?
I get permission denied error during file sync
e.g.
dev:app sync  Waiting for initial sync to complete
dev:app sync  Initial sync completed
start_dev: initial sync: upstream: apply changes: apply creates: upload archive: after upload: rpc error: code = Unknown desc = untar all: decompress: create /app/.eslintrc.js: open /app/.eslintrc.js: permission denied

What did you expect to happen instead?
devspace dev should not hang in filesync but sync the files and open a shell in the dev pod
How can we reproduce the bug? (as minimally and precisely as possible)
run devspace dev
My devspace.yaml:
version: v2beta1
name: mtracegit

# This is a list of `pipelines` that DevSpace can execute (you can define your own)
pipelines:
  # You can run this pipeline via `devspace deploy` (or `devspace run-pipeline deploy`)
  deploy:
    run: |-
      run_dependencies --all                            # 1. Deploy any projects this project needs (see "dependencies")
      ensure_pull_secrets --all                         # 2. Ensure pull secrets
      build_images --all -t $(git describe --always)    # 3. Build, tag (git commit hash) and push all images (see "images")
      create_deployments --all                          # 4. Deploy Helm charts and manifests specfied as "deployments"
  # This is the pipeline for the main command: `devspace dev` (or `devspace run-pipeline dev`)
  dev:
    run: |-
      run_dependencies --all       # 1. Deploy any projects this project needs (see "dependencies")
      ensure_pull_secrets --all    # 2. Ensure pull secrets
      create_deployments --all     # 3. Deploy Helm charts and manifests specfied as "deployments"
      start_dev app                # 4. Start dev mode "app" (see "dev" section)

# This is a list of `images` that DevSpace can build for this project
# We recommend to skip image building during development (devspace dev) as much as possible
images:
  app:
    image: myorgcrdev.azurecr.io/mtrace
    dockerfile: ./api/Dockerfile

# This is a list of `deployments` that DevSpace can create for this project
deployments:
  app:
    # This deployment uses `helm` but you can also define `kubectl` deployments or kustomizations
    helm:
      # We are deploying this project with the Helm chart you provided
      chart:
        name: .devspace/chart-repo
      # Under `values` we can define the values for this Helm chart used during `helm install/upgrade`
      # You may also use `valuesFiles` to load values from files, e.g. valuesFiles: ["values.yaml"]
      values:
        image:
          repository: myorgcrdev.azurecr.io/mtrace
          tag: latest
        connectorDng:
          image:
            repository: myorgcrdev.azurecr.io/mtrace-connector-dng
            tag: latest
        global:
          postgresql:
            auth:
              createExistingSecret: true

# This is a list of `dev` containers that are based on the containers created by your deployments
dev:
  app:
    arch: amd64
    # Search for the container that runs this image
    imageSelector: myorgcrdev.azurecr.io/mtrace
    # Replace the container image with this dev-optimized image (allows to skip image building during development)
    #devImage: ghcr.io/loft-sh/devspace-containers/javascript:18-alpine
    devImage: ghcr.io/loft-sh/devspace-containers/typescript:18-alpine
    # Sync files between the local filesystem and the development container
    sync:
      - path: ./api/:./
        disableDownload: true
        excludePaths:
          - integration-tests/
          - src/

    # Open a terminal and use the following command to start it
    terminal:
      command: ./devspace_start.sh
    # Inject a lightweight SSH server into the container (so your IDE can connect to the remote dev env)
    #ssh:
    #  enabled: true
    # Make the following commands from my local machine available inside the dev container
    proxyCommands:
      - command: devspace
      - command: kubectl
      - command: helm
      - gitCredentials: true
    # Forward the following ports to be able access your application via localhost
    ports:
      - port: "9229"
      - port: "8080"
    # Open the following URLs once they return an HTTP status code other than 502 or 503
    open:
      - url: http://localhost:8080

# Use the `commands` section to define repeatable dev workflows for this project
commands:
  migrate-db:
    command: |-
      echo 'This is a cross-platform, shared command that can be used to codify any kind of dev task.'
      echo 'Anyone using this project can invoke it via "devspace run migrate-db"'
hooks:
  - events:
      - before:deploy
    command: if [ -d '.devspace/chart-repo/.git' ]; then cd ".devspace/chart-repo" && git pull origin main; else mkdir -p .devspace/chart-repo; git clone --single-branch --branch main git@github.com:myorg/mtrace-chart.git .devspace/chart-repo; fi

Local Environment:

DevSpace Version: 6.3.13
Operating System:  mac
ARCH of the OS: ARM64
Kubernetes Cluster:
Cloud Provider: local kind cluster using podman
Kubernetes Version:
(☸|kind-mtrace:mtrace)➜  ~ kubectl version
Client Version: v1.31.1
Kustomize Version: v5.4.2
Server Version: v1.31.0

Anything else we need to know?
If I enter the dev pod and type whoami , I  see user 1001 instead of root

I fixed it with this config
# This is a list of `deployments` that DevSpace can create for this project
deployments:
  app:
    # This deployment uses `helm` but you can also define `kubectl` deployments or kustomizations
    helm:
      # We are deploying this project with the Helm chart you provided
      chart:
        name: .devspace/chart-repo
      # Under `values` we can define the values for this Helm chart used during `helm install/upgrade`
      # You may also use `valuesFiles` to load values from files, e.g. valuesFiles: ["values.yaml"]
      values:
        securityContext:
          runAsUser: 0

but is it required to run the devcontainer as root?

@frank-bee
Currently, yes it's required to run the ghcr.io/loft-sh/devspace-containers/typescript:18-alpine image as root to avoid this issue. This image is provided for convenience, but if you have restrictions that prevent you from running it as root, then it should be straightforward to use the Dockerfile here as a starting point to customize it for your needs.
A common pattern we use is just define a "dev image" build in the devspace.yaml, since our development image needs quickly expanded outside the common use case.
Another solution might be to use pod patches to set runAsUser: 0. This should work with any deployment method / helm chart.


Architecture design

Goal

Distributed system
Low billing
High performance
Logging of every move

System Environment

Ubuntu 18.04 LTS
.NET Core 2.1
Microsoft Azure Storage

MySQL as persistence relational database in server.
Redis as cache application for performance purpose
RabbitMq as message broker
Microsoft Azure Storage as FOREVER persistence storage.
Admin site

ss


How to disable the mpi during the installation.

I have installed it successfully, but wheni run it. it feeds back the following error:
OSError: libmpi.so.20: cannot open shared object file: No such file or directory

Hi, @ztdepztdep. I think this somehow relates to the mpi4py installation (or, in particular, your MPI installation); see https://bitbucket.org/mpi4py/mpi4py/issues/134/install-issue-libmpiso20-cannot-open for some hints regarding how to debug mpi4py. Meanwhile, I agree with you that it is better to allow users to manually disable parmetis in the installation process via some environment variables (or something else, I will think about this).
I will work on this when I have some spare time. As a temporary workaround, you can install from the source and manually set mpi4py=None in conf.py (hope this works)...


Can't select current folder while inside

There is no way to use the alt+enter shortcut to echo current folder. As a workaround, I find myself having to navigate further down and press alt+enter on the .. entry.
I would suggest adding the . entry, or providing a new shortcut to select the current location.

More intuitive than you think because pressing alt+enter on the .. entry was the first thing that I tried.

Thank you for the suggestion, it was added in https://github.com/Araxeus/ls-interactive/releases/tag/v1.7.0
(You can now also download ls-interactive from cargo, if you have rust installed on your pc)

Thank you for implementing this! It's much easier to use now!

so how to stay in current folder while inside?


Use translations insead of Persian keywords or phrases

I've checked some views and noticed that you're using pure phrases like the exact Persian keywords in your views.py which is not recommended to be honest. A better approach is to use Django Translations instead. Using non-English keywords (specifically in the .py files) may cause Unicode issues on some system species and Python is actually working on that though.
Thanks for your cool project by the way. :heart: :)

Hello there.
Thanks for reporting the issue.
The point is that this project is completely native to Iranian users and schools. Is there a specific reason we should use the Django i18n?

Django Translation System not only provides your project, translation compatibility but it allows you to use non-English keywords and phrases in your actual scripts. The majority of the current developments are the base fundamentals of future implementations and features. That's the main reason in fact.


Suggested open for unknown attribute on module is placed inside the module

Suggested open for unknown attribute on module is placed inside the module
namespace Test

// Import missing reference
[<EditorBrowsable>]
module Code =
   // It ends up here while it should be placed outside the module to bring the attribute in scope.

   let x = ()

And here's another set of improvements for previously not supported cases for top-level modules: b827329, 5c0522a.


Backend api :add member to flat.

Pull Request Information
Relates to Github Issue: #52
What is the PR Type:  Feature Branch.
Please select type that applies from below:

feature

Describe the added/fixed behaviour:
Add the backend api and unit test for the api.
api : add user to one flat.
Checklist

[x] Pull request has suitable title, adequately described and associated with a github issue
[x] Branch name follows correct convention (e.g. feature/[id]-some-feature)
[x] Branch builds and executes with no errors
[x] Test cases has been generated for the feature and test cases pass
[ ] Updated relevant wiki/documentation

Due to some changes for the API. New tests are required too.


Fixed #26226 -- Made related managers honor the queryset used for prefetching their results.

PR for CI only, work and discussion tracked at https://github.com/django/django/pull/6159.

This won't work as soon a queryset with multiple results uses prefetch related. This is because the queryset added to each model instance related manager's cache is not bound to the instance in anyway. For example, given the following scenario:
first_parent = Parent.objects.create()
second_parent = Parent.objects.create()
first_child = Child.objects.create(parent=first_parent, name='Simon')
second_child = Child.objects.create(parent=second_parent, name='Loic')

first_parent, second_parent = tuple(Parent.objects.prefetch_related(
    Prefetch(
        'children',
        Children.objects.filter(name='Loic'),
    )
))

# This will fail because the prefetched children queryset is not bound to each p
self.assertEqual(
    list(first_parent.children.all()),   # Uses the prefetch results
    list(first_parent.children.all().all()),  # SELECT * FROM child WHERE parent_id IN (first_parent.id, second_parent.id) AND name = 'Loic'
)

# The first_parent.children.all().all() queryset should be SELECT * FROM child WHERE parent_id = first_parent.id AND name = 'Loic'

Right! We do need to bind to that exact instance. How about we call _filter_queryset(rel_qs), the resulting query will have IN and = but that should be optimized away by the db. That way we don't involve the .all() method which could be customised.

Given this is mostly a consistency fix and this queryset won't be used 99% of the time I guess using rel_qs could be a good compromise. Let's move the discussion to #6159.


Error when pressing "study now" on a deck

Running with latest update from the addon page. Error code:
Error
An error occurred. Please start Anki while holding down the shift key, which will temporarily disable the add-ons you have installed.
If the issue only occurs when add-ons are enabled, please use the Tools > Add-ons menu item to disable some add-ons and restart Anki, repeating until you discover the add-on that is causing the problem.
When you've discovered the add-on that is causing the problem, please report the issue on the add-on support site.
Debug info:
Anki 2.1.49 (dc80804a) Python 3.8.6 Qt 5.14.2 PyQt 5.14.2
Platform: Windows 10
Flags: frz=True ao=True sv=2
Add-ons, last update check: 2022-12-12 18:24:41
Caught exception:
Traceback (most recent call last):
File "aqt\webview.py", line 41, in cmd
File "aqt\webview.py", line 142, in _onCmd
File "aqt\webview.py", line 595, in _onBridgeCmd
File "aqt\overview.py", line 91, in _linkHandler
File "aqt\main.py", line 672, in moveToState
File "aqt\main.py", line 694, in _reviewState
File "aqt\reviewer.py", line 141, in show
File "aqt\reviewer.py", line 161, in refresh_if_needed
File "aqt\reviewer.py", line 209, in nextCard
File "aqt\reviewer.py", line 284, in _initWeb
File "aqt\webview.py", line 472, in stdHtml
File "aqt\hooks_gen.py", line 4099, in call
File "C:\Users\mertt\AppData\Roaming\Anki2\addons21\368380974\src\reviewer.py", line 66, in try_append_wrapper
reviewer.toolkit_wrapper = ReviewWrapper(reviewer, content, _try_get_current_did())
File "C:\Users\mertt\AppData\Roaming\Anki2\addons21\368380974\src\reviewer.py", line 112, in init
self.load_options(did)
File "C:\Users\mertt\AppData\Roaming\Anki2\addons21\368380974\src\reviewer.py", line 148, in load_options
self.append_marker_html()
File "C:\Users\mertt\AppData\Roaming\Anki2\addons21\368380974\src\reviewer.py", line 182, in append_marker_html
self.content.body += self.marker_html()
File "C:\Users\mertt\AppData\Roaming\Anki2\addons21\368380974\src\reviewer.py", line 171, in marker_html
.replace('marker_color', self.toolkit_config[Config.LEECH_COLOR])
KeyError: 'leechColor'

Fixed and scheduled for release (should happen in a bit)!
Thanks for the report!


[BUG] Crash the process when running recognize Vault

Description
When running the recognize Vault, it raises an error because the plugin cannot read the content in the Markdown file. The error log is provided below:
TypeError: Cannot read properties of undefined (reading 'position')
plugin:consistent-attachments-and-links:916 Error stack:
    at parseLink (plugin:consistent-attachments-and-links:14563:47)
    at testEmbed2 (plugin:consistent-attachments-and-links:14653:31)
    at eval (plugin:consistent-attachments-and-links:14729:29)
    at eval (plugin:consistent-attachments-and-links:14471:36)
    at async resolveValue (plugin:consistent-attachments-and-links:1040:16)
    at async eval (plugin:consistent-attachments-and-links:14218:23)
    at async resolveValue (plugin:consistent-attachments-and-links:1040:16)
    at async eval (plugin:consistent-attachments-and-links:14124:28)
    at async eval (plugin:consistent-attachments-and-links:1763:25)
    at async run (plugin:consistent-attachments-and-links:1795:18)

Steps to Reproduce
1 . Click on 'recognize Vault'
2. Notice that 'the plugin cannot read the Markdown content'
Expected Behavior
The plugin the smoothly run the recognize vault process.
Actual Behavior
The error log is provided below:
TypeError: Cannot read properties of undefined (reading 'position')
plugin:consistent-attachments-and-links:916 Error stack:
    at parseLink (plugin:consistent-attachments-and-links:14563:47)
    at testEmbed2 (plugin:consistent-attachments-and-links:14653:31)
    at eval (plugin:consistent-attachments-and-links:14729:29)
    at eval (plugin:consistent-attachments-and-links:14471:36)
    at async resolveValue (plugin:consistent-attachments-and-links:1040:16)
    at async eval (plugin:consistent-attachments-and-links:14218:23)
    at async resolveValue (plugin:consistent-attachments-and-links:1040:16)
    at async eval (plugin:consistent-attachments-and-links:14124:28)
    at async eval (plugin:consistent-attachments-and-links:1763:25)
    at async run (plugin:consistent-attachments-and-links:1795:18)

Environment Information

Plugin Version: [3.21.12]
Obsidian Version: [1.7.7]
Operating System: [Mac OS]

Attachments
The detailed photo can refer in below about this issue:

Confirmations

[X] I attached a video showing the issue
[X] I attached a sample vault where the issue can be reproduced
[X] I have tested the issue with the latest version
[X] I have checked for existing issues

@william123438 you tick checkbox that you attached the vault while you didn't

Please check the sample vault for your reference. Thank you. I produced this issue in the sample vault.
obsidian_sync_copy.zip

I'm seeing similar errors  for "Check Vault Consistency" and "Reorganize Vault" functions: TypeError: Cannot read properties of undefined (reading 'position')

Fixed in https://github.com/dy-sh/obsidian-consistent-attachments-and-links/releases/tag/3.21.13


 Please approve and merge.

Hi @aditya059, I cannot bypass the lint errors which are preventing the merge.
Looking at the output, these seem to be 3 simple fixes and it might be the easiest to just add these into this merge.
Please let me know whether that's a possibility.

@gvenzl
Fixed. Please merge

Great, thank you @aditya059!


added complete link for nodejs learning resource fixed 404 not found error

I have fixed a 404 page not found error. I have fixed the issue by changing  variable  node-guide value from
https://nodejs.dev/en/learn to https://nodejs.org/en/learn/getting-started/introduction-to-nodejs
error file :
/docs/latest/tutorial/tutorial-1-prerequisites.md
line no : 134
prev : [node-guide] : https://nodejs.org/en/learn/
added  [node-guide] : https://nodejs.org/en/learn/getting-started/introduction-to-nodejs
https://github.com/user-attachments/assets/2957bcd9-ee49-42da-9d2b-43ed2d57e451

Hi @Anujverma89, this needs to be fixed upstream at https://github.com/electron/electron/blob/main/docs/tutorial/tutorial-1-prerequisites.md


Referenced issues not linked correctly

When I look at a commit with a URL like this that as a reference to an issue:
https://server.com/{org}/{repo}/commit/{commit-hash}
The link to the issue is not set correctly. It is missing the repository. The link I expect would look like this
https://server.com/{org}/{repo}/issues/18
But the link I get actually looks like this:
https://server.com/{org}/issues/18

This appears to be everywhere where issues are linked.

Looks like this is a duplicate if #2229.


Sharing failed when using Windows path

When using "E:\document\temp", the valid path cannot be divided correctly

The issue will be resolved in next release.


Test potential mirror node paging history limit

Problem
The Mirror Node plans to rollout the new sharded database in its 0.114 version, and as result users will not be able to see transaction histories beyond certain size with a single API call.
The range may apply to

/api/v1/accounts/{id}
/api/v1/contracts/results
/api/v1/contracts/results/logs
/api/v1/transactions

However, it’s not applied to the /contracts/{id}/results endpoint
Solution
Review the code and confirm if there are flows where we utilize time ranged calls on the affected endpoints.
We should double check for the use of the next link in MAPI list calls.
These must be honored beyond the size of the response page.
Additionally, take a local node env and utilize main or an alpha or rc version of 0.114 mirror node and configure the new env variable hedera.mirror.rest.query.bindTimestampRange to something smaller and enable hedera.mirror.rest.query.maxTransactionsTimestampRange for easier testing.

verify acceptance tests run with no issues
verify newman

Alternatives
No response

@victor-yanev I see this ticket has been moved to Done in the board, could you please confirm if this is resolved? If yes please close the ticket as complete.


App Version Updated to Version 1804

The application version has been updated from version <IP_ADDRESS> to version 10.0.1804.0. I've not updated the major or minor version; so it's still version 10.0 as of now. The 1804 signifies that it is the April 2018 code.

Since there is no automated versioning system implimented yet, this allows us to at least easily update the version without worrying about whether to increment the major or minor version.
Note: I originally wanted to go with something like 180418, (YYMMDD) to include the day. This way, whoever commits a code, they would just update the version number to reflect the date. Then it'd be super easy to tell which "build" one is runnign for troubleshooting reasons. But I went with just YYMM (1804) because we'd only need to increment the version at least once a month.
But I'm interested to hear what you think of how versioning should be done.

I appreciate the idea of date-based versioning scheme a lot. <3
Will the granularity of year + month be enough? Probably, but maybe we could just use one additional digit in release version number to indicate week of the month, eg. today would be 10.0.18044 (or 10.0.18043, if we're counting from 0). We can't be sure if we're not going to release two versions on the same month.
As we're talking about the fourth (build) version number, we could use it to indicate the number of commits in master branch since the previous release, but that would make the repo manager/owner increment it on every accepted pull request. (It would make 10.0.18044.77 then). But I'm afraid that I am overcomplicating right now...

We definitely need a versioning scheme.  Hard coding it to res.rc does not seem like the right idea.
I'm still interested in appveyor.com; that should provide us a lot of flexibility.

@craigwi yes I agree. I intended this to be an interim/temporary thing. Currently, version <IP_ADDRESS> is hard coded. So, if something is going to be hard coded, I thought it would be something other than <IP_ADDRESS> to distinguish it from release. Perhaps others beg to differ?

And I think using appveyor thing is brilliant. Played around with it. It can be set up pretty easily, it appears.

Perhaps version should also include m or r to signify a master or retro build

@ZanderBrown Weren't we going to make the retro branch number 9.0?

@thecatkitty , yes we were, ignore me :slightly_smiling_face:

@thecatkitty I definitely wanted to do something with the very last revision number, and number if PR seems like an awesome idea. However, there's is one teeny tiny major problem.
It's something I cannot comprehend at all, and would LOVE it if someone knew why this problem I'm about to describe exists.
Windows Store, for whatever reason, no longer accepts Windows 10 packages that have a version number in the fifth (aka revision) section to be anything other than zero. I don't understand this at all. I had to redo my entire version in for one of my apps that I tried to submit to the store because of this restriction.
So, for example, a package of version, say, <IP_ADDRESS> will throw an error if you try to upload it to the store. You must increase its version number to something like <IP_ADDRESS>. Essentially, the last number must be zero.
This wasn't the case in Windows 8.x. And when you install or update apps in the store, you'll notice that all Windows 10 apps have a version with the last number being zero. And if you install an app that still hadn't yet been upgraded to target Windows 8, you can see that the version number either in the Store or the settings app likely has a non-zero value in its last number.
This means the last number is essentially useless as a versioning scheme. A non-zero value can only signify beta releases and be a package incapable of being uploaded to the store
Anyine have any idea for the reasoning behind this artificial restriction?

@NazmusLabs is the .exe updated on get hub?


Language mismatch in errormessages

Some Error messages are always in German (EMail, Website)

Looks like an error in  APEX
Itemtype Text Subtypes URL and E-Mail  seems to return error message wrong language, messages are not included in messages-files where other error-messages could be found.

Caused by browsers.
The messages are build in the browser. Unfortunatelly they appear in the GUI-language of the browser, but not in the language the pages are show.


ReadCalFile not changing calibration

I'm trying out a new calibration file and it looks like the ReadCalFile function isn't working as I expect.
I open up the file, then execute:
FragmentTree->Scan("GetCharge():GetEnergy():1.261460*GetCharge()-0.823068:1.26289*GetCharge()-2.024","ChannelNumber==64 && DetectorType<2")

The third column is what I expect the default calibration to be, and the fourth column is the new calibration contained in my new cal file. This scan produces:
************************************************************
*    Row   * GetCharge * GetEnergy * 1.261460* * 1.26289*G *
************************************************************
*       21 * 219.25976 * 275.76229 * 275.76258 * 274.87683 *
*       25 * 300.49850 * 378.24438 * 378.24554 * 377.47382 *
*       30 * 761.77838 * 960.13037 * 960.13042 * 960.01860 *
*       31 * 1092.5474 * 1377.3830 * 1377.3823 * 1377.7429 *
*       32 * 388.20599 * 488.88305 * 488.88264 * 488.23685 *

As expected, columns two and three are nearly identical. Then I load my new cal file with TChannel::ReadCalFile and see the confirmation of parsed 1111 lines. I execute the same scan and get:
************************************************************
*    Row   * GetCharge * GetEnergy * 1.261460* * 1.26289*G *
************************************************************
*       21 * 219.25976 * 275.76229 * 275.76258 * 274.87683 *
*       25 * 300.49850 * 378.24438 * 378.24554 * 377.47382 *
*       30 * 761.77838 * 960.13037 * 960.13042 * 960.01860 *
*       31 * 1092.5474 * 1377.3830 * 1377.3823 * 1377.7429 *
*       32 * 388.20599 * 488.88305 * 488.88264 * 488.23685 *

where I now expect columns two and four to be similar, but instead see that columns two and three are the same, indicating that the calibration hasn't been applied as I expected.
Anyone have any idea?

Can you print out the channel?
TChannel::GetChannelByNumber(64)->Print();
see if this changes when you load the cal.

Before:
GRG16WN00A	{
Name:      GRG16WN00A
Number:    64
Address:   0x0000110f
Digitizer:
EngCoeff:  -0.823068	1.26146
Integration: 0
ENGChi2:   0
EffCoeff:
EFFChi2:   0
TIMECoeff: 0	271.279	-0.739159
FileInt: 1
}
//====================================//

After:
//====================================//
GRSI [1] TChannel::ReadCalFile("47Ca_walk_corrected_newcal.cal")
parsed 1111 lines.
(Int_t)75
GRSI [2] TChannel::GetChannelByNumber(64)->Print()
GRG16WN00A	{
Name:      GRG16WN00A
Number:    64
Address:   0x0000110f
Digitizer:
EngCoeff:  -0.823068	1.26146
Integration: 0
ENGChi2:   0
EffCoeff:
EFFChi2:   0
TIMECoeff: 0	271.279	-0.739159
FileInt: 1
}
//====================================//

This is what my cal file shows:
GRG16WN00A	{
Name:      GRG16WN00A
Number:    64
Address:   0x0000110f
Digitizer:
EngCoeff:  -0.823068	1.26146
Integration: 0
ENGChi2:   0
EffCoeff:
EFFChi2:   0
TIMECoeff: 0	271.279	-0.739159
FileInt: 1
}
//====================================//

Sorry, screwed that up. This is what my cal file contains:
GRG16WN00A	{
Name:      GRG16WN00A
Number:    64
Address:   0x0000110f
Digitizer:
EngCoeff::  -2.024	1.26289
Integration: 0
ENGChi2:     0.000000
EffCoeff:
EFFChi2:   0.000000
TIMECoeff:  0.000000	271.279000	-0.739159
}

To make sure I was loading the correct file, I deleted most of the lines. The new cal file now has many fewer lines than the original and I see an appropriate reduction in the parsed XXX lines output, but still no change in the printed channel.

Strange. It works for me. Let me see if I can break it.

Best part of being a physicist: "Let me see if I can break it."

Can you delete everything from your cal file except that channel?

Still the same result.

Stupidity check for my functions calls:
GRSI [0] TChannel::GetChannelByNumber(64)->Print()
GRG16WN00A	{
Name:      GRG16WN00A
Number:    64
Address:   0x0000110f
Digitizer:
EngCoeff:  -0.823068	1.26146
Integration: 0
ENGChi2:   0
EffCoeff:
EFFChi2:   0
TIMECoeff: 0	271.279	-0.739159
FileInt: 1
}
//====================================//
GRSI [1] TChannel::ReadCalFile("testcal.cal")
parsed 14 lines.
(Int_t)1
GRSI [2] TChannel::GetChannelByNumber(64)->Print()
GRG16WN00A	{
Name:      GRG16WN00A
Number:    64
Address:   0x0000110f
Digitizer:
EngCoeff:  -0.823068	1.26146
Integration: 0
ENGChi2:   0
EffCoeff:
EFFChi2:   0
TIMECoeff: 0	271.279	-0.739159
FileInt: 1
}
//====================================//

This is really strange. It works perfectly on my side. Is there any way it is changing a different channel?

The mnemonic, channel number, and address are all the same.

I just mean internally. Like did someone hack in a plus/minus one somewhere to make channels go up to 64 for instance.

....
found it.
double colon.

Oh jeese...Brings up whether we should have the code say: I parsed this many lines but had no idea what you were talking about with this many lines."

Potentially.

Thanks for the assistance.


Rename getLogger() to get_logger()

What type of PR is this? (check all applicable)

[X] Refactor

Have you discussed this change with the InvokeAI team?

[ ] Yes
[X] No, because: trivial fix

Have you updated all relevant documentation?

[X] Yes
[ ] No

Description
It annoyed me that the class method to get the invokeai logger was InvokeAILogger.getLogger(). We do not use camelCase anywhere else. So this PR renames the method get_logger().

Why not just totally rename it?

I don’t generally like to change a published API, but I’ve gone ahead and renamed to get_logger() everywhere and removed the old call’s name.

How did this auto-merge if flake8 didn't pass?


Dynamic expressions "hidden" does not work on components located in a group.

Description of the bug
When I try to use the expression "hidden" which points to a component that is included in a Group, the expression does not work.

Group containing radio button component.

{
        "id": "d8a834ba-88e4-433f-8278-0d0474960cd1",
        "type": "Group",
        "children": [
          "fdacd8f7-a563-4d55-8dc6-27320ba44e76",
          "172b8fbb-bea6-4ac6-8979-429ba5823d87",
          "925f5744-d164-4ce0-9415-7237ad87f2cc"
        ],
        "textResourceBindings": {
          "title": "xDnqTKrIwvuE"
        },
        "maxCount": 0,
        "dataModelBindings": {}
      },
      {
        "id": "fdacd8f7-a563-4d55-8dc6-27320ba44e76",
        "type": "RadioButtons",
        "textResourceBindings": {
          "title": "CQVGjiuhq9Qx"
        },
        "dataModelBindings": {
          "simpleBinding": "Skjemadata.brukSosialeMedia.sosNettverkJaNei"
        },
        "optionsId": "jaNei",
        "required": false
      },

This Radio button should be hidden using the  "hidden": Expressions that point ot the component "fdacd8f7-a563-4d55-8dc6-27320ba44e76" that is a part of the code above.

{
        "id": "3e6a0cfd-a41f-4ee2-99d8-e81cbd241ee6",
        "type": "RadioButtons",
        "textResourceBindings": {
          "title": "Scesd4hEzy8V"
        },
        "dataModelBindings": {
          "simpleBinding": "Skjemadata.brukSosialeMedia.brukAvOvennevnte.utviklingJaNei"
        },
        "optionsId": "jaNei",
        "required": false,
        "hidden": [
          "notEquals",
          [
            "component",
            "fdacd8f7-a563-4d55-8dc6-27320ba44e76"
          ],
          1
        ]
      },

Feilmelding i konsollen.

The expression work when i remove the  "fdacd8f7-a563-4d55-8dc6-27320ba44e76" from the Group.

Steps To Reproduce

clone this repo  https://altinn.studio/repos/ttd/ra-0419/src/branch/errorHidden
run the app local.
page  Bruk-av-nettside.json contain the expression that fails.

Additional Information
No response

I have looked into this a bit, and it appears that the problem lies in the LayoutNode.closest()-method. This function only looks at itself, its immediate siblings and then traverses upwards. This means that if, as in this case, the component you are looking for is not an immediate sibling of any of the parent nodes going upwards, but instead the child of such a sibling, it will not be able to find it.
@olemartinorg The comment above the closest()-function clearly states that it only checks upwards through the hierarchy, but I suppose you would expect the component-expression to be guaranteed to find the component as long as it is properly defined?
@andreas-didriksen A workaround is simply referring to the dataModel directly instead of referencing components. But it looks like you have figured that out already.

@bjosttveit Yup, I agree with your assessment. I don't think we should change the meaning of closest(), but I guess the expressions engine should make one more attempt at finding a component if it's not among the closest ones.
Just assuming, but I think this could work in the component lookup function (defined in src/featues/expressions/index.ts):
       const node = this.failWithoutNode();
-      const component = node.closest((c) => c.id === id || c.baseComponentId === id);
+     const closestComponent = node.closest((c) => c.id === id || c.baseComponentId === id);
+     const component = closestComponent || (node instanceof LayoutPage ? node.findById(id) : node.top.findById(id));
       const binding = component?.item?.dataModelBindings?.simpleBinding;

If you want to look into it, make sure to create a shared unit test for the expressions engine (look into src/features/expressions/shared-tests/functions/component/). Bonus points if you copy the tests over to app-lib-dotnet and make sure they run there as well! I usually use Meld for diffing the folders in the two projects when I sync tests between them.

Something like that should do it

@andreas-didriksen this should be fixed now in main.


I've found a bug

Bug: There cannot be bugs.
But it is a very good software. I, use it for my 200+ member network. Good work. 5 Stars.

It's not a bug, it's a main feature


[Bug] Only suffort '\n' for line delimiter when streamload

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
1.1.0
What's Wrong?
For now, when we use http inputstream to do stream load, we can only use  \n as line delimiter，otherwise, doris will return json line parse error.if one column has '\n' internal，it will cause another line parse error. i think we should think about it to avoid them.
What You Expected?
we can use another special characters for line delimiter.
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

doris support config line_delimiter.

doris support config line_delimiter.

I had done some test , line_delimiter can only use ‘\n’

doris support config line_delimiter.

I had done some test , line_delimiter can only use ‘\n’
support line_delimiter
https://doris.apache.org/zh-CN/docs/data-operate/import/import-way/stream-load-manual

@Wilson-BT

is there any update for this problem, I found the same。I had the same problem in 1.0.0 version

is there any update for this problem, I found the same problem in 1.0.0 version I my case, if i config the line_delimiter, the streamload task is success, but the data can not be queried。 Even if I use the json format，it is still not effect

what does you mean "the data can not be queried"?


Request: Add variable for today's date

I have a thing in the frontmatter called "DateStarted" that I previously combined with the Templater plugin, so that when I insert that template it would automatically add today's date, in the format YYYY-MM-DD.
would it be possible to add something to this plugin? So that when you add a book note it automatically gets today's date.
Thanks :)

Would this work for you? DateStarted: {{DATE:YYYY-MM-DD}}
You just put it in the template.


Toc Restructure

Restructure table of contents. Left toc represents file tree toc and allows user to quickly navigate to other pages. Right toc represents in-page section contents.

Two exceptions:

Index page doesn't need right doc since it is the same as left one:

API page concatenates right doc into left one:

Also add highlighting for navigation entry and toc entry to show users their current position.
@mli @zackchase @piiswrong

I like the new TOC a lot, thanks.
It needs a force fresh to make the new TOC appear
There are two tiny issues

TOC is missing here http://mxnet.io/tutorials/python/mnist.html
Unable to unfold an item through clicking

For http://mxnet.io/tutorials/python/mnist.html page, the reason is that in this example we are using a higher version of jquery which is not compatible with bootstrap. Maybe we can modify our notebook to fix this.
Will add unfold on clicking.


feat: add AdEvent enum to have an exhaustive list of all possible AdEvent values

To test this PR, you have to enable ads in your project, see:
https://react-native-video.github.io/react-native-video/component/props#adtagurl (it appears the link needs to be fixed too) and iOS and Android parts of https://react-native-video.github.io/react-native-video/installation

On iOS, add this to your Podfile:

$RNVideoUseGoogleIMA = true

On Android, add this to buildScript.ext section of your android/build.gradle file:

        RNVUseExoplayerIMA = true

Then, you should be able to do:
import { AdEvent } from 'react-native-video'

const MyComponent = () => {
  const onReceiveAdEvent = (adEvent: AdEvent) => {
    if (adEvent === AdEvent.COMPLETED) {
      console.log('Ad completed!');
    } else {
      console.log(adEvent);
    }
  };

  return (
    <Video
      adTagUrl='https://pubads.g.doubleclick.net/gampad/ads?iu=/21775744923/external/vmap_ad_samples&sz=640x480&cust_params=sample_ar%3Dpremidpostoptimizedpodbumper&ciu_szs=300x250&gdfp_req=1&ad_rule=1&output=vmap&unviewed_position_start=1&env=vp&impl=s&cmsid=496&vid=short_onecue&correlator='
      onReceiveAdEvent={onReceiveAdEvent}
    />
  );
};

@freeboub yes exactly, because otherwise the types won't be accurate for iOS! :/


[Bug] [SFTP] SFTP connector miss fs.sftp.keyfile option

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
I find we cannot set fs.sftp.keyfile in sftp connector, this means we only support use username and password to create ssh connection, but I find in the source code, there exist fs.sftp.keyfile
https://github.com/apache/seatunnel/blob/c23070919c8abde6e95e42c28095ca318dddc5e7/seatunnel-connectors-v2/connector-file/connector-file-sftp/src/main/java/org/apache/seatunnel/connectors/seatunnel/file/sftp/system/SFTPFileSystem.java#L120-L132
so this is a missing option?
SeaTunnel Version
dev
SeaTunnel Config
-

Running Command
-

Error Exception
-

Zeta or Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Can you test whether that code can run normally with a blank password

Can you test whether that code can run normally with a blank password

Is this related with this issue?

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。


[BUG] "Class tymon.jwt does not exist" - Laravel 5.4

In raising this issue, I confirm the following (please check boxes):

[x] I have read and understood the contributors guide.
[x] I have checked that the bug-fix I am reporting can be replicated, or that the feature I am suggesting isn't already present.
[x] I have checked the pull requests tab for existing solutions/implementations to my issue/suggestion.

My familiarity with the project is as follows (check one):

[ ] I have never used the project.
[x] I have used the project briefly.
[ ] I have used the project extensively, but have not contributed previously.
[ ] I am an active contributor to the project.

My project stack details

Framework (Laravel):
Framework Version: 5.4
JWT Auth Guard Version: ^1.0
tymon/jwt-auth Version: 0.5.*

Laravel 5.4 throws the error "Class tymon.jwt does not exist" after installing jwt-auth-guard and configuring it as described in the docs.

Same here

ReflectionException: Class tymon.jwt does not exist in /src/vendor/illuminate/container/Container.php:719

Finally! I was outdated.
You need this version in composer.json, not 0.5!

"tymon/jwt-auth": "^1.0@dev",

So, the Provider now is:
$app->register(Tymon\JWTAuth\Providers\LumenServiceProvider::class);
And you must publish your config/jwt.php again to avoid conflicts.
(I just copied from this gist: https://gist.github.com/jonsa/90a8bf9b4ecac7f9dd2d)

Thanks @libasoles
Got it, closed!


Fix typo

Thanks for the fix. Be sure to check out your name as a contributor about the 3rd of next month on the What's new page in the Community contributors section at the bottom.


Add trackEventWithOptionsAndMeasurements method

Tracking events with name doesn't really allow us to provide information around these events. I have added a new method called trackEventWithOptionsAndMeasurements which plugs into the hockey app method for providing options and measurements with each event.

Hey @martincik !
What's the status of this pull request? Are you going to merge it soon?

I tried to reach @brianfoody with a feedback, but he's not replying and I want to know the answers before I merge it. I don't want to merge this just because it's contribution.
@brianfoody Can you get back to me with the feedback.
BTW: Thanks so much for the contribution. Very much appreciated.


STATIC : static

Hi, please assign me issue #190 STATIC : static

Hi @NripeshN,
I am eager to contribute to this issue by implementing support for the STATIC keyword in the DirectX backend. Could you please assign it to me?
Thank you!

@CrossGL-issue-bot assign me

Assigning @Firaas-Ahmedkhn as I believe they have already created a PR for this issue.


Controlling table update frequency

This topic needs to be fully covered in Community.
https://deephaven.io/enterprise/docs/development/ltm/

This is : https://deephaven.io/core/docs/conceptual/query-engine/engine-locking/
We need to update Enterprise docs.

What actual work does this ticket entail? @margaretkennedy


Problem with kmongo-serialization

When tried to use a list with LocalDateTime
@Serializable
data class PointModel(
    @Contextual val _id: ObjectId,
    val name: String,
    @Serializable(with = LocalDateTimeSerializer::class)
    val datePoints: List<LocalDateTime>
)

i get this error
Serializer has not been found for type 'LocalDateTime'. To use context serializer as fallback, explicitly annotate type or property with @Contextual

try to use this
val AppJson = Json { serializersModule = IdKotlinXSerializationModule }

But I need it to be a list.

I fixed
val date : List<@Contextual LocalDateTime>


Suggestion of an additional benefit offered by Power query

[In the paragraph entitled "How Power Query helps with data acquisition" https://docs.microsoft.com/en-us/power-query/power-query-what-is-power-query#how-power-query-helps-with-data-acquisition, it would be helpful to add the benefit of "Delivery (or Sharing) of the cleaned, shaped processed data.". I think this would further enhance this section.]

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

ID: 6c9b1e7c-616b-87e0-7858-7f9e83b69ea5
Version Independent ID: 5ffba9ae-ac08-3375-df06-507e7b6336f3
Content: What is Power Query? - Power Query
Content Source: powerquery-docs/power-query-what-is-power-query.md
Service: powerquery
GitHub Login: @mllopis
Microsoft Alias: millopis

Great suggestion @Josh-BI-UK !
While Power Query is the tool that helps you create queries that shape your data, Dataflows is the component that lets you load the output of your queries into the destination of your choice and then consume such data using your tool of choice.
Perhaps we need another matrix like the one from the "What is Power Query?" article in the Dataflows article.

Please ignore (I'm using bot automation to add tags). #please-open

Thanks @Josh-BI-UK for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Ds9 display closing seconds after opening

I am trying to execute this code in a script:
#Create region file
    master_list = []
    if man_sources:
        for ogc in man_sources:
            master_list.append('circle {} {} 10 #color=red'.format(ogc[0],ogc[1]))
    if casjobs_sources:
        for ogc in casjobs_sources:
            master_list.append('circle {}d {}d 12 #color=green'.format(ogc[0],ogc[1]))
    #DS9
    viewer=imexam.connect()
    viewer.load_fits(fits_image)
    viewer.scale()
    for reg in master_list:
        viewer.set_region(reg)

When I run the script it opens ds9 and then displays the circles for a millisecond and then quits.  I have grabbed this newer version of imexam and the problem still persist.
I am using the 7.1 version of ds9.

I haven't had a chance to try and replicate this - but have you tried starting ds9 yourself and connecting to the window?
viewer=imexam.connect("name_of_ds9_window")  #which defaults to ds9, or you can name it when you startup ds9. You can also give it the XPA name referenced in the menu

haven't been able to replicate, @asymone do you still have this issue?

I think this has been fixed


Consider CICD workflow run with no build jobs unsuccessful

Issue:
A CICD workflow run with no build jobs is considered successful.
Solution:
Check if the CICD has at least one build job.

Will merge this into ALDoc PR when merged


Block allocator is slow with large file trees

When the block allocator runs it traverses the whole file tree to find unused blocks.  This is slow.  On my set up I see around 125us per block read where each block is 4kb which works out to be 30ms per MB of data.  So on a large SD card this is a significant delay - 1.5 seconds with only 50 MB of data.  On a 16GB SD card that is half used this allocation time is around 4 minutes.  (Not sure if SD use is way outside the design bounds of littlefs).
The block allocator is run on demand.  So on the first file write when the write buffer was greater than the sector size I get a 1.5 second delay, and then every (lookahead_size * 8) blocks it runs again.  With only 50 MB data.
Two things will help me - forcing the block allocator to run so that I don't get a delay on the first file write, and sizing the lookahead_size so that it does not run in the middle of writing a file.  The first I can add easily enough, but the second is problematic if you don't know the exact size of the file you will write and the lookahead_size needs to grow as more blocks are used on the media.
Increasing the allocation size will help me, but I'll still run into this problem again eventually.  Currently I am using read_size = 512, prog_size = 512, block_size = 4096.  I'm not sure if that is typical for a SD card - I just tweaked the block size to get the read/write speed I needed (around 2MB/s).
Other ideas?  Breaking the SD down into volumes to reduce the file tree size.  Still the overall time is the same and I have fixed volume sizes and then another layer to translate block numbers.
I could force the allocator to run in a low priority thread, but then the device is essentially unusable for a time period.
I could cache the used block list in faster non volatile memory, which would completely defeat the purpose of a resilient file system.
More complex would be to maintaining two free lists, one in use and the other updated at the same rate as block writes.  When one list runs out switch them over.  The next list would need to be updated if any block write falls into its block range.  This would help the situation of running the block allocation in the middle of writing a large file but would not get around the need to run it initially.  Plus you'd still need to scan the whole file tree.
Why don't I use FAT?  Because even if you are careful with power loss detection and graceful shutdowns it will eventually corrupt.  I'd need to write a chkdsk which runs every boot (none exist for FAT on a microcontroller).
I'm hoping that I'm missing something simple here, as I would like to use littlefs.
Edit: Related conversations https://github.com/littlefs-project/littlefs/issues/463 https://github.com/littlefs-project/littlefs/issues/75
Edit2: I see this issue has been bought up and discussed many times.  I'll close this issue.

I'll close this as it has been covered in https://github.com/littlefs-project/littlefs/issues/75


Fix object modified errors

Description
Changes proposed in this pull request:

Update the correct status as early as possible in the controller instead of the doing this in the sub reconciliations. I think the controller is the better level to handle state updates for the CR. We can also think about moving the status handling to the controller package.
Add retry on conflict for update of status, lastAppliedConfiguration annotation and finalizer removal since those have been a constant source of this issue and we can't avoid it.
Avoid technical "exit status 1" error description in Istio CR when Istio installation in external process failed.
Removed result handling from Status handling to make its responsibility more clear.
Remove ErrorRetryTime to use the DefaultControllerRateLimiter behaviour instead of a linear retry time.

Related issue(s)

https://github.com/kyma-project/istio/issues/192

/retest pull-istio-operator-integration-gcp

/test pull-istio-operator-integration-gcp

/retest

/retest


just a noob question

how do i add this on alert view?

UIAlertController isn't really meant to have subviews added to it (and Apple's docs discourage it).

The view hierarchy for this class is private and must not be modified

There are some who still do it but I'd likely just create your own modal (example here or use something like SVProgressHUD


Switch  to c8 for coverage

c8 is quite more lightweight and probably faster too.

Wow, nice packagephobia feature!
nyc and c8 maintains the same guy and I heard in recent podcast with him that c8 is still far from perfect because v8 slow responses to contributions and a lot of cases still not covered.
Though I definitely like install size reduction.
Do you see any difference in report?

Have you thought about using coveralls, codegov or something similar? I can only judge from the summary report, and it seems with c8 the coverage is increased
------------------------------------|---------|----------|---------|---------|----------------------
File                                | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
------------------------------------|---------|----------|---------|---------|----------------------
All files                           |   91.18 |    85.28 |    94.2 |   92.08 |

------------------------------------|---------|----------|---------|---------|----------------------
File                                | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
------------------------------------|---------|----------|---------|---------|----------------------
All files                           |   95.86 |    91.34 |   92.71 |   95.86 |

Your call about the final choice. I only use c8 in 2 other projects and I haven't had any important issues personally. :)

I had not success with coveralls setup. Looks like it's just broken.
I guess these a few percents are not covered features. Not a big difference. Probably good to go. It will be improved with a time anyway.

We don't need the app, we can just use the action :)
If you have added the repo to coveralls, I can make a PR later.

Not a big deal. I'm fine to look into CI or local output. Bot commits only add noise in notifications.

I agree hence why I always the comments disabled.
Anyway, not a big deal


Android User Experience.

What this does...:

This only works in Android with Chrome (maybe Firefox too). <meta name="theme-color" content="#21412a"> sets the theme color (the whole thing where the URL is in Chrome) to #21412a.
When adding a shortcut of the App on the Android Homescreen this will be the images used as an icon. (Change it to what you want and compress it)

{
      "src": "desktop.jpg",
      "sizes": "347x400",
      "type": "image/jpg"
}

"background_color": "#21412a" .. When opening  the App from the Homescreen this will be the background color while loading
"short_name": "Resource Center" is the App name in the Homescreen.
"name": "MarCom Resource Center" when you open the app this is the text that will be shown while the app loads.

More info here https://developers.google.com/web/fundamentals/web-app-manifest/
To test this:

In you .env file set the NODE_ENV to production.
Start you dev server.
In the console you will be shown a local IP address.
With an Android device connected to the same network as your server, open Chrome with that IP address BUT on PORT 9000.
Once the webpage displays in the upper right corner of Chrome click the 3 dots and from there select Add to homescreen or something similar.
Close Chrome, and go to your Homescreen and open the App.

There are still many things that can be done. Also, it may not work propely because the Service Worker need to be on a HTTPS server, but I had no issue.

Thanks @SebastianCApostolescu ! This is really cool. I had not even thought about doing this.


How to run yarn setup:oss on Windows?

When run it in cmd, it said '.\scripts\setup-env-for-oss' not recognized as an internal or external command

Hey there, right now our setup instructions assume a macOS machine with Xcode installed. You may be able to try working around with a docker image or virtual machine with macOS and Xcode but none of us have tried before. If you do have success would love to hear about it!


[prism] Dev prism builds for python  and Python Direct Runner fallbacks.

This change does two tasks:

Enables Python SDK developers to have prism automatically build and run locally using local changes to Prism when using PrismRunner.
Sets Prism to be attempted to be used in the Python DirectRunner in some circumstances.

The main change is to the prism_runner behavior when the SDK reports a .dev version.
In that case:

Try to go install github.com/apache/beam/sdks/v2/go/cmd/prism to the binary cache directory.
If that doesn't succeed, inspect the output for module errors

If there are no module errors, then the local version of prism couldn't compile, OR go isn't installed. Either way, this is a developer addressable error.
If there are module errors, then those modules don't have access to a version of Beam Go, so instead install the latest released version of prism.

This can be avoided by having a go.work file set up to direct Beam to a local version of the beam repo. This will be documented in a forthcoming SDK Development guide.

Arguably independently, is Attempt To Use Prism in the direct runner.

This currently only happens if we can't use the FnAPI runner.
If Prism fails on receiving the job, on job submission, then fallback to the DirectRunner instead.

Other changes are a refactor to reverse a few if blocks and return earlier, improving readability and reducing indentations.
Part of #32564
Fixes #32877.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

I discovered the root of my problem was using a newer yapf than Beam does, (0.40, instead of 0.29), so now I can have the formatting enabled on save, which will help! Local Linting on the other hand eludes me. I'm fortunately not tickling that as often.


Add support for Kafka Streams

I am trying to get a Spring Kafka application using Kafka Streams working. I'm getting a runtime exception when the code is trying to create a state store.
Caused by: java.lang.RuntimeException: librocksdbjni-linux64.so was not found inside JAR.
	at org.rocksdb.NativeLibraryLoader.loadLibraryFromJarToTemp(NativeLibraryLoader.java:110)
	at org.rocksdb.NativeLibraryLoader.loadLibraryFromJar(NativeLibraryLoader.java:78)
	at org.rocksdb.NativeLibraryLoader.loadLibrary(NativeLibraryLoader.java:56)
	at org.rocksdb.RocksDB.loadLibrary(RocksDB.java:64)
	at org.rocksdb.RocksDB.<clinit>(RocksDB.java:35)
	at com.oracle.svm.core.classinitialization.ClassInitializationInfo.invokeClassInitializer(ClassInitializationInfo.java:375)

Sample code here.
Here are the hints I've added to get things working to this point.
From looking at the quarkus implementation, it looks like they are registering the default key/value serdes, as well as the configured exception handler also. It would be good if the support here also did similar.

I am discussing that one with the GraalVM team.

Based on GraalVM team feedback, you can try to extract the native library from the jar and put it in the same folder as native-image, and if that doesn't work, add the current directory to the java.library.path. Please let me know how it goes.

Ok, I've just been using the 'bootBuildImage'  buildpack build up to now. I'm guessing I'll not be able to do the above using that approach?

I've extracted the librocksdbjni-linux64.so lib from the org.rocksdb jar and placed it in resources/META-INF/native-image.
I've done a quick test and I'm not reproducing the above issue.
I'm not sure if this is a good solution though. From looking at the quarkus code base, it looks like this type of issue will crop up in a few different areas for this project.

We should probably add dedicated support to do that automatically I agree.

I've extracted the librocksdbjni-linux64.so lib from the org.rocksdb jar and placed it in resources/META-INF/native-image.
I've done a quick test and I'm not reproducing the above issue.
I'm not sure if this is a good solution though. From looking at the quarkus code base, it looks like this type of issue will crop up in a few different areas for this project.

Sorry, I spoke too soon. I took another look at this, just extracting the .so didn't work

@sdeleuze We added the native support only for the regular Kafka binder, but not for Kafka Streams binder. There is this issue. I've seen this error from Kafka Streams binder while looking at it initially. I guess these are all related, do you know the status of the error on that rocksdb/jni issue?

@sobychacko Not 100% sure, but you can try to put librocksdbjni-linux64.so in resources/META-INF/native-image manually for now and see how it goes.

Hello all, Did anyone manage to solve this issue?

I didn't investigate it further anyway, sorry

This issue is now tracked by https://github.com/spring-cloud/spring-cloud-stream/issues/2323 and should target resolution leveraging Spring Boot 3.


Added CSS to style.css

Aligned items in the 5-Day Forecast card using Flex display and CSS.

Alignment in the 5-Day FOrecast weather card looks good. The app is functioning properly. No issues were found. Merging to the main branch.


Update build list to include an alert state

As raised in #4018, a better UI for a queue backup would be to show the affected builds in an alert state:

Show an alert state on each hanging build in the build list if build.state == triggered and build.date > 5m ago. No need for redis here
Alert state should be very minimal, a FA icon -- ie https://fontawesome.com/icons/exclamation-triangle?style=solid
Perhaps mouse over on icon shows a brief message "This build is still waiting to be built"
Link still goes to detail page
There should be a warning box (using the standard notification pattern!) that says something along the lines of "This build has not started yet. This can be due to heavy build volume or to a problem with Read the Docs. If the problem persists, contact us"

Where should the warning box be on the list page or the detail page?


IP Address of cloudflared client displayed in traefik logs

Hello,
I have cloudflared running in docker container and all tunnels are configured online in cloudflare.
In same docker stack i have traefik and nginx running, however when checking traefik access logs i se IP address of docker router as accessing the endpoint.
Middleware is configured in file provider and when looking at traefik api page, i see "cloudflarewarp" middleware loaded.
Here is the content of docker-compose.yml:
version: '3.8'

services:
  traefik:
    image: traefik:v2.10
    command:
      - "--api.insecure=true"
      - "--providers.docker=true"
      - "--providers.docker.exposedbydefault=false"
      - "--providers.file.directory=/config"
      - "--providers.file.watch=true"
      - "--entrypoints.http.address=:80"
      - "--entrypoints.https.address=:443"
      - "--certificatesresolvers.resolver.acme.dnschallenge=true"
      - "--certificatesresolvers.resolver.acme.dnschallenge.provider=cloudflare"
      - "--certificatesresolvers.resolver.acme.dnschallenge.resolvers=<IP_ADDRESS>:53,<IP_ADDRESS>:53"
      -<EMAIL_ADDRESS>      - "--certificatesresolvers.resolver.acme.storage=/letsencrypt/acme.json"
      - "--experimental.plugins.cloudflarewarp.modulename=github.com/BetterCorp/cloudflarewarp"
      - "--experimental.plugins.cloudflarewarp.version=v1.3.3"
      - "--accesslog=true"
      - "--accesslog.fields.headers.defaultmode=keep"
    environment:
      CF_API_EMAIL<EMAIL_ADDRESS>      CF_DNS_API_TOKEN: <secret_token>
    ports:
      - 80:80
      - 443:443
      - 8080:8080
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - /mnt/2TB/opt/controller/traefik/data/letsencrypt:/letsencrypt
      - /mnt/2TB/opt/controller/traefik/data/config:/config
    restart: on-failure
  cloudflared:
    image: cloudflare/cloudflared:latest
    restart: on-failure
    command: tunnel --no-autoupdate run --token <token>
  public_html:
    image: flashspys/nginx-static:latest
    labels:
      - "traefik.enable=true"
      - "traefik.http.routers.public_html.rule=Host(`domain.org`)"
      - "traefik.http.routers.public_html.entrypoints=https"
      - "traefik.http.routers.public_html.middlewares=hsts@file,cfip@file"
      - "traefik.http.routers.public_html.tls.certresolver=resolver"
      - "traefik.http.routers.public_html.tls.domains[0].main=domain.org"
      - "traefik.http.routers.public_html.tls.domains[0].sans=*.domain.org"
    volumes:
      - /mnt/2TB/opt/controller/public_html:/static
    restart: on-failure

Content of /config/middlewares.yml:
http:
  middlewares:
    cfip:
      plugin:
        cloudflarewarp:
          disableDefault: true
          trustips:
            - "<IP_ADDRESS>/32"
    hsts:
      headers:
        browserXSSFilter: true
        contentTypeNosniff: true
        forceSTSHeader: true
        stsIncludeSubdomains: true
        stsSeconds: 63072000
        frameDeny: true

Log when accessing domain.org:
controller-traefik-1  | <IP_ADDRESS> - - [24/Jan/2024:10:50:12 +0000] "GET / HTTP/1.1" 403 175 "-" "Mozilla/5.0 (Linux; Android 10; K) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Mobile Safari/537.36" 331 "public_html@docker" "http://<IP_ADDRESS>:80" 0ms

IP Address <IP_ADDRESS> is gateway IP of docker compose created network.
As for plugin configuration, I've tried following:
disableDefault: true
trustips:
  - "<IP_ADDRESS>/16"

disableDefault: true
trustips:
  - "<IP_ADDRESS>/32"

disableDefault: true
trustips:
  - "<IP_ADDRESS>/32" # IP of container running cloudflared

disableDefault: false

And i always get the same log output.
I would really appreciate assistance on this.

This can be closed but here's an idea to update docs to help anyone struggling with this issue later.
You need to set following to static config, in my case in cli:
--entryPoints.web.forwardedHeaders.insecure

OR (even better, more secure) specify origin IP, IP or CIDR of your cloudflared host:
--entryPoints.web.forwardedHeaders.trustedIPs=<IP_ADDRESS>/32,<IP_ADDRESS>

Make sure the web in your static config points to name of your entrypoint.


session.go - allow all keys to be optional for a domain

Allow all keys to be optional for a target domain and allow a complete session to be detected.  For example, using the following configuration snippet in a phishlet with other required auth_tokens:

domain: 'login.example.com'
keys: ['key,opt']

doesn't work with google. I tried compiling the code. Doesn't capture any cookies.

What was your phishlet configuration?  I don't want to troubleshoot the phishlet, however, would like to ensure the code changes work in all valid configurations.


feat(doc): do not italicize the @dev NatSpec

Component
Forge
Describe the feature you would like
Issue
forge doc adds an asterisk * to the Markdown paragraphs generated for the @dev NatSpec tag.
This is not great because it screws up the formatting for anything more complicated than a line, e.g.:
/// @notice Creates a stream with the provided start time and end time as the range of the stream. The stream is
/// funded by `msg.sender` and is wrapped in an ERC-721 NFT.
///
/// @dev Emits a {CreateLockupLinearStream} and a {Transfer} event.
///
/// Notes:
/// - As long as they are ordered, it is not an error to set a range that is in the past.
///
/// Requirements:
/// - `params.recipient` must not be the zero address.
/// - `params.totalAmount` must not be zero.
/// - `params.range.start` must not be greater than `params.range.cliff`.
/// - `params.range.cliff` must not be greater than `params.range.end`.
/// - `msg.sender` must have allowed this contract to spend at least `params.totalAmount` assets.
/// - If set, `params.broker.fee` must not be greater than `MAX_FEE`.
///
/// @param params Struct that encapsulates the function parameters.
/// @return streamId The id of the newly created stream.
function createWithRange(LockupLinear.CreateWithRange calldata params) external returns (uint256 streamId);

forge doc generates the following Markdown for the function above:
### createWithRange

Creates a stream with the provided start time and end time as the range of the stream. The stream is
funded by `msg.sender` and is wrapped in an ERC-721 NFT.

*Emits a {CreateLockupLinearStream} and a {Transfer} event.
Notes:
- As long as they are ordered, it is not an error to set a range that is in the past.
Requirements:
- `params.recipient` must not be the zero address.
- `params.totalAmount` must not be zero.
- `params.range.start` must not be greater than `params.range.cliff`.
- `params.range.cliff` must not be greater than `params.range.end`.
- `msg.sender` must have allowed this contract to spend at least `params.totalAmount` assets.
- If set, `params.broker.fee` must not be greater than `MAX_FEE`.*

It adds that extra *, which (i) doesn't work in this case because this is a paragraph with a list, and (ii) even for cases where it works (single lines), it looks ugly, IMO.
Solutions

Completely remove this asterisk.
Let users choose whether they want the asterisk via a config option under [doc].

Additional context
No response

Ah interesting. For context the rationale here was:

@notice and @dev are for different audiences, so it would be nice to distinguish them somehow so each type of reader knows what's relevant
The primitive-dodoc hardhat plugin puts @dev tags in italics as shown there, which is why we went that route (ref https://github.com/foundry-rs/foundry/pull/2701#issuecomment-1376009358).

Wonder if there's any alternative solutions that keep the visual separation but doesn't break formatting like this

Ah interesting. For context the rationale here was:

@notice and @dev are for different audiences, so it would be nice to distinguish them somehow so each type of reader knows what's relevant
The primitive-dodoc hardhat plugin puts @dev tags in italics as shown there, which is why we went that route (ref feat(forge): doc #2701 (comment)).

Wonder if there's any alternative solutions that keep the visual separation but doesn't break formatting like this

The primitive dodoc does not generate well formatted output to begin with. Look at the markdown tables it generates versus forge doc.
notice and dev are not exclusionary audiences, though. It's impossible for the author to presage what the reader think is relevant. Its up to the author to document what they assume to be relevant.
Styling should be decoupled from how the output is produced. Providing a template like mentioned by @PaulRBerg would be a great start. mdbook is already severely limited in terms of producing a usable documentation website. Better to focus on producing usable markdown output and leaving the rest of the styling to the end user.

Ah interesting. For context the rationale here was:

@notice and @dev are for different audiences, so it would be nice to distinguish them somehow so each type of reader knows what's relevant
The primitive-dodoc hardhat plugin puts @dev tags in italics as shown there, which is why we went that route (ref feat(forge): doc #2701 (comment)).

Wonder if there's any alternative solutions that keep the visual separation but doesn't break formatting like this

The primitive dodoc does not generate well formatted output to begin with. Look at the markdown tables it generates versus forge doc.
notice and dev are not exclusionary audiences, though. It's impossible for the author to presage what the reader think is relevant. Its up to the author to document what they assume to be relevant.
Styling should be decoupled from how the output is produced. Providing a template like mentioned by @PaulRBerg would be a great start. mdbook is already severely limited in terms of producing a usable documentation website. Better to focus on producing usable markdown output and leaving the rest of the styling to the end user.


Javascript "Rename symbol" does not rename destructured variables

VSCode Version: Code 1.16.1 (27492b6bf3acb0775d82d2f87b25a93490673c6d, 2017-09-14T16:38:23.027Z)
OS Version: Windows_NT x64 10.0.15063
Extensions: Extensions are disabled

Steps to Reproduce:
Given the following code:
let obj = { x: 'x', y: 'y', z: 'z' };
const { x, y, z } = obj;
console.log(x, y, z);  // "x" "y" "z"

If I execute the "Rename Symbol" command while the cursor is on the x property of obj and rename the property, the destructured variable is not renamed and the code is left in a invalid state.
let obj = { newNameX: 'x', y: 'y', z: 'z' };
const { x, y, z } = obj;
console.log(x, y, z); // undefined "y" "z"

I would expect the x variable to be renamed to newNameX and the usage of x in the console.log to also be renamed.
This problem also exists for Typescript code.
This appears to have been reported and fixed once before (#6227).

Reproduces without extensions: Yes

Thanks. I believe this is already tracked by https://github.com/Microsoft/TypeScript/issues/15787
Closing as upstream


\begin{DndMonster} from .tex export is not recogised when using the {dndbook} from rpgtex

Hello there,
I used your custom tool to create homebrew monsters for my 5e campaign.
To have a more concise db I'm putting everything into a .tex file to have a single pdf to use a Campaign Manual
I'm using rpgtex/DND-5e-LaTeX-Template, so I setup everything accordingly, but when trying to add your .tex it generated errors as reported in the log

LaTeX Error: Environment DndMonster undefined.
\DndMonsterType [etc.] Undefined control sequence.

% Changing book to article will make the footers match on each page,
\documentclass[10pt,twoside,twocolumn,openany,nodeprecatedcode]{dndbook}

\usepackage[english]{babel}
\usepackage[utf8]{inputenc}

\usepackage{lipsum}
\usepackage{listings}

\usepackage{background}
\usepackage{float}
\usepackage{graphicx}
\usepackage{ifoddpage}
\usepackage{multicol}

\usepackage{tikz}
\usepackage[singlelinecheck=false]{caption}
\usepackage{shortvrb}
\usepackage{stfloats}
\usepackage{fp}
\usepackage{xstring}

% Start document
\author{Melvasul}
\title{Campaign Manual}
\date{2024}

\begin{document}

% Your content goes here
\maketitle

\chapter{Monsters}
 \subsection{Bloodflies}
 Bloodflies are large flying insects armed with a poisonous stinger. They are
 usually found in large, but can also be found in grasslands. Their distinct
 sense of smell enables them to detect their prey despite their undeveloped
 visual organs. On the hunt, bloodfly swarms surround their victim and overwhelm
 it with a multitude of attacks. Once their prey is incapacitated, they feed on
 its blood, which is their main source of nourishment.
 \\[1\baselineskip]
 \begin{DndMonster}[]{Bloodfly}

	\DndMonsterType{Medium beast, unaligned}
	\DndMonsterBasics[
	  armor-class = {14},
	  hit-points = {\DndDice{6d8+6}},
	  speed = {30 ft. fly}
	]

	\DndMonsterAbilityScores[
	  str = 10,
	  dex = 18,
	  con = 12,
	  int = 2,
	  wis = 14,
	  cha = 4,
	]

	\DndMonsterDetails[
	  saving-throws = {DEX +6, WIS +4},
	  %skills = {},
	  %damage-vulnerabilities = {},
	  %damage-resistances = {},
	  %damage-immunities = {},
	  %condition-immunities = {},
	  senses = {Passive Perception 12},
	  languages = {},
	  challenge = {1},
	]

	% Traits
	% Actions
	\DndMonsterSection{Actions}

	\DndMonsterAttack[
		name=Sting,
		distance=melee,
		type=weapon,
		mod=+6,
		reach=5,
		%range=20/60,
		targets=one target,
		dmg=\DndDice{1d6+4},
		dmg-type=piercing,
		plus-dmg={\DndDice{1d4} poison},
		%or-dmg=\DndDice{1d6+2},
		%or-dmg-when={},
		extra={}
	  ]
	% Spellcasting

\end{DndMonster}

% End document
\end{document}

How would I solve this?
ValleyOfMines.log

hi! sorry for the delay!
Debugging LaTeX stuff is always a bit of a struggle for me. Funnily enough, the rpgtex branch does not actually work in my local installation (unsure why, seems like some out of date font packages in there?). So at some point i changed my local installation (texlive 2022) to use https://github.com/ashonit/DND-5e-LaTeX-Template as the dnd template base. That branch has been updated more recently, so i'd give that version of the template a try. If you're still running in to issues, it might be that the template wasn't fully loaded (the error in question here looks like the monster environment was missed somehow)

After a clean reinstall of my whole texlive environment, I found out it was a problem with the installation not being 100% correctly setup.
Now I'm encountering other problems such as
Package etoolbox Error: Toggle 'bool-bg' undefined
Package etoolbox Error: Toggle 'multitoc' undefined.
Package etoolbox Error: Toggle 'bool-footer-scroll' undefined.

but I'm 100% sure they are not related to your splendid tool or the library itself, thank you again!


add keystroke to scroll to the beginning and to the end of the input json data (CTRL+*)

@evandrix I got this features at https://github.com/simeji/jid/pull/52
You can scroll top and bottom of the json data by CTRL-T and CTRL-G.

done


Accept Collections of Groups

Also renames GroupAdapter.putGroup(Group) to add(Group) to be more consistent
with method naming

https://github.com/Genius/groupie/issues/39

👍


New Crowdin updates

Jon, how would I go abou that? On Crowdin I can't see how to change english base entries.
by Ivan:
Please, don't change localized keys directly in code. Better change the value on https://crowdin.com/project/freeaps-x
And I prefer that custum insulin settings are not in the main section.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

You can PM me the strings that need to be added and I'll add them to the Base language folder and then Crowdin will update the rest of the necessary files automatically

Aha, I see now. This change you're suggestion requires change to Ivan's code..

ok, thanks.


Fix view source and edit message popover behavior.

We now make sure that View Source and Edit Message options don't misbehave upon selection when feature is already active.
Earlier if View Souce and Edit Message were chosen after being chosen once, then the message narrow shifted to next message. Now we make sure that upon re-selection, the feature is closed.
Fixes #3802

Hi, I'll soon update this PR with changing necessary text in popover.
Thanks!

This PR has been updated.
Thanks!

@timabbot This PR has been updated again. Suggestions welcomed.
Thanks!

The code looks good; @rishig what are your thoughts on what we should call this menu item?

Thanks for finding and working on this @khantaalaman! I think the naming issue is hard enough that we should just remove the menu item. (Possibly we should just remove the chevron menu entirely while we're in view source or message edit mode? I doubt "Collapse" works properly in this mode, and I can imagine subtle future bugs with "link to this conversation" or other things we add to this menu in the future.)

That proposal works for me.  @khantaalaman would you be up for changing this to work that way?

@timabbott I would like to complete this up to perfection. I am unable to completely get what @rishig proposed. It would be really helpful if you could just explain it once again.
Thanks!

Sorry for being unclear! The proposal is to remove (hide) the following three icons when we are in message edit or view source mode:

Feel free to ping me on chat.zulip.org if it is still unclear!

HI @khantaalaman, just checking in. How are things going with this?

Closing this, given that the author is unresponsive and the approach we've landed on in discussion differs from what this does.


Adding files with basic infrastructure

Basic class structure for PipelineDP.

DPEngine with the constructor and 1 function aggregate() without implementation.

Also it contains:
a.BudgetAccountant (for using in DPEngine).
b.PipelineOperations (for using in DPEngine).

An example of using of DPEngine (not doing much yet)

Almost empty unittests

Can we move the tests to a test folder?

Sure, done.
Also README.md has been extended
PTAL


GEOSwift # TopologyException: side location conflict at 8.6666822676303692 49.6224086061181.

I use this geojson. It contains geometries (polygons and multi polygons) marking regions in Germany.
I want to find out in with region (geometry) a specific map location is contained in.
This is what I do (shortened at some places)
    public func regionIdFor(location: CLLocationCoordinate2D) -> (regionId: Int?, partregionId: Int?) {
        guard let features = try? Features.fromGeoJSON(GeoHelper.geoJSONURL) else { return (nil, nil) }
        guard let wayPoint = Waypoint(latitude: location.latitude, longitude: location.longitude) else { return (nil, nil) }
        for region in features! {
            if let regionGeometry = region.geometries?.first {
                // check if user location is within the region
                if regionGeometry.contains(wayPoint) {
                    // GF is the partregion id
                    guard var partregionId = region.properties?.value(forKey: "GF") as? Int else { return (nil,nil) }
                    print(partregionId)
//                    return (regionId, partregionId)
                }
            }
        }
        return (nil, nil)
    }

For some locations I see the error in my terminal:
GEOSwift # TopologyException: side location conflict at 8.6666822676303692 49.6224086061181.

Every time this error occurs the function above finds multiple containing geometries. However this should not be possible because when I display the regions on the map they do not overlap at all.
Why does this happen?

I use the geos isValid() function to find out that one of the polygons in the geojson is invalid. Its the one:
      "geometry_name": "THE_GEOM",
      "properties": {
        "GF": 111,
        "GEN": "Oberrhein und unteres Neckartal",
        "DES": "Pollenfluggebiet",
        "bbox": [
          7.51140615808012,
          47.5773391340742,
          9.17334188771982,
          49.6252264420181
        ]
      }

How can I find out what's wrong with it? How can I fix it?

I could find the bug in the geojson.
I added this to the Geometry extension, maybe this is also a good idea to add to the code of this repo:
    func isValid() -> Bool  {
        let isValid = GEOSisValid_r(GEOS_HANDLE, self.geometry)
        return isValid == 1
    }

@funkenstrahlen, good job! That was a nice detective work.
Could you create a PR adding the isValid property and some tests.
Cheers!

I see what I can do. But might take some time.


Portfolio Plan (beta) dates

Hi,
What dates does Portfolio plan use to align the epics/features. I observe that even after providing Starting Date and Target Dates it did not pic those dates.
I will be importing a set of epics & features and I would like the portfolio plan to honour those dates.
Please can you let me know what field to use for this.

Hi,
How can we mark PortFoli Plan as favorite?
Thank you!

Sorry, but this code in this repo is not currently being maintained. We recommend you use the Delivery Plans feature that is core to Azure DevOps.


Using handlebars inside the file headers does not work

I create all my markdown files with a header like
---
filename: {{ site.baseurl }}/contracts/Lock
slug: Lock
type: contract
---
Just a Test {{ site.baseurl }}

The usage of {{ site.baseurl }} inside the header gives me an error
(): did not find expected key while parsing a block mapping at line 2 column 1
The second usage outside of the header works just fine.
So how can I get the value of baseurl inside the header?
I build the header with a function in a helpers.ts class but did not find a way to access the value so I thought I could use the handlebar.
My workaround would be to hardcode the baseurl into my helpers.ts class but I do not like it.

Can you show the function you wrote in helpers.ts?

export function fileHeader(this: DocItemWithContext)
{
	if (this.nodeType === "ContractDefinition")
	{
		const dic = this[DOC_ITEM_CONTEXT];
		if (!dic.page) return "**helpers.fileHeader:page property must be present!**";
		const purePath = extractDocPath(dic);

		let ret = "---\n";
		ret += `filename: ${purePath}\n`;
		ret += `slug: ${dic.item.canonicalName}\n`;
		ret += `type: ${dic.item.contractKind}\n`;
		ret += "---";
		return ret;
	}
}

function extractDocPath(dic: DocItemContext): string
{
	const regex = /\.md$/;
	if (!dic.page) return "/index";
	let result = dic.page.replace(regex, "");
	if (result[0] !== "/")
	{
		result = "/" + result;
	}
	return "/my-contracts" + result;
}

If you output a Handlebars tag from a helper it will not be rendered.
I'm actually not sure where baseurl is coming from, so I don't know if this will help you, but you can access the global site variable in the helper like this:
export function fileHeader(this: DocItemWithContext, opts: any) {
  const site = opts.data.site;
  ...
}

@frangio Not so fast.
opts.data.site holds data regarding the contract.
baseurl is a variable taken from the _config.yml file in the doc root folder defining the documentation build properties.
I did not see a way to access this value inside the helper.ts

@migoldfinger Sorry, I'm not sure what _config.yml is but it is not handled by solidity-docgen in any way.


when load config file which contains chinese , "input length = 1" error will occur

only on windows os , I guess its a String.getBytes() problem, using defaultCharset

we have discussed this in DingDing.

Closing this issue due to no feedback.


Fix Typo

The height dimension didn't change in the example

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/Archive/CNTK-Evaluate-Image-Transforms.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


mcpwm phase difference appearing where none configured, and configuring has no effect (IDFGH-10561)

Answers checklist.

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

General issue report
Since I don't have a 2-channel oscilloscope, I've connected a simple LED across the esp32 mcpwm output pins. When connected between L and H of same phase (arm), The LED is off as expected (no dead-times configured), since both signals are HIGH and LOW at the same times, and toggle at the same time.
But if I connect it across the L pins across two phases (arms), since I haven't configured any phase difference, I expect these signals to also be identical hence the LED should be off (same duty is set on both phases). But actually, the LED is dimly lit at 500Hz, and it shines fully at 25kHz (reaches full intensity even before that), indicating a phase difference that increases with frequency.
I don't see a logical explanation for this. Even worse, when I try to configure a phase difference across the phases (arms) to try to neutralize this, I realize it has no effect, no matter what the phase difference. I configure 3 separate timers in the same mcpwm group, connect one operator to each timer, one comparator to each operator, and two generators to each operator. Then I configure the sync strategy as follows:
  const uint32_t PHASES=3;

  ESP_LOGI(tag_mcpwm, "Creating TEZ sync source from timer0");
  mcpwm_timer_sync_src_config_t timer_sync_config=
  {
      .timer_event = MCPWM_TIMER_EVENT_EMPTY,
      .flags= {.propagate_input_sync=1, },
  };
  ESP_ERROR_CHECK( mcpwm_new_timer_sync_src(mcpwm_timers[0], &timer_sync_config, &timer_sync_source) );

  ESP_LOGI(tag_mcpwm, "Set other timers sync to the first timer");
  mcpwm_timer_sync_phase_config_t sync_phase_config=
  {
      .sync_src=timer_sync_source,
      .count_value=static_cast<uint32_t>(phase_diff*mcpwm_ticks),
      .direction=MCPWM_TIMER_DIRECTION_UP,
  };

  for (auto phase=1;phase<PHASES;phase++)
  {
      ESP_ERROR_CHECK(mcpwm_timer_set_phase_on_sync(mcpwm_timers[phase], &sync_phase_config));
  }

  ESP_LOGI(tag_mcpwm, "Created TEZ sync source from timer0. phase diff ticks: %lu",sync_phase_config.count_value);

But no matter what value I use for sync_phase_config.count_value, the LED always stays at the same brightness, which only depends on frequency. Am I doing something wrong?

nevermind, I resolved it on my own after looking closely at the example. It turns out, the sync strategy must be set after the timers have been enabled and started, not before.