Canvas editor

Current manifest editor creates (arbitrary?) canvasses of 1000x1000px. Aspect ratio can only be adapted by fitting canvas to image.
For composing canvasses consisting of multiple image annotations, both the size and the proportion of the canvas should be adaptable.
Size is a tricky thing when one starts scaling annotations on the canvas.
See for example the canvasses with multiple objects here: https://heritage.tudelft.nl/en/exhibitions/novieten. The aspect ratio of the canvas determines the way the objects appear on the exhibition page.

Currently, canvas size is also related to zoom controls. So If I create a new canvas, add an image to it, I cannot zoom it because the canvas is 1000px (while the image is much larger of course).

Cropping images in current tool resets their size so that the resulting crop needs to be scaled back on the canvas. As there's nothing outside of the canvas, this is often impossible.
How to conceptualise the canvas and its surroundings in the editor? Maybe it's an endless drawing board on which the canvas itself is presented as a (kind of) annotation. I'm thinking of the artboards as used in Illustrator. Isn't it theoretically possible to put annotations halfway on the canvas? Can they have negative coordinates? Creatively, these kind of options are inspiring, but they require a more flexible approach to what a canvas is, separate from the annotations.

To further the Illustrator analogy: for composition, it's beneficial to have the equivalent of smart guides or a grid to snap to (also when scaling annotations).
For anyone wondering what all this might be good for: think of Aby Warburg's Mnemosyne Atlas or the work of Batia Suter. Or a (non-IIIF) project like this allowing visitors to create their own custom display of archeological objects: https://belowthesurface.amsterdam/en/vitrines.
While Mirador offers a (non-IIIF) way to juxtapose different manifests next to each other, the manifest editor allows editors to actually compose canvasses of multiple IIIF-images from different sources, while being able to crop/rotate/scale each of them. Especially the modifications show the curator's hand, which is often not tangible in digital presentations. For me, this is one of the most exciting aspects of the editor.

For negative xy, not clear if this is possible or allowed, but wh could be larger than canvas size: https://www.w3.org/TR/media-frags/#naming-space

Some similar work here:
https://github.com/atlas-viewer/atlas/blob/master/src/world.ts#L317-L343
What this model lets you do is create a world with a varying size, and then place images at fixed size in it. So you could, for example, have 4 1000x1000 images tiled in a 2x2. However you're not limited to painting the images at the size they are. Instead you could paint them all at 500x500 and still have a world that is 1000x1000.
The choice of how "big" the canvas ends up being could be done automatically from maybe the largest asset, or could be an editorial choice. If you image having a composition that looks like this:
 [ 250 x 250 ]   [ 1500 x 1500 ]
 [ 500 x 500]    [ 500 x 500 ]

(all squares displayed at the same size)
Because the 250x250 is not very high resolution, you may make an editorial decision to make the canvas width in this case 750x750. So a balance between the max (3000x3000) and the minimum (500x500) for the given image sizes.

To be clear: I don't think there's much of a use case for manually setting canvas dimensions (contrary to aspect ratio, where I do see a use case), so it makes sense to do this automatically on the basis of the largest, or perhaps rather the smallest annotation on the canvas (to avoid scaling beyond original size). I hope the calculations are manageable with a large number of annotations on a single canvas...

If you think of a tool like Lucidchart, you never really need to think about the "paper" size. Although it still operates in units of A4 (or whatever you have chosen) you always end up just cropping to content.
It's a special case to say that you really want "paper" size of something other than the best fit.
Speaking of Lucidchart, I wonder how much like the editing / layout experience of this tool the manifest editor is per canvas? A layout surface...

We have to remember that unlike Lucidchart, most users of the manifest editor will only be putting one object (image) on each page (canvas), and won't be thinking about the fact that the canvas dimensions and extent could be different from their image. Whatever we do it shouldn't complicate this base use case but be a natural extension of it when someone is making a canvas with complex composition. By complex, I mean anything other than one image filling the entire canvas.

Common scenario: I have 100 book page images and I need to arrange them in sequence, make a table of contents, add some metadata.
In the user's model the canvas doesn't really exist, it's just a sequence of images.
Less common scenario - I want to make this:

...so I need to be able to arrange and stack things, and think about the shape of the page/canvas distinct from its content.
The challenge is to make both of these feel natural. They aren't special modes or different UIs.

@tomcrane in that example above, if you had just created that - as a user how would you like the options for the size of the overall canvas to be presented? We could take into account the largest possible, and largest without losing detail (i.e. based on the smallest image).

The Mona Lisa example? The difficulty here is that there is only one piece of content. In normal operation, that would dictate the dimensions of the canvas and I'd end up with just the full Mona Lisa.
To get the effect I want, I'd need to separately make the shape of the canvas, and then move the Mona Lisa partially of off it.
If you mean the immediate multi-image above example - I don't see a problem with just defaulting to the largest possible. They are just integers, not pixels, so we have plenty of room. This illustrates a problem with the previous editor, when you drag an image in that is much "bigger" than the canvas... that "bigger" doesn't make any sense, because there is no required relationship between the canvas coordinates and the pixel dimensions of the image resources; we just set them the same for single-image canvases for convenience.
Say I have a 2000 x 1000 w,h canvas and I drag a 10,000 x 10,000 pixel image onto it... the editor has no information at all to determine how much of the canvas that image takes up, so it definitely shouldn't appear huge, much bigger than the canvas. There should in fact be some default behaviour that just makes the introduced image a comfortable size in the viewport. The editor has no idea what you'll be doing with that image and shouldn't make assumptions based on its pixel dimensions.

Almost all the time it will be fine for the manifest editor app to just make that decision for the user - to expand the canvas integer dimensions to accommodate the numeric pixel values.
The canvas could be deliberately "locked" by the editor in two special cases:

the user has modified the canvas boundaries (e.g., by dragging, or entering numbers manually) to be different from the outer bounding rectangle of the resource(s) and then saved the manifest. This adds a dimensions-locked behaviour to the canvas automatically - user is up to something so don't interfere. E.g., in https://github.com/digirati-co-uk/iiif-manifest-editor/issues/17#issuecomment-963323034 the user saved the canvas with borders beyond the bounding rectangle of the content, so just leave it and lock it. It would be very rare that the w,h the canvas ends up with was unsatisfactory from a "resolution" point of view - but even then (even more edge) you could still modify w,h later.
the user explicitly checks a box in the UI that does the same thing - they want to keep the current coordinate system even if they later add a detail image that is a "higher resolution" - this is so edge-case that we shouldn't design the UI around it.

[x] lock canvas dimensions

Most users, all of the time do not need to think about Canvas dimensions at all.
Some users, some of the time will need to modify canvas dimensions or have multiple resources and/or resources that don't fill the canvas.

I've been thinking: why was control of canvas aspect ratio important in the first place? You could indeed arrange and scale annotations on the canvas freely and determine canvas dimensions based on their shared outline and the smallest/largest annotation.
The reason why we needed control over aspect ratio had to do with the difference between preview and zoom modal in our digital exhibitions. The size of the preview blocks is determined by behaviour properties (eg. w-4, h-6). In order to fit the canvas content correctly in the block, it needed to have the same proportions.
Here're two screenshots of the problem. div.canvas-preview__center is larger than the preview block (it's a standard square canvas). In order for the preview to render correctly, I placed the objects on the top of the canvas. This explains why, when opening the modal, they're not in the center but at the top of the viewer (second image).

(See exhibition made with the old manifest editor here: [https://heritage.tudelft.nl/en/exhibitions/rise-of-a-campus])
This issue could, I presume, also be solved within the exhibition component rather than the manifest editor. Because currently we're setting the dimensions twice, using behaviour properties and the canvas. (I remember that initially the plan was to generate thumbnails of mixed-media canvasses to avoid this problem, although then you'd still need to control the thumbnail's aspect ratio.)
Also take into account that you'd usually want to add some outer margin to mixed-media canvasses (although this could also be added later in the viewer application).
I have not rethought the exhibition component and would be open to reconsider our approach to preview/modal...

(https://github.com/digirati-co-uk/iiif-manifest-editor/issues/17#issuecomment-963300881) Agreed about regions solving need for negative coordinates indeed.

Task - design the canvas UI
This feels like:
Most users never consider the canvas dimensions at all. They just think in terms of content because there's only one piece of content. The manifest editor just works for them and they don't set Canvas dimensions. It's just a sequence of images - or AV.
The editing surface doesn't even suggest a canvas. It's just an image.
Adding more than one painting resource to the canvas changes the behaviour slightly. The canvas will still default to the bounding rectangle, but you can see that there is a canvas, as long as there's a disjoint (content 2 not wholly enclosed by content 1)
There's a button that makes the canvas borders editable so you can move them. You can activate this, and they stay at the dimensions they were when you pressed the button, unless you drag them.
When this is activated you can also enter width and height manually and see the effects.
(mockups to follow)

I started a discussion for this at https://github.com/digirati-co-uk/iiif-manifest-editor/discussions/29

We should close discussion on this and agree what we're going to do across:

[ ] what's solved in the exhibition code rather than in providing settings in the Manifest
[ ] Aspect ratio settings #206
[ ] Cropping #31

This has been improved, although having "fixed" size or aspect ratio canvases is not something we support. With the other tickets opened, closing this one for now.


Revert whole hog (roadhog ultimate) to OW1

Probably not a high priority, but just a bug I noticed while playing. When you ult as road hog, you have to hold the fire button vs him justing auto firing.

Reopening as the feature is only partially complete


Flipfix

now we have test_sameData (test.test_importers.TestFromFourierData) ... FAIL. I am investigating.


Resolve flake8 linting issues

To better comply with Python coding standards, it would be good to resolve current flake8 linting issues.

Should be closed by #16


Add dark mode treatment to sponsor logos.

Before:

After:

I took the Runway logo white from their website. Bottom of their main page, so it should be an official logo treatment for dark backgrounds too.

I do notice an issue in the after image.

Build real-time chat messaging in less time. Rapidly ship highly reliable chat in-app messaging with Stream's SDK. Get Started.

<picture><source srcset="/images/sponsors/stream-white.svg" media="(prefers-color-scheme: dark)"></picture>
Build real-time chat messaging in less time. Rapidly ship highly reliable chat in-app messaging with Stream's SDK. Get Started.

I think the resulting structure is good, but why the blue hover on the entire text all of a sudden?

Closed due to pushing this branch to upstream repository.


Copy breadcrumbs

It would be great to have the ability to copy the current breadcrumbs, which could be very useful for team communication and debugging purposes.

I suggest adding a 'Copy Breadcrumbs' option to the context menu, allowing it to copy the breadcrumbs. In this case, it should copy 'a.b[1].c.e[2]' to the clipboard.

/duplicate of #58678

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Fix extra background space on Android

@keybase/react-hackers
This PR reverts @chrisnojima's padding PR for iPhone X in favor of @buoyad's.  Seems to make things better on Android for me -- @MarcoPolo, what do you think about iOS?

Does it work on ipHone X?

I'll try it now.

I think both did. This one is more isolated though so it didn't affect android and non x. We need to update react native for the larger fix so

Seeing this on iPhone non-X:
JSON value 'unset' of type NSString cannot be converted to a YGValue

Just use undefined
On Wed, Dec 6, 2017 at 9:12 PM Chris Ball<EMAIL_ADDRESS>wrote:

Seeing this on iPhone non-X:
JSON value 'unset' of type NSString cannot be converted to a YGValue
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/pull/9830#issuecomment-349839899, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfhU_FZrwUVvl8ObWDi6ME4p2tZjSOSks5s90mjgaJpZM4Q44JA
.


Add Array.append simplifications

Adds Array.append simplifications #174

[x] Array.append Array.empty array -> array
[x] Array.append array Array.empty -> array
[x] Array.append (Array.fromList [ a, b, c ]) (Array.fromList [ d, e, f ]) -> Array.fromList [ a, b, c, d, e, f ]

I extracted listAppendChecks into a generic collectionAppendChecks. I noticed it got very close to collectionUnionChecks and changed the error messages to match collectionUnionChecks because they were nicer.
Now collectionAppendChecks and collectionUnionChecks are pretty much the same check, except that the former tries to combine append [ a ] [ b ] into [ a, b ].
I think we can make that work for dict/set as well in a next step, completely fusing the 2 collection checks.
The only difference is that sets and dict need to have the elements added to the end rather than at the beginning, so union (fromList [ a ]) (fromList [ b ]) into fromList [ b, a ]. I'll do that in a follow-up PR.
Can be reviewed commit by commit.

This has been on my wishlist for some time. Very cool!

I suspect there could be issues with cases like
Array.append ([ 1 ] |> Array.fromList) ([ 2 ] |> Array.fromList)

Would be cool to add tests for these cases

I added more tests, it looks like things are working as expected :+1:


[CBR-298] Remove integration test script

Description
Removes integration test script. Use nix based script to run integration tests in future
Linked issue
https://iohk.myjetbrains.com/youtrack/issue/CBR-298
Type of change

[ ] 🐞 Bug fix (non-breaking change which fixes an issue)
[ ] 🛠 New feature (non-breaking change which adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)
[x] 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc
[ ] 🔨 New or improved tests for existing code
[ ] ⛑ git-flow chore (backport, hotfix, etc)

Developer checklist

[x] I have read the style guide document, and my code follows the code style of this project.
[x] If my code deals with exceptions, it follows the guidelines.
[x] I have updated any documentation accordingly, if needed. Documentation changes can be reflected in opening a PR on cardanodocs.com, amending the inline Haddock comments, any relevant README file or one of the document listed in the docs directory.

Testing checklist

[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

QA Steps

Screenshots (if available)

Good one @adinapoli-iohk !
Also, @akegalj, I just noticed the base branch. This task is part of CBR-288's story, so let's use:
https://github.com/input-output-hk/cardano-sl/tree/Squad1/CBR-288/review-wallet-peripheral-utilities
as a base! (My bad though, I didn't update the story ticket with the branch :upside_down_face:)


Remove unneeded TestHydrateHelmComponents

TestHydrateHelmComponents has been flaky in the 1.15 release branch CI. The Helm function that's being tested in TestHydrateHelmComponents has been removed from main. However, we don't want to cherrypick that breaking change to release branch. Instead, remove TestHydrateHelmComponents directly.
b/290521459

/lgtm
/approve

Looks like the source branch was created in this repo (GoogleContainerTools/kpt-config-sync), please clean up the branch after the change is submitted

/lgtm


Documentation needs updating

Digging around some of the issues here, it seems that there are more options available than what's in the README.md.
Also, there should be clarification that if you pass a variable, instead of a hardcoded string, into the Worker call, that it will not be picked up by the WebPack compilation, and will use the browser's native implementation w/o module compilation.

These would both be great additions to the documentation. The latter is noted during compilation as an error/warning, but I don't think it's surfaced properly in the docs.

Sorry for the delay, finally got around to updating the readme!


Donating Funds to ENS-Normalize.JS

Hi ENS-Normalize.Js maintainers. I'm Lucas, a core contributor to the Drips project. I help recipients of donations through their learning and onboarding process.
Ethers.JS has been using https://github.com/adraffy/ens-normalize.js. It's one of their most critical dependencies!
Ethers.JS core contributors (mostly RicMoo) appreciate the work you all do and want to make sure that you are all well-funded to keep doing it. For that reason, they have decided to donate some funds to your project through Drips.
Who can I talk to about it?
CC: @adraffy @jeetiss (👋 @ricmoo)

hello @Lfada — I should of investigated this further—as I didn't really become aware of this until seeing it on Twitter today.  What can I help you with?

Hello, @adraffy, great to hear you saw the Twitter activity going on. Just saw that you claimed!!!
How would you rank your experience from 1 to 10 (excellent)? Also, if you have a few minutes, sharing What you liked the most, what could be improved, and opportunity/es for improvement would be super helpful for us :)

The whole experience was pretty slick and simple.  Great for teams of all sizes.  8/10
I misjudged the gas necessary to bootstrap the account, so approximate setup fees for the initial claim would be nice (setup + claim + swap to ETH).  If an address is registered w/o ENS, you should encourage setting a primary!


Upgrade ssh-agent-tls from 0.0.2 to 0.0.3

fixes #1158

Codecov Report

Merging #1213 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1213      +/-   ##
============================================
+ Coverage     52.61%   52.66%   +0.05%
  Complexity     1614     1614
============================================
  Files           268      268
  Lines         13026    13026
  Branches       1504     1504
============================================
+ Hits           6853     6860       +7
+ Misses         5655     5647       -8
- Partials        518      519       +1


Automated test(s)

Idea:

clone repo
make test
tests run in all their glory, starting with a simple talker/listener example

Ideally, this could be done in a non-ament way, but will still be automated on Jenkins.

Since the package already uses CMake why not use the standard approach:

clone repo
invoke cmake
invoke make test

ament_tools will handle it well even if it is just a plain CMake project.

how about these steps

clone repo
invoke make (which invokes cmake)
invoke make test

and this too

clone repo into ROS workspace
build with ament
same tests will get run with ament test

Also:
I like the idea of reusing the existing apps by creating a test executable that forks those processes, since code reuse is always great, but this would probably only work on POSIX; is that OK?

A related task is configuring Jenkins to do what we want once there are automated tests in place :muscle: I just added the option to test a branch for the freertps project on ci.ros2.org, which is useful for my workflow since I like branches... I tested it by making a bogus branch test_jenkins which puts trash in the CMakeLists.txt. Predictably, the CI job for that branch failed. So I think the feature works...
http://ci.ros2.org/job/freertps/18/


Returning resolved event move object instead of bytes

This PR added logic to convert event data bytes to SuiMoveStruct in SuiEvent, we also implemented conversion for rest of the Event types
here is an example event json output from transactions.json
...
        "events": [
          {
            "moveEvent": {
              "type": "0x2::DevNetNFT::MintNFTEvent",
              "fields": {
                "creator": "0xd9ecb26707ff4060fca732f69d7b6376a11f0b1c",
                "name": "Example NFT",
                "object_id": "0xcc55d5c436b338c65172bdc58504ff6d9278dc4a"
              }
            }
          },
          {
            "newObject": "0xcc55d5c436b338c65172bdc58504ff6d9278dc4a"
          }
        ]
...

added back event bcs bytes as per @damirka 's request
here is a sample event json response in transaction effect
https://github.com/MystenLabs/sui/blob/14d39342a5d0c6507c41c2aa3b961f23c0ac5b17/crates/sui-open-rpc/samples/transactions.json#L96-L111

friendly reminder :) @666lcz

I have relaxed the TS definition to any to unblock you from merging. Will follow up with another PR to add the type definitions later.

I have relaxed the TS definition to any to unblock you from merging. Will follow up with another PR to add the type definitions later.

thanks @666lcz !

I have relaxed the TS definition to any to unblock you from merging. Will follow up with another PR to add the type definitions later.

I have relaxed the TS definition to any to unblock you from merging. Will follow up with another PR to add the type definitions later.

thanks @666lcz !


✏ Update copy and site header

Removed "Security" and "DAO" from header links (looks cleaner and having DAO in the header makes it sound like we are a DAO already) + dded "Web3 Pledge" to make the page more visible.
Edited newsletter signup copy
Other minor copy polish for clarity, grammar, and pith.

Looks good!

 Yes, in this case that feels like overkill... but it's a good practice to follow.

testings steps: check the


When will i18n be supported?

When will i18n be supported? For instance, in Chinese, Japanese, German, or French, I see that all the supporting phrases are in English, with no language settings available.

Is it really an issue? Software development is in English and it's not obvious to me that having instructions in English and processed text in another language is meaningfully harmful to performance. I'm not a maintainer though

Tagging #729 for related commentary on using DSPy with different languages -  Internal signatures can be configurable to different languages.


Nonvoting Preliminary Work

mostly this switches the inc_dr_end to use inc_enumerate_signatures
more work will follow but for now we need this in master to get a production sheet working

do NOT delete the nonvoting branch yet! it is still in use for one of our deals


Is it possible to use groq API

Hi,
groq should be OpenAI compliant: https://console.groq.com/docs/openai
If I try to set it, I have 404 error
Thank you

you need to try provide the full completions API path, ie: https://api.groq.com/openai/v1/chat/completions

API path confirmed by the groq docs: https://console.groq.com/docs/api-reference#chat

Grazie @lmangani these my steps
D SET VARIABLE openprompt_api_url = 'https://api.groq.com/openai/v1/chat/completions';
D SET VARIABLE openprompt_model_name = 'groq-llama3-70b';
D LOAD open_prompt;
D SET VARIABLE openprompt_api_token = 'gsk_CIhrphxxxxxxxxxxxxxxxxxxxxxxxxxxjBXPEIRHuz';
D SELECT open_prompt('Write a one-line poem about ducks') AS response;

And I get
┌──────────────────────────────────┐
│             response             │
│             varchar              │
├──────────────────────────────────┤
│ Error: HTTP error 404: Not Found │
└──────────────────────────────────┘

Thank you

The groq documentation states the API exists. Perhaps other groq users can assist here... if you find the reason let us know

Hi @lmangani if I run a RAW call to the API
curl https://api.groq.com/openai/v1/chat/completions \
  -H "Authorization: Bearer $GROQ_API_KEY" \
  -H "Content-Type: application/json" \
  -d '{
    "model": "mixtral-8x7b-32768",
    "messages": [
      {
        "role": "user",
        "content": "Ciao! Come stai?"
      }
    ],
    "temperature": 0.7,
    "max_tokens": 1024
  }'

I have a reply
{"id":"chatcmpl-2560ba1b-3a69-43f3-a882-1c85d9d09700","object":"chat.completion","created":1730046840,"model":"mixtral-8x7b-32768","choices":[{"index":0,"message":{"role":"assistant","content":"Hello! I'm just a computer program, so I don't have feelings, but I'm here and ready to help you with any language-related questions you have. Is there something specific you'd like to know or practice today?\n\nIf you're a language learner, it can be helpful to practice short, focused conversations to build your skills. Here are a few questions you could ask me to get started:\n\n* How do you say [word or phrase] in [language]?\n* Can you give me an example sentence using [word or phrase] in [language]?\n* How would you translate this sentence from English to [language]: [sentence]?\n* Can you explain the difference between [word or phrase] and [word or phrase] in [language]?\n* Can you help me practice pronouncing [word or phrase] in [language]?\n\nI'm looking forward to chatting with you!"},"logprobs":null,"finish_reason":"stop"}],"usage":{"queue_time":0.012524184,"prompt_tokens":15,"prompt_time":0.002070754,"completion_tokens":204,"completion_time":0.328595312,"total_tokens":219,"total_time":0.330666066},"system_fingerprint":"fp_c5f20b5bb1","x_groq":{"id":"req_01jb7d7m3aet6bdgqqctm88txy"}}

Does this help us understand the problem?

Not really, since you're receiving a 404 for the same destination, unless something is wrong with the parameter.
The only thing I'm thinking is you might have an older version of the extension loaded. If that's the case try this:
SELECT set_api_url('https://api.groq.com/openai/v1/chat/completions');
SELECT set_api_token('gsk_CIhrphxxxxxxxxxxxxxxxxxxxxxxxxxxjBXPEIRHuz');
SELECT set_model_name('groq-llama3-70b');

and then
SELECT open_prompt('Write a one-line poem about ducks') AS response;

If any of the above works, it means the community extension repo is still serving an older release and we'll investigate.

The conclusion is that I am really stupid: it works perfectly.
I got the model name wrong, so it didnt find it and I had 404. The correct name is llama3-70b-8192`.
I take advantage of this for a question: how do I set them as default? I add them to the .duckdbrc file.
Thank you and sorry for the time I am wasting.

Ciao @aborruso thanks for sharing an update! I really appreciate it. This is all new, so don't feel at fault. What we need to do is collect and document more working examples, and this definitely helps do that.
In the next release we're going to add SECRETS which are persistent, right now you have to pass the parameters into the session manually or via SQL statements in your .duckdbrc if you're using the DuckDB CLI.


Allow use of team drives

Updated API allows to pass argument stating compatibility with team drives.

Thanks for this very useful support of team drives.
small issue : gdrive list -q with  clause return empty
example : gdrive list -m 1000 -q "'xxx' in parents or 'yyy' in parents"
whereas it returns ok when you do it separately like gdrive list -m 1000 -q "'xxx' in parents

@kryptonic Strange... It works for me like a charm...

Would also really need this!

Please check gdrive share not working

Failed to share file: googleapi: Error 400: Bad Request. User message: "", invalidLinkVisibility

Please check gdrive share not working

This pull request does not alter the share command in any way as far as I am concerned.

FYI I have packaged an unofficial version (obviously) with this change included: https://github.com/mcartoixa/gdrive/releases/tag/2.1.1-a
Use at your own risk!...


Release v1.4.5

Fixed bug in creating bundle CI
Fixes for github bots to work correctly
In case of an error, the entire error will be returned to the output
Added script file launch test

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

src/index.js
97.61% <100.00%> (ø)


New internal variable for L2 MTU

Also updated the fragmentv4 comment about the meaning of MTU.
Requested on https://github.com/Igalia/snabbswitch/pull/126

LGTM.


fix(DataStore): serialize IncomingAsyncSubscriptionEventPublisher events

Issue #
https://github.com/aws-amplify/amplify-swift/issues/2982#issuecomment-1671713659
Description
Before the PR changes, the publisher would emit events concurrently which caused some of them to be missed by the subscriber even though the subscriber applied back pressure to process one event at a time (demand of max(1)). For more details regarding back-pressure, see https://developer.apple.com/documentation/combine/processing-published-elements-with-subscribers
The change, to publish the events though the same TaskQueue ensures that the events are properly buffered and sent only when the subscriber demands for it.
General Checklist

[ ] Added new tests to cover change, if needed
[ ] Build succeeds with all target using Swift Package Manager
[ ] All unit tests pass
[ ] All integration tests pass
[ ] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] Documentation update for the change if required
[ ] PR title conforms to conventional commit style
[ ] New or updated tests include Given When Then inline code documentation and are named accordingly testThing_condition_expectation()
[ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (f64c471) 67.90% compared to head (1f194d2) 67.98%.
Report is 3 commits behind head on main.

Files
Patch %
Lines

...Sync/IncomingAsyncSubscriptionEventPublisher.swift
76.92%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3489      +/-   ##
==========================================
+ Coverage   67.90%   67.98%   +0.07%
==========================================
  Files        1080     1080
  Lines       36406    36415       +9
==========================================
+ Hits        24723    24755      +32
+ Misses      11683    11660      -23

Flag
Coverage Δ

API_plugin_unit_test
67.23% <ø> (ø)

AWSPluginsCore
64.18% <ø> (ø)

Amplify
48.12% <ø> (+0.01%)
:arrow_up:

Analytics_plugin_unit_test
81.16% <ø> (ø)

Auth_plugin_unit_test
78.98% <ø> (+0.02%)
:arrow_up:

CoreMLPredictions_plugin_unit_test
59.44% <ø> (ø)

DataStore_plugin_unit_test
80.72% <76.92%> (+0.30%)
:arrow_up:

Geo_plugin_unit_test
70.75% <ø> (ø)

Logging_plugin_unit_test
63.34% <ø> (ø)

Predictions_plugin_unit_test
37.16% <ø> (ø)

PushNotifications_plugin_unit_test
87.13% <ø> (ø)

Storage_plugin_unit_test
77.51% <ø> (ø)

unit_tests
67.98% <76.92%> (+0.07%)
:arrow_up:

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

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


Any plans for a reporting tool on how many tokens are created on the platform?

Totals
By Day
By Week
By Month
By Year
Something with graphs would be much appreciated.
Also any comparison info would be also super cool. (i.e. x% up or down on previous day/week/month/year
List of top tokens by value
top collections
top NTFs
top fixed supply
Not sure if this belongs in the dApp or elsewhere.

I have had it on my to-do list for awhile now to create a 'metrics' website. This data would also be handy in the dApp.
@CHMEX could you use the above list to start creating API's from dexstats.info for us to pull and use in the website/dApp?
This will also be very handy data in the explorer @pbca26 .

@piggydoughnut Is this data captured on the token explorer?

@NutellaLicka there is a number of total records in Elastic, so that will be the number of tokens on the platform. Easily accessible too.


TypeError: Only HTTP(S) protocols are supported

Using Nextjs on vercel, using the guide as described here:
https://docs.upstash.com/redis/quickstarts/vercel
(btw, these key names 'UPSTASH_REDIS_REST_URL', 'UPSTASH_REDIS_REST_TOKEN', no longer correlate with what you get in the details tab so it's a source of confusion.)
results in the error:
TypeError: Only HTTP(S) protocols are supported
Both locally and on vercel.
My Nextjs was generated using the create-next-app.

Hey!
I don't understand what you mean the key names don't correlate with the details tab, can you elaborate?
I can see them in our console

The error message is a pretty good indication, that whatever you are passing to Redis is not properly prefixed with https://. Can you double check your url is correct?

Hey @chronark you are right on the money.
I couldn't see the rest API example, since it is below the "connect to your database" box. It didn't make sense to me that one of the connection methods would be at some other place.
However I did see this, and instead now getting this error on vercel:
WARN	Error: connect ECONNREFUSED <IP_ADDRESS>:6379
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1159:16) {
  errno: -111,
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 6379
}

Actually never mind my precious message. It looks like that error was from a previous deploy. Currently debugging a 500 on the endpoint. Will fix, and come back or close.

👍  I'll update the docs to make it clearer and we'll think about merging the connection options at some point.

So it turns out my original TypeError, my 500 error error - UpstashError: Unauthorized, and my latest TypeError: Cannot read properties of undefined (reading 'startsWith') are all due to authentication errors.
This happens when the URL or REST token is incorrect, or read incorrect from my dot env file.
I figured this is worth knowing, as it's a bit strange. But I will close this issue as the original problem is resolved.
Thank you!

Cannot read properties of undefined (reading 'startsWith') is a result of passing undefined instead of a string
If you are using typescript, it should warn you about this.
We provide const redis = Redis.fromEnv() as static constructor, that reads from env and provides a helpful error message if UPSTASH_REDIS_REST_URL or UPSTASH_REDIS_REST_TOKEN is not found in the environment.
However that's as far as we can go I think. storing and loading the url and token is up to the user and they need to ensure it works.

@chronark The cause was dotenv serving an old version. But I figured it might be worth documenting that there was some confusion on my part.
I very much appreciate the quick help!

Hey @chronark you are right on the money. I couldn't see the rest API example, since it is below the "connect to your database" box. It didn't make sense to me that one of the connection methods would be at some other place.
However I did see this, and instead now getting this error on vercel:
WARN	Error: connect ECONNREFUSED <IP_ADDRESS>:6379
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1159:16) {
  errno: -111,
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 6379
}

Idem, is very easy to miss & a simple notice would save so much time.

We were just talking about this last week and will move a rest example to the top of the page


Feature Request: Puppeteer options in the scraper constructor

It would be useful edit options of puppeteer in the constructor, for example { headless: true } for SSR (server side) operations.

Is puppeteer integrated into this package? Not really or am I wrong? Currently I'm using this to fetch the Video URL and then I use pupeteer separately to get the "real" mp4 file link

You can pass arguments to puppeteer with the options argument of the Scraper.scrap method:
scraper.scrap("http://example.com", {
    runnerOptions: {
        puppeteerConfig: {
            headless: true
        }
    }
});

or alternatively by changing the the Scraper.defaultOptions property:
scraper.defaultOptions = {
    runnerOptions: {
        puppeteerConfig: {
            headless: true
        }
    }
};

but it shouldn't be necessary to pass headless: true as an option as it is the default.
Note: You can only pass a puppeteerConfig to Scraper using puppeteer


react-native module

Hi,
Would it be possible to also add a react native module wrapper for this code, so shell commands can be called from there?

@anilgulecha , I do not use react native (yet :), so I am not the right person to do it ..


Make default new file properties optional

All the four default properties that are automatically added to a new journal file can be made optional so a user can opt out, better still, the file can be totally blank, the properties conflict with other plugins and some custom scripts. thanks.

The plugin is build around using that properties to know about note (what journal and what dates). So it is not possible to make them optional without major rewrite.
Can you please provide more info on what problems properties are causing. There were issue with Templater but they are fixed in 1.3.0

I use a plugin that automatically adds my tasks to my kanban board but when I do not specify a date for the task, it picks the date of the daily journal as the task date which sometimes is not what I want, I will want the task to be undated but it just takes the date of the journal.

What plugin are you using? is it CardBoard? or some other?

journals

no, I meant - which plugin do you use for kanban that has issues with properties that journals plugin is adding?


Error running exchange_with_ipc.py after generating blocks_rpc.json

After generating blocks_rpc.json, the following commmand exchange_with_ipc.py resulted in error.
python exchange_with_ipc.py --ipc-path=~/.local/share/io.parity.ethereum/jsonrpc.ipc --input=blocks_rpc.json --output=blocks_rpc_output.json
Traceback (most recent call last):
File "exchange_with_ipc.py", line 20, in
response = socket_exchange(args.ipc_path, ''.join(line_batch), args.ipc_timeout)
File "/home/user/ethereum-etl/ethereumetl/socket_utils.py", line 13, in socket_exchange
sock.connect(socket_path)
FileNotFoundError: [Errno 2] No such file or directory

Thank you for reporting this issue.
Try using --ipc-path=$HOME/.local/share/io.parity.ethereum/jsonrpc.ipc - looks like the ~ is not expanded by the shell as it's in the middle of the string. I will update README also.
However, parity is not supported by this tool at this point. The reason is geth and parity ipc work in a slightly different way as I found out:

geth ipc will output an empty string at the end of the response so it's easy to determine when the data has been received completely
parity outputs empty strings even if it didn't finish outputting the response.

The way it's handled in web3py, for example, is it will try to parse json after every recv and if a JSONDecodeError happened this means that the response is not full yet and it will continue to wait for the remainder of the response https://github.com/ethereum/web3.py/blob/master/web3/providers/ipc.py#L175 Parsing JSON adds a lot of overhead that's why I don't use web3py and wrote a custom socket handling.
web3py also doesn't support batch requests for now https://github.com/ethereum/web3.py/issues/832
I have a few TODOs for this project:

Unit tests
Send batch requests http://www.jsonrpc.org/specification#batch.
Support Parity
Add HTTPProvider
Error handling and logging

Any help would be appreciated.

Thank you for the help!
I haven't done much programming in Python myself, most of my experience is Java/Kotlin. For unit testing I think we can use https://docs.pytest.org. It's the same tool that web3.py uses https://github.com/ethereum/web3.py. I suspect they already apply all the best practices so we could borrow configs, structure, workflows from there.
A good start can be ethereumetl.service.EthErc20Processor.filter_transfer_from_receipt_log. The tests should give it test logs and check that the result is correct.
Btw, I have a business idea: SaaS solution for analysts and developers - "SQL for Blockchain".

Simple web console for running SQLs
Built on top of AWS - cost efficient
Users pay per query
Can provide access to raw CSV data if needed
Supports Ethereum at first, later Bitcoin and other blockchains can be added
Internal transactions will be supported

If you know any businessmen, project managers, developers, including yourself, who might be interested to join, or if you have any ideas please let me know<EMAIL_ADDRESS>

@jonjet I added support for Parity. Please help test when you have time.


unable to run node_exporter playbook when offline and SELinux enabled on RedHat

Background:
We are using RedHat with SELinux enabled in an offline environment.
When SELinux is enabled, the node_exporter playbook wants to make sure two packages are present: python3-libselinux and python3-policycoreutils.
Problem:
The task to check for the presence of the packages always fails due to Yum which is unable to fetch metadata from the Yum repositories. Note that the packages are already installed so why does Yum wants to fetch repo metadata.
I've noticed that when I run "yum check-update" beforehand with internet temporarily enabled, the task does succeed afterwards when offline...

While I agree that this is a bug in the ansible module I don't think that this is something we should account for, it feels strange to have a host in a air gapped/offline environment but still have yum/dnf configured to use public mirrors.
In this situation I would set up a internal yum mirror, but if that's out of the question then you could even set up a local directory mirror for yum that's on the host itself.


Standard jinja defaults for v6+ templates

Fix #247.

Codecov Report

Merging #363 (acd80a7) into master (76d6c2f) will decrease coverage by 14.55%.
The diff coverage is 99.23%.

@@             Coverage Diff             @@
##           master     #363       +/-   ##
===========================================
- Coverage   96.95%   82.40%   -14.56%
===========================================
  Files          35       34        -1
  Lines        2170     2228       +58
===========================================
- Hits         2104     1836      -268
- Misses         66      392      +326

Flag
Coverage Δ

unittests
82.40% <99.23%> (-14.56%)
:arrow_down:

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

Impacted Files
Coverage Δ

tests/test_prompt.py
11.71% <50.00%> (-88.29%)
:arrow_down:

copier/template.py
98.85% <100.00%> (+2.60%)
:arrow_up:

tests/helpers.py
100.00% <100.00%> (ø)

tests/test_answersfile.py
100.00% <100.00%> (ø)

tests/test_cli.py
100.00% <100.00%> (ø)

tests/test_complex_questions.py
20.19% <100.00%> (-79.81%)
:arrow_down:

tests/test_config.py
98.68% <100.00%> (-0.04%)
:arrow_down:

tests/test_copy.py
100.00% <100.00%> (ø)

tests/test_demo_update_tasks.py
100.00% <100.00%> (ø)

tests/test_migrations.py
92.91% <100.00%> (-7.09%)
:arrow_down:

... and 25 more

Continue to review full report at Codecov.

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


Fix division by zero in CLI output when no inter frame is coded

Fixes #732.

Coverage remained the same at 84.56% when pulling 21cf5dacc2a4ac1cf41faa60e2c796f23d96bb45 on rzumer:l1_fix into ed3cf8345f26040bdf0ebf488b2db23c22ce6c97 on xiph:master.

Did this accidentally pick up other changes?

Oh nvm looks like the sum of all the commits is ok.


[Doc] External links ending with .md do not link to the correct page in read-the-docs

Hi, I am a developer of MMDet3D. Recently I found a bug in the read-the-docs page of MMDet3D (please refer to this). In a word, the underlying parsing machanism of sphinx will view an external link (which is not a relative path) ending with .md as something other than a external link. Therefore, when a user clicks the link in a generated read-the-docs webpage, the link will jump to a 404 page because .md is stripped in the target webpage.
Example:
https://mmdetection.readthedocs.io/en/latest/model_zoo.html#training-speed-benchmark
Click detectron2 in the sentence "We compare the training speed of Mask R-CNN with some other popular frameworks (The data is copied from detectron2)". The webpage jumps to https://github.com/facebookresearch/detectron2/blob/master/docs/notes/benchmarks, which is stripped of .md and shows a 404 error.
I have searched for solutions in Google and among the issues of sphinx and recommonmark but found nothing useful. Therefore I provide a workaround, which is to add a / after .md. Do you have a better solution, such as modifying the building script of read-the-docs? If not, the workaround is easy and solves this tiny bug.

Thank you, we will fix this bug.
If we find a better solution, we will record it in this issue.

Thank you, we will fix this bug.
If we find a better solution, we will record it in this issue.

I can also open a pull request.

Thank you, we will fix this bug.
If we find a better solution, we will record it in this issue.

I can also open a pull request.

PRs are welcomed! Thanks for your contribution.


Does suport agent upload data cache to MQ  , oap pick data from MQ?

Please answer these questions before submitting your issue.

Why do you submit this issue?
[x] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?
Does suport agent upload data cache to MQ first , later oap pick data from MQ?
I find OAP receiver parse Big trace segement（more then 1000+ spans） in heavy load status，CPU reach nearly 100%。（8C16G）。

No, even it is, don't help the CPU payload. It will only make you cost more CPU.

No, even it is, don't help the CPU payload. It will only make you cost more CPU.
I think the benefit of MQ is that the disk exchange to the CPU，agent respone no timeout，and Reduce CPU pressure  ，Balance busy and idle time of the OAP  。

But with that, the value of a monitoring system is keeping reduce. With several minutes latency to trigger alarm is meaningless. If you want, agent could override a trace sender by a plugin. I just said from my understanding, it is useless for a monitoring system.


eye and gaze tracking in opencvsharp

Hi, I want to understand that part of the screen, for example, the computer that the user's eye is looking at.  Wherever the eye moves, the mouse pointer moves as well. How to write this program in C # language and image processing and webcam and opencvsharp library. Does Libray openvsharp . have an example for eye tracking?

OpenFace: https://github.com/TadasBaltrusaitis/OpenFace, is free and has eye tracking estimation. You need to build it in Visual Studio 2017. Once you have the FeatureExtraction.exe, you can call it from Unity with:
//Launch OpenFace FeatureExtraction programatically.
private Process gaze = new Process();
string arg = " -q -g -3Dfp -aus -pose -gaze -device 0 -of " OutPutFileName";
gaze.StartInfo.Arguments = arg;
gaze.StartInfo.CreateNoWindow = true;
gaze.StartInfo.UseShellExecute = false;
gaze.Start();
The OutPutFileName output file will be a .csv which you can open in Excel to see the user's eye gaze.

It is possible to learn step by step through the image or possibly make a video. I am a beginner.

thank you . It is possible to learn step by step through the image or
possibly make a video. I am a beginner.
On Fri, Nov 20, 2020 at 3:28 PM Eoghan Hynes<EMAIL_ADDRESS>wrote:

OpenFace: https://github.com/TadasBaltrusaitis/OpenFace, is free and has
eye tracking estimation. You need to build it in Visual Studio 2017. Once
you have the FeatureExtraction.exe, you can call it from Unity with:
//Launch OpenFace FeatureExtraction programatically.
private Process gaze = new Process();
string arg = " -q -g -3Dfp -aus -pose -gaze -device 0 -of "
OutPutFileName";
gaze.StartInfo.Arguments = arg;
gaze.StartInfo.CreateNoWindow = true;
gaze.StartInfo.UseShellExecute = false;
gaze.Start();
The OutPutFileName output file will be a .csv which you can open in Excel
to see the user's eye gaze.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/shimat/opencvsharp/issues/1084#issuecomment-731127726,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE2ULSQFZMAMNY2YVHGLR2LSQZKWBANCNFSM4TOLWZWA
.

@A00107408  What do you think about this algorithm? Does the algorithm detect the eye? https://github.com/Star-Clouds/CenterFace  . What is the difference between the algorithms of the two? https://github.com/TadasBaltrusaitis/OpenFace and https://github.com/cmusatyalab/openface

I have worked on this topic for weeks. I ended up failing. Because I think it's impossible to perform high-accurate gaze tracking with ordinary webcam. Typical eye trackers are equipped with several near-infrared emitters, which can illuminate the eye to identify the pupil’s position and diameter .
Furthermore, the algorithm of the eye-trackers has been improving for decades. The eye is a very complex system. For example, pupil movement and attention movement are not simply linear, and there are saccades between fixations.
Then, I tried WebGazer.js, which is a full-fledged webcam eye-tracking library developed by Brown University and Pomona College. After calibration, WebGazer.js can achieve more precise effects. However, the precision of WebGazer.js is just about
100px (Dimri, 2018), and the accuracy at the edge of the screen is lower; therefore, WebGazer.js is not accurate enough for web evaluation.
So I suggest you turn to dedicated eye tracking device, like Tobii.

I have worked on this topic for weeks. I ended up failing. Because I think it's impossible to perform high-accurate gaze tracking with ordinary webcam. Typical eye trackers are equipped with several near-infrared emitters, which can illuminate the eye to identify the pupil’s position and diameter .
Furthermore, the algorithm of the eye-trackers has been improving for decades. The eye is a very complex system. For example, pupil movement and attention movement are not simply linear, and there are saccades between fixations.
Then, I tried WebGazer.js, which is a full-fledged webcam eye-tracking library developed by Brown University and Pomona College. After calibration, WebGazer.js can achieve more precise effects. However, the precision of WebGazer.js is just about
100px (Dimri, 2018), and the accuracy at the edge of the screen is lower; therefore, WebGazer.js is not accurate enough for web evaluation.
So I suggest you turn to dedicated eye tracking device, like Tobii.


Fix jarhell of common-utils

Signed-off-by: Zhongnan Su<EMAIL_ADDRESS>Description
While adding common-utils to sql project for transport interface feature[https://github.com/opensearch-project/sql/pull/533], there's a jarhell issue caused by common-utils being packaged into ml-commos jar. Need to change it from implementation to compileOnly
| Caused by: java.lang.IllegalStateException: jar hell!
| class: org.opensearch.commons.ConfigConstants
| jar1: /Users/szhongna/Desktop/Projects/OpenSearch/sql/doctest/build/testclusters/docTestCluster-0/distro/2.0.0-INTEG_TEST/plugins/.installing-4482172287902453920/common-utils-<IP_ADDRESS>-SNAPSHOT.jar
| jar2: /Users/szhongna/Desktop/Projects/OpenSearch/sql/doctest/build/testclusters/docTestCluster-0/distro/2.0.0-INTEG_TEST/plugins/.installing-4482172287902453920/opensearch-ml-client-<IP_ADDRESS>-SNAPSHOT.jar

Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #328 (0d95d99) into main (42fd67e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #328   +/-   ##
=========================================
  Coverage     92.16%   92.16%
  Complexity      360      360
=========================================
  Files            51       51
  Lines          1111     1111
  Branches         51       51
=========================================
  Hits           1024     1024
  Misses           69       69
  Partials         18       18

Flag
Coverage Δ

ml-commons
92.16% <ø> (ø)

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

Continue to review full report at Codecov.

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

@anirudha @penghuo  Need 1 more reviewer.

@zhongnansu feel free to backport to 1.x, 2.x branch if needed.


Errors in logfile

2023-07-16 17:04:14.933 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.h_m_v_d_zijden_current_electricity_market_price (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
2023-07-16 17:04:14.944 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.h_m_v_d_zijden_next_hour_electricity_market_price (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
2023-07-16 17:04:14.952 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.h_m_v_d_zijden_lowest_energy_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
2023-07-16 17:04:14.960 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.h_m_v_d_zijden_highest_energy_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
2023-07-16 17:04:14.991 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.h_m_v_d_zijden_average_electricity_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
2023-07-16 17:04:15.002 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.h_m_v_d_zijden_current_percentage_of_highest_electricity_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
2023-07-16 17:04:15.015 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.h_m_v_d_zijden_time_of_highest_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('timestamp') it is using; expected None; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
2023-07-16 17:04:15.025 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.h_m_v_d_zijden_time_of_lowest_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('timestamp') it is using; expected None; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.

Similar list of errors in my log file
Entity sensor.entso_e2_highest_energy_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
Entity sensor.entso_e2_average_electricity_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
Entity sensor.entso_e2_current_percentage_of_highest_electricity_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
Entity sensor.entso_e2_time_of_highest_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('timestamp') it is using; expected None; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
Entity sensor.entso_e2_time_of_lowest_price_today (<class 'custom_components.entsoe.sensor.EntsoeSensor'>) is using state class 'measurement' which is impossible considering device class ('timestamp') it is using; expected None; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.

v0.4.0 is released this version contains fixes that solve a lot of warnings and known issues. This issue is probably also solved.
Please update to the latest version.
Closing this issue for now. If you still have issue please feel free to open a new issue.


Enabled fixed ip for tracing loadbalancer.

/cc @sdake @linsun @morvencao @costinm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: gyliu513
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: gyliu513, morvencao
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

@gyliu513: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
1841fd2f5e80d0d5f4d45fe261ca80f645e0a702
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Dice Score

Just wanted to make sure I'm looking at this correctly: the Average Dice Coefficient which is printed out after you run test should be different then the DICE score which is shown on the  'dice.png'

Yes, that's correct! The average DICE score is printed over the batch that's currently being trained over. However, the DICE Score on 'dice.png' is calculated for that particular image.

I guess my question is this: A low Dice 'loss" reflects a better forward propagation as a smaller loss indicates you don't have to adjust your parameters during training. However, for Dice "Score", two samples with good overlap (ie. segmentation map closely matches ground truth) should technically be higher(eg. 1 for max similarity, 0 for minimum similarity). Is that a true statement?  If that is the case, then the average DICE "loss" is printed out over the training samples where as the Dice "Score" would be in the 'dice.png' file.

You got it right! Given two images, a DICE score of 1 means perfect overlap, and a DICE score of 0 means no overlap at all. However, from losses.py, you can see the DICE Loss is calculated as loss = 1 - torch.sum(dice_eso) / dice_eso.size(0), which is 1 minus the average DICE Score across all images in the batch.


requesting formatspades in the results directory

Description of feature
It would be great to get the NFCORE_METATDENOVO:METATDENOVO:FORMATSPADES output in the results directory since the modified headers are in the bam files and dealing with those files would be much easier without looking for the original reference file used to align reads.
Cheers

fixed in #286


Use NumPy 1.24 in CI and bump baseline API

Refs #7243
Blocked by #7245

/test mini

/test mini

Test failure is irrelevant.

@emcastillo Failed to backport automatically.

Not cleanly cherry-picked


OutputProcessor

This extracts the logic which sections and rows should be included in an output file from Abstract into a separate class, OutputProcessor. This focusses the formatters on the actual formatting aspect.
The language fallback mechanism was previously also handled by the formatters, but in my opinion it's actually an output processing aspect, too. It's very unlikely that one wants a language fallback for one format but not for another (say Android yes, Apple no). I think it's way more likely that one want's the same fallbacks for all formatters. (Maybe not the same fallbacks for all projects, however that would require the ability to configure the fallbacks).
All tests still pass. Feedback appreciated.

Looks good! Thanks for pulling this out into its own commit and PR.


[REFACTOR] :recycle: Seprate admin action's in it's own endpoint.

Description
Refactor the code to differentiate between /admin and /user's endpoints. All performable API calls for admin can have thier own endpoint for better security and management. It would also avoid the confusion for future Developer's.
Current Behaviour (as of currently available endpoint's):

[ ] /users/markverified/:id  (PATCH) - Marks a user verified.
[ ] /users/:id (DELETE)  - Delete's a user.

Expected Behaviour

[ ] /admin/users/markverified/:id  (PATCH) - Marks a user verified.
[ ] /admin/users/:id (DELETE)  - Delete's a user.
. . .

/users/:id (PATCH) - Updates a user

@is-it-ayush I have created this route so that user can update his username so shouldn't
it be as PATCH /users/:id

@aayushchugh . Thank You! I updated the original issue.


OpenAI Assistant Module not being shown in the js docs

Why aren't I able to find OpenAI Assistant where it was before?

I needed to input the direct link found via a doc search

Here is the link https://js.langchain.com/docs/modules/agents/agent_types/openai_assistant#assistant-tools
I am using Brave Browser, I've already done an empty cache and hard reload.

🤖
Hi @ed3899! I'm Dosu, a friendly bot here to assist with bugs, answer your questions, and help you contribute to the project. While we wait for a human maintainer, feel free to ask me anything about LangChainJS. I'm here to help! 😊
It seems like you're having trouble accessing the OpenAI Assistant module in the LangChainJS documentation. This could be due to a number of reasons, including changes in the documentation structure or issues with the website's indexing.
From the context provided, it appears there was a similar issue in the past where users were having trouble with the ChatOpenAI function. However, this seems to be a different issue as it pertains to accessing the documentation rather than a specific function.
As a first step, I would recommend checking if the issue persists across different browsers and devices. This can help rule out any local issues with your browser or device.
If the issue persists, it could be that the documentation has been restructured or the OpenAI Assistant module has been moved or renamed. In this case, you might want to use the search function on the documentation website to locate the module.
If you're still unable to find the OpenAI Assistant module, it would be helpful if you could provide more details about the issue. For example, are there any error messages displayed? Are you able to access other parts of the documentation? This information can help in diagnosing the issue.
I hope this helps! If you have any further questions or need more assistance, please don't hesitate to ask.
Sources

ChatOpenAI from doc throwing a request error

This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.
Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.
If you want to continue the conversation, start your reply with @dosu-bot.

Checked on Google Chrome:

And on Microsoft Edge:

Same issue. It is not showing up

We hid it since it's not quite like the other agents, but we can add it back in!

@jacoblee93

We hid it since it's not quite like the other agents, but we can add it back in!

Yup, me and my team at @Gptcustom would appreciate that.
We are currently working a lot with them.
One of my team members noticed @ulisesmora is the inability of the Assistant Agent to reply to a specific thread.
To give you some context, before we were doing:

Langchain + Xata to give the illusion of memory (context)
Xata to vectorize pdfs
ConversationalRetrievalQAChain(legacy)

Now, with assistants things are different:

They can consume files
They've got functions
They handle context a.k.a via threads

However, when it comes to replying.
const assistantResponse = await assistant.invoke({
  content: "Hello world!",
});

It looks like the assistant can only be invoked generally, not on a specific thread. In that case, how do we tap into a specific thread? (to keep the conversation going with a user)
Please let us know if we are missing something.
We are thinking about maybe creating our own executor but we would like to explore simpler alternatives before that.
Thanks!


[Google Translate] Switch texts when switching languages

Extension
https://www.raycast.com/gebeto/translate
Description
When switching languages between themselves, the texts inside the bodies should also change. As original google translator does. Otherwise it doesn't make sense.
Who will benefit from this feature?
Everyone who needs it!
Anything else?
https://translate.google.com/
This is the reference!

Thanks for your suggestion! I'll try to implement it soon 🙌


[Clash of Nodes] Validator Staking & Explainer Material

https://docs.availproject.org/docs/operate-a-node/become-a-validator


How does it work?

Do you have some explanations how drift is detected? For example what statistical test is used for which data type?  I can't find it in the docs. Thanks!

Hi @pplonski! Here is the relevant section of the docs: https://docs.evidentlyai.com/reports/data-drift#how-it-works. There is a default logic that picks a statistical test based on feature type, the number of unique values, and the number of observations. You can also modify it by choosing a different test or adding your own.

Thank you!


merging back potentially split shares into corresponding fields of Block.Data

Currently, there is no method to parse back the block.Data. This will be required to read the block though (#212).
Ideally, this would be implemented in a separate PR. The way this would work is to use the fact, that the shares are encoded with a length delimiter. So by reading the length from a share, we should be able to deduct the number of shares a Tx (messages / interm. state roots/ evidence) actually spans and merge the shares back together (if necessary).
ref: https://github.com/lazyledger/lazyledger-specs/issues/147

can we close this issue?

Yes, I think this is completed. Otherwise this would not work: https://github.com/lazyledger/lazyledger-core/blob/7fca45bbd271909c5c14bf1ce60071481540d600/p2p/ipld/read.go#L27


Set values to default and additional checks if they are unfilled

Envoys send EnvoyHealthCheck after ShardInfo, and there is a short period between them. The director filled part of values (last_updated, valid_duration) in shard_info after EnvoyHealthCheck getting and didn't check if they are not set  in GetEnvoys API.
last_updated and valid_duration values are now filled in by default on envoy registration.
Additional checks that last_updated and valid_duration are not set were added in GetEnvoy API.

ok to test

ok to test


Upgrade Risc0 to v0.19.0

Description
This PR upgrades Risc0 from v0.18 to v0.19. While the API changes are minimal, the "big" new feature seems to be in-VM recursion. This PR does not test in-VM recursion to see if behavior is as expected or if more changes are needed from our part to support it.
Starting from v0.19, Risc0 also makes available precompiled cargo-risczero binaries as part of GH release artifacts (for x86_64 Linux and Apple Silicon), which means we can skip compilation and install those directly in CI. Fixes #1036.

Related to https://github.com/Sovereign-Labs/sovereign-sdk/issues/633.

Rountrip hint (de)serialization seems buggy; working on fixing it.


MSPSDS-154: Improved source in investigation and product

Investigations and Products now use a more appropriate concept of a source

Closingmoved to #9


OMPL export file lacks options metadata

To reproduce

have subscriptions from Feedbin (not sure if the account type matters)
check option for "Always Use Reader View"
export feeds
The feed in resulting OPML has no metadata corresponding to the option.

Ideally OPML could contain all the data required to restore one's subscriptions.

When exporting OPML we’re making sure it’s of the same format that other readers expect.
Other readers know nothing about our extra metadata, and so we don’t include that in the exported OPML, because it could prevent OPML import from working with other readers.

It seems like that’s against the spirit of XML, ignoring attributes that you don’t know about. What are these other readers doing instead?

I haven’t seen OPML with custom attributes from any other reader. Doesn’t mean it doesn’t exist, but I’ve never seen it.
We’re following the “be conservative in what you send” rule with this, to make sure our OPML is compatible with every other RSS reader.

Tiny Tiny RSS does this by default.


3.c.1

General:

[x] Update data source to: Occupation stats from CPOP for all Prov/Terr, by gender
Statistics Canada. Table 98-10-0447-01 Occupation unit group by highest level of education, major field of study, age and gender: Canada, provinces and territories
DOI: https://doi.org/10.25318/9810044701-eng

2 series

Health worker density, by type of occupation (per 10,000 population)

Density of medical doctors: the sum of all 2
Occupation unit code:
31101 Specialists in surgery
31102 General practitioners and family physicians
Density of nursing and midwifery personnel:: the sum of all 3
NOC =
31301 Registered nurses and registered psychiatric nurses
31302 Nurse practitioners
31303 Physician assistants, midwives and allied health professionals
Density of dentists:
NOC = 31110 Dentists
Density of pharmacists:
NOC = 31120 Pharmacists
Second series:
- Health worker distribution, by sex and type of occupation as defined above

Percentage of male medical doctors: Male doctors as percentage of all medical doctors at national level.
Percentage of female medical doctors: Female doctors as percentage of all medical doctors at national
level.
Percentage of male nursing personnel: Male nursing personnel as percentage of all nursing personnel at
national level.
Percentage of female nursing personnel: Female nursing personnel as percentage of all nursing
personnel at national level.

EN Page:

[x] def and concepts: This indicator measures the densities of medical doctors, nursing and midwifery personnel, dentists as well as pharmacists per 10,000 population. The indicator also measures the health worker distribution by gender for selected occupations. The occupations are based on the National Occupational Classification [NOC] 2021 Version 1.0 occupation codes as mapped to the International Standard Classification of Occupations (ISCO-08).
[x] Comments and limitations: remove the text
[x] Method of computation:
Calculated by dividing the number of health care professionals working in direct care in each health region or province/territory by population estimates and multiplying by 10,000.

Density of medical doctors = ((NOC: 31101 Specialists in surgery + NOC: 31102 General practitioners and family physicians) / Population) x 10,000
Density of nursing and midwifery personnel = ((NOC: 31301 Registered nurses and registered psychiatric nurses + NOC: 31302 Nurse practitioners + NOC: 31303 Physician assistants, midwives and allied health professionals) / Population) x 10,000
Density of dentists = (NOC: 31110 Dentists / Population) x 10,000
Density of pharmacists = (NOC: 31120 Pharmacists / Population) x 10,000

[x] Update sources to :
Statistics Canada. Table 98-10-0447-01  Occupation unit group by highest level of education, major field of study, age and gender: Canada, provinces and territories

and the following for POP: Statistics Canada. Table 17-10-0005-01  Population estimates on July 1, by age and gender

[x] Same in French

I have implemented the changes required! However, when you click on the second series, the main data that shows up first is Alberta, Female. This is a known Open SDG issue where OpenSDG takes values alphabetically when there are no headline data where there are multiple series, which is why Alberta and Female are chosen when switching between the two series.

[ ] For the Density, please remove "Density of" in front of the occupation under sub-category Type of occupation
[ ] for both radio button categories, is it possible to have the default chart showing all occupation?

for both radio button categories, is it possible to have the default chart showing all occupation?

For now, I don't think it is possible. I have searched through Open SDG's documentation and I haven't found a way to set the default display for charts that have two or more series with no headline data. It is something that we could ask the Open SDG team in a ticket to see if that feature could be added or not!


cloudlog: 2.4.10 -> 2.4.11

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for cloudlog is: Web based amateur radio logging application built using PHP & MySQL
meta.homepage for cloudlog is: https://www.magicbug.co.uk/cloudlog
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.4.11 with grep in /nix/store/04gw6r3wvyvgdkys64sm5d5jhd7r2bb4-cloudlog-2.4.11
found 2.4.11 in filename of file in /nix/store/04gw6r3wvyvgdkys64sm5d5jhd7r2bb4-cloudlog-2.4.11

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cloudlog

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/04gw6r3wvyvgdkys64sm5d5jhd7r2bb4-cloudlog-2.4.11 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cloudlog https://github.com/r-ryantm/nixpkgs/archive/eed58e57b53289c91b42e54caf59079f22b9a34b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/eed58e57b53289c91b42e54caf59079f22b9a34b#cloudlog

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/04gw6r3wvyvgdkys64sm5d5jhd7r2bb4-cloudlog-2.4.11
ls -la /nix/store/04gw6r3wvyvgdkys64sm5d5jhd7r2bb4-cloudlog-2.4.11/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cloudlog

Maintainer pings
cc @mattmelling for testing.

Tests pass, all looks good.
Result of nixpkgs-review pr 263272 run on x86_64-linux 1

1 package built:

cloudlog


Keyboard in CAPS mode

The problem is new HTC Sense 3.5 builds, they changed something in their default touch input keyboard so virtual keyboard stays in CAPS mode constantly. Other software keyboards work fine.
Landscape mode is more of a problem. There it happens always. Specially if you change orientation of phone/terminal while keyboard is open. Keyboard stays in CAPS mode until you reset it in some third app
The solution for this problem in Better Terminal was to change >Input method< to Word-Based from Character-Based. ConnectBot doesn’t have such option.

Which device?
HTC Rhyme is working correctly..


Fixed the ustream extractor to use the current ustream API

The current ustream support is broken. I've used the tip from https://github.com/rg3/youtube-dl/issues/6943#issuecomment-144583467 to create a fix

Cherry-picked in 7fd4ed9939d7c467c61663cad7189ee8ad27c89b, thanks.


could support some demo for it?

this is a greate project,but i'dont kown how can i use it? can you give me some support,please? And I'd want to it support OSGB？ Looking forward to your reply.

I wrote this project many years ago. I've forgotten most of the code. So I can only give you a hint. You can have a look at me.cxj.j3dtiles.model.v1 package. The class B3dm, I3dm, PointCloud, Composite are the model class. Their read method create a model instance from the 3dtiles json file.


Create CONTRIBUTING.md of Image

Added the contribution guidelines for this repo in the root  .
https://github.com/appwrite/appwrite/issues/2110

@neelshah2409 please create a PR only after you are assigned to a issue.


Failed to deserialize torrent metainfo

$ imdl torrent show tmp/somefile.torrent
error: Failed to deserialize torrent metainfo from `tmp/somefile.torrent`: Error: bencode encoding corrupted (Keys were not sorted)

Created by uTorrent/2210
File cannot be provided, but works fine in normal torrent clients.
imdl version 0.1.8

That's interesting!
BitTorrent metainfo is stored in a format called bencode, which requires that keys in dictionaries be stored in sorted order. Intermodal rejects .torrent files where the keys aren't in order.
It sounds like other torrent clients don't actually do this, which is surprising.
I'd like to get my hands on the torrent file, just to make sure that the keys are actually out of order, but I understand if the contents are sensitive. I might be able to write a tool that dumps the keys in a torrent file, which would help diagnose the problem without exposing any information about the contents.
Is this the only torrent file that you've had issues with, and did you create it yourself?

same here with torrents from oppaitime.

@awsms Is it the same bencode encoding corrupted (Keys were not sorted) error? Torrents with unsorted keys are technically invalid, so I'm unsure what should be done.


Fix getData crashing in offline

Fixes #8357
The issue was that node property of FCacheNode pokes one level deeper than needed in this case: (FCacheNode.m:66)
- (id<FNode>)node {
    return self.indexedNode.node;
}

Hey there! So you want to contribute to a Firebase SDK?
Before you file this pull request, please read these guidelines:
Discussion

Read the contribution guidelines (CONTRIBUTING.md).
If this has been discussed in an issue, make sure to link to the issue here.
If not, go file an issue about this before creating a pull request to discuss.

Testing

Make sure all existing tests in the repository pass after your change.
If you fixed a bug or added a feature, add a new test to cover your code.

API Changes

At this time we cannot accept changes that affect the public API.  If you'd like to help
us make Firebase APIs better, please propose your change in a feature request so that we
can discuss it together.

@googlebot I signed it!

Thanks for the investigation and PR!  Note the CI failure can be disregarded due to the known issue #7476.

Hey @hybridcattt  I'm working on Google Open Source Peer Bonus nominations and would like to recognize your contribution to improving Firebase Real Time Databas quality. I couldn't find an email address for you, so if you're interested, please email it to paulbeusterien at google.com and I'll make the nomination.


can't run docker command from ubuntu 16.04

my os version : Ubuntu 16.04.1 LTS (Xenial Xerus)
docker version: Docker version 1.11.2, build b9f10c9
docker run -d --privileged -p 1723:1723 -v {local_path_to_chap_secrets}:/etc/ppp/chap-secrets mobtitude/vpn-pptp
docker: Error response from daemon: create {local_path_to_chap_secrets}: "{local_path_to_chap_secrets}" includes invalid characters for a local volume name, only "[a-zA-Z0-9][a-zA-Z0-9_.-]" are allowed.
See 'docker run --help'.

any ideas?

@coney I think {local_path_to_chap_secrets} just the placeholder ,it mean it should be changed to the real path of the chap-secrets of your host so that it can be mounted to the container.

@coney I'm closing this issue as @NikoTung gave the correct answer to your question.


More fixes to make bootc-image-builder work.

@sallyom @Gregory-Pereira @lmilbaum @MichaelClifford PTAL