Expose a variable where each dependencies are cloned to

Is your feature request related to a problem?
I want to dynamically access the directory from an imported devspace.yaml file from another repository. As I can see, in imports the repositories are cloned into ~/.devspace/dependencies/<dynamically-created-dir>. I want to know in advance, what the <dynamically-created-dir> is, such that I can access certain files from that repository (scripts in my case).
Which solution do you suggest?
I suggest to expose a variable such that these dynamically created dirs are properly accessible.
Alternatively, I suggest to include another field in imports (for git repos) which can overwrite the <dynamically-create-dir> to a static one provided by the user through devspace.yaml.
Which alternative solutions exist?
None
Additional context
Ask if needed, please.
Code Example
version: v2beta1
name: devspace.yaml

# Import a template
# Depending on the URL and branch, tag or revision, the resulting path is different,
# which makes it difficult to "guess" the path in advance.
imports:
  - git: <URL-to-repo1>
    branch: main

vars:
  # this could be a possible runtime variable exposed
  # would resolve to: ~/.devspace/dependecies/<URL-to-repo1>-<revision>/<branch>/<tag>
  IMPORT_DIR_REPO1: ${runtime.imports.clonepath}
``

@marcm-ml Hello! I've been assigned to look into this.
Could you add more context on why you'd like to access these script files from your dependencies?
My first guess is that you'd like to execute them. There might be a more encapsulated way to do that using imported functions.
I put together an example where an import defines a function that calls a relative script.... and oops, it doesn't work. That's a shame because that would avoid the need to get the full path to where the dependency is cloned.
If we got that working would that avoid the need to access the paths of imports / dependencies or is there another reason for wanting that information?

Hey @lizardruss
I saw you comment as an email and I think all of my issues could be solved if the each devspace.yaml would be aware in which directory it is lying (not executed). For example, this could be a variable similar to DEVSPACE_NAMESPACE. If the devspace.yaml can then access this variable, we can just have relative imports based upon this. I think making this an internal variable would be quite easy as this could either be the path to the devspace.yaml where devspace dev is executed in or the cloned git path (available from getDependencID here) or the local path to an imported dependency (also easy to get from the underlying yaml file and its parent path)
However, this variable cannot be the same for my main project (located in ".") and the imported dependency ("~/.devspace/...") otherwise it is the same issue, e.g. we cannot have relative imports in the imported dependency. This could be achieved with runtimes variabel or by internally overwriting the variable at import time similar how we can overwrite vars for imported dependencies? But this might be trickier

@marcm-ml Yeah, I'm trying to arrive at a solution that won't be problematic. Do you need this for both dependencies and imports or only imports?
Originally I was thinking runtime variables of the form ${runtime.import[0].path} or ${runtime.dependency.dep-name.path} could be used. The limitation is that runtime variables are not available in pipelines, which might limit the usefulness. Referencing imports is also awkward, since its an array. It should be possible to add a pipeline function to read the runtime vars.. but will discuss with my colleagues before going that route.
Then I was thinking we could populate vars from runtime variables, like your example, but that's not currently possible due to config load sequencing. There's a similar issue with the built-in ${DEVSPACE_*} style variables, since their definition is static and needs to happen before loading dependencies occurs, IIRC.
Next, the question you saw (but then I deleted).. would it be better to have these scripts encapsulated by an exported function? Then I realized that imports just get copied into the current config, so any paths would be relative to the current project at execution time, which isn't what you want. It would also be a large undertaking to get imported functions to work that way.
imports also allow importing variables and resolving naming conflicts, but I'm not sure that it would work for a built-in variable like ${DEVSPACE_CONFIG_PATH}, if that were to be added. It's really only intended to import user defined vars. I'll try it out just to be sure. I don't think modifying imports to work with built-in variables would be good, since you'd need to always rename them otherwise really odd things might happen.
So with all that said.. getting the path where the import / dependency is stored is the easy part, but making a coherent way to reference it, and have it work consistently in different parts of the DevSpace schema is pretty hard! Your alternative suggestion of adding an option to specify where the dependency should get stored would likely be the easiest choice. An example of what that might look like:
version: v2beta1
name: devspace.yaml

vars:
  IMPORT_1_CLONE_PATH: ~/.dependencies/my-special-import-path

# Import a template
# Depending on the URL and branch, tag or revision, the resulting path is different,
# which makes it difficult to "guess" the path in advance.
imports:
  - git: <URL-to-repo1>
    branch: main
    clonePath: ${IMPORT_1_CLONE_PATH}

This keeps things simple since you'd be able to use that anywhere vars are available, but puts deciding the paths on you. I'd also need to test that the devspace reset dependencies command also works correctly if this is the solution. I'll keep testing things out to see if there's something more convenient. This is the most appealing option in terms of complexity, so I'll probably proceed with this unless you're opposed.
One more suggestion might be to inline your scripts into functions in the imported devspace.yaml. That way you won't have to think about paths at all but I don't think that will always be practical.

Hey @lizardruss, what is the progress on the PR draft? I would really love this feature and it would make our workflow much simpler

We have the same "problem". Would love to just reference a bash file inside the imported devspace path. That way we don't need to "wrap" the content of these scripts into devspace functions.

@marcm-ml @mrkwtz Would you two be willing to try out or at least review this PR?
It's been a little bit since I've looked into this issue, but I remember being uncertain whether the solution in the PR would actually meet your needs. Feedback would be great, but I think it still needs review from our product team to move forward too.

Hey I tried the MR and this would solve my problems. We use a central template which contains some script, configs, etc. We want to copy them into the pod at runtime. Albeit we now solved it differently using prebuilt docker images. However, it would still be a very appreciated feature and would make some of our future usecases possible which we have already worked out but are just waiting for this MR to be merged and released into the cli.
Cheers

Hey @lizardruss, thanks for your comments / PR. I guess it could somehow solve our "problem" but it's not very user friendly. I'll try to describe our use-case and why.
So let's assume we have two repositories: one that includes scripts for automatically getting database credentials (from here on called "db-creds") and one that hosts a ML microservice (from here on called "ml-service). Both have a devspace.yaml. For the ml-service for local development developers need to obtain database credentials to pull and push data there.
Right now what they need to do is: clone the db-creds repo, execute the scripts that obtain these credentials, clone the ml-service repo. To improve that workflow it would be nice if we could just import the db-creds repo via DevSpace inside the ml-service devspace.yaml. So the devspace.yaml inside the db-creds repo could define functions like "get-credentials" which could look like
functions:
  get-credentials: |-
    ./db-creds.sh "some" "args"

if some args or the name of the script changes, the services (like ml-service) that use that import would not need to be bothered and just call "get-credentials". With your solution, the repo that uses the db-creds functions, would need to know the name of the script etc. It would be nice if DevSpace would be aware of the contexts for both the imported devspace.yaml and the own one.
So if the ml-service devspace.yaml calls a "get-credentials" function like above DevSpace knows that it's imported and is also aware of the directory context of that import if the function is executed (i.e. it knows that it has to look in the imported folder for the db-creds.sh).
I hope I could explain our use-case a little bit better. If you have any questions I'm more than happy to answer them :) That feature would allow us to re-purpose so much of our already existing tooling (scripts) via DevSpace, it would be awesome.

Ok i could finally give it a chance to build a binary from the MR and test it with out setup. Unfortunately, the way it is implemented right now is not sufficient. I can get the runtime variables via the get_runtime_variable function in the pipelines. However, I cannot access it via ${runtime.XXX} in the devspace.yaml which would really be the case I wish I could use it. Am i missing something? I believed I could simply do something like that:

vars:
  PATH: ${runtime.imports[0].path}

but that does not work.

Specific use case:
Some of our dependencies have their own helm chart defined in the helm directory. Currently the way we have implemented the deployment of these dependencies is that we do something along the lines of:
Project'a A devspace.yaml:
vars:
  SYNC_DIR: "."
deployments:
  application:
    helm:
      name: ${SYNC_DIR}/helm

To make the above work when using Project A as a dependencies for project B, SYNC_DIR would not resolve properly and we override SYNC_DIR specifically for project B to point to the local dir of project A. However, when using git this does not really work as we have no way of knowing the clone dir in advance

Okay, I think what I mentioned above can be partially resolved with dependencies. As with many things, it came down to us mixing up imports and dependencies. When using imports the relative paths are obviously resolved incorrectly when assuming that what you import defines something that is specific to project-b but not true for project-a (such as helm chart location). However, if you do the same via dependencies (e.g. define project-b as dependency in project-a) then the helm upgrade command is correctly executed inside the folder where project-b will be cloned to or is referenced

@marcm-ml Glad you were able to get it working with dependencies!
@mrkwtz I wonder if your use case could be addressed by defining a custom pipeline login that executes the required script in db-creds, and then including that as a dependency and executing it with run_dependency_pipelines --pipeline login. This essentially turns a dependency pipeline into an importable function.
The main issue with getting this to work for imports is that imports do not have an execution context, and instead are intended only as a way to deduplicate repeat configuration. To get them to have their own execution context would be quite a bit of refactoring. Let us know if the dependency-pipeline-as-a-function would be an acceptable solution.
Thanks!

Any chance this can be somewhat expanded, to have the path that an import got cloned to? 😅
Currently i have to copy a file from the repo that got cloned into position, but i end up doing..
cp ~/.devspace/dependencies/whatever-the-repo-name-is/... to/path

Would be awesome if i could just do
cp ${runtime.imports[0].cloned-to-path}/... to/path

Or something like it 😅
(I did check the pull request, but don't see anything in regards to the cloned path, only the import path)

@karbowiak runtime.dependencies.[name].path from the PR should be the path the dependency was cloned to.
I still wonder though if using a pipeline defined by the dependency would be a better path. I may put together an example to try it myself.


add timestamp to BotUttered message representation

Proposed change:
This adds the event timestamp to the message representation of BotUttered events. This is useful for custom channels, especially for communication with Rasa X, which uses the timestamp as the id.
Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog
[x] reformat files using black (please check Readme for instructions)

cc @tmbo @hotzenklotz

@jstriebel looks like a good idea to me :) thanks for the PR. Could you please add the change to the changelog as well?

@imLew Sure! Done.


feat: Add support for EFS CSI driver to iam-role-for-service-accounts-eks

Description

Add support for EFS CSI driver to iam-role-for-service-accounts-eks

Motivation and Context

Allow users to use EFS CSI driver using IRSA role

Breaking Changes

No

How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

This PR is included in version 4.18.0 :tada:


Added number of rounds and carthage support

Hello. I have added number of rounds and shadow track to RangeCircularSlider, so it is possible to fully mimic Apple bedtime feature now. Added carthage support (with prebuilt binary) as well.
I have also reintroduced example project, adding a segmented control on clock pane, to show difference between 1 and 2 rounds.
Thanks again for the library - it works pretty nice and I have good experience using it. I removed previous pull request, and merged recent changes into my fork - so it should be possible to merge this automatically now.

Please look at #16


Fix ERR Handling zcli project service-import

⚠ The Issue

💡 Proposed Idea

Better Err Handling like other commands 😸

Show the available commands if any
Show the available params we are supposed to use if any

🤔 Where to make the changes?
You can make the necessary changes here src/cmd/projectServiceimport.go and src/i18n.
🧭 For more info
Join our discord server https://discord.gg/3yZknaRhxK

You can also reply here if you're confused about something

hey, could i work on this @nermalcat69 😸


[Issue]: std:enable_if compiler error.

Problem Description
What can be a reason of below compiler error during calling of orortcCompileProgram? I see that OrochiUtils::getFunction is using -std=c++17" option to compile kernel.
In file included from C:\Users\Piotr\AppData\Local\Temp\comgr-2cfecd\input\third_party/MiniNN/MiniNN/Gpu/Kernels/NetworkGpuKernels.h:26:
In file included from third_party/MiniNN/contrib/rocWMMA/library/include\rocwmma/rocwmma.hpp:29:
In file included from third_party/MiniNN/contrib/rocWMMA/library/include\rocwmma/internal/io_config.hpp:29:
In file included from third_party/MiniNN/contrib/rocWMMA/library/include\rocwmma/internal/broadcast.hpp:29:
In file included from third_party/MiniNN/contrib/rocWMMA/library/include\rocwmma/internal/types.hpp:131:
third_party/MiniNN/contrib/rocWMMA/library/include\rocwmma/internal/vector.hpp:145:28: error: no template named 'enable_if' in namespace 'std'; did you mean '__hip_internal::enable_if'?
typename std::enable_if<(std::is_same<U, T>{}) && (Rank > 1)>::type* = nullptr>
^~~~~~~~~~~~~~
__hip_internal::enable_if
Operating System
Windows 10.0.19045
CPU
AMD Ryzen 7 5800X 8-Core Processor
GPU
AMD Radeon RX 7900 XTX
ROCm Version
ROCm 6.0.0
ROCm Component
No response
Steps to Reproduce
No response
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

I used another copy of hip driver binaries what solves problem.


Ex3: add points to discuss

Exercise 3:

prove that there is a zero in the scattering matrix
show that zero is eliminated with complex production

Closed by #12


Are BTrees and Binary Trees the same?

Quote from README
There are 3 segments to Big Tree consisting of Tree, Binary Tree (BTree), and Directed Acyclic Graph (DAG) implementation.

Though used in this context as an abbreviation, i think a BTree is not a binary tree

Thanks for pointing this out! I have made the correction in v0.6.7


Import it all archive

This is just a closable WIP tracking commit so these commit hashes get registered in PR land.

Closing (that was the point after all)


Updating spin test page and including second example in the process

Content must go through a pre-merge checklist.
Pre-Merge Content Checklist
This documentation has been checked to ensure that:

[x] The title, template, and date are all set
[x] Does this PR have a new menu item (anywhere in templates/*.hbs files) that points to a document .md that is set to publish in the future? If so please only publish the .md and .hbs changes in real-time (otherwise there will be a menu item pointing to a .md file that does not exist)
[x] File does not use CRLF, but uses plain LF (hint: use cat -ve <filename> | grep $'\r' | wc -l and expect 0 as a result)
[x] Has passed bart check
[x] Has been manually tested by running in Spin/Bartholomew (hint: use PREVIEW_MODE=1 and run npm run styles to update styling)
[x] Headings are using Title Case
[x] Code blocks have the programming language set to properly highlight syntax and the proper copy directive
[x] Have tested with npm run test and resolved all errors
[x] Relates to an existing (potentially outdated) blog article? If so please add URL in blog to point to this content.

Had to add the package lock json files back, sorry.
The build failed without new versions of them as part of this pr.
Not entirely sure if we can add them both the the .gitignore file or if this will cause future problems.

We shouldn't gitignore them.  I guess this is just a random wonder of NPM,

@itowlson it is an artifact of npm install instead of npm ci. npm insatll does not respect the lockfile and upgrades dependencies as long as they satisfy the requirements in package.json.  npm ci on the other hand respects the lockfile.


Convert the Snake 2D text tutorial into a series of video tutorials

Convert the Snake 2D text tutorial into a series of byte-sized (2-3 minute long) video tutorials

Videos to be hosted on Fragcolor's official YouTube channel
YouTube video links to be embedded in the Snake 2D text tutorial on learn.fragcolor.xyz/snake

Hey team! Please add your planning poker estimate with ZenHub @Gilgrion @guusw @Kryptos-FR @sinkingsugar @ZerusD


Broken links

For: https://jslee02.github.io/awesome-collision-detection/

3D collision detection: a survey (2001), P. Jiménez et al. [pdf]

I saw the link is FTP. I think it'd be much better to replace it with a direct HTTPS PDF link. https://users.soe.ucsc.edu/~pang/161/w06/notes/jtt01.pdf. Archive.

Fast Approximate Nearest Neighbors with Automatic Algorithm Configuration (2009), M. Muja and D. Lowe. [pdf, code]

The old PDF link is down. I found a working link at PSU here. Also an archive of the old link.

Contributions are very welcome! Please read the contribution guidelines first. Also, please feel free to report any error.

Unfortunately, CONTRIBUTING.md is down.

For: https://github.com/jslee02/awesome-collision-detection

Robust contact generation for robot simulation with unstructured meshes (2013), K. Hauser. [pdf, web]

PDF and site are down. Here's a better PDF link. Archive of the website.

Collision Detection (code) by Jeff Thompson - This book explains the algorithms behind those collisions using basic shapes like circles, rectangles, and lines so you can implement them into your own projects.

The code link (https://github.com/jslee02/awesome-collision-detection/blob/master) is down.

Contributions are very welcome! Please read the contribution guidelines first. Also, please feel free to report any error.

As aforementioned, CONTRIBUTING.md is down.
I don't get physics, but whatever you're doing is cool, and I hope lots of good comes out of it!

Forgot to include the first PDF's archive, it's here: https://web.archive.org/web/20210222085803/https://users.soe.ucsc.edu/~pang/161/w06/notes/jtt01.pdf

Thank you for pinpointing the broken links! They should be fixed by https://github.com/jslee02/awesome-collision-detection/commit/fbb12e312416187e8d6bf612435d6162da4f7f2d


Cosmos: Stake Confirm

Stake - Confirm (Mockup)
As a user, I want to be able to confirm the details of my staking transaction before executing

Progress Column
As a user, I want to know what steps are required to stake and which step I’m on

Step 1: Confirm Details with animated spinner
Step 2: Broadcast TX

Confirm Details
As a user, I want to be able to confirm the details of my staking transaction before executing

The following details are displayed on the confirmation screen

Stake: Amount being staked in both crypto and fiat
Validator: Validator being delegated to (Default is ShapeShift; users will not be able to select in MVP; just support a default validator for each chain for now)
Average APR: Current staking APR for the selected validator

Validator commission is accounted for when calculating APR

Gas Fee: Gas cost in both crypto and fiat

User can select Low, Med, or High gas fee based on network conditions

Estimated Yearly Rewards: Estimated rewards in crypto and fiat calculated by multiplying input amount by average APR
Copy: While your tokens are staked you will not be able to transfer or spend them. It will take X days to unlock them after you unstake them.

x = the number of days for the respective Cosmos SDK chain’s unbonding period

Confirm & Broadcast

If a user clicks “Confirm & Broadcast,” they are prompted to confirm the transaction on their wallet and the button displays a pending state with, “Confirm on Wallet”
If tx is not broadcast successfully, the ‘Confirm & Broadcast’ button turns red and displays “Error staking. Try again.”
If user rejects tx on their wallet, the ‘Confirm & Broadcast’ button goes back to the active “Confirm & Broadcast” state and user can try again
If a user confirms the tx on their wallet their tx is broadcasted successfully and they are routed to the Stake - Pending step

The Confirm Details icon turns into a Check icon
The ‘Broadcast Tx’ step turns into an animate spinner

f. Copy: While your tokens are staked you will not be able to transfer or spend them. It will take X days to unlock them after you unstake them.

x = the number of days for the respective Cosmos SDK chain’s unbonding period

Reopening since broadcast should be done in the same issue.


Method Restriction

Idea: Plugin that restrict api access only with allowed http methods.
Example:
I have a api to manage categories (GET, POST, PUT and DELETE), but for some user, I need that the user has access only the GET method, and for another user, only POST and PUT methods.
Plugin usage:
curl -X POST http://kong:8001/apis/mockbin/plugins \
    --data "name=method-restriction" \
    --data "config.methods=GET, POST, PUT, DELETE"

curl -X POST http://kong:8001/apis/mockbin/plugins \
    --data "name=method-restriction" \
    --data "config.methods=GET" \
    --data "config.consumer_id=876bf719-8f18-4ce5-cc9f-5b5af6c36007"

(This will be included in the upcoming new router, which will enable a more complex routing of incoming traffic to upstream services. APIs will be defined with:
{
  "name": "",
  "uris": ["/path1", "/path2"],
  "headers" : {
    "Host": ["domain.com"],
    "X-API": ["api-xxx", "api-yyy"]
  },
  "methods": ["GET", "POST", "X-CUSTOM"]
}

In this model, all properties must match except the ones with several values (X-API, uris and methods), for which only one is enough to fulfill the requirement. Removing the beginner label for now and putting this under the 0.10 milestone.

But, after this new feature the PR #1414 will be necessary?

#1411 would still be relevant for Consumer-level method restriction I believe. The new resolver mentioned would globally restrict APIs to the specified methods, but there is still room for a plugin taking care of this at the Consumer level.

A basis restriction will only be the first step. I agree with thibaultcha, that a consumer level restriction to methods should be the final target. We have exact that use case....

I agree guys.
So, I will wait for a new version, after that, I will continue with the development.

Well, I have been thinking about this customer plugin, and I have a two questions.
1 - The priority will be greater than API restriction?
2 - ACL integration?

An update on this: see the feat/new-router branch to track the progress on the new router which will implement method restriction directly into the core.
Also, I'll be closing this as we consider it part of #505, thanks!

@thibaultcha  awesome!!!


"ReferenceError: document is not defined" in worker

Describe the bug
I am getting a document is not defined error from a service worker once the app is built - this does not happen in development. The error is as follows:
Uncaught ReferenceError: document is not defined
    at stakers-73lEuv0j.js:1:1406
    at stakers-73lEuv0j.js:17:405829

Reproduction
Deployment with bug: https://paritytech.github.io/polkadot-staking-dashboard/
Steps to reproduce

Go to https://github.com/paritytech/polkadot-staking-dashboard and clone the project.
Run yarn && yarn build && yarn preview.

System Info
Browser: Brave / Chrome
@vitejs/plugin-react-swc": ^3.5.0
typescript": ^5.3.3
vite": ^5.0.7

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Try adding the following configuration:
// vite.config.ts
export default defineConfig({
  worker: {
    format: 'es'
  }
})

Appears to be fixed, will double check and close if working, thank you @XiSenao!

This may be an issue that still needs to be discussed. @rossbulat

the error suggests that you are trying to access the document object in a context where it is not available. The document object is part of the browser's DOM (Document Object Model) and is typically available in a browser environment, not in a Node.js or server-side context.

I'll close this issue as a duplicate of #15310 as the reproduction isn't minimal.


⚠ Bofrost.es has degraded performance

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

HTTP code: 200
Response time: 1895 ms

Resolved: Bofrost.es performance has improved in e888139 after 27 minutes.


Update README.md

Added Proxeus to "Workflow & Automation"

I can't find a node based UI.

@moklick you're not alone - please find a description and screenshot's in our User's Guide
It is a central part of the architecture and user interface, that the project's documentation does not yet do justice. I actually updated the READMEs today to make this clearer ^_^

The node editor is a key feature, at the top of the user interface of Proxeus. We will try to make this clearer in the documentation and website. Please let me know if you'd like to see a screencast, or have some other criteria that needs to be fulfilled.


Analyzer is missing error for getter/setter assignability in extension methods

Analyzer is missing the check for assignability of a getter and setter in an extension method, as tested here.

@leafpetersen the initial attempt to fix this bug accidentally swapped the roles of getter and setter (it checked that the getter type was assignable to the setter type rather than the other way around).  Would you mind adding a language_2 test that verifies that the order?  (I guess it will have to be an NNBD-related test, since the order only matters when NNBD is enabled).  I want to make sure that we have the necessary test coverage to ensure that the CFE team implements the check in the same way

I will add a test, but want to resolve a couple of questions about how we organize these tests first.  Expect an email shortly.

The initial fix for this bug was correct (the proposal reversed the order used in the language spec and has been updated). https://dart-review.googlesource.com/c/sdk/+/113041

@stereotype441 wrote:

accidentally swapped the roles of getter and setter (it checked that the
getter type was assignable to the setter type ...)

@bwilkerson wrote:

The initial fix for this bug was correct

I'm not quite sure how these two comments interact, but looking at https://dart-review.googlesource.com/c/sdk/+/113041, the intended property seems to be "the getter return type is assignable to the setter parameter type". This is also the direction which is specified in the language specification (since a07b2a10c6da58e0f3c461d778e51b128e2772a5).
Of course, with a symmetric assignability rule it makes no difference, but given that NNBD changes assignability such that it isn't symmetric, it makes sense to consider whether we should duplicate the requirements on getters/setters such that it stays symmetric.
I created https://github.com/dart-lang/language/issues/531 to clarify this issue.

At the time that Paul's comment was written the proposal stated the opposite order for comparison than what the spec contained. Leaf updated the proposal to match the spec, making the original implementation correct.

OK, thanks! It might still be desirable to require assignability in both directions, and dart-lang/language#531 would still serve to clarify that.


Implement Preview Service

This solution implements a very simple Preview Service that will render the matched row from the CSV file as description list.
Note that I also added a generic .gitignore file in a separate commit. I know that many developers nowadays are using global .gitignore files, but providing a .gitignore (and a .editorconfig) in the repository is much appreciated by new contributors.
Closes #28.

Thanks for this.  Let me give it a spin and I'll post back.  In the meantime would you mind adding an issue and referring to it in the original post as per the contributing guidelines?
FWIW, I think this is a great idea.  I had always pictured people making csv files with references to data taken from other services like Wikidata whence the preview service can be borrowed, but I completely get how this is useful in the more general case.

Looks good.  I've merged this now.  Sorry for the delay.  FWIW, I think I'm missing a dependency between the steps in my GitHub actions and that explains the failed test.  I'll have to look at that.
Also, as you've probably noticed this is on the develop branch.  I guess I should do a release soon.  Can I ask your opinion on whether you felt the change to the command line interface was easy enough to adapt to?

Oohh..  I also noticed that when the service is shut down then hovering over it shows "unable to connect".  Since these are imagined to be temporary services, that's kind of annoying.  Also, since you may be making many temporary services running on the same port the preview service may not make sense if it's showing data from the current service.
I'm not sure what the right thing to do is.  What do you think about not having this on by default?  You can always add it by supplying a config to point to it.  This sort of side steps the issue until we come up with a better idea.

Can I ask your opinion on whether you felt the change to the command line interface was easy enough to adapt to?

It took me a second to get used to the switch. But I see the further potential of the new approach.
Going to adress the other comment in #28 .


Easi 2613/user account migration plan participants and providers

EASI-2613
Changes and Description

Migrate user Plan Participants and Providers to use the user table
Update postman collection

NOTE, this needs the change in this PR to pass unit tests as expected *

How to test this change

Run the app. Update plan participants and providers to make sure it functions as expected.

PR Author Review Checklist

[ ] Met the ticket's acceptance criteria, or will meet them in a subsequent PR.
[ ] Added or updated tests for backend resolvers or other functions as needed.
[ ] Added or updated client tests for new components, parent components, functions, or e2e tests as necessary.
[ ] Updated the Postman Collection if necessary.

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.
Always make comments, even if it's minor, or if you don't understand why something was done.

Closing in favor of https://github.com/CMSgov/mint-app/pull/458 which includes these changes.


:yo_yo: Update dynamic backends support to handle the "don't pool" flag.

This doesn't actually implement any interesting functionality -- it's not clear what pooling would mean within the context of Viceroy -- but it allows programs that use this flag to function properly.
Because this just bit me, I've also removed the bit of code that checks for unknown flags and errors. I think it actually makes more sense for old versions of Viceroy to ignore flags they don't understand. Admittedly, I'm betting that future changes will be like this one (in that they don't change any, or change very little, Viceroy behavior), and not settings which will totally change behavior.

Well, OK. I was going to argue a bit, but then we discovered that we were a few functions behind in the backend introspection API, and that this was going to cause problems. Which then made me think "well, that means we can't be super tolerant of unknown things in general", so ... OK, agreed.
Also, this PR just got bigger. Sorry for the surprise embiggening.


[CommonHTML] Inconsistent horizontal spacing when loading a new font

In the example below, the spacing between the :: and the = is not the same on the first and fifth equations (it's more visible on the fifth). These equations load a new font, which might be the cause for the larger space. The equations 1–4 are strictly identical, and so are the equations 5–8. I wasn't able to build a smaller example, the problem disappears when removing one of the : or when removing the \raise.

<!DOCTYPE html>
<html>
  <head>
    <script type="text/x-mathjax-config">
      MathJax.Hub.Config({
        extensions:["tex2jax.js"],
        tex2jax:{displayMath:[["\\[", "\\]"]]},
        jax: ["input/TeX", "output/CommonHTML"]
      });
    </script>
    <script type="text/javascript" src="MathJax/MathJax.js?config=default"></script>
  </head>
  <body>
    <!-- Loads bf font: -->
    \[a \raise0.4mu{::}\hspace{-8mu}=\mathbf{\unicode{x03BB}}\]
    \[a \raise0.4mu{::}\hspace{-8mu}=\mathbf{\unicode{x03BB}}\]
    \[a \raise0.4mu{::}\hspace{-8mu}=\mathbf{\unicode{x03BB}}\]
    \[a \raise0.4mu{::}\hspace{-8mu}=\mathbf{\unicode{x03BB}}\]
    <!-- Loads sf font: -->
    \[a \raise0.4mu{::}\hspace{-8mu}=\mathsf{\unicode{x03BB}}\]
    \[a \raise0.4mu{::}\hspace{-8mu}=\mathsf{\unicode{x03BB}}\]
    \[a \raise0.4mu{::}\hspace{-8mu}=\mathsf{\unicode{x03BB}}\]
    \[a \raise0.4mu{::}\hspace{-8mu}=\mathsf{\unicode{x03BB}}\]
  </body>
</html>

Thanks for the report. We can reproduce the issue and will look into it.

OK, here's what's happening:  when the content of the \mathsf command is being typeset, MathJax has to load the data for the sans-serif font, which means MathJax has to pause in order to wait for that to load.  Once that loads, it restarts the typesetting operation and lays out the expression again.  But it turns out that the information about the width of the earlier :: characters doesn't get cleared, and when they are played out again, their width is added to the element again.  Usually, this width isn't used, but the \raise{} introduces an <mpadded> element, and in that case, the width is used, and since it is wrong, the output end up twice as wide as it should.
The solution is to clear the bounding box information so that any earlier bounding box data (left over from an earlier attempt to typeset the expression) is discarded.
Here is a patch that you can include in your pages for now, until the bug is fixed in MathJax itself.  Add the following script
MathJax.Hub.Register.StartupHook("CommonHTML Jax Ready", function () {
  var MML = MathJax.ElementJax.mml;
  var TOCHTML = MML.chars.prototype.toCommonHTML;
  MML.chars.Augment({
    toCommonHTML: function (node,options) {
      this.CHTML = null;
      TOCHTML.call(this,node,options);
    }
  });
});
</script>

just before you load MathJax.js itself, or include the content of the script in your configuration file, if you are using a local configuration.

=> renamed title.

Thanks for the quick patch! And sorry for the title, I must've been distracted as I actually checked that the bug occurs with CommonHTML (and not with HTML-CSS).


Issue #4: Implement better handling of trailing punctuation.

Update example.js to refer to correct index location.
Update logic for handling trailing punctuation that immediately
follows a non-text node, such as a link.
Add test cases covering updated logic.
All tests passing.

I don't think this totally fixes the original concerns with #4. I really only looked at trailing punctuation use case.

I’m closing the issues here and archiving the repo. It isn’t really maintained, was never finished (it’s not a task very suitable to syntax trees, but could work in a compiler/generator/stringifier). Prettier is the better option.


Toy model for users

Scaffold User model, view, controller, and spec stubs.
Run rails scaffold User name:string email:string
Run bundle exec rake db:migrate
Run rails console || rails c and then once in pry:

Run User.count
=> 0

Then to create new user to test migration table do:
User.create(name: 'somename', email: 'someEmail'
Run: User.count
=> 1
Run User.first
=> <ActiveRecordAssociation:User:...id: 1, name: 'someName', email: 'someEmail'..>

Successfully created some users from console


Webfinger configuration templating

Is your feature request related to a problem? Please describe.
When having a lot instances configured in your Webfinger config and running Webfinger in Kubernetes, you might quite easily reach the ConfigMap size limit of 1MiB (https://kubernetes.io/docs/concepts/configuration/configmap/#:~:text=A ConfigMap is not designed,separate database or file service.)
This is because the configuration needs to be quite verbose:
kind: ConfigMap
apiVersion: v1
metadata:
  name: webfinger-config
data:
  webfinger.yaml: |
    ---
    instances:
      - claim: "instancenumber"
        regex: "^0001$"
        href: "https://0001.this-is-my-ocis-domain.test"
      - claim: "instancenumber"
        regex: "^0002$"
        href: "https://0002.this-is-my-ocis-domain.test"
      ...
      ...
      ...

Describe the solution you'd like
An option to use the value of regex capture groups in the href, which would funktion as template.
Therefore we would configure something like that:
kind: ConfigMap
apiVersion: v1
metadata:
  name: webfinger-config
data:
  webfinger.yaml: |
    ---
    instances:
      - claim: "instancenumber"
        regex: "^[0-9]+$"
        href: "https://{{ <regex-match> }}.this-is-my-ocis-domain.test"

Describe alternatives you've considered
Use multiple ConfigMaps and glue config files together in a init container. But the proposed solution also has the advantage of not need to explicitly configuring all permutations.
Additional context
none

Since we're reaching the 1MB limit, please raise issue priority. Thank you

Should already work:
https://github.com/owncloud/ocis/blob/e316d8ea3a94242d887a33431a7fcba36c40db73/services/webfinger/pkg/relations/owncloud_instance_test.go#L28-L36

Confirmed, that the claim based approach is already working. Thanks


ols no longer indexing wrapper.odin in odin-gl

With respect to https://github.com/vassvik/odin-gl, OLS is no longer indexing symbols in https://github.com/vassvik/odin-gl/blob/master/wrappers.odin . No relevant error output from the LS request in VSC. The rest of the symbols from the package seem to be indexing fine.

It's caused by "when !ODIN_DEBUG".
The language server right now only has hardcode checks for binary comparisons i.e. ODIN_OS == "windows". I have now committed a version where it always indexes when conditions that don't contain ODIN_OS.

Heck yeah! Thanks for the quick fixes.
I'll let you decide on closing since idk if you want to track that in an issue or not.


How to reproduce the results of experiments that are shown in Table 7

Thanks for your open source code!
I'm trying the reproduce the results of experiments in Table 7 in the ICLR paper: 'Zero-Shot Performance of CLIP in VQA'.
Would you provide code or detail information for this?

Hi @raven38, thanks for your interest! We just append each label used in VQAv2 to question as [Question] + [Label] for each question and compute the most similar label with the [image] embedding of the question to compute the zero-shot performance of CLIP.


Allow SmartContract to use a hash with or without 0x prefix

What current issue(s) does this address, or what feature is it adding?
Allow SmartContract to use a hash with or without 0x prefix as output by neo-python contract operations.
How did you solve this problem?
Check for 0x prefix and remove it
How did you make sure your solution works?
Manual test
Are there any special changes in the code that we should be aware of?
No
Please check the following, if applicable:

[ ] Did you add any tests?
[X] Did you run make lint?
[X] Did you run make test?
[X] Are you making a PR to a feature branch or development rather than master?
[X] Did you add an entry to CHANGELOG.rst? (if not, please do)

looks like a pycodestyle issue :)

Coverage decreased (-0.01%) to 78.862% when pulling fb1fd4980af60757dd68175f873be19b91621cfa on smart-contract-0x into dc4526bb94292fcd57bc3f690b4c8868006bee05 on development.


Legend fields do not change when changing the field on the Style

STR

Create a map with a point layer.
Click on layer STYLE.
Set the marker width or fill to BY VALUE. Select a field. Now the legend is showing the field name.
Select a different field. The legend is still showing the old field name.

Expected result

The legends should show the new field name.

cc @CartoDB/support

@ramiroaznar you mean the legend's title right?

@nobuti Yes!


add seller and buyer as argument to _createListing() and _buyListing()

add seller and buyer as argument to _createListing() and _buyListing()

add an address seller argument to _createListing to be used instead of msg.sender
add a address buyer argument to _buyListing to be used instead of msg.sender

Done in pr "issue16" and issue"17-18"


Remove some old modules

TranscriptSNPs and TranscriptAlleles were long ago superceded by modules in the ensembl-variation repo. The only other reference I can find was a use statement in the webcode which then makes no further reference to the method pulled in (I've put in a pull request to remove this too).

Coverage increased (+0.56%) to 74.61% when pulling 804c0ca6093bf22d7c8fddfcb8366cecabf36700 on willmclaren:master into 6911c2ffa274e7cb84aeee21173463b1d5cdd6cd on Ensembl:master.

Thank you, much appreciated.


Rebuild the syntax with regions

Hi @mariozaizar. I learned some things since my first attempts in 2015-2017 at michaelblyons/SublimeSyntax-Email. I hope they are welcome here.
This PR:

[x] Navigate to headers and body sections (Cmd+R).

This context awareness will also be helpful for future improvements.

[x] Highlighting within headers.
[x] Closer to suggested scope documentation.
[x] Hover preview for Base64 text.
[x] Hover preview for Base64 images.
[x] Highlight Base64 and Quoted-Printable only if the header says to
[x] Highlight HTML only if the header says to
[x] prototype Quoted Printable into HTML where appropriate.

Maybe future:

[ ] Regression tests.
[ ] Allowances for Content-transfer-encoding appearing before Content-type.

Thanks for your PR @michaelblyons ❤, I will review this soon.
Can you provide before/after screenshoots highlithing the new parts for reference. Thanks!

I'll try to do that next 7 days or so. Wait for a few more pushes. This weekend, I found and fixed some small issues with a few emails I tried with. I also have some more improvements.
One of those is hover preview for Base64 text.

Before you merge, I want to see if I can render attached base64 image data in the hover.

Image hovers work now, and a lot of the quoted-printable and base64 rules are now boxed into the context where they are expected.
Before:

After:

Wow, thanks for all of these! @michaelblyons
Sorry for the waiting, it's been crazy at work and family. But I'm planning to get this reviewed and/or merged this weekend.

Bump.


It does not seem to work for any reason

Hi @chengsokdara
The package does not seem to work for me for no apparent reason.
I have a pretty simple setup just for demo purpose and when I start recording I do get the following logs in the console

These do suggest that everything seems to be working fine but when I log the transcript, I always get the following => {blob: undefined, text: undefined} as the output.
My recording status is also true when I am speaking but for some reason the output blob and text is always undefined.
My env:

Mac M1
CRA
React 17.0.2
use-whisper 0.2.0

Following is a snippet of what I have done:
import React from "react";
import {useWhisper} from "@chengsokdara/use-whisper";

const App: React.FC = () => {

  const {startRecording, stopRecording, transcript, recording} = useWhisper({
    apiKey: key,
  });
  console.log("transcript", transcript);
  console.log("recording", recording);
  console.log("...........");

  return (
    <>
      <button onClick={() => startRecording()}>start</button>
    </>
  );
};

export default App;

Thanks in advance
Cheers

same for me

Simply logging isn't enough since it is a promise. Also where's your stop recording button? Transcript won't update until recording has stopped. You can try making it async and adding an await to make sure to promise is fulfilled before logging. Try putting {transcript} into your return inside a div or any element and you'll notice that after pressing stop and waiting a couple seconds, it'll update with your transcript.

I'm having the same issue. I can't figure out how to run a live transcription. It really shouldn't need a stop button, since the point of a live transcription is not to have to click on a button to get the transcription.


Improve Cluster synchronization to seed

What this PR does / why we need it:
This PR refactors the Cluster synchronization to the seed when a shoot shall be reconciled or deleted. Earlier it was possible that the shoot's status is set to Error in case the Cluster sync failed even if the shoot is outside of its maintenance time window.
With this PR, we have refactored the way how Garden, Seed, Shoot, and Operation objects are built (using the builder pattern). Also, we have refactored the shoot controller to only sync the Cluster resource right before the reconciliation starts. In the reconcileShoot function it's also synced right after the reconciliation succeeded. Also, if a cluster has Failed status or shall be ignored the Cluster resource is synced in order to allow the extension controllers to get to know about this status.
/cc @timuthy
Release note:

An issue that caused the `Shoot`'s `.status.lastOperation.state` to be set to `Error` although no actual reconciliation operation is executed for the `Shoot` has been fixed.

@gardener/gardener-maintainers has someone capacity to double check? @rfranzke and I paired for this PR but another round of review would be helpful.

/kind/bug

/exp/expert
/platform/all

/exp/expert

/platform/all

/exp/expert
/platform/all

/kind/epic
/kind/poc

/kind/epic
/kind/poc

/kind/epic
/kind/poc

Just a note on the bot labels up there:

the first is not valid for PR
the second is not defined in he bot manager

Will take a look today, sorry for the delay

The behaviour I described in gardener/gardener-extension-provider-aws#62 (review) is coming from this PR.
I think the reason is that the Cluster initially is created without .status.lastOperation and this prevents the ShootNotFailed predicate to be satisfied?

Hm maybe, will check. Nice catch anyways. The ShootNotFailed predicate can be improved. As a mitigation until all extension have revendored we can probably add a "fake status" to the shoot manifest that is inside the Cluster object.

Mainly, it would be nice to a) have default implementations for e.g. getting the cloud profile and secret binding to remove unnecessary calls to builder methods

Default implementations require that certain arguments are always passed to Build() (e.g., the client). We thought/discussed about it and didn't like it too much. For example, when you already have the CloudProfile and use the WithCloudProfileObject builder function, and the only purpose of the context/client passed to Build() is to have a default implementation then it's the opposite of convenient for the caller. Consequently, we introduced WithCloudProfileFrom where clients/listers can be passed only if it needs to be read.

and b) rather pass a client to Build() and remove the clients/informers/listers from the builder methods.

Depends on https://github.com/gardener/gardener/pull/2217#discussion_r417071667

Mainly, it would be nice to a) have default implementations for e.g. getting the cloud profile and secret binding to remove unnecessary calls to builder methods

Default implementations require that certain arguments are always passed to Build() (e.g., the client). We thought/discussed about it and didn't like it too much. For example, when you already have the CloudProfile and use the WithCloudProfileObject builder function, and the only purpose of the context/client passed to Build() is to have a default implementation then it's the opposite of convenient for the caller. Consequently, we introduced WithCloudProfileFrom where clients/listers can be passed only if it needs to be read.

Well, good point. I agree with you, that wouldn't be convenient.
Would it be an option to add a WithClient method to the builder, to pass a Client for all the default implementations? (Yes, the context would have to be stored in the builder, which is bad practice...)
If not, I would keep passing the client to the builder methods, but maybe streamline all builders to use controller-runtime clients.

Can you take another look?
@ialidzhikov for the problem you encountered I added https://github.com/gardener/gardener/pull/2217/files#diff-8f27551340700250a13bbbe0ea0507b0R143-R154 in the shoot_control.go,  I hope it helps until #2265 is revendored by the extensions. WDYT?

@ialidzhikov for the problem you encountered I added https://github.com/gardener/gardener/pull/2217/files#diff-8f27551340700250a13bbbe0ea0507b0R143-R154 in the shoot_control.go, I hope it helps until #2265 is revendored by the extensions. WDYT?

I still reproduce it with the master branch. So I think we have to revendor the extensions


CMIP6_license  reworded

Here is the new wording for the license:
"CMIP6 model data produced by  is licensed under a Creative Commons Attribution "[*] Share Alike" 4.0 International License (http://creativecommons.org/licenses/by/4.0/). Use of the data should be acknowledged following guidelines found at https://pcmdi.llnl.gov/home/CMIP6/CitationRequirements6-0.html.  Further information about this data, including some limitations, can be found via the further_info_url (recorded as a global attribute in data files)[and at ]. The data producers and data providers make no warranty, either express or implied, including, but not limited to, warranties of merchantability and fitness for a particular purpose. All liabilities arising from the supply of the information (including any liability arising in negligence) are excluded to the fullest extent permitted by law."

@taylor13 this is a dupe of #133 so closing this and migrating the content across


added condition for amazon linux

This resolves #365

Excellent. Thank you!

Released as version 6.1.1.
https://forge.puppet.com/elastic/logstash/6.1.1/readme


RFC: Singleton types

Yeah I suppose the point of it is to show that it's possible to not increase the time complexity when comparing strings in the type checker. Pointer address is certainly the most robust strategy one can take. You're right that the best way to implement this is to follow same direction we took for AstLocals, but with global interning support.


WIP: Mistral Support

The implementation of the Mistral model with Mistral Client python framework.
This implementation is still under development, and further refinements and testing are required to ensure full functionality and robustness.

[ ] Tests
[ ] Clean Code
[ ] Fix typo
[ ] Fix Style
[ ] Steaming
[ ] Json Model and Func calling
[ ] Public Inferences Params

@samuelcolvin I'm having some trouble with the Stream Structured part. Could you take a look at it?

Good I solved it. @samuelcolvin

@YanSte BaseModel isn't yet supported by partial validation, this is explained in data at https://docs.pydantic.dev/latest/concepts/experimental/#partial-validation.
Use a TypedDict and it should work.

orted by partial validation, this

Yes, I remember now, I overlooked this part.
To ensure we don't miss it again, maybe add an update to documentation to include this information.

Hi @samuelcolvin,
I need some assistance in writing the test to simulate a call from Mistral. Could you please help me with this?
Thank you.

I've got this merged with main, just let me merge it with your latest pushes

You should definitely check the changes I made, in particular commenting out # created=1704067200,  # 2024-01-01 in the tests.models.test_mistral.chunk function. I dropped it to get the tests passing but if you want to confirm the timestamp setting is correct you might want to tweak that.
But I figured I would probably be faster to set up the model settings stuff and deal with the refactor to the new unified ModelResponse type, and that you'd appreciate the help. If you don't feel free to revert my push!


fix deprecations  config-testing

blocked by
https://github.com/tektoncd/pipeline/issues/6848
https://github.com/knative/pkg/issues/2725


save graph as image for responsive graph on mobile

http://techslides.com/save-svg-as-an-image
to be integrated within npm run build and replace graph images for mobile view

try https://github.com/sampumon/SVG.toDataURL
thanks @joerx

thanks @cheeaun

Resolved by 7bc48d6854f8c82c4831075a05dd04d8c1effb6b. No longer need to save as image.

:ghost: :tada:


Add Support for ZIA Rule Labels API Endpoints

Request Details
The ZIA API allows users to create Labels for policy rules.
pyZscaler should support these endpoints to allow users to create these Rule Labels.
Reference
https://help.zscaler.com/zia/api#/Rule Labels

[ ] Code In Progress
[ ] Code Complete
[ ] Tests In Progress
[ ] Tests Complete

Feature complete, awaiting next release.

Added in 1.2.0


Need version 3.0.0 for android

Am unable to find, 3.0.0 version on maven repo, please help getting the link so that I can add dependency into my project as there are few issues seems to be fixed in 3.0.0.
Also if there is no uploaded version of 3.0.0, please let me know the latest branch which I can pull and build my self and any steps to build that.

@trnchawla Can you please share the built version of it with me???


Reading on CSRF in Spring Boot and Spring Framework

Context
On of my projects needs to enable Cross-Site Request Forgery (CSRF) protections. As we are using Spring Boot for the API part and our web UI is a Single-Page Application (SPA), CSRF may be differerent.
I'm looking into learn what is CSRF and how should I use Spring Security to protect against this attack.

As always, OWASP has to most comprehensive writing on this topic: https://cheatsheetseries.owasp.org/cheatsheets/Cross-Site_Request_Forgery_Prevention_Cheat_Sheet.html#use-built-in-or-existing-csrf-implementations-for-csrf-protection

Spring Security also has a dedicate section for this: https://docs.spring.io/spring-security/reference/features/exploits/csrf.html.


Bug/86 call parse json file with true parameter

Fixes #86

@jfmdev Sorry I missed this comment. Then everything was working good.


fixes for hmin in render-arcs and render-gaps

https://github.com/drom/wavedrom/issues/154
really there are one line in each render-gaps and render-arcs that changed:
lane.phase = ...
The rest of the render-arcs changes are just retabbing (it was off by one character)

This doesn't fix arcs that get shifted off the left (they are still visible) but I think that's a general concern for arcs and phase in general that I'll try to handle separately.

Also, I noticed lib/lane.js seems to have defaults for lane.  Should lane.xmin_cfg and lane.xmax_cfg be put in there?

@itsayellow you don't need xmin_cfg and xmax_cfg in the defaults if you are checking for the existence of those keys before you use them.

copying this comment from above so it doesn't get buried under the "outdated diff":
As for the xmin_cfg and xmax_cfg defaults and checking: I define defaults for them in parse-config.js so that everything after parse-config has access to default values regardless of whether they're set. I did this instead of wrapping every mention in e.g. (lane.xmin_cfg || 0).
I was reading one of the old bugs (https://github.com/drom/wavedrom/issues/116) and got the impression that there might be some uses of this code that doesn't involve running parse-config.js first (and instead using lane.js.) So I'm just wondering if setting defaults in parse-config.js will ensure that they are executed, or if there are models of running this code that won't execute parse-config.js first.

@itsayellow good point. Initial lane specification should contain parameters needed for rendering but not depending on specific data set. We definitely need to clean this thing at some point. For now I am fine with any working implementation. Let me know when you think I can pull.

Then yes go ahead and pull.  It is working now as long as parseConfig gets executed (as it should be in normal operation.)


Design : first iteration of what Combined settings should look like

Based on earlier discussions :

serilog/serilog#1038
serilog/serilog#1043

And prototypes from:

serilog/serilog#1041

I have failed to keep working on this U_U.
The goal was to be able to express configuration both in C# code and in config files, in a way that a single sink can be configured half in code / half in config file, which is not currently possible (i.e. you cannot override a method call param used in C# from a config file).
The idea was to merge together in a given order :

C# code based on expressions
appSettings/xml config
other like Microsoft.Settings*

Let's see if I find time to properly polish it :P

Basically, it's kind of a mixture of :

it's painful to have to maintain both appSettings and Configuration
Microsoft.Configuration.* is nice, but there are a few caveats and we may not want to take a hard dependency on it
we already have ReadFrom.KeyValuePairs that exposes key-value settings in a format that is quite specific to Serilog
there is a need for hybrid config : define config in code ... but allow to override some params of what are really methods calls in the end ...

The general idea was therefore to have all "Settings providers" end up generating key-value pairs in the Serilog-recognized format ... Possible providers being :

appSettings / XML
appSettings.json / Microsoft.Configuration.*
actual method calls (through ExpressionTrees)

and then being to merge all those, the way ConfigurationBuilders do it , in a last one wins sort of style.
There have been discussions about not wanting to take on a dependency on Microsoft.Config here : https://github.com/serilog/serilog/issues/1038#issuecomment-337376758
So yes, standardization (see https://xkcd.com/927/) is part of the idea. The config reloading is not really the focus here though :-)
For someone who is on vacation, I find you extremely involved in GitHub issues and PRs ;)

For someone who is on vacation, I find you extremely involved in GitHub issues and PRs ;)

Haha, my wife felt the same way...
Thanks for the clarification, and the example provided by this test looks very cool.
https://github.com/serilog/serilog-settings-combined/blob/dev/test/Serilog.Settings.Combined.Tests/CombinedSettingsMixTests.cs#L23
I wasn't really suggesting a hard dependency on M.E.C but rather pointing out the flexibility in that system. I actually wound up here because I was searching for ReadFrom.KeyValuePairs syntax (I've only used it for unit tests) and landed on one of the threads that sparked this repo. But yes, that's the same concept.
Even though this is very cool, it gives me the feeling of going to extreme measures to service a simple-sounding requirement. I know this would be waaaay out of scope for this repo but just thinking out loud, one of the other M.E.C conventions that might be nice to borrow (some day... Serilog v3...) is config by POCOs. Again, not proposing any dependencies, just the concept. Then config is relatively trivial. But I know that would amount to completely overhauling configuration.
Definitely watching with interest! In particular the SQL sink has a bunch of config points that probably never change that could benefit by being baked into code (table definition...).

The general idea was therefore to have all "Settings providers" end up generating key-value pairs in the Serilog-recognized format

Definitely sold on the concept at this point (especially if your idea in https://github.com/serilog/serilog/issues/1178 happens), but I'm still The New Guy around here and haven't seen how big ideas get set into motion. Would such a significant change to the settings pipeline and providers get serious consideration?
I ask because I'd love to tackle parts of this now that I have time to focus on Serilog again. I was already lamenting that the various settings packages are left to implement the reflect-and-invoke process internally. Hindsight being 20/20, I think the ideal scenario is what you've described -- settings packages (including this one) ingest and translate various config sources to the baseline KVP format -- then Serilog itself invokes the resulting combined KVP list. Is that last part also under consideration, by any chance?
It's sort of ironic that you mentioned avoiding hard M.E.C dependencies given that I'd recently created some in the SQL sink to deal with the many complexities of the ColumnOptions class (of course, the .NET Framework variant has equivalent appSettings XML dependencies). Once everything clicked about how KVP could represent a baseline format I realized the headaches which come with that multi-targeting split would just go away.
It's also a little ironic that the SQL sink has a miniature version of combined settings: you can pass a ColumnOptions object in code, and simultaneously pass an IConfigurationSection representing ColumnOptions and the config will overlay anything in the code version. (Of course, this is still config-in-code in some sense because you have to pass the config collection yourself in the same call.)
I mention all this stuff about the SQL sink's use of ColumnOptions because the arg-matching and invocation step of any final implementation of these ideas probably needs a way to populate a LoggerConfiguration argument with either a subset of the KVPs (equivalent to a config section in M.E.C) or at a minimum, a reference to the entire KVP list. Actually, the SQL sink has use-cases for both: the entire config is useful to resolve connection strings by name stored outside of the Serilog section. There are similar questions I've fielded about complex parameters on the App Insights sink.
I know that part is a bit further out than you're probably looking right now, but this seems like a good place to mention a concern that crosses many boundaries in the Serilog ecosystem.

Is this still being looked into?

Not currently - would still be very interesting to look into, but low on bandwidth :-)


KOGITO-4657 Experiment with Kogito Quarkus Platform BOM + Quarkus

This is an experimental PR to add support to our own Quarkus-style Kogito Platform BOM.
This will be superseded in the future, but it may be necessary ad-interim for supported releases. It may be also useful as an alternative to Quarkus archetypes (we may drop those in favor of this)
I am creating this as a PoC to start a discussion and to see if this is something feasible for support with the given timeframe.
How this works
A new kogito-quarkus-parent-quarkus-platform-descriptor is added with a plugin. The plug-in generates a JSON file that contains metadata for the Quarkus CLI. When the Quarkus CLI is invoked, the metadata is fetched alongside the BOM and a list of the given extensions is then displayed for users to pick.
Read more in the Quarkus documentation
How to use.
Once all is installed locally:
mvn io.quarkus:quarkus-maven-plugin:1.11.6.Final:create -DplatformGroupId=org.kie.kogito -DplatformArtifactId=kogito-quarkus-parent -DplatformVersion=2.0.0-SNAPSHOT

follow the instructions. You can add the kogito extension here or later; once you are in the fresh project directory, you can now:
$ mvn quarkus:list-extension

the Kogito extension will be displayed alongside the other core extensions:
Current Quarkus extensions available:
Kogito                                             kogito-quarkus
Kogito - Decisions                                 kogito-quarkus-decisions
Kogito - Predictions                               kogito-quarkus-predictions
Kogito - Rules                                     kogito-quarkus-rules
Amazon Alexa                                       quarkus-amazon-alexa
Amazon DynamoDB                                    quarkus-amazon-dynamodb
Amazon IAM                                         quarkus-amazon-iam
Amazon KMS                                         quarkus-amazon-kms
AWS Lambda                                         quarkus-amazon-lambda
AWS Lambda HTTP                                    quarkus-amazon-lambda-http
...

Caveats

I am using kogito-quarkus-parent as a BOM. This is suboptimal for 2 reasons

it is not designed as a BOM
it is named -parent (weird naming convention for a BOM)
so we may want to create an ad-hoc BOM

kogito-bom declares artifacts that are in kogito-apps. Because the plug-in fetches dependent artifacts while generating the metadata, and because this platform BOM is runtimes, they are probably being pulled from nexus at the moment. Not sure if this is a real issue, because they are being pulled just for processing. Solutions may be:

move this platform BOM to kogito-apps (not great) or
create another BOM specific to the extension where kogito-apps artifacts are not present (this is probably what we want)
this may be a non-issue because the plugin may be further configured to ignore some artifacts

I haven't created an integration test for this; but we should probably create an invoker-style integration test with the same contents of the Quarkus archetype

As part of the platform building process, this is pulling the dependencies from the entire Quarkus Core BOM, so this will slow down our builds (at least when we bump Quarkus). Is this OK? Shall we create a specific profile?

/cc @aloubyansky

Note, the reason the json generating plug-in resolves all the artifacts from the bon is that it is trying to catch all the artifacts that are Quarkus extensions. You can configure the plugin to not process artifacts, e.g. from a specific groupId.
As to the BOM, it has to be convenient for the end user of your extensions. So if you can have a single bom that contains everything you customers need, that would be the best, IMO.

The Linux check has failed. Please check the logs.

This means kogito is creating its own platform. That when quarkus releases users won't be update and community and camel extensions not available.
I'm not a fan of this approach unless kogito will release same time as quarkus and consider use  universal platform as base.
Any reason not going for the just BOM approach ?

LGTM, just check my comments

Please, make sure you understood Max's points above. Perhaps you already have considered that. I just want to be sure we are making conscious decisions here.

I think, for the given timeframe, we'll have to keep archetypes. We may consider using this as a way to test kogito-examples (on master) before they land in Quarkus platform, though. I don't know how that would work though

closed as outdated


Upgrade django sample project to use render method

Use render method to render the view as it is a replacement for
render_to_response.
Exclude backward compatibility for render_to_response method as
developers using it are likely to be on one of the latest versions of
Django that has support for render method. Therefore, render_to_response
support may be excluded altogether from this sample project.

I signed the CLA as a corporation :: Footbits, Inc.

Thank you for the contribution!
(1) Please conform to chris.beams.io/posts/git-commit/#seven-rules with your commit message.
(2) Why maintain backward compatibility to pre-Django-1.3? That's looking five-ish years back, isn't it? That feels like ancient history and isn't worth the extra code that I see in the current draft of your change. Am I wrong, or missing some important information?
(3) I'm still seeing the "cla: no" label on this contribution - you may have signed the CLA as a corporation, but it looks like the identity with which you authored the commits isn't associated with that corporation (in a manner determinable by the robot). Are you sure you're using exactly the accounts and email addresses that you intend to use?

Thanks, Nathan!
I read through the link and modified my commit heading and message appropriately. It is definitely more concise and readable now.
Initially, I was thinking to include support for pre-Django-1.3 as there are many apps that still use it. Hence, it is still supported partly due to the fact there is not much maintenance cost.
Rethinking about it after your comment, since this is a sample project, developers using it are likely
to be on one of the latest versions of Django that has support for render method. Therefore, render_to_response support may be excluded altogether from this sample project. I have committed another change for this.
I am looking into number 3.
On the other note, a commit that was meant for another pull request got pulled in here (my second commit for this PR). Can you accept it as part of this PR or would it be better to create another PR after this one is closed?

My name is Nathaniel. :-)
Commits don't travel around the world by accident - git users push and pull them. Look at the top of this page where the text "zixan wants to merge 3 commits into google:master from ZIXANLABS:master" is displayed. Is that a true statement? Do you want to merge three commits into this project? Specifically the three commits currently at the tip of the "ZIXANLABS:master" branch? I doubt it, and if I'm right then you should update the branch "ZIXANLABS:master" to have just the one commit that you want merged into this project in this pull request.
The title of this pull request is "Upgrade django sample project to use render method". If there are commits that you've drafted that don't fall under that description, they probably belong in another pull request.
Git is hard; the management appreciates your diligence. :-)

Thanks, Nathaniel.
I have made changes as per your recommendations.
Please review and let me know if further clarifications/changes are needed.

Please squash your commits - this change should be done all in one commit.

Squashed four commits into one. Please review. Thanks.

Looks much better; thanks for the contribution.


Firebreak - Swagger validation

What you did
Explorer Swagger request validator library to validate swagger.json file (generated through annotation) with expected/actual responses https://bitbucket.org/atlassian/swagger-request-validator
Swagger can be validated in multiple ways with the library. Below are the ones that are close to frameworks we use / independent of frameworks (also variants includes using springmvc, mockmvc, spring web client)
1. Simple request/response validation
Validates swagger.json with expected responses (json)
Pros

Quick feedback
Independent of framework

Cons

Can't actually test if swagger file is incorrect. An incorrect swagger with an incorrect expected responses will still pass

2. Pact provider validation
Validates swagger.json with a pact interaction (local file / from pact-broker)
Pros

Quick feedback

Cons

Only useful when am actual consumer generates pact interactions. Generating interactions ourselves is not useful
Can't actually test if swagger file is incorrect. Am incorrect swagger with an incorrect expected responses will still pass

3. Rest assured
Validates swagger.json with actual responses to publicapi application
Pros

Invokes requests on application which can detect if swagger itself is invalid

Cons

Takes longer to run but shouldn't be a deal break

What can it detect
Validator can detect the following in both swagger.json / responses (mock / actual responses)

Schema mismatches ( objects, fields, datatype, enumerate values..)
Unexpected fields (additional fields in swagger but not in response and vice verse)
Request/Response headers

Can also whitelist any validation errors if it is persistent issue that can't be fixed / to be ignored
Any issues

Encountered issues with dependencies especially com.github.fge.json-schema-validator as it conflicts with another to read JsonSchema. Excluding this library seems fine but may have impacted existing integration tests using mockserver
Seems to work fine on both Java 8 & Java 11, but pacts provider tests doesn't work the latest (3.6.2) pact library.

Additional info
Swagger request validator is listed as open source tool for Java by Swagger.io https://swagger.io/tools/open-source/open-source-integrations/

Closing for now but can be referenced if we want to add swagger validation


Ability to analyze just one system

Would it be good to support analysis over just one system?

I guess that's not the main point of the toolkit, but I think it's probably pretty easy to implement now, so why not?


Windows keep losing focus

OS: Windows 10 Pro x64 Build 10.0.16299
Issue Descriptions: Other open windows are losing focus when the Player is running in the background. I might be in the middle of typing in Outlook, Chrome, etc... and the window just stops taking my input. Might be related to this issue in Electron. I did not have this issue before installing the player.
Unfortunately I can't really put down when it happens, it's not on changing a song. It might be when a window update happens, since it seems to be connected to when Win10 reloads some windows (Outlook, System windows, Skype).
Steps to Reproduce: Randomly happens and is thus not really reproducible. I have attached the debug_info.zip

Randomly happens and is thus not really reproducible

This means I can't really help, it doesn't even sound like you can pinpoint this as being GPMDP and not something else.

https://github.com/electron/electron/pull/10783

This is a PR that we never merged, this can't be the cause of your issue.


Wiring

Can you please show the wiring of the encoder pins? In the code there are in and output pins that somehow are both marked as pins for the rotary encoder. I thought the encoder only has pin1, pin2 and GND

@devmapal  Hi, thanks for answer :) I am working on littele bit larger in scope Smart Home Automation Project :)  link: https://github.com/Simple-Home. If you would want participate (by any miracle let me know :)) :). But back to your project are you running ESP from internal battery ? i am bud i cant get longer than 9h because i am using ESP 01 there is no possibility of deep sleep did you run into same problem ? because if i am able to run it form internal battery i can gide hole system in original case :)

@GamerClassN7 In fact I've into the same issue. There is a deeper slee

@GamerClassN7 I'vs run into exactly the same issue. There is a deeper sleep mode the ESP8266 supports which would allow it to run for years. In this mode it doesn't process interrupts anymore, so you would loose the ability to manually set the temperature.
Another alternative would be to use a second microcontroller for controller the rotary encoder that can go into a deep enough sleep state but still be woken up by interrupts. Then you could hook up both the manual controls as well as the EP to it. There are many microcontrollers e.g. from Atmel that support this. It does make the design quite a bit more complex though.
A third option (which I went for for now) is simply hook the thermostat up to a powersocket :-)

@devmapal HI, i find a weary word problem for no reason my thermostat goes to 17 degree, how i switch termostat to fully manual mode ? I am suspicious of some smart feature :D

@devmapal  i run into the problem with ESP 01 i wont boot when conected to termostatic valve did you run into same issue ?  what type of board did you used


add h5py to dockerfile

Add h5py to dockerfile. Fixes #11356

Can one of the admins verify this patch?

I signed it!

Fixed as requested. The list wasn't alphabetical to start with, so I fixed that too.

Thanks, @yhenon

PR merged. The changes will take effect in the tensorflow/tensorflow release images after the next release and in the tensorflow/tensorflow:nightly* images after the next nightly build.


Add tests for $elemMatch projection operator

Refs FerretDB/FerretDB#257.
Refs FerretDB/FerretDB#331.

With that PR merged, other FerretDB PR checks fail – main dance tests functionality that is not merged yet.
I think we should agree to merge two PRs at the same time.

agree


feat(help): Add built-in help function

This PR implements:

The client can be initialized with a helpCommand, which will generate a help function.
For example, passing helpCommand: "!help" will add the commands !help and !help <function>.

The callback of bot functions now receive the resolved functions as the second argument

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Create custom style prop or more prop to customize style

Is your feature request related to a problem? Please describe.
I add w-full class on Button component to use it by full width, but internal class w-fit ignore it.
 <Button type='button' className='w-full bg-primary-500' onClick={handleOpenAddView}>
   글 작성하기
 </Button>

Describe the solution you'd like
maybe 2 option (just suggestion). Also, I think other components will need this option too.

add width prop on Button component. then user can customize button component's width

// for full width button
 <Button width='full'>
   sdfsdf
 </Button>

// for CTA button 🟩🟩🟩  🟦
<Button width='0.75'>
  🟩🟩🟩
</Button>
<Button width='0.25'>
  🟦
<Button>

add custom style prop to customize free without priority issue

 <Button custom='w-full'>
   custom prop should have high priority then internal class
 </Button>

Additional context
discord thread

I think there's too many possibilities to add options as unique props, but we definitely need an "escape hatch" to override className. As you suggested, custom.
After you brought that up on Discord, I found this: https://github.com/richardgill/tailwind-override which I think we can use to add that escape hatch. That would make it so className can be provided to each component, and any classes will be applied without needing !important.

@tulup-conner how about this? https://github.com/ben-rogerson/twin.macro
tailwind-override doesn't seem to be updated anymore

@tulup-conner how about this? https://github.com/ben-rogerson/twin.macro
tailwind-override doesn't seem to be updated anymore

Yeah, tailwind-override isn't the best option for that reason and because it seems to have some outstanding unfixed bugs. They also chose to use regex to parse class names which inflated the bundle size.
Twin brings in a lot of dependencies and unrelated functionality. The dependencies thing is something we really want to avoid.
This isn't really my area of expertise, so I'm doing the same thing you are, looking around for libraries that have done it right already.

Following up, we are starting to complete theme support for components now; see #153, #155, #156.
Note: The API is unstable (almost certain to change in some details before "done")

See #160, it's in draft right now.

See #160, it's in draft right now.
It just occurred to me that there are probably situations where you might want to add a className in a subset of your buttons, but not all of them. Brainstorming..

So cool !!
If I want to use Button w-full in specific component, Is this what I should be doing?
const theme = {
  button: {
    base: 'w-full',
  },
};

or should like this?
const theme = {
  button: {
    base:  'group flex h-min w-full items-center justify-center p-0.5 text-center font-medium focus:z-10',
  },
};

@emewjin the second option.

We should document that behavior for sure ^


[WIP] Fix issue 201

Fix issue 201

Separate TParameterDefinition from TPropertyDefinition

[ ] Waiting for #206 -> WIP

[ ] Ensure that you followed http://eclipse.github.io/winery/dev/ToolChain#github---prepare-pull-request. Especially, we require a single commit
[x] Ensure that the commit message is a good commit message
[ ] Ensure to use auto format in all files
[ ] Change in CHANGELOG.md described
[x] Tests created for changes
[ ] Screenshots added (for UI changes)

Can one of the admins verify this patch?


For Azure AD tenant, per user MFA is technically available. This caused confusion to our customer. It would be great to make it straight in the doc that per user mfa is not available in Azure AD Free tenant.

[Enter feedback here]

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

ID: feb03e63-f543-d85b-9daf-16b4d1e13a60
Version Independent ID: da9dcdd2-70d4-e08e-18f2-6e6a1489937a
Content: Azure AD Multi-Factor Authentication versions and consumption plans
Content Source: articles/active-directory/authentication/concept-mfa-licensing.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@jerryliangt Thank you for your feedback. I am updating the document as per your suggestion.


outdated and cleanup have false positives

[x] Confirmed this is a problem with running a brew command and just brew installing one or more formulae? If it's a formulae-specific problem please file this issue at https://github.com/Homebrew/homebrew-core/issues/new
[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed all issues and retried your prior step?
[x] Ran brew config and brew doctor and included their output with your issue?

I have a custom formula foo in a local tap.  The version is "2.1.20_1".  Only one version of foo is installed in the cellar, and brew outdated keeps giving me:
chdiza/dizaone/foo (2.1.20_1) < 2.1.20_1

which is wrong.  Also, brew cleanup -n gives:
Warning: Skipping (old) /usr/local/Cellar/foo/2.1.20_1 due to it being linked

which is also wrong: that's not an old version of anything.
I only noticed this problem because I just bumped to "2.1.20_1" from "2.1.20_0".  Apparently the latter gets transformed by brew internals into "2.1.20", which failed to fool brew internals.
The formula foo.rb, by the way, looks like this (shrunken to show a minimally reproducible test case):
class Foo < Formula
  url "file:///usr/local/hb/ChangeLog.md"
  version "2.1.20_1"

  def install
     system "touch", "#{prefix}/foofoo"
  end
end

Here's the relevant stuff:
brew config
HOMEBREW_VERSION: 1.2.0-66-g459fef3b0
ORIGIN: https://github.com/Homebrew/brew
HEAD: 459fef3b09b25d3e24cce6aa6f2e3a7bd5460a2b
Last commit: 3 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 151887da68c0678c594b6909ac2c271958ada9a8
Core tap last commit: 9 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_REPOSITORY: /usr/local/hb
HOMEBREW_CELLAR: /usr/local/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://homebrew.bintray.com
CPU: quad-core 64-bit ivybridge
Homebrew Ruby: 2.0.0-p648
Clang: 8.1 build 802
Git: 2.13.0 => /usr/local/bin/git
Perl: /usr/bin/perl
Python: /usr/bin/python
Ruby: /usr/bin/ruby => /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby
Java: N/A
macOS: 10.12.4-x86_64
Xcode: 8.3.2
CLT: <IP_ADDRESS>.1.1492020469
X11: N/A

$ brew doctor
Your system is ready to brew

This is definitely a bug. Think I know what's going on; should have a fix shortly. Thanks for the lovely detailed report @chdiza.

The problem comes from the "_1" being stripped in one part of the code, but not in another, which messes up as comparison.

@chdiza Homebrew treats underscores in versions a bit specially, and I think that's what's causing the problem here. I'm interested to know where the version scheme with underscores is coming from? Is there a reason it has to use underscores?

There are at least a few of them
gnu-cobol.rb:25:    version "2.0_nightly_r658"
tbb.rb:5:  version "2017_U6"
tmpreaper.rb:6:  version "1.6.13_nmu1"
vbindiff.rb:5:  version "3.0_beta4"

Not at computer now so can't confirm, but I think it's only a problem when it's an underscore followed exclusively by digits.

I'm interested to know where the version scheme with underscores is coming from? Is there a reason it has to use underscores?

It comes from me :)  There's no reason why it has to use underscores.  It just looks better, and I know that there are other formulas that use underscores so I thought those are allowed.

Yeah, Homebrew expresses versions ending in /_\d+\Z/ as Homebrew-specific revisions to formulae, so '1.0.0_2' would be 'revision 2 of the formula for 1.0.0'.
It would probably be possible to make Homebrew handle this correctly, but since it doesn't affect any of our formulae (I checked) it probably isn't something we're going to work on ourselves. We'd definitely accept a PR though!


Mbed not enough SPI stack size for debug mode on DISCO board

Problem
For the DISCO board, the SPI stack size is set to 1024. It is not enough space for debug mode (shell example).
BLE event is service by background callback and it requires more stack to get there.
Proposed Solution
Increasing the size to 2048 solves the problem.

The fix is committed to the feature-chip branch of mbed-os repo

The fix merge to the latest mbed-os version


Language select

[ ] Remove flags
[ ] The default value should say English, EN, or en (shorter names might be better for mobile version if we don't change the layout) - we can also use an icon
[x] Update dropdown style to white bg and default shadow style (light and blue'n'green theme).
[ ] Update dropdown style to dark bg for semi-dark theme.

@m0ksem still relevant?

@m0ksem still relevant?

Yap. We haven't removed flags yet. But I don't think that it's important.

There is no language select for now.


App doesn't start after the upgrade

Description
After the upgrade from 24.2 to 24.2.1, the app crashes during the start

1727163969816.log
DBeaver Version
24.2.1
Operating System
Windows Server 2016 Data Center
Database and driver
MS SQL Server
Steps to reproduce
Start the app
Additional context
No response

Same issue today on Windows Server 2016

@oguzgundogdu Hello, how did you install dbeaver? And what exactly version of WIndows Server 2016 do you use. You can find it if you execute winver

@oguzgundogdu Oi, então eu instalei usando o link Windows (zip) https://dbeaver.io/files/dbeaver-ce-latest-win32.win32.x86_64.zip, depois de extraído na máquina foi realizado a tentativa de abrir o dbeaver.exe e é apresentado o erro indicado pelo @Matvey16. A versão usando o winver é Windows Server 2016 14393

@Matvey16 Hi! I just upgraded as I always do through the upgrade dialog. Then I tried to make a clean install by uninstalling and installing again.
Here below, my version. It's very close to @avlisesolrac's one

Thanks

Windows Server 2016 is no longer supported by Eclipse SWT. See https://github.com/eclipse-platform/eclipse.platform.swt/issues/1500

The issue comes from the platform on which DBeaver is based.
Here are 2 options that might help:

Downgrade DBeaver to 24.2.0
Try upgrading Windows. If no additional minor upgrades are available, upgrade to a higher major release of Windows.

Windows Server 2016 is no longer supported by Eclipse SWT. See eclipse-platform/eclipse.platform.swt#1500

Well, maybe some sort of warning before/instead of the "update available" info would have been useful here.

Yes, we're going to implement it. Here is the issue about it #34315


LocalTunnel server on DigitalOcean droplet with Ubuntu help

I am trying to run this localtunnel server on my DigitalOcean droplet with Ubuntu.
I added a domain on this droplet.
http://examplesite.in
I installed as below:
git clone git://github.com/defunctzombie/localtunnel-server.git
cd localtunnel-server
npm install
When I run this command I got some error :
bin/server --port 1234
Then I installed :
apt install golang-golang-x-tools -y
Again run this command and no error :
bin/server --port 1234
I get this response on the putty.
Defaulting to port 8080Listening on port 8080
Is this the expected response on success ?
Then on my raspberry pi if I run this command:
lt --host http://examplesite.in:1234 -p 3000
nothing is happening and I don't get any url.
I am sure only few more steps for success which I don't know that.
I wonder this localtunnel works great on localtunnel.me server.
Also given open source code.
But no step by step tutorial for freshers like me to run our own server in DigitalOcean like cloud.
"You can set up DNS entries for your domain.tld and *.domain.tld (or sub.domain.tld and *.sub.domain.tld)."
"The server can accept incoming TCP connections for any non-root TCP port (i.e. ports over 1000)."
I couldn't understand what it is.
It it nothing but normal domain.com or domain.in ?
Can any one make a step by step tutorial to may our own server in DigitalOcean droplet ?

Okay. Managed to make this work under a subdomain using the instructions below. Figured this should also work if you're using the main domain for the server, so I'm posting it here. While your setup may not be similar, consider the following steps and tweak depending on your process:

Register a wildcard subdomain entry on your DNS records. Depending on your registrar, this is normally allowed and should be as easy as adding an A record with *.domain.tld as your host. In my case, I want to set the server up under a subdomain, so I had to register both the main subdomain lt.my-domain.com and wildcard subdomain *.lt.my-domain.com.
Login to your DigitalOcean droplet via SSH and set up your reverse proxy. In my case, I used Nginx for my web server and configured it as a reverse proxy for the LocalTunnel server. Since LocalTunnel server is basically just another NodeJS app, you can actually follow DO's guide on setting up Nginx as a reverse proxy for NodeJS applications. I'm also running several NodeJS applications on the same web server, so I had to configure it like a virtual host, listening to other ports -- otherwise if you will be using a dedicated server for the tunneling service, it's totally fine to just use port 80.
Setup SSL for your main domain/subdomain and wildcard subdomain. You can use Let's Encrypt for this one, but there's a specific command to generate an SSL for both the main domain/subdomain and wildcard subdomain. I followed this guide for the specific command.
While I theoretically think this will run under PM2 or even systemd, I'm only executing it as a regular NodeJS app with the following command for two reasons: node -r esm ./bin/server --port <port_number> --domain lt.my-domain.com --secure

The package is using ES6 modules. Without a compiler like Babel, it won't run on its own hence the need to execute node -r esm. I don't wanna hassle myself into figuring it out -- yet.
I'm not gonna use the service very often, so I'm only executing it from the command line instead of executing it as a daemon service

DigitalOcean droplets does not block any port nor have any firewall configurations by default, so if you've set it up right, you should have a working LocalTunnel server at this point. All you need to do is run a LocalTunnel client from your local machine. In my case, I installed LocalTunnel as a global module so I just need to execute lt --host lt.my-domain.com --port <local_port>. Tested it with the --subdomain command as well to reserve a specific subdomain for the tunnel and it works as is.

Hope that helps.

Thanks micahbule
I will check this well later.
Time being I managed many things by my own.
Thanks again for your detailed help.

Update: Managed to install it as a docker image on my server using the instructions on README -- of course passing in the --domain and --secure flag, as well as specifying the specific port that my Nginx reverse proxy is forwarding requests to.

@micahbule I also managed to get it working on DO using Ubuntu. But the main problem is it is not working properly. most of the time, i got 404 error and another issue is cannot request a new subdomain from the server after client restart. my client is Raspberry Pi. Any suggestion to improve this?

@micahbule I also managed to get it working on DO using Ubuntu. But the main problem is it is not working properly. most of the time, i got 404 error and another issue is cannot request a new subdomain from the server after client restart. my client is Raspberry Pi. Any suggestion to improve this?

It takes time for the server to clear any used subdomains for some reason, haven't really dived into it. Not entirely sure as well why you're getting a 404. Maybe check the service you're tunnelling first?

This was a great help though the biggest issue was the DO ngnix docs being outdated and missing a step:
Once you've created your server config in the /etc/nginx/sites-available/domain.com you need to link it to the sites-enabled with the following command
sudo ln -s /etc/nginx/sites-available/domain /etc/nginx/sites-enabled/


pulsar admin  not acessing bookies

Hello I am trying to look at a message on the ledgers via the admin command of the pulsar client:
/bin/pulsar-admin topics get-message-by-id persistent://public/default/testtopic9 -l 4 -e 3 however it doesnt work, returning the following error:
null

Reason: java.util.concurrent.CompletionException: org.apache.pulsar.client.admin.internal.http.AsyncHttpConnector$RetryException: Could not complete the operation. Number of retries has been exhausted. Failed reason: pulsar-broker-3.pulsar-broker.pulsar.svc.cluster.local

I've done  bin/pulsar-admin brokers list and it returns all bookies as expected:
"pulsar-broker-0.pulsar-broker.pulsar.svc.cluster.local:8080"
"pulsar-broker-1.pulsar-broker.pulsar.svc.cluster.local:8080"
"pulsar-broker-2.pulsar-broker.pulsar.svc.cluster.local:8080"
"pulsar-broker-3.pulsar-broker.pulsar.svc.cluster.local:8080"

I have also verified that that ledfer id and entry exist using the reader api.
I am running this on kubernetes via minikube and I installed using the the helm chart for with 1 replica of each, but changed it the replicas from 1 to 4:, https://github.com/apache/pulsar-helm-chart/blob/master/examples/values-minikube.yaml .
Other than that all configurations remain unchanged.

The request looks like redirected to another broker. Can you try with pulsar proxy? You can expose the proxy port on local and use pulsar-admin to connect proxy then execute that command.

That was it, I was connecting directy to the broker service instead of to the proxie's webservice webServiceUrl


MoreDetailsBox Styling

Fixes #issue_number
What was changed?
added the css file for the moredetails box
Here, describe what part of the application you changed (e.g. login page, database, etc.). If possible, mention what specific components were added, removed, or modified.
I added the css file for the moredetails box, nothing was removed
Why was it changed?
Need to incorporate styling into moredetails box
Here, describe the issue that you are fixing with this code, and why your code fixes it.
This code sets up the more details box to have styling
How was it changed?
Changed by adding the
Here, get into detail about what files you modified, and talk about the most important lines in regards to fixing the issue.
This CSS styles the modal overlay, modal content, and buttons to make the interface look cleaner. Uses flexbox to center the modal and ensure the other elements have an even distribution. The buttons have hover effects and rounded corners for better usability and a cleaner look.
Screenshots that show the changes (if applicable):

Fixes #issue_number
Please replace issue_number with the issue number that this PR resolves.


Sending variable to spark as an array

Bug Report
Hello,
Sending a variable into a spark component as an array value, kills the request and shows a blank white page. For example:
<main id="structure" data-on-load="{{ spark('_components/article', {
  id: entry.id,
  tags: entry.objTags.ids()
}) }}">

Plugin Version
1.0.0-alpha.5
Craft CMS Version
5.5
PHP Version
No response

Please check the error logs for an exception message and stack trace.

It appears that trying to send any form of array variable, just kills the memory. Even with 2G set, it still dies.
[21-Nov-2024 09:48:19 Australia/Brisbane] PHP Fatal error:  Allowed memory size of<PHONE_NUMBER> bytes exhausted (tried to allocate 262144 bytes) in /var/www/html/vendor/yiisoft/yii2/base/Module.php on line 784
[21-Nov-2024 09:48:19 Australia/Brisbane] PHP Fatal error:  Allowed memory size of<PHONE_NUMBER> bytes exhausted (tried to allocate 262144 bytes) in Unknown on line 0
Sending a blank array, does the same thing:
<main id="structure" data-on-load="{{ spark('_components/article', {
  id: entry.id,
  tags: []
}) }}">

Can you show me what’s in the template at _components/article?


symlink /usr/local/bin/consul to the install location

This fixes issue https://github.com/johnbellone/consul-cookbook/issues/351

This patch breaks windows installation