Jr fork/ergo serch input

simple implementation to add extra time selector buttons for more ergonomic selection of catalogs

I merged #14 this morning, which introduced many conflicting changes with this PR. To make up for that, I have done the rebase for you and force-pushed. I hope I reconciled the changes correctly; please check my work. :-)
Will comment later on your question on Slack about refreshing. Thinking....

After my last cleanup commit, now looking at the timedelta validator: I'm not sure why it is needed here, because the RButtons in the gui would only be checked, when one of the 4 timedeltas defined for these buttons are set. So if someone tried to set a different timedelta via CL, it would simply changes since and until widget but not check a RadioButton, so what's the matter for the validator then?

If someone set a custom timedelta via the CL, the UI would be in a state where, looking at it, you wouldn't be able to tell whether the since/until were absolute or relative. But maybe that's not such a big problem; maybe it's OK to allow some things from the CL that aren't possible to do through the UI and assume that if someone did this they would understand why since/until would update themselves whenever the catalog refreshed. It's a good point.
I would be OK with omitting the validator, at least until we hear from some user who got confused by this "secret feature". :-D

P.S. Thanks for fixing he signs in my tests, @JulReinhardt. :-D

Yes, true the UI would not determine between relative and absolute, but why does it matter as long as the since/until display are correctly displayed?

My suggestions for now are committed.
I commented the timedelta() restrictions out for now, since I don't see a clash. But happy to have it back in.

I think commented-out is a good state to leave them in. If it becomes a problem, we'll have a solution already thought-out and ready. If it's never a problem, we can delete the comment someday.

Ok, Travis Check failed (on mac)? So are we ready for merge?

The flake8 build failed too. Mind patching these little nit-picks from flake8 up so it's happy?
./bluesky_widgets/qt/search_input.py:13:1: F401 '..components.search.search_input.SearchInput' imported but unused
./bluesky_widgets/qt/search_input.py:15:1: E302 expected 2 blank lines, found 1
./bluesky_widgets/qt/search_input.py:182:5: E303 too many blank lines (3)

The Mac builds fail while trying to install numcodecs, not related to this PR, so we can address that separately (now tracked in #19) and proceed with merging this once flake8 is mollified. :rocket:

The board is green minus the known unrelated OSX build failures noted in #19. Thanks @JulReinhardt!


Ensuring that configs passed via the command line are respected.

Attempts to fix #2636

Coverage remained the same at 60.915% when pulling cc6d53fdeb4ff3cd24dd96ac2caa0b292f101208 on 2636-config-ignore into 721fb8ec70c929fd88ccb837c166a7a572588b47 on master.

Yes, some very rudimentary tests of your branch indicate it does fix -c and --config functionality.

Thanks Kevin, merged it.
Also tested it locally, and and reading the config file via command line works as intended!


An initial plan is available for integrating into Istio

An XDD exists for the Secretless - Istio integration
Parent of #354 - Istio can work with an external DB

Current efforts aren't pushing in this direction - closing this issue for now.


dbt schema generation

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/45
Originally created by @joshlambert on 2018-05-09 20:08:50

As part of the larger goal to have a schema library, dbt should also output both what it expects as input as well as the eventual output.
This will help us achieve to goals:

A catalog of the schema for each ELT job, and as well what is expected and output by dbt: https://gitlab.com/bizops/looker/issues/46
A common data model and mapping tool, to map a users custom fields to what is ultimately expected by dbt: https://gitlab.com/bizops/bizops/issues/9

View 4 previous comments from the original issue on GitLab


Feature build python 3

he changes that i made in the build.py for privly applications is made in from line 70 to 72. The problem was that the type of prettyHTML came out to be different in python2(str) and python3(bytes).So i committed the changes can be seen in line number 70 to 79.The lines changed can be seen in the above commit.

Just one thing,I added and committed only the build.py file and when i created a pull request,All the files (html) are here.Am i missing something sir.

If I recall correctly @smcgregor would prefer this PR target the privly-applications develop branch.  Also, it appears that more than just the two commits you describe were included in this PR.


Can not get device router

** Supporting Details **
Provide the following:

Browser Version: edge
Headscale Version: 64323e49c52a034d57867383549a51c65dfeddd6
Any Browser Errors (control+shift+i in chrome to see) GET https://xxxx:10443/api/v1/machine/14/routes 404 (Not Found)

Describe the bug
Default, headscaleAPIMachineOrNode is null, so it use machine as the router query path.
https://github.com/gurucomputing/headscale-ui/blob/64323e49c52a034d57867383549a51c65dfeddd6/src/lib/devices/DeviceCard/DeviceRoutesAPI.svelte#L9
https://github.com/gurucomputing/headscale-ui/blob/64323e49c52a034d57867383549a51c65dfeddd6/src/lib/common/apiFunctions.svelte#L159

machine needs to be updated to node, see #120

The machine or node check was removed, but clearly some references to machine got missed. Getting taken care of now


404s in documentation, links to .md instead of .html

The ToC on the left is fine, but on top-level pages that link to other sub-pages, there are links that end in .md which should end in .html. e.g.:
https://boats.gitlab.io/failure/howto.html
https://boats.gitlab.io/failure/guidance.html

Will be fixed when I deploy master to gitlab


Not getting recent fix

Hi! can you please tell me why I'm not getting the fix from https://github.com/basarat/typescript-collections/pull/101 when I do npm install?
Thank you

Hi @Pogix3m , I just published 1.3.3. Check it out!


document existing Keycloak configuration

Signed-off-by: Kai Helbig<EMAIL_ADDRESS>

Sorry, I forgot to push two commits.


[REVIEW] Mitype

Project link
https://github.com/Mithil467/mitype

Reviewed with the awesome EddieHub on live stream https://www.youtube.com/watch?v=e5EDm9S3m-4
Looks good, here are some thoughts:

I would reduce the topics
releases to be automated


Error with app that has baseUrl defined

When trying to setup Mirage, I am consistently getting the error even though all configuration appears to be correct:
Mirage: Your Ember app tried to GET '/models', but there was no route defined to handle this request. Define a route that matches this path in your mirage/config.js file.

The only thing that fixes this error is changing my config/environment.js baseUrl value to /, instead of what it is now. It seems to me that baseUrl should have no effect on Mirage, but in this case it definitely appears to. Branch this is occurring on here.

I just ran into the same issue, and it seems that the way to fix it for me was to set my namespace to '/api' in the mirage config.  Before this, I had it set to 'api', which was resolving to a relative url.
I'd recommend trying to set this.namespace = '/'; if you don't already have one set up, and see if that helps :)

@MiracleBlue thanks for the tip. Unfortunately, it didn't seem to make a difference in this case :( have tried every combination of configs I can think of...

@andycrum That is very unfortunate!  Could you share your configuration file with us in a gist?

@MiracleBlue https://github.com/andycrum/ember-data-model-maker/blob/6d90e2a5bc611c78207990549f05ec42d3c2b222/app/mirage/config.js
That branch is the one failing. I've changed it a bit since pushing there, but still have the same top-level this.get('/models'), etc.

Looks fine to me.  That's strange that setting the namespace didn't help.  My configuration looks very similar to yours except I have the namespace property which points to '/api' for me, which overrides the baseUrl only when the '/' is at the start of the namespace property.

Sorry I can't be of more help :( hope you figure out what's going on!

@MiracleBlue your comment pointed me in the right direction. I set this.urlPrefix = '/'; and everything is working now. haha.
@samselikoff it looks like it's an issue with the logic in this function: https://github.com/samselikoff/ember-cli-mirage/blob/master/addon/server.js#L197. I haven't looked that far into the root cause yet, but I was able to debug and found that Mirage was registering models as the route instead of /models. No idea if this is related to the baseUrl thing or not. Actually very curious how no one else ran into this, unless everyone happens to have a urlPrefix defined

Ah interesting!  I certainly don't have a urlPrefix defined as far as I can tell.  If I run into the same situation where my namespace isn't overriding the baseUrl in the future this will be very useful to know.

If true this is definitely a bug. A failing test would be great help!
On Sun, Oct 11, 2015 at 8:05 PM Nicholas Kircher<EMAIL_ADDRESS>wrote:

Ah interesting! I certainly don't have a urlPrefix defined as far as I can
tell. If I run into the same situation where my namespace isn't overriding
the baseUrl in the future this will be very useful to know.
—
Reply to this email directly or view it on GitHub
https://github.com/samselikoff/ember-cli-mirage/issues/302#issuecomment-147257395
.

@andycrum @MiracleBlue Anything additional we should fix here now that baseURL has been removed in Ember-CLI?  If not, I'd like to close this issue as we're trying to get the issue tracker into a more reasonable state so we can see the issues that are still outstanding 😀

@acorncom @andycrum @MiracleBlue what are we supposed to set now that baseURL is gone? I tried setting this.urlPrefix = '/'; and that did not seem to help. Please advise!

@rwwagner90 There's a new issue at #901, I'm unclear on the issue so probably best to watch that issue


osd-11758 Implement checks for generic endpoints on GCP

The current GCP implementation for egress supports features subnet-id, instance-type, region, cloud-tags, debug, timeout.
A brief documentation on how to use it is here.
Flags "image-id" and "kms-key-id" currently do nothing when used with the "--gcp" flag. This feature will be added soon.
Currently egress checks for generic and aws endpoints are done (default image).

@hbhushan3 would you mind updating the title to give a brief summary of what the PR is attempting to do?

@fahlmant currently egress command works for gcp - "./osd-network-verifier egress --gcp --subnet-id $SUBNET_ID" (instructions in readme. Lmk if you find any bugs or if code needs update. I'm working on adding Flags "image-id" and "kms-key-id" for gcp; will update test cases

Codecov Report

Merging #123 (59c43a0) into main (eff4acd) will decrease coverage by 7.20%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #123      +/-   ##
==========================================
- Coverage   27.39%   20.19%   -7.21%
==========================================
  Files          17       18       +1
  Lines         719      926     +207
==========================================
- Hits          197      187      -10
- Misses        500      718     +218
+ Partials       22       21       -1

Impacted Files
Coverage Δ

cmd/egress/cmd.go
0.00% <0.00%> (ø)

pkg/cloudclient/aws/private.go
46.38% <ø> (+0.53%)
:arrow_up:

pkg/cloudclient/gcp/gcp.go
0.00% <0.00%> (-80.96%)
:arrow_down:

pkg/cloudclient/gcp/private.go
0.00% <0.00%> (ø)

pkg/helpers/helpers.go
54.54% <ø> (ø)

pkg/output/output.go
42.85% <0.00%> (+5.71%)
:arrow_up:

pkg/errors/errors.go
50.00% <0.00%> (+7.14%)
:arrow_up:

/label tide/merge-method-squash

/lgtm

/approve


non-breaking space -> error

Describe the bug
Running mdformat on
# X

x

(# X\n\xa0\n\nx\n, note the non-breaking space) results in the error message
The formatted Markdown renders to different HTML than the input Markdown. This
is likely a bug in mdformat. Please create an issue report here, including the
input Markdown: https://github.com/executablebooks/mdformat/issues

Reproduce the bug
See above.
List your environment
No response

Thanks for reporting!
It seems this was fixed by https://github.com/executablebooks/mdformat/pull/352 and released in mdformat 0.7.17


1.10.2 Creative netherrack dimlet gives chisel variant

RFTools 5.83 and Rftoolsdim 4.54 (In both AllTheMods modpack ver 2.1 and AllTheMods:Expert 1.0.24 modpacks)
Creating a netherrack dimlet by "cheating" (shift + double-click) when used in a dimension creates a chisel variant of netherrack instead of vanilla netherrack. Tried on multiple dimensions, every time the material would come out as chisel and not vanilla.
I have not yet tested with a material absorber or other material types to see if it's limited to just netherrack or if other materials are affected as well.

It's not a Netherrack-specific issue -- the same thing happens with Glowstone for example.


Warning regarding useNativeDriver when displaying ImageViewer

I'm using React Native 0.62 and when I display a ImageViewer it thows me various warnings regarding useNativeDriver:

Does anyone knows why?
Thanks in advance

I have the same problem

+1 i had the same issue.

node_modules/src/image-viewer.component.tsx
node_modules/built/image-viewer.component.js
find Animated.timing
add useNativeDriver: false


Enchantment Glow not Staying

Hi,
I recently downloaded and installed the plugin, and I love it! However, when I put and enchantment on an item, if I close and reopen my inventory, or open my inventory after having been given an enchanted book, the items lose their enchanted glow, no matter what. I'm not sure what the error could be, and I checked the config, only to find nothing about the issue present. Please let me know how to fix this, and in the meantime, I'll just uninstall and reinstall the plugin, keeping my same config.
Thanks, Ty Howard.

That happens because the enchantment isn't really a enchantment. It is only a lore. So that's why it gets removed. Not much I can do about it. It will happen.


Fix cluster name in Getting Started Locally With Extensions

How to categorize this PR?

/area documentation
/kind bug
What this PR does / why we need it:
After following https://gardener.cloud/docs/gardener/deployment/getting_started_locally_with_extensions/#setting-up-the-kind-cluster and running make kind-extensions-up I noticed:
$ kind get clusters
gardener-extensions

Also
https://github.com/gardener/gardener/blob/e89c6d3b0bc51db61dde19696420b1ed4c684227/hack/kind-extensions-up.sh#L14
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

/lgtm
/approve


Naming collision with multiple foreign keys referencing one table

Tables with multiple foreign keys that point to the same foreign table result in naming collisions.
A simplified schema of what I'm working with:
CREATE TABLE "address" (
  "id" integer NOT NULL PRIMARY KEY AUTOINCREMENT,
  "name" text NOT NULL
);
CREATE TABLE "mail" (
  "id" integer NOT NULL PRIMARY KEY AUTOINCREMENT,
  "sender_id" integer NOT NULL,
  "recipient_id" integer NOT NULL,
  FOREIGN KEY ("recipient_id") REFERENCES "address" ("id"),
  FOREIGN KEY ("sender_id") REFERENCES "address" ("id") ON DELETE NO ACTION ON UPDATE NO ACTION
);
CREATE INDEX "idx_mail_recipient_id" ON "mail" ("recipient_id");
CREATE INDEX "idx_mail_sender_id" ON "mail" ("sender_id");

mail.xo.go ends up containing this:
// Address returns the Address associated with the Mail's RecipientID (recipient_id).
//
// Generated from foreign key 'mail_recipient_id_fkey'.
func (m *Mail) Address(db XODB) (*Address, error) {
	return AddressByID(db, m.RecipientID)
}

// Address returns the Address associated with the Mail's SenderID (sender_id).
//
// Generated from foreign key 'mail_sender_id_fkey'.
func (m *Mail) Address(db XODB) (*Address, error) {
	return AddressByID(db, m.SenderID)
}

I'd like to suggest naming those functions differently. How about using the column name without _id? e.g. Recipient() and Sender()? or maybe even combine it with the table name: RecipientAddress() and SenderAddress()?
Unless of course I'm missing something. I could probably copy the templates and modify them for now, but I'd rather see this fixed :)

Yes, I see the issue. I'm working on a fix for this now. What will end up happening is that there will likely be 2 new command line options that toggle behavior for this, however I'm looking into seeing if there is a way to detect this automatically and force the behavior.

I've pushed a change that should fix this in the default case, and introduces a new command line option, --fk-mode that provides a number of different types of foreign key modes. The default, smart will look if there are other foreign keys with that same parent name and use the field foreign key mode, otherwise if there are no collisions, then the parent foreign key mode will be used.
A summary of the foreign key modes / behaviors:
smart  - default behavior; will use parent when no name collisions, and field when there are
parent - foreign key funcs on generated types as will be in the form of '<RefTable>'.
field  - foreign key funcs on generated types as will be in the form of '<RefTable>By<FieldName>'
key    - foreign key funcs on generated types as will be in the form of '<RefTable>By<KeyName>'

For example, if you have a simple authors and books relationship (as in the examples), then the following will be the result for the different modes:
parent - func (b *Book) Author()
field  - func (b *Book) AuthorByAuthorID()
key    - func (b *Book) AuthorByFk123() // (assuming the foreign key name is 'fk_123')

Let me know if that works for you.
I am hesitant to do anything more complicated or to change the templates, as I think if there is more custom logic/behavior necessary, then one likely needs to use custom templates for their schema as it is not possible to accommodate all the different schema designs styles, and also still work across the supported database types, here.

Thank you very much! The change works fine for me.
Just one minor thing: the default argument for --fk-mode shows as a pointer to FkMode.
  --fk-mode FK-MODE, -k FK-MODE
                         sets mode for naming foreign key funcs in generated Go code [values: <smart|parent|field|key>] [default: smart] [default: 0xc82000e978]

I'm aware of the issue -- it's a problem with the package I'm using for command line argument parsing, which is why I manually put the other [default: smart] text there.
Glad it works for you!


python310Packages.sensor-state-data: 2.6.0 -> 2.7.0

Description of changes
https://github.com/bluetooth-devices/sensor-state-data/blob/main/CHANGELOG.md#v270-2022-09-06

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 191658 run on x86_64-linux 1

11 packages failed to build:

home-assistant-component-tests.bluemaestro
home-assistant-component-tests.bthome
home-assistant-component-tests.govee_ble
home-assistant-component-tests.inkbird
home-assistant-component-tests.moat
home-assistant-component-tests.qingping
home-assistant-component-tests.sensorpro
home-assistant-component-tests.sensorpush
home-assistant-component-tests.thermobeacon
home-assistant-component-tests.thermopro
home-assistant-component-tests.xiaomi_ble

30 packages built:

python310Packages.atc-ble
python310Packages.bluemaestro-ble
python310Packages.bluetooth-sensor-state-data
python310Packages.bthome-ble
python310Packages.govee-ble
python310Packages.inkbird-ble
python310Packages.moat-ble
python310Packages.qingping-ble
python310Packages.sensor-state-data
python310Packages.sensorpro-ble
python310Packages.sensorpush-ble
python310Packages.thermobeacon-ble
python310Packages.thermopro-ble
python310Packages.tilt-ble
python310Packages.xiaomi-ble
python39Packages.atc-ble
python39Packages.bluemaestro-ble
python39Packages.bluetooth-sensor-state-data
python39Packages.bthome-ble
python39Packages.govee-ble
python39Packages.inkbird-ble
python39Packages.moat-ble
python39Packages.qingping-ble
python39Packages.sensor-state-data
python39Packages.sensorpro-ble
python39Packages.sensorpush-ble
python39Packages.thermobeacon-ble
python39Packages.thermopro-ble
python39Packages.tilt-ble
python39Packages.xiaomi-ble


Test fix prevents testing plugins

This line causes problems when I try to test my plugins. Since I'm running my plugin tests from a different repo, the file path doesn't render correctly and my tests throw an exception.
A comment in the rendered file says it's needed to work around a TinyMCE bug. I tried running the Camaleon test suite with that line commented out and it passed, so maybe the workaround is no longer needed.
If it's still needed, can we just place the necessary content in a conditional block in the view, instead of rendering it from a file?

@brian-kephart you are right, can you please make a pull request with the change?
Regards!


First Person Issue 1.4.1

Just updated to 1.4.1 and I am experiencing that when zooming to first person, the camera lock up to a static position (in third person). Meaning I can run around and away from the camera positions, very weird and could almost be a funny extra mode to capture something in, though yeah, clearly we also need the regular first person view :)

It would be helpful if you provided some details with a bug report. I cannot reproduce this by simply zooming in and out.
What were you doing what it happened?
Do you have any other mods installed?
What are your settings?

Strange, I am fairly sure it must be connected (like with the last bug, reverted to 1.3.1 for all to work fine). Here's are the basic details, if its down to settings I'll have to check later on. Was not doing anything, as experienced it twice before downgrading, once in Dawnbreak Village, just standing outside a house (admin mode and some toggleobserve mode on/off most likely). Second time was in the Copper Mines, just hanging around (again with admin mode, but not using anything beyond toggleobserve and addtime)... at what point it seemed to be easy to reproduce so avoided it and later reverted to 1.3.1.
Other Mods:
BepInExPack_V_Rising 1.0.0
Wetstone 1.1.0
HideInterface 1.2.0

I had the same issue, but only when playing on a dedicated server that I set up locally. Solo & Private games don't seem to suffer from this issue. I went as far as installing the mod on the server as well but it didn't help (tested with 1.4.1 of the Mod, and Bepinex v1.0.0)
When playing on a dedicated server and zooming in, at the point where the camera is supposed to transition into first person view, instead of first person view the camera gets stuck in place and you can move the character independent of the camera.

I tried a few times and cannot reproduce this. Could you post the server settings and ModernCamera settings?

That's makes this a mystery then; I've no idea of any ModernCamera settings > Installed, using F11, nothing more nothing less. In regards to the server, it's a dedicated one and I think nothing special settings wise but see attached files. Revering to 1.3.1 makes everything work just fine for some reason. I can test on a 2nd client this week, just in case and or without this mod installed and get back top you.
ServerGameSettings.json.txt
ServerHostSettings.json.txt

Okay, I can confirm that starting the game only with your mod still causes the issue (1.4.1) so it's not related to the other mod at glance. Any thoughts on what I can supply you with and or test on my installation?

I use a local dedicated server when testing. Even using your server and client settings with different types of servers I cannot reproduce at all.

Wow, okay, that's crazy. Maybe I need to try to reinstall the server and just copy over the DB and/or try to take the server DB to a local save (presuming that also work) and see if the error occurs then as well. That you can't reproduce and that I still have this issue makes no sense to me at this point.

Here's how it looks for reference:
https://user-images.githubusercontent.com/16366292/178098553-8abee5d1-e506-4c47-b230-a27b3e4e09ed.mp4

Actually looks kind of cool.
Is there any errors in either the client or server logs?

Haven't heard anything new about this issue


Value injection does not work with structs

The following code does not compile:
import poodinis;

struct MyStruct
{

}

class MyClass
{
    @Value("")
    MyStruct myStruct;
}

void main()
{
    auto dependencies = new shared DependencyContainer();
    dependencies.register!(ValueInjector!MyStruct, MyStructInjector);
    dependencies.register!(MyClass);
}

class MyStructInjector : ValueInjector!MyStruct
{
    MyStruct get(string key)
    {
        return MyStruct();
    }
}

Here are the error messages:
../../.dub/packages/poodinis-8.0.0/poodinis/source/poodinis/container.d-mixin-397(397,24): Error: undefined identifier app.MyStruct
../../.dub/packages/poodinis-8.0.0/poodinis/source/poodinis/container.d(321,24): Error: template instance poodinis.container.DependencyContainer.callPostConstructors!(ValueInjector!(MyStruct)) error instantiating
../../.dub/packages/poodinis-8.0.0/poodinis/source/poodinis/container.d(265,54):        instantiated from here: resolve!(ValueInjector!(MyStruct), ValueInjector!(MyStruct))
../../.dub/packages/poodinis-8.0.0/poodinis/source/poodinis/autowire.d(214,28):        instantiated from here: resolve!(ValueInjector!(MyStruct))
../../.dub/packages/poodinis-8.0.0/poodinis/source/poodinis/autowire.d(138,49):        instantiated from here: injectValue!("myStruct", 0LU, "", false, MyClass)
../../.dub/packages/poodinis-8.0.0/poodinis/source/poodinis/autowire.d(118,3):        ... (3 instantiations, -v to show) ...
../../.dub/packages/poodinis-8.0.0/poodinis/source/poodinis/container.d(146,47):        instantiated from here: register!(MyClass, MyClass)
source/app.d(18,17):        instantiated from here: register!(MyClass)
dmd failed with exit code 1.

After some research, I found out, that this is not a bug limited to ValueInjector. Template classes in general have this problem. For example, the following program also does not compile:
app.d:
module app;

import poodinis;
import test;

void main()
{
    auto dependencies = new shared DependencyContainer();
    dependencies.register!(TestClass!TestStruct);
    dependencies.register!(AnotherTestClass);
}

class AnotherTestClass
{
    @Autowire TestClass!TestStruct testClass;
}

struct TestStruct
{

}

test.d:
module test;

class TestClass(T)
{

}

The reason is, that in poodinis.container.DependencyContainer.callPostConstructors and poodinis.autowire.AutowiredRegistration.getPreDestructor only the module of the type that is currently being injected are imported. We also need to import the modules of the template arguments of the type that is currently being injected. Something like the following:
string moduleMixins(Type)()
{
	string ret = `import ` ~ moduleName!Type ~ `;`;
	static if (__traits(compiles, TemplateArgsOf!Type))
	{
		foreach (TemplateArg; TemplateArgsOf!Type)
		{
			static if (!isBasicType!TemplateArg)
				ret ~= `import ` ~ moduleName!TemplateArg ~ `;`;
		}
	}
	return ret;
}

and in poodinis.container.DependencyContainer.callPostConstructors then
mixin(moduleMixins!Type());

instead of
mixin(`import ` ~ moduleName!Type ~ `;`);

The same also needs to happen in poodinis.autowire.AutowiredRegistration.getPreDestructor. Maybe somewhere else, too, I don't know about that.

I fear, to fix this properly, we need to do this recursively. The template arguments might be instanciated templates themselves...

Thank you for reporting this issue. Unfortunately I was so busy fixing this error I didn't see that you replied meanwhile. In the end it seems we came to the same conclusion and same fix. I've just fixed this in the develop branch myself.
Thank you regardless for taking the time to look into it.
Indeed we should do this recursively, I will add this.

What is still a mystery to me is that the currently existing unit test, which also injected a struct, did succeed.

Will be fixed in 8.0.1 which I will release shortly

As I mentioned earlier (in my very long reply), all of this also need to be done in poodinis.autowire.AutowiredRegistration.getPreDestructor. Thanks for working on this so quickly!

I've applied recursively importing types everywhere now

Thank you, looks good!


Is it possible to learn good spectrogram representation without good attention alignment?

Were you able to achieve good alignment? If so, at what epoch / iteration did you achieve it?


Minimal interface, with regard to the type of the keys

Hi, i am pasting this from Reddit, because i had some messages disappearing there, so i prefer to continue the discussion here on Github.
To summarise, i began asking you what is the utility of the fancy types for the keys, and you answered:

You got this right. You can use the Raw interface if you like, but I don't really get why anyone would want this.
The type safe interface basically only requires you to list the keys you are going to use. (Ok currently additional smart constructors are needed) Which is really very little work (this was actually a very important design criteria) and saves you from misspelling keys or using the wrong key!
Also, because the keys encode the type you are going to retrieve, you will never need any type annotations. If keys were plain strings and you would retrieve a value, let's say from the repl, a type annotation would be necessary. - So especially if you just want to try it out an explicit key type makes sense.
Ok, having said that: I am not opposed to including an additional interface where keys are strings, and values are encoded/decoded for you. If people want/need that, I am happy to accept my first pull request :-)
in fact - even the example in the Readme, would need an explicit type annotation, when written with strings as keys:
user <- localStorage.getItem "UserConfig"
log $ gShow user
This would not work without a type annotation, as gShow is also generic - so the compiler has no way of figuring out of what type the retrieved value is.

Alright, this is interesting. Thus the type fanciness of the keys allows the compiler to infer the type of the value. Well, this is a valid reason, thanks for elaborating. The value added was not clear to me at first. I will close this issue, but it will stay as a form of documentation for the future. Cheers!

Thanks!
Wuhu -my first issue and it is closed already :-)

Updated the readme accordingly.

:+1:

@eskimor Could the interface be simplified though?
Why go over the trouble of requiring the key to be generic just to call gShow to get the string representation when we could let the user supply the string value for the key:
data Key a = Key String

myKey :: Key Foo
myKey = Key "foo"

The string value can then be extracted with (\Key str -> str) instead of gShow.

That is actually a quite elegant solution. The manual string specification makes it a little less type safe: If you accidentally (copy&paste) use the same string twice, you would get a runtime error, instead of a compiler error. Other than that I like it, but I also don't really see a problem with the status quo, so I am a bit hesitant to do this breaking change.

Another advantage of the above approach is that one can actually specify the key to be used. This can be useful if you have some legacy data in the localstorage that you don't want to throw away.

Or you just want more readable keys, which is nice for debugging. I think I am sold and would accept a PR ;-)


870 rename notify of removal

Hey @rlgreen91 , have you used your credientials recently and confirmed that they worked independently? I recall you mentioning that you were using different IAM credientials for this S3 access. Could there be something about that change that is affecting the fog configuration?

Ok, so this branch had issues, mostly due to configuration problems on my end.  I have set up a new dev environment, made these changes there, and will create a PR from that.  So now, this one can die.


Allow for injecting custom IDS statuses with IDSResolver.queryMule

This doesn't allow for forcing an 'available' status with a cli flag, I'll implement that next. Just added that; I think the CLI library we use is kinda messed up, but more on that later.
I've verified that this works when IDS returns a plist for the destinations, but sometimes it returns an NSKeyed(Una|A)rchiver thing (I think it did that on earlier versions of ventura? It used to do that on my computer, but doesn't anymore. I don't know what changed, honestly), and I can't guarantee this works for that. I may have to install an extra partition and play around with what version of ventura it's on to make it work.
Also this makes scratchBox async and adds the extra glue necessary for that.
The function that you'll want to implement the 'querying echobot' functionality in is IDSResolver.queryMule. right now it returns 'available' for every status, so we won't want to roll out this change until we've also implemented the echobot stuff. Now it just checks to see if we've manually overwritten any statuses, e.g. with the cli commands, and uses those, otherwise returns unavailable for everything.
The code's kinda ugly and could probably be made simpler by making a func like func hijackIDSResponse(in dict: inout OS_xpc_object) (especially since that would allow for early exit instead of a bunch of nested if { } else { } statements... now that I'm thinking about it, I probably should do that), but I'll get around to that. Maybe when I add the send-message cli flag.
Ok, now re: the CLI library being messed up: I can't get the overwriteAvailability flag (to tell the send_message cmd to report a specific availability) to work nicely. E.g. if I do barcelona-mautrix send_message iMessage;-;+12345678900 "example message" true, it says that I've passed in a non-bool value for overwriteAvailability, but if I switch the order of the last two arguments, it gives the same error. Then, if I completely get rid of the example message argument, it thinks that I haven't passed in any value for overwriteAvailability and thinks that the message I want to send is "true". The only way I've found to trigger it correctly is to call barcelona-mautrix send_message iMessage;-;+12345678900 true true. this way, your text message will be true, but it will correctly process that you've passed in a value for overwriteAvailability and will get its value correctly. @bradtgmurray since I think you'll be using this the most, if you could try it out once merged and let me know if I'm crazy, that would be nice.
@myyra just like a cursory review would be great, make sure there's no weird code smells since I wrote most of this while tired.

Looks good! The extra logs are helpful, though I worry about pulling in more stuff than we need in our little CLI.
One thing I noticed is that when running the command the Got reply for __sendMessage log never came out, which implies there is no IDS lookup happening? Were you able to get a lookup to trigger?

Looks good! The extra logs are helpful, though I worry about pulling in more stuff than we need in our little CLI.
One thing I noticed is that when running the command the Got reply for __sendMessage log never came out, which implies there is no IDS lookup happening? Were you able to get a lookup to trigger?

Going to merge this to continue testing


Unblock existed wallet

Need more information:

when we block wallet?
do we need "Forget this wallet" button?
how UI should know that wallet is blocked?


refactor: override entity assignment in Node class

What does this pull request change?
Fixes lint errors in Node class
Why is this pull request needed?
Better to use types correctly than to ignore the warnings
Issues related to this change:

If specialized classes is setting entity themself, should it not be removed from the parent class? If that is possible ofc. Can "NodeBase" do anything with the entity if it does not know it's type?

NodeBase has an is_empty function which returns not self.entity, so it does use the variable. It also has a repr function which assumes entity is a dict, that one should probably be moved into Node instead


Line ending combobox is inactive

Line ending combobox is unavailable after the extension update
Describe the bug

OS and Version: Windows_NT x64 10.0.19045
VS Code Version: 1.96.2
Serial Monitor Extension Version: 0.13.1

Screenshots

Resolved.
Message encoding should be: Send as Text


fixed fireball rank 12 level

Got a message at level 61 that I was using a Fireball (Rank 11) instead of Rank 12, went to trainer to double check that Rank 12 Fireball is indeed at level 62. Hopefully I edited the correct file. Love the addon, thanks!

Thanks for the report!

Ability data comes from Gogo's revised ability list, please leave a comment for any ability data changes you have in mind.

All changes are from the GSheet, looks like the same situation to Shadowbolt https://www.curseforge.com/wow/addons/ranksentinel?comment=12
Updated with v2.0.1b


docs: 📚 Create showcase for form field usage in Storybook

Stories
As a user of the Synergy Design System I want to have a demo which shows how to build html forms using Synergy components.
As a developer of the Synergy Design System I want to have a demo to make sure that the created form field behaves as expected.
As a developer of the Synergy Design System I want to have a demo in a separate package to validate that the SDS can be flawlessly used as an external library.
Acceptance critera

status of the form field is submitted via the form data when triggering the submit action
data binding is working
reference to SDS package is working

See also #142 and #143.

blocked by https://github.com/SickDesignSystem/design/issues/41


Error Data Source ardoq_components

Hello,
I have an error with the Data Source ardoq_components, the config is as per documented
# request details for a workspace with the name "Some workspace"
data "ardoq_workspace" "someworkspace" {
    name = "Some workspace"
}

# returns all components in workspace
data "ardoq_components" "all" {
  # use the retrieved data from above
  root_workspace = data.ardoq_workspace.someworkspace.id
}

# output all components
output "all_components" {
  value = data.ardoq_components.all.components
}

Error is:
Error: components.0.fields.data_object_s: '' expected type 'string', got unconvertible type '[]interface {}', value: '[Finance]'
│
│   with data.ardoq_components.all,
│   on ardoq.tf line 36, in data "ardoq_components" "all":
│   36: data "ardoq_components" "all" {

``
Thank you to have a look at it.

Thank you very much for this issue.
This was easily reproduced.
I was so busy with, learning go, getting the provider to a somewhat functioning level, learning about CI/CD and more...
That I forgot to test for Ardoq fields, other than text.
Currently I am stuck with the data type of string for all custom fields, so I've create a new version of the ardoq-client-go which just converts any Ardoq type to string.
I presume the field that gave you trouble was of the Ardoq type "Select multiple list"
For now I have converted this to "[Option 1 Option 2]"
Although I think a value of "Option 1,Option 2" might be better.
For now this is fixed for the data_source ardoq_component.
I will test and see the behaviour for creating components and custom fields
Version 0.1.2 is available on the Terraform reigstry.

OK with V0.1.2, all fine, no breaking errors.
I am interested to push the investigation to create ardoq components from my terraform cloud infra.
I will narrow my test-bed to a single workspace.
Thank you again for your diligence.


Add telnet connect FSM for XE when no password is required

The Finite State Machine (FSM) in the connection module is not able to handle ‘no login’ configuration under line vty.
line vty 0 4
exec-timeout 0 0
password lab
no login

@jamesjwei, please email me the DUT credentials.

@jamesjwei I can't reproduce it. It works with latest condoor.


Can deepspeed-MII run on AMD GPU?

Can deepspeed-MII run on AMD GPU?
susie.sun@yz-amd1:~$ deepspeed --num_gpus 1 pipe.py
[2023-12-05 09:42:21,499] [INFO] [real_accelerator.py:161:get_accelerator] Setting ds_accelerator to cuda (auto detect)
[2023-12-05 09:42:21,780] [WARNING] [runner.py:203:fetch_hostfile] Unable to find hostfile, will proceed with training with local resources only.
Traceback (most recent call last):
File "/home/susie.sun/.local/bin/deepspeed", line 6, in
main()
File "/home/susie.sun/.local/lib/python3.10/site-packages/deepspeed/launcher/runner.py", line 422, in main
raise RuntimeError("Unable to proceed, no GPU resources available")
RuntimeError: Unable to proceed, no GPU resources available
susie.sun@yz-amd1:~$ cat pipe.py
import mii
pipe = mii.pipeline("mistralai/Mistral-7B-v0.1")
response = pipe(["DeepSpeed is", "Seattle is"], max_new_tokens=128)
print(response)

Hi @sunpian1, our custom kernels do not currently support AMD GPUs. It is something we have considered, and I will share this with the other devs. Thank you!

@mrwyattii thanks for your answer!


Remove eigenDA/client in favor of authenticated client

https://github.com/Layr-Labs/eigenda/pull/551

Closed by: https://github.com/Layr-Labs/eigenda-proxy/pull/23


The system has a super admin tool to manage adding/editing/updating data in the system

Rails admin / administrate / etc. Let's get this in from the outset.

First closed ticket on the project! 🍾


MINIFICPP-1091 - Update LICENSE and NOTICE

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

reviewing

@apiri @arpadboda thanks for the reviews, addressed the review comments in a new commit.


[🐛 Bug]: ESLint rule no-app-not-found-runtime does not work when using a src folder

next-on-pages environment related information
System:
	Platform: darwin
	Arch: arm64
	Version: Darwin Kernel Version 23.0.0: Fri Sep 15 14:41:43 PDT 2023; root:xnu-10002.1.13~1/RELEASE_ARM64_T6000
	CPU: (10) arm64 Apple M1 Pro
	Memory: 16 GB
	Shell: /bin/zsh
Package Manager Used: npm (10.1.0)

Relevant Packages:
	@cloudflare/next-on-pages: 1.7.2
	vercel: N/A
	next: N/A

Description
When using a src folder at the top level of the application, the rule no-app-not-found-runtime does not match the file src/app/not-found.tsx and therefore does not show any errors.
Reproduction

Create a Next.js project with this structure:

src/
  app/
     not-found.tsx

Run linting with the rule no-app-not-found-runtime active (currently it's exported as no-app-not-found, see other issue: #530)

Pages Deployment Method
None
Pages Deployment ID
No response
Additional Information
No response
Would you like to help?

[X] Would you like to help fixing this bug?

Thanks a lot for the issue @mattiasw 😃
Yeah you're again right, I think that this can be very simply fixed by updating accordingly this regex:
https://github.com/cloudflare/next-on-pages/blob/bf57ec9d01b354ea6c9b9bd17c683d592d5c6ae9/packages/eslint-plugin-next-on-pages/src/rules/no-app-not-found-runtime.ts#L6
Again I am happy to fix this or let you apply the change if you'd like to 😃 , please let me know what you prefer 🙏

I can do this one too.


Collect exceptions from all processes and collect from startup

Summary
This change gives the exceptions history feature the ability to collect exceptions from startup (prior behavior was that it started a few seconds after process discovery) and to collect from all processes (prior behavior only collected from the default process). There is currently one takeback with this change: when a process shuts down, the exception history information for the process is removed and no longer accessible; I would prefer to address that in a separate change.

Release Notes Entry

/backport to release/8.x


I Don't understand last minute claim.

so i set tup the bot and everything is working perfect but i have a question about what "last minute claim" is, idk if im just dense but the explanation was kinda confusing, i was also wondering what soul kakera is and what the diffrent colors of kakera mean?

"min_kak - A minimum kakera value ### to snipe a claimable character regardless of whether it's in the series/name lists
Last_True - (True|False) enable Last Minute Claim windows

last_claim_min - (1-180) the window the window is open for e.g. 10 means last 10 minutes

min_kak_last_min - same as min kak but only within the last minute claim window"

https://github.com/vivinano/MudaeAutoBot/issues/143#issuecomment-1055812011

So how last minute claims work is that if your claim is still available it will claim during the last claim window fitting the min kak there

so if
last_claim_min: 60
min_kak_last_min: 500

that mean the script will claim wishes and chars from name_list
then if 1h left for reset claim the script will claim any char has 500kak value
soul kakera mean Kakera reactions on char that have 10 keys or more

LVL 10 (Chaos) => Kakera reactions on this character cost half the power for the character owner.

Your characters with 10+ keys consume half the power (15%)

this cleared it up for me,
Thank you so much!

#143 (comment)

So how last minute claims work is that if your claim is still available it will claim during the last claim window fitting the min kak there

so if
last_claim_min: 60
min_kak_last_min: 500

that mean the script will claim wishes and chars from name_list then if 1h left for reset claim the script will claim any char has 500kak value
soul kakera mean Kakera reactions on char that have 10 keys or more

LVL 10 (Chaos) => Kakera reactions on this character cost half the power for the character owner.

Your characters with 10+ keys consume half the power (15%)


[Feature Request][Go SDK]: Add textio.ReadWithFilename transform

What would you like to happen?
I would like a transform that reads a text file (splittable implementation) and emits the file path paired with each line, i.e. returns a PCollection<KV<string, string>>
Issue Priority
Priority: 2 (default / most feature requests should be filed as P2)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[X] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue


How do I claim rewards?

Is it possible to claim rewards through this API? I would be very thankful if someone directed me to an example if it's possible! Thank you in advance.

You can claim reward using the method claimRewardBalance. Here is an example:
steem.broadcast.claimRewardBalance(privatePostingWif, 'username', '0.000 STEEM', '0.000 SBD', '123.093932 VESTS', function(err, result) {
  console.log(err, result);
});

Thanks! How can I find out the amount that is available? Is there anyway to tell it to claim whatever is available? If not, how do I calculate the available amount?

You can get user current reward balances using the method getAccounts. The parameters are: "reward_sbd_balance", "reward_steem_balance" and "reward_vesting_balance".

Perfect! Thanks heaps :]


Fixed docker network remove command

Please edit dind-cluster.sh and then run build/genfixed.sh to update 'fixed' scripts.

Thanks!


/linodes list view does not play nicely with groups

Screenshot?

Is it just the padding issue you are seeing here?

Yes.

Cool.


JBPM-4859 - Edit FormService and ImageService tests

closes #345

+1
merged into 6.4.x, thanks @jakubschwan


Push login group loading to sync task

This should give a speed increase to logins. It will put group loading on the next tick.

Can one of the admins verify this patch?  Type 'ok to test' to test.

You can actually remove that completly. It's a leftover from reinforcement decay (which was initially in NameLayer) and there is no good reason to preload all groups of a player.

Works for me. I'll swap it when I get back from getting food.
On Sat, Aug 13, 2016, 11:05 PM Maxopoly<EMAIL_ADDRESS>wrote:

You can actually remove that completly. It's a leftover from reinforcement
decay (which was initially in NameLayer) and there is no good reason to
preload all groups of a player.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Civcraft/NameLayer/pull/213#issuecomment-239655083,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIZCbWKPs9fHhANNydC6Ztc2wFk8l0O8ks5qfpQHgaJpZM4Jj0eM
.

So now we can just check for player group existence on first login and otherwise not touch groups on login?

Yes since they get loaded on first use and then cached
On Sun, Aug 14, 2016, 6:55 AM ttk2<EMAIL_ADDRESS>wrote:

So now we can just check for player group existence on first login and
otherwise not touch groups on login?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Civcraft/NameLayer/pull/213#issuecomment-239669741,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIZCbUJUCbJIBsCHG3_G17dGZeMB-2UFks5qfwIcgaJpZM4Jj0eM
.

Update commit, should be good to merge.


support various http status code

We are looking if a resource can be serialized as JSON API format with various HTTP status code returned to a client, for example, we need to response a user resource with 300 HTTP status code to the client when one of business validations is not passed. we tried this codes, but looks it doesn't work.
[ReturnsResource(typeof(AuthenticateResource))]
public IHttpActionResult CreateSession(UserModel model) {
return Content(System.Net.HttpStatusCode.Ambiguous, model, new JsonApiMediaTypeFormatter())
}

Can you elaborate on "it doesn't work"? What does happen? As far as I'm aware Saule doesn't mess with status codes outside of what the spec dictates.

we saw a 500 internal server error when using the version<IP_ADDRESS>8, but it works great if we switch to the latest beta version from nuget.

Yeah, I really need to get that release out. I don't want to without docs for all the features tho :smile: (#139 is the tracking issue).

1.6 is released, so this should be resolved.


ValueError: continuous format is not supported

Hi there,
With ABOD, I'm getting this error message:
ValueError                                Traceback (most recent call last)
<ipython-input-158-c00d8d10c5e7> in <module>
      4 y_test_scores = clf.decision_function(X_test) # raw outlier scores
      5 y_test_proba = clf.predict_proba(X_test) # outlier probability
----> 6 evaluate_print("ABOD", y_test, y_test_scores) # performance evaluation

~\AppData\Local\Continuum\anaconda3\lib\site-packages\pyod\utils\data.py in evaluate_print(clf_name, y, y_pred)
    281     print('{clf_name} ROC:{roc}, precision @ rank n:{prn}'.format(
    282         clf_name=clf_name,
--> 283         roc=np.round(roc_auc_score(y, y_pred), decimals=4),
    284         prn=np.round(precision_n_scores(y, y_pred), decimals=4)))
    285

~\AppData\Local\Continuum\anaconda3\lib\site-packages\sklearn\metrics\ranking.py in roc_auc_score(y_true, y_score, average, sample_weight, max_fpr)
    353     return _average_binary_score(
    354         _binary_roc_auc_score, y_true, y_score, average,
--> 355         sample_weight=sample_weight)
    356
    357

~\AppData\Local\Continuum\anaconda3\lib\site-packages\sklearn\metrics\base.py in _average_binary_score(binary_metric, y_true, y_score, average, sample_weight)
     71     y_type = type_of_target(y_true)
     72     if y_type not in ("binary", "multilabel-indicator"):
---> 73         raise ValueError("{0} format is not supported".format(y_type))
     74
     75     if y_type == "binary":

ValueError: continuous format is not supported

Any suggestion to rectify this?
Kind regards
Morgan

My suggestion is to print out the value y_test_scores. It is expected to be a continuous vector.
I guess the returned value is all 1 or 0 or very close values?
Hope this helps!
Yue


Bluetooth Classic

Hi, can we use this plugin to scan Bluetooth Classic devices as well?

Hi @bojankostic81,
I'm the friendly issue checker.
It seems like (100.00 %) you haven't used our issue template :cry: I think it is very frustrating for the repository owners, if you ignore them.
If you think it's fine to make an exception, just ignore this message.
But if you think it was a mistake to delete the template, please close the issue and create a new one.
Thanks!


Azure pipelines support

Please add support for Azure pipelines

This'd be great to have - would you be up for opening a PR?

Duplicate of #56 ?
Also, how would one go about adding/debugging a provider, is there a guide @JasonEtco ?


v.0.6.3 to Dev

See individual commit details for more information.

Hello @mikkokotila! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file talos/autom8/automodel.py:

Line 120:80: E501 line too long (80 > 79 characters)

In the file talos/reducers/GamifyMap.py:

Line 41:80: E501 line too long (86 > 79 characters)

In the file test/commands/test_latest.py:

Line 36:80: E501 line too long (96 > 79 characters)

Pull Request Test Coverage Report for Build 546

104 of 171 (60.82%)  changed or added relevant lines in 17 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.2%) to 85.979%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

talos/autom8/autoparams.py
0
1
0.0%

talos/reducers/reduce_run.py
7
9
77.78%

talos/utils/experiment_log_callback.py
9
11
81.82%

talos/autom8/autoscan.py
3
6
50.0%

talos/reducers/GamifyMap.py
40
51
78.43%

talos/reducers/forrest.py
1
14
7.14%

talos/reducers/trees.py
1
16
6.25%

talos/templates/datasets.py
1
21
4.76%

Totals

Change from base Build 527:
-3.2%

Covered Lines:
1300

Relevant Lines:
1512

💛  - Coveralls


Associations.AssociatedCompany and Associations.AssociatedContacts

Hi
I Am trying to pull Associations.AssociatedCompany and Associations.AssociatedContacts data for all deals and these values are set to nothing, do you have some sample code to do this?
Paul

Appears to be an issue when deserializing the deal JSON data into a DealHubSpotModel object.
The JSON result from the example get all deals code includes the associations but they aren't assigned to the object when converted.

Issue is in RequestDataConverter's FromHubSpotListResponse method in the following block of code:
// Convert all the entities
var jsonEntities = expandoDict[propSerializedName];
foreach (var entry in jsonEntities as List)
{
// convert single entity
var expandoEntry = entry as ExpandoObject;
var dto = ConvertSingleEntity(expandoEntry, Activator.CreateInstance(genericEntityType));
// add entity to list
listAddMethod.Invoke(listInstance, new[] { dto });
}
The expandoEntry object has the association values.  After the highlighted line is called dto is a DealHubSpotModel object but without the associations.

Is this going to be fixed and the NuGet Package updated?

Hi @PaulWHY, thanks for your feedback! If you're able to open a PR with the relevant fix I can get that merged & NuGet updated as soon as possible. Otherwise, I'll try and take a look at this issue in the next few days.
Thanks @bfutrell70!

I am unable to open the PR in VS 2015, if you could fix over the next few days that would be great
Thanks
Paul

@clarkd You're welcome. I tried to see if I could fix it but so far haven't had any luck.  I figured if I could at least report where it occurred it might give someone else an idea to resolve the problem.
I initially thought the issue was caused by the IgnoreDataMember attribute assigned to the DealHubSpotModel's Associations property but even after setting it to [DataMember(Name = "associations")] the same issue occurred.
I also noticed that in the EngagementHubSpotModel the Associations property had a DataMember attribute assigning "associations" as the name.  Unfortunately I haven't had a chance to whip up some test code to see if Engagements have the same issue.

Something else it could be....
Besides the dto object losing the associations when being passed into ConvertSingleEntity there doesn't appear to be any code within that method to look for either "associations", "associatedVids" or "associatedCompanyIds".
There is code in there to find "dealId", "companyId", "vid" and "properties".

I think I may have have figured it out - maybe.
In the RequestSerializer class there is a virtual method for deserializing a HubSpotEntity and one for deserializing a HubSpotListEntity.
In the method that deserializes an entity there is a call to the HubSpot object's FromHubSpotEntity.  For the DealHubSpotModel this sets the associations based on the data passed into it as a parameter.
The same method in the DealListHubSpotModel is empty so it doesn't know to do something with the list of deals.

I'd need to have a look but it sounds like the list's method might need to
invoke the method for each deal?
On Thu, 14 Jun 2018, 21:13 Brian Futrell<EMAIL_ADDRESS>wrote:

I think I may have have figured it out - maybe.
In the RequestSerializer class there is a virtual method for deserializing
a HubSpotEntity and one for deserializing a HubSpotListEntity.
In the method that deserializes an entity there is a call to the HubSpot
object's FromHubSpotEntity. For the DealHubSpotModel this sets the
associations based on the data passed into it as a parameter.
The same method in the DealListHubSpotModel is empty so it doesn't know to
do something with the list of deals.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/squaredup/HubSpot.NET/issues/20#issuecomment-397423746,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADng6679y4mQJhXztcdvmI2K_DDoOQ5Kks5t8sPYgaJpZM4UCj0K
.

That appears to be the only way to assign associations since the Associations property is read-only.
I did step through the code and added some dummy assignments to DealListHubSpotModel's ToHubSpotEntity and FromHubSpotEntity and added some breakpoints to see if the two methods were actually invoked.  So far they aren't.

Any updates?

@chrassendk Not right now unfortunately - will hope to dig into this soon!

The below will return associatedVids and CompanyIds though perhaps not in the way originally intended. I have been testing this with Deals only.
Add to RequestDataConverter.cs before it checks all the properties
`
        if (expandoDict.TryGetValue("associations", out var associationsData))
        {
            var contactList = new List<long>();
            var companyList = new List<long>();

            foreach (var assoc in associationsData as ExpandoObject)
            {
                if (assoc.Key == "associatedVids")
                {
                    var contList = (IList<object>)assoc.Value;
                    foreach (var co in contList)
                    {
                        contactList.Add((long)co);
                    }
                }
                else if (assoc.Key == "associatedCompanyIds")
                {
                    var coList = (IList<object>)assoc.Value;
                    foreach (var co in coList)
                    {
                        companyList.Add((long)co);
                    }
                }
            }
            var associations = new Api.Deal.Dto.DealHubSpotAssociations();
            long[] contacts = contactList.ToArray();
            long[] companies = companyList.ToArray();
            associations.AssociatedContacts = contacts;
            associations.AssociatedCompany = companies;
            var associationsProp = dtoProps.SingleOrDefault(q => q.GetPropSerializedName() == "Associations");
            associationsProp?.SetValue(dto, associations);
        }

        // The Properties object in the json / response data contains all the props we wish to map - if that does not exist
        // we cannot proceeed

`
Update DealHubSpotModel.cs to add set as per below.
`
    [IgnoreDataMember]
    public DealHubSpotAssociations Associations { get; set; }

`

I think the same issue is still exists in nuget package, that associatedCompany and associatedContacts not mapped.
anyone have idea how to solve this.


feat: single annotation dialog step 2

#450
Implements step 2 for the download annotation dialog

Refactor to pass activeAnnotation instead of annotation map
Adds react-query for client side fetching
Refactor file size fetching to be more efficient

instead of fetching all file sizes for the page, fetch on demand when user views file size in the dialog

Fix navigation for step 2

Demos
https://dev-jeremy-dl-modal.cryoet.dev.si.czi.technology
Configure Download

Download Options
.mrc Direct Download

.mrc Curl

.mrc AWS

.mrc API

.ndjson Direct Download

.ndjson Curl

.ndjson AWS

.ndjson API

.zarr AWS

.zarr API

Hey Jeremy! Looking good!
I have the feedback in the google doc below, can you cross out the ones that are fixed and you can leave a comment for the ones you have questions.
Thank you so much!!
Feedback doc: https://docs.google.com/document/d/1QyIex7NhRj53a2BOzps-HWJAPpXc6UDkyPX8XnQ8M-U/edit#heading=h.wc4bnytgbsc0

@codemonkey800 I noticed that the subhead copy inside all of the AWS CLI tabs still says "AWS S3" -- could you update that as well?


Bug 2047693:[Alibaba] Remove code from getSecurityGroupIDByTags

Each time a security group is created, the Tag is unique. I think it's not I don't think specifying resource groups is necessary.

/bugzilla refresh

/retest

The machine controller log as below
I0128 05:21:54.071310       1 controller.go:76] controllers/MachineSet "msg"="Reconciling" "machineset"="jiwei-505-5bszr-worker-ap-northeast-1b" "namespace"="openshift-machine-api"  E0128 05:22:03.976941       1 instances.go:838] error list resourcegroups: Post "https://resourcemanager.aliyuncs.com/?AccessKeyId=LTAI5tMvCUa483ih4rEyX6Ai&Action=ListResourceGroups&Format=JSON&Name=jiwei-505-5bszr-rg&RegionId=ap-northeast-1&Signature=4kST9M529w658z8NxhAgRzz02OA%3D&SignatureMethod=HMAC-SHA1&SignatureNonce=0a6ad2034b882cd48e67d8d3ea7800bc&SignatureType=&SignatureVersion=1.0&Timestamp=2022-01-28T05%3A21%3A53Z&Version=2020-03-31": context deadline exceeded (Client.Timeout exceeded while awaiting headers) E0128 05:22:03.976975       1 instances.go:394] Unable to determine resource group ID for machine "jiwei-505-5bszr-worker-ap-northeast-1a-5cxxf", err "error list resourcegroups: Post \"https://resourcemanager.aliyuncs.com/?AccessKeyId=LTAI5tMvCUa483ih4rEyX6Ai&Action=ListResourceGroups&Format=JSON&Name=jiwei-505-5bszr-rg&RegionId=ap-northeast-1&Signature=4kST9M529w658z8NxhAgRzz02OA%3D&SignatureMethod=HMAC-SHA1&SignatureNonce=0a6ad2034b882cd48e67d8d3ea7800bc&SignatureType=&SignatureVersion=1.0&Timestamp=2022-01-28T05%3A21%3A53Z&Version=2020-03-31\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)"
So the api timeout not other reason

/retest-required

the aws test failures don't look related to this PR, i think they are just being flaky. we run those tests to ensure that we are not breaking the release payload.

/lgtm


emulator wont work on site

i followed all the steps , and even looked up some other steps to try and fix my issue but still dosent work, please help me out! i can give you my code so you can see if its right and my website, its on my website but i just get a blank white screen? what do i do?

give me url and I'll try

stargaminglive.tk  ...it is on the front page , you will see a white box, i dont know why its doing that

did you see the blank white box i was talking about?

im sorry i ment its a all black box

To play this game, please, download the latest Flash player!

My Code

To play this game, please, download the latest Flash player!

To play this game, please, download the latest Flash player!

I see an error: http://stargaminglive.tk/work/sega.swf not found

try to change 'work/Nesbox.swf' to 'flash/Nesbox.swf' and put Nesbox.swf to the 'flash' folder

its here http://stargaminglive.tk/flash/sega.swf

yes i did all that

:) no, I see this on your page 'swfobject.embedSWF('work/Nesbox.swf', 'emulator', '640', '480', '11.2.0', 'flash/expressInstall.swf', flashvars, params, attributes);'
replace 'work/Nesbox.swf' with 'flash/Nesbox.swf'

see they are in flash folder

next error, the emulator doesn't support .sms roms

ok i downloaed a smd rom of the game, its not working still


Javascript on mobile not running

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report  <!-- Please search GitHub for a similar issue or PR before submitting -->
[x ] Feature request
[ ] Documentation issue or request
[x] Other: <!-- Please describe: -->

Current behavior

Everything works fine, but when I view through mobile, my javascript does not run. It runs fine on chrome desktop, but it doesn't on my chrome mobile. It how's all the HTML but js does not run
Expected behavior

Environment
Browser:

- [ x] Chrome (desktop) version XX
- [ ] Firefox version XX
- [x ] Safari (desktop) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:

- Live Server: <!-- Check which version is installed -->
- Platform:  <!-- Mac, Linux, Windows -->
- Visual Studio Code: <!-- `code --version` -->

Others

I seem to have the same Issue, although I am running Live-server with NGROK to have that public URL, not sure if that would have something to do with.


my reimplement

Hi, I have reimplement your job, https://github.com/lck1201/simple-effective-3Dpose-baseline.
Please consider add it to your README. Thanks!

Will this help in your 2D pose estimation ? https://github.com/rwightman/posenet-python

@basicvisual  thanks, I will try

dear @lck1201,
can you represent more detail about " How to use"?
The data you gave is not downloaded?

@NguyenDangBinh  Hi，you could submit issue under my repo, so I could check it reguarly...
What do you mean 'not downloaded'? Can you open the link?

Thanks a lot for your contribution. What framework is your implementation on? I wanna add a note on my readme indicating that.

@una-dinosauria Hi, it's implemented on Mxnet/Gluon


Mllib feature coverage

Codecov Report

Merging #47 into develop will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##           develop       #47   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           15        16    +1
  Lines         1035      1047   +12
=========================================
+ Hits          1035      1047   +12

Impacted Files
Coverage Δ

src/zero_one/geni/ml.clj
100.00% <ø> (ø)

src/zero_one/geni/default_stop_words.clj
100.00% <100.00%> (ø)

src/zero_one/geni/ml_feature.clj
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Added missing changelog entries for npm buildpacks

The composite buildpack heroku/nodejs should have had entries added for the new npm buildpacks merged in PRs:

#623
#625
#685

@colincasey I disabled auto-merge, since it looks like there should have been a merge conflict in the ## Removed section. Maybe try rebasing this?

weird, the merge didn't complain about a conflict but duplicated the # Removed section twice

CI has been running ~15 mins and some jobs haven't even started yet - due to waiting for instances of the pub-hk-ubuntu-22.04-large runner to be available. It might be the case that switching those jobs to the smaller default runner may end up being quicker (even with fewer CPUs/less memory), due to less wait time. (This could be offset by splitting the tests more and/or reducing the size of some of the test fixtures, so they install fewer packages etc).


Add Real-Time Log Display in GitHub Actions Console

Currently, I am using the GitHub Action provided by Digital Ocean to deploy my application. However, I noticed that unlike the Digital Ocean panel, GitHub Actions doesn't display real-time logs during the deployment. This makes it challenging to track the progress of the deployment and identify any potential issues that may arise during the process.

Request:
I would like to request the addition of a feature that allows displaying real-time logs directly in the GitHub Actions console. This would make the deployment process more transparent and efficient, enabling easier identification and resolution of any issues that may occur.
Benefits:

Improved Visibility: Displaying real-time logs in the GitHub Actions console would provide a clear view of the deployment status, allowing developers to track progress more efficiently.

Quick Diagnostics: With the ability to instantly view logs, it would be easier to identify potential errors or failures during the deployment process, enabling immediate correction.

Enhanced Continuous Integration: By showing real-time logs, continuous integration would be more effective as developers would have precise information about each step of the process.

We've just released v2.0.0 of the action which optionally prints the build and/or deployment logs after the deployment and also surfaces those in an action output for you to use in subsequent steps. It's not real-time for now, but it should fulfill the described goal. Please check it out: https://github.com/digitalocean/app_action/discussions/122.


Processor: HarvestRewards

This PR introduces the processor for the HarvestRewards instruction.
This instruction calculates a holder's share of any unseen total rewards,
then harvests those as well as any unharvested rewards.

@joncinque I found an issue with the way I was updating the unharvested rewards, so the third commit I just recently added addresses it, as well as updates the tests to check for what you were after with updating unharvested rewards when the pool doesn't have enough.


QnAMaker/MSBot needs retry criteria when creating kb

Tool
Name: QnAMaker/MSBot
Version: 1.1.0/4.3.2
OS: Win 10 Build 17763
Describe the bug
See https://github.com/Microsoft/BotBuilder-Samples/issues/1187 for more details.
Summary
MSBot frequently (up to 70%) of the time runs qnamaker create kb ... before QnAMaker Cognitive Service is finished being set up on the backend. This will most frequently result in an "invalid subscription key" error, but may also result in a "QnaRuntimeError - Authorization Failed" or a nondescript "Failed" error.
This can be solved easily by adding a 60+second wait between AZ CLI reporting that the Cognitive Service has been created and running qnamaker create kb .... A better solution might be having QnAMaker API provide some sort of "Backend Ready" status check API that can be programmatically checked, or by giving msbot/qnamaker the ability to retry X times after X seconds if one of those errors is present.
To Reproduce
Steps to reproduce the behavior:

Run the msbot clone services ... command for any sample that uses QnAMaker
It will fail many, but not all times at the qnamaker create kb step

Expected Behavior
msbot clone services ... command to run without failures and QnA KB created successfully.
[bug]

@rokulka Regarding your comment, "Create KB is called immediately after creating QnAMaker Cognitive Service resource, it fails because the backed updates are not complete yet."
Would any of the below API requests allow for checking when the backend is complete (and ready to run qnamaker create kb...)?


4.14.0-0.okd-2024-01-06-084517 monitor-plugin failed

Describe the bug
After updating to 4.14.0-0.okd-2024-01-06-084517 monitor plugin stopped working, from time to time it works for a minute or so, and then goes down again.This happens on 3 of 3 clusters that i updated.
Error:
Failed to get a valid plugin manifest from /api/plugins/monitoring-plugin/
SyntaxError: JSON.parse: unexpected end of data at line 1 column 1 of the JSON data

I have checked logs from cluster-monitoring-operator, console-operator and monitor-plugin pods, but there is nothing useful.
Version
4.14.0-0.okd-2024-01-06-084517
How reproducible
Update to 4.14.0-0.okd-2024-01-06-084517 .
I did one update from 4.14.0-0.okd-2023-11-14-101924, and two from 4.13.0-0.okd-2023-10-28-065448.
Log bundle
Download

OCP bug - see https://issues.redhat.com/browse/OCPBUGS-23516

Exact same problem here, console says "Dynamic plugin/monitoring plugin degraded" every couple of minutes and the "observe" console menu is on and off...

Same here Failed to get a valid plugin manifest from /api/plugins/monitoring-plugin/ SyntaxError: The string did not match the expected pattern.

Looks like we're just waiting on QA to wrap up getting the fix into the 4.14 stream:
https://github.com/openshift/console/pull/13517

Hello,
@vrutkovs Can you tell me on which 4.14 release this got fixed ?
Right now we've tried:
4.14.0-0.okd-2024-01-06-084517
4.14.0-0.okd-2024-01-26-175629
but both have the issue.
Thank you.


[Driver] Denote ABI type in driver invocation

Addresses a FIXME to specify ABI type in the driver invocation.

See https://github.com/apple/swift/pull/439 for additional context.

@modocache As discussed in another PR, we'd need to change the CMake logic to put the library in a different directory.  Do you have access to a Linux machine so that you can test?

There are a few places to look for changes.  One is cmake/modules/AddSwift:44
# Compute the library subdirectory to use for the given sdk and
# architecture, placing the result in 'result_var_name'.
function(compute_library_subdir result_var_name sdk arch)
  set("${result_var_name}" "${SWIFT_SDK_${sdk}_LIB_SUBDIR}/${arch}" PARENT_SCOPE)
endfunction()

or starting a 1102 in the same file:
1102   # On Linux and FreeBSD add the linker script that coalesces protocol
1103   # conformance sections. This wouldn't be necessary if the link was done by
1104   # the swift binary: rdar://problem/19007002
1105   if("${CMAKE_SYSTEM_NAME}" STREQUAL "Linux" OR
1106      "${CMAKE_SYSTEM_NAME}" STREQUAL "FreeBSD")
1107     list(APPEND link_flags
1108         "-Xlinker" "-T"
1109         "-Xlinker" "${SWIFTLIB_DIR}/${SWIFTLIB_SINGLE_SUBDIR}/swift.ld")
1110   endif()

There are quite a few places where SWIFTLIB_DIR (and similar) is referenced.

There's another issue with ABIs that would be nice to have fixed.  In the Foundation build scripts, I have to assume gnueabihf if it's an ARM, otherwise it won't build.

This sounds like something for @jrose-apple.

Sorry to keep this open so long. It's outdated to the point that the FIXME being addressed doesn't even exist anymore!


Add AWS CLI and environment patch

Description of your changes
The changes enable using AWS CLI and use in-cluster ProviderConfig to pass the configuration.

Added AWS CLI to the build
Added valueRef to environment variables parameters
Added air.toml for hot reload

Fixes https://github.com/crossplane/crossplane/issues/4141 https://github.com/crossplane/crossplane/issues/5830
I have:

[x] Read and followed Crossplane's
contribution process. Also see docs.
[x] Added or updated unit tests for my change.

Updated to follow this proposal, allowing access to any context key  or composite value

@clementblaise great job. This will be super useful for many Crossplane users.


On create room, it should go to add members screen directly

When I create a room, the room is created as empty room. I think a empty room doesn't make sense, so always I go to add members screen. Please, could be this step automatic.

Sim6,
it is an expected behaviour.
We're going to implement a second room creation page in which you will add members before creating it.
So, when you will tap on the (+), you will have choice to create an empty room or select members to invite in the created room.

Ok, thanks!


Avoid the need for global install during "npm build"

No-argument "npm link" updates the global package repository, which is unfortunate and sometimes impossible (by default on Linux, the global repository isn't writable without sudo).
This changes a couple things to avoid that:

With --noGlobalInstall, avoids the "npm link" calls in the prepare{Es6,UMD,Additional}DevPackages build tasks
Regardless of the flag, directly links to the temp-install for testing, rather than linking from the global repository

(Maybe the "npm link" calls could be removed entirely instead of flag-disabled, but I wasn't sure if other people used the global linkage in their workflow... I'm only testing "npm run build" / "npm run start" at the moment.) The way shelljs is used in these spots is also changed to be more compact and robust.

I need validation from @sebavan but he is oof for a month so please bear with us :)

@egnor any updates on the comments ?

Ooh sorry the comments look quite sensible I just hadn't circled back to
it. If timing matters I can prioritize it for sure!
On Tue, Jun 8, 2021, 6:52 AM sebavan @.***> wrote:

@egnor https://github.com/egnor any updates on the comments ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BabylonJS/Babylon.js/pull/10295#issuecomment-856787511,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACEKC22IBJAFXA7XCFTCFTTRYOC5ANCNFSM437BS35Q
.

no rush and no worries I ll keep it open for now

pinging @sebavan

@egnor About the console, yup would be great to place them back in as I can see those easily from some monitoring tools :-)
About npm link, I am all in to do it if noGlobalInstall is not passed in. Basically, we rely on it heavily when developping locally. like anywhere in my environment I can use my local babylon or mainly one of them from any other projects. Also the dependency is then working automatically without changing the package json to know where the other babylon packages are.
I ll merge as soon as you ll have both changes in :-)
Thanks again

Thank you :-)


TODO: Implement function in transcript2 class to compute jM and jI sam fields

Output sam files from the STARlong aligner contain two custom tags (jM and jI) that describe whether each splice junction is canonical and where each intron begins and ends. TranscriptClean uses these tags when correcting noncanonical splice junctions. However, not everyone can use STARlong to align their transcripts. Computing the jM and jI fields directly in my script would expand the splice junction correction feature to more people.

This functionality has been implemented and tested.


DateRangePicker enableOutsideDays breaks date selection

Hello,
I'm migrating from v4 to v12.1.2 and I encountered a weird issue which I narrowed down to the use of enableOutsideDays prop. See this demo, when the prop is present, whenever you try to select a date range, the following error occurs : Uncaught TypeError: Cannot read property '2017-07-16' of undefined which corresponds to line 622 of DayPickerRangeController.jsx :
const modifiers = new Set(month[iso]);

Am I missing something or ... ?

Same problem

I guess you could label this as a bug.

Hi @LoicUV @RichardLindhout, I think this should be fixed in v12.2.0

It was def not functioning beforehand though. I believe #575 contains the solution!


Add Citations

Should use the new citation API in https://github.com/qiime2/qiime2/issues/387
(I'm not sure if there are any for this)

There were none.


Garbage collection being blocked?

Hey there,
This is related to the memory leak I mentioned on Twitter a few days ago. The leak itself seems to be have been fixed, but now it seems that garbage collection is never running.
My test setup for analyzing these leaks has been very primitive, so it could be that there is a flaw in my process. In any case, I'll go over the full history to hopefully shed more light on it:
I've been having memory leak problems on my client's NodeCG servers for some time now. Nobody else running NodeCG seemed to have leaks though, and their RAM usage was quite stable. After turning a bunch of things off one-by-one, I realized that the leak was coming from my lfg-sublistener bundle. Further experimentation revealed that it was specifically twitch-irc that was causing the leak. Disabling twitch-irc and relying purely on lfg-sublistener's API polling resulted in stable RAM usage that did not deteriorate over time.
I set up periodic heap dumps using node-heapdump, and these dumps backed up my hypothesis. Each dump had more and more data from twitch-irc in it, and it seemed that this data never got garbage collected. All the data seemed to point to the channelUserData property in the now-removed data.js library.
Schmoopie refactored the offending part of the code and it looked like the leak was fixed. I pulled down his changes onto my test server, and the heap dumps looked clean. No rising RAM usage, no huge pile of channelUserData.
However, when I turned off the periodic heap dumps, the exact same problem returned. RAM usage started steadily rising while chat was active, and it never went down. Baffled, I turned the heap dumps back on. The problem once again went away. It seems that the act of turning on these heap dumps causes the problem to go away.
 Sure enough, this made the problem go away as well. It would seem that the leak itself has been fixed, but Node is for some reason deciding to never run garbage collection.
Below I've attached a screenshot of my server's RAM usage. On the left, you can see the RAM usage of a server running twitch-irc normally, with no heap dumps and no forced garbage collection. The ram usage has been steadily climbing whenever the channel it is listening to goes live and its chat activity picks up. The blue lines are where I'm editing and restarting NodeCG to have periodic forced garbage collection every 30 minutes. After this, RAM usage starts out by rising very fast and peaking at 300MB, but then levels out and stabilizes at about 138MB.

I've tried to do research into Node's garbage collection to understand why this might be happening, but so far I've not been able to find anything that would explain this pattern I'm seeing. Was this something you also saw when profiling twitch-irc on Strongloop?

I also think that the image schmoopie showed me on twitter after he fixed the leak is inaccurate. In it, you see that heap and GC are flat and close to zero, yet RSS is continuing to climb. I observed something similar when using StrongOps, and it seems to be caused by turning on heap profiling.
On the left of the graph, we see NodeCG running lfg-sublistener. On the right, I have turned on heap profiling in StrongOps and the graph goes haywire.

Hello @Lange,
Yes I did notice that and I believe it's related to the event listeners, but I still can't confirm anything yet. I can see the exact same results with 2 servers out of 3 so far, with Strongloop I was able to see the memory leak which is now fixed, but I also saw that the RSS was going up and down, which is normal and depends on the page file, but sometimes it wasn't going down enough. I have a feeling that this is due to an event listener that is including an object and the fact that the object is getting used over again and again, it doesn't get GC'ed properly.
I will need some time to figure this out, for now I'm busy with my job and personal life :/ It is one of my priority though.
Thanks!

Yeah it's weird. I've been trying to attack this from all sides to find the root cause, but haven't been successful yet.
My production setups are just barely stable enough to work right now, so this isn't a disaster situation for me. They're running out of RAM every few hours, but they're stable enough to restart themselves with only about a minute of downtime.
I'll keep working away at this as well, and I'll let you know if I discover anything useful.

I made a fork of twitch-irc that is gutted down to just the parts that I need for lfg-sublistener. The good news is that twitch-irc, when run in a minimal test case, has very stable RAM usage. The bad news is that when run in NodeCG as a whole, something is still slowly leaking.
So I've probably got a leak in my code somewhere, but I'm struggling to find it because of the aforementioned heapdump problem. I'll keep looking.

Is there a temporary workaround for the time being? Maybe force GC every 15-30 minutes?

Been working on this for a while now, this is what I have found so far and everyone is welcome to replicate my steps to confirm the results. I tested everything on Windows and Ubuntu 14.04.
Many times I tried using profilers and many memory management applications for Node and as soon as I started profiling, the RSS went up like crazy and this is a problem for me when my goal is to see if there is a memory leak or not. The only memory leak I was able to fix was the channelUserData thing, but now we are talking about the RSS going up, which might be due to another memory leak. I decided to test with different modules, I tried using a very basic express application and another one with node-irc, both of them started with 40mb RSS and when I started the profiler, they increased to 170mb RSS. Anyways, I used node-heapdump and started creating heapdumps every 60 secs and found out that it might be because of this function. Everywhere in the memory, when there was an array somewhere, this function was added to it. I removed it and changed the mods.clean(''); line in client.js to a for loop and ran the script again and checked the heapdump to make sure this function is no longer being used. I was also looking for data not being garbage collected, but found nothing in about 20-25 different heapdump files. Now that it was gone, I removed node-heapdump and started profiling my own way. I'm using this in my code in client.js and it is still running as I am writing this post..
function format (val){
    return Math.round(((val / 1024 / 1024) * 1000) / 1000) + 'mb';
}

setInterval(function () {
    console.log('Memory Usage :: ' +
            ("\tRSS: " + format(process.memoryUsage().rss)) +
            ("\tHeap Total: " + format(process.memoryUsage().heapTotal)) +
            ("\tHeap Used: " + format(process.memoryUsage().heapUsed))
    );
}, 5000);

I also tried running the script with --trace-gc and I can confirm that twitch-irc gets GC'ed.. the delay is just higher with Node v0.12. Node is garbage collecting twitch-irc every 10mins or so, the first GC increases the RSS a bit and then it is stable, the heap used increases until it gets GC'ed and goes back down. The following screenshot is an instance that has been running the module for 72 hours on 15 channels, it is not running with strongloop or any profilers, just the code I provided above..

"Heap used" goes up and down as you can see and RSS isn't that high for an application that is analyzing so many lines right now (LOT of chat activity). Everytime there is a message on Twitch, the module receives 4-5 messages. Twitch sends informations with jtv before every message and the module throw it back to you in the user object of the chat and action listeners. Also, Node is single threaded, which might be the cause of high memory consumption when being on many many channels.
The "Heap used" was going up and up while writing this post, RSS is going up too and then suddenly, see what happened:

I will still run my tests on different servers, but maybe it is fixed now. I am not closing this issue as long as it is not confirmed for good.


Only write to the manifest during dep init

This closes #454. Now the only command that writes the manifest is dep init. This does leave dep remove and dep ensure --override in an inconsistent state until the new command set is implemented.

Excellent, thanks!


Remove edit mode toggle

[Issue] https://github.com/Samsung/TAU-Design-Editor/issues/209
[Problem] Edit mode toggle is currently not needed.
[Solution] Dissable edit mode button from toolbar
Signed-off-by: Pawel Kaczmarczyk<EMAIL_ADDRESS>

PR merged


Program log: Error: Account is frozen

We've updated our NFTs to Programmable NFTs. Since then, we updated our SDKs for our Gemfarm. Our NFTs are able to be unstaked, but they are now stuck in our gembank. Anyone have suggestions for this issue?
Transaction simulation failed: Error processing Instruction 0: custom program error: 0x11
Program bankHHdqMuaaST4qQk6mkzxGeKPHWmqdgor6Gs8r88m invoke [1]
Program log: Instruction: WithdrawGem
Program TokenkegQfeZyiNwAJbNbGKPFXCWuBvf9Ss623VQ5DA invoke [2]
Program log: Instruction: Transfer
Program log: Error: Account is frozen
Program TokenkegQfeZyiNwAJbNbGKPFXCWuBvf9Ss623VQ5DA consumed 4294 of 169386 compute units
Program TokenkegQfeZyiNwAJbNbGKPFXCWuBvf9Ss623VQ5DA failed: custom program error: 0x11
Program bankHHdqMuaaST4qQk6mkzxGeKPHWmqdgor6Gs8r88m consumed 34908 of 200000 compute units
Program bankHHdqMuaaST4qQk6mkzxGeKPHWmqdgor6Gs8r88m failed: custom program error: 0x11

All you need to do now is update (lib\gem-farm\common\gem-bank\index.ts) with:

However you will now be stuck with me at Signature verification failed.

@hercules-sensei  All you need to do now is update (lib\gem-farm\common\gem-bank\index.ts) with:
https://github.com/gemworks/gem-farm/blob/main/app/gem-farm/src/common/gem-bank/index.ts
However you will be stuck with me at Signature verification failed. More to follow...

@hercules-sensei All you need to do now is update (lib\gem-farm\common\gem-bank\index.ts) with:
https://github.com/gemworks/gem-farm/blob/main/app/gem-farm/src/common/gem-bank/index.ts
However you will be stuck with me at Signature verification failed. More to follow...

Exactly what you said happened. But feel closer to the finish line, will update if we crack it.

we are thinking about converting to pnfts also, we considered seeing if it was possible to force unstake all NFTs in the gemfarm before we converted


incompatibility between pip V22 json output and sqlitebiter

Describe the bug
pip V22 has a new inspect command which produces a json output. https://github.com/pypa/pip
using sqlitebiter to create a sqlite database with the output raises an exception
Expected behavior
convert json output to sqlite atabase
To Reproduce

activate a virtual environment
python3 -m pip install pip --upgrade with several packages
pip install a few pacakges
pip inspect -local >json_output.json
sqlitebiter file json_output.json table

Environments
Please complete the following information:
- uname: Linux
- Python version: CPython 3.8.10
- sqlitebiter version: 0.36.1
- platform version: Ubuntu 20.04
- SimpleSQLite version: 1.3.0
- pytablereader version: 0.31.3

Installation Method: pip / deb package  / apt(PPA) / brew (choose one of these)

Additional context
trace is in https://bpa.st/J5SQ
json file renamed in .txt
json_local_requirements.txt

@piscvau
You might convert the JSON value to convertible format with jq command.
e.g. https://github.com/thombashi/sqlitebiter/issues/58#issuecomment-532936700


Support unnamed requirements in uv tool install

Summary
This PR adds support for (e.g.) uv tool install git+https://github.com/psf/black.
Closes https://github.com/astral-sh/uv/issues/4664.

👍 I just copy-pasted from the others, agreed.


Data visualization project idea

I just found this site for calculating college costs, and all of it is open source on GH.
http://www.college-costs.com/
https://github.com/cuthbertLab/collegeCosts
I'm not sure if we could reach out to the creator of this to see about his thoughts on this being a data vis challenge, but I think that campers could come up with some really cool ways of displaying this information.

@brianamarie great idea. We can consider doing this later through curriculum expansion.