Simplebrowser crashes accessing www.os2world.com

Console output is:
`[151:4:0113/122057.387000:ERROR:disk_data_allocator.cc(174)] DISK: Cannot write to disk. written = -1 FILE_ERROR_FAILED
[151:3:0113/122057.387000:ERROR:disk_data_allocator.cc(174)] DISK: Cannot write to disk. written = -1 FILE_ERROR_FAILED
`
Followed by an exceptq log (attached)
63c14cb0-0094_01-SIMPLEBROWSER-exceptq.txt

Looking at the trap, cd1a30d060fc2b650af4eba7ef2789d21b698ce8 should fix it.


(PUP-6124) Update Acceptance tests for huaweios

Waiting for CLA signature by @LuvCurves
@LuvCurves - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppet.com/
Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppet.com/community/trivial_patch_exemption.html

CLA signed by all contributors.

@LuvCurves looks good. Could you update the commit summary about why the change to double quotes is necessary?

@joshcooper Done.

@LuvCurves sorry I misspoke. Could you update the body of the commit message? I'm not going to remember the details of why we made this change in the future.

@joshcooper Done.

@puppetlabs-jenkins retest this


send --text - reads in all the text before sending

I wanted to stream over a pipe, so I tried piping into wormhole send --text -, but that seems to read in all the data first, so it ran out of memory.

Yeah, sorry, the send-text protocol currently only sends a single message, and doesn't support streams.
When we get the "Dilation" protocol (#312) implemented, then streams should be pretty easy, and --text - won't need to buffer everything first.


Fix invalid links in the Markdown files

Reason/Context
We have recently started validating links in the markdown files across all of the repositories across the organization.
As I can see in the latest check by the time of writing this issue, there are some invalid links in some markdown files that need fixing.
How to Resolve the issue?

If you haven't read our contributions guide, please give it a read to see how contribution works in AsyncAPI.
open the latest run of the Check Markdown Links and fix them in your own fork (you can test if the check passes by running the Check Markdown links (Weekly) in your own fork).
Open a Pull Request with the changes.

How to fix the links

If the link is moved to a new place, edit the link to point to the appropriate place.
If it is not supposed to be checked or it is a sample URL, add it to the ignorePatterns section of mlc_config.json at the root of the repo.
If you still don't know what to do with a specific link, ask the maintainers for help.

Examples
If you still have some issues, please check some example fixes in other repositories:
https://github.com/asyncapi/spec/pull/769
https://github.com/asyncapi/parser-js/pull/540
https://github.com/asyncapi/glee/pull/288
https://github.com/asyncapi/java-spring-template/pull/219

/gfi docs


Use Spring Data Range as an alternative to PostgreSQL range type

Currently, to map the range Hibernate types, there are already 2 options

Range class from Hypersistence Utils
Range class from Guava

I would like to add Range class from the Spring Data as a third option.
This would facilitate using the same Range class for use with Spring Data JPA and Spring Data Elasticsearch and perhaps in other Spring Data projects.
The changes with tests are made only to the 6x modules, as the 5x modules uses the Spring Data commons 2.7x where the version of the Range class is incompatible when using Date/Time types e.g. LocalDate, ZonedDateTime etc.

Thanks, I merged it.


CLOUDSTACK-9765: centos: packaging: fix missing path in spec file

Related to 6051fa503a2da7b96256f21e6b7c42aa1f3cb01e
/cc @rhtyd (shapeblue)

Thanks @resmo can you re-target this PR for 4.9 branch? Let's get this in both 4.9 and master.

@rhtyd done. changed base to 4.9 ;)

@blueoranguten package

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✖debian. JID-468

@borisstoyanov is there a way to get the packages build by @blueorangutan?

Unfortunately no at this stage, but we can talk about this with @rhtyd

LGTM. @karuturi not a blocker, but then we'll need to document the workaround (manual fix) in the release notes. If RC1 has any blockers, let's include this in RC2.

This LGTM too, we hit this issue during our upgrade


TypeError: tensor is not a torch image.

thank you for your amazing work. But I met a little problem. when I run  python3 gendata.py ./images, I got this
Traceback (most recent call last):
File "gendata.py", line 100, in
device, G, encoder, dlatent_avg, faceimg, truncation_psi=0.5, num_steps=1
File "/home/ubuntu/extend/dq/StyleFlowPytorch-main/webui/imageencoder.py", line 63, in encode_real_images
images = torchvision.transforms.Normalize(mean=0.5,std=0.5)(images/255.)
File "/home/ubuntu/anaconda3/envs/psp_env/lib/python3.6/site-packages/torchvision/transforms/transforms.py", line 175, in call
return F.normalize(tensor, self.mean, self.std, self.inplace)
File "/home/ubuntu/anaconda3/envs/psp_env/lib/python3.6/site-packages/torchvision/transforms/functional.py", line 209, in normalize
raise TypeError('tensor is not a torch image.')
TypeError: tensor is not a torch image.
Could you tell me how to solve it. thanks a lot

Seems impossible.  torch.Tensor(images).to(device) will turn images from numpy to torch tensor. You can add print(images) to check it after torch.Tensor.
images = torch.Tensor(images).to(device)
images = torchvision.transforms.Normalize(mean=0.5,std=0.5)(images/255.)

thanks for your reply. Could you tell me your Pytorch and cuda's version?

RTX3090
NVIDIA-SMI 470.63.01    Driver Version: 470.63.01    CUDA Version: 11.4
efficientnet-pytorch==0.7.1
pytorch-lightning==1.5.10
torch==1.9.0+cu111
torchaudio==0.9.0
torchcontrib==0.0.2
torchdiffeq==0.2.2
torchfile==0.1.0
torchmetrics==0.7.3
torchnet==0.0.4
torchvision==0.10.0+cu111


Document dependsOn a Component resource

Document dependsOn a Component resource.  (A new feature in 0.17.1). See https://pulumi-community.slack.com/archives/C84L4E3N1/p1551945676230900.

Closing out stale issue. This is documented in https://www.pulumi.com/docs/intro/concepts/programming-model/


Add a replace_types option to ConfigDict

Change Summary

Related issue number
Fixes #6045
Introduced new config option replace_types that allows to patch fields annotations after class creation:
LaxStr = Annotated[str, BeforeValidator(str_validator)]

class LaxStrBase(BaseModel):
    model_config: ConfigDict = {'replace_types': {str: LaxStr}} # !!!!!!!!!

class Data(LaxStrBase):
    f: str
    i: str
    d: str

user = Data(f=1.1, i=1, d=Decimal('1.1'))
# will equal to {'f': '1.1', 'i': '1', 'd': '1.1'}

this also allows a pydantic global override string behaviour:
BaseModel.model_config['replace_types'] = {str: LaxStr}

Checklist

[x] Unit tests for the changes exist
[x] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[ ] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

I like this idea! My main concern is the implementation, in particular, that it is done by replacing fields and only for BaseModel. It seems to me like it could be made to work for dataclasses and typeddict without much more effort.

replacing fields and only for BaseModel.

not sure I understand... - it should work for global (BaseModel), some base model (and only child classes) and a single class:

# 1 this changes globally
BaseModel.model_config['replace_types'] = {str: LaxStr}

# 2 this changes types only for child classes inherited from LaxStrBase

class LaxStrBase(BaseModel):
    model_config: ConfigDict = {'replace_types': {str: LaxStr}}

# 3 and for one single model:

class MyModel(BaseModel):
    model_config: ConfigDict = {'replace_types': {str: LaxStr}}
    a: str
    b: str
    x: OtherStr

I mean:
from typing_extensions import TypedDict
from pydantic import ConfigDict
from pydantic.dataclasses import dataclass

@dataclass
class Model1:
    x: str
    __pydantic_config__ = ConfigDict(replace_types={str: int})

class Model2(TypedDict):
    x: str
    __pydantic_config__ = ConfigDict(replace_types={str: int})

A couple more things to think about:

Handling generic types (replace_types={List: Dict}), how can users handle replacing the origin vs. replacing a parametrized version
Recursive replacements, both that {str: Annotated[str, ...]} should cause recursive replacement and that {str: int, int: float} should (or not?) lead to str being replaced with float.

Handling generic types (replace_types={List: Dict}), how can users handle replacing the origin vs. replacing a parametrized version

I do not see any practicality in such a case, but even if it used - it will not replace concrete annotation with origins because the equal statement is used, and it will not work for cases without passed origins:
List != List[str]
List != List[Any]
str != Annotated[str, 'meta']
ans so on

Recursive replacements, both that {str: Annotated[str, ...]} should cause recursive replacement and that {str: int, int: float} should (or not?) lead to str being replaced with float.

Again - practicality is questionable, but yeah why not let user loop all the way str -> int -> float for some unknown reason (just keep keys of the dict in a required sort order)
I think the main practicality here is in enhancing types with extra Annotated markers or dates with datetimes with timezones. I do not see any useful case where you replace all your strings with integers or any other unrelated to each other types.
I guess if you really need some magic with handling container types and origins - the replace_types can be a callback function that takes class and FieldInfo and patches is it in any imaginable way:

def patch_strings(cls: BaseModel, field_name: str, info: FieldInfo):
    if info.annotation == str or info.annotation.origin == str:
         info.annotation = LaxStr
         info.summary = 'it can take numbers ;)'
    if name == 'age':
         info.annotation = Annotated[int, Range(0, 100)]

class LaxStrBase(BaseModel):
    model_config: ConfigDict = { 'replace_types': patch_strings }

Thanks for thinking about these things. I agree these aren't dealbreakers for the feature, I just wanted us to think about them a bit, thank you for keeping the focus on the core use cases this is solving. Practically speaking I see the value in this and think it'd be totally okay to start with no generics or recursive replacement support. I do think we need to get this to work with TypedDict, dataclasses at least, those two can also have a Config. Do you want to give that a shot?

@adriangb back from #6045
I think patching FieldInfo.annotation might not perfect - I'm trying to  do monkey patching on other large project but stuck with this:
from pydantic import BaseModel, Field, BeforeValidator
from typing import Annotated
from pydantic.v1.validators import str_validator
from pydantic._internal import _fields
from pydantic._internal import _model_construction

LaxStr = Annotated[str, BeforeValidator(str_validator)]

# patching orig_collect_model_fields

orig_collect_model_fields = _fields.collect_model_fields

def patched_collect_model_fields(*a, **kw):
    fields, classvars = orig_collect_model_fields(*a, **kw)
    for f in fields.values():
        if f.annotation == str:
            f.annotation = LaxStr
    return fields, classvars

_model_construction.collect_model_fields = patched_collect_model_fields

class Data(BaseModel):
    gender: str = Field('F', max_length=1)

print(Data(gender='M'))

gives:
gender
  Value should have at most 1 item after validation, not 1 [type=too_long, input_value='M', input_type=str]
    For further information visit https://errors.pydantic.dev/2.0.3/v/too_long

probably some extra thing need to be done ?

Indeed that does seem brittle. I think this needs to happen during core schema generation.

@vitalik is this PR still on progress or it's ready for review?

I think it may still need some work but primarily it requires our input

closing as we're working on another solution.

@samuelcolvin can you please share what's planned to solve this issue?


tests.unit._rpc_test.RPCTest shouldn't have to have a tearDown method

All it does is delete things that should be deleted anyway.
When I run the test without tearDown method on my terrestrial workstation, it passes, and when I run it without tearDown on my workstation in the cloud, it hangs.

Maybe a destruction order problem. Maybe.

This was fixed in pull request 6857.


eclipse fails to install with "missing filename"

On my NixOS unstable system, when eclipses.eclipse-platform or eclipses.eclipse-sdk is added to nixpkgs.config.packageOverrides on the system configuration file, nixos-rebuild fails with the message
# nixos-rebuild switch
[...]
eclipse/readme/readme_eclipse.html
missing filename
builder for ‘/nix/store/i8lfzisbx4i6vq5z84gb9av337b3m2m0-eclipse-platform-4.5.2.drv’ failed with exit code 1
[...]

Maybe fixed by https://github.com/NixOS/nixpkgs/pull/15058

Yes, it seems to be fixed by #15058.

Thanks for testing, I merged it now.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/760 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit dd39cfb:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-02-02-03-14-38 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Display save button on companies, add canCreate flag to companies/sites/site areas

Temporarily display 'save' button to fix bug
Need to further discuss about the way to fetch/set canCreate

Temporarily display 'save' button to fix bug

I think by scratching your head a bit more, you can find a proper fix:
*ngIf="canUpdateCompany" miss a condition to handle the update authorization, the proper fix is to find the right one.

@Fonsu As discussed via email, pass the available canCreate, canUpdate that you have in the list to the pop-up and use them.

@Fonsu Any news on this one?

@Fonsu Any news on this one?

I'm done with companies, can you take a look? At the moment I'm also replicating these changes to Sites and Site Areas, need to check if this approach will cover all other use cases.
createWithAuthorizations is a temporary method to avoid changing 10+ create actions with potential regressions but my idea would be to remove the original create method eventually, once all use cases are covered and double checked.

@Fonsu Any news on this one?

I'm done with companies, can you take a look? At the moment I'm also replicating these changes to Sites and Site Areas, need to check if this approach will cover all other use cases.
createWithAuthorizations is a temporary method to avoid changing 10+ create actions with potential regressions but my idea would be to remove the original create method eventually, once all use cases are covered and double checked.

If you want me to take a look, move your issue to To be reviewed ;-)

@Fonsu I finally merged it to have a better readability on the next changes.
I tested it and I didn't find any showstopper.
Other devs will also test their code with these changes


Fix disconnect issues

Issues:

When the agent disconnects, the container uses 100% of CPU and does not respond.
When Connection Closed is received, the thread is not closed properly

Solution
Implement a separate cleanup method to take care of the pipes and use an event to signal when the program should disconnect. In case the agent disconnects, the program enters a waiting state with a delay of 1 second until the agent comes back online.
Definition of Done
The PR shall be merged only if all items mentioned in CONTRIBUTING.md have been followed. In case an item is not applicable as described, please provide a short explanation in the description.

[x] All steps in CONTRIBUTING.md have been handled

@GabyUnalaq, let's directly bump the version up for another rc.


Adding commas as the separator between per elements makes it easy to …

…copy from numpy outputs.
Related to issue #28
Testing logs
❯ tsc
❯ node out/test/runTest.js
Found existing install in /Users/vela/WorkDir/vscode-pydata-viewer/.vscode-test/vscode-darwin-arm64-1.89.1. Skipping download
2024-05-24 19:42:49.982 Electron[26936:1092594] WARNING: Secure coding is automatically enabled for restorable state! However, not on all supported macOS versions of this application. Opt-in to secure coding explicitly by implementing NSApplicationDelegate.applicationSupportsSecureRestorableState:.
[main 2024-05-24T11:42:50.182Z] update#setState disabled
[main 2024-05-24T11:42:50.184Z] update#ctor - updates are disabled by the environment
2024-05-24 19:42:50.595 Code Helper (Renderer)[26941:1092707] CoreText note: Client requested name ".NewYork-Regular", it will get TimesNewRomanPSMT rather than the intended font. All system UI font access should be through proper APIs such as CTFontCreateUIFontForLanguage() or +[NSFont systemFontOfSize:].
2024-05-24 19:42:50.595 Code Helper (Renderer)[26941:1092707] CoreText note: Set a breakpoint on CTFontLogSystemFontNameRequest to debug.
Loading development extension at /Users/vela/WorkDir/vscode-pydata-viewer
Started local extension host with pid 26980.

  Extension Test Suite
    ✔ Sample test
  1 passing (2ms)
[main 2024-05-24T11:42:51.319Z] Extension host with pid 26980 exited with code: 0, signal: unknown.
Exit code:   0
Done

Thank you for your PR!


faster and nothrow random sample for RHT

Btw, why array container is used in RHT?

Btw, why array container is used in RHT?

We may want to ping @DmitryOlshansky for this. Otherwise, @9il feel free to merge the changes yourself.

i did not change the algorithm, but RNG library.
@ljubobratovicrelja Do you have an visual example for RHT. RNG library is new, and it will be nice to have a proof that this PR does not introduce quality regression.

Do you have an visual example for RHT. RNG library is new, and it will be nice to have a proof that this PR does not introduce quality regression.

Yes, the example Dmitry's made shows the visual. Running examples/rht should create results similar as this one.

Yes, the example Dmitry's made shows the visual. Running examples/rht should create results similar as this one.

OK, it is a time for my first DCV installation :) How I can do it for OS X?

How I can do it for OS X?

Let me merge #93. Otherwise you'll have linking issues.

Let me merge #93. Otherwise you'll have linking issues.

Do I need to install some libs before run dub for RHT example?

Let me merge #93. Otherwise you'll have linking issues.

You're good to go.

Do I need to install some libs before run dub for RHT example?

rht example links to whole dcv, but it needs only dcv:core. If you want, you can change it so it links only to dcv:core, and commit that here. That way you wont need anything preinstalled.
And in case you ever need ffmpeg and glfw, you'd might want to install ffmpeg using the travis script, and glfw3 should be ok to install with homebrew.

Btw, why array container is used in RHT?

Can't recall the details but I think I needed reference semantics for the array, maybe it's possible with built-in arrays I just didn't spent too much time on it.

@9il I'd like to draft a new release with recent changes. Would you like me to wait for this to be merged?

@9il I'd like to draft a new release with recent changes. Would you like me to wait for this to be merged?

No, this is not urgent PR

Merging this one... After full 5 years and some. :) Things are not working throughout the whole library, also rht examples do not compile because of changes within mir, so a lot of work to do.


clean out JACDAC of CODAL

We still have the legacy JACDAC implementation in CODAL. How much of it do we still need?

Good question! I believe all we're using now is the single wire serial abstraction and I put that in common codal, not in the JACDAC folder. Will remove!

@mmoskal Do you think anything different?

nuke it


KAFKA-9491; Increment high watermark after full log truncation

When a follower's fetch offset is behind the leader's log start offset, the follower will do a full log truncation. When it does so, it must update both its log start offset and high watermark. The previous code did the former, but not the latter. Failure to update the high watermark in this case can lead to out of range errors if the follower becomes leader before getting the latest high watermark from the previous leader. The out of range errors occur when we attempt to resolve the log position of the high watermark in DelayedFetch in order to determine if a fetch is satisfied.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

1 job passed, 1 had 3 flaky failures:

kafka.admin.DeleteConsumerGroupsTest.testDeleteWithMixOfSuccessAndError
kafka.admin.DescribeConsumerGroupTest.testDescribeGroupWithShortInitializationTimeout
org.apache.kafka.streams.kstream.internals.KStreamImplTest.shouldSupportTriggerMaterializedWithKTableFromKStream

Merged to trunk, 2.5 and 2.4. There were some conflicts when I tried to cherry-pick to 2.3, so leaving that @hachikuji.


track Scala.js master

not to be merged until a test run passes

test run: https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-integrate-community-build/167/

test run green. merging

@sjrd FYI


Is there a way to delete a cache?

I wanted to delete a previously made cache, is there a way to do that? or the only way is to removeAll from it?

Setting the cached object to nil removes the object from the cache.
cache["name"] = nil


ReaderAttributes.h does not include Guid.h

I'm just getting a project set up, and encountered the following build error.
liam@drungus fc (master #)$ make
c++ -std=c++11 -MD -MP -Os -ggdb -fPIC -fno-common -fvisibility=hidden -Wall -Werror -Wundef -I./vendor/Fast-RTPS-v1.0.0-1415-g7b323d1/build/install/include -c src/listener.cpp -o src/listener.o
In file included from src/listener.cpp:1:
In file included from src/listener.h:7:
In file included from ./vendor/Fast-RTPS-v1.0.0-1415-g7b323d1/build/install/include/fastrtps/attributes/SubscriberAttributes.h:19:
./vendor/Fast-RTPS-v1.0.0-1415-g7b323d1/build/install/include/fastrtps/attributes/../rtps/attributes/ReaderAttributes.h:83:2: error: unknown type name 'GUID_t'
        GUID_t guid;
        ^
1 error generated.
make: *** [src/listener.o] Error 1

If ReaderAttributes.h requires GUID_t, it seems like it should include Guid.h.
I worked around it for now by adding #include "fastrtps/rtps/common/all_common.h" before I include ReaderAttributes.h

Hi @liamstask.
This issue was solved in @d26c1a5. Thanks


Added rolling restart task when the RKE2 version changes

Description
Issue #61 - When changing RKE2 version, the role installs the new version, but doesn't restart the rke2-server service for it to take effect.
Added a rolling service restart to each host, with a check to make sure k8s is ready on all nodes before starting the next.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Small minor change not affecting the Ansible Role code (Github Actions Workflow, Documentation etc.)

How Has This Been Tested?
Tested locally on a 3 node cluster. Downgraded and upgraded

Thanks! It looks good.

@twentybit : The rolling restart task will never run if the playbook has failed. For example: In case, if the playbook starts to hang at the Start RKE2 service on the rest of the nodes task. The rke2 binary file on the nodes is in new version, however, version of node is still in old version when showing in runtime with kubectl.  To handle this case, I suggest to correct the task as bellow:
  ansible.builtin.shell: |
    set -o pipefail
    #    {{ rke2_bin_path }} --version | grep -E "rke2 version" | awk '{print $3}'
    {{ rke2_data_path }}/bin/kubectl --kubeconfig /etc/rancher/rke2/rke2.yaml get nodes {{ inventory_hostname }}|tail -n1|awk '{print $NF}'
  args:
    executable: /bin/bash
  changed_when: false
  delegate_to: "{{ active_server | default(groups[rke2_servers_group_name].0) }}"
  register: installed_rke2_version
  when: '"rke2-server.service" in ansible_facts.services'


Fixed model card data type

Description
Updated data type for the compare models input data type.
There was a validation error occurring on model compare request since terrarium taskrunner sends the input payload with { model_cards: [...] } while this service expects { cards: [...] }.
This change fixes the issue.
Related PR: https://github.com/DARPA-ASKEM/terarium/pull/4024
Resolves #(issue)

Didn't realize this was an external change. @jryu01 Let's change the hmi-server side of it.

Yeap then I'll make the change in the hmi-sever side

The change will be made in hmi-sever closing the issue.


add go version of update script

Please give me your thoughts! This isn't perfect code, but getting your eyes on it early is important.
The parallelism is limited to within one (distro,release,arch,flavor) line. This is to stop processing a "line" when BTF is detected in a kernel. The assumption is future kernels will also have BTF.
Some additional notes:

I'm using tar as a command for the compression, because using Go libraries resulted in much worse compression ratios.
The github.com/therootcompany/xz library is much faster than github.com/ulikunitz/xz
I wasn't able to access amazon package repos from outside an instance

very nice work @brycekahle. I'm accepting this as is, and I can start suggesting changes in PRs and asking your reviews, and things like that. Then I can add the update script to automation again.

I created a simple makefile in the root directory and moved the btfhub tool to cmd/btfhub structure. "make btfhub", "make clean" and voilá. Will go through the tool, automate its execution, and the pushes to archive, etc.
Will send some PRs your way during this work. Please let me know if any change affects you badly and we can discuss them. Thanks!


Fixes issue with metadata not being available on initial load

This moves observeMeta() to after createDataObservable() so that the subscription path is ensured to be loaded when the gauge is initially rendered.  Without this, zone data may not be available, causing highlights to not render.

I'll push a release today. Mind if I tag and list you as a contributor?

Not at all, thanks!
On Mon, Aug 5, 2024, 11:59 AM David G @.***> wrote:

I'll push a release today. Mind if I tag and list you as a contributor?
—
Reply to this email directly, view it on GitHub
https://github.com/mxtommy/Kip/pull/450#issuecomment-2269712723, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AATELJKFY6MJZVFMK3IJBO3ZP7DQJAVCNFSM6AAAAABL65XLHKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRZG4YTENZSGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Rebase proxy

Fixes #154

@sebastienvas @ayj the build job is deadlocked: https://istio-testing.appspot-preview.com/job/manager/job/presubmit/415/console any ideas?

Jenkins job manager/presubmit passed

Codecov Report

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

@@           Coverage Diff           @@
##           master     #183   +/-   ##
=======================================
  Coverage   31.45%   31.45%
=======================================
  Files          19       19
  Lines        1758     1758
=======================================
  Hits          553      553
  Misses       1154     1154
  Partials       51       51

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 662cb65...9ca9aca. Read the comment docs.


Cannot git cesium native using instructions on webpage

I tried git clone git@github.com:CesiumGS/cesium-native.git --recurse-submodules in a command windows
and I get the following error message
Cloning into 'cesium-native'...
The authenticity of host 'github.com (<IP_ADDRESS>)' can't be established.
ED25519 key fingerprint is SHA256:+DiY3wvvV6TuJJhbpZisF/zLDA0zPMSvHdkr4UvCOqU.
This key is not known by any other names.
Are you sure you want to continue connecting (yes/no/[fingerprint])? y
Please type 'yes', 'no' or the fingerprint: yes
Warning: Permanently added 'github.com' (ED25519) to the list of known hosts.
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
If I download the zip file from git and try to CMake it fills my C drive then then aborts
Bruce

@bclay1297 have you configured your git client for ssh access to github?
https://docs.github.com/en/authentication/connecting-to-github-with-ssh/about-ssh
If you have trouble getting SSH to work, you could always try cloning over https instead:
git clone --recurse-submodules https://github.com/CesiumGS/cesium-native.git

I'm closing this because it's either a problem with your configuration or a connectivity problem with GitHub, and in either case not something that we can fix.


Should probably run with strictDeps = true

It is very easy to trigger needless rebuilds of dependencies between multiple cargo inheritance steps, by just introducing seamingly unrelated binary, only because by default mkDerivation seems to make dependencies of your build inputs also available in the derivation.
So I had a one off test, imported some extra binary in nativeBuildInputs just for that one step, and I see openssl-sys and downstream crates are rebuilding because of that. Turns out that binary was using a some library which is them being added to PKG_CONFIG_PATH, which makes build.rs of openssl-sys want to rebuild. Tricky to debug.
I was told on NixOS Matrix channel that using strictDeps = true; is generally a good idea, so I'm going to do just that, but crane could do that by default, given how it is used. Might save some users lots of trouble, at the minor cost of having to list all direct build inputs.

Haven't used it myself before, what are the impacts of setting strictDeps? I'm assuming it will change what things are actually set in the various environment variables, but I'm not fully certain whether that might "break" existing derivations
FWIW adding a new nativeBuildInput will cause a rebuild anyway (since that input will show up as an environment variable to the derivation), though not including it in the pkg-config variables would avoid potentially invalidating build-script caches

I'm assuming it will change what things are actually set in the various environment variables

I'm not entirely sure. I've found:

StrictDeps enforces desirable properties like outputs not retaining references to build tools. It also prevents mistakes that break cross-compilation.

It seems to me that without it Nix mkDerivation's infer a lot of stuff.

FWIW adding a new nativeBuildInput will cause a rebuild anyway (since that input will show up as an environment variable to the derivation

Yes, but it will not trigger cargo rebuilding already built dependencies, which right now does happen. E.g. I build the whole workspace, inherit the result into a test where I import some binary, and bam, I see the project being rebuilt again.

whether that might "break" existing derivations

It does. I started using it and had to move certain things between buildInputs and nativeBuildInputs (pkg-config has to be in nativeBuildInputs or cargo will not find stuff).

Although I do agree that strictDeps is probably desirable to always have on going forward, I worry that this is a departure from being a thin wrapper around mkDerivation (i.e. I worry people will be confused why stuff doesn't work with mkCargoDerivation). Though maybe we could highlight it in all the examples as a start :thinking:

I think you're right. It is confusing, especially when it happens implicitly.


video stylize always generate the same style video?

hi and thanks for ur great work,
First Step:
_animatediff stylize create-config YOUR_SRC_MOVIE_FILE.mp4_
Second Step:
I've try about 10 checkpoints by edit  'path' in prompt.json, then
_animatediff stylize generate STYLYZE_DIR -L 16_

but animatediff seems always generate the same style video,

Can you help me think about where the problem might be?
thanks

yes sure!
you can see logs  below:
13:39:13 INFO     Using generation config: stylize\2023-10-05T23-05-50-sample-mistoonanime_v20\prompt_tmp.json                                                                                cli.py:288
INFO     is_v2=True                                                                                util.py:346
INFO     Using base model: runwayml\stable-diffusion-v1-5                                                                                cli.py:306
INFO     Will save outputs to ./stylize\2023-10-05T23-05-50-sample-mistoonanime_v20\2023-10-07T13-39-13-hangzhou-xxmix9realistic_v40                                                                                cli.py:314
Preprocessing images (controlnet_tile)  94% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╺━━━━━━━━━━━━━━━ 15/16  [ 0:00:00 < 0:00:01 , 26 it/s ]
Saving Preprocessed images (controlnet_tile) 100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 16/16  [ 0:00:01 < 0:00:00 , 8 it/s ]
Preprocessing images (controlnet_ip2p)  94% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╺━━━━━━━━━━━━━━━ 15/16  [ 0:00:00 < 0:00:01 , 27 it/s ]
Saving Preprocessed images (controlnet_ip2p) 100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 16/16  [ 0:00:01 < 0:00:00 , 8 it/s ]
13:39:18 INFO     Checking motion module...                                                                                generate.py:321
INFO     Loading tokenizer...                                                                                generate.py:335
INFO     Loading text encoder...                                                                                generate.py:337
13:39:19 INFO     Loading VAE...                                                                                generate.py:339
INFO     Loading UNet...                                                                                generate.py:341
13:39:23 INFO     Loaded 453.20928M-parameter motion module                                                                                unet.py:578
13:39:24 INFO     Using scheduler "k_dpmpp_sde" (DPMSolverSinglestepScheduler)                                                                                generate.py:353
INFO     Loading weights from E:\animatediff-cli-prompt-travel\data\models\sd\xxmix9realistic_v40.safetensors                                                                                generate.py:358
13:39:28 INFO     Merging weights into UNet...                                                                                generate.py:375
INFO     Creating AnimationPipeline...                                                                                generate.py:421
INFO     No TI embeddings found                                                                                ti.py:102
INFO     loading c='controlnet_tile' model                                                                                generate.py:450
13:39:30 INFO     loading c='controlnet_ip2p' model                                                                                generate.py:450
13:39:31 INFO     Sending pipeline to device "cuda"                                                                                pipeline.py:22
INFO     Selected data types: unet_dtype=torch.float16, tenc_dtype=torch.float16, vae_dtype=torch.bfloat16                                                                                device.py:90
INFO     Using channels_last memory format for UNet and VAE                                                                                device.py:111
13:39:34 INFO     Saving prompt config to output directory                                                                                cli.py:367
INFO     Initialization complete!                                                                                cli.py:375
INFO     Generating 1 animations                                                                                cli.py:376
INFO     Running generation 1 of 1                                                                                cli.py:386
INFO     Generation seed:<PHONE_NUMBER>06100

xxmix9realistic_v40.safetensors：
https://github.com/s9roll7/animatediff-cli-prompt-travel/assets/9162812/3468adb7-2f31-45f5-86dd-38b799c4f80b
darkSushiMixMix_225D.safetensors：
https://github.com/s9roll7/animatediff-cli-prompt-travel/assets/9162812/8f1dd255-0868-4f0a-b4a5-77783219625f

https://www.reddit.com/r/StableDiffusion/comments/16vlk9j/guide_to_creating_videos_with/

By default, controlnet_tile is set to 1.0 and controlnet_ip2p to 0.5 strength,
This is a setting that outputs a picture that looks exactly like the original picture, so if you want to change it, you should replace the tile with another controlnet.

https://www.reddit.com/r/StableDiffusion/comments/16vlk9j/guide_to_creating_videos_with/

very nice! I solve this issue follow the steps writing in reddit.


How to install gym-retro-integration UI for windows error :(

Issue summary
Hi, I am trying to follow and install gym-retro-integration UI for windows but when i install gym retro and try to execute this statement " pip3 install -e . " it returns me following error mentioned in the image attached

System information

[Operation System: Windows 10 Pro 64 Bit]
[Python version: 3.8.5]
[Gym Retro version: 0.8.0]

I ended up commenting out some pieces of the setup.py if you take a look at the issue about multiple times I c won't claim it's the right way, but it worked for my needs

Can you please tell me which lines you commented and where is setup.py located? thanks

diff --git a/setup.py b/setup.py
index fab566c..c6c2b91 100644
--- a/setup.py
+++ b/setup.py
@@ -11,25 +11,11 @@ VERSION_PATH = os.path.join(os.path.dirname(os.path.abspath(file)), 'VERSION
SCRIPT_DIR = os.path.dirname(os.path.abspath(file))
README = open(os.path.join(SCRIPT_DIR, "README.md")).read()
-if not os.path.exists(os.path.join(os.path.dirname(file), '.git')):

use_scm_version = False

shutil.copy('VERSION', 'retro/VERSION.txt')
-else:

def version_scheme(version):

   with open(VERSION_PATH) as v:

       version_file = v.read().strip()

   if version.distance:

       version_file += '.dev%d' % version.distance

   return version_file

def local_scheme(version):

   v = ''

   if version.distance:

       v = '+' + version.node

   return v

use_scm_version = {'write_to': 'retro/VERSION.txt',

                  'version_scheme': version_scheme,

                  'local_scheme': local_scheme}

+#if not os.path.exists(os.path.join(os.path.dirname(file), '.git')):
+use_scm_version = False
+shutil.copy('VERSION', 'retro/VERSION.txt')
+

setup.py is inside the retro repository it appears you downloaded.

I think just if you look in setup.py there is some stuff going on that appears to be checking what you have checked out and building that, but since I"m guessing you just cloned and didn't check out a tag/branch, that might be why it does it, I just commented it out, for better or worse.

I think just if you look in setup.py there is some stuff going on that appears to be checking what you have checked out and building that, but since I"m guessing you just cloned and didn't check out a tag/branch, that might be why it does it, I just commented it out, for better or worse.

could you share the setup file you used. will be really useful thanks.


libgpm may delete some signal handlers

Problem:  libgpm may delete some signal handlers
Solution: restore these signal handlers after calling gpm
When 'mouse' is set, vim is trying to detect mouse support on startup. Eventually, vim resorts to using libgpm as the final method of mouse detection. This library may delete some signals handlers that were initially set up by vim.
This is how:

mch_setmouse() calls gpm_open()
Gpm_Open is executed, which returns early on line 210 [1]
Keep in mind that lines 353-373 [2] are skipped, so gpm_saved_suspend_hook and gpm_saved_winch_hook are empty
Finally, Gpm_Close is called, which will reset [3] SIGWINCH and SIGTSTP to an empty sigaction.

[1] https://github.com/telmich/gpm/blob/e82d1a653ca94aa4ed12441424da6ce780b1e530/src/lib/liblow.c#L210
[2] https://github.com/telmich/gpm/blob/e82d1a653ca94aa4ed12441424da6ce780b1e530/src/lib/liblow.c#L353-L373
[3] https://github.com/telmich/gpm/blob/e82d1a653ca94aa4ed12441424da6ce780b1e530/src/lib/liblow.c#L419-L424
Fixes: #12154

To reproduce this you need vim compiled with +mouse_gpm and kitty terminal.
Also ttymouse must be empty and mouse non-empty. Eg. set mouse=a ttymouse=
Test_signal_TSTP was failing on kitty for the same reason.

Thanks, I guess it should probably also reporte to the gpm upstream project. ping @telmich


mockModeNoCallback option and controller cache

Please see my question about context.headers[] before merging.

mockModeNoCallback config option. Swagger examples show res.end() in the controllers while mock mode use cb(). This makes a huge difference for projects using custom fitters
after the router.
Return content-type header on Accept: /
Simple default content-type detection
Place auto-generated mock controller function in cache
Made logging a bit more verbose
ignoreMissingHandlers option as described in swagger-tools router docs

Sorry for the bundled pull request ;)

Coverage decreased (-1.05%) to 95.902% when pulling 533a18d568a6b8cdfe10217677a3d57b21df73cd on moander:feature/router-stuff-1 into cc89cb2b7e6964cb1579ce2ec47f65175924a5c4 on theganyo:master.

Coverage decreased (-1.5%) to 95.447% when pulling a944384bebaf23111c47028ac99cb4d044d2d1c5 on moander:feature/router-stuff-1 into cc89cb2b7e6964cb1579ce2ec47f65175924a5c4 on theganyo:master.

Thanks for your contribution. Would you be able to provide any tests to verify your work?

Swagger examples show res.end() in the controllers while mock mode use cb(). This makes a huge difference for projects using custom fitters
after the router.

I think it's a deceiving problem, we need to fix these examples.
In this sense, they just show what not to do...


Create "Install Keptn" section

Move the installation content that is currently in 0.xx.x/operate into a top-level section and break out individual pages for "Create k8s cluster", "Install Keptn in an existing cluster", "Install CLI and Keptn", etc.  Also include pages about using Helm (which is kind of there but pretty confusing), "Multi-cluster setup", "K8s support & Cluster size", etc
Perhaps include Quick Start as first page in this section?
Top page of each section/subsection should list the steps involved, in order, with links to individual pages as well as relevant general information.  (For example, the section on installing a k8s cluster will briefly discuss the options that are particularly suited for self-study, demos, and development environments; the section on installing Keptn might explain that this can be done manually, using the kindkeptn docker image, or using the helm chart).  The summary lists might include links to pages currently in other sections (e.g. manage/git-upstream, Istio installation in continuous-delivery/expose-services, set up monitoring for Dynatrace/Prometheus) although some effort will be made to consolidate all this information into this section.
Some pages to be included are:

Install a k8s cluster (list all the options in current docs but include intro that identifies the options that are particularly appropriate for study/demonstration and for development
Install Keptn in this cluster (including authenticate Keptn CLI and Keptn Bridge)
Install Keptn in an existing (production) k8s cluster
Kubernetes support & cluster size
Configure Helm on Keptn (for exposing the Bridge, customizing ports, etc)
Troubleshooting

Maybe other topics will be added.
For  detailed explanation of the proposed structure, see https://docs.google.com/document/d/1fQaLxsnIRjHfqzZl-D2vJ8VoeYpaSEdg0CIH9IbmxyU/edit# .
Implementation plan:

This will be implemented for  the current-plus-2 version of the docs so that it is published for the first time when a new software release is published; this change is too disruptive to be implemented piecemeal is a published doc set.
We will first create the new section to include the existing material from the installation docs and deleting the old version of those docs.   This first pass will also include a first pass at the section intro information.
On a subsequent pass, we will incorporate material from other docs (such as tutorials).
We will then compare these contents with equivalent information included in other docs, especially the tutorials

What Done Looks Like

New top-level section entitled "Install Keptn" with comprehensive information for all implementations
Duplicate installation material removed from other docs as much as possible

outdated


Github Action workflow to deploy to Github Pages

Fixes #885
Changes:
Added a Github actions workflow which

converts readme.md to HTML
pushes the code to gh-pages branch

Thanks! Unfortunately the idea for the page has changed and we are no longer using the Readme.md. Hope to see more contributions in future.


As a project lead, I want to perform a lit review on new CNN topics so that we can confirm direction.

Completed during winter break.


Better error reporting when a composition root has not been setup.

Currently, CreateContextBuilder() throws a null pointer exception if a composition root has not been setup. It should probably throw an execption telling what exactly is wrong and what to do about it.

Seems unimportant, it's an issue which is easily spotted, especially when using trunk-based programming


error when running

Hey
Great idea...  Unfortunately it doesn't work.  Does it support python 3?
I downloaded the entire pkg.  I then either pip installed everything (definitely did "pip install noisemix") or verified that the required package can be imported successfully in py.
When I executed in the noisemix-master/noisemix directory:
$: python -m noisemix ~/Models/data/train.csv -format fastText
it generated an err msg:
$: /home/shogo/miniconda3/bin/python: No module named noisemix.main; 'noisemix' is a package and cannot be directly executed
Were you able to run it?
Thx for your help!
S

Were you able to use it with git clone?
I will try to repro the error tomorrow.

Hi Adam
Thx for getting back to me.  I cloned the repo, and figured that if I have the sc, I can just mod it and get it to work.  But you know how it is… Just never got around to do it.
The idea of noisemix is fab!  And I would love to have it work out of box.  Maybe if you don’t mind taking a minute to see you were able to replicate the err msg and repo, it’d be great.  Thx!
SH

On Nov 11, 2018, at 11:19 AM, Adam Bittlingmayer<EMAIL_ADDRESS>wrote:
Were you able to use it with git clone?
I will try to repro the error tomorrow.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/noisemix/noisemix/issues/5#issuecomment-437696700, or mute the thread https://github.com/notifications/unsubscribe-auth/AGeQPP3l-8uTF0hCD-NFuW6OrMGdY7Lpks5uuHg0gaJpZM4YPkOr.

@studiocardo I just tried it and it worked fine for me.
Maybe the issue is MiniConda?  And are you running from within the repo?
git clone https://github.com/noisemix/noisemix.git
cd noisemix/
python3 -m noisemix ...

(It worked without or without the cd step.)
When I do python3 I get:

Python 3.7.0b4 (default, May 14 2018, 18:57:32)
[Clang 9.1.0 (clang-9<IP_ADDRESS>)] on darwin


test with userPrivateKeyFile

Are you sure this if statement in line 100 is right ?
https://github.com/speatzle/go-passbolt-cli/blob/0449e9983cd98c7a946a901770bcc50fa50c8633/cmd/root.go#L100-L105
If no error raised and option userPrivateKeyFile isn't empty, the contents of the PrivateKeyFile is never saved in config file.

This if statement checks first that there was no error reading the flag and second that the user Provided a KeyFile Path (that it is not empty).
If that is the case then the UserPrivateKey gets replaced with the contents of the file at UserPrivateKeyPath.
This Piece of Code is only there to replace the UserPrivateKey with the file contents of UserPrivateKeyPath if UserPrivateKeyPath is set.

Ah i see the problem now, thank you. This did work in my test anyway so ill need to look into this.

fixed with release v0.1.6

Many thanks, i appreciate your work ! 😉


dags tab is empty

I am using below command
docker-compose -f docker-compose-CeleryExecutor.yml up but still dag is empty

Can you please guide me what is issue.
I have checked all the container and its running fine.

I believe you need to see if your docker machine has access to the folder that was mounted with dags file.
I did add permission to the folder that has the project.


Correct name for Vietnamese language

Vietnamese in Vietnamese is "Tiếng Việt"; use this in the language chooser etc.
/language vi
/kind cleanup

/lgtm
/approve


Improve confirmation dialogs

We need to rethink the properties that allow or stop confirmations for different actions.

Maybe we also need to change the confirmation message on restore database action. See https://github.com/dbeaver/dbeaver/discussions/16395#discussioncomment-3041666


Possible completionHandler for FUIArray's observequery?

The problem I'm having is I have 1000s of values. I need to make sure all my data is synced before i can accomplish the next task. Is there a completion handler or a notification that lets you know when all the batch updates are done, for swift?

This is exposed in a FUICollectionDelegate method.


chaire-lib/transition: Remove all imports from '[..]/projects' directory

All these imports should be replaced with registration functions (or some such mechanism, validate for each case) that can be called at application start-up. See the i18next.ts file in chaire-lib-backend for such a mechanism.

We have no such imports anymore. And I don't understand the issue anymore here, so closing


Fixes for table queries not returning 404 for missing entities

Added test case for missing entities, when using direct reference or query of objects in Table.
Added corresponding logic to return 404
Should address Issues #20 and #19

Should also resolve #26


Support streaming IO by removing need for seek in skip_sync

This moves skip_sync to the end of the block (where the sync marker exists) rather than the start (where it will exist only when there was a previous block), and has it require the sync marker is present, and not try to handle the case where it isn't present. This means we no longer depend on seek, and can support streaming IO.
Fixes #39 for me.

Thanks! Will check soon-ish...

Hi, and thanks a lot for this patch.
It's also fixing the issue when there is a corrupted sync marker at the end of file.
I ran into this recently, in that case fastavro is entering infinite loop because of seeking back.

Thanks. In 0.9.6


Deploy single action problem

Hello,
When I try to deploy a single Action, I have this error :
$ aio app deploy -a someAction  ›   Error: Cannot read properties of undefined (reading 'getProject')
After some research, I see there is a "publish" flag initialized here :
https://github.com/adobe/aio-cli-plugin-app/blob/378781925c07a630fa4887c35ab9a60bfa7e4c94/src/commands/app/deploy.js#L35
flags.publish = flags.publish && !flags.action
This flag is used to initialize libConsoleCLI variable wich is used for "getProject" function...
I think you have to remove "&& !flags.action" to let single action to deploy.
Thank you.
Damien.

@purplecabbage can we get some eyes on this? Is it better if I create an internal Jira ticket to match?

This appears to be an issue with the Production workspace, and the --action flag resulting in a logic error that resulted in libConsoleCLI not to be initialized.
If --action is set, it should never go in here (in checking the extension state): https://github.com/adobe/aio-cli-plugin-app/blob/378781925c07a630fa4887c35ab9a60bfa7e4c94/src/commands/app/deploy.js#L88

PR #808

https://github.com/adobe/aio-cli-plugin-app/releases/tag/12.2.4


Is unilm mask_prob_eos not used?

when i use unilm in a abstract task , the eos can't predict well, The text often repeats the last server words, and i haven't  see the mask_prob_eos used in this file , https://github.com/microsoft/unilm/blob/master/unilm-v1/src/biunilm/run_seq2seq.py
is it nessary for nlg task?

Hi @qiansi , the argument mask_prob_eos is ignored in the implementation. The EOS token is masked and predicted in the same probability as other tokens. The refactored code at https://github.com/microsoft/unilm/tree/master/s2s-ft should be easier to use.


Setting AKMicrophone device changes the entire system output

Hello guys,
I don't know if this is a bug or a designed feature, but on macOS setting the device of an AKMicrophone also changes the system wide input.
let microphone = AKMicrophone()
let microphoneDevice = AudioKit.inputDevices!.first(where: { $0.name == "My Microphone" })!
try? microphoneAudioNode.setDevice(microphoneDevice)

Result: the microphone is initiated correctly, but macOS also changes the microphone to "My Microphone'
Is there any solution for this? To create an AKMicrophone input with a custom microphone?
Thank you

I believe this is better described in this open issue: https://github.com/AudioKit/AudioKit/issues/2130 so I will close it here for now. It is on our radar.

I see.. the same question for output.
Is it possible to set the output without changing the system wide settings?
let outputDevice = AudioKit.outputDevices!.first(where: { $0.name == "Other Speakers" })!
try? AudioKit.setOutputDevice(outputDevice)

Thanks


Reading content state on the Flutter side

Do we have any possible way to read the content state on the Flutter side after it updates?
We have this but I tried implementing it on my branch.
It will emit an event onto the stream when you start your activity, returning the ActivityId and the AppGroupId since its part of the ContentState defined in the package (I think?) but when I update the activity state through a push notification it will not emit anything onto the stream.

@istornz or @jolamar do you have any suggestions or ideas for this?

@arnar-steinthors
is there any updates ?


”Turnu Roșu” is listed in the drop-down menu

Description:
"Turnu Roșu" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Turnu Roșu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Turnu Roșu, SIBIU".


Would you mind adding few words on the main doc to "document" this addition? You can do this in the paragraph about CSS.

 Good to go?


Fix refresh cache behavior

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[x] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necessary)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes / reffers to the following issues: ...
Pull Request Description
This pull request do the OPPOSITE thing about #2032 . It modify that description about SDWebImageRefreshCached from

If a cached image is refreshed, the completion block is called once with the cached image and again with the final image.

to

The completion block is called once with the cached image and again with the final image.

Which means the completion block with SDWebImageRefreshCached MUST called twice. One is from SDImageCache memory-disk cache and one from NRURLCache system cache or NSURLSession network response(It just use that NSURLRequestUseProtocolCachePolicy and let system maintains).
Why I suggest to change this is that our old behavior is sometime misdirected. Even if the server reponse the same imageData, we still call that completion block twice at some cases and not really obey what it says. So it's better to ensure that MUST be called twice and let end-user to render their views depended on the callback times.
Moreover, the SDWebImageDownloaderIgnoreCachedResponse will be not used for SDWebImageRefreshCached. It will kept just for compatibility which check the cache before any network request and immediately return.
This PR must be conflicted with #2032

After consideration. I think we should not change the behavior in the patch release. So I will close this PR and choose #2032 to merge. If someone have other idea or any question, you can reopen and comment here.


Update the README to markdown

I also pushed a couple of changes removing the NEWS and Changelog files, but can revert these if you want. They seem to be dead and unnecessary with git
Also updated the TODO to include newly implemented APIs.

@hughbe,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


simple key mapping for page areas

BREAKING CHANGE
This replaces the (complicated) insertion of page data into layouts with a simple 1:1 key mapping. The old way was extremely flexible, but complicated and slow (and it turns out was more than we actually needed). The new way allows us to bake some assumptions into our data and kiln, and lays the foundation for switching layouts.
Here's what the old data looked like:
# page
pageRef:
  layout: /layout/ref
  areaA: /a/string/ref
  areaB:
    - /another/ref
    - /a/third/ref

# layout
layoutRef:
  propA: ['areaA']
  propB:
    -
      _ref: /some/layout/component
    - areaB

Here's what the new data looks like:
# page
pageRef:
  layout: /layout/ref
  propA:
    - /a/string/ref # all page areas look the same
  propB:
    - /another/ref
    - /a/third/ref

# layout
layoutRef:
  propA: propA # layout prop and page prop have the same name, allowing for easy insertion and switching
  propB: propB
  propC:
    -
      _ref: /some/layout/component

👍


Invalid asset name: undefined

Hi, I would like to inquire about this issue wherein we encountered an Invalid asset name: undefined on our stellar address GBIJTZVWS6GSTEDGF4V6TKVZ6N7SJXWTWCLGFH66RN5A6RW7OVCLSGFF

I have the same error for EPIC ( fastepic.eu )
"Invalid asset name: undefined." at https://stellar.expert/explorer/public/asset/EPIC-GD4YEDHMQK2HD7DMKAG554JK4TVOTQNPWTKR2KHL5UCSJ6ART2UA2E32-1
c@https://stellar.expert/212.18acec95c0d7039b280c2c54176aad42.js:2:7620
parse@https://stellar.expert/212.18acec95c0d7039b280c2c54176aad42.js:2:8804
@https://stellar.expert/212.18acec95c0d7039b280c2c54176aad42.js:2:20975
b@https://stellar.expert/212.18acec95c0d7039b280c2c54176aad42.js:2:20962
sa@https://stellar.expert/app.7f31f1f39850c68c2484.js:2:124010
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:104.0) Gecko/20100101 Firefox/104.0

@fastepic yours is ok now, i checked it. how did you do it?

Good day,
I have an issue as well. I received the same
Here is from my tpgtoken.io Distribution Account:
GBIJTZVWS6GSTEDGF4V6TKVZ6N7SJXWTWCLGFH66RN5A6RW7OVCLSGFF
"Invalid asset name: undefined." at https://stellar.expert/explorer/public/account/GBIJTZVWS6GSTEDGF4V6TKVZ6N7SJXWTWCLGFH66RN5A6RW7OVCLSGFF?filter=all
TypeError: Invalid asset name: undefined.
at new c (https://stellar.expert/212.18acec95c0d7039b280c2c54176aad42.js:2:7620)
at c.parse (https://stellar.expert/212.18acec95c0d7039b280c2c54176aad42.js:2:8804)
at https://stellar.expert/212.18acec95c0d7039b280c2c54176aad42.js:2:20975
at Array.map ()
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/105.0.1343.33
======================
and here is from my tpgtoken.io Issuer Account:
GANJHXWRSHC6AQDG3S4LW3VIM5HOKOSOGWJJPUXE7RJLEZNGMJ33ELS6
"Invalid asset name: undefined." at https://stellar.expert/explorer/public/asset/TPG-GANJHXWRSHC6AQDG3S4LW3VIM5HOKOSOGWJJPUXE7RJLEZNGMJ33ELS6?filter=all
TypeError: Invalid asset name: undefined.
at new c (https://stellar.expert/212.18acec95c0d7039b280c2c54176aad42.js:2:7620)
at c.parse (https://stellar.expert/212.18acec95c0d7039b280c2c54176aad42.js:2:8804)
at https://stellar.expert/212.18acec95c0d7039b280c2c54176aad42.js:2:20975
at Array.map ()
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/105.0.1343.33
Please let me know what caused this error and how it can be fixed? Any updates would be helpful.
our email is<EMAIL_ADDRESS>Thank you so much!

The problem was caused by a metadata-related update we rolled out recently.
Everything has been fixed, asset and liquidity pools metadata remain backwards-compatible.
Sorry for the inconvenience.

"Cannot read properties of undefined (reading 'find')" at https://stellar.expert/explorer/public/account/GDBXCWE52NKAPPCP6LYNFLADNDGH6Q6FPGRS42MN4YNIDGSQGL46K5G6
TypeError: Cannot read properties of undefined (reading 'find')
at https://stellar.expert/explorer.a4ec1dcb347e6cd992c09396f59db808.js:1:23810
at aa (https://stellar.expert/app.e67c0ab26dca498f8264.js:2:432718)
at Ka (https://stellar.expert/app.e67c0ab26dca498f8264.js:2:442236)
at Ws (https://stellar.expert/app.e67c0ab26dca498f8264.js:2:485817)
at Cu (https://stellar.expert/app.e67c0ab26dca498f8264.js:2:472121)
at Ou (https://stellar.expert/app.e67c0ab26dca498f8264.js:2:472049)
at Au (https://stellar.expert/app.e67c0ab26dca498f8264.js:2:471912)
at bu (https://stellar.expert/app.e67c0ab26dca498f8264.js:2:468899)
at https://stellar.expert/app.e67c0ab26dca498f8264.js:2:418637
at t.unstable_runWithPriority (https://stellar.expert/app.e67c0ab26dca498f8264.js:2:513298)
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Get this unhandled error on my account balance


REST API: private message require recipients in json encoded list

Fixes #3142

@timabbott I've changed the tests accordingly. Please have a look

@sinwar @timabbott I wonder if there has been some miscommunication on this issue.  I don't think we want to force the use<EMAIL_ADDRESS>when we're sending a message to Alice.  This will break a lot of API code, including some of our own bots.  The common case here is to have one recipient, so we should optimize the predictability of that case.  For huddles, we have a lot of frontend code that assumes comma-delimited lists.

I think you're right that maybe we actually don't want to change this.  Sorry for the confusion @sinwar!  I guess we should just close as wontfix #3142, right?

@timabbott #3142 says the docs conflict with the code; if we shouldn't change the code, then we should change that issue to say "update the docs to say that the current behavior is what's expected". :)

@brainwane yes we should.

I updated the issue description in #3142 @sinwar :)

Hello @sinwar, a Zulip maintainer reviewed this pull request over a week ago, but you haven't updated your pull request since. Please take a look at the requested changes and update your pull request accordingly.
Thank you for your valuable contributions to Zulip!

Closing this, since I think we've decided against this change.


Building the csharp docker container under directory 0.11 fails

The error message is:
E: Version '<IP_ADDRESS>-1~bpo8+2' for 'libgrpc0' was not found

Seems like if I use 0.11.1-1~bpo8+1 then the package is found, as indicated by the packages index here. Is this the right package or it has not been published yet?


[BUG] Window of launched apps are behind others

description
I'm using Cerebro on latest macOS with dual-screen (well, 3 exactly). It shows as expected on the main screen, but any application it launches will appear behind all other applications windows.

expectations
The application I'm launching should show its window on top of any existing window.
system information
  System:
    OS: macOS High Sierra 10.13.6
    CPU: x64 Intel(R) Core(TM) i5-4288U CPU @ 2.60GHz
    Memory: 287.36 MB / 8.00 GB
    Shell: 5.3 - /bin/zsh
  Binaries:
    Node: 10.4.0 - ~/.nvm/versions/node/v10.4.0/bin/node
    Yarn: 1.7.0 - /usr/local/bin/yarn
    npm: 6.1.0 - ~/.nvm/versions/node/v10.4.0/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Utilities:
    Make: 3.81 - /usr/bin/make
    GCC: 4.2.1 - /usr/bin/gcc
    Git: 2.15.2 - /usr/bin/git
  Servers:
    Apache: 2.4.33 - /usr/sbin/apachectl
  Virtualization:
    Docker: 18.03.1 - /usr/local/bin/docker
  Languages:
    Bash: 3.2.57 - /bin/bash
    Go: 1.10.3 - /usr/local/bin/go
    Java: 1.8.0 - /usr/bin/javac
    Perl: 5.18.2 - /usr/bin/perl
    PHP: 7.1.16 - /usr/bin/php
    Python: 2.7.15 - /usr/local/bin/python
    Ruby: 2.3.7 - /usr/bin/ruby

Cerebro: 0.3.1

Also I'm using [CMD] + [SPACE] as my launching shortcut, if it can make any difference...

Sounds like this has been fixed in 0.3.2! Thanks!

whew, I was totally on the wrong repo, sorry, too much hours of geeking lately :-/


Support for OSX

Thanks for working on this!
I just wanted to try it out, but it seems like OSX is not supported yet?
$ go get -u github.com/derekparker/delve/cmd/dlv
go build github.com/derekparker/delve/proctl: no buildable Go source files in /Users/felix/code/go/src/github.com/derekparker/delve/proctl
# github.com/derekparker/delve/goreadline
38: error: use of undeclared identifier 'rl_catch_sigwinch'
38: error: use of undeclared identifier 'rl_resize_terminal'; did you mean 'rl_reset_terminal'?

Some time ago I tried to create an interface to lldb on OS X and I had some trouble with the code signing. What I remember is that you can use the codesign tool to add a signature to existing binaries. So I guess you should be able to use normal gc and then add the signature afterwards. Manpage: https://developer.apple.com/library/mac/documentation/Darwin/Reference/ManPages/man1/codesign.1.html

@neelance that looks interesting, however the issue is still that the codesign tool is going to look in the binary for the Info.plist section. See: https://developer.apple.com/library/mac/documentation/Security/Conceptual/CodeSigningGuide/Procedures/Procedures.html
What caught my eye there was:
5. Add the following arguments to your linker flags:
-sectcreate __TEXT __info_plist Info.plist_path

where Info.plist_path is the complete path of the Info.plist file in your project.

If codesign can take a path to an Info.plist file instead of looking for it in the __info_plist section, that would be great. Then the normal Go toolchain would work fine, and gccGo would not be needed.

@neelance actually I was wrong(-ish). We can embed the Info.plist into the binary without using gccgo by using cgo and C variable attributes. This will allow us to use the normal Go toolchain + codesign on OS X.

I don't think we have to. The codesign man page says:
     --prefix string
             If no explicit unique identifier is specified (using the -i option), and if the implicitly generated identifier does not contain any dot (.) characters, then the given
             string is prefixed to the identifier before use. If the implicit identifier contains a dot, it is used as-is. Typically, this is used to deal with command tools without
             Info.plists, whose default identifier is simply the command's filename; the conventional prefix used is com.domain. (note that the final dot needs to be explicit).

It seems like that you don't need an Info.plist file or section if you pass the data via command line args. I'll give it a try.

Yeah I was going to say that I don't think GDB has one (I haven't explicitly looked at the code though...) https://sourceware.org/gdb/wiki/BuildingOnDarwin.

GDB embeds one in darwin-nat.c I think
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=blob;f=gdb/darwin-nat.c;h=b44dcb4d1dd013245e2cb5ee2fa6f8ef15a84401;hb=HEAD#l154

@landaire @neelance - @pnasrat is right, it's embedded in gdb/darwin-nat.c, and we can embed it in exactly the same way using C variable attributes which will work with the normal Go toolchain.

I still don't see why we need it. I just tried codesign on one of the binaries created by gc and it worked fine.

Right, codesign will work without an embedded plist file, however as far as I know, the keys in the Info.plist file we have to embed are used by OS X, like the SecurityFramework (I think) to allow us access to another Mach task.

@neelance did you try it specifically with delve?

As discussed in Code Requirements, the system often uses the Info.plist file of an application bundle to determine the code’s designated requirement. Although single-file tools don’t normally have an Info.plist, you can add one.

And

To see how this works in practice, assume the user has granted permission to the Apple Mail application to access a keychain item. The keychain uses Mail’s designated requirement to identify it: the keychain records the identifier (com.apple.Mail) and the signer of the application (Apple) to identify the program allowed to access the keychain item. Whenever Mail attempts to access this keychain item, the keychain looks at Mail’s signature to make sure that the program has not been corrupted, that the identifier is com.apple.Mail, and that the program was signed by Apple. If everything checks out, the keychain gives Mail access to the keychain item. When Apple issues a new version of Mail, the new version includes a signature, signed by Apple, that identifies the application as com.apple.Mail. Therefore, when the user installs the new version of Mail and it attempts to access the keychain item, the keychain recognizes the updated version as the same program and does not prompt the user for verification.

Source
I think what it's really used for in this context is the SecTaskAccess key: http://os-tres.net/blog/2010/02/17/mac-os-x-and-task-for-pid-mach-call/
This is also present in GDB's Info.plist. Note I haven't looked at delve's source to see what APIs it uses,

Yeah seems like you are right. The identifier, signature and requirements are stored by codesign, but the SecTaskAccess key is not. It's only in the Info.plist file, which gets secured by the signature.

I found manually codesigning gdb on OSX Yosemite to be painful.  Any improvements would be appreciated by many -- I'm sure.

Update on this issue: The branch darwin-support contains my current efforts towards adding OS X support. OS X support will be my absolute top priority for the time being, with 1 other nagging issue coming in at a close second.
If anyone else happens to be working on OS X support in their own free time, please base your work off of the darwin-support branch, and post any issues or arch-specific code design questions here.

Yet another update:
I now have Delve compiling in OS X using a bunch of unimplemented stub functions. The good news is that it compiles, which means all Linux specific stuff has been sorted out and moved to the correct place. The next step is the fill in the stub functions, most of which I already have a clear idea of what the Linux -> Mach translation looks like.
Hoping to have all stubs implemented and all tests passing as soon as possible. The other nagging issue has been resolved, so basically 100% of my efforts are focused of OS X support at the moment.

:+1: great

You deserve a beer!

:+1:  from the :1234:

I have just merged initial support for OS X into master. I plan on making an actual announcement tomorrow, but I'd like to give a chance for anybody watching this thread to check it out and flush out any major issues before then, if there is any.
Please note that Delve is still in beta, so there may well be general issues, however the major thing here is OS X support.

@derekparker Awesome stuff, Thank you!

Had a small issue with the make install command, $(which dlv) is return blank, but this could just be me. $(shell which dlv) resolved that for me.

Thank you!

@nowk thanks, fixed.

looking forward to it.

great job derek, i'll be sure to try it.

Wow, I can't wait to give this a shot.  :)


⚠ TorrentLeech has degraded performance

In 0a6bf91, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 1399 ms

Resolved: TorrentLeech performance has improved in a380a03 after 5 minutes.


Vue devtools tab doesn't retain it's position when reordering Chrome devtools tabs

Chrome devtools remember the position they were in if they are reordered by the user (via drag-drop). The Vue tab is the exception; it is always placed on the right end of the set of tabs.

That's not the case for me - it does retain position for me. Either way, this is a Chrome issue and cannot be fixed by vue-devtools itself.

My bad, it happens in version 62 as well but I thought it was a bug because other extensions did retain their positions but Vue is the only one (among those extensions) that only appears when the app is a Vue app

Chrome doesn't seem to persist position of tabs that are not always visible.. Since the vue devtools tab only appears if the current page contains a vue app, its tab position does not persist.
There is a similar issue for the react extension that behaves the same way.
Could we not change the behavior of the vue devtools plugin to always be present and simply display some sort of "Vue not present on the current page" message instead of omitting the tab in that case? That should be enough of a change to allow the tab order to persist... and seems like it'd be a pretty easy change to make. I'd submit a PR, but not familiar with chrome plugin dev to know where to look in the code.

It's been reported as a bug to Chrome team: https://bugs.chromium.org/p/chromium/issues/detail?id=771144 ... no movement on it though...


Dynamic devise class_name within routes

Make it possible to complete the process of overriding Spree.user_class while using solidus_auth_devise. Before this PR if a developer took advantage of setting their own Spree.user_class - for example.
User = Class.new(Spree::User)

initializer "spree.auth.environment", after: :load_config_initializers do |app|
  Spree.user_class = "::User"
end

The hardcoded 'Spree::User' (string) within the routes file would override this customisation and continue to incorrectly use Spree::User rather than User. Then meant when using devise built in helpers the user class returned would be incorrect - for example.
Before this PR current_spree_user.class => Spree::User
After this PR current_spree_user.class => User

That failure looks unrelated, re-running tests.
Also this looks like its a good change to me, @brendandeere? You  know this code better than I do.

Restarted the build again. Still randomly fails for unrelated reasons.

:+1: For using the built in solidus user class hook.
A custom user class would still need to implement the api expected by SolidusAuthDevise in order to function proplerly, but that seems like a reasonable expectation of someone providing a custom user class.
It might be worth looking into this hook and changing when it is run.
If you explicitly set a user_class in config/initializers/solidus.rb, I believe the auth devise gem would still override that setting with its own user class.
It could be worth while to make auth devise set it's default value at a different time which still allows a developer to override the default from within an initializer file, rather than specifying an :after clause.

@karlentwistle would you mind to rebase on current master? That should fix the failing build

Sorry for the delay I just returned from honeymoon. I have just rebased off master.


[BUG] Item Permissions not functioning in 1.11.2

Server version: 1.11.2 git-Spigot-3fb9945-2b6c9f4
RPGItems: v3.5.262 for 1.11.2 and v3.5.125 for 1.8
Recently updated from 1.8 to 1.11.2 and while testing out some thigns, making sure it worked correctly, I gave a player one of our previous weapons that had a permission on it.
In 1.8, this worked just fine, and anyone who had that item but lacked the permission node set on it were unable to use the item in question.
In 1.11, this does not work at ALL! Anyone who gets their hands on these items can use them, even if they do not have the permission set on it.
Hopefully this can be fixed before my players figure it out...
Here is an example item which worked fine in 1.8, and only players who had the permission rpgitem.infernalblade were able to use it, but now anyone is allowed to:
  infernal:
    name: infernal
    id: 895
    haspermission: true
    permission: rpgitem.infernalblade
    display: '&4&lＩＮＦＥＲＮＡＬ ＢＬＡＤＥ'
    quality: TRASH
    damageMin: 35
    damageMax: 35
    armour: 0
    type: Long Sword
    hand: Two Handed
    lore: Born from the hate, greed, and evil of humanity, this sword was created as Divinity Blade’s evil twin, mirroring it in both strength and magic ability. Unlike Divinity Blade, it was born from evil and darkness.
    description:
    - '&f&6&lSpecial Quest Item'
    - '&f&8&lBaneful Instrument'
    item: IRON_SWORD
    ignoreWorldGuard: false
    item_data: 0
    powers:
      '0':
        powerName: potionself
        cooldown: 1400
        amp: 2
        time: 200
        type: DAMAGE_RESISTANCE
        consumption: 0
      '1':
        powerName: potionhit
        chance: 33
        duration: 30
        amplifier: 0
        type: HARM
        consumption: 0
      '2':
        powerName: potionhit
        chance: 7
        duration: 200
        amplifier: 2
        type: SLOW
        consumption: 0
    recipechance: 6
    hasRecipe: false
    dropChances: {}
    hitCost: 1
    hittingCost: 1
    blockBreakingCost: 1
    hitCostByDamage: false
    maxDurability: -1
    defaultDurability: -1
    durabilityLowerBound: 0
    durabilityUpperBound: 250
    forceBar: false
    showPowerText: true
    showArmourLore: true
    damageMode: FIXED
    customItemModel: false

Just me experiencing this issue I take it?


Add types (TS support)

If interested, 1f9097e  has types for TypeScript. Travis is failing because of outdated scripts in the travis system, which won't affect the package.
I don't have time to rewrite the travis logic, but if you're interested in adding types, you can find it in the commit mentioned!

Can you go ahead and remove https://github.com/caitp/node-mock-stdin/pull/20/commits/2f6f8cfda53acbdab670e85f2e170293e944103b from your branch? Should be good to go then

Should be set!


Mark self(backend) as defunct if cannot communicate with leader and notify associated recorder

The sequence to be ensured here is:
t0=Failure to report load to leader as observed by backend, tick is not incremented in next report if failure observed
t1.a1=Failures continue to happen, leader marks backend as defunct, stops assigning new process groups to it
t1.a2=Reports succeed, back to state before t0
t2.a1.b1=Failures continue to happen, backend marks self as defunct, sends 5xx for /poll when existing assigned recorders make requests. Ensure t2>t1, i.e. backend marks self as defunct only after leader marks it as defunct
t3.a1.b1=Backend starts sending exploratory reports (tick=0)
t4.a1.b1=Recorder gives up on backend since it keeps receiving 5xx, calls /associate to get a new backend, leader assigns a new backend to recorder's process group, de-associates the defunct backend
t5.a1.b1=N successive exploratory ticks succeed, backend marks self as available again, sends usual reports to leader
t6.a1.b1=Reports succeed, eventually leader marks backend as available again
Description:
If backend is not able to talk to leader (in case of n/w partition) for some successive load reports, it should mark self as defunct and respond to all /poll requests with 5xx. /profile calls should not be errored.
The above is required so that recorder detects something to be wrong with backend and calls /association again to get a healthy backend via leader.
Backend should mark self as available again if it can communicate with leader for some successive report intervals.

Some pictorial description of sequence of events will help.

Updated description with a textual timeline description.


Update update.md

update paths in the manual update section, these were not accurate

The reason .env wasn't included is because it is not provided by us. I consider this change unnecessary - What would the user replace it with?

I should have explained that.
the reason I mentioned .env is because I find a better/safer approach to copy user files (uploads, database, .env, css) to the the unpacked files shipped with the release. this way people know also .env needs to be there.

The user.css file and the SQLite DB is also created by us. Ah no, you're right. However, I don't think it's bad to mention it.

user.css is in the zip, so if you uploaded would replace the customised version. Ditto sqlite.

the reason I mentioned .env is because I find a better/safer approach to copy user files (uploads, database, .env, css) to the the unpacked files shipped with the release. this way people know also .env needs to be there.

I see your point, but that instruction is stated as the reverse, in which case it could cause confusion.


ClearlyDefinedPackageCurationProvider: Minor code style fixes

See individual commits.

@sschuberth just updated the commit message titles for semantic commits.


Improved CLI, code restrukturierung and Inital support for node-gtk

Here is inital support for node-gtk (I have also convert the two gjs examples to note-gtk), I know the project is called ts-for-gjs and not ts-for-node-gtk but I hope my adjustments will still be accepted (Maybe this project could get more contributors in this way and if there is no separate project where many tasks are the same).
For this I have restructured the code so that it doesn't get too confusing by increasing the complexity:

Moved CLI specific code to cli.ts
Add a new TemplateProcessor class to generate definitions from templates to reduce the GirModule class
Add a new Transformation class to transform methods, enums, etc for the specific environment (gjs or node)
Convert string templates to ejs templates
Because there are now more source files, I have moved them all to src/
The compiled source files now in lib/
I have moved the dependencies loading to module-loader.ts
I have moved the file parser to module-loader.ts
I have moved the cli specific to ./src/commands/* and ./src/config
All types now in ./src/types, each type / interface in his own file
Added bin/bin.js, required to use ts-for-gir as a cl tool by npm
Added ESLint with the Prettier plugin, I tried to keep the code style (for example, there is a rule to not use simicolons).
ESLint and Prettier are also used to validate and auto format the generated template files.
I've changed the default output dir to @types/ to make it easier to generate type definitions in other projects without having to specify the output path each time
The output path now contains subfolders for Gjs and / or node-gtk (the directory name node-gtk is required to get typescript automatically working with import * as gi from 'node-gtk')
There is now a new option called --build-type which can be "lib" or "types", "lib" is the default for generating types for Gjs and the way it was before. I added "types" for node-gtk and generates the types e.g. as you would publish them on DefinitelyTyped.
Source maps are now also generated, which simplify debugging, Zhe IDE can now points directly to the typescript files e.g. at a breakpoint
Updated dependencies
Replaced commander.js with oclif because commander.js was a difficult to use with multiple arguments
Added wild card support for module names to load (e.g. Gtk*)
Ask which module version should be used if several are found
It can now be specified which modules should be ignored
A config can now be created in which the cli options are available (e.g. the modules that should be ignored)
Added new reserved keywords for functions, class names, etc
Fixed data type errors
Added a new list

I've also prepared the project to publish it on npm to use it as a real cli tool, for this I have also improved the CLI arguments and updates the README (this fixes #18 and #1):
Show general help
$ ts-for-gir --help
Typescript .d.ts generator from GIR for gjs or node-gtk

VERSION
  ts-for-gir/1.0.0 linux-x64 node-v12.14.1

USAGE
  $ ts-for-gir [COMMAND]

COMMANDS
  generate  Generates .d.ts files from GIR for gjs or node-gtk
  help      display help for ts-for-gir
  list      Lists all available GIR modules

Generate .d.ts files
$ ts-for-gir generate --help                                                                                :(
Generates .d.ts files from GIR for gjs or node-gtk

USAGE
  $ ts-for-gir generate MODULES

ARGUMENTS
  MODULES  [default: *] GIR modules to load, e.g. 'Gio-2.0'. Accepts multiple modules

OPTIONS
  -b, --buildType=lib|types        [default for gjs: lib, default for node: types] Force the definitions generation type
  -e, --environments=gjs|node      [default: gjs,node] javascript environment
  -g, --girDirectory=girDirectory  [default: /usr/share/gir-1.0] GIR directory
  -h, --help                       show CLI help
  -i, --ignore=ignore              [default: ] modules that should be ignored
  -o, --outdir=outdir              [default: @types] directory to output to
  -p, --print                      print the output to console and create no files
  -v, --verbose                    Switch on/off the verbose mode
  --configName=configName          name of the config if you want to use a different name
  --ignoreConflicts                Do not ask for package versions if multiple versions are found
  --pretty                         prettifies the generated .d.ts files

EXAMPLES
  # Run 'ts-for-gir generate' in your gjs or node-gtk project to generate typings for your project, pass the gir modules you need for your project
  ts-for-gir generate

  # You can also use wild cards
  ts-for-gir generate Gtk*

  # If you want to parse all of your locally installed gir modules run
  ts-for-gir generate '*'

  # Generate .d.ts. files only for gjs
  ts-for-gir generate '*' -e gjs

  # Generate .d.ts. files only for node
  ts-for-gir generate '*' -e node

  # Use a special config file
  ts-for-gir generate --configName='.ts-for-gir.gtk4.rc.js

  # Generate .d.ts. files but not for Gtk-3.0 and xrandr-1.3
  ts-for-gir generate --ignore=Gtk-3.0 xrandr-1.3

List available GIR modules
$ ts-for-gir list --help
Lists all available GIR modules

USAGE
  $ ts-for-gir list MODULES

ARGUMENTS
  MODULES  [default: *] GIR modules to load, e.g. 'Gio-2.0'. Accepts multiple modules

OPTIONS
  -g, --girDirectory=girDirectory  [default: /usr/share/gir-1.0] GIR directory
  -h, --help                       show CLI help
  -i, --ignore=ignore              [default: true] modules that should be ignored
  -v, --verbose                    Switch on/off the verbose mode
  --configName=configName          name of the config if you want to use a different name

EXAMPLES
  # Lists all available GIR modules in ./vala-girs/gir-1.0
  ts-for-gir list -g ./vala-girs/gir-1.0

  # Lists all available GIR modules in /usr/share/gir-1.0 but not Gtk-3.0 and xrandr-1.3
  ts-for-gir list --ignore=Gtk-3.0 xrandr-1.3

I have renamed the package name to ts-for-gir to make it clear that it's no longer just for gjs. So if you publish the package on npm that would be the name with my changes.
If more changes are necessary before my changes are accepted, please let me know.
I would like to submit more adjustments for node-gtk in the future :)
And this is what it looks like:

Edit
Added more descriptions of my changes
Edit 2
Again

Okay I think I have it now, I re-edited my description above.
I have run the generator over all vala-girs, for that I extended the cli much more and I found many bugs such missing reserved keywords, variable names that start with a number and so on.
You can try it by your self by running npm run build:test:girs.
I have added the vala-girs as a submodule, so the following is necessary to run this new test:
git submodule init
git submodule update

P.S.
I'm sorry about the big PR, I will keep it smaller in the future

I found and fixed another small bug:
This was the list of the reserved function names:
export const RESERVED_VARIABLE_NAMES = [
    in: 1,
    function: 1,
    true: 1,
    false: 1,
    break: 1,
    arguments: 1,
    eval: 1,
    default: 1,
    new: 1,
    extends: 1,
    with: 1,
    var: 1,
    class: 1,
    delete: 1,
    return: 1,
}

if it has now been checked if a function name is reserved, the following was called
if (RESERVED_VARIABLE_NAMES[name]) {
    ...
}

But if name === 'toString' the check is true because the method toString exists in such objects.
So now I've converted RESERVED_VARIABLE_NAMES to an array and the check looks like this now:
if (RESERVED_VARIABLE_NAMES.includes(name)) {
    ...
}

Hi @JumpLink ,
I tried to follow your installation steps from https://github.com/JumpLink/ts-for-gjs/blob/master/README.md but somehow I cannot find a ts-for-gir cli tool afterwards.
I use NodeJs v12.15.0 on Arch Linux.
What am I doing wrong?

Hi @JumpLink ,
I tried to follow your installation steps from https://github.com/JumpLink/ts-for-gjs/blob/master/README.md but somehow I cannot find a ts-for-gir cli tool afterwards.
I use NodeJs v12.15.0 on Arch Linux.
What am I doing wrong?

You need to run npm link in the root of the project because the module is currently not published on npm but with npm link you can simulate this

Thanks @JumpLink.
The issue was that I somehow mixed up your repo and this repo and in the end nothing worked :(
I uninstalled all installed npm packages and followed your instructions and now it works.
Thanks, again for your help!

Regarding the -g option of the tool. Would it be possible to extend the tool to set multiple directories it should look at.
As an example, I would like to add St-1.0 which gir-file can be found at /usr/share/gnome-shell.
It depends on files which can be found at /usr/lib/mutter-5, which depend on files in /usr/share/gir-1.0.
Currently when I try generate the St-1.0 ts file the tool displays a lot of errors that it cannot find those dependencies as they are located in other directories.
It would be great if this would work: ts-for-gir generate -g /usr/share/gir-1.0 -g /usr/lib/mutter-5 -g /usr/share/gnome-shell St-1.0 -e gjs

Hi @dan1707, thank you for your Feedback and yes I will implementate this. I will have time for it in 1 or 2 weeks, I think. Maybe sooner

@dan1707 Okay, I got it :)
Tested on Ubuntu with
ts-for-gir generate St-1.0 -g /usr/share/gir-1.0 /usr/share/gnome-shell /usr/lib/x86_64-linux-gnu/mutter-5/ -e gjs

This also works
ts-for-gir generate St-1.0 -g /usr/share/gir-1.0 -g /usr/share/gnome-shell -g /usr/lib/x86_64-linux-gnu/mutter-5/ -e gjs

Can you test it?

@mmstick Looks like you are using Mac OS? If so, I need to test this on a Mac

That's a negative. Pop!_OS (Ubuntu) 20.04

Wich node.js version? Installed with apt or nvm?

Installed with apt: 10.15.2~dfsg-2ubuntu1

I haven't had time to review in detail, but I like the approach and it looks high quality. I'm happy that we fix up any issues we find later on the main branch.

Thank you very much!

I have the same issue as mmstick

@mmstick, @jambonmcyeah If you still have the problem, it should be fixed now with the change from oclif to yargs in #44. However, node 16 is now required there