Markdown going all to the right

My markup is going all to the right of my text area, like the video shows. I used the same model as the example:
https://codesandbox.io/s/configured-marked-input-forked-4svlb7?file=/src/Button.tsx
But the behavior is like the video and not like the example (expected behavior). What can be going wrong? Since I used the same elements, I expected similar behaviour. Thank you!
I would like to have them in between my text, like this:

https://github.com/Nowely/marked-input/assets/6011385/9235a5bc-9d20-4979-9060-c1d011fff167

Update:
Solved by adding this to the button css:
float: "none",
it was set automatically:
float: "right",


[Deployer] Cleanup Gconfig references

using Gconfig multiple times instead of self.config.
should add function to gconfig to override values if seperate config is passed in.

deployer has been moved to backends, so cleanup is not needed.


Improve report formatting

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Files
Coverage Δ

src/flamme/section/mapping.py
100.00% <100.00%> (ø)

:loudspeaker: Thoughts on this report? Let us know!.


Rendering bug: content from all project tabs is displayed

As you can see, from the root of the project, the releases are visible, if the screenshot extended further, the files would also be visible.
Previously, the UI had these in seperate "tabs".

Seems like the issue is that the javascript bundle referenced by that page (https://pypi.org/static/js/warehouse.3b55beb2.js)  is 404-ing, that's why the login links on the top are missing as well.

I purged the cache for this page and the issue was resolved. Seems like the actual issue is why the cache was out of date.
@alex Have you seen this on any other project pages?

Also seeing it on https://pypi.org/project/wheel/
A cache purge fixed it. Going to close this.


Spike: How much of the extension is broken if it support 0.17+/0.18+

The original plan to support 0.19+ is a MVP solution. It assumes OmegaConfigLoader and OmegaConfigLoader is the only configloader for 0.19.  This has limited the extension for 0.19 only user.
Background
0.19 is released in 2023 Dec, the adoption has been relatively slow (as Kedro project tend to last quite long and team rarely upgrade the library version unless there are some necessary change). The adoption is ~30%

Context
OmegaConfigLoader is actually introduced in 0.18 earlier (forgot the exact version, probably mature around 0.18.12+). If we look into the stat, 0.18.14 alone has ~40% of Kedro user. This mean that we can actually support this group of user already.

Further pushing the idea, we can support non-OmegaConfigLoader user (or even 0.17+). Until today we haven't introduced any specific OmegaConfigLoader feature, Add navigation support for OmegaConfig Syntax. So it could be relatively easy to support older version of Kedro. Even if there are omegaconfig specific feature, it only affect some advance feature of code navigation (i.e. pipeline.py -> parameters.yml). Other feature such as:

hover
top-level pipeline.py -> config /  config -> pipeline.py shouldn't be affected.

The exact impact is unclear, and thus this ticket is about doing a spike to map this out.
cc @astrojuanlu

Or we can use kedro-vscode to convince more people to upgrade to Kedro 0.19.+ because it doesn't support older versions.

As much as I'd like more people to use the extension, it's probably a lot of effort for us to support and as @merelcht says, the fact that only Kedro 0.19+ is supported could be a nice incentive to upgrade.

I think it's a good thing to reserve important new user-facing features for later versions of Kedro. In the past we've been guilty of issuing breaking changes with very little incentives for people to migrate.

I understand the argument here, but I think

As much as I'd like more people to use the extension, it's probably a lot of effort for us to support

The intuition here is actually opposite, it may be easy to add support for older versions without much effort.

could be a nice incentive to upgrade.

There could be still incentive to upgrade i.e. feature parity if we starting to implement things like https://github.com/kedro-org/vscode-kedro/issues/8, so only OmegaConfigLoader user get accurate navigation while others get some basic feature only.

Or we can use kedro-vscode to convince more people to upgrade to Kedro 0.19.+ because it doesn't support older versions.

I am not a big fan of keeping feature just to force people to upgrade, our versioning strategy is usually around breaking changes/non-breaking, we didn't do this for OmegaConfigLoader.
With that said, I don't plan to do this before the marketing effort (Youtube, Linkedin post) go out. It's ideal if this actually drive more 0.19 adoption.

For the record, I did a quick test on 0.18.14 + OmegaConfigLoader. The only breaking changes are _data_sets -> _dataset. When running in debug mode the hover and go to defintion function are running just fine, except I get a weird server pannic that kills the server without any log.

We could do something like "partial 0.18.x" support and not invest in edge cases which crop up

My recommendation would be to try to find Kedro 0.19+ projects that break the extension (I already found a few), try to focus on those, and when (1) the plugin is more solid, (2) there's more awareness, and (3) users pressure us to add support for older versions, we evaluate the effort

My recommendation would be to try to find Kedro 0.19+ projects that break the extension (I already found a few)

Can you help to report those cases?

Surely 😄

Maybe the extension helps adoption some how, after 2 months the adoption of 0.19 is >50%

I will close this issue as I think the benefit of supporting older versions make less sense now.


Error when running rpm-ostree update

Describe the bug
Dear bluefin maintainers,
I ran rpm-ostree update manually today, just to notice that it fails. :(
I included the output of the command below. I hope you can help me fix this.
Thanks in advance!
What did you expect to happen?
I expected to be notified in some way, that my system hasn't updated since 2024-10-09 due to an error.
Also I expected this error not to happen. ':)
Output of bootc status
apiVersion: org.containers.bootc/v1
kind: BootcHost
metadata:
  name: host
spec:
  image: null
  bootOrder: default
status:
  staged: null
  booted:
    image: null
    cachedUpdate: null
    incompatible: true
    pinned: false
    store: null
    ostree:
      checksum: 1d9afcb5b0d9dd7d6336023278134807e70e1e9ec02162e20e9d24b9dadf0a28
      deploySerial: 0
  rollback:
    image: null
    cachedUpdate: null
    incompatible: true
    pinned: false
    store: null
    ostree:
      checksum: b01eb2861bc2ed79e08a97c02d53c96486d38bac2f90eda826f72f1508fcff99
      deploySerial: 0
  rollbackQueued: false
  type: null

Output of groups
istvan wheel lxd libvirt incus-admin docker

Extra information or context
Output of rpm-ostree update
> rpm-ostree update
note: automatic updates (stage) are enabled
Pulling manifest: ostree-image-signed:docker://ghcr.io/ublue-os/bluefin-dx:40
Checking out tree 110919f... done
Enabled rpm-md repositories: updates fedora copr:copr.fedorainfracloud.org:hikariknight:looking-glass-kvmfr rpmfusion-free-updates-testing rpmfusion-free rpmfusion-nonfree-updates-testing rpmfusion-nonfree updates-archive
Updating metadata for 'rpmfusion-free-updates-testing'... done
error: Updating rpm-md repo 'rpmfusion-free-updates-testing': Failed to download gpg key for repo 'rpmfusion-free-updates-testing': Curl error (37): Couldn't read a file:// file for file:///etc/pki/rpm-gpg/RPM-GPG-KEY-rpmfusion-free-fedora-40 [Couldn't open file /etc/pki/rpm-gpg/RPM-GPG-KEY-rpmfusion-free-fedora-40]```

Interesting, I cannot remember doing this, especially since disabling a repo is non-trivial.
Here is my solution for future reference:
I had to edit:

/etc/yum.repos.d/rpmfusion-free-updates-testing.repo
/etc/yum.repos.d/rpmfusion-free.repo
/etc/yum.repos.d/rpmfusion-nonfree-updates-testing.repo
/etc/yum.repos.d/rpmfusion-nonfree.repo
and in all these files change the line "enabled=1" to "enabled=0"

It works again. I still wish I would have gotten a notification that something is broken. And I really wonder why I had rpmfusion enabled, I wouldn't even know what for. Thanks for the incredibly quick help though!


Enhance SQLMesh to Support Audits for External Sources

Description:
Currently, SQLMesh supports external sources but lacks the functionality to perform audits on them. The schema.yaml file for external models does not include an interface for adding audits, limiting the ability to ensure data integrity and reliability for downstream tables.
Objective:
Implement a feature within SQLMesh to enable audits for external sources, allowing users to verify that downstream tables receive the expected data from these external models.
Context: https://tobiko-data.slack.com/archives/C044BRE5W4S/p1712156303563069?thread_ts=1712149507.981779&cid=C044BRE5W4S

implement by https://github.com/TobikoData/sqlmesh/pull/2715


How to turn on debug log in plugin

Hi all,
I found that there are many plugins, like cache_range_requests support the DEBUG_LOG.
How can we turn this DEBUG_LOG in our configuration after installing ATS?
Thanks.

Hi.
You have to enable debug logs and specify the tag. The cache_range_request debug tag seems to be the plugin name(usually is).
Not sure which ATS version are you using but if you are building out of master, then you have a .yaml file, otherwise you have a .config file.
records.config set the following two config records to:
CONFIG proxy.config.diags.debug.enabled INT 1
CONFIG proxy.config.diags.debug.tags STRING cache_range_requests

records.yaml run the following command( or edit the file manually)
traffic_ctl config set  proxy.config.diags.debug.enabled 1 --cold
traffic_ctl config set  proxy.config.diags.debug.tags cache_range_requests --cold

Thanks.

Thanks a lot, @brbzull0 !
Let me try your instruction.

@xn-zhao Did this resolve your issue?  If so, can you please close it.


maxLength based on rawText or text?

Does maxLength takes rawText or text into account? This seems to behave differently in Android versus iOS.
In Android, it does count the mask characters while in iOS uses the rawText.
What's the general rule for this?

I am also interested in this topic, any updates?


Why pt Print filenames -g spent more time than ag?

search keyword pt faster than ag
/home/pub/linux$ time pt struct  . | wc -l
1497329    real    0m3.571s    user    0m9.428s   sys     0m4.416s
/home/pub/linux$ time ag struct  . | wc -l
1504023    real    0m7.315s    user    0m9.172s   sys     0m2.280s

/home/pub/aosp$ time pt out . | wc -l
11658    real    0m1.283s    user    0m14.760s    sys     0m5.456s
/home/pub/aosp$ time ag out . | wc -l
11658    real    0m3.339s    user    0m2.200s      sys     0m8.352s

Print filenames ag faster than pt
/home/pub/linux$ time pt -g  . | wc -l
59444    real    0m1.074s    user    0m2.556s    sys     0m0.740s
/home/pub/linux$ time ag -g  . | wc -l
59444    real    0m0.310s    user    0m0.244s    sys     0m0.076s

/home/pub/aosp$ time pt -g . | wc -l
249043    real    0m4.348s    user    0m9.412s    sys     0m3.728s
/home/pub/aosp$ time ag -g . | wc -l
249043    real    0m1.839s    user    0m0.956s    sys     0m0.936s

The results are just the opposite, Is there anything setting for fast filenames search?
os: ubuntu 16.04
ag version: 0.31.0
pt version: 2.1.5

@fcying You could try this commit to see whether it helps.

@tw4452852  thanks for your commit.
I test your commit, it fast than ag now.
/home/pub/linux$ time pt -g . | wc -l
59444    real	0m0.203s    user	0m2.176s    sys	0m0.436s

/home/pub/linux$ time ag -g . | wc -l
59444    real	0m0.297s    user	0m0.244s    sys	0m0.064s


Testing observations

Having done some testing as part of my setup and I have noticed the following:
Site return redirection:
When a user hits a view/page that requires authentication the user is sent to the ADFS login screen, the user signs in and is returned to the URL defined in the “LOGIN_REDIRECT_URL”

Is there any way of passing the current location over to ADFS so the user can return to the same page, I’m not sure if this is a limitation of this module or ADFS.  (It’s my understanding that ADFS can do this.)

So for me this solution works well on sites that require a login to the whole site where the user hits the login page as the first point of call rather than a site that has mixed content which displays differently depending on your authentication state.

Logout:
I’m getting mixed results when it comes to logging out a user.

I assume I would need to logout of both Django and close the ADFS session, logging out of Django is not enough as I found that when I revisit the login process I get automatically authenticated (no ADFS sign in required)

Having a function in your code to provide this in a single template call much like the {{ ADFS_AUTH_URL }} context call would be helpful.

User “Staff” and “Superuser” Status
I didn't see an option to be able to add a user with "Staff" or "Superuser" rights

My thoughts were to have two groups one for Users and another for Admins, I have this working to a point, my user gets placed in to the relevant group in django based on the AD group membership however I'm unable to assign "Staff" or "Supperuser" rights to the user as its added.

Site return redirection:
This is indeed a limitation of the OAuth2 authorization code grant type itself. You have to register a redirect URI and it's only this URI that ADFS will redirect to. This is a security measure so the authorization code is only sent to a trusted URI and not to an attacker.
A bypass could be something that remembers the page that triggered the redirect to ADFS an direct the user again to this page when authentication is completed. But the possible ways of doing that are so diverse that it's outside the scope of this package.
Logout:
The ADFS server remembers your authentication in a cookie. So as soon as you are redirected again to the ADFS server, you will automatically be logged in again. That was also the scope of this package. Seamless (e.g. without user interaction) single sign on to your site with ADFS as the back end.
While there is a logout URL on ADFS (/adfs/ls/?wa=wsignout1.0), I never got it to work.

UPDATE
User “Staff” and “Superuser” Status:
I fixed my own point here, I realized that I could send a claim field to the app to apply the "is_staff" option when creating the user.
I did this by updating my CLAIM_MAPPING and by adding the following claim rules:

"CLAIM_MAPPING" : {"first_name": "given_name",
"last_name": "family_name",
"is_staff": "is_staff",
"email": "email"},

RULES
Admin Group / is_staff = True

Rule Template
Claim Rule Name
User's Group
Outgoing Claim Type
Outgoing claim value

Send Group Membership as a Claim
is_staff_[ADMIN_GROUP]_True
[AD Admin Group]
is_staff
True

User Group / is_staff = False

Rule Template
Claim Rule Name
User's Group
Outgoing Claim Type
Outgoing claim value

Send Group Membership as a Claim
is_staff_[USER_GROUP]_False
[AD User Group]
is_staff
False

As for the is_staff and is_superuser fields, they can indeed be set like that. This is because both are a BooleanField and those convert from a string value to a boolean automatically.
Two things are possible here:

Document setting those fields that way
Adding extra functionality to support those 2 specific fields.

My preference goes to 1 as it allows to support all possible fields you might want to add to the user model of Django.

RE User “Staff” and “Superuser” Status:
I guess this is your call.
RE Logout:
I have been testing options for the logout and have come up with the following:
setting.py add:

LOGOUT_REDIRECT_URL = 'https://[ADFS SERVER FQDN]/adfs/ls/?wa=wsignout1.0'

django-auth-urls.py update as follows:

from django.conf.urls import url
from django.contrib.auth.views import logout
from . import views

urlpatterns
= [
url(r'^login$', views.OAuth2View.as_view(), name='login'),
url(r'^logout/$', logout, name='logout'),
]

Now when I visit the https://[WEB SERVER FQDN]/oauth2/logout I get logged out of django and redirected to the ADFS logout page.
Its just one method which might work for some.

While testing I found an issue with setting is_staff via a claim. While it's easy to give a user staff access, it's very difficult to take it away again. Also, if your user is not in one of the groups you specify, the claim isn't sent, triggering an error. So this doesn't scale.

OK thanks for the update on this.
In my case I have the two groups defined in my "Issuance Authorization Rules" So I guess I will only ever get a user that is in one group or the other.

I've opened separate issue for your 3 remarks.


[WIP] Add docker template

Adding a template for the creation of a Docker hosted endpoint

@bording can you take a look and tell me what you think of this?
The biggest thing I did that deviates from the other templates is the addition of a --endpoint "enterEndpointNameHere" option which flushes out the endpoint name during configuration as well as the folder name in Docker. I haven't added an update to the README for this yet because I'm wondering if there's reason to add it to the other templates (consistency maybe?)
I'm also wondering if adding --endpointName is enough or if I should take it one step farther and add things like persistence and transport selection to it too? None of the others do anything like this so it would be a pretty big difference.
I still need to flush out some comments in the Program.cs file depending on what we decide for the above questions.

@bording I looked at the current templates and see what you were talking about around them setting the endpoint name using -o and/or -n. Part of me says that's good enough, part of me says that an explicit --endpointName is slightly clearer and more extensible. The question really becomes, do we need those things?

@dbelcham take a look at the changes I've made. We've discussed most of them.
We still need to add some sort of Ctrl+C to exit dev experience that works when not using Windows.
We also still need to wire up the new template into the tests.


NodeJS 4.x LTS support

Version 0.10.x is getting a bit long in the tooth.

FWIW, tests failing with single message from NodeJS 4:
\n#\n# Fatal error in heap setup\n# Allocation failed - process out of memory\n#\n'

Process exits with code 4. Not sure how to debug further. Google not helping.

Changes look good to me. I imagine the failures are the result of the memory limitations we place on the Node processes we launch during tests. We can probably fix this by starting up a simple Node 4.4 process, seeing how much RAM it uses and adjusting https://github.com/praekelt/vumi-sandbox/blob/develop/vxsandbox/tests/test_worker.py#L391. One would think 256MB would be enough for Hello World. ;)

Also, tx for working on this!

@hodgestar: I tried doubling those 2 values but had no luck :/
Will dig more when I get the time.

According to node's FAQ:
Q: I'm experiencing reliability/memory spiking issues on a low-memory system, what's going on?

A: By default, --max_old_space_size (which controls the upper limit of the V8 heap) is ~1.5GB. If your system has less memory than that, you will need to explicitly set this option to the appropriate amount of memory available to that process. For example, --max_old_space_size=128 will allow 128MB for that process.

Could explain we are seeing those memory limit issues, though using --max_old_space_size=128 doesn't seem to change anything. I didn't investigate too deeply though, so maybe there is a way it could still help us.

Node will start when the virtual memory rlimits are removed but --max-old-space-size doesn't seem to help so far...

👍

:+1:


How can we trigger error responses?

Stripe suggests the use of test cards against their dev APIs to trigger different error responses: https://stripe.com/docs/testing#cards.
We'd like to do the same while in testing, but couldn't find how to do that with fake stripe. How do you do it? Do you think we could include those in the gem?
Thank you! :)

One way we could achieve this feature would be to make the fixture file used per action configurable.
For example, if we wanted creating a charge to fail (the default could remain create_charge), we could make config such as:
FakeStripe.set_fixtures({ create_charge: 'create_charge_failure' })

that would then be looked up when the mocked endpoint is defined.
post '/v1/charges' do
  FakeStripe.charge_count += 1
  json_response 201, fixture(FakeStripe.get_fixture_for('create_charge'))
end

The developer would then need to create a properly failed API response in a create_charge_failure file.
Did y'all have other API's in mind?

👍 I would like to be able to test in a System Test that UI is rendered based on various error codes from those test cards from the page @tute noted

Closing this because fake_stripe has been deprecated: https://github.com/thoughtbot/fake_stripe/pull/154


Immidiatly closes upon opening the file

It immidiatly closes python when i run setup.py

Hey! Can you share any error message or whatever your console shows?


Making use of console.error

NodeJS outputs errors to stderr. Making use of console.error(...) instead of console.log(...) makes node-source-map-support compatible with IDEs like Webstorm to get stack traces.

Nice, thanks.


Added leaderboard functionality to Tabular

Issue #, if available:
Description of changes:
Added leaderboard functionality to Tabular
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

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

@Innixma @jwmueller I changed the PR merging rule to only require 1 approval and hope to move faster. Let me know if it is okay for you guys.

Seems fine for now, once we have more contributors we may need to change it back to 2.


fix crd customization

make install fails probably due to kustomization file missing. This PR adds it.
Before
❯ make install
Downloading<EMAIL_ADDRESS>/Users/ashith/git/ashithwilson/istio-fortsa/bin/controller-gen-v0.14.0 rbac:roleName=manager-role crd webhook paths="./..." output:crd:artifacts:config=config/crd/bases
Downloading<EMAIL_ADDRESS>/Users/ashith/git/ashithwilson/istio-fortsa/bin/kustomize-v5.3.0 build config/crd | kubectl apply -f -
Error: unable to find one of 'kustomization.yaml', 'kustomization.yml' or 'Kustomization' in directory '/Users/ashith/git/ashithwilson/istio-fortsa/config/crd'
error: no objects passed to apply
make: *** [install] Error 1

After
❯ make install
/Users/ashith/git/ashithwilson/istio-fortsa/bin/controller-gen-v0.14.0 rbac:roleName=manager-role crd webhook paths="./..." output:crd:artifacts:config=config/crd/bases
/Users/ashith/git/ashithwilson/istio-fortsa/bin/kustomize-v5.3.0 build config/crd | kubectl apply -f -
clusterrole.rbac.authorization.k8s.io/manager-role created

Can you please rebase your branch from main?


Softlock if the player ignore a item

Version
3.0.8 (a1e8985)
Seed
R730-6WE2-XJZ9207Z9Y100E09W080000200000WZZH1Z
Scenario
RE 3 (Jill)
Player Character
Ada
What happened?
I wasn't sure if i should report this issue but...
I decide to confirm if #453  was crashing, so using the log i tried to go to the place where it crashed as fast as i could and well...
After i grabed what originally was the Lockpick (Oil Machine in this rando) the game change to night-time, but i realized i don't have the lockpick, after seeing the log it should be where the Lighter Oil should be, but since is Night-Time, the Ligher Oil is not there anymore, so i can't go back and grab it, causing a softlock, unless i use Debug mode
This is more my fault for ignoring the basement but i guess any other player could have this same problem
Anyway here is a save
SaveData.zip
(Also, zombies in the room of the save are supposed to be laying, but since the game obliges them to do the animation of the cutscene the frooze after doing it, they still can harm the player and be harmed and go back to normal later so...)

...Nevermind the last part, if you load from the save what i said about the zombies don't happen, the zombies load normally


Update citation of SPDI Paper

Citing the peer-reviewed version rather than the preprint.

I appreciate the PR!


migrate fails with (1146, "Table 'db.auth_group' doesn't exist")

I have a ViewSet which references RoleViewSetMixin:
class MyViewSet(RoleViewSetMixin):

When I try to run migrate for a new installation, the process crashes with:
django.db.utils.ProgrammingError: (1146, "Table 'db.auth_group' doesn't exist")

drfr_err.txt
This is because the RoleViewSetMixin class contains a static call which tries to read from models which are not present (because migrate hasn't been run):
DEFAULT_GROUPS = [group.name.lower() for group in Group.objects.all()]

The solution is to move that initialisation to a constructor (something like):
class RoleViewSetMixin(object):
    """A ViewSet mixin that parameterizes DRF methods over roles"""

    _viewset_method_registry = set(getattr(settings, "VIEWSET_METHOD_REGISTRY", DEFAULT_REGISTRY))

    def __init__(self):
        groups = [group.name.lower() for group in Group.objects.all()]
        self._role_groups = set(getattr(settings, "ROLE_GROUPS", groups))

What are your thoughts?  Happy to submit a PR if required.
There is a workaround which is to uncomment all paths from urls.py, run manage.py migrate auth, then re-add the URLs and run manage.py migrate

PRs welcome! I'm not really working with django these days but this seems like a straightforward fix.

PR (has been tested).

Closed by #7


Microsoft.WindowsTerminal.Preview new version 1.15.2002.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hello @mivsvit, The package manager bot determined that the metadata was not compliant.

 Manifest: The manifest file path must match PackageIdentifier and PackageVersion properties from manifest. actual 'manifests\m\Microsoft\WindowsTerminal\Preview\1.15.2003.0', expected 'manifests\m\Microsoft\WindowsTerminal\Preview\1.15.2002.0'

Looks like a transient error as the pipeline was running while the manifest was being corrected.

#68845

Will submit new request with correct folder name


Use configparser

This simplifies neuralmonkey.config.parsing. By using configparser from the Python standard library, we ensure that our INI syntax is compatible with the most widely used standard.
This also cancels the (undocumented) “global config” feature, because configparser does not seem to support global properties.

Very nice, I would just name better the functions and add a better docstring, see comments in the diff.

By the what was the undocumented global config?

Global config is the part of the ini file without any section header. This is needed.
It is not implemented yet so it is naturally undocumented. There is only a mention about it, as it is necessary if we want to be able to build only parts of the configuration file.
Which means we will have to re-think the entrypoint declaration. Or, use another config parser (which would add a dependency, however).
We could also specify the entrypoint via command line (which is ugly).

I suppose we could keep the [main] section for the global config, though.

There could be a [main] or [global] section. And it could even create an object that would run the whole thing and we could merge all the commands into just neuralmonkey config.ini.

Then you would have to change the configuration between run and train phases, which we are trying to avoid. I prefer multiple binaries right now. In future, we can change it to commands of one runnable file, so the command line would be something like
neuralmonkey config.ini any_command
or
neuralmonkey any_command config.ini
which would look up the [any_command] section and run the whole thing from this entry point.
Anyway, these configparser changes can be merged.

Besides, run script needs another dataset configuration as its argument, which would first have to be handled somehow.

I am working on this now, in the experiment_object branch. This should refactor out all the ugly things, including learning_utils.


[update] rubocop 1.18.4

Coverage decreased (-3.2%) to 81.516% when pulling d218275b70c6b677a4029dc4d99516a06166990c on fix-rubocop-issues2 into fc420614acd6a6accffb7889db659a3fa1b2e4b3 on master.


Modular contact form not processing

Hi,
I have followed the steps on https://learn.getgrav.org/forms/forms/how-to-forms-in-modular-pages to add a form to my modular website at https://steyaert.be/marleen/.
But when I fill out the form and click the send button ("Verzend") nothing happens. I'm not guided to the thankyou page, the maillogs on my server tell me that there is no mail sent.
I've set the Mail Engine to SMTP (changed it to PHP as a test), the default from and to are defined.
I enabled the debugger but also nothing there, just like there is nothing to see in logs/grav.log.
I also noticed that adding the form definition in modular.md doesn't show my form but adding the definition to form.md does show it.
I also tried to base this on the deliver theme where it does work if I test but still can't seem to pinpoint the issue.
You can download the entire source from https://steyaert.be/marleen.tgz if you'd like.
I hope you can guide me in the right direction so that I can solve this.
Cheers

I'll try to take a look at your source if i get a minute.

Hi @rhukster , have you been able to take a look at this recently? :)

Hi, I have disabled caching but this didn't help.

I found a bug which could be related to this issue. Though I'm not sure if it affects also modular pages.. But if you are adding content dynamically, it'll make form not to work.

Hi, all content is loaded from the pages itself and nothing comes from external resources. But did you manage to fix this bug? Otherwise I could test this on my modular website.

Regarding to what you just said, I don't think my fix affects your issue. There's no tgz file anymore in the above url, so I cannot see what's going on.

Oh yes, I had removed the tgz. I have put it back so you can download it again.

I have just updated the form plugin to v2.11.4 and now I get the following error when filling out the contact form:
[2018-02-16 08:36:16] grav.CRITICAL: Call to a member function post() on null - Trace: #0 [internal function]: Grav\Plugin\FormPlugin->onPageInitialized(Object(RocketTheme\Toolbox\Event\Event), 'onPageInitializ...', Object(RocketTheme\Toolbox\Event\EventDispatcher)) #1 /var/www/sites/steyaert.be/web02/sites/marleenlabbeke/vendor/symfony/event-dispatcher/EventDispatcher.php(184): call_user_func(Array, Object(RocketTheme\Toolbox\Event\Event), 'onPageInitializ...', Object(RocketTheme\Toolbox\Event\EventDispatcher)) #2 /var/www/sites/steyaert.be/web02/sites/marleenlabbeke/vendor/symfony/event-dispatcher/EventDispatcher.php(46): Symfony\Component\EventDispatcher\EventDispatcher->doDispatch(Array, 'onPageInitializ...', Object(RocketTheme\Toolbox\Event\Event)) #3 /var/www/sites/steyaert.be/web02/sites/marleenlabbeke/vendor/rockettheme/toolbox/Event/src/EventDispatcher.php(23): Symfony\Component\EventDispatcher\EventDispatcher->dispatch('onPageInitializ...', Object(RocketTheme\Toolbox\Event\Event)) #4 /var/www/sites/steyaert.be/web02/sites/marleenlabbeke/system/src/Grav/Common/Grav.php(288): RocketTheme\Toolbox\Event\EventDispatcher->dispatch('onPageInitializ...', Object(RocketTheme\Toolbox\Event\Event)) #5 /var/www/sites/steyaert.be/web02/sites/marleenlabbeke/system/src/Grav/Common/Processors/PagesProcessor.php(26): Grav\Common\Grav->fireEvent('onPageInitializ...', Object(RocketTheme\Toolbox\Event\Event)) #6 /var/www/sites/steyaert.be/web02/sites/marleenlabbeke/system/src/Grav/Common/Grav.php(132): Grav\Common\Processors\PagesProcessor->process() #7 /var/www/sites/steyaert.be/web02/sites/marleenlabbeke/system/src/Grav/Common/Grav.php(379): Grav\Common\Grav->Grav\Common{closure}() #8 [internal function]: Grav\Common\Grav::Grav\Common{closure}('pages', 'Pages', Object(Closure)) #9 /var/www/sites/steyaert.be/web02/sites/marleenlabbeke/system/src/Grav/Common/Grav.php(355): call_user_func_array(Object(Closure), Array) #10 /var/www/sites/steyaert.be/web02/sites/marleenlabbeke/system/src/Grav/Common/Grav.php(133): Grav\Common\Grav->__call('measureTime', Array) #11 /var/www/sites/steyaert.be/web02/sites/marleenlabbeke/index.php(52): Grav\Common\Grav->process() #12 {main} [] []
I updated the .tgz on https://steyaert.be/marleen.tgz.

Can you get the whole traceback for the error? Especially I need the file/line in the form plugin.

Oh, this was the entire error I got from the logs, the page in my browser only gave me this error: https://cl.ly/0Z0U0o1K2b0J even though I have debug logging enabled.

Please change Error handler -> Display errors to Full Backtrace Error in the configuration.

Oh ok, this is the full traceback: https://cl.ly/0o3a293v1D2O

I believe I found the bug. Already fixed it in the form plugin.

Did you fix it in v2.11.4 because I am already running this version. Or will you release a new version with this fix?

It is released now.

Hi, this release has solved the error I got but the initial problem still exists. I am not able to send out any mails with the form.


Remove archived pages from mirorring page list

Short description

This PR removes archived pages from the list of mirroring pages.
Proposed changes

- Exclude explicitly or implicitly archived pages from page.mirroring_pages

Change the field mirrored_page deletable
Allow page deletion/archiving when there is no mirroring page or only mirorring pages that are archived (in other word: prevent deletion/archiving only when there are non-archived mirorring pages).
Make migration to set mirrored_page to None if a page is archived or belongs to an archived region.
Set mirrored_page of a page object to None when it gets archived or belongs to a region which gets archived

Side effects

When a page A is mirroring another page X and A gets archived, now X can be archived too because A is archived. But when A gets restored, the problem occurs that A is mirroring after the restration not X but the first page of the region, which was exactly the reason why we wanted to prevent archiving of mirrored pages and show mirroring pages as list. Therefore the field mirrored_page gets set to None when a page becomes archived.

Resolved issues

Fixes: #2365

Pull Request Review Guidelines

@david-venhoff that's a good idea 😃 If the next reviewes are for it, I'll change the implementation in that way.

@timobrembeck @david-venhoff
Your sugestions are now applied 😸

It seems removing of archived pages does not work if redis is activ. I'll do an update for that.

Removing of the target pages does not work correctly/quickly if redis is activated. Cache invalidation is added.


[ENH] Add Hidalgo annotation

Reference Issues/PRs
Drafting a template for the Hidalgo algorithm, an estimator in the annotation module (issue #2813). Ideas will be discussed as a community about an annotation base class object, what it could look like.
What does this implement/fix? Explain your changes.
Introduces a new concept/algorithm for segmentation given a (multivariate) timeseries.
intrinsdim.py -> output is scalar value representing intrinsic dimensionality
hidalgo.py -> extends the above model, assuming the data exists on a mixture of manifolds (each with corresponding dimension), output is a vector of size N x 1 corresponding to each the multivariate input of size N x c. For example, if we assume there are K=3 manifolds then an output vector for N=10 would look like (d_1, d_3, d_2, d_2, d_2, d_3, d_1, d_1, d_1, d_1), where d_k is the dimensionality for manifold K.
Does your contribution introduce a new dependency? If yes, which one?
TDB.
Hidalgo relies on Gibbs sampling (MCMC) with a custom likelihood function in the model. Specifically, it is assumed that the neighborhood of a point is more likely to contain points from the same manifold than from different manifolds. This requirement is enforced with an additional term in the likelihood that penalizes local inhomogeneity [points belonging to different manifolds].
Options for implementing the Gibbs sampler include:

including the C code from https://github.com/micheleallegra/Hidalgo/blob/master/python/gibbs.c
re-writing the model and Gibbs sampler in python
interfacing pymc, seeing if our model can be fed into an 'out-of-the-box' Gibbs method

Notes
References: https://arxiv.org/abs/1902.10459

[x] Gibbs sampler code from this repository has been rewritten (c++ to python)
[x] Unit tests written as reference
[x] Paired programming
[x] Example notebook created

@lmmentel @Lovkush-A @fkiraly @miraep8

@KatieBuc is this ready for review? :)

@aiwalter why yes, yes it is*
*currently running all tests, assuming all is well or that I'm busy fixing things in the next few hours/day.

Tests are passing! Okay, so this is Hidalgo as a Transformer.
A couple of questions:

Since predict can't write to self, how else can we get important algorithmic information? There are various outputs from Gibbs sampling, i.e.

        # _d = np.mean(bestsampling[:, :K], axis=0)
        # _derr = np.std(bestsampling[:, :K], axis=0)
        # _p = np.mean(bestsampling[:, K : 2 * K], axis=0)
        # _perr = np.std(bestsampling[:, K : 2 * K], axis=0)
        # _lik = np.mean(bestsampling[:, -1], axis=0)
        # _likerr = np.std(bestsampling[:, -1], axis=0)

which are commented out for now - any ideas?

depending on the operating system, the dtype of an array was either printed, or not - which meant the doc string example matched depending on the operating system i.e.

PASSED: Install and test / test-windows (3.9) (pull_request)

FAILED: Install and test / test-unix (3.8, ubuntu-20.04) (pull_request)

2022-08-23T10:40:15.1569321Z ___________ [doctest] sktime.transformations.series.hidalgo.Hidalgo ____________
2022-08-23T10:40:15.1570690Z [gw1] linux -- Python 3.8.13 /opt/hostedtoolcache/Python/3.8.13/x64/bin/python
2022-08-23T10:40:15.1570983Z 074     Examples
2022-08-23T10:40:15.1571236Z 075     --------
2022-08-23T10:40:15.1571528Z 076     >>> from sktime.transformations.series.hidalgo import Hidalgo
2022-08-23T10:40:15.1571842Z 077     >>> import numpy as np
2022-08-23T10:40:15.1572080Z 078     >>> np.random.seed(123)
2022-08-23T10:40:15.1572319Z 079     >>> X = np.random.rand(10,3)
2022-08-23T10:40:15.1572548Z 080     >>> X[6:, 1:] = 0
2022-08-23T10:40:15.1572810Z 081     >>> model = Hidalgo(K=2, burn_in=0.5, n_iter=50, seed=10)
2022-08-23T10:40:15.1573081Z 082     >>> Z = model.fit_transform(X)
2022-08-23T10:40:15.1573288Z 083     >>> Z
2022-08-23T10:40:15.1573483Z Expected:
2022-08-23T10:40:15.1573885Z     array([0, 0, 0, 0, 0, 0, 1, 1, 1, 1], dtype=int64)
2022-08-23T10:40:15.1574106Z Got:
2022-08-23T10:40:15.1574297Z     array([0, 0, 0, 0, 0, 0, 1, 1, 1, 1])

so I made it Z.tolist(), hoping that's okay? open to other suggestions!
@fkiraly @lmmentel @aiwalter @Lovkush-A @miraep8

also @lmmentel I think it's pretty trivial to make gibbs_sampling a non-class method, if we just pass in
        zeta = self.zeta
        q = self.q
        K = self.K
        n_iter = self.n_iter
        fixed_Z = self.fixed_Z
        use_Potts = self.use_Potts
        estimate_zeta = self.estimate_zeta

these variables instead, and a couple of those then into _update_zeta_prior, could do this if you think it's important?

nice @KatieBuc 🎉

Since predict can't write to self, how else can we get important algorithmic information?

I guess you mean transform? Good question if fit is empty, maybe we need to make an exception here that we allow state changes, what do others think?

depending on the operating system, the dtype of an array was either printed, or not - which meant the doc string example matched depending on the operating system i.e.

Is it possible that you use one of our sample data, e.g. load_airline, load_shampoo_sales, load_longley? Then the output would be pandas. If you want to show output, it might help to shorten the data or just take load_longley, its quite short so output will be short as well.

Since _transform can't write to self, how else can we get important algorithmic information?

A simple solution to this problem I've used before:

run the parts that write to self in _fit
in _transform, check whether the data is identical to the data seen in fit. If yes, run the rest. If not, run the entire algorithm, but do not write to self.

A simple solution to this problem I've used before:

run the parts that write to self in _fit
in _transform, check whether the data is identical to the data seen in fit. If yes, run the rest. If not, run the entire algorithm, but do not write to self.

Hmm in this case, if we separate logic into fit and predict/transform, then the data must be identical (and errors will/should be thrown otherwise). Also, the commented out variables are the last task of _transform, there's really nothing else to run anyway - it would be cool to have a kind of get_params/set_params but for auxiliary outputs; parameters that are found in predict or transform that are of interest for subsequent analysis. What do you think?
What is the simplest solution to this for now? Even if we put all the logic in _fit, and refit to ensure the algorithm can run, we still don't get the fitted_model returned i.e.
def _transform(...):
    if np.allclose(X,fitted_model.X):
        return fitted_model.Z
    fitted_model = _fit(...)
    return fitted_model.Z

@fkiraly

@fkiraly ignore my last comment^^
just put everything in fit, see last commit - should be fine now? (i.e. no separation of logic, passing required unit tests, predict is basically just selecting what to return from the fitted object)

:tada:


print(os.date("%s")) crashes the sim

Place print(os.date("%s")) crashes the sim in main.lua and it crashes.  It should raise an error at the very least,

This quite a strange crash indeed.
It also doesn't seem to crash a built project. For instance, you can run the same code over at https://www.solar2dplayground.com/ and it'll just print "%s".

Is this only happening on Windows? I just tried on my Mac, cannot reproduce

Yeah, we're both using Solar2D Simulator on Windows.

This is fixed with 2022.3685+


Loading prefetched small variants results is slow

Describe the bug
Loading prefetched small variant resuls is slow.
To Reproduce
Create small variant query. Make varfish fetch the result. Look at the query results.
Expected behavior
Should be quite speedy.
Additional context
N/A

Root Cause Analysis
The query to fetch the small variant results does not incorporate case_id, thus all table partitions need to be queried.

Resolution Proposal
Make the queries to fetch the results also consider case_id to make queries on the partition tables fast.
Affected Components
Varfish server.
Affected Modules/Files

variants.queries

Required Architectural Changes

None

Resolution Sketch

See above, make the queries also check the case_id column.


[beta-1.66] Backport fix for git2 safe-directory disable

This is a beta backport of #11366.

@bors r+

:pushpin: Commit 6774bc1118ca114d536f94db34d671c6699c4284 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 6774bc1118ca114d536f94db34d671c6699c4284 with merge 20e4f6082183b9e559b330ffb40125c3b09299bf...

:broken_heart: Test failed - checks-actions

I added #11332 and #11335 to get CI passing.

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing d65d197ad5c6c09234369f219f943e291d4f04b9 to rust-1.66.0...


Email address parser

Hi,
If the email address with quotes in the name is parsed (for example, "Gogh Fir" <tesst@test.com>), then the flag Quoted will be set, indicates then the local-part needs quotes too. As a result, String () will return "tesst" @ test.com.
This is due to the fact that the QcontentSMTP () function is called when parsing the name in quotation marks and mistakenly sets the flag for local part.
Thanks.
package main

import (
	"log"
	"github.com/flashmob/go-guerrilla/mail"
)

func main() {
	addr, err := mail.NewAddress(`"Gogh Fir"<EMAIL_ADDRESS>	if err == nil {
		log.Println(addr.DisplayName)
		log.Println(addr.String())
	}
}

Gogh Fir
<EMAIL_ADDRESS>

Thanks for reporting this! It's a bug that needs to be addressed. Would you
be interested in submitting a PR?
On Sun, 26 Apr 2020, 10:45 int01<EMAIL_ADDRESS>wrote:

Hi,
If the email address with quotes in the name is parsed (for example, "Gogh
Fir" tesst@test.com), then the flag Quoted will be set, indicates then
the local-part needs quotes too. As a result, String () will return "tesst"
@ test.com.
This is due to the fact that the QcontentSMTP () function is called when
parsing the name in quotation marks and mistakenly sets the flag for local
part.
Thanks.
package main
import (
"log"
"github.com/flashmob/go-guerrilla/mail"
)
func main() {
addr, err := mail.NewAddress("Gogh Fir"<EMAIL_ADDRESS>if err == nil {
log.Println(addr.DisplayName)
log.Println(addr.String())
}
}
Gogh Fir
<EMAIL_ADDRESS>—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/flashmob/go-guerrilla/issues/207, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE6MP5ZLNSTY4L57W6VDLTROOG3DANCNFSM4MRAEH4Q
.

It seems enough in the file "address.go" in the displayName() function add s.LocalPartQuotes = false.
func (s *RFC5322) displayName() error {
	defer func() {
		if s.accept.Len() > 0 {
			s.addr.DisplayName = s.accept.String()
			s.accept.Reset()
			s.LocalPartQuotes = false // !!! THIS LINE !!!
		}
	}()
	// phrase
	if err := s.word(); err != nil {
		return err
	}
	for {
		err := s.word()
		if err != nil {
			return nil
		}
	}
}


Package Setup Issues in lucide-vue-next Leading to Module Resolution Errors

Package

[ ] lucide
[ ] lucide-angular
[ ] lucide-flutter
[ ] lucide-preact
[ ] lucide-react
[ ] lucide-react-native
[ ] lucide-solid
[ ] lucide-svelte
[ ] lucide-vue
[x] lucide-vue-next
[ ] Figma plugin
[ ] source/main
[ ] other/not relevant

Version
latest
Can you reproduce this in the latest version?

[X] Yes
[ ] No

Browser

[ ] Chrome/Chromium
[ ] Firefox
[ ] Safari
[ ] Edge
[ ] iOS Safari
[ ] Opera
[X] Other/not relevant

Operating system

[ ] Windows
[ ] Linux
[ ] macOS
[X] Other/not relevant

Description
We're encountering an issue where the lucide-vue-next package seems not to be properly configured, resulting in module resolution errors. This issue has been observed when attempting to import the package, as shown in the error message below:
Error: Cannot find module '/Users/christian/Workspace/ark-ui/node_modules/lucide-vue-next/dist/esm'

Additionally, upon reviewing the package.json for lucide-vue-next, numerous warnings and suggestions have been identified, which might be contributing to this problem. Here is a screenshot of the linting output:

Could we look into improving the package setup to prevent such issues? Suggestions and contributions to resolve these warnings are also welcome.
Steps to reproduce
Go to<EMAIL_ADDRESS>and check warnings.
Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)

Thanks for reporting will check these issues!

I am having the same issue. I am trying to write Stories for my components library using Histoire which uses vite.js and the reported error is thrown. This basically prevents me from either writing Stories/Documentation or from using these icons.

through some debugging, I found that if the file node_modules/lucide-vue-next/dist/esm/lucide-vue-next.js would be renamed to index.js (node_modules/lucide-vue-next/dist/esm/index.js) then it works just fine.

To fix this problem so that I could write Stories/Documentations I had to change the alias in the Vite build to pass the full path of node_modules.
That way it worked:
import type { UserConfigExport } from 'vite'

const viteConfig: UserConfigExport = {
  resolve: {
    alias: {
      'lucide-vue': resolve(__dirname, 'node_modules/lucide-vue-next/dist/cjs/lucide-vue-next.js'),
    },
  },
}


use full path to inputs.list file

The inputs.list file specifying input files for an array job was accidentally left out of the switch to absolute paths in #343
Addresses #345

There are some leftover path errors that cause files to be stored out side the plantit folder.
For example: 'task c4bfcdbc-a2f8-4e4f-a626-e0edf39d7057 on Puma’ shows dirt_out.csv in home directory and in the task directory.
(puma) [bucksch@junonia ~]$ pwd /home/u29/bucksch (puma) [bucksch@junonia ~]$ ls dirt_out.csv  plantit  test (puma) [bucksch@junonia ~]$


Add ref to the Modal content and add props to the ModalBackdrop

Codecov Report

Merging #941 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #941      +/-   ##
==========================================
+ Coverage   99.77%   99.77%   +<.01%
==========================================
  Files         306      306
  Lines        7630     7632       +2
  Branches     1851     1851
==========================================
+ Hits         7613     7615       +2
  Misses         17       17

Impacted Files
Coverage Δ

src/components/modal/Modal.tsx
100% <ø> (ø)
:arrow_up:

src/components/modal/ModalBackdrop.tsx
100% <100%> (ø)
:arrow_up:

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 4da7423...2cdd6e0. Read the comment docs.


Incorrect handling of UTF8 messages

Hi,
the problem:
when you try to send a message with any UTF8 character, it fails
actual result:
it raises with:
/bundle_cache/gems/nats-streaming-0.2.2/lib/stan/client.rb:193:in `initialize': U+0100 from UTF-8 to ASCII-8BIT (Encoding::UndefinedConversionError)
        from /bundle_cache/gems/nats-streaming-0.2.2/lib/stan/client.rb:193:in `new'
        from /bundle_cache/gems/nats-streaming-0.2.2/lib/stan/client.rb:193:in `publish'

As stated here, using #b on message string resolves the problem.

Thanks @wallyqs . Wouldn't it be safer to always use #b on any sending message? I know this is extra effort to encode to ASCII-8BIT, but on the other hand any UTF8 character may crash sending the whole message, which sounds like a much worse scenario. I'd vote for support UTF8 by default.

@blelump I think you're right, it would be both more convenient and handling as bytes using #b throughout would be closer to treating the payload opaquely which is one of the things that we try to do in the clients. Thanks for the feedback!

Just a note, I'm writing payload in JSON format. In this case, ascii_only: true is handy option.
decoded_payload = JSON.generate({ hello: 'こんにちは！' }, ascii_only: true)

sc.publish(subject, decoded_payload) do |guid, error|
end


Cypress tests failing in action

In some cases cypress tests will pass locally but fail when running in the cloud. After some debugging I came across this familiar error:
Automatic publicPath is not supported in this browser

This occurs when code splitting is enabled and the browser cannot determine the host of where to fetch the chunk files. This can be confirmed by disabling code splitting for bundle.js in webpack.dev.js and then the tests pass in the cloud.
The root cause seems to be due to using webpack 5 with this version of cypress but we have other sites that are working so something else may be causing this. Creating this issue to further investigate or how we want to handle this. Cypress just a few days ago released the fix to this and it looks like they're disabling code splitting so that may be an option we could consider (just for the modern bundle.js in webpack.dev.js)
Related links with some workarounds I couldn't get to work:
https://github.com/cypress-io/cypress/issues/8900
https://github.com/cypress-io/cypress/issues/18435#issuecomment-972309941

First thing to do should be to test updating Cypress on site in question.

No longer an issue.


[incubator/haproxy-ingress] haproxy-ingress v0.7.1

What this PR does / why we need it:

update haproxy-ingress to v0.7.1
fix wrong default value for controller.initContainers

Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/ok-to-test

/lgtm

/approve


[BUG] Bad cert for rust-atelier.dev

Describe the bug
SSL_ERROR_BAD_CERT_DOMAIN when accessing https://rust-atelier.dev/

Thanks, I’ll look into the GitHub config

Made changes to Pages settings (not sure why it keeps forgetting these). Will take a while to replicate.

Appears to be working again.


Added XSC contract ABI

Crowdstart Capital (CSC) is one of the first technology accelerators to offer its own ERC20 digital tokens. CSC’s overall goal is to support the evolution of the blockchain technology ecosystem and direct this evolution toward the development of enterprise-ready tools. XSC tokens are being directly distributed to the active developer community laying the basis for a cryptoeconomic incentive scheme designed to support the further evolution of the blockchain ecosystem.
Website: http://crowdstart.capital
Email<EMAIL_ADDRESS>Slack signup form: http://bit.ly/2zMX0hm

@crowdstartcapital Do you have a support email, please? https://myetherwallet.github.io/knowledge-base/tokens/token-creators-add-your-token-to-myetherwallet.html

Hi Taylor,
Thanks for reviewing our pull request. Users can always reach us at<EMAIL_ADDRESS>and users specifically in need of technical support can reach us at<EMAIL_ADDRESS>We aren't selling the tokens, but rather giving them away to developers as an incentive for contributing to the blockchain ecosystem so I have not submitted a pull request for adding the token sale address & manual gas limits. Just let us know if you've got any other questions.
Best regards,
Rebecca Johnson


Remove empty types & Rename Misc.go to something better? :D

FileIO.go

Removed empty types, moved some Methods from Misc.go to other classes, renamed Misc.go to errorhandler.
close?


[QA] "Sum a Terra" skin not loading in backpack

Explorer Alpha build versions:

v0.21.0

Issue Description:
The "Sum a Terra" skin does not load correctly in the backpack preview. This issue prevents the skin from being visible or properly displayed in the backpack view.
WEARABLE URN:
https://decentraland.org/marketplace/contracts/0xbf65a0f5a6115ca5cd1ec9c13c993d412eaa009c/items/0
USER LOGS:
https://drive.google.com/file/d/16SGzGSpH4GvFp_Uy0m4A44a6ZQqIWFuf/view
STR:

Launch the explorer.
Open the backpack preview.Equip the "Sum a Terra" skin.
Equip the "Sum a Terra" skin.

Expected behavior:

Current behavior:

Apparently its displaying the legs and it shouldnt. This may be a duplicate of #1770

It is showing it correctly on my end:


[Docs] BackAndroid documentation needed

"To get BackAndroid to work you need to return false in your back event handler. This will prevent exiting the app and you can handle the transition to the previous screen. For an example check out the UIExplorer sample app.
I'm sure I documented this somewhere, but it probably got overwritten at some point (we sync code between a few places and sometimes this happens). Will add it back."
As per: https://corbt.com/posts/2015/09/16/one-day-with-react-native-for-android.html#comment-2256643777

Note that this isn't just a documentation issue -- using the default template project BackAndroid won't work at all because the onBackPress event is never handed to the JS. I've opened #2801 to amend this.

Also @brentvatne I think your description of the return is backwards -- to indicate that the event has been handled I need to return true.

Awesome, thanks @corbt!

This has been fixed.


Show leverage on Dashboard Positions

Suggestion
Positions don’t show leverage so hard to know which position is close to being liquidated, also type of margin is unclear (is it deposited margin, accessible margin, or remaining margin)

Totally agree with this one. At the very least we should display position size, potentially leverage as well.
Another suggestion is a way to flag positions that can be liquidated or are close to liquidation.


Added functionality to 'dcos marathon task stop <task_id> --wipe' in …

…order to wipe stateful services
Background:
I’ve started writing a deep dive on setting up stateful services in marathon. One thing is wiping a task from state including reservations and persistent volumes. That’s possible in marathon via a POST to /v2/tasks/delete (yes, POST to /delete, don’t ask) as described here https://mesosphere.github.io/marathon/docs/rest-api.html#post-v2-tasks-delete – for the sake of having the deep dive tutorial merely on dcos cli without any direct marathon api interaction, I’ve added the tasks delete command. It doesn’t cover all possibilities of the marathon api, but it would be good to be able to wipe tasks via the cli, I guess.

LGTM. Thanks for adding this :)


Repository status?

Are there plans to start work on this sometime soon? I'm really missing the support in C#

Ah, I guess https://www.nuget.org/packages/Google.Apis/ exists. What's the status of this repo, then?

This repo is going to consist of wrappers around lower level API libraries to make them significantly easier (and more idiomatic) to use.
You can see some prototyping work in my fork here:
https://github.com/jskeet/gcloud-dotnet/tree/experimental-gcs

Now that I've started merging things into the repo, hopefully things are becoming clearer. Closing for now - feel free to reopen.


Feature/automated testing

This adds a ScanRunnerPart for testing a scan block. Scan parameters are first loaded from a user-written YAML file, and then the part can be ran to go through each set of parameters. The part writes scan data and a small report file inside a defined top-level output directory.

Codecov Report

Merging #281 into master will decrease coverage by 0.05%.
The diff coverage is 84.4%.

@@            Coverage Diff             @@
##           master     #281      +/-   ##
==========================================
- Coverage   92.22%   92.16%   -0.06%
==========================================
  Files         206      207       +1
  Lines       10482    10735     +253
==========================================
+ Hits         9667     9894     +227
- Misses        815      841      +26

Impacted Files
Coverage Δ

malcolm/modules/scanning/parts/__init__.py
100% <100%> (ø)
:arrow_up:

malcolm/modules/scanning/blocks/__init__.py
100% <100%> (ø)
:arrow_up:

malcolm/modules/scanning/parts/scanrunnerpart.py
84.27% <84.27%> (ø)

malcolm/modules/pmac/parts/beamselectorpart.py
93.33% <0%> (+6.66%)
:arrow_up:

...les/ADPandABlocks/parts/pandaalternatingdivpart.py
83.33% <0%> (+35.18%)
:arrow_up:

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 7138264...c23185a. Read the comment docs.

I combined your suggestion into a single object as the generator contains all of the physical parameters.
Let me know if there is anything else I should modify - otherwise I believe it is ready.

I have added abort which first stops the stored context and then aborts the current scan (I believe this is the correct order?).
More tests were also added to improve coverage.

I believe this is now at the point ready for merging, if you are happy @thomascobb


Add list navigation to header menus

Floating-ui has a "hook" to enable this, for which the floating implementation in appkit needed some massaging in order to work correctly with nested and/or variable menuitems in header menus.
Theoretically, this shouldn't break anything, but I think it would be best to test this with each application that currently uses appkit. At least with Tobira, but maybe also Studio and Editor.
Tobira has a related PR which is necessary for this to work correctly: https://github.com/elan-ev/tobira/pull/997.
This PR is related to the issue https://github.com/elan-ev/tobira/issues/753.

Humph... I've ran into another issue introduced by the suggested changes that I don't understand. Specifically by passing the context.getItemProps() thing. The onClick handling in the series block sorting menus is broken with this, but I suppose that is better discussed in https://github.com/elan-ev/tobira/pull/1001. And maybe easier to debug for other people (cough, cough, you) once this PR here is merged.

Neat!


fix: devShells default output

The flake output devShell has been deprecated in favor of devShells for 2 years now:
https://nixos.org/manual/nix/stable/release-notes/rl-2.7.html?highlight=devShell#release-27-2022-03-07
Nix itself already warns about this, and it is feasible that at some point the backwards compatibility will be dropped.

@a-kenji
After fixing CI settings, I'll merge this PR. Please wait a sec.

No need to rush :).

Oh no it'sy bad :<

No worries, stuff happens :).

@a-kenji my nix doesn't warm about it tho, so I'll look into that

@Sigmanificient try running:
nix flake check github:kyu08/fzf-make/fe8fde47880317ec20a77f23735c4fb45e5d410d

6 month in, never stop learning new stuff about nix, what a rabbit hole


TODOs

Basic Information
Frontend

[x] Wipe the boilerplate
[x] Install tanstack query
[x] Install a ui library (probably shadui)
[x] Install axios for easier handling of the sessions (see https://github.com/codersforcauses/csf)
[ ] Enforce code style + file naming conventions with a stronger linting config

Backend

[x] Setup postgres
[ ] Some boilerplate for verifying CRUD works
[ ] Install and setup djangorestframework-simplejwt

Other

[ ] Dockerise backend (or everything) for deployment (maybe)
[ ] CI

[ ] Lint and formatting checkers
[ ] API tests

[ ] Setup mkdocs boilerplate for documentation (see https://github.com/codersforcauses/csf)

Other Information
[Are there any additional information that we should know about?]

Can't dockerise everything whilst in a dev container https://github.com/codersforcauses/automated-setups/issues/10


Allow checks to be documented which are unenforced.

For example:
// SomeFunc requires:
// * "some other func called first"

Could you please clarify? You can always make a condition as a constant true and include a description. Is that what you had in mind?

I thought about adding a clarification to the readme, but actually decided against it since we forbid unverified contracts in our code base at Parquery due to potential bugs related to documentation rot.


[Serialization] Serialize internal closure labels

Since https://github.com/apple/swift/pull/36551 we are keeping track of internal closure labels in the closure’s type. With this change, wer are also serializing them to the swiftmodules.
Furthermore, this change adjusts the printing behaviour to print the parameter labels in the swift interfaces.
I’m not sure if printing the parameter labels in all cases (especially print_types.swift) is actually beneficial because I don’t have a detailed understanding of what they are used for.
Resolves rdar://63633158

@swift-ci Please test

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 0dbec0df26797db6a12dde5b8d67c113247db5e4

@swift-ci Please test


Replace HasCurrentUser mixin with injection

We were basically mixing HasCurrentUser into everything anyways so I figured "let's just inject it into the controllers and get it over with"
I'm making a big assumption that Ember lazily injects to solve infinite recursion problems, but I think it's fairly safe.  We'll know for sure once Travis builds it.
Evidently I was wrong, I'll switch this to an ObjectProxy-based service (those are an official role now) since ObjectController is deprecated anyways.  Not a complex refactor.
Oh and on the topic of injection, we can probably reenable model injection in app.js now that the patch landed for that

we can probably reenable model injection in app.js now that the patch landed for that

As far as I'm aware, that was introduced in Ember 1.9+ and we are still on 1.8. I'm playing with updating Ember and its dependencies in a branch at the moment, so I can switch it there.

ObjectProxy approach works well, I used that in a different Ember app a few months ago.

And finally, 3 months later, this is done and ready for review :dancer:

I don't think you commited the services file

Balls. Brb lemme go do that

Fixed

Looks good :+1:
Gonna need some updating with the Ember update

Yeah I figured it'd need some adjusting for that, but it should be mostly the same, besides having to rebase.  Probably a lot of merge conflicts.  I guess that image is quite fitting then, isn't it?


Derivation outputs order

In the readme and the examples, this pattern can be observed:
hello-world = (rustPkgs.workspace.hello-world {}).bin;
#                                                ^^^^ this part

If the .bin is intended to be used by default, is there any reason why outputs is defined like this?
https://github.com/cargo2nix/cargo2nix/blob/c149357cc3d17f2849c73eb7a09d07a307cdcfe8/overlay/mkcrate.nix#L319
In particular, putting "bin" first would make that the default so the .bin stuff could be omitted. From the manual:

The first element of outputs determines the default output.

I found it useful to swap the order here, for example, but maybe I'm just doing something wrong.

Takes 4 lines.  I had to make the tera output use the out attribute explicitly, but other than that, it looks like a naive swap.  I'll get it into the unstable branch.  Right now I'm focused on fixing features for target-specific dependencies / features.  That will be the 0.12.0 release pretty soon.

Nice!
I'm wondering if, since the bin output only includes $bin/bin/{the,built,binaries} while out contains $out/{bin,lib,nix-support}, if "bin" should only come first for top-level builds, since it seems like the stuff in out contains the rlibs which I'm guessing are used for dependency-building stuff.

AFAIK "top-level builds" are derivations not referenced downstream in dependent derivations.  We don't know when we build one if it's top level or not.  Re-ordering the outputs depending on the situation would create two unique derivations, but if we only use one of the outputs from both, it's virtually the same as having two outputs.  The difference is choosing outputs versus choosing derivations.
The oddball for our output is that we include not just rlibs but linking info that needs replayed downstream.  As things are, a crate derivation doesn't know its linking info up front.  It gets it by looking at the output of the upstream dependencies.  This is what a lot of behavior in mkcrate.nix actually is, recording and replaying linking information.

Closing this.  Please use the unstable branch to include changes discussed in this issue.
Fix will ship in 0.12.0 release as well.


Change default branch to master

This should be more obvious to people unfamiliar with GitHub Pages.

Merging despite flaky HTML proofer link detection failure.


feat(cli): Add size-plugin

What kind of change does this PR introduce?
Feature
Did you add tests for your changes?
No
Summary
Adds size-plugin for pretty output size tracking & display.

(the same PR as https://github.com/developit/preact-cli/pull/624, raised on next branch, just so that the commit log doesn't get messy)

can you post a pic with --esm flag as well?

@prateekbh with --esm flag during build

@developit anyway we can sort this alphabetically so everything is comparable to each other?

any reason we're holding this off?


[Storage] Expose services to be configurable for account SAS generation

As title states 😄

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-blob


Android : If QR code is scanned iteratively it stops fetching wallet address

What happened:
QR scan stops fetching wallet address from second scan onwards
What you expected to happen:
QR scan should fetch wallet address regardless of attempts
How to reproduce it (as minimally and precisely as possible):
Balance > Up arrow in Portfolio card > Click QR scan
Anything else we need to know?:
No

only happens on android, works on ios


base64: invalid input

When I use iTerm2 terminal and work with the file, it shows:

base64: invalid input

But still generating some part of a wrong certificate in hex and generating a wrong SMS Retriever hash code as well.
But when I use regular Terminal, it works well.
A sample wrong output:
➜  Desktop ./sms_retriever_hash_v9.sh --package com.example.test --keystore /path/keystore.jks

package name: com.example.test
keystore file: /path/keystore.jks

File /path/keystore.jks is found.

Enter keystore password:  password
base64: invalid input

certificate in hex: 328202c5308201ada00362010202043d1336f8300d06092a864486f70d01010b050030133111300f0603550403430853

SHA-256 output in hex: bf0ae40bf16485b2320765f3c824e263dfba8d09ec46307d6b4dd3153cfbc287

First 8 bytes encoded by base64: ywrkS/HkhbI

SMS Retriever hash code:  ywrkS/HkhbI

Machine: Mac OS 10.14.5
iTerm2 version: 3.2.9

Please add --ignore-garbage to the command:
# Retrieve certificate from keystore file. Decoded with Base64 and converted to hex
cert=$(keytool -list -rfc -keystore $keystore | sed  -e '1,/BEGIN/d' | sed -e '/END/,$d' | tr -d ' \n' | base64 --decode --ignore-garbage | xxd -p | tr -d ' \n')

So it can solve the invalid input error in some terminals, like iTerm2.
https://gitlab.com/gnachman/iterm2/issues/7912

I am closing this issue/PR, as it has been migrated to the new repo linked above in the comments. Thank you!


Listener address/port

I tried to run the example grammy server from the documentation. But how do I know what host/port the server is listening to? I'm trying to read the code but there's no obvious answer(at least to me)

Found it, it's 4000.

I should definitely log that out though, so thanks for raising it!

This is printed when you start the server.  Thank you!


Misc questions

First of all thanks for this great lib !
I have a few questions about the runtime:
The lib uses custom base64 functions. Have you considered using the platform instead ? (Buffer on node and atob and btoa in the browser). I understand that the current decode might be more lenient but I'm not sure if this is really needed and if it is it should not be to hard to implement.
Have you considered using const enums in the runtime ? i.e. for ScalarType. This could also save some bytes - it would technically be a breaking change if someone uses the enum in their code.
I am happy to submit PRs for those if it can help.
Thanks

Hey @vicb,
const enums
Earlier versions did use const enum, but this caused problems:
const enum in a module breaks the react build process. React enforces the TypeScript compiler option isolatedModules, which does not allow const enum. It is likely that there are other situations where a user needs isolatedModules.
Therefore, const enum was dropped in #4. This increased the webpack build size by ~4%. But code size is still a fraction of all other protobuf libs I know.
base64
The functions compile to about 1.6kb minified JS. I think it should be possible to drop 1kb by using platform features. That would be a small but considerable improvement.
The functions are used by the JSON format (reflection-json-reader.ts, reflection-json-writer.ts) and grpc-web-format.ts.
The leniency in decoding has two reasons:

compatibility with the protobuf JSON format
Compatibility is currently tested with the protobuf conformance test suite. The test suite only runs against nodejs. When we introduce platform specific code, some more test coverage for base64 is necessary. Unit tests can be added to the runtime, which is tested against both nodejs and browser.

allow inner padding for easy reading of chunked grpc-web response
I think it is fine to rewrite the grpc-web function and drop support for inner padding.

A PR would be welcome!

Thanks for you quick and detailed answer.
Good to know about const enum breaking react.

But code size is still a fraction of all other protobuf libs I know.

I used mapbox/pbf before which generates really tiny files - but no typescript and not maintained either.
I'll send a PR for the base64 functions in the coming days.

Regarding base64, atob() and btoa() only support unicode string but not arbitrary binary data.
It means that it would not be possible to use them in the browser where the space saving would be most required :(
It's a shame but I have to close this issue. Sorry.

No worries, @vicb
If you have any other suggestions, please let me know.
BTW, thanks for pointing out mapbox/pbf!  Added it to the code size benchmarks:

generator
version
optimize for
webpack output size

pbf
3.2.1

22,132 b

protobuf-ts
1.0.5
size
42,353 b

protobuf-ts
1.0.5
speed
72,855 b

ts-proto
1.26.0

111,825 b

google-protobuf
3.12.2

396,934 b

Looks like we have a new target to beat :)

@timostamm before finding your great plugin I thought about creating mine so I have some ideas on what I would like, maybe I can share some of it if you are interested. I could also help implementing some of those if you are interested.

Use JS Map instead of objects for maps - whatever way should be documented,
Being able to control what is generated, i.e. drop JSON for people who are not using it. Merge and is could also be candidate,
Use ts-morph instead of tapping directly in the TS API. Probably to be considered only if the code generation should be changed (i.e. to implement the previous bullet point)

It would be nice to add an eslint config to the project to automate code formatting and ease contributions.

JS Map:
The problem with Map is accessing the items. Map.keys(), entries() and values() return IterableIterator.  Using IterableIterator in for loops requires the TypeScript compiler option `downlevelIteration, which is still disabled by default. See stackoverflow.
Control what is generated:
Yes, it would be very beneficial if unused features would be excluded in production builds.
But I am not convinced that plugin options are the way to go. I think the generated code should be as stable as possible. A plugin option should not break userland code if possible.
There is an alternative I have been thinking about. If there was a function toJson(msg) instead of MyMessage.toJson(msg: MyMessage), tree shakers would strip the function from production builds if it is not used. If you are interested I can commit this in an experimental branch.
tslint:
Absolutely agree.
ts-morph:
The compiler API is painfully verbose. ts-ast-viewer.com helps a lot... Have no experience with ts-morph. Might be worth it.
Other ideas:

Make it easier to work with reflection. Goal: easily generate a form for a protobuf message (including validation with custom options, for example max-length, etc.).
Documentation: better readability, switch to docusaurus?

Map
The easy solution is to use Array.from(...).
Slightly less easy but more memory efficient is to call next().
What is generated
Agreed that compiler options might not be ideal... but it's typescript, you would now right away if you miss a method and stripping code would be opt-in. I think mapbox/pbf also allow to generate read or write only methods.
But if there is a way without compiler options it might be better.
Other topics
What about creating issues for all know item to work on ?
Could be one issue per identified item + one "misc issues" with smaller/exploratory ideas.


AssertionError on delete files

I got this error when deleting files from admin "Filer folders" view:

error appears after press "Yes" in confirmation form of delete_view
but when i try to delete from simple view by admin action "Delete selected.."(http://<IP_ADDRESS>:8000/admin/filer/image/) i got no errors.
using filer latest from develop branch

@itatelier Official Django 1.10 support will be released in 1.2.6. We are going to release an rc today and a final shortly after

still have this issue on version 1.2.6

I'm also having this problem. Did you found any solution?

Nope.. waiting =)

Any news about it?


Cannot create IdentityPolicy with Principal Service

In version 0.7.0 it is not possible to create an IdentityPolicy with an AWS service as the principal.
Example of such a policy
  IdentityPolicy:
    Type: Custom::IdentityPolicy
    Properties:
      Identity: !GetAtt 'DomainIdentity.Domain'
      PolicyName: AllowMailFromCognito
      PolicyDocument:
        Version: '2012-10-17'
        Statement:
          - Effect: Allow
            Principal:
              Service: cognito-idp.amazonaws.com
            Action:
              - ses:SendEmail
              - ses:SendRawEmail
            Resource: !Sub 'arn:aws:ses:${AWS::Region}:${AWS::AccountId}:identity/${DomainIdentity.Domain}'
      ServiceToken: !Sub 'arn:aws:lambda:${AWS::Region}:${AWS::AccountId}:function:binxio-cfn-ses-provider'

This fails with the error message in CloudFormation:
Received response status [FAILED] from custom resource. Message returned: invalid resource properties: <instance> is not valid under any of the given schemas
The schema in https://github.com/binxio/cfn-ses-provider/blob/master/src/identity_policy_provider.py#L38 only allows for AWS, but not for Service

Hi @danielvijge, sorry about that .This problem is solved in  version 0.7.1


Fix metadata searches for submission and allow alerts to have multilevel metadata

This patches changes the underlying index template therefor you can't just update a running instance with this patch. We will have to change the index template and reindex all data to keep old data in this new schema.
This should allow submission metadata fields to be queried and should also allow alert metadata to have multilevel data like the submission had.
I have also modified the create_test_data script so it generates multilevel keys for metadata fields so we can test this properly. To test, do the following:

spin up the minimal dev dependency compose file
Launch a web server
run the create_test_data script with the "full" parameter so it creates sample alerts and submissions with single and multilevel metadata keys.
Open the UI in your browser and start doing queries targetting the metadata in the search page.

Codecov Report

Merging #22 into master will increase coverage by 0.03%.
The diff coverage is 93.33%.

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
+ Coverage   60.72%   60.76%   +0.03%
==========================================
  Files         115      115
  Lines        9833     9843      +10
==========================================
+ Hits         5971     5981      +10
  Misses       3862     3862

Impacted Files
Coverage Δ

...emblyline/datastore/support/elasticsearch/build.py
88.46% <75.00%> (-0.76%)
:arrow_down:

assemblyline/odm/base.py
88.99% <100.00%> (+0.05%)
:arrow_up:

assemblyline/odm/models/alert.py
100.00% <100.00%> (ø)

assemblyline/odm/randomizer.py
95.75% <100.00%> (+0.58%)
:arrow_up:

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 2939f96...7e67d8a. Read the comment docs.


Official F-Droid Release

I see that the app has been released on Izzy's repo in issue #9, but some people do not want to add another repo to F-Droid.
It would be nice if it can be added to the official F-Droid repo.
This is the inclusion policy that must be met to be able to submit. More info about submitting can be found here.
Here is the issue tracker to request your package to be available on F-Droid. After submitting, they can assist on what is needed to be done (if anything) to get it on F-Droid.

Will do. Thanks for the links!

Will do. Thanks for the links!

This will take a a bit of time to get done. Facing issues trying to work with fdroid's app inclusion tools.

This will take a a bit of time to get done. Facing issues trying to work with fdroid's app inclusion tools.

Refer MR on fdroid

PSA (because lots of activity has happened over at the fdroid repo which is not reflected here): The F-Droid inclusion process is on hold until a certain dependency is removed, making the inclusion process easier and less of a maintenance burden for everyone involved.

@veejayts What's the latest update on this? Were you able to move away from that dependency?

Still being worked on over at the dev_v3 branch.

@veejayts Thanks!
Curiosity piqued. What made you decide on the OpenCV 3.x series instead of 4.x?

Any update on this?

https://gitlab.com/fdroid/rfp/-/issues/2336

Any more updates? I can't find the app on fdroid.

last update is from 2021 https://gitlab.com/fdroid/fdroiddata/-/merge_requests/8336#note_505861119

+1

Are there any plans to add this application to the official F-Droid repository?

Well, being delisted from google play store doesn't have much relation to NOT being listed on fdroid. Am I wrong?

  No promises, currently I'm the only one working on this app, I'll make improvements, fixes and releases when I find the time.
FYI The dependency itself is not closed source, it's a binary build dependency which is not hosted on any repo (afaik) so it has been bundled directly.
Why can't it be unbundled easily? It's the core dependency of the app and an Android specific library (https://github.com/jhansireddy/AndroidScannerDemo). I'm re-implementing this core functionality in Flutter instead of tweaking the existing library in Java and compiling this to fit our app. Hosting it is no longer viable either since it is kinda old at this point and will require a rework in of itself.
Aside from this, Android has had a lot of updates regarding app permissions which I'm honestly yet to fully look into.


Bootstrapped core dependencies: context and nats connection.

Right now there isn't any global context created while bootstrapping the application, which causes any extension needing a context to create its own.
Also, any extensions using NATS would want to use the same connection object, so this would probably be well suited to create somewhere in the initiation process as well so it could be passed down as a dependency, much like the context issue above.

Yeah, I did hack this version together, but will need to structure it a bit better.


s2i compoentn fix: use Config instead of ContainerConfig for port detection

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug
What does does this PR do / why we need it:
s2i builder images available on OCP 4.6 has metadata where exposed ports are listed only in Config section.
Previously we were incorrectly using Exposed ports from ContainerConfig.
Which issue(s) this PR fixes:
fixes https://github.com/openshift/odo/issues/3882
How to test changes / Special notes to the reviewer:
odo create nodejs --s2i on OCP 4.6 used to fail, now it should work. you can check issue for more info

ok  	github.com/openshift/odo/pkg/notify	1.024s [no tests to run]
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x1dfa60f]
goroutine 84 [running]:
testing.tRunner.func1(0xc0005e4700)
	/usr/local/go/src/testing/testing.go:874 +0x69f
panic(0x1fede60, 0x324bb20)
	/usr/local/go/src/runtime/panic.go:679 +0x1b2
github.com/openshift/odo/pkg/occlient.getExposedPortsFromISI(0xc00002e000, 0x6, 0xc000481d80, 0x1, 0x1, 0x0)
	/go/src/github.com/openshift/odo/pkg/occlient/occlient.go:604 +0xdf
github.com/openshift/odo/pkg/occlient.(*Client).GetExposedPorts(0xc0000e9290, 0xc00002e000, 0x21bd5b6, 0x6, 0xc00002e000, 0x0, 0x0)
	/go/src/github.com/openshift/odo/pkg/occlient/occlient.go:797 +0x51
github.com/openshift/odo/pkg/occlient.(*Client).NewAppS2I(0xc0000e9290, 0x21b80d6, 0x4, 0x21d7b14, 0x21, 0x0, 0x0, 0x21b7b72, 0x3, 0x21bd5b1, ...)
	/go/src/github.com/openshift/odo/pkg/occlient/occlient.go:838 +0x2ac
github.com/openshift/odo/pkg/occlient.TestNewAppS2I.func1(0xc0005e4700)
	/go/src/github.com/openshift/odo/pkg/occlient/occlient_test.go:1251 +0x107b
testing.tRunner(0xc0005e4700, 0xc000095c00)
	/usr/local/go/src/testing/testing.go:909 +0x19a
created by testing.(*T).Run
	/usr/local/go/src/testing/testing.go:960 +0x652
FAIL	github.com/openshift/odo/pkg/occlient	0.245s

seems like a genuine panic

Can we write unit test for this?

Can we write unit test for this?

@grantomation done

/approve
/lgtm


winston detetction not sufficient

Description
i'm trying to use pino over winston and I have to hack the logger instance to look not like the simpleified winston config.
  // HACK
  // eslint-disable-next-line @typescript-eslint/no-explicit-any
  (logger as any).color = undefined;

Without this, I get
X [MissingPeerError]: Missing peer dependency: winston. Please install it to use the feature.
    at ne (/Users/danielwhite/Repos/prism-service/node_modules/express-zod-api/dist/index.cjs:5:8015)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Mr (/Users/danielwhite/Repos/prism-service/node_modules/express-zod-api/dist/index.cjs:18:1169)
    at async Nr (/Users/danielwhite/Repos/prism-service/node_modules/express-zod-api/dist/index.cjs:18:1838)

Expected
I would not expect express-zod-api to allow pino with no modifications
Reproduction
import pino, { BaseLogger, LevelWithSilent, LoggerOption } from 'pino';

export function createLogger(level: LevelWithSilent): BaseLogger {
  const options: LoggerOptions = {
    level
  };
  const logger = pino(options);
  return logger;
}

Context

Node.js version: 20.11.1
express-zod-api version: 17.2.0

how do you use that function createLogger that you made?
Please provide more context for reproducing the issue, @daniel-white
Please also read the corresponding article in Readme:
https://github.com/RobinTail/express-zod-api?tab=readme-ov-file#customizing-logger

I think I'm getting now what's happening there.
Please stand by....

I made a test for the function isSimplifiedWinstonConfig() against the pino instance as pino({level: "info"})
https://github.com/RobinTail/express-zod-api/pull/1607
And that test passes. The function basically tells whether or not should winston be loaded instead.
Can not reproduce that behaviour so far.
What is your actual version of pino, @daniel-white
Could you run yarn why pino and tell me what is says?

hm interesting i'm using 8.19.0 as well. looks like a good test to have and ill keep playing on my end.

Are you sure you're supplying "info" to createLogger()?
Not one of these: "silent", "warn", "debug" , @daniel-white

Since when it's "debug", the test fails and that's I believe exactly the case, @daniel-white

@RobinTail i was doing a bit more. i'm getting that value from envvars, i opened it up to more levels. it might have been on debug yesterday
I'm allowing
['error', 'warn', 'info', 'debug', 'trace']

@daniel-white ,
🚀 the issue fixed in the recently released version 17.2.1 .
Thank you for the contribution!


Error Importing Tensorflow

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution:  (Windows 10 64 bits)
Mobile device: None
TensorFlow installed from (source or binary):
TensorFlow version:  2.3.0
Python version: 3.8.4
Installed using virtualenv? pip? conda? : pip
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory: Nvidia GeForce 920M 2GB

Describe the problem
When trying to import tensorflow, i got the following error:

Traceback (most recent call last):
File "C:\Users\Muril\Anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 64, in
from tensorflow.python._pywrap_tensorflow_internal import *
ImportError: DLL load failed while importing _pywrap_tensorflow_internal: Não foi possível encontrar o módulo especificado.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "deep_versions.py", line 2, in
import tensorflow
File "C:\Users\Muril\Anaconda3\lib\site-packages\tensorflow_init_.py", line 41, in
from tensorflow.python.tools import module_util as module_util
File "C:\Users\Muril\Anaconda3\lib\site-packages\tensorflow\python_init.py", line 40, in
from tensorflow.python.eager import context
File "C:\Users\Muril\Anaconda3\lib\site-packages\tensorflow\python\eager\context.py", line 35, in
from tensorflow.python import pywrap_tfe
File "C:\Users\Muril\Anaconda3\lib\site-packages\tensorflow\python\pywrap_tfe.py", line 28, in
from tensorflow.python import pywrap_tensorflow
File "C:\Users\Muril\Anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 83, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "C:\Users\Muril\Anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 64, in
from tensorflow.python._pywrap_tensorflow_internal import *
ImportError: DLL load failed while importing _pywrap_tensorflow_internal: Não foi possível encontrar o módulo especificado.
Failed to load the native TensorFlow runtime.

Provide the exact sequence of commands / steps that you executed before running into the problem
# tensorflow
import tensorflow
print('tensorflow: %s' % tensorflow.__version__)

Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
I'm using Anaconda, and installed through pip in the command prompt. Using python -VV i got the following:
Python 3.8.3 (default, Jul  2 2020, 17:30:36) [MSC v.1916 64 bit (AMD64)]
Later, i tried install using conda install -c conda-forge tensorflow, and got the following:
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: |
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found
to be incompatible with the existing python installation in your environment:

Specifications:

  - tensorflow -> python[version='3.5.*|3.6.*|>=3.5,<3.6.0a0|>=3.6,<3.7.0a0|>=3.7,<3.8.0a0|3.7.*']

Your python: python=3.8

If python is on the left-most side of the chain, that's the version you've asked for.
When python appears to the right, that indicates that the thing on the left is somehow
not available for the python version you are constrained to. Note that conda will not
change your python version to a different minor version unless you explicitly specify
that.

The following specifications were found to be incompatible with your system:

  - feature:/win-64::__cuda==10.1=0

Your installed version is: 10.1

@muriloasouza
What is make/model of your cpu?
I suspect your cpu model does not support AVX instructions sets.See hardware requirements
Make sure to download the latest microsoft visual c++ redistributable from here.
.Also, please follow the instructions from to install from Tensorflow website.
Please, check Your CPU/Python is on 32 bits?Please, refer #36167 and see if it helps you.Please, refer similar issues #36167 #36151 #36138 #36054 #36045 #36020 #36003 #35988 #35903 #35880 #35865 #35805 #35789 #35773 #35772 #35767 #35766 #35749 #35721 #35618 #35204
Thanks!

@ravikyram
1 - It's an Intel Core i7-5500U;
2 - Not really sure what microsoft visual c++ i should download there, x64? x86? ARM64?
3 - Do i really need to install those versions of Python (3.6 or 3.7)? I've seen it should work in Python 3.8;
4 - Both my CPU and Python are on 64 bits (as you can see in the python -VV result i posted earlier).


Add boolean parameter about initialized

Hi. I try to use this mod and according setting, sometimes confuse to detect SRanipal is initialized or not.
i'm now trying to detect some geek way (existing param check and not zero), but also think if add boolean param to detect initialize is more confortable.

I don't think I'll be adding a bool parameter to check for module initialization any time soon, you can tell if things have successfully started in the MelonLoader console at the moment. That said, you could modify the source code and add this functionality yourself using ParamLib


Having trouble with push_files

I'm not sure what is going on here.  I am an owner of the organization, but somehow I don't have rights to push_files to student repos.  Here is an example of the error I get.
Authenticating to GitHub...
Found organization at: https://api.github.com/orgs/KJHS-Coding
Adding remotes and pushing files to student repositories.
jakiestfu --> git@github.com:KJHS-Coding/jakiestfu-Python-Tour.git
fatal: remote jakiestfu already exists.
Warning: Permanently added the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

I am running create_team, create_repos, and push_files - in that order.  push_files is the only step that fails.

I had the same problem and this helped me - thanks!


matplotlib cannot be interactive

import matplotlib.pyplot as plt
plt.ion()  # this create the crash
plt.plot([1,2])
A window appears (gui=qt) but gets freeze without any plot. The window must be killed (closed) which crash the REPL.However the prompt was back.
Without plt.ion() it works after calling plt.show(), but the figure cannot be changed dynamically.
Thanks
python 2.7, VS2013pro, PTVS 2.2, matplotlib 1.4.3

We recommend configuring IPython (our doc page) for this. UI and REPLs generally don't work well together unless it's been explicitly designed into both parts (in this case, the UI does not support our REPL).

Thanks for answering.
My original question was related to python interactive, not ipython interactive.
However I did some test, here are the results.
in "python interactive" as described above plt.ion() corrupts the REPL.
in "ipython interactive" there is no need to configure "Interactive Mode:" as "IPython w/o Pylab" as claim in the doc to embed a plot/image within an external window. One just has to:
In [1]: %matplotlib tk
In [2]: plt.plot([1,2]) # pops a window, and give the prompt back
plt.ion() is not needed, it is by default.
PS: In fact I found it is impossible to use "IPython w/o Pylab". Once click "OK", and call "Configure interactive window", this option is back to "IPython".
Regards.

Yes, the fact that plt.ion() corrupts the REPL is why we recommend using IPython mode where it is handled better.
The switching between "IPython" and "IPython w/o Pylab" is a bug we should investigate though.


[Blazor] Referencing the project name

This issue is intended for project maintainers and developers that are building custom tooling and/or projects for Blazor. Now that Blazor is going to be renamed to Razor Components, how are we going to reference the Blazor when describing our project? Are we going to reference to the Blazor or Razor Components?
As I can see the Blazor is mentioned in README along with Razor Components.
Also what will happen with the web page?

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


fix: change key name for new version of semantic release

Need to update branch -> branches for new version

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Improve relationships docs

Hey Artem this looks great, in general.
But, I'm not sure why you're not working in the brandicted repo.  If your PR came from a branch in the brandicted repo, I could do that. But since the PR comes from your repo, I can't.

One topic not covered much: one-to-one relationships. How should the developer decide which side of the relationship to put the relationship field on in this case? Maybe it depends on if they plan to nest the relationship or not.


periodic_adv not working with nRF5340 DK

Describe the bug
periodic_adv sample is not working with nrf5340dk_nrf5340_cpuapp
it is working for nrf52840dk_nrf52840 after adding CONFIG_BT_LL_SW_SPLIT=y to proj.conf
To Reproduce
Steps to reproduce the behavior:
Open the sample from the SDK. build for nrf5340dk_nrf5340_cpuapp
Expected behavior
To work
Impact
showstopper
Logs and console output

Starting Periodic Advertising Demo
Failed to set periodic advertising parameters (err -5)
[00:00:00.442,626]  bt_hci_core: HW Platform: Nordic Semiconductor (0x0002)
[00:00:00.442,657]  bt_hci_core: HW Variant: nRF53x (0x0003)
[00:00:00.442,657]  bt_hci_core: Firmware: Standard Bluetooth controller (0x00) Version 2.5 Build 99
[00:00:00.444,641]  bt_hci_core: Identity: CC:E6:FA:E1:A6:E6 (random)
[00:00:00.444,641]  bt_hci_core: HCI: version 5.2 (0x0b) revision 0x0000, manufacturer 0x05f1
[00:00:00.444,641]  bt_hci_core: LMP: version 5.2 (0x0b) subver 0xffff
[00:00:00.447,082]  bt_hci_core: opcode 0x203e status 0x01

Environment (please complete the following information):

Zephyr SDK
SES

Would appreciate some help

If you are using nRF connect SDK, please raise the issue at https://devzone.nordicsemi.com/
Otherwise, if using latest Zephyr, please provide the commit hash you are using, also please provide detailed build and flashing commands etc as was requested in To Reproduce template when creating the issue.

Closing the issue as periodic advertising feature is experiment and work is still in progress.