avoid bundling crypto on browser with esbuild

In theory it should be simple to make a package isomorphic such that
const crypto;

if (isNode)
  crypto = async import('crypto');
else
  crypto = async import('crypto').subtle;

However, when bundling with esbuild for our browser bundles it has the unfortunate side effect of evaluating conditional import behind an if that would typically never be evaluated during run time. In order to resolve this we can use the external directive from esbuild in order to never bundle crypto and evaluate it at compile time instead. I can't imagine we would ever want to bundle crypto at all regardless of isomorphism so it seems that adding this directive to our bundle configs should be a net positive.

Closing after discussion with @frankhinek we do in fact have a few limited places where we polyfill crypto for browser bundles. We can revisit another time when all crypto polyfills have been removed


Add Form4 api

Added API that pulls Form 4.

@crperezt can you resolve the conflicts? The changes look fine. I will accept it.


Created deck editor

Context
Some initial version of a deck editor, tightly coupled with the collection view.
Deck Creator #40
Describe Your Changes
Deck editor page added.
Index page updated with a link to decks.
Collection display now features a list of the player's decks (part of #93)
Checklist

[X ] I have performed a self-review of my code
[X ] I ran make check and fixed resulting issues
[ ] I ran the relevant tests and they all pass
[ ] I wrote tests for my new features, or added regression tests for the bug I fixed

Previous PR: #96
Merging this as we discussed.
@eviterin could you add issues for merging collection & editor, and then for connecting the editor to the chain?


Add Client.csproj. Fixes #139

Not merging this right now because the build is sometimes failing on Razor.VSCode-CI-Public even though it always works on Razor.VSCode-CI-Official. The intermittent failure is this error:
npm ERR! enoent ENOENT: no such file or directory, rename 'D:\a\1\s\client\node_modules\.staging\microsoft.aspnetcore.razor.vscode-90813625\node_modules\@types\node' -> 'D:\a\1\s\client\node_modules\.staging\@types\node-ea9b1830'

This is a familiar problem. NPM has a long history of intermittent failures of this kind on Windows. Usually they are fixed by upgrading or downgrading to a different version of NPM. Since Razor.VSCode-CI-Official uses a different version of NPM (6.0.1, where it works) than is used on Razor.VSCode-CI-Public (6.4.1, where it doesn't work), that most likely explains it.
@natemcmaster Do you know if there's a way to configure which version of Node/NPM should be installed in the build environment?

Yes, you can configure the node version with this build task: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/tool/node-js

Thanks @natemcmaster - that looks perfect! The docs there about all the tasks look really useful too.

OK, should work reliably now.


Suggestion to add php doc in the model generation

Description
When generating the model in php, it would be interesting to add documentation on the class, especially to help IDEs to recognize attributes.
/**
 * @package App\Models
 *
 * @property integer $id
 * @property integer $order
 * @property integer $homepage_order
 * @property string $title
 * @property string $tagline
 * @property string $slug
 * @property string $layout_block
 * @property Carbon $published
 * @property Carbon $created_at
 * @property Carbon $updated_at
 */
class Reference extends Model {...}

I know I'm nitpicking on a detail, but I already had fun adding it to the free version of craftable.
Implementation in model.blade.php (for Craftable free version)
{{-- PHP DOC --}}
/**
 * Class {{ $modelBaseName }}
 * {{ '@package' }} App\Models
 *
@if(isset($columns))
@foreach($columns as $column)
 * {{ '@property' }} {{ $column['type'] }} ${{ $column['name'] }}
@endforeach
@endif
 * {{ '@property-read' }} string $resource_url
 */ {{-- END PHP DOC --}}
class {{ $modelBaseName }} extends Model

Hi!
We actually omitted docblock definitions since Craftable PRO no longer uses blade views. Instead, we generate TypeScript definitions which we use in the generated Vue files. We might consider returning them, but even the Laravel codebase is moving away from this type of definition. Alternatively, we might introduce the view model concept, but it's not on our roadmap.
I will keep this thread open for future reference, and we will also talk about this internally to see what are our options.

Hello,
Thank you for your answer.
The new typescript integration is great and is very useful for front-end development 👌
My suggestion was more about the use of php models doc or type inside the backend: api controllers, commands, cron jobs...
It is true that it would be interesting to join the two, but I do not know if it's possible for now..


Reconnected, but not really

We have a livestreamer program inside a robot that streams the video to people who watch in the browser. We log reconnect and disconnect like so:
OnReconnected: func() {
	LOG.Info("reconnected to room")
},
OnReconnecting: func() {
	LOG.Info("reconnecting to room")
},
OnDisconnected: func() {
	LOG.Info("disconnected from room")
	// cleanup and restart connection omitted
},

When the internet connection is interrupted, the client tries to re-connect. I have seen this work well, but sometimes the livekit sdk thinks it successfully reconnected but no media (video & audio) comes through. This happened:
// before this moment everything works beautifully
// at 22:22:39 we have a network interruption.
2023/05/15 22:22:41 "level"=0 "msg"="ICE disconnected"
2023/05/15 22:23:05 "level"=0 "msg"="ICE failed"
22:23:05	INFO	reconnecting to room
2023/05/15 22:23:05 "level"=0 "msg"="resuming connection..." "reconnectCount"=0
2023/05/15 22:23:11 "level"=0 "msg"="received offer for subscriber"
2023/05/15 22:23:11 "level"=0 "msg"="ICE connected"
22:23:11	INFO	reconnected to room
22:23:14	INFO	reconnecting to room
2023/05/15 22:23:14 "level"=0 "msg"="resuming connection..." "reconnectCount"=0
2023/05/15 22:23:14 "level"=0 "msg"="received offer for subscriber"
2023/05/15 22:23:14 "level"=0 "msg"="remote offer restart ice while ice gathering"
22:23:14	INFO	reconnected to room
2023/05/15 22:23:16 "level"=0 "msg"="accept remote restart ice offer after ICE gathering"

// We know from other system logs that from 22:22:45 on the internet was all good again.
2023/05/15 22:23:46 "level"=0 "msg"="ICE failed"
22:23:46	INFO	reconnecting to room
2023/05/15 22:23:46 "level"=0 "msg"="resuming connection..." "reconnectCount"=0
2023/05/15 22:23:46 "level"=0 "msg"="received offer for subscriber"
2023/05/15 22:23:46 "level"=0 "msg"="ICE connected"
22:23:46	INFO	reconnected to room

// Around 22:23 participants report that the image froze.
// At 22:24:53 an engineer confirms the issue and restart the go-program, restoring the stream.

It seems like Livekit thinks it is connected, but it isn't.
These are the screenshots from Livekit dashboard:

Any idea what the problem could be?

Is there any extra logging that we could enable to help find the root cause? I'm happy to add extra logs in production so that when this happens again we better understand the problem.

As a workaround, are there any metrics/hooks in this sdk that we could use to detect this situation? Maybe there is a way to see the number of frames arrived at the participants, and we could compare that to the number of frames sent, to automatically force-restart?

Tonight it happened again. Video was cut for 2 minutes while the logs say that it is connected.

very interesting use case here!
I think this might be an issue with reconnect handling with the Go SDK. It's possible that while the subscriber connection is reconnected, it's not set up to republish the tracks that your robot is publishing. we'll take a look

Reconnect has proven to work much better since this change. Thanks!


bridging: reduce #ifdef USED_IN_CPP_SOURCE in bridging headers

Especially avoid any constructors in #ifdef USED_IN_CPP_SOURCE blocks, because this breaks Windows ARM64.

@swift-ci test

@hjyamauchi Can you please check that this change doesn't break anything on Windows ARM64? It should not, because it should make it even more robust.

@eeckstein Sure thing.
I have been locally testing the Windows ARM64 compiler with this patch: https://github.com/swiftlang/swift/pull/77193 since October 1 or so.
I will test the Windows ARM64 compiler with this change (#77185) and rebase https://github.com/swiftlang/swift/pull/77193 after it and send it for review, if necessary.
Thanks for taking care of this.

@swift-ci please test macOS Platform

@eeckstein My testing of the ARM64 compiler is a bit behind in terms of the git history where I've so far validated up to Oct 2 2024 or so  (I'm working my way up toward tip of tree) and this PR doesn't cleanly apply at that point. It'd need to take me a while to test this patch. Alternatively, you are welcome to merge this now and I will follow up with testing later and a change if necessary.
As you probably understand, the rule of thumb is (1) never put a constructor inside #ifdef _USED_IN_CPP_SOURCE...#endif or (2) match the condition of whether there is at least one user-defined constructor between when _USED_IN_CPP_SOURCE=1 and when _USED_IN_CPP_SOURCE=0 in any of the Bridged C++ class/structs. This PR looks good to me.

Oh, one thing I just noticed: USED_IN_CPP_SOURC is a typo? Probably doesn't matter?
https://github.com/swiftlang/swift/blob/132726c5daea0d7c7a0928a26877f2436b9d37d9/include/swift/Parse/ParseBridging.h#L19
#ifdef USED_IN_CPP_SOURC
#include "swift/Parse/Parser.h"
#else
namespace swift {
class Parser;
}
#endif

USED_IN_CPP_SOURC is a typo?

Yes, thanks! This is not needed at all. I cleaned up the header includes in the bridging files a bit.

@swift-ci test

@swift-ci test

👍


A11 pipeline

Pipeline for Dataset A11: not ready yet because of municipalities are not yet mapped correctly to spatialunit_uids.

Closed because of difficulty for spatial mapping of municipalities


correct films example

continuity bug in documentation, probably due to change in mental model between one paragraph and the next.

I've merged this into the lib feature branch we're currently working on. Thanks for the patch :)


Investigate 403 assaytype error from CloudWatch alarm

[2024-10-30 17:27:53] ERROR in hubmap_translator:898: Exceptions during executing translate()
Traceback (most recent call last):
  File "/usr/src/app/src/./hubmap_translator.py", line 772, in translate
    result = f.result()
  File "/usr/lib64/python3.9/concurrent/futures/_base.py", line 439, in result
    return self.__get_result()
  File "/usr/lib64/python3.9/concurrent/futures/_base.py", line 391, in __get_result
    raise self._exception
  File "/usr/lib64/python3.9/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/app/src/./hubmap_translator.py", line 625, in _exec_reindex_entity_to_index_group_by_id
    self._transform_and_write_entity_to_index_group(entity=entity
  File "/usr/src/app/src/./hubmap_translator.py", line 672, in _transform_and_write_entity_to_index_group
    private_transformed = transformer.transform(json.loads(private_doc),
  File "/usr/src/app/src/./hubmap_translation/addl_index_transformations/portal/__init__.py", line 71, in transform
    add_assay_details(doc_copy, transformation_resources)
  File "/usr/src/app/src/./hubmap_translation/addl_index_transformations/portal/add_assay_details.py", line 135, in add_assay_details
    assay_details = _get_assay_details(doc, transformation_resources)
  File "/usr/src/app/src/./hubmap_translation/addl_index_transformations/portal/add_assay_details.py", line 50, in _get_assay_details
    raise e
  File "/usr/src/app/src/./hubmap_translation/addl_index_transformations/portal/add_assay_details.py", line 42, in _get_assay_details
    response.raise_for_status()
  File "/usr/local/lib/python3.9/site-packages/requests/models.py", line 1024, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 403 Client Error: FORBIDDEN for url: https://ingest.api.hubmapconsortium.org/assaytype/4274c023697e1fd68433f4b8c7a150c0

requests.exceptions.HTTPError: 403 Client Error: FORBIDDEN for url: https://ingest.api.hubmapconsortium.org/assaytype/28adfdab2632a2da190b1ffda55edbce
requests.exceptions.HTTPError: 403 Client Error: FORBIDDEN for url: https://ingest.api.hubmapconsortium.org/assaytype/4274c023697e1fd68433f4b8c7a150c0
requests.exceptions.HTTPError: 403 Client Error: FORBIDDEN for url: https://ingest.api.hubmapconsortium.org/assaytype/b279489f1381b60810d6494a68b829ac
requests.exceptions.HTTPError: 403 Client Error: FORBIDDEN for url: https://ingest.api.hubmapconsortium.org/assaytype/be25590e514b671f5adc79e3abb210a8

John and I tested with our personal tokens, all worked fine.

From CloudWatch logging search, could not find any /reindex/<id> made from outside that match the target. Only found the one I triggered in Postman AFTER the alarm.

Can't reproduce and couldn't figure out based on logging. There's a chance that the token expired.


Build fails probably due to spellchecker

Building latest Ferdi git fails:
make: *** [spellchecker.target.mk:121: Release/obj.target/spellchecker/src/main.o] Fehler 1
make: Verzeichnis „/tmp/yaytmp-1000/ferdi-git/src/ferdi/node_modules/@felixrieseberg/spellchecker/build“ wird verlassen
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/lib/node_modules/node-gyp/lib/build.js:194:23)
gyp ERR! stack     at ChildProcess.emit (events.js:210:5)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:272:12)
gyp ERR! System Linux 5.3.11-arch1-1
gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /tmp/yaytmp-1000/ferdi-git/src/ferdi/node_modules/@felixrieseberg/spellchecker
gyp ERR! node -v v13.1.0
gyp ERR! node-gyp -v v6.0.1
gyp ERR! not ok
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!
Breche ab...
Arch Linux, latest Kernel etc...
Not sure what the problem is...

This looks to be a problem in electron-spellchecker: https://github.com/electron-userland/electron-spellchecker/issues/165

Any easy way to disable spell checker plugin on build to temporarily fix this?

We can temporarily fix by using a hardcoded version electron-spellchecker (the last known working release) instead of letting semantic versioning pick up the faulty version.

If you give me a hint where to look I can try writing a small patch for the arch installer only

I just took a look and it's coming from a dependency of dependency, so we can't fix it the way I mentioned previously. I have mentionned the author of the dependency's dependency in this comment as we can't open issues on the faulty dependency repository.

Not sure what happened there (semver mishap?) but this doesn't seem to be an issue anymore.


Add non watch store

Problem:
For now, we will disable watch function in HPA store when the cluster
version is lower than 1.12. But like GKE, they don't enable HPA v2beta2
api in version > 1.12 yet.
Solution:
Use apiservice to get the available api groups in the cluster.
For HPA, we will detect v2beta2.autoscaling api service to determine
the watch func of hpa store will serve or not.
Related PR:
https://github.com/rancher/types/pull/857
Related issue:
https://github.com/rancher/rancher/issues/20758

Rebased

LGTM

lgtm


feat(UniAndGroup): #919 introduce combinedWithUni

adds syntactic sugar to flatten the result of a combinedWith call

@jponge okay, had to do a force-push, but I merged upstream/main too, because I saw you merged main a minute ago

Replaced by #1362
Thanks @lordvlad!


User Settings UI not aligning properly when switching between servers

The user settings panel gets aligned to the left instead of the right when switching servers / going to dms

I have disabled collapsing of user settings in the plugin config

I found a solution, not sure if it's possible I just modded the css.
class="panels-3wFtMD" > container-YkUktl > + justify-content: space-between

Thanks for the report and the help! Should be fixed in the next release


Gridmet in pipeline

Add gridmet file and associated targets to pipeline.

Thanks! This looks good to me. Should this be merged after the S3 PR? It looks like it's built off of that branch

Yep. We should merge the S3 PR first.


Fails to build

ERROR: _fs_changed: Invalid export preset name: windows. Make sure `export_presets.cfg` is present in the current directory.
   At: editor/editor_node.cpp:753

This error appears for linux, mac, and windows
The export presets are as they should be, and are in the the same directory as project.godot
export_presets.zip
The job, yml is located at the default location
jobs:
  Godot:
    runs-on: ubuntu-latest
    strategy:
      matrix:
        platform: [windows, linux, mac]
    steps:
      - uses: actions/checkout@v2
        with:
          lfs: true
      - name: Build
        id: build
        uses<EMAIL_ADDRESS>        with:
          name: StructureMaker
          preset: ${{ matrix.platform }}
          debugMode: "false"
      - name: Upload Artifact
        uses: actions/upload-artifact@v2
        with:
          name: Client - ${{ matrix.platform }}
          path: ${{ github.workspace }}/${{ steps.build.outputs.build }}

Hi @Jatoxo! :wave:
So you're using v1.4.0. Try updating to v1.4.1 which has a bugfix related to file paths.
If that doesn't fix it, I can give a look into your specific project.
Thanks!

Great, now it works just fine!


[wiki] link to wal-set is down

remove the link?

https://github.com/dylanaraps/bin/blob/master/wal-set on the customisation page


Wiki DNS broken

Hey guys, I think the DNS on your wki sight is broken. DNS is supposed to resolve to https://<IP_ADDRESS> but theres no DNS entry found for roadrunner.dev

DNS is OK, http://whois.domaintools.com/roadrunner.dev Maybe your ISP blocks it?

@wolfy-j You're right. Switched DNS providers and works fine. My bad.


Colors

📑 Description
Approach to custom color management.

tailwind.config.js can be used to introduce new colors like primary.
User creates a config file config.js where he extends Frame colors configuration using what ever combination of color classes and giving it a custom name: info in the below example.
Frame reads config.js and creates a type for the union of predefined colors and newly defined files.
All colors are now available and type safe to components inheriting from Frame

Below see the example - go to /color-test route.
When you open src/routes/color-test/+page.svelte in the editor you'll notice that:
  <!-- typescript claims no error as info is defined as available colors -->
  <Alert color="info" border>Info color</Alert>
  <!-- typescript error as info1 is not a defined color -->
  <Alert color="info1" border>Info color</Alert>

Weak points to solve with config.js

file must exist to avoid import problem
structure can be cumbersome and should be Typescript defined
location must be manage to avoid '../../config.js' in import

Status

[x] Not Completed
[ ] Completed

ℹ Additional Information
BTW, I've noticed we have lib/alert/Alert.svelte and lib/alerts/Alert.svelte. Clear duplication.

Regarding Alert, we keep this one:
// Alerts
export { default as Alert } from './alerts/Alert.svelte';


Reworked python health checking and added tests

Health Checking should now be able to be packaged and has unit tests.
I expect unit tests to fail because there is a dependency on #6437 being pushed to PYPI (Currently we are unable to have grpcio as a dependency).  However, I was able to build against a local repository with the change in #6437 and the unit tests passed.

@soltanmm
running
/workspace/grpc2/src/python/grpcio_health_checking/tmp/bin/python setup.py install on the generated sdist failse with
error: The 'grpcio>=0.13.1' distribution was not found and is required by grpcio-health-checking
(tmp is a fresh virtualenv)
This is more the result I expected given that it failed as a build dependency as well.  I still believe this to be an artifact of the generated egg missing version info, and #6437 is the easy way to fix that, but am open to alternate solutions.

You python setup.py install'd from the sdist...? One is typically not supposed to do that with an sdist. What does pip install <sdist here> have to say?

pip works fine.  I think the issue is with setuptools, easy_install <sdist here> also fails.  I think the issue is with the egg installation.  If we don't want to support that, then I think we are all set.

Also, out of curiosity, do you on your work machine also see a difference between sudo which pip and which pip? I've not yet thought through how that particular indication may affect this but given what I've seen thus far our work machines seem to have broken Python environments by default. Having mused that, I'll take another look soon-ish.

Oh, yes, eggs are deprecated.

If we are not concerned about egg installation, I think this is all set.
I've tested (with a fresh virtual env each time)
python setup.py sdist
pip install <sdist_tgz>
import grpc_health.health.v1.health; import grpc_health.health.v1.health_pb2
python setup.py bdist_wheel --universal
pip install <bdist_whl>
import grpc_health.health.v1.health; import grpc_health.health.v1.health_pb2

Sorry this slipped again. LGTM.


Allow mtu task to configure multiple NICs

What does this PR do?
Allow mtu task to configure mtu settings of multiple NICs.
Do you have any concerns with this PR?
No
How can the reviewer verify this PR?
Run mtu config task with multiple NIC mtu settings.
Any background context you want to provide?
This was introduced by code restructuring and reformatting.
Screenshots or logs (if appropriate)
N/A
Questions:

Have you connected this PR to the issue it resolves?
#116

Does the documentation need an update?
No

Does this add new Python dependencies?
No

Have you added unit or functional tests for this PR?
No, use existing integration testing for mtu config.

Looks ok.
Regards,
Chirag Gupta


[Bug]: Hard to drag a column to resize the width

Contact Details
No response
What happened?
Hard to drag a column to resize the width. I tried and feel that is too hard. Sometimes it doesn‘t work and sometimes it could be slow and not the size I wanted.
What platform were you using?
Desktop
Version of the plugin
3.4.0
Relevant log output
No response
Are you using the latest version of the plugin in your Obsidian vault?

[X] I have verified that I am on the latest version

Are you check if there is a similar issue?

[X] I have verified that there are not similar issues

Resizing columns is working okay for me on Mac, but does not work on Linux.  (Obsidian v1.4.16 on both platforms)


certbot-dns-godaddy==2.9.0

Hey, just updated to the latest docker img and i get the following error
[2/28/2024] [5:46:24 PM] [Global   ] › ⬤  debug     CMD: . /opt/certbot/bin/activate && pip install --no-cache-dir  certbot-dns-godaddy==$(certbot --version | grep -Eo '[0-9](\.[0-9]+)+')  && deactivate
[2/28/2024] [5:46:25 PM] [Certbot  ] › ✖  error     ERROR: Could not find a version that satisfies the requirement certbot-dns-godaddy==2.9.0 (from versions: 0.1.0, 0.1.1, 0.2.0, 0.2.1, 0.2.2, 0.2.3, 0.2.4, 2.6.0, 2.7.4, 2.8.0)
ERROR: No matching distribution found for certbot-dns-godaddy==2.9.0**

I don't see such version in pypi

for anyone else encountering this, for now hardcode the version 2.8.0 in global/certbot-dns-plugins.json
                "name": "GoDaddy",
                "package_name": "certbot-dns-godaddy",
                "version": "~=2.8.0",
                "dependencies": "",
                "credentials": "dns_godaddy_secret =<PHONE_NUMBER>abcdef0123456789abcdef01234567\ndns_godaddy_key = abcdef0123456789abcdef01234567abcdef0123",
                "full_plugin_name": "dns-godaddy"
        },

for anyone else encountering this, for now hardcode the version 2.8.0 in global/certbot-dns-plugins.json
                "name": "GoDaddy",
                "package_name": "certbot-dns-godaddy",
                "version": "~=2.8.0",
                "dependencies": "",
                "credentials": "dns_godaddy_secret =<PHONE_NUMBER>abcdef0123456789abcdef01234567\ndns_godaddy_key = abcdef0123456789abcdef01234567abcdef0123",
                "full_plugin_name": "dns-godaddy"
        },

Yeah, I also have this issue. Although, I'm not sure how to modify or even find global/certbot-dns-plugins.json when it comes to docker.

On my Raspberry Pi 4 (arm64) using image with 2.11.1 tag, I have to change to "version": "~=2.6.0", or I would encounter the Failed to renew certificate npm-2 with error: Error determining zone identifier for __domain__: 404 Client Error: Not Found for url: https://api.godaddy.com/v1/domains/__domain__ error when renewing certificate.

Fixed it by installing a specific certbot version:
pip3 install certbot==2.8
This works, because it sets all dependency plugin versions to 2.8 as declared in global/certbot-dns-plugins.json:
...
        "godaddy": {
                "name": "GoDaddy",
                "package_name": "certbot-dns-godaddy",
                "version": "=={{certbot-version}}",
...
        }

The issue with certbot at version 2.9 is that there is currently no certbot-dns-godaddy v2.8: https://github.com/miigotu/certbot-dns-godaddy

How would one go about getting this fixed in the docker container?

How would one go about getting this fixed in the docker container?

https://stackoverflow.com/questions/30172605/how-do-i-get-into-a-docker-containers-shell#30173220

How would one go about getting this fixed in the docker container?

https://stackoverflow.com/questions/30172605/how-do-i-get-into-a-docker-containers-shell#30173220

I know how to get INSIDE the container and execute things inside of it, but once you stop, reboot, etc, all changes are lost, so how do we make this change permanent?

How would one go about getting this fixed in the docker container?

copy the file from this repo, place it somewhere on your machine, do the change, and mount it in docker
it should be mounted to /app/global/certbot-dns-plugins.json

How would one go about getting this fixed in the docker container?

copy the file from this repo, place it somewhere on your machine, do the change, and mount it in docker
it should be mounted to /app/global/certbot-dns-plugins.json

Thanks, that fixed it for me!

Me too!

I created a pull request to pin the certbos-dns-godaddy version at the latest, v2.8.0.
It introduced some other issues for me, namely an issue with certbot/plugins/dns_common_lexicon.py regarding the recent update about delegated subdomains.
Try this PR at your own risk, but I'd appreciate the feedback if anyone else can test and confirm:
Docker Image for build 1 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-3639

In my initial testing it seems I have fallen onto issues with subdomains with wildcards possibly mentioned by @wolviex.
*.sub.example.com for example will route the API request to sub.example.com instead of example.com on the GoDaddy API. Causing a 404 error.
Unsure if this issue is an upstream certbot change or NPM.

Currently running into this error after downgrading to certbot==2.8.
Error determining zone identifier for <DOMAIN>: 401 Client Error: Unauthorized for url: https://api.godaddy.com/v1/domains/<DOMAIN>.
Ask for help or search for solutions at https://community.letsencrypt.org. See the logfile /tmp/letsencrypt-log/letsencrypt.log or re-run Certbot with -v for more details.

Does anyone have any suggestions? Would the jc21/nginx-proxy-manager:github-pr-3639  image pull fix this?

No, jc21/nginx-proxy-manager:github-pr-3639 would only remove the need for manually downgrading Certbot to 2.8.
I believe your issue is the same as I noted above regarding subdomain delegation.
I believe this was introduced in dns-lexicon 3.17 here: https://github.com/certbot/certbot/pull/9821 and I suspect that certbot-dns-godaddy needs to accommodate for it. I'm looking at contributing to that repo anyways, so there may be some further updates to this issue.
TL;DR The Error determining zone identifier is not an NPM issue
HERE BE DRAGONS
I haven't had a chance to dig in yet, but the workaround for me was to remove/comment out the line:
204: 'delegated': domain, in
/opt/certbot/lib/python3.11/site-packages/certbot/plugins/dns_common_lexicon.py
This is a pain in the ass, and I never recommend changing site-package files.

remove/comment out the line: 204: 'delegated': domain,
I'm not familiar with python, I edited the site-package file & rerun the SSL Certificate creation & got the same Error determining zone identifier.

Do I have to recompile the certbot package @wolviex?

Do I have to recompile the certbot package @wolviex?

You may need to restart the container

Container was restarted & the problem still persists, line of code is commented out as well.

solved by https://github.com/NginxProxyManager/nginx-proxy-manager/releases/tag/v2.11.2


Call double request whenever load page data table

Hi,
I try to run your example : https://vuejs-datatable.patrickstephan.me/demonstrations/usage/ajax.html
When always reload page, the library call 2 request to backend  to get data.
Why library do that? It is not necessary. Please help me solve it.

Duplicate of #32.
I'm trying to solve this issue, but I already have a lot of work. I will @mention you if I did significant advancements on this.

Closed by v2.0.0-alpha.0

Great Work @GerkinDev


Fix typos

Just some small typo fixes!

 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.


Infinite "Setting LED" a5dba60

When I set effect other than "solid" and I turn off leds I get output:
Message arrived [bruh/porch/set] {"state": "ON"}
?ěôţ?Ĺ“ @ěôţ?Ŕ˙?i�ţ?SL @state
Message arrived [bruh/porch/set] {"effect": "lightning", "state": "ON"}
lightninglightning
Message arrived [bruh/porch/set] {"state": "OFF"}
? ˙?SL @state
Setting LEDs:
r: 0, g: 0, b: 0
Setting LEDs:
r: 0, g: 0, b: 0
Setting LEDs:
r: 0, g: 0, b: 0
Setting LEDs:
r: 0, g: 0, b: 0
Setting LEDs:
r: 0, g: 0, b: 0
Setting LEDs:
r: 0, g: 0, b: 0
Setting LEDs:
r: 0, g: 0, b: 0
And eps writes it until I turn on leds again.
Also as you can see, effect name is doubled.

Ah yes, I'm all to familiar with garbage output on the serial monitor. This is pretty common for me. For starters, I changed the baudrate down to 9600 (I couldn't get any output at all on 115200.)
My best guess is that it may be linked to the issue I submitted about changing effect/animation speed. Once I change the effect enough and then toggle the LEDs off and on I start getting that garbage output, followed by the controller dropping the MQTT connection.
I'm not sure if the effect name string displaying twice is an issue. When I watch the state topic the MCU (when it's cooperating) reports the selected effect properly.

It's not even about the garbage, but it still sets LEDs to r: 0, g: 0, b: 0, even if it is already turned off.

Interesting. I'll check into this. So far everything I have setup has been really stable.. or at least I thought so. :P I'm checking into this and will get back to you soon.

The loop is because of this code:
`void showleds() {
if (stateOn) {
FastLED.setBrightness(brightness);  //EXECUTE EFFECT COLOR
FastLED.show();
if (transitionTime > 0 && transitionTime < 130) {  //Sets animation speed based on receieved value
FastLED.delay(1000 / transitionTime);
//delay(10*transitionTime);
}
}
else {
setColor(0, 0, 0);
}
}`
if stateOn == false then setColor....

The double effect comes from void processJson() and void callback().
In processJson:
    if (root.containsKey("effect")) {
      effect = root["effect"];
      effectString = effect;
      Serial.print(effectString);
      twinklecounter = 0; //manage twinklecounter
    }

In void callback:
  ...
  if (!processJson(message)) { **<-- calls function processJson, causes the first effect print**
    return;
  }

  if (stateOn) {

    realRed = map(red, 0, 255, 0, brightness);
    realGreen = map(green, 0, 255, 0, brightness);
    realBlue = map(blue, 0, 255, 0, brightness);
  }
  else {

    realRed = 0;
    realGreen = 0;
    realBlue = 0;
  }
  Serial.print("callback: "); **<-- my debug**
  Serial.println(effect); **<-- second print**
  ...

I've debugged the code and found out why there is a loop when state is OFF and effect is !solid.
The code first checks what to do (the effect). If it is an effect (anything but solid) then it calls the function showleds(). In the showleds() it checks on status. If status is OFF then setColor(0,0,0)... continuous...
When the effect is solid, that part of the code doesn't call function showleds():
  if (startFade && effectString == "solid") {
    // If we don't want to fade, skip it.
    if (transitionTime == 0) {
      setColor(realRed, realGreen, realBlue);

      redVal = realRed;
      grnVal = realGreen;
      bluVal = realBlue;

      startFade = false;
    }
    else {
      loopCount = 0;
      stepR = calculateStep(redVal, realRed);
      stepG = calculateStep(grnVal, realGreen);
      stepB = calculateStep(bluVal, realBlue);

      inFade = true;
    }
  }

  if (inFade) {
    startFade = false;
    unsigned long now = millis();
    if (now - lastLoop > transitionTime) {
      if (loopCount <= 1020) {
        lastLoop = now;

        redVal = calculateVal(stepR, redVal, loopCount);
        grnVal = calculateVal(stepG, grnVal, loopCount);
        bluVal = calculateVal(stepB, bluVal, loopCount);

        if (effectString == "solid") {
          setColor(redVal, grnVal, bluVal); // Write current values to LED pins
        }
        Serial.print("Loop count: ");
        Serial.println(loopCount);
        loopCount++;
      }
      else {
        inFade = false;
      }
    }
  }

Maybe, instead of checking the state in showleds(), there can be an effect called "off"? Then when state is OFF, don't do any effect, go directly to "effect off"...?

Final thoughts ;-) Because the whole code is a continuous loop, is doesn't matter how the code is. When effect is set to solid, it does a setColor() without any Serial.print();. When the effect is !solid, than the showleds() function is called and there is also the setColor() function, only with a Serial.print()... remove this Serial.print(), and it doesn't loop anymore.
In my opinion the best way is to print text is in the payload function, not the continuous loop (only for debugging).

Thanks much for the suggested change, @zuidwijk. My apologies for the issue. Yeah, that bit at the end of the code is a lazy way of making the effect stick after turning off the strip in HA. When you "turn on" the strip in HA, it remembers the last effect it was running and resumes it.
The latest version I just uploaded should resolve this issue. Let me know if it doesn't! Cheers!


Release/v2.3.0 security fixes: chore(fix): fixed springboot security vulnerabilities and added codeql workflow

Why we create this PR?

Added CodeQL workflow to analyze the codebase as a requirement from the release guidelines
Removed veracode workflows from git actions (replaced by CodeQL).

What we want to achieve with this PR?
To meet the trg-8 requirements
What is new?
Added

Added CodeQL code analysis to the workflow to comply with the release guidelines

Deleted

Removed veracode workflows from git actions (replaced by CodeQL).

Security Fixes:

Fixed the following spring security vulnerabilities:

CVE-2024-22234
CVE-2024-22257
CVE-2024-22243
CVE-2024-22259

This PR is closed becasue the branch is renamed. Opened another PR here #293


Chart Points hover problem

On hovering over the old points, chart changes.
Possible solution -
Add react-chartjs-2 wrapper for react and use its component.

Added react-chartjs-2, changed the Bar.jsx in https://github.com/GermaVinsmoke/bmi-calculator/commit/92a91b151e70083a053afb97a2d88105bac82de0


fix use_BAS_dictionaries

variable is not evaluated without $

Thanks, merged!
Note that I don't recommend using the BAS dictionaries anymore

Thank you.
Any reason why you do not recommend them anymore?
I am mainly using it for pathological speech and speech analysis and am not
that deep into all tricks of speech recognition.
Thanks &Cheers
bmilde<EMAIL_ADDRESS>schrieb am Mi. 19. Feb. 2020 um 13:42:

Thanks, merged!
Note that I don't recommend using the BAS dictionaries anymore
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/uhh-lt/kaldi-tuda-de/pull/35?email_source=notifications&email_token=ABRCLOGUHQC23LMFTYIUQJ3RDUSKBA5CNFSM4KHRQB42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMHVHQI#issuecomment-588207041,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABRCLOHAN5IKUN7Y4FHLFITRDUSKBANCNFSM4KHRQB4Q
.


Jewel custom affixes only adding as suffix

In trying to add a jewel which has #% increased Maximum Mana on it, I noticed PoB tried to add it as a suffix even though it is a prefix, and it is listed under the prefixes section. Having tested several other custom affixes for jewels they all add as suffixes. I haven't tested other item types.

PoB v1.4.33
Win10
Let me know if you need more info or there's anything else I can do to help :)

That's not a bug; custom affixes always display at the end, after any other modifiers on the item, just as crafted affixes do in-game (that's why they use a different colour to normal modifiers).


nopparuj model

add Nopparuj's model in Nopparuj's folder

lgtm, i've moved some files along the way.
thanks, merging now.


BoundServiceAccountTokenVolume refresh token

Hi!
I builded the latest dockerfile for papertrail, but the latest kubernetes serviceaccount 1hour expiration token can't refreshed by the image.
How can it be solved?

👍 on this topic, we are now running latest version of fluentd-kubernetes-daemonset (fluentd-kubernetes-daemonset:v1.14-debian-kafka2-1) in EKS 1.21, and we are seeing it is using an old token in Cloudwatch logs (using this method)

@ashie are there any plans for such support?

How about setting /var/run/secrets/kubernetes.io/serviceaccount/token to bearer_token_file of fluent-plugin-kubernetes_metadata_filter?
https://github.com/fabric8io/fluent-plugin-kubernetes_metadata_filter#configuration
As I described at https://github.com/ManageIQ/kubeclient/issues/561, it seems that it's already implemented, but not configured by default (the default value is nil in both kubeclient and fluent-plugin-kubernetes_metadata_filter).

@ashie I can give it a try. But I was expecting fluentd to be already using by default as I am not setting any other configuration to ensure fluentd authenticates in Kubernetes API.

@ashie I have tried with that setting, although it still appears fluentd to be reusing an old token (probably it does not refresh token as expected?).

Sorry, although it's implemented in master branch of kubeclient but not released yet.

@ashie I have tried with that setting, although it still appears fluentd to be reusing an old token (probably it does not refresh token as expected?).

Sorry, although it's implemented in master branch of kubeclient, it's not released yet.

@ashie, Hi,
Do you know when will you released the fix ? thanks in advance.

ref: https://github.com/fabric8io/fluent-plugin-kubernetes_metadata_filter/issues/323

The fix I contributed has been released: https://github.com/fabric8io/fluent-plugin-kubernetes_metadata_filter/issues/323#issuecomment-1165744181
https://github.com/fabric8io/fluent-plugin-kubernetes_metadata_filter/pull/337

@ashie And others, please see my update above.

Thank you for notifying it. I'll wait your report.

@ashie It fully works! Sorry for the alarm!
Testing details are in the PR: https://github.com/fabric8io/fluent-plugin-kubernetes_metadata_filter/pull/337

I see, thanks for your effort!

Now rebuilding images...

Done.

@ashie have you tried this?
I used the tag v1.14.6-debian-kafka2-1.1, but still seems to use a token older than 1h after looking for this in cloudwatch following this procedure.

Please see #1368

Did the underlying kubeclient got updated? is there any way to make it refresh each hour to no longer have the stale-token annotation?
If this is not the case, is there a way to test that the fix works without waiting 90 days?


Updating build flow for multiple ip instantiations

This PR updates the build flow to support multiple IP instantiations that vivado-create-block can then take advantage of
Replaces #57

I can confirm that this PR works on FPGA.


('instruction', '')   # user instructions for select, rawselect, checkbox TypeError: object() takes no parameters

Rasa version: 0.13.7
Python version: 3.6.5
Operating system (windows 8):
Issue:  I am trying to fire this command "python -m rasa_core.train -d domain.yml -s data/stories.md -o models/current/dialogue -c config.yml" I tried many  possibilities but dont know why it is getting wrong.
Please help me
Thnak you.
Content of configuration file (config.yml):
language: "en"

pipeline:
 - name: "nlp_spacy"
   model: "en"
 - name: "tokenizer_spacy"
 - name: "ner_crf"
 - name: "intent_featurizer_spacy"
 - name: "intent_classifier_sklearn"

policies:
  - name: KerasPolicy
    epochs: 100
    max_history: 5
  - name: FallbackPolicy
    fallback_action_name: 'action_default_fallback'
  - name: MemoizationPolicy
    max_history: 5
  - name: FormPolicy

Content of domain file (domain.yml) (if used & relevant):
  intents:
      - greet
      - PF_UAN_Number
      - Update_Bank_Account
      - Update_Mobile_Number
      - Update_PAN
      - Update_Name
      - previous_PF_number
      - reason_No_Salary
      - Update_Date_of_Birth
      - Experience_Relieving_letter
      - Offer_letter
      - Less_salary
      - form_16
      - Permanent_ID_Card
      - Promotion_letter
      - Maternity_Leave_Benefits
      - Leave_Policy
      - paternity_leave
      - Regularize_LOP
      - leave_Child_birth
      - Miscarriage_Medical_termination_of_pregnancy
      - Tubectomy_operation
      - Apply_for_leaves
      - Book_transportation_Drop
      - apply_Mobile_Reimbursement
      - fill_appraisal

  templates:
      utter_greet:
          - 'Hello, how can I help you?'
          - 'Hi, I am here to help.'
      utter_PF_UAN_Number:
          - 'How can I update my PF number?'
      utter_Update_Bank_Account:
          - 'How can I update my bank account?'
      utter_Update_Mobile_Number :
          - 'How can update my mobile number?'
      utter_Update_PAN :
          - 'How can I update my pan number?'
      utter_Update_Name:
          - 'How can I update my name?'
      utter_previous_PF_number :
          - 'How  can I continue with my previous PF number?'
      utter_reason_No_Salary :
          - 'I havent received my salary how could I know the reason?'
      utter_Experience_Relieving_letter:
          - 'How can I get my experience / relieving letter?'
      utter_Offer_letter :
          - 'How can I get my offer letter?'
      utter_Less_salary:
          - 'I have received less salary ,how could this happen?'

  actions:
      - utter_greet
      - action_db

  policies:
      - 'KerasPolicy'
      - 'MemoizationPolicy'

Thanks for raising this issue, @paulaWesselmann  will get back to you about it soon.

Hey @sushilr007, can you copy your full error message please?
Also you don't list the policies in your domain file, maybe that fixes your error.

My issue has been fixed.
There was dependency problem regarding numpy  and prompt-toolkit.
Thank you so much.

My issue has been fixed.
There was dependency problem regarding numpy and prompt-toolkit.
Thank you so much.

hey @sushilr007 I'm also having same problem. which version  of numpy and prompt-toolkit did you install.

@PrakashRaj37
prompt-toolkit    2.0.9
numpy      1.13.0+mkl

mkl

what does it means?

Faced issue 1 apr 2020: resolved by upgrade pip install -U prompt-toolkit to 3.0.5 ver and numpy to 1.18.2, (rasa latest 1.9.3)


How to delete a paused/stopped resource

Hello,
Is there a option for deleting paused or stopped resource through the web interface?
I want to add new resource in the resource list but I want to delete the old one.
Best regards,

Just committed (43590f9e5d9e100748f03fc6a85347cfb9e71bb5) a fix to this that should resolve the issue. You will need to rebuild from source to get the fix. We might push DEB package updates at some point. Working through a new plugin for hashcat is the priority right now.


Unhandled error occurred

After a fresh install at first start up, I received this error:
Exception Name:
FileNotFoundException
Exception Message:
Could not load file or assembly 'CefSharp.Core.dll' or one of its dependencies. The specified module could not be found.
Exception Stacktrace:
at System.Reflection.RuntimeAssembly.nLoadFile(String path, Evidence evidence)
at System.Reflection.Assembly.LoadFile(String path)
at GrocyDesktop.Program.CurrentDomain_AssemblyResolve(Object sender, ResolveEventArgs args)
at System.AppDomain.OnAssemblyResolveEvent(RuntimeAssembly assembly, String assemblyFullName)

This is not really something I can help with.
The mentioned DLL is shipped with the installer, most probably it gets blocked by your anti malware software or something.
Please double check that.
Should be located at (after the first start of grocy) at C:\Users\<USERNAME>\AppData\Roaming\grocy-desktop\runtime-dependencies\<IP_ADDRESS>\cef\x86.
Maybe try to restart by deleting the whole C:\Users\<USERNAME>\AppData\Roaming\grocy-desktop\runtime-dependencies folder.

For what it's worth;
CefSharp has a few dependencies:
Microsoft .NET 4.5.2 or greater
Visual C++ Runtime 2015 or greater. (VC++ 2017 and VC++ 2019 are backwards compatible)

CefSharp has a few dependencies:
Microsoft .NET 4.5.2 or greater
Visual C++ Runtime 2015 or greater. (VC++ 2017 and VC++ 2019 are backwards compatible)

Thanks for mentioning that.
I've also read the CefSharp docs and maybe that's why the installer exactly requires / grocy-desktop has bundled exactly that.

I think I wasn't clear, I had the same error as above and after manually installing the C++ Runtime, the error was gone.

Can't be the case, all the needed dependencies are shipped/included.
Can also be easily reproduced by taking a fresh Windows VM or something, it will work, I always test exactly that after updating CefSharp for grocy-desktop.
In all those cases (not only for grocy-desktop, more generally), it was about that some fancy antimalware tools block or delete some DLLs, however. And however someone uses such fancy tools, I'll also never get behind that...

I've found the issue. There was already an installation of the Runtime, but the x64-version, which obviously doesn't work on an x86-system. Maybe Grocy sees Runtime is installed, but can't/doesn't check if it's the right version for the system?
Anyway, got it fixed, thanks for the very nice piece of software.

x64 and x86 runtimes can of course co-exist on the same system, grocy-desktop is a x86 application and also only bundles Cef x86 - and x86 programs can't even technically use x64 DLLs. So also can't be really the source for this problem...


Merging different shape inputs & predicting labels from multiple output layers

[X] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/keras-team/keras.git --upgrade --no-deps

[X] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

I'm trying to implement a model with functional API where output of Dense Output_0 layer is feed back as input to separate/multiple sub-model like layers to predict labels from each of those different 3 Dense/Softmax layers.
Issue is, I don't know how to combine two variable shape inputs in a way it helps connected sub-model based parts to predict labels corresponding to output from Dense Output_0.

For Example:

Lets assume, Dense Output_0 is [0.9, 0.9, 0.1], so here Hidden 1 & Hidden 2 sub-model parts will be triggered and model predicts labels from Dense Output_1 & Dense Output_2. I can understand that Dense Output_3 will also predict labels but here I want to make those predictions to be really irrelevant for this particular one since it got input 0.1.
I'm sorry if this is something which is already discussed. In that case, I would really appreciate if anyone can point me to relevant documents/issues.

Closing the issue as the above situation can be handle by some engineering efforts on Keras Merge  layers and proper use of Functional API.
:)


ggplot2 output size

Hi Eric,
I was able to follow the instruction and set R up in Zeppelin. However, it seems like most of the ggplot2 output have a fixed size, do you know if that's something that can be adjusted in the notebook?
Thanks

Hi Robert,
The ggplot2 images adjust based on the size of the paragraph. Is this something that can help?

If you want to generate a predefined plot width and height, I guess parameters can be given (see e.g. http://stackoverflow.com/questions/17181051/r-ggplot2-size-of-plot-area).
Maybe you are thinking to parameters in the paragaph magic %r(...), or in the paragraph first line, or as a general interpreter configuration? Can you elaborate?
Thx, Eric

Just pushed ggplot2 image width definition with {"imageWidth": "200px"}

What do you think of this (you can test in the rscala-z branch)?
You also follow this R interpreter as contribution to Apache Zeppelin on https://github.com/apache/incubator-zeppelin/pull/702


Skull ids bug

I was working with the skulls and when I have a skull in my hand and use /ex narrate "<player.item_in_hand>" it will return the proper ids like: i@creeper_skull, i@zombie_skull and for the dragon: i@skull_item,5
Now when I have a script with the event "on player breaks blocks" and then "narrate <context.material.name>" it will always return "skull" whether it was a dragonskull or creeperskull that was destroyed.

Because you used "material.name". Which returns the ... material name...
Everything is operating as expected here.


when will the egui version be updated to 0.25.0?

Some features i would like to use in the new egui version are not supported in here. Are there any workarounds?

See my PR at https://github.com/optozorax/egui-macroquad/pull/39 .

https://github.com/optozorax/egui-macroquad/issues/36#issuecomment-1925458022


How to display class name of detected object

Regarding the tutorial you made on face detection using Detectron2, i want to output the bounding box on detected objects with the class name, instead of displaying the scores only. I would be glad if you could explain how to achieve that.

Hello Musbell, did you found a way to make the class names visible along with output?


Docker Daemon configuration

Hey
Can I configure docker daemon to always forward certain devices automatically by default without usage of --device flag in run cmd.
I want to basically always forward gpu devices to all containers in ec2 gpu instance.
Thx

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@jas99  I think there is no way to do this, you have to add --device flag in run cmd per container

@coolljt0725 It will be nice to have though.

I don't think this is something we'd add as a daemon option, as it's really something specific to a container. Perhaps other maintainers think differently on this.

@thaJeztah If I start gpu instance I want all containers on that host to have gpu capability. ECS service from AWS as of now didn't exposed device forwarding. My query emerged from that. Maybe creating container wise default settings for all options might be something that is use of to docker users.
Thx


Helm chart forces zookeeper hosts to be set even if using druid-kubernetes-extensions

Affected Version
Apache Druid Helm chart 0.3.2
Description
If a user deploying via the Apache Druid helm chart is while using druid-kubernetes-extensions then there is no need to set a zookeeper host however the helm chart will fail if the user does not.

I can help me with the problem so can I help you with that if you assign this issue to me

I can help me with the problem so can I help you with that if you assign this issue to me

Thanks but I created the issue because I already had a PR to solve the problem. https://github.com/apache/druid/pull/13747


Add missing and for propagating DCAP version

Previously the version was not being propagated to the library names.
Also needed to be explicit on all the libraries to install or else newer
ones might get installed.

Current dependencies on/for this PR:

main

PR #5   👈

This comment was auto-generated by Graphite.

Successful run is here https://github.com/mobilecoinfoundation/sgx/actions/runs/5673720430/job/15376043286


Feature: fmt::Write for serial::Tx

Hi,
I'm new to this HAL, rather new to Rust and I still have a lot of confusion, so apologize if this does not belong here or is a stupid question.
I'm using the usart (like in the example) which provides the function Tx.write(byte). But there is no functionality to send a (formatted) string provided.
This can be implemented with something like:
use core::fmt::{self, Write};

struct Format_Tx {
    Tx :  stm32f1xx_hal::serial::Tx<USART1>,
}

impl fmt::Write for Format_Tx {
    fn write_str(&mut self, s: &str) -> fmt::Result {
        for byte in s.bytes() {
            block!(self.Tx.write(byte)).ok();
        }
        Ok(())
    }
}

This enables the write macro, if you create a Format_Tx "relay" struct. For example write!(my_format_tx, "Formatted {} String", 123);
Could  stm32f1xx_hal::serial::Tx  implement this?
Then writing strings to the serial out would be as easy as write!(mytx, "foobar"). I guess this is a common use case.
Regards Jounathaen

@jounathaen Indeed we should have such an implementation. It's probably just an oversight that it doesn't exist yet and there's no blanket implementation.
It shouldn't be necessary to have your proxy structure, you can impl it directly on the Tx struct, something like (copied from stm32f0xx-hal and untested on stm32f1xx-hal):
impl<USART> Write for Tx<USART>
where
    Tx<USART>: embedded_hal::serial::Write<u8>,
{
    fn write_str(&mut self, s: &str) -> Result {
        s.as_bytes()
            .iter()
            .try_for_each(|c| nb::block!(self.write(*c)))
            .map_err(|_| core::fmt::Error)
    }
}

Shouldn't this impl be directly in embedded_hal?

@TeXitoi I think it should be possible to have a blanket impl in embedded_hal.

impl<T> Write for T where T: embedded_hal::serial::Write<u8>

There is no embedded_hal type, thus I think the orphan rule prohibit it (not sure).


Update node-fetch to 2.6.7

Fixes CVE-2022-0235
Packages: ui, dev, appservice, databases

@bwateratmsft Did all 4 in one PR to save you some clicks 😄


Php diff 97

I don't understand why you abandoned #100.
It was almost done. Just needed below change, I think:
    /**
     * Get diff statistics
     *
     * @return array
     */
    public function getDifference(): array
    {
        $return = [
            'inserted'     => 0,
            'deleted'      => 0,
            'notModified'  => 0,
            'replaced'     => 0,
        ];

        foreach ($this->getGroupedOpCodes() as $chunk) {
            foreach ($chunk as [$string, $one, $two, $three, $four]) {
                switch ($string) {
                    case 'delete':
                        $return['deleted'] += $two - $one;
                        break;
                    case 'insert':
                        $return['inserted'] += $four - $three;
                        break;
                    case 'replace':
                        $return['replaced'] += $two - $one;
                        break;
                }
            }
        }

        $return['notModified'] = count($this->old) - $return['replaced'] - $return['deleted'];

        return $return;
    }

Calculating those stats aren't the job of a generator anyway.

I don't understand why you abandoned #100.

Brain fart :D


README Debian needs  libxkbcommon-dev

Description
On debian/Ubuntu libxkbcommon-dev needs to be present to be able to build from source.
Motivation and Context
Help others build from source
How Has This Been Tested?
$ cargo install --root "$HOME/.cargo" --path .
....
error: linking with `cc` failed: exit status: 1
note: /usr/bin/ld: cannot find -lxkbcommon

$ sudo apt install libxkbcommon-dev

$ cargo install --root "$HOME/.cargo" --path .
  Installing gpg-tui v0.7.0 ($HOME/Dokumenter/git-workfolder/gpg-tui)
    Updating crates.io index
   Compiling gpg-tui v0.7.0 $HOME/Dokumenter/git-workfolder/gpg-tui)
    Finished release [optimized] target(s) in 22.16s
  Installing $HOME/.cargo/bin/completions
  Installing $HOME/.cargo/bin/gpg-tui
   Installed package `gpg-tui v0.7.0 ($HOME/Dokumenter/git-workfolder/gpg-tui)` (executables `completions`, `gpg-tui`)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation (no code change)
[ ] Refactor (refactoring production code)
[ ] Other

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.
[ ] I have formatted the code with rustfmt.
[ ] I checked the lints with clippy.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thanks!


"parallel input mismatch" when "from *" added to front of forked/join query

Repro is from a community/zync user of Zui Insiders, so based on timing it is presumed that the repro is with Zed commit 9c0f097 (or something close to that).
The user shared a sophisticated query that's being withheld from this public issue due to its proprietary nature. However, of potential significance is the fact that it included fork and join pieces. The user indicated this query worked as expected in isolation. However, they then attempted to issue the same query across all pools in the Zed lake lake by using the from * in front of the query, which resulted in the following error message:

FYI while the fix is still being worked on, I was hacking around trying to repro with a simpler query and couldn't produce the same error message, but as @nwt explained, what I observed shares a root cause with what the community user experienced.
My attempt was to base a repro on the examples in the Zed join tutorial.
$ zed -version
Version: v1.2.0-49-g9c0f0973

$ curl http://localhost:9867/version
{"version":"v1.2.0-49-g9c0f0973"}

$ zed ls
people.ndjson 2Ej1jPGGEPEJAiwQg7eS8tcyWoz key ts order desc
fruit.ndjson 2Ej1k95ZqocRjE8Qa8RNl2tn92o key ts order desc

$ zed query 'from people.ndjson'
{name:"morgan",age:61,likes:"tart"}
{name:"jessie",age:30,likes:"plain"}
{name:"quinn",age:14,likes:"sweet",note:"many kids enjoy sweets"}
{name:"chris",age:47,likes:"tart"}

$ zed query 'from fruit.ndjson'
{name:"strawberry",color:"red",flavor:"sweet"}
{name:"avocado",color:"green",flavor:"savory"}
{name:"banana",color:"yellow",flavor:"sweet"}
{name:"dates",color:"brown",flavor:"sweet",note:"in season"}
{name:"apple",color:"red",flavor:"tart"}
{name:"figs",color:"brown",flavor:"plain"}

$ cat forked.zed
from * | fork (
  => has(color) | sort flavor
  => has(age) | sort likes
) | inner join on flavor=likes eater:=name

What I observed is that it only works sometimes.
$ while true; do zed query -I forked.zed; echo "just ran the query" ; done
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
{name:"figs",color:"brown",flavor:"plain",eater:"jessie"}
{name:"strawberry",color:"red",flavor:"sweet",eater:"quinn"}
{name:"banana",color:"yellow",flavor:"sweet",eater:"quinn"}
{name:"dates",color:"brown",flavor:"sweet",note:"in season",eater:"quinn"}
{name:"apple",color:"red",flavor:"tart",eater:"morgan"}
{name:"apple",color:"red",flavor:"tart",eater:"chris"}
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
just ran the query
{name:"figs",color:"brown",flavor:"plain",eater:"jessie"}
{name:"strawberry",color:"red",flavor:"sweet",eater:"quinn"}
{name:"banana",color:"yellow",flavor:"sweet",eater:"quinn"}
{name:"dates",color:"brown",flavor:"sweet",note:"in season",eater:"quinn"}
{name:"apple",color:"red",flavor:"tart",eater:"morgan"}
{name:"apple",color:"red",flavor:"tart",eater:"chris"}
just ran the query
just ran the query
...

I won't try to transcribe the explanation for how the from / fork combo's current behavior is causing this. Suffice to say, an effective workaround is to put a sort between the two.
$ cat forked-plus-sort.zed
from * | sort | fork (
  => has(color) | sort flavor
  => has(age) | sort likes
) | inner join on flavor=likes eater:=name

$ while true; do zed query -I forked-plus-sort.zed; echo "just ran the query" ; done
{name:"figs",color:"brown",flavor:"plain",eater:"jessie"}
{name:"banana",color:"yellow",flavor:"sweet",eater:"quinn"}
{name:"dates",color:"brown",flavor:"sweet",note:"in season",eater:"quinn"}
{name:"strawberry",color:"red",flavor:"sweet",eater:"quinn"}
{name:"apple",color:"red",flavor:"tart",eater:"chris"}
{name:"apple",color:"red",flavor:"tart",eater:"morgan"}
just ran the query
{name:"figs",color:"brown",flavor:"plain",eater:"jessie"}
{name:"banana",color:"yellow",flavor:"sweet",eater:"quinn"}
{name:"dates",color:"brown",flavor:"sweet",note:"in season",eater:"quinn"}
{name:"strawberry",color:"red",flavor:"sweet",eater:"quinn"}
{name:"apple",color:"red",flavor:"tart",eater:"chris"}
{name:"apple",color:"red",flavor:"tart",eater:"morgan"}
just ran the query
{name:"figs",color:"brown",flavor:"plain",eater:"jessie"}
{name:"banana",color:"yellow",flavor:"sweet",eater:"quinn"}
{name:"dates",color:"brown",flavor:"sweet",note:"in season",eater:"quinn"}
{name:"strawberry",color:"red",flavor:"sweet",eater:"quinn"}
{name:"apple",color:"red",flavor:"tart",eater:"chris"}
{name:"apple",color:"red",flavor:"tart",eater:"morgan"}
just ran the query
...

Verified in Zed commit 3d32d97.
Circling back to repro, I've since recognized why I wasn't able to reproduce the same error message that the community user reported. The missing ingredient was to add yet another pool beyond the ones I wanted to join. So revisiting the repro with 9c0f097, now I've got a third pool which contains the sample.zng data from the Brim repo.
$ zed -version
Version: v1.2.0-49-g9c0f0973

$ curl http://localhost:9867/version
{"version":"v1.2.0-49-g9c0f0973"}

$ zed ls
people.ndjson 2F0LUwg94Nt0w35X7pEO2JHnT3n key ts order desc
fruit.ndjson 2F0LVYBdPfP2La9LceJF5eMTKgg key ts order desc
sample.zng 2F0LVwivM0HIs3FIZEQc6TJTjcd key ts order desc

$ zed query -I forked.zed
status code 500: parallel input mismatch: 3 parents with 2 flowgraph paths

Now at Zed commit 3d32d97 that has the fixes from #4087:
$ zed -version
Version: v1.2.0-59-g3d32d974

$ curl http://localhost:9867/version
{"version":"v1.2.0-59-g3d32d974"}

$ zed ls
sample.zng 2F0LqAnYiFjhRFcrvY8KbQiMuqM key ts order desc
people.ndjson 2F0LpEq8tcQsXuDFvTL3U1syhxg key ts order desc
fruit.ndjson 2F0Lpj7MIpKOOaOmOT8YZ1FzXWl key ts order desc

$ zed query -I forked.zed
{name:"figs",color:"brown",flavor:"plain",eater:"jessie"}
{name:"strawberry",color:"red",flavor:"sweet",eater:"quinn"}
{name:"banana",color:"yellow",flavor:"sweet",eater:"quinn"}
{name:"dates",color:"brown",flavor:"sweet",note:"in season",eater:"quinn"}
{name:"apple",color:"red",flavor:"tart",eater:"morgan"}
{name:"apple",color:"red",flavor:"tart",eater:"chris"}

The fixed Zed is also present in the latest Zui Insiders build 74, so the original community user should be able to move past this issue now.

Thanks @nwt!


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
17 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
20 seconds
deprecated-base-image-check

✅ Succeeded
20 seconds
sanity-inspect-image

✅ Succeeded
58 seconds
clamav-scan

✅ Succeeded
16 seconds
clair-scan

✅ Succeeded
11 seconds
sbom-json-check

✅ Succeeded
12 seconds
sanity-label-check

✅ Succeeded
11 seconds
sanity-optional-label-check

✅ Succeeded
5 seconds
show-summary


Prerender run very slow

hi there
i have one website now i want share in facebook but prerender working very slow so facebook can't read meta in my website . let me know why ?  my website load very fast
my website is http://hiquiz.net

Hi there,
I just responded to your email. Let's continue the conversation there!
Best,


Update main to prepare for automated codeflow

Remove all the extra items from the versions files that shouldn't be needed.
Update runtime to the latest 9.0 version
Update maui to the 8.0 shipping maui versions
Add Aspire
Add the runtime branch dependency so we can try adding darc flow

@MiYanni @Forgind I got a successful build with my branch here that appears to have the right content. Any concerns with merging? https://dev.azure.com/dnceng/internal/_build/results?buildId=2351940&view=results
Any concerns with me pushing a release/8.0.2xx branch? I have a build from that as well.
https://dev.azure.com/dnceng/internal/_build/results?buildId=2351951&view=results

@marcpopMSFT :shipit:


Literate Elm?

I couldn't find anything about literate programming in Elm, but I think it would be a good candidate for it. Is anything like this possible or in the works? I am more than happy to have a go at implementing something that would parse literate elm programs if there is call for it and we can agree on some sensible syntax for it.
As a super-quick intro to those not familiar with the concept, literate programming is essentially flipping the concept of comments vs. code in the source file: text that is not marked up in any way is considered documentation, and code is specifically marked up to indicate it is code. The whole source file can still compile in the same way that a normal code file would.
For example, what if the compiler could handle markdown-style documents, compiling all the code-annotated parts into the program? This example supposes the compiler would treat anything indented 4 spaces as code, and anything "fenced" with triple-backticks (```) as code.
mathemator.elmd
# Mathemator

This program takes mathematics to the next level!!! Why just add
two things when you can add infinity things?!!

## Addition

**additionator** adds as many things as you like together! Wow!
*Note: numericals sounds fancier, right?*

    additionator : List Float -> Float
    additionator numericals = effin' (+) 0 numericals

## Subtraction

**subtractionator** is like **additionator**, but a lot more negative.
It pretty much just brings everybody down. Probably just don't invite it.

    subtractionator : List Float -> Float
    subtractionator victims = effin' (-) 0 victims

## Multiplication

***Caution: this can make some really big numbers!*** Make sure you
have enough room to store all your numbers - maybe a *warehouse* or something.

    multiplicatinator : List Float -> Float
    multiplicatinator dimensions = effin' (*) 1 dimensions

## Utilities

This takes the core logic of the mathemator funcitions so that it is not repeated.

I could have written it with folds and stuff, but I love typing and I'm not
some kind of Einstein math robot, gosh, get off my back man!
I worked in a bit of a *nautical* theme here to help with understanding, since
there's all this *Float*ing going on.

```
effin' : (Float -> Float -> Float) -> Float -> List Float -> Float
effin' swab yarrrrr plank =
  case plank of
    [] -> yarrrrr
    ye :: deck -> effin' swab yarrrrr deck |> swab ye
```

:+1:

There are several possible formatting styles:

markdown (plain or github-flavoured)
"bird" style: prefix code lines with >
latex stye. I have not learned any latex, but it features in Literate programming | haskell wiki
etc.

There is also the question of whether to include a macro system. My gut feeling is that it would not add much, since Elm is already fine with definitions in whatever order. It may not add much of use.
A few other features I would like in a literate elm compiler:

[ ] can put text between any code lines, even if it would split up an expression.

I start an if-expression:
    if wantToGo
      then "sure"
I don't really want to give a clear "no" here. Keep 'em guessing.
      else "maybe... maybe"

[ ] all error messages use the correct line and column numbers. It would be easy (depending on formatting style) to just strip out the non-code lines and feed them into the normal elm compiler, possibly sacrificing the proper context information.

[ ] ability to include non-elm code blocks, which are treated as documentation.
 So I write Elm with the 'fence' syntax and no language specified:

 ```
 getName : { name : Maybe String } -> String
 getName { name } = Maybe.withDefault "Anonymous" name
 ```

 or with elm specified...

 ```elm
 type alias Person = { name : Maybe String }
 ```

 ... although that may be better to use for showing elm code that is not evaluated

 Then I can write some JavaScript that the compiler will just ignore.
 Mainly handy for writing docs that compare elm with other languages,
 I would guess.

 ```javascript
 function getName (person) {
   return person.name ? person.name : 'Anonymous'
 }
 ```

 Another thought, for JavaScript specifically, is to possibly specify some
 of it to include in the compiled .js file, but I doubt this is something
 that anyone would really want to encourage.

What is your goal here?
I have done some literate programming in Haskell and was not very into it. It ended up making my blog posts much harder to read because I kept having to show code that didn't really matter to getting an understanding of the key points. So I ended up with stuff that I didn't really work as a blog or as code.
Do you have a usage scenario that'd motivate further work on this? From there I can say some stuff about prioritization.

@evancz my main motivation for suggesting this is just that it seems like a fun idea that might be fun to try implementing, so not much beyond a possible weekend project at the moment.
I can imagine a few scenarios where literate syntax might be a bit nicer to work with, but I have no pressing or immediate need:

small modules that have README.md with all the code and docs, and  elm-package.json, and nothing else. The imagined benefit would just be reducing the chance of the information in the readme getting out of sync with the code.
applications in teaching, where there might be a lot more text than code. It may be marginally easier for an educator to write, but whether there is any substantial value will depend on whether students find it easier to get started. I am thinking here particularly of younger students - primary school level. This would probably need a thesis to investigate before I could really say whether there would be any real benefit.

Okay, makes sense. I think there's a pretty low chance that this'd make it into the compiler for a few reasons:

Supporting an alternate parser is a big deal. I don't think there is adequate justification for that maintenance burden right now.
I prefer having one good way rather than two competing ways with different strengths. In this case, my experience would lead me to believe that students would do better with normal doc comments. That way there is no "new syntax" to learn when switching between literate and illiterate code. I have observed that even tiny changes can be pretty confusing for new people, and that's from total beginners to folks with programming jobs.

That said, I think it's possible to do this outside the compiler which would cool. If you have a pre-processor that strips out all the literate parts and removes whatever the "code line starter" character is, you could just feed that to Elm. You can leave all the vertical indentation the same so the line numbers would work out. This'd mean you'd get normal Elm errors that are correctly aligned with row and column.
So that'd be my recommendation. Now that I say it, maybe that'd be my recommendation even if I wanted to fold it in officially ;)

maybe that'd be my recommendation even if I wanted to fold it in officially. It is certainly a better route to that because it is zero risk for the compiler

I agree, as long as it is getting the positions right in error messages it is certainly good enough for me. Also sounds pretty straightforward, so I may have time to implement it soon.

@davidmason Hey has any work been done around it? I am really impressed by this idea.

@jigargosar No work so far. As you can guess from the delay in my response, I have had a lot on (daughter born in December and had a sick relative who passed away earlier this year).
Anyone could have a go at implementing this. It would really just be a script that reads in a literate code file, replaces lines that are not Elm code with empty lines, then feeds it to the elm compiler. It could be as simple as:

lines with ``` or ```elm start a block of code

lines with ``` end the current code block

in a code block, all lines are kept
outside a code block, lines with 4+ spaces at the start are kept, lines without are stripped

@davidmason I appreciate your response. Specially after all you have gone through.
I was just wondering if you had made any progress, if so I would love to see it.
I understand that this might be easier to implement. But, from my past experience, every time I have started to work on a weekend project, it tends to expand itself in a bigger scope then initially estimated. Case in point jigargosar/elm-simple-gtd ;)
Once again, really appreciate your response.

I think it would be fun to make, so I will keep it in mind. I am slowly catching up on everything, so I may make some time some month soon to have a crack at it.
If I can use a library to get an AST, that will avoid the incidental complexity of trying to do the initial parsing myself. From an AST with line/column numbers it should be straightforward.
This looks like it would do: https://github.com/textlint/textlint/tree/master/packages/markdown-to-ast
Code is:
"type": "CodeBlock",
"lang": "elm", // or null


kubeadm init fail with kubernet service failed to start because CRI not implemented

(ubuntu1604, kubernets 1.26, containerd 1.26)"command failed" err="failed to run Kubelet: validate service connection: CRI v1 runtime API is not implemented for endpoint "unix:///var/run/containerd/containerd.sock": rpc error: code = Unimplemented desc = unknown service runtime.v1.RuntimeService"

Hi @dantanqikun, can you tell me the version of containerd installed.
/priority awaiting-more-evidence

your version of containerd is too old.
go to the "container runtime" page of k8s.io and install the latest containerd.
for more support questions please use discuss.k8s.io or slack.
/kind support
/close

Find the new version solved the problem, thx guys.


delete old metrics in puma collector

refs #69 #34

@SamSaffron  can't figure out why travis fails under 2.6 ruby , seems some technical problem

Thanks, I adapted this in e2d75a5aa84bc71e8bfcf07d8a406e1145ed9958 and credited you as a co-author.


[WIP][CARBONDATA-3914] Fixed exception on reading data from carbon-hive empty table.

Why is this PR needed?
Reading data from empty carbontable through hive beeline was giving 'Unable read Carbon Schema' exception, when no data is present in the carbon table.
What changes were proposed in this PR?
Return the empty data if no data is present.
Does this PR introduce any user interface change?

No
Yes. (please explain the change and update document)

Is any new testcase added?

Yes

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/4121/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/2380/

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/2382/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/4123/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/4124/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/2383/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/2386/

LGTM

spark 3.4 one test failure is a random issue I presume as these changes are unrelated and build passed before adding comments. We need to track this and fix

retest this please


mac-rel-css is taking >1h30 on macpro1

See http://build.servo.org/builders/mac-rel-wpt/builds/2266 for example

macpro1 is now out of rotation.


JS: Example test case isn't fully checked when updating the test framework

Steps to reproduce:

Find an outdated JS kata that uses Node v8 or v10 with the deprecated Test framework.
Update the assertion in the Test Cases (replace them with Chai's assertions), but leave the Sample Test unchanged.
Update the Node version to Node v12
Republish

At this point, it should fail to republish because the sample tests hasn't been updated, but no. The republish will succeed and the sample test will stay that way.

Passing the Example tests is not necessary ( I have intentionally designed at least one kata that way ).
I understand the problem, but the scenarios seem mutually exclusive.


OpenAI API Error: Request failed with status code 429 - Too Many Requests

After installing and running aicommits a few times in a subdirectory, the command returns:
▲ No staged changes found. Make sure there are changes and run `git add .`

However, I did change directory to the repository root and ran aicommits it returned:
▲ Generating your AI commit message...

▲ OpenAI API Error: Request failed with status code 429 - Too Many Requests

How is this possible? I never used the OpenAI API before, only the chat version. Does the aicommits command also sent a request to the OpenAI API if there is nothing staged? It would help a lot if there was some kind of verbosity flag to get a better understanding of what is happening.
Now aicommits always runs, no matter what you type behind it. This most likely caused me to almost brute force the OpenAI API.
Besides the problem I encountered this issue also proposes 3 feature requests:

[ ] Add the ability to run aicommits anywhere in the repository (like git status).
[ ] Add the some kind of verbosity flag.
[ ] Change the ease of executing the main command, e.g. error on unknown input (like aicommits --help).

Look this https://platform.openai.com/docs/guides/error-codes/api-errors

@luukvhoudt Due to a lack of attention, I did not configure a payment method in OpenAI and for this reason I was receiving the error 429, I had no "balance". After defining a payment method, the CLI started to work perfectly.

Thank you @ribeiroevandro! That's correct, you likely need to add a payment method to OpenAI.
@luukvhoudt We're planning on adding a --help command. Also, what do you mean by the ability to add aicommits anywhere in the repo?

Closing this for now since it was answered above

@Nutlope here are some steps to reproduce what I encountered:

Create a git repo
Create a file and a dir
Change the file contents
Cd to the dir
Run aicommits
> No changes are detected.
Run git status
> file in repo root is detected as a change.

The feature request was to have aicommits work the similar to git status for this func in particular.


ARROW-17962: [Java] Remove unused schema creation from try with resources

CI still seems to fail here.

@lidavidm Thanks for the offer. IDK if this is related or not, but there was a bug in the production code where a VSR was created for exporting, but was not closed. That is fixed here. If you could unstick the workflows when you have time that would be great.

@lidavidm Thanks for the offer. IDK if this is related or not, but there was a bug in the production code where a VSR was created for exporting, but was not closed. That is fixed here. If you could unstick the workflows when you have time that would be great.

Also, I passed the ArrowSchema into the import method. Strictly speaking, the ArrowSchema is optional, but it might be the case in this round-trip test that if you pass one in one direction you should pass one in the other

Looks like it still fails, but it's just due to Checkstyle
[WARN] /arrow/java/c/src/main/java/org/apache/arrow/c/Data.java:170: 'try' child has incorrect indentation level 4, expected level should be 6. [Indentation]

@github-actions submit java

@github-actions crossbow submit java


Lovelace card overhanging

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Card is overlapping with others in view.
Expected behaviour
Card should be flush with those next to it.
Steps To Reproduce
No response
Card Version
6.1.2
Home Assistant Version
2021.11.5
Configuration
No response
Relevant log output
No response
Code of Conduct

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

Hi, I'm not sure what is causing that to happen. Possibly some of the div's that are required for the card to work correctly.
If you are able to pinpoint what is causing that I can look at fixing it. However I don't have the same theme you are using so not sure I would find it unless I can get that theme somewhere

@Andrewpaps  please could you test with the latest version, this may now be resolved by #552

Closing this out as there has been no update, let me know if that didn't resolve the issue

Neither the Home Assistant update nor any Integration update have resolved the issue unfortunately.

You can fix it by setting the top and bottom padding to 0px of ha-card.
card_mod:
  style: |
    ha-card {
      padding-top: 0px !important;
      padding-bottom: 0px !important;
    }

You can fix it by setting the top and bottom padding to 0px of ha-card.
card_mod:
  style: |
    ha-card {
      padding-top: 0px !important;
      padding-bottom: 0px !important;
    }

Already tried it, doesn't fix the issue as card_mod only applies to padding within the card.
As you can see, for some reason the extra padding is happening exterior to the card but is still definitely being caused by the atomic calendar card itself.

@Andrewpaps ... I am experiencing the same issue. Did you ever get it resolved?


remove Storage Test Namespace from parent namespace

In issue https://github.com/Azure/azure-sdk-for-net/issues/15853 we saw that our tests were failing to catch when we accidentally set a constructor to internal, preventing users from being able to create instances of public class objects.
Our test namespace are included in the parent namespace, which allows the test namespace to call properties, constructors and objects that are internal.
This could be a bug and not a bug. Because we could need these access these internal objects for our tests but we don't want to expose them publicly. At the same time we do want to be able to write tests/code that users would be writing as well.
Open to any ideas or help on how to do this for our existing tests. Or if we should write separate tests in a namespace outside of the existing tests that have access to call our internally set objects.

In issue #15853 we saw that our tests were failing to catch when we accidentally set a constructor to internal, preventing users from being able to create instances of public class objects.

Was the constructor actually changed from public to internal or was it always internal? ApiCompat should catch an API break like this.

Our test namespace are included in the parent namespace, which allows the test namespace to call properties, constructors and objects that are internal.

The reason that that internal methods can be called is because InternalsVisibleTo is enabled for the project - https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/storage/Azure.Storage.Files.Shares/src/AssemblyInfo.cs#L7


Merge Zally and Zally-Web-UI Repositories

As Zally maintainer
I want to have the source code of all components in one place
So that we can implement richer integration tests
And we can provide easier and better getting started guides and scripts
Acceptance Criteria

Zally Web UI source code is moved to this repository.
Zally Web UI links to the merged repository
Travis configuration is adjusted and works
Root README.md describes the project and all its components

As an intermediate step we can include webui as a submodule.
Also we need to check if there is any way to keep the Git history of webui project. I will take a look on it.

@maxim-tschumak , I've created a merge prototype in #447 .
We still need to think how to transfer issues and how to deprecate the old repo.


Upgrade aws sdk from v1 to v2 in repository-s3 plugin

This is an enhancement to upgrade AWS SDK from v1 to v2. This will be the first step in upgrading to v2 which will responsible solely to migrate existing v1 APIs to v2 sync APIs. Future iterations (outside of the scope of this feature) will involve exposing async counterparts of existing plugin APIs which will be based on non-blocking IO.

Taking this up

Closing as a dup of https://github.com/opensearch-project/OpenSearch/issues/7312, doing it in a combination in https://github.com/opensearch-project/OpenSearch/pull/7372.


use provided yapf interface

The checker was not using the yapf format checker provided to users.
Forcing the CI to use the checker provided to users increases consistency
with testing.

Huh, looks like changing to v1 actions was the solution @ncrubin - see #79.


First login really slow dashboard load time

Hello,
if you do a first login after a while (don't know the exact time),
the load times for dashboard are excessive, even more than 30 sec, 1 minute and so on,
even on private browsing (no browser plugins etc.).
While If I sign out and then sign in again, it takes 3-5 seconds to load the dashboard.
So, is it a cache issue? How you exactly manage the cache, in order to avoid this issue?
thanks for your help

Hi,

How you exactly manage the cache, in order to avoid this issue?

For the initial Dashboard load only a cache provided by Webmin is used.

Is it a cache issue?

Yes.

Hi,

How you exactly manage the cache, in order to avoid this issue?

For the initial Dashboard load only a cache provided by Webmin is used.

Is it a cache issue?

Yes.

ok, and is it possible to set in some way? Where's this Webmin cache stored and set? Thanks

Webmin ⇾ Webmin Configuration: Background Status Collection:

Ok, so, put the case I want a long life cache, one week: I could set 10080 minutes and then, If I need, I would do a manual refresh on the dashboard? Is it working that way? Thanks

Was Webmin running all that time you weren't there?

Was Webmin running all that time you weren't there?

Isn't Wembin service always running? I'm not sure to understand what you mean...

I'm not sure to understand what you mean...

What is the main goal of asking at the first place then? What are you trying to achieve? If you want to report any bug --  just do it.
I am busy right now re-working the theme.

That's what I wrote at first. To avoid waiting so long the first time you login in a week (example).

Do you expect me to know what is happening on your system, without knowing myriads of underlying details? :slightly_smiling_face:
Anything could happen alongside, and it well could be a completely unrelated problem. For instance, there was other heavy process running in background, at the same time while you were visiting the Dashboard, which created an impression that it's a theme or Webmin issue, when it's actually not.
If you have  exact details or steps to reproduce an issue, I'd be interested.

At first you wrote that's a cache issue. And is a cache issue for sure because it does the same over time.
That's why I asked:
Ok, so, put the case I want a long life cache, one week: I could set 10080 minutes and then, If I need, I would do a manual refresh on the dashboard? Is it working that way? Thanks
but, instead of answering the questions, you did another question. Where do I come in this? Honestly....
thanks 👍

Where do I come in this

Where are you actually coming from any why bother at all? :slightly_smiling_face:

but, instead of answering the questions, you did another question.

I cannot provide an answer (particularly exactly) if I don't even know what the real question is, and what problem is being solved.
It may look as a clear and easy to answer question to the one who asks it (like this one) but an answer, most of the time, depends on so many abandoned details.

thanks :+1:

Not at all.

Put the case I want a long life cache, one week: I could set 10080 minutes and then, If I need, I would do a manual refresh on the dashboard? Is it working that way?

Yes, manual refresh on the dashboard (including side slider) does force background recollection.
Now, I'd like to ask though - did you see a tooltip text or a delay is too long and you missed it?

Ok, there's a bug. I can't set any number of minutes greater than 60

I will report to Webmin, thanks.

Isn't cache period for more than 60 minutes disregard the idea of caching? Perhaps better to disable collection at all and perform on demand only?

Isn't cache period for more than 60 minutes disregard the idea of caching? Perhaps better to disable collection at all and perform on demand only?

nevermind, I just changed the homepage to show a fast module instead of the system information. Thanks

Were you waiting untill the dashboard load was finished? It could be possible to click any other link before the load was finished to terminate it, right?


TST: tests for GH4862, GH7401, GH7403, GH7405

closes https://github.com/pydata/pandas/issues/4862
closes https://github.com/pydata/pandas/issues/7401
closes https://github.com/pydata/pandas/issues/7403
closes https://github.com/pydata/pandas/issues/7405
minor code change to https://github.com/pydata/pandas/pull/9061; otherwise only tests. the code change is to avoid https://github.com/pydata/pandas/issues/9170 issue.

@jreback took _shallow_copy/Index to a separate functions, and removed check_dtype=False.

@behzadnouri ok this looks good. and tests ok on windows.
can you add a release note (you can put all the issues on the same one I think). ping when pushed.

@jreback done!

thanks! a 4-some! excellent


Added keymode to constructor params to avoid calling _deriveDukptSess…

Added keymode to constructor params to avoid calling _deriveDukptSessionKey twice to use non-default modes

@dpjayasekara can you check this out.
It helps passing key mode to constructor, supports running node-dukpt tests on node v4.8.3

@awebdev Will do that. Thanks for the contribution.

@awebdev Please kindly note that there are multiple issues in this pull request which I have commented on. As other key modes are not supported yet, providing keymode in the constructor is discouraged. Also, since this is merely a minor improvement, will not be merged right now.

@dpjayasekara I do agree these are minor changes but calling _deriveDukptSessionKey twice to decrypt with pinkey does not make any sense.

Yes, these minor issues will be addressed and some performance improvements will be implemented soon in a new release. Please kindly note that this pull request contains multiple issues as I commented due to which I cannot merge this.