Further ideas on how WASI/Wasm modules could be used in Hugo

Placeholder.

I'm very happy that Wasm/WASI support has landed in Hugo. Nice work!
I've been using Hugo to build my website for a while and I've noticed that the syntax highlighting provided by Chroma hasn't been as nice as what other highlighters like Syntect or Shiki can produce. I wonder if there could be a way to have another syntax highlighter that uses Syntect/Shiki via Wasm?
One thing to note is that Syntect especially has a long warmup time when loading stylesheets (around 100-200ms last time I tried) but is very fast once initialized. It would be good to have a way for programs to serve more than one request.
As for extending the Wasm support in Hugo, I think the natural choice would be to have some kind of a plugin/extension system. I think Zed has the most advanced WASM plugin system of all, since it uses the WIT format to define an interface for extensions and exposes the API as a Rust crate.

I think the natural choice would be to have some kind of a plugin/extension system

I was thinking of something similar to use wasm as a "plugin" or "function" that can be hooked into the templating system.
A starting point could be supporting using a custom wasm file for string-to-string transformations, which we could easily hook into things like safeHTML or unmarshal.
Often the times I found it tedious to implement things such as splitting a page into different chunks or parsing a Jupyter file which could be more straightforward in a higher-level languages like JavaScript or Go.
potential design for the primitive:
wasm.Transform PATH INPUT [OPTIONS]

transformWasm []byte := content.ReadFile(path)

type TransformInput struct {
	Input   string      `json:"input"`
	Options interface{} `json:"options"`
}

As a module author or end user, I could just define things in JavaScript, similar to the greet.js
const transform = function (input, options) {
  // custom data transformation, e.g.
  return input.toUpperCase()
};

This script could be compiled to uppercase.wasm and included with the module or website.
Then, using it in a partial would be straightforward:
{{ $output := wasm.Transform "/partials/wasm/uppercase.wasm" $input "" }}

This could potentially address requests for "custom functions"

https://discourse.gohugo.io/t/helper-functions-acccessible-by-template/1785
https://github.com/gohugoio/hugo/issues/10633

@imfing thanks for the input. In my head, any use of wasm in Hugo needs to be relatively coarse grained -- there is a relatively big start up cost so you need to think of each wasm file as "a server". So uppercase.wasm may not be the best fit.
Thinking out loud here of some examples that I would find useful:

Image processing for formats not supported by Hugo/Go, e.g. if we added a libwebp "wasm adapter", we could get rid of Hugo's extended build.
Embed Tailwind4 as wasm.
Adding some wasm content adapter plugin API (e.g. sqllite)
Adding some wasm content renderer plugin API (e.g. asciidoc?)
Adding some wasm ESBuild plugins API (e.g. Svelte, Vue)


Cached results should be based on view type

Hi,
first of all thank you for the continuous development of the package :)
Currently the cached results of the API are shared across all view types, therefore changing the view type with refresh=False yields in an incorrect json, which is not only confusing but can result in nasty bugs.
Example (my use case):
paper_refs = AbstractRetrieval("2-s2.0-84951753303", view="REF").references
...
paper_issn = AbstractRetrieval("2-s2.0-84951753303", view="FULL").issn

Output:
paper_refs: [<Reference, len() = 16>, <Reference, len() = 16>, ...]
paper_issn: KeyError: 'coredata'

And switching the statements:
paper_issn = AbstractRetrieval("2-s2.0-84951753303", view="FULL").issn
...
paper_refs = AbstractRetrieval("2-s2.0-84951753303", view="REF").references

Output:
paper_refs: [<Reference, len() = 16>, <Reference, len() = 16>, ...] // these references are different from the REF view, as they should be
paper_issn: '0926227X'

Since the cache is extremely valuable in my use case (saves ~60% of API requests) I need to be able to switch views without refreshing the json.
Can the cached jsons from different views be treated as different?

Yes, I've been thinking about this issue for a while.
My currently preferred solution is to have different subfolders for each view and cache files there.
I'll think a bit more about it and implement it next week.

Related: #81 , #66.

Files are cached according to their view in subfolders to the previously existing folders.  The patch will go live with the next major release (i.e. scopus 2.0), somewhen the coming week.  The change affects all classes, even those where only one view is implemented.
If you want to make use of previously cached files, you have to move them:

~/abstract_retrieval/ to
~/
~/
~/

Thank you so much for the hard, excellent work!


meta: Staging server

Deploys all branches somewhere on every push, so we have a central place to look at and discuss.

Question - I don't think we need a real server for this now right? I could probably set up Travis to deploy <branch>-example.html pages to an S3 bucket upon successful build?

Yes, if we are willing to use some of the Transloadit’s own test APIs for actual uploads. Which is probably the way to go.

:+1:


[CHORE] Add a README to wasmcloud-core

https://crates.io/crates/wasmcloud-core has no README!
We should say something about it 😄

Test

Test comment


Tabular: Minor improvements

Issue #, if available:
Description of changes:

GBM preset -> GBMLarge and moved to 'GBM' list so it is filtered out by exclude_models.
Documented 'multimodal' hyperparameters preset.
Minor cleanup on TextPredictor.predict.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-964-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-964/2/index.html

Job PR-964-4 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-964/4/index.html


[WIP] Bill customer by Invoice.

Waiting for feedback here to finish this.

@alya for showing invoice payment status and pending invoices for additional licenses, should we just add a new heading which shows a list of due invoices with their description?

@alya for showing invoice payment status and pending invoices for additional licenses, should we just add a new heading which shows a list of due invoices with their description?

Sure, that seems reasonable. Maybe it's best if you draft something and post screenshots, and I can help iterate on the details?

Sure, that seems reasonable. Maybe it's best if you draft something and post screenshots, and I can help iterate on the details?

Okay, I can open a separate PR for that.

This overall looks good, though needs more testing; I'll post about a plan for that in Zulip. Thanks for all the work on this @amanagr!
A couple more notes I'll post here because the GitHub UI is really struggling with this PR:

NOTE: Only use last_send_invoice to display invoice due information and not to verify payment.

It would be good to extend this comment to explain why we should only use it for that.

days_until_due = 1

I think we might want to look at displaying the end of their free trial as the due date here, in a follow-up PR. It feels like that would set expectations nicely for folks who are on a free trial -- if they don't pay before the expiration date, they will have a service interruption.
But for folks without a free trial, immediately seems fine!


Merge pull request #2 from VladmirPutgang/Reset-accomplishments-to-my…

…-level
Reset checklist & change attributions to new src

Merging master over to main after creating main and changing it to the default branch


Remove AFS proxy settings

[Enter feedback here]
I am wondering why the cmdlets “Remove-StorageSyncProxyConfiguration” doesn’t exist in this guide. As we were looking for a way to remove the existing proxy configuration, but couldn’t figure out how to do it… post trying this command it worked:

Can this be added please?

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

ID: 72b865a2-4634-dee0-50b2-72ef9e0bbce0
Version Independent ID: bbd0ac41-a189-9aad-2479-1b6d9e14efe2
Content: Azure File Sync networking considerations
Content Source: articles/storage/file-sync/file-sync-networking-overview.md
Service: azure-file-storage
GitHub Login: @khdownie
Microsoft Alias: kendownie

@Abu3lia-MS
Thanks for your feedback! We will investigate and update as appropriate.

@Abu3lia-MS I agree we should include that cmdlet in the docs, but this guide isn't the right place for that kind of detail. I suggest we include it here instead: https://learn.microsoft.com/en-us/azure/storage/file-sync/file-sync-firewall-and-proxy

Added to this article: https://learn.microsoft.com/en-us/azure/storage/file-sync/file-sync-firewall-and-proxy
#please-close


How to launch the app with specific name?

There is a game named "com.tencent.tmgp.sgame", when launch it from renderdoc, it will always fail due to wrong activity name.
Checked the renderdoc Log, it will launch with "com.tencent.tmgp.sgame/com.tencent.tmgp.sgame.SGameActivity".
While manually launch it, from the adb log it was "com.tencent.tmgp.sgame/.SGameActivity".
How can I set in the RenderDoc? Thanks.

The GUI uses the following adb command to get the activity name to launch:
adb shell cmd package resolve-activity -c android.intent.category.LAUNCHER %s
Can you run that for your package and tell us what activity name it is reporting? (Might as well just paste the whole output) It looks for the "name=" string so its possible we may need to cut off the package name from the activity.
Also can you give any more details about the exact error when it fails due to the wrong activity name?

I'm trying to be a bit more reactive in closing issues, and as part of that if an issue is sitting indefinitely needing more info from the original reporter I will close it after a while.
A closed issue of course can easily be re-opened if more info is provided, so please if you can provide the information requested above then please comment and we can investigate. But having old issues cluttering the list doesn't help anyone if there's no solution to the original problem.


New design of PP Express Checkout page

Hi team,
at this moment we have such design of EC page: https://www.evernote.com/shard/s406/sh/25841a61-2d92-49e5-947a-7b32f65c3ad9/7c0ea25b2d5bebdb but we would like to use the new one:
https://www.evernote.com/shard/s406/sh/7572fa08-fa6c-4cae-a4bc-70d1a406edcb/5dca1013161f3db6
I was informed by PP Integration Engineer that the new design is incompatible with the Digital Goods flow at this time. And he asked me to contact you for solving this issue.
Could you please assist us in this? Please inform me if you need additional information.

Digital goods is currently being ramped across from the legacy checkout app.
For now, could you just use regular Express Checkout without digital goods?

Hi, thank you.
Would it be possible for us to use a new design along with Digital Goods in future?

You can use it right now -- my recommendation is just to use regular Express Checkout, and do not pass digital goods parameters.
If this isn't possible, then yes, in future your site will be auto-migrated over to the new PayPal checkout design.


[!] Android toolchain - develop for Android devices is partially installed; more components are available. (Android SDK 27.0.3)

[!] Android toolchain - develop for Android devices is partially installed; more components are available. (Android SDK 27.0.3)
what to do?

https://stackoverflow.com/questions/46557775/how-to-fix-partially-installed-sdk-android-studio
Please consider asking support questions in one of the other channels listed at http://flutter.io/support .

If this issue persists for you even after the recommended fix from StackOverflow ^, changing Java version to 8 worked solved this for me. (I was using JDK 11)


Revert from Async to Sync

Begin work to revert from Async to Sync for Java-Kiota components.
fixes #175

appreciate the feedback and insight @baywet , thanks!


Resolve build warnings

Delimination Criteria:
The following warnings cannot be resolved yet because they depend on an external issue (https://github.com/RicoSuter/NSwag/issues/4681) being resolved first.

/home/schauties/git/nexus/src/extensibility/dotnet-extensibility/DataModel/Representation.cs(114,10)
/home/schauties/git/nexus/src/extensibility/dotnet-extensibility/DataModel/Resource.cs(41,10)
/home/schauties/git/nexus/src/extensibility/dotnet-extensibility/DataModel/Resource.cs(48,10)
/home/schauties/git/nexus/src/extensibility/dotnet-extensibility/DataModel/Resource.cs(55,10)
/home/schauties/git/nexus/src/extensibility/dotnet-extensibility/DataModel/ResourceCatalog.cs(43,10)
/home/schauties/git/nexus/src/extensibility/dotnet-extensibility/DataModel/ResourceCatalog.cs(47,10)


Cannot create collection if location path has brackets

I try to create a collection where location is "/Users/xxxx/Desktop/bruno (test)" and I get "An error occurred while creating the collection".
No spaces works, spaces works, no brackets works, brackets doesn't work.
Bruno v1.9.0
MacOS 14.1.1

I assume this is related to / caused by not being able to have special characters in the "Folder Name" input.

experiencing this as well on Windows

Same thing with a hash in the path.


Add naming conventions and requirements to documentation

Getting the following bug in excpetions.py while using a connect command. Both the object being connected are valid and exists. Connect command is working fine elsewhere
File "/usr/lib/python2.7/wsgiref/handlers.py", line 85, in run
    self.result = application(self.environ, self.start_response)
  File "/usr/local/lib/python2.7/dist-packages/django/contrib/staticfiles/handlers.py", line 67, in __call__
    return self.application(environ, start_response)
  File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/wsgi.py", line 241, in __call__
    response = self.get_response(request)
  File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py", line 177, in get_response
    response = self.handle_uncaught_exception(request, resolver, sys.exc_info())
  File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py", line 219, in handle_uncaught_exception
    return debug.technical_500_response(request, *exc_info)
  File "/usr/local/lib/python2.7/dist-packages/django/views/debug.py", line 66, in technical_500_response
    html = reporter.get_traceback_html()
  File "/usr/local/lib/python2.7/dist-packages/django/views/debug.py", line 286, in get_traceback_html
    c = Context(self.get_traceback_data())
  File "/usr/local/lib/python2.7/dist-packages/django/views/debug.py", line 278, in get_traceback_data
    c['exception_value'] = smart_unicode(self.exc_value, errors='replace')
  File "/usr/local/lib/python2.7/dist-packages/django/utils/encoding.py", line 39, in smart_unicode
    return force_unicode(s, encoding, strings_only, errors)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/encoding.py", line 71, in force_unicode
    s = unicode(s)
  File "/usr/local/lib/python2.7/dist-packages/neomodel/exception.py", line 35, in __str__
    trace = "\n    ".join(self.java_trace)
TypeError

In my case the issue was with relationship name having space in it e.g "belongs to".

Thanks for the information @siddharthm. I'm glad you were able to figure out the issue. I've updated the issue's name to ensure we add naming conventions and requirements to the documentation to help mitigate this issue in the future.
I'm pretty sure this requirement is down at the Neo4j layer but I could only find requirements on identifiers and not relationships in particular.

@bleib1dj Thanks a lot

I think it is possible the relationship name needs quoting with back ticks, however i believe the neomodel code doesn't quote them.

@robinedwards good catch. What are your thoughts on adding in the capability?

Doesn't seem like it would be too much trouble to wrap relationship names or node labels in backticks, since it doesn't seem to have any negative effect for names without spaces.

The thing is they do break when they are mapped to python identifiers such as properties (RelManagers)

I have made a note of this in the documentation


tilt: 0.17.2 -> 0.17.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/tilt-dev/tilt/releases.
meta.description for tilt is: "Local development tool to manage your developer instance when your team deploys to Kubernetes in production"
meta.homepage for tilt is: "https://tilt.dev/"
meta.changelog for tilt is: ""
Updates performed

Golang update

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

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.17.10 with grep in /nix/store/hkbfk0mfxs58q69qnvyffq1a5ci3lbc8-tilt-0.17.10

directory tree listing: https://gist.github.com/f6eac73fc8d51729ecebebfbf54c1492

du listing: https://gist.github.com/c9c4d42e3ec451eec33438dd7b4e513b

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
tilt

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hkbfk0mfxs58q69qnvyffq1a5ci3lbc8-tilt-0.17.10 \
  --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 tilt https://github.com/r-ryantm/nixpkgs/archive/e3dd2a3fedd08f1f3c196f035f8b77ae3bb74217.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hkbfk0mfxs58q69qnvyffq1a5ci3lbc8-tilt-0.17.10
ls -la /nix/store/hkbfk0mfxs58q69qnvyffq1a5ci3lbc8-tilt-0.17.10/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 1

1 package built:

tilt

Maintainer pings
cc @anton-dessiatov @kalbasit for testing.

Already a newer version on master.


Adding Jade is a clean, whitespace sensitive syntax for writing html.

Hey @BykovDenis, thanks for submitting this PR.
I am seeing a lot of customizations in your code, like color and styles. I wonder if you are going to submit it to this repo or another repo that you are really working on.
If you do want to introduce Jade into this repo, I may need more time to evaluate it. Also, could you exclude the build folder and translate the comments into English, which is more understandable for most people?


docs: Add Issue Templates

Changes
Adding Issue templates
Template path  .github/ISSUE_TEMPLATE
Template Types

Contribution towards GSsoc`22, Thank you

@maityamit Go Ahead!

Is someone working on this issue??

@TG922 Yes , I already created a PR , but not accepted till now


Dropping support for node.js 4.x and 9.x

I'd like to submit a blanket PR to drop support for node.js 4x and 9.x . This would be a semver major across the board.  Given that we're no longer worried about the constructor pattern issue with TypeScript, does anyone have an objection to dropping these across the board?
@GoogleCloudPlatform/node-team @GoogleCloudPlatform/yoshi-nodejs

As we are looking at rev-ing some of the libs anyway with recent changes I think this is a solid idea. Are you planning on tackling this work? It should mostly be updating package.json and test configs right?

We should be able to use the @google/repo to do it - @alexander-fenster has a bunch of good examples in there 😄

We are waiting for a final conversation with legal on this one.  As soon as that's cleaned up, we can use repo to blanket drop it.

This is fixed now!


Data for camp details and its distribution (currently sparse) see https://process.keralarescue.in/relief

Subject of the issue
There is currently no way to visualise how the camps are distributed throughout the districts
Actual behaviour
Developed a dashboard for visualising the camp details. Currently, the data is largely sparse with many info not added properly. Child counts, male/female counts may not match up to the total.
Please see, https://process.keralarescue.in/relief
Request improved data collection for accurate info dessimination.

Anyone confirm this DMO portal populated, from any 3rd party ?

@laxman954 based on the data from https://www.keralarescue.in/camp_requirements/?district=&name__icontains=&taluk__icontains=&village__icontains=

@ssat335 do you have an idead of how these page constructed?


mat4.perspective problem?

version 1.3.7 work well, but version 2.6.1 not work well,
someone get problem ?

@lmj01 Passing Math.Infinity or null to the last parameter now properly creates an infinity-perspective matrix: https://github.com/toji/gl-matrix/pull/318

I wrap gl-matrix for my code, i commit the mat4.perspective, it work fine, but i set the prespective , the result is not in scene viewport, may it will change in shader multiply order

@lmj01 Sorry, this is a bit hard to understand.
So, you've already pinned down the issue to mat4.perspective? If  you use the

@stefnotch , sorry,  i don't gei much more information about the problem.
I test for single method, it get the result,
my scenario is , when I set mat4.targetTo, and multiply the perspective project, the result is very unpredicate. in old version, i don't have the problem, so i got the confused by this,
and because i comment the the mat4.perspective, it work well,
[call this function, all the parameter are the same]

If you use the code I posted above mat4.perspective does it work?

Which old version are you exactly referring to? 1.3.7? If yes, please take a look at https://github.com/toji/gl-matrix/issues/142

code is very common, i got the problem, the #142 give me to cause


[Request]: Invoke-WGUpgrade -all /full

Describe the request
Invoke-WGUpgrade -all / >>winget upgrade --all
Invoke-WGUpgrade  --full   /winget upgrade --all "--include-unknown"
one-line upgrade everything could be useful esp on scheduled tasks
PowerShell version
7.2
Platform
Windows 10 Pro or Enterprise

I guess I don't see a problem with this approach. The -Full parameter may have to be dynamic since --include-unknown looks like a recent addition to winget.

I've published version 1.4.0 to the PowerShell Gallery with updates to InvokeWGUpgrade`.


Add support for PHP 8.2

Looking for Laravel 10 support as well

Looking for Laravel 10 support as well

That could be done in a separate PR after this one gets merged

Done! Added support for Laravel 10 as well. Thank you!


No dark mode at all

Recently I reinstalled Ferdi, now I've not dark mode at all whatsapp, telegram, etc. they were dark before, now activating dark mode does nothing, any idea on how to fix this or why was working before?
Ferdi ver 5.5.0 (x64)
Platform win32

Reinstalled ver 5.5.0 and now got dark mode on Telegram, Messenger, and Skype, still no dark mode on whatsapp, reinstalled whatsapp service without any change

Status page on whatsapp is shown in dark mode

Did you fix this? I'm having the same issue after install

Sorry still not working for me, whatsapp and telegram not respond to dark mode setting. I closed the issue because got no response and thought that I was the only one with this problem.

I don't get dark mode in WhatsApp or Telegram either. Just noticed that I get dark mode in GitHub Notifications but not when I load an issue (like this one) so it's definitely intermittent!

I shall take this issue up and investigate.

@zeeh1975 Can you try deleting your darkmode.css file in your recipe folder in-case you haven't done that already @markgaze ? I just tried to reproduce this and I get the dark mode theme 100% of the time.

Will give it a go @mahadevans87 and let you know 🙂

I removed the darkmode.css from the whatsapp recipe folder and it made no difference 😕
My Facebook Messenger and Google Calendar directories don't have darkmode.css in so can't do it in there unfortunately.

Interestingly, I just repeatedly toggled Dark Mode on and off in the services and managed to get them to switch to dark mode... So the workaround for this might be to toggle off and back on Dark Mode in the service edit menu. Would be interesting to see if this fixes it for @zeeh1975 too

Yes. I think we shouldn't consider a darkmode.css file which is of 0 bytes. Will see if I can put up a fix for that.

Removing the 0 bytes darkmode.css and disabling and re-enabling dark mode fix the problem on both whatsapp and telegram.

I don't have a darkmode.css in %APPDATA%/Ferdi/recipes/* and disabling and re-enabling dark mode in either the services or globally doesn't fix the issue. This issue occurs on a fresh install of Ferdi.

I have a similar issue with Google Calendar still but if I disable and re-enable then right-click on the service and Reload, then it eventually switches to dark mode.

I don't have a darkmode.css in %APPDATA%/Ferdi/recipes/* and disabling and re-enabling dark mode in either the services or globally doesn't fix the issue. This issue occurs on a fresh install of Ferdi.

What I do was, going to configure service, on bottom has a button that says "Open darkmode.css" that opens a explorer window on service profile, there I deleted darkmode.css, then back to Ferdi, disabled dark mode and then re-enabled again.

I've completely reinstalled Ferdi and now I have a 0-byte darkmode.css. I've deleted it and followed @zeeh1975's steps, but Whatsapp is still not in dark mode, even after reloading the service, reloading the whole of Ferdi, restarting Ferdi, or rebooting my PC.

New Ferdi user here, so I have a fresh install on macOS Mojave. I can't get any of the services to switch to dark mode.... Gmail, Slack, WhatsApp , Messenger, JIRA, Google Calendar - all display in the default (white bg).

New Ferdi 5.5.0 user here as well, also no dark mode on any services with a fresh install. I've tried removing and re-adding services but the issue persists. No darkmode.css is created when toggling dark mode on or off in a service, and when "Open darkmode.css" is clicked, it creates an empty file.

I have this issue as well. Tried every tip but nothing worked.

I fresh installed a few minutes ago for the first time and have this issue with facebook messenger. None of the tips work either

Hello for three days, the dark mode doesn't enable in two services: Slack and Telegram .. strange, tryed to reload many times without success, is there a way to force it? Thanks a lot

Hello for three days, the dark mode doesn't enable in two services: Slack and Telegram .. strange, tryed to reload many times without success, is there a way to force it? Thanks a lot

Ok, dark mode problem only in Telegram, I've activated the dark mode under visual menu (CTRL+ALT+D) but Telegram remains white, no dark mode... strange.

Ok, dark mode problem only in Telegram, I've activated the dark mode under visual menu (CTRL+ALT+D) but Telegram remains white, no dark mode... strange.

On a fresh install of 5.5. (x64), I also have a 0 byte darkmode.css file. Going to try to manually enter any defaults I might find in the repo, as a workaround. This is probably the cause of issue #158 ?

I've workaround the problem installing another telegram service: Telegram React, this one goes with daakmode without any problem.

Good to know, for telegram. A fix is still needed for this issue with several other services.

Hi All.
I just found this and although it is a little late to reply, if anyone is having this issue I have maybe found a fix.
OneNote wasn't working for me in dark mode, so I copied the darkmode.css from Google Calendar to OneNote then reloaded OneNote and boom!
Might work for others as well, definitely worth a try!


Emolsson/11215

tests that we don't leak references when getting the "Cannot start multiple ..." error

+1 after nits


Convert dpx output to be ioproxy enabled

Signed-off-by: Kimball Thurston<EMAIL_ADDRESS>Description
Converts dpx output (input is already) to use the IO proxy layer (I recently
had a reason to write dpx, but couldn't make a memory stream).
Tests
just tested with dpx image tests
Checklist:

[x] I have read the contribution guidelines.
[ ] If this is more extensive than a small change to existing code, I
have previously submitted a Contributor License Agreement
(individual, and if there is any way my
employers might think my programming belongs to them, then also
corporate).
[x] I have updated the documentation, if applicable.
[x] I have ensured that the change is tested somewhere in the testsuite
(adding new test cases if necessary).
[x] My code follows the prevailing code style of this project.

Did you clang-format OutStream.h by hand? We had excluded it from "make clang-format" because it's imported from another project and we wanted to retain the ability to easily diff between them. I'm tempted to recommend reverting the format-only changes to existing code.

But I'm not really going to get bent out of shape about it. In truth, it has been extremely rare over the years for us to need to diff these files versus the originals. And with IOProxy, the files really do have differences, whereas they used to be identical.

Looks like CI is failing the "imageinout_test", which (among other things) for all file types supporting proxies, will write a file and write a proxy and then verify that they are byte for byte identical.

I did reformat the outstream when I did the pull request because I looked at the code after the comment in the PR template - and it wasn't following the same style, so I thought it just hadn't been visited in a while to have been formatted. Happy to revert that if you'd prefer.
I ran ctest (i.e. ninja -C  test) and that succeeded for me, I wonder what I'm missing that imageinout_test doesn't run...

Try running by hand, it lives in BUILDAREA/bin/imageinout_test

I mildly prefer that you remove the formatting change, but only if it's super easy. It's not worth the hassle of extra work, I'm just being superstitious about wanting "imported" source to restict diffs versus upstream to only the things we purposely change.

ah, I forgot dpx puts the filename in the header. I've changed the test  write proxy to just use the disk filename, do you know of any issues with that? I believe I reverted the change to formatting OutputStream.cpp

That sounds reasonable. In the imageinout_test we are also careful to set the DateTime attribute to something fixed so that files don't differ because of that metadata.

Would you mind also fixing builtinplugins.rst to note that DPX takes proxies for output as well as input now?


Add check name to ArgumentError

Pull Request Checklist
Added "value" variable to output of ArgumentError message.
Description
Added "value" variable to output of ArgumentError message.
Related Issue
Motivation and Context
"sensu_check name invalid" needs more information about which check name is failing the ArgumentError. Otherwise it makes it hard to debug when you have thousands of checks.
How Has This Been Tested?
Edited code on our puppet server, pushed out, works great. Told me the name of the check which had accidentally gotten a space in the name during a giant sed.
General

[x] Update README.md with any necessary configuration snippets - No new info.

[x] New parameters are documented - no new parameters

[x] New parameters have tests - no new parameters

[ ] Tests pass - bundle exec rake validate lint spec - haven't tested this.

@amccrea Thanks for the contribution! This will definitely help out debugging.


Make the currently active editor more distinguishable from others

As a user of a large screen space (two monitors) I keep my editor in one side and several terminal windows on the other side.
When my eyes swift from the editor the the terminals and back again, I would like a "very obvious" visual hint of which was the tab I was working on.
Currently all tabs look equally important, but I would greatly appreciate a clear visualization of the active tab
Some random thoughts:

only the active tab should highlight the current line

syntax in the active tab should "stronger" than in the non active tabs. Non active tabs should have a "weaker" visual, but maybe not to the point of making them blurry

The active tab should look very - more different than tabs in the same group (and other groups too)

@fgarcia with the recent support of theming and settings you can actually tweak the background color and foreground color of the active tab:
Something like this in settings to demonstrate what is possible:
"workbench.experimental.colorCustomizations": {
	"activeTabBackground": "#ff0000",
	"activeTabActiveGroupForeground": "#0000ff"
}

Would that help you?

Those settings don't really help that much. They can highlight which tab is active but I still cannot immediately visualize where I was working when I came back to the editor
In the attached screenshots there are four areas where I could have been working. While working outside the editor there is no visual hint in which area I was working

Now look at my terminals. I think one could tell where I was working

Got it, not ideal :thumbsup

Great! I think the easiest and most relevant first step to improve is having an option to highlight the current line "only" in the most recent file

@fgarcia in our latest release we added a new color that helps to make the active tab more stand out:
"workbench.colorCustomizations": {
    "tab.activeBorder": "#ff0000",
    "tab.unfocusedActiveBorder": "#000000"
}


B.1. Define and document the domain model

...  i.e. how to map the relational model into an object oriented model. This requirement is closely related to A.3, but should focus on the object-relational mapping, i.e. how is the relational model in the database mapped into the object-oriented domain model.

Have made the domain model in a UML diagram and implementet the object classes.
Need to make a model that shows the modeling fra relations to objects.


Spike: Test out Zoneless Change Detection

Which scope/s are relevant/related to the feature request?
Don't know / other
Information
This is a placeholder to test out Analog with the experimental Zoneless change detection provider.
Describe any alternatives/workarounds you're currently using
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[ ] No

I have a small zoneless app with ssr and event replay actions using nx.
When I migrated to analog.js, I could see the ssr was deactivated, but running the app I get this exception:
ypeError: Cannot read properties of undefined (reading '_ejsa')
    at getJsactionData (chunk-T5GBXR7Y.js?v=4e96f082:22766:20)
    at chunk-T5GBXR7Y.js?v=4e96f082:22807:41
    at _ZoneDelegate.invoke (zone__js.js?v=4e96f082:295:158)
    at _ZoneImpl.run (zone__js.js?v=4e96f082:94:35)
    at zone__js.js?v=4e96f082:1956:30
    at _ZoneDelegate.invokeTask (zone__js.js?v=4e96f082:321:171)
    at _ZoneImpl.runTask (zone__js.js?v=4e96f082:131:37)
    at drainMicroTaskQueue (zone__js.js?v=4e96f082:476:23)
    at invokeTask (zone__js.js?v=4e96f082:395:11)
    at ZoneTask.invoke (zone__js.js?v=4e96f082:381:38)

When I enable SSR using analog, I get this exception:

This is just informative, maybe do you have an idea of the problems we could found with a zoneless app.
I'm really interested to use analog because nitro and the api routes, I'll keep digging inside analog

Its seems that the analog content provider needs somehow zone. when zonejs is not imported i got the following error. With zonejs importat everything works fine.
C:/workspace/herold-solutions/node_modules/@analogjs/content/fesm2022/analogjs-content.mjs:150
    const macroTask = Zone.current.scheduleMacroTask(`AnalogContentResolve-${Math.random()}`, () => { }, {}, () => { });
                      ^
ReferenceError: Zone is not defined
    at waitFor (C:/workspace/herold-solutions/node_modules/@analogjs/content/fesm2022/analogjs-content.mjs:150:23)
    at __vite_ssr_import_5__.Observable.pipe.__vite_ssr_import_6__.map.content.content [as _subscribe] (C:/workspace/herold-solutions/node_modules/@analogjs/content/fesm2022/analogjs-content.mjs:187:13)
    at Observable._trySubscribe (C:\workspace\herold-solutions\node_modules\rxjs\dist\cjs\internal\Observable.js:41:25)
    at C:\workspace\herold-solutions\node_modules\rxjs\dist\cjs\internal\Observable.js:35:31
    at Object.errorContext (C:\workspace\herold-solutions\node_modules\rxjs\dist\cjs\internal\util\errorContext.js:22:9)
    at Observable.subscribe (C:\workspace\herold-solutions\node_modules\rxjs\dist\cjs\internal\Observable.js:26:24)
    at C:\workspace\herold-solutions\node_modules\rxjs\dist\cjs\internal\operators\map.js:9:16
    at OperatorSubscriber.<anonymous> (C:\workspace\herold-solutions\node_modules\rxjs\dist\cjs\internal\util\lift.js:14:28)
    at C:\workspace\herold-solutions\node_modules\rxjs\dist\cjs\internal\Observable.js:30:30
    at Object.errorContext (C:\workspace\herold-solutions\node_modules\rxjs\dist\cjs\internal\util\errorContext.js:22:9)
Node.js v20.15.0

Hey @brandonroberts, thanks for the recent changes towards supporting zoneless apps! :)
Tried setting up a zoneless (analog) app in my demo monorepo. Having problems removing zone.js imports I found this issue. I setup the demo app to use the trpc integration and was running into the following problem:
NG05000: Angular detected that hydration was enabled for an application that uses a custom or a noop Zone.js implementation. This is not yet a fully supported configuration. Find more at https://angular.dev/errors/NG05000
Angular is running in development mode.
/…/node_modules/@analogjs/trpc/fesm2022/analogjs-trpc.mjs:384
    const macroTask = Zone.current.scheduleMacroTask(`AnalogContentResolve-${Math.random()}`, () => { }, {}, () => { });
                      ^

ReferenceError: Zone is not defined
    at Module.waitFor (/…/node_modules/@analogjs/trpc/fesm2022/analogjs-trpc.mjs:384:23)
    at new AnalogWelcomeComponent (/…/apps/analog-commerce/src/app/pages/analog-welcome.component.ts:73:36)
    at NodeInjectorFactory.AnalogWelcomeComponent_Factory [as factory] (/…/apps/analog-commerce/src/app/pages/analog-welcome.component.ts:94:94)
    at getNodeInjectable (/…/node_modules/@angular/core/fesm2022/core.mjs:6066:44)
    at instantiateAllDirectives (/…/node_modules/@angular/core/fesm2022/core.mjs:12813:27)
    at createDirectivesInstances (/…/node_modules/@angular/core/fesm2022/core.mjs:12209:5)
    at ɵɵelementStart (/…/node_modules/@angular/core/fesm2022/core.mjs:24369:9)
    at Module.ɵɵelement (/…/node_modules/@angular/core/fesm2022/core.mjs:24427:5)
    at HomeComponent_Template (/…/apps/analog-commerce/src/app/pages/(home).page.ts:12:35)
    at executeTemplate (/…/node_modules/@angular/core/fesm2022/core.mjs:12169:9)

Node.js v18.20.4

Looking at this error and your pr, I'm  pretty sure we need an early return at this location aswell. What do you think?

This morning I was further migrating my demo repo and thought I might just aswell look a this trpc zoneless issue and pulled the latest changes from the beta branch. With these 3 lines added in wait-for.ts:
  if (typeof Zone === 'undefined') {
    return prom;
  }

the trpc package seems to work fine in zoneless mode. Since these are just 3 (or 5) lines of changes, I think there is no value of me submitting a RP based on my local branch fix-trpc-zoneless.
@brandonroberts, do you think theres a need of having a zoneless demo/testing app inside the analog repo going forward? Or in other words; do you think zoneless mode will require more efforts to be fully supported inside analog in the future? If yes, I might be able to help here, since I'm planning to investigate more into zoneless change detection in the next few months anyway.

Thanks @tobiasegli. Yes, I think that would be a valuable patch for trpc also as long as its not a breaking change. Having people be able to test Zoneless support moving forward would be helpful.
I think its too early to have a separate Zoneless app in this repo, but we can put together an example repository separately and point people to that if needed.

@brandonroberts: Ok, this makes sense. I also ran the pnpm test command on my local branch which succeeded.
So it might be feasible to "merge" this into the trpc package and release it with a 1.7.3-beta.4 or something like that.
If this change got merged I could continue investigating on a zoneless analog demo app inside of my private repo. If I would run into more issues I could also just fork analog and start patching things there alongside a zoneless demo.

@tobiasegli yep, if you submit the PR, we can get a beta release out. @analogjs/trpc is on version 0.2.45 but we can get a beta of 0.2.46-beta.1 out after its merged.
Let's track that here https://github.com/analogjs/analog/issues/1295


can't connect to firebase

Compilation error: no matching function for call to 'FirebaseESP32::begin(const char [88], const char [40])'
try to connect with the fire base and this error is popped up every time

https://github.com/mobizt/Firebase-ESP32/tree/master/examples


Enable S3, Delete collection loading infinite

Hi~
Sorry. I have infinite loading when I turn on S3, turn off it's ok. Could you give me some suggest where can I check and fix it?
Sorry if i upset you. I'm really love PocketBase and Golang. I'm trying to be pro like you 😬

What do you mean by "infinite loading" and when do you experience it, during delete?
In "Admin UI > Settings > File storage" do you have a green checkmark saying that the s3 config is OK?
If yes, could you provide an example structure of your collection (Admin UI > Settings > Export collections)?
In "Admin UI > Settings > File storage" do you have a green checkmark saying that the s3 config is OK?
In any case, you can start PocketBase with --debug and see if there is a console error.
Also please make sure to test with the latest v0.10.4 release.

@hungcrush I've just tested it locally with the Vultr S3 compatible Object Storage offering and it is working fine for me.
There was an issue in v10.2 and older versions related to the s3 upload, but it should be fixed in the latest release.
If after the upgrade you are still experiencing issues, please provide more details and I'll investigate it more thoroughly.

Hi @ganigeorgiev I just download official version on pocketbase.io

And this is my S3 config:

If you need please give me your email, i will send you my config

You can attach your collections configuration as json here or send it to support @ pocketbase.io.
I suspect this is a permission issue or incorrect S3 configuration (eg. do you really need the "Force path-style addressing"?)
The only way I was able to reproduce it was due to invalid S3 config because currently we don't have an explicit timeout for the S3 operations (it is in the roadmap and will be added together with the individual request timeouts).
Please check:

Does uploading a file works?

Do you have any special access/permission policies for your S3 storage/account (eg. disallowing deletion)?

Hi @ganigeorgiev
I found the problem.

I suspect this is a permission issue or incorrect S3 configuration (eg. do you really need the "Force path-style addressing"?)

I tried to disable this and change from
https://cf-simple-s3-origin-cloudfrontfors3-713801267216.s3.amazonaws.com to https://s3.amazonaws.com
It works well now.

Does uploading a file works?
=> Upload well in both cases

Thank you so much.

Thank you for confirming.
Proper configurable timeouts will be added in the near future.
I'll consider in the next release to improve the automated S3 config check so that it can throw an error when saving the S3 settings.


Empty files in Share on Synology...

I have for every backup also an empty share in an other directory under the Share on my Synology NAS.
Made a description about this on the Community Page.
Description:
https://community.home-assistant.io/t/samba-backup-create-and-store-snapshots-on-a-samba-share/199471/158
Is this as intended? and if so, what is the function of these files?

It seems that you have some kind of "recycle bin feature" enabled on your NAS, which means that files are not actually deleted, but instead moved to that folder. Those files should already be deleted.
The samba-tmp123 is just an empty file which is created and deleted to check write-permissions. This happens if you (re)start the add-on.


Add Supportability content for Transparent Networks on Azure VM

We are seeing a lot of customers using this documentation when using Containers on Azure VMs. It would be good idea to add a note to the transparent network description stating that MAC Address Spoofing is not supported on Azure VMs.
transparent – containers attached to a network created with the 'transparent' driver will be directly connected to the physical network through an external Hyper-V switch. IPs from the physical network can be assigned statically (requires user-specified --subnet option) or dynamically using an external DHCP server.

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

ID: 556473c3-3630-dc56-caad-a3d72d34e9e1
Version Independent ID: 560ad6ca-e017-db71-4605-a73e9f1b8aed
Content: Windows Container Networking
Content Source: virtualization/windowscontainers/container-networking/network-drivers-topologies.md
Service: windows-containers
Product: windows-containers
GitHub Login: @JMesser81

Hi, I totally agree with you. One question that comes up immediately and would be a nice addition: is there a workaround to achieve the same result (multiple containers on multiple IP's accessible externally) for transparent networks with Azure VM's? Thanks!


fix: add additional container definitions for lightweight AZDO jobs/steps

Pull Request Template
📲 What
Alpine containers for TFS jobs/steps
🤔 Why
Why have you implemented it?
🛠 How
How have you implemented it?
👀 Evidence
Link to builds/artifacts/etc.
🕵 How to test
Notes for QA

Hey @dnitsch, we've got a different image structure now, we've moved off of Alpine to Ubuntu Minimal, so the new images we have are compatible with AzDo as direct images if you so choose.


How to use the scripts

Hello,
i want to use the scripts to get connected to an online server after flashing. Is this possible and is there any example i can try?
Can the script start AFTER flashing the CM4?

Is it possible to start a script after flashing process?
Is it possible to a enumeration as a serial number to the devices?
Can I add a certificate during flashing process?

Is it possible to start a script after flashing process?

When adding a script in the web interface you can choose between "pre" and "post" installation.
"pre" means before flashing.
"post" means after flashing.

Is it possible to a enumeration as a serial number to the devices?

Not functionality we provide by default.
cmprovision includes source code though, and you can hire any random PHP developer of your choice, to add such things to your requirements.

Can I add a certificate during flashing process?

I am afraid I do not know what type of certificate you are interested in.
We do not support programming secure boot keys/certificates, if that is what you mean.
Do can generate client-side certificates typically used to communicate with an IoT/cloud service.
If you need an example of a script that does X, I suggest you ask on the forum though.
This are not cmprovision bugs for which the github issue tracker is meant.
Keep in mind that for every issue you create here, everyone that is subscribed to this repository gets an e-mail notification.


Gradient of Parameter ssd1_batchnorm0_beta on context gpu(0) has not been updated by backward since last step

I'm trying to execute the code as it is .. but i dont know why i'm getting this error ..
"UserWarning: Gradient of Parameter ssd1_batchnorm0_beta on context gpu(0) has not been updated by backward since last step. This could mean a bug in your model that made it only use a subset of the Parameters (Blocks) for this iteration. If you are intentionally only using a subset, call step with ignore_stale_grad=True to suppress this warning and skip updating of Parameters with stale gradient " on trainer.step(stepsize)

I tried to run word_and_line_segmentation.py end to end and I did not see the error.

Would it be possible to post the exact arguments you used?
Could you confirm that this line ran correctly?

thanks for your response sir ... i have solved this issue .. i reduced the batch size from 32 to 20 .. i don't know why(i'm new in mxnet) but that was the reason ..
and Sir in IAMdataset there are 1500+ images but why only 967 data for training and 232 data for testing was used?? and how can i increase the training dataset number .
sir, i am a Bangladeshi student and currently doing my undergraduate thesis , my topic is "Bangla handwritten ocr system " , from page to word level segmentation i'm following (implementing) your system which i did ,  i ran the "line_word_segmentation" notebook for word segmentation and set epoch 400 .. but its not detecting word segments properly  . how many epoch should i set for word segmentation ? and sir, "<EMAIL_ADDRESS>"  this is my email . it will be a great help for me if i could contact you through mail .

The IAM dataset has train, validation and test data. We did not use the validation data for training nor testing.
We set the number of epochs to 400 for word segmentation.


Chrome Zpl Printer no longer installable?

Hey @brianzinn !

If you want to preview the label you can install the Zpl Printer extension from the chrome web store and set it up to listen on port 9102

Unfortunately it looks like this extension https://chromewebstore.google.com/detail/zpl-printer/phoidlklenidapnijkabnfdgmadlcmjo is no longer available..?

Any suggestions there.
Thanks for making this software! We're trying to support a zpl printer but don't actually have one... I'm suprised they don't provide an emulator themselves or at least a fake test device with their printer software!

https://github.com/MrL0co/ZplPrinter/releases

So that'll work with this emulator? @parallels999
Thanks

thanks for sharing @parallels999 - so, have you setup and then it runs on port 9102 and acts the same as labelary website?

Yes

Thanks @tnrich for creating this issue and @parallels999 for a solution. If this doesn't fix for you please re-open.  I have updated the main page readme with instructions.  Cheers.

https://github.com/sbinkert/ZplPrinter/issues/7#issuecomment-1331183007


UI 2.0: Build Pipeline for Apps (inclusion into Core's build pipeline)

After pip installing an app, the build UI commands (introduced in #3203) should include and build those app views with the core views.
This should be demonstrated with the example plugin being built when installed locally.

This would effectively create the migration guide for all apps on how to start authoring react views in their apps.

TODO: Review that this is working as expected.


Add in config to avoid AST in some join cases

This is intended to be a test to see if it will improve the performance in some cases so we can possibly come up with a heuristic in the future.

build

Would it make sense to have RapidsMeta have the ability for expressions to be disabled for AST individually if spark.rapids.sql.expression.ast.classname is false? We probably still want an overall AST join enable as implemented here.

Possibly.  I'm not sure that we ever want to actually merge this in.  It is here as a tool for now to see if a specific query would get faster or slower with it.

Closing for now. We can open this again if we need to.


Auto-Editor incorrectly expects audio files without ever creating them.

Auto-Editor incorrectly expects audio files without ever creating them. This causes a crash without the program ever generating an output.
This only applies when cutting video with more than one audio tracks when exporting to adobe premiere pro.
This doesn't happen with davinci because I never updated the code so look at that for fixes.

Fixed in newest build


fix: crash by iteration on non-existing directory

Pull request
Issue tracker
Fixes will automatically close the related issue
Fixes #
Feature
Describe feature of pull request
Unit test

[x] Done

Manual test

[x] Done

Code Review

Unit and manual test pass
GitHub Action CI pass
At least one contributor reviews and votes
Can be merged clean without conflicts
PR will be merged by rebase upstream base

Additional Info

squashed


Add cropping mode to Reformat node

Add toggle to switch between scaling to fit, and cropping to fit.
Additional controls might be needed to define how to centre cropping, and whether to distort when scaling to fit

Closing this, since Reformat is dead (replaced by Resize), and we have a Crop node now.


SEGSAdmin doesn't set log_generic to appropriate values

Summary
SEGSAdmin generates a new cfg if one does not exist, but it doesn't correctly set the value for log_generic from the template file.
We should set log_generic         = *.debug=true\nqt.*.debug=false so that debug output is properly shown in the console window. Building with Release should disable this output anyway.
It's useful to allow it to be configured, but setting to false by default seems like unexpected behavior and may make troubleshooting with a user unfamiliar with the code base more difficult.

Proposing we either A) show a special input field for log_generic only, or B) simply hide it from SEGSAdmin altogether.

I'd hide log_generic from SEGSAdmin, and add the appropriate value to the generate log file code within SettingsDialog.cpp


Show cluster distances of each patient in one cluster to the centroids of all other clusters

display them optionally or by-default when original cluster distance view is opened?


File hook should be available at Project level

From an administration point of view it would be better if the filehooks plugin was available at the project level in Bit Bucket rather than at the repository level.
When project administrators create new repositories they have to remember to enable the file hook plugin, whereas if it was enabled at project level the settings would be inherited from the project.
There are two plugins in Bit Bucket provided by Atlassian which work at the project level: Verify Committer and Reject Force Push

With later versions of Bitbucket this is supported I had anyways to support this in the next version.

That would be great 👍

Fixed in 3.1.0 which is available now in the Atlassian Marketplace. Please note that the UI is functional but needs some rework which will be done in one of the next versions, see also #36.

Wow that was quick! Great - look forward to using it.


🐞 [Bug]: dashboard disclaimer mentions playground

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?

I find this on the dashboard, it mentions a playground, while that's not a thing anymore
Steps To Reproduce
go to the dashboard dashboard.grid.tf
go to virtual machines
go to full vm
you will find the disclaimer
which network/s did you face the problem on?
Main
version
3.13
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records

Relevant log output
-

Verified:

Devnet
c98400f


Type changed for envelopeResult from getLroMetadata

When calling getLroMetadata for createOrUpdate for the template resource, I got logicResult with right TypeSpec type Product, but envelopResult with a wrong calculated TypeSpec type ProxyResource as follow screenshot.
After some investigation, I found the type is changed after calling getEffectiveModelType here. Since the resource model has a path parameter, so it will enter the logic of finding a naming model which is same to the resource model without this path parameter (filter logic). When it has no exactly same model, it will also find a model that is same with it with the same filter logic, then the final result will fall to the original model. But after the complicated process, the type has totally changed and the naming info lost, it make the emitter hard to distinguish if it is the same resource model. Either fix the getEffectiveModelType logic for templated model or change the logic of getLroMetadata coudl resolve it. I don't know whether it should be opened as a core compiler problem or a getLroMetadata problem, If wrong place, I could move it.

I think this is by design, the finalEnvelopeResult (which matches the finalResult) is the value that *should be used by emitters.

Let me explain more clearly. The main problem here is for the following TypeSpec, when we call getEffectiveModelType, we will get the result type with same structure with Product, but wrong name ProxyResource. I suppose it should return the type of Product.
model Product is ProxyResource<ProductProperties> {
  @doc("Name of product.")
  @pattern("^[\\w][\\w\\s]{1,48}[\\w]$|^\\.default$|^\\.unassigned$")
  @key("productName")
  @path
  @segment("products")
  name: string;
}

Let me explain more clearly. The main problem here is for the following TypeSpec, when we call getEffectiveModelType, we will get the result type with same structure with Product, but wrong name ProxyResource. I suppose it should return the type of Product.
model Product is ProxyResource<ProductProperties> {
  @doc("Name of product.")
  @pattern("^[\\w][\\w\\s]{1,48}[\\w]$|^\\.default$|^\\.unassigned$")
  @key("productName")
  @path
  @segment("products")
  name: string;
}

@timotheeguerin do you think it is a bug for getEffectiveModelType?

This should be handled by the fix here: https://github.com/Azure/typespec-azure/pull/1416
Closing as duplicate


fix: include slug for bookmarked answers

Checklist

[x] tested locally
[ ] added new dependencies
[ ] updated the Swagger API docs
[ ] updated the Postman collection
[x] added a test
[x] linter passes
[x] format check passes

Fixes #172

No problem 🙏🏾
As soon as the Swagger Docs are done, we will merge


u-boot: check if kernel and dtb have been loaded before crc checks

Changelog-entry: u-boot: check if kernel and dtb have been loaded before crc checks

@balena-ci I self-certify!


Add SWE-agent tools as sandbox plugins

Based on sandbox plugins introduced in https://github.com/OpenDevin/OpenDevin/pull/1255, this PR adds command line tools from SWE-Agent as a plugin that can be initialized and available for all agents.
This PR also slightly adds some lightweight dependencies that the SWE agent requires, which is pretty inconvenient and slow (we need to re-run apt-get update and then install) to add post hoc.
Here's a demo by running python3 opendevin/sandbox/docker/ssh_box.py:
Be sure to docker pull ghcr.io/opendevin/sandbox:xw-swe-agent-tool-plugins and use this image for testing.

So we actually had started taking a different approach here: https://github.com/OpenDevin/OpenDevin/pull/846
The idea was to convert all of SWE-agent's commands into Actions. We already have support for most of them, with a little translation.
IMO this will be important for an actual SWE-agent integration into OpenDevin--if we just expose these commands as bash commands, we don't get any structured data about e.g. what files are being edited and how.
I suppose one approach would be to merge this PR as-is, but have the agent implementation eventually intercept open, scroll_up, scroll_down etc and convert them into Actions. WDYT?

@rbren, I totally understand your concern about translating these commands into structured data - however, I kinda don't want to constraint ourselves to those list of SWE-Agent actions, which may grow and become unmanagable at some point.
What we quantitatively verified (across 10+ LLMs) in the CodeAct paper is that:

code (e.g., raw bash command) has the benefit of compositionality (i.e., it gets more things done in one command), say if the agent now what to save all the output of search_dir into one file for future use, they can just run search_dir XXX > note.txt.  And quantitatively, it translates to better final task performances, especially complex ones. Translating everything to structured Action may constraint the agent's flexibility to perform tasks.

The idea of converting every SWE-Agent action into a structured Action and parsing the agent's output is essentially a variant of "Text/JSON as action" discussed in the paper. It can get simple things done, but you must keep adding/expanding the action space when the task gets harder / ask for more. For the same example, you may need to introduce a different Action if you want to save the results of search_dir XXX to a file, which is much more complex than having the agent execute search_dir XXX > note.txt. Based on my experiences, it could be a complete nightmare when you try to parse the model's action ("e.g., intercept open, scroll_up, scroll_down etc and convert them into Actions"), what if the model just echo something like echo "scroll_up is fun?" that shouldn't be executed at all? Your parser eventually will get extremely complex in order to handle most (not all!) of these cases, and this is only for ONE action.

IMO this will be important for an actual SWE-agent integration into OpenDevin--if we just expose these commands as bash commands, we don't get any structured data about e.g. what files are being edited and how.

The assumed benefit of structured Action for tracking file edits might not be valid.
There are thousands of ways the agent could modify / edit files in the workspace. They may accidentally delete some files via rm without using these APIs, run a git pull that completely changes the content of the directory, or apply a git patch to change the context of files. IMO, casting these SWE-agent actions into the Action class is not gonna be a fundamental solution. Rather, I think it'd probably be more efficient if we pursue a git-style approach to monitor the file change (I think there's some Linux primitive (inotify?) that allows you to do that very efficiently).

However, at the end of the day, each agent's implementation can (and should) be completely isolated. I actually prefer that we maintain a very small set of "core" Actions well (e.g., BashRun, PythonRun). And if there's really a need for structured Action, we can starts an "auxiliary action library folder" (or actions/contrib?) that provide those actions by wrapping around core actions like BashRun and/or PythonRun? WDYT?

Some good points here! I'm in the process of reading your paper.
One of my main goals for OpenDevin is to provide users with a transparent window into what the agent is doing. That way users have a sense of control and certainty. If the agent is just running in a black box, the user will worry that it's doing the wrong thing, wasting time, or even doing something harmful. A black box also makes it harder to measure and debug failures.
The best way to provide visibility is to put as much of the agent's behavior as possible into structured data. That doesn't mean everything needs to be structured data though! We definitely need to give the agent the chance to run arbitrary bash/python/etc as it goes about its task.
To put it succinctly: actions should be structured as much as possible without constraining the agent.

I suppose one approach would be to merge this PR as-is, but have the agent implementation eventually intercept open, scroll_up, scroll_down etc and convert them into Actions. WDYT?

This is what I was getting at here--we can give the agent access to all the raw SWE-agent bash commands like scroll_up, but when we notice a really simple, non-composed command, we translate it into a structured Action. We should be able to do this with ~100% precision, so long as we accept imperfect recall.
The Action will have the exact same effect as the bash, so far as the agent can tell. But it lets us provide a lot more feedback to the user as to what the agent was doing.

There are thousands of ways the agent could modify / edit files in the workspace

Totally agree we can't use read/write Actions as a source-of-truth about file changes. Instead, we should only treat read/write Actions as a partial record.

I think it'd probably be more efficient and fundamental if we pursue a git-style approach to monitor the file change

Someone brought up the idea of basically archiving the whole workspace at every step, so you can fast-forward and rewind! Listening to inotify could be helpful as well. But I don't think either is a replacement for having structured Actions

I actually prefer that we maintain a very small set of "core" Actions
Completely agree here. I don't think we'll add more than a few beyond what we have today.

And if there's really a need for structured Action, we can starts an "auxiliary action library folder" (or actions/contrib?) that provide those actions by wrapping around core actions

I could be convinced otherwise, but I don't even want to maintain auxiliary/contrib actions. Agents can create pseudo-actions inside their implementation, and translate them into core Actions. Maybe if we see the same pseudo-actions getting used over and over we should explore some kind of auxiliary angle.

"Someone brought up the idea of basically archiving the whole workspace at every step, so you can fast-forward and rewind!"

I like this one! It is just like the one in Devin's demo video, where you can scroll it forward and backward to check the agent's action.

I could be convinced otherwise, but I don't even want to maintain auxiliary/contrib actions. Agents can create pseudo-actions inside their implementation, and translate them into core Actions.

Exactly! I like this approach. I only mentioned auxillary is because I'm worried we might add too many actions for SWE-Bench eventually..

One of my main goals for OpenDevin is to provide users with a transparent window into what the agent is doing. That way, users have a sense of control and certainty.

If we want the user to feel sense of control and certainty, we can:

As you suggest, we can try to get structured output and highlight what's being changed in the UI.
Add a confirmation mode of execution, the user needs to click a button to confirm the execution of certain actions before they get executed by the agent.
Or we just try to keep track of the entire workspace using git on the backend and make a commit on the backend for every action taken (so that we can fast-forward & rewind). Or i believe docker layer (that only track changes i think?) might also be a good alternative here.

For now, i think the most easy way would be (1) and (2) above. I'm not against getting structured output info for display, as long as that parsing process itself does not constraints/limit the agent in anyway (i.e., it is just for user's information).

The Action will have the exact same effect as the bash, so far as the agent can tell. But it lets us provide a lot more feedback to the user as to what the agent was doing.

So how about we keep this PR as is and let agent have access to the raw bash command, and we starts a new PR and add a field (e.g., OperationType) to the base ExecutableAction that attempts to understand/classify what a generic bash Action is doing by parsing the content of core action (e.g., BashRun) to structured format that can be displayed by the frontend? This is just for user info and has NO way to impact the agent's performance. If agent implementer prefers structured action, they wrap around these core actions in their implementation.
This way, we can get away with only keeping these core Actions without expanding them, yet having ways to get structured info out of them for sense of control.
In the future, it is also possible to add a very fast, cheap smaller LLM specialized in interpreting commands to OperationType that improves the parsing progress (e.g., parse scroll_up bash commands - similar to argument extraction task what i did in this paper). WDYT?

Or we just try to keep track of the entire workspace using git on the backend and make a commit on the backend for every action taken

I did this in an early agent I wrote! It works OK but kind of pollutes the git log. Probably something we could explore though...git is super powerful.

add a field (e.g., OperationType) to the base ExecutableAction that attempts to understand/classify what a generic bash Action is doing by parsing the content of core action (e.g., BashRun) to structured format that can be displayed by the frontend? This is just for user info and has NO way to impact the agent's performance. If agent implementer prefers structured action, they wrap around these core actions in their implementation.

I'm not sure about this--let me think more before we go down this path

In the future, it is also possible to add a very fast, cheap smaller LLM specialized in interpreting commands to OperationType that improves the parsing progress

This is a really neat idea!

This definitely makes for a more powerful agent, so let's get it in, and then we can figure out the observability/structure piece later


Build Prerequisites requires github id and password for i donot reasons

I tried to use the Build Prerequisites but I am not able to install it  and even when i tried fill my github id pass i am getting errors

Fixed by #54.
Wait for a new release or compile from main.

/close


Not python 3 friendly

Hi,
Love the package, I'm keen to try it out for myself.  Just wanted to point out that it doesn't seem to play well with Python 3.  Some errors are easily fixed by running 2to3 on the relevant .py files, but some are not.  For example, try running install_data.py using Anaconda Python 3.  I get the following:
[zamparol@gpu-1-14 Basset]$ python install_data.py -r
[edited for brevity]
Traceback (most recent call last):
  File "/cbio/cllab/nobackup/zamparol/Basset/src/seq_hdf5.py", line 130, in <module>
    main()
  File "/cbio/cllab/nobackup/zamparol/Basset/src/seq_hdf5.py", line 46, in main
    seqs, targets = dna_io.load_data_1hot(fasta_file, targets_file, extend_len=options.extend_length, mean_norm=False, whiten=False, permute=False, sort=False)
  File "/cbio/cllab/nobackup/zamparol/Basset/src/dna_io.py", line 293, in load_data_1hot
    seq_vecs = hash_sequences_1hot(fasta_file, extend_len)
  File "/cbio/cllab/nobackup/zamparol/Basset/src/dna_io.py", line 267, in hash_sequences_1hot
    seq_vecs[header] = dna_one_hot(seq, seq_len)
  File "/cbio/cllab/nobackup/zamparol/Basset/src/dna_io.py", line 137, in dna_one_hot
    seq = seq[seq_trim:seq_trim+seq_len]
TypeError: slice indices must be integers or None or have an __index__ method

The same script seems to succeed using Anaconda Python 2.7.1 (though I can't be sure, the seq_hdf5.py step takes a while to complete).  I'll use that for my purposes, but maybe you should update the readme to explicitly say python 2 is required?

i met the same problem using the docker image, with Python 2.7.6

I guess the error in  line 92 of current seq_hdf5.py while running python 2.7.5  is related
[ca445@cbsugpu01 bassetfiles]$ python /usr/local/Basset-0.1.0/src/seq_hdf5.py -r -c -v 3000 -t 3000 learn_cd4.fa lt.txt learn_cd4.h5
85261 training sequences
3000 test sequences
3000 validation sequences
Traceback (most recent call last):
  File "/usr/local/Basset-0.1.0/src/seq_hdf5.py", line 130, in <module>
    main()
  File "/usr/local/Basset-0.1.0/src/seq_hdf5.py", line 92, in main
    train_seqs, train_targets = seqs[i:i+train_count,:], targets[i:i+train_count,:]
TypeError: slice indices must be integers or None or have an __index__ method


prefix policies and abac support

Prefix policy support and ABAC support

hi @slvishal, can you please close this PR as we are not actually using this fork anymore, we are using directly this one https://github.com/pycasbin/flask-authz


Fix ResourceServer::resourceReaderCallback types.

Resolves #233. I don't know why this passed through Travis.

Before creating a pull request

[x] Document new methods and classes
[x] Format code with make format

Before merging a pull request

[x] Set version target by selecting a milestone on the right side
[x] Summarize this change in CHANGELOG.md
[x] Add unit test(s) for this change

I don't know why this passed through Travis.

Let me take a look at this issue.


Privileged users need to see registration code

An issue came up in 2022 that required looking up a users registration code. Currently this takes postgres queries to achieve. Consider adding a field to the user profile information that renders the registration code for privileged users such as Mods/THO/TwitarrTeam.

The Regcodes page in the Admin panel now has a 'search by user' form where you can input a partial username, get username matches, and from those view all the alt accounts and regcode for a user. Using this requires TwitarrTeam access or higher.


How do i use Azurite Blob Storage using Python ?

i use docker-compose for launch container with app, DB and Azurite:
version: '3.8'

services:
  api:
    # app part here

  db:
    # db part here

  azurite:
    image: mcr.microsoft.com/azure-storage/azurite
    hostname: azurite
    volumes:
      - ./cert/azurite:/azurite_data
    command: "azurite --blobHost <IP_ADDRESS> --blobPort 10000"
    ports:
      - "10000:10000"
    restart: always
    networks:
      - net

now i have a three containers and i'm trying to teach docker container with app on Python3.10 uploads, gives and deletes blob-data in another docker container with Azurite,
how i can do it ? what libraries should i use on the python-app container side for connect to azurite container ?

@4vadim4
You question looks more like a python SDK question instead of Azurite question.
I don't quite understand your scenario for Azurite, from your description.
However, You can just use Azurite like using the Azure storage server.
The connection string for Azurite is different, you can find detail from https://github.com/Azure/Azurite#connection-strings

@4vadim4
I will close this issue as we not get responds from you for long time.
Feel free to contact us again if need any further assistance on Azurite.


Chrome Cookies not Working

Chrome cookies never worked .I am using Cookie Editor extension for importing cookies but session never generated .I am doing wrong or  wrong cookies are generating ?

Please provide more detailed information about bugs, OS versions and browser versions.


rateLimit not working

My rateLimit with Amazon SES is 14 mails/s, but if I set a rateLimit equals to 14 in the ses transport, I get Maximum sending rate exceeded errors. However if I set the rateLimit to 10, I don't have any error.
Do you know if it's a lack of precision or does the problem come from Amazon SES ?

@FabriceGaudin how comes you closed this issue?

@massimocode With this PR: https://github.com/andris9/nodemailer-ses-transport/pull/18, now it's working fine for me

Yep I see how it's being done. I was looking a the source code and was going to say it looks like it could never be a problem, I didn't notice it came from your PR! Thanks!


Comment update: clarify we need sdk in install-dotnet.yml due to asp.net core runtime

For context, please see:

https://github.com/Azure/azure-sdk-tools/pull/5405#discussion_r1105006774

@weshaggard should I remove the build link even with this?

I always manually retain builds I link to.

@weshaggard should I remove the build link even with this?_

I'd say it still isn't worth it. If you want log messages to be maintained I'd suggest copying the part that is interesting into this PR so folks could find it later if needed.

/check-enforcer override


Fix Dermatology typo

Deprecate misspelt method.
Update tests.
Fix data file.

Signed-off-by: kingthorin<EMAIL_ADDRESS>

How are release/change notes generated for this project? Should I be doing something specific to ensure this is included?

based on git history, usually done by github

there is net.datafaker.script.ProvidersDocsGenerator which generates provider docs with version from @since javadoc annotation


Observable.Window with (skip < count) produces no overlap

I've previously used this to produce a sliding window (overlapping windows)
e.g. Window(o, 5, 2)  =>
1 2 3 4 5
3 4 5 6 7
5 6 7 8 9
But now it seems to only have the full count on the first. The other windows all have length "skip"
Could be my misapprehension, but this seems to be a change in behaviour if the count, skip overload
can produce overlapping windows
Here's a simple example (FSharp):
#r "../packages/log4net/lib/net45-full/log4net.dll"
#r "System.Core.dll"
#r "System.dll"
#r "System.Numerics.dll"
#r "../packages/System.Reactive.Core/lib/net45/System.Reactive.Core.dll"
#r "../packages/System.Reactive.Interfaces/lib/net45/System.Reactive.Interfaces.dll"
#r "../packages/System.Reactive.Linq/lib/net45/System.Reactive.Linq.dll"
#r "../packages/System.Reactive.PlatformServices/lib/net45/System.Reactive.PlatformServices.dll"
#r "../packages/FSharp.Control.Reactive/lib/net45/FSharp.Control.Reactive.dll"

open FSharp.Control.Reactive
open System.Reactive.Linq

let t = seq { 1 .. 20 } |> Observable.ofSeq

Observable.Window( t, 5, 2)
|> Observable.map Observable.toArray
|> Observable.concatInner
|> Observable.map (Array.map (sprintf "%d") >> Array.toSeq >> String.concat ", ")
|> Observable.add (printfn "%s")

I get the output:
1, 2, 3, 4, 5
6, 7
8, 9
10, 11
12, 13
14, 15
16, 17
18, 19
20
I'm using System.Reactive 3.1.1 on .NET 4.5

I ported this to C# to try and create a simpler repro:
class Program
{
    static void Main(string[] args)
    {
        var range = Observable.Range(1, 20);
        var window = Observable.Window(range, 5, 2);
        window.Subscribe(items =>
        {
            items.ToArray().Subscribe(Output);
        });

        Console.ReadLine();
    }

    static void Output(int[] array)
    {
        Console.Write("[");
        for (var i = 0; i < array.Length; i++)
        {
            Console.Write(" " + array[i]);
        }
        Console.Write(" ]");
        Console.WriteLine();
    }
}

This seems to be working as expected:
[ 1 2 3 4 5 ]
[ 3 4 5 6 7 ]
[ 5 6 7 8 9 ]
[ 7 8 9 10 11 ]
[ 9 10 11 12 13 ]
[ 11 12 13 14 15 ]
[ 13 14 15 16 17 ]
[ 15 16 17 18 19 ]
[ 17 18 19 20 ]
[ 19 20 ]
[ ]

I also kludged together a different version which also works like the C# version:
let formatWindow(observable: IObservable<int[]>): unit =
    observable
    |> Observable.map (Array.map (sprintf "%d") >> Array.toSeq >> String.concat ", ")
    |> Observable.add (printfn "[ %s ]")

Observable.Window(t, 5, 2)
|> Observable.map Observable.toArray
|> Observable.add formatWindow

Looking at your sample, the only conceptual difference is with how concatInner might be working. I'm avoiding that explicitly, but that also just wraps Observable.Concat(sources), so I'm not sure that's the issue here.

Could be my misapprehension, but this seems to be a change in behaviour if the count, skip overload can produce overlapping windows

Do you have a version of this (including FSharp.Control.Reactive) that worked previously?

Ah, I see the problem. You're quite right about the concatInner being the difference.
I found the earlier example I was thinking of and it turned out I'd used mergeInner
What's happening appears to be that concatInner only subscribes to the inner observables
on completion of the preceding observable, which is understandable. But they run concurrently,
so merge is appropriate. My test case was at fault!
Sorry to cause you work unnecessarily, I should have spotted that.
Thanks for taking a look


Merged PR:
"btf_parser.cpp" fails when the " uint32_t line_info_record_size" is trying to calculate from the start to the end. However, the line_info_record_size is not checked for at least a size of 4 bytes.
In fact bpf_line_info_t has the following information:
bpf_line_info_t
+0x000 insn_off         : Uint4B
+0x004 file_name_off    : Uint4B
+0x008 line_off         : Uint4B
+0x00c line_col         : Uint4B
This means that line_off needs to be at least 4B and this check hasn't been done before. When a smaller size of line_off is passed, the program will crash.
verifier fuzzer is crashing in this line
Follow up: created issue #1790 to address update of documentation.


IpAllocation Policy - Secondary Range Names

Hello everyone,
This week I begin to receive this error:
Error: googleapi: Error 400: Pod secondary range name (public-services) should not be the same as service secondary range name., badRequest when I  recreate a cluster.
I in my plan module add the same name in cluster and service secondary ranges names.
+ ip_allocation_policy {
          + cluster_ipv4_cidr_block       = (known after apply)
          + cluster_secondary_range_name  = "public-services"
          + services_ipv4_cidr_block      = (known after apply)
          + services_secondary_range_name = "public-services"
        }

Look in detail the name is from vpc_network module and isn't possible to create a new subnetwork with a different name on this module.
I believe Google made few changes to the API.

Same errors

I am also having same error.

Add to create another block of ip for pod

Seeing the same issue right now. What did you do exactly to fix the issue? @Jojoooo1

Doing some testing right now, but I changed the name for services_secondary_range_name. This should fix the issue hopefully too. Need to verify the cluster is working properly after it comes up.

secondary_ip_range {
range_name = "public-services"
ip_cidr_range = cidrsubnet(
var.secondary_cidr_block,
var.secondary_cidr_subnetwork_width_delta,
0
)
}
secondary_ip_range {
range_name = "public-cluster"
ip_cidr_range = cidrsubnet(
var.secondary_cidr_block,
var.secondary_cidr_subnetwork_width_delta,
1 * (1 + var.secondary_cidr_subnetwork_spacing)
)
}

@Jojoooo1 Thanks. So you have to make changes to both the vpc-network module and the gke-cluster module?

Nevermind. I was able to finally get it to work by doing what you suggested @Jojoooo1.
Had to patch both modules for it to work properly.
https://github.com/brianpham/terraform-google-gke/commit/f2bd4ce91ba615a50cde366d84aaf27b77477b8e
https://github.com/brianpham/terraform-google-network/commit/10f743327dacb3e4552da288b03f965d3881a835
Another workaround is just commenting out this line if you don't really want to point to your cidr block. GKE will just manage it for you if you don't include it.
https://github.com/gruntwork-io/terraform-google-gke/blob/master/modules/gke-cluster/main.tf#L71

Hi everyone, thank you for raising this issue. Did anybody need to migrate Terraform state after changing/adding the additional secondary_ip_range? I'm mindful of having to publish a migration guide in order to get a quick fix out or finding a way to maintain backwards compatibility. I have a feeling the latter will not be possible in this case as it seems we are conflicting with GCP's API.

@robmorgan Not sure about that. I re-created my cluster since I had to update the cidr blocks.

Released in https://github.com/gruntwork-io/terraform-google-gke/releases/tag/v0.8.0


checks for whether cuSOLVER is enabled aren't needed anymore

This PR removes some checks that were supporting CUDA Toolkit versions older than 8.0.  It seems that older Toolkit versions are no longer supported.
Aside from the checks, the only other change is the second commit which enables a commented-out cusparse.getStream method also requires CUDA Toolkit >=8.0.

How about cupyx/linalg/sparse/solve.py, cupyx/scipy/linalg/decomp_lu.py, and cupyx/scipy/sparse/linalg/solve.py?

Thanks @leofang, looks like my grep missed those the first time around. The only remaining instances I see now are the definition of the constants and one test (test_cusolver_enabled).

pfnCI, test this please.

Jenkins CI test (for commit<PHONE_NUMBER>b7c8dc1124916b634338425db31ec6, target branch master) failed with status FAILURE.

pfnCI, test this please.

Jenkins CI test (for commit<PHONE_NUMBER>b7c8dc1124916b634338425db31ec6, target branch master) succeeded!


impl(GCS+gRPC): alternative routing format for upload id

This is the analog of #11253 for one RPC that needs hand-crafted metadata decoration.

This change is

Added a test, thanks.


Get rid of original events in Aggregate/FunctionalCommandService

It should not be the Aggregate's or FunctionalCommandService's responsibility to expose an exhaustive event history to consumers.  This won't scale.  Either the state captures a synopsis of the required event information, or the event stream is projected to a separate read model fit for the consumer's purpose.
More to the point, the "Original" event array blocked adding support for loading from a cached snapshot + stream tail (see upcoming PRs).
Maintain OriginalVersion instead.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I am not willing to remove the original events collection. There must be another way to do it.
I would recommend opening an issue to discuss the proposed changes before implementing it. There's (usually) a reason for certain things to be in the code base, and taking things out because some use case doesn't fit is not a good idea. There must be a way to find balance between the use cases.

There's (usually) a reason for certain things to be in the code base, and taking things out because some use case doesn't fit is not a good idea.

I understand completely.  I'll see if I can find a way to satisfy both - in my mind, having all events as baggage on aggregate was orthogonal to the whole idea of event sourcing.  I could only see a few tests which made use of them in this repo, so I took too much of a liberty.
BTW, these PRs were meant to be discussion vehicles with running code, not final submissions.

Sure, I understand that it may have merit to use the event history as state in scenarios where typical aggregate history is negligible, or the number of "hot" aggregates is low.
But comparing to "the vast majority of ES systems", and considering scenarios with millions of events per aggregate (which I think is perfectly valid), I think it's wrong to make the event history a compulsory part of the core Aggregate type (or "folded event stream").
Caching, persisted snapshots and incremental read models should ensure that only new events are iterated - reading and iterating (or loading into RAM) the full stream must be considered costly and prohibitive.
I have no problem with the event history being optional in Aggregate: e.g. configurable by the aggregate factory, added via a specialized Aggregate type, or provided via callbacks - anything but compulsory 😃.

Yeah, it makes sense, although I don't think that an aggregate with millions of events is something that could be considered "normal". I've seen cases like that, and it was always something weird with the model.
Regardless, I agree that it should be made optional.

Actually, the issue with OriginalEvents is not existent as it won't be populated with anything more than the events (including snapshots) from the store. So, if snapshots are there and only one snapshot and 10 events are loaded, the collection will contain 11 elements. It won't have more than required.

Sorry - I've have to make progress with my application, and have decided to not use Eventuous - I just can't find the time to achieve the behaviour I need while ensuring backwards compatibility.
Unless told not to, I'll close these PRs soon.

Sure, no problem. As I said, the claim of the PR that having the events collection won't scale isn't exactly right as it should contain only what's physically loaded for restoring state. It should not contain more elements than what's loaded from the database: events, or snapshot + events. So, I believe this PR can be closed.


Integration with Travis CI

html-proofer tool used for *.html files validation. I do have similar checks for my blog and it saves me all the time pointing to potential issues.
Sample build
https://travis-ci.org/vgaidarji/square.github.io
Issues
html-proofer found one issue

Broken link fix
https://squareup.com/directory -> https://squareup.com/pos/customer-directory in eacf331f.

I'll upgrade the ruby in a separate pr etc


ci: skip npm instll -g ... if they are already installed

Codecov Report

Merging #72 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #72   +/-   ##
=======================================
  Coverage   97.89%   97.89%
=======================================
  Files          15       15
  Lines         903      903
  Branches      182      182
=======================================
  Hits          884      884
  Misses         19       19

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 4c1ee63...626e94b. Read the comment docs.


Use "module load wflogger"

Need to set it up.

When release 0.1.0 is ready, do:

[ ] clone release to: /apps/jasmin/supported/wflogger
[ ] create /apps/jasmin/modulefiles/wflogger/0.1 to updated the path so you can just run module load wflogger


Incorrect doc of notifySignatureTimeout() and notifyRedemptionProofTimeout()

In Deposit.sol, as shown here, it is stated that failure to provide a signature or a redemption proof consitutes fraud.
This is inconsistent with the eponymous functions of DepositRedemption.sol, as shown here, which calls startLiquidation(false).

Yeppppp this was on my radar thank you :bow::bow::bow:

I'm doing a mega-branch with a fairly large cleanup of Deposit.sol from an API + docs perspective, captured in #695. Closing this issue in its favor; expect to see this early next week, though you can have a sneak peek at https://github.com/keep-network/tbtc/tree/notification-station .


Fix debug assert in getConstructorDefinedThisAssignmentTypes

Fixes #39704
It turns out the above has been lingering around for a while; this one shows up in the top100 tsserver traces somewhat often, i.e. https://github.com/microsoft/TypeScript/pull/52317#issuecomment-1397828761
The requests all happen in babel's testing artifacts, which have the extension js but are in fact not JS. The one in the repro contains TypeScript code.
It seems like the bug is clear; this loop iterates over declarations to build an array of types, except that sometimes it can continue and skip over a declaration it doesn't like. But then, we pass that (smaller) array of types along with the original declaration array. The two need to be the same size as they are effectively zipped, but the loop explicitly doesn't guarantee that the two are going to be the same length. So, we can just build the list of declarations we actually cared about and then provide that instead.
But, I have absolutely no clue how to actually test this; it takes tsreplay 700 some requests to get the files into the right state for this to appear and I quite know how to reduce that when I can't reproduce it myself in the editor.

Oh, huh. I think #51524 is for this exact bug, which is #51521.

#51524 seems more correct to me.

@typescript-bot pack this

@typescript-bot user test tsserver
@typescript-bot test tsserver top100

Pulled the other test (with with attribution)

@typescript-bot pack this
@typescript-bot user test tsserver
@typescript-bot test tsserver top100

I actually think this PR is not quite right either.
Why are we even trying to go down this path?
It really does feel like the right fix is to fix up the logic in setValueDeclaration - however, that function seems to be a very terse pile of jenga blocks. So maybe it's just a targeted fix in these cases.
You can check out #52323 to see my latest attempt.


Add imageSrcCallback config option.

Adds an imageSrcCallback config option, which accepts a (string) => string callable. This should cover injecting any feasible urlparams.

looks generally good to me.
As I started writing the other nit, I realised it is a bit misnamed, it's not really a callback, maybe a formatter, so imageSourceFormatter?
one slight nit: usually I would prefer to have the identity function as the default, so inside libpannellum you always call imageSourceFormatter. But this is mainly to assert that it should never be null, and always a function type, and pannellum is pretty much a type-free zone, so it's no big deal here :)

vis a vis the type: yeah, I would normally use an identity function too rather than branching, but I think they were using sometimes-null functions elsewhere so mildly leaned in favour of stylistic consistency in the absence of strong typing benefits.


bootique-maven-plugin to simplify maven-shade-plugin setup

It is probably worth creating a special bootique-maven-plugin that hides the complexities of maven-shade-plugin and together with bom config #15  would make importing Bootique much less invasive. Specifically framework mandated parent won't be needed.

closing... moved to a separate project: https://github.com/nhl/bootique-maven/issues/1

The plugin was actually implemented here: https://github.com/bootique-tools/bootique-maven-plugin


Ideal banks sort order

Hi, is it possible to put largets banks first?
ING, rabo and ABN etc ???
(comment from customer who found it strange that Triodos or van Lanschot was the first in the list)

Hi Sean,
This is not possible. There is a feature request to do this inside magento see: #261
Regards,
Rik
Adyen


Add types for typescript

It's always nice to have type information and it would be great to have type information for this awesome package.
I'm thinking to create @types/react-icomoon or maybe even better to just re-write this project (well, single js file) to TypeScript. Then users will know props just from autocompletion for example.
What do you think?

A great idea. Instead of creating a separate dependency for this small component or rewriting this component with typescript, it can be sufficient to create only an index.d.ts file.

any pr for this?

any pr for this?

No for now. If there is no help from the community, I will solve it this week. @davidkhierl

any pr for this?

No for now. If there is no help from the community, I will solve it this week. @davidkhierl

seems the recent publish are not properly typed hence had some problem in my project so I made a pr to fully type the project.
#13

I'm sorry it's so late. TypeScript support is now ready.
v2.3.0


add an unstable task::blocking function

This add an unstable task::spawn_blocking function to unblock people (lol) waiting for #65. I'm not sure if this is the right API, but we've talked about adding spawn_local at some point in the future too, so it doesn't seem too wrong either.
Regardless, I think it's time to provide people with at least "unstable" access to this. Thanks!
Closes #219, closes #65
cc/ @zkat
Screenshot

I still think we should merge this, but I'm almost wondering if the mapping of blocking -> future shouldn't be part of the thread submodule.
There's something about blocking tasks that makes me feel like it should be part of that. Not sure tho; feel like we may need to talk it through sometime.

I see the issue that this function will be effectively stable. The proliferation of reaching for an unstable flag to expose things people are asking for will either force people to choose the unstable flag all the time and not caring about it and taking the ability from us to iterate on the solution.

Having to make some changes after running cargo update is better than having to maintain a fork, that is out of date. And isn't semver supposed to alleviate this? I don't understand all this commitment to find a solution that will be supported indefinitly.

After some reflection I see the problem. Whenever a crate exports a trait the crate can only occur once in the dependency tree.

Yes please! Is this moving forward? :D

@skade We've been talking about this for months now, and we haven't made any tangible progress on an API design. Currently there's quite a few other things in-progress, and several other things marked as priority before we could get around to designing this. Where I estimate it'll probably be at least another month or two before this can be prioritized, which imo is too long.
I think the higher order bit would be to unblock people who have been waiting on this feature for a long time already, so they can move forward and build projects on top of async-std. We can provide detailed upgrade instructions if we decide to stabilize a different version of the API.
cc/ @stjepang I'd be curious for your input here also.

I'm comfortable with stabilizing this function, provided we do two changes:

Rename to something simpler, perhaps task::blocking().
Return a regular task::JoinHandle<T> from the function.

 That means inside blocking tasks we should be able to use task-locals, task::current() and so on.

@stjepang and I had a chat yesterday, and we're on the same page about what to do. I agree with what he's said in https://github.com/async-rs/async-std/pull/251#issuecomment-538843574.

Alright, this PR now exposes task::blocking, and implements #219 as requested. This should be good to review now!

I've addressed all feedback, and things are compiling + tests passing so I think this should be good to merge!


feature: Add support for the HTTPS protocol for stream server

Signed-off-by: Starnop<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Add support for the HTTPS protocol for stream server
Ⅱ. Does this pull request fix one issue?

None.
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it

update the code of  [kubernetes-incubator/cri-tools]

#L137 http->https
#L152 Insecure: true -> CertFile:"client.crt",KeyFile:"client.key",CAFile:"ca.crt".

make crictl & make install-crictl.
setup CA with openssl, create CA-Signed Certificates for server and client.
run pouchd with command pouchd --enable-cri --server-tls --tlskey server.key --tlscert server.crt --tlscacert ca.crt
use cmd crictl exec to verify the results .

Ⅴ. Special notes for reviews

Please help to review this. Thanks a lot. @YaoZengzeng

Please make sure that you have communicated with the apiserver to double check whether and how to use the verified connection. @Starnop

Close it temporarily.