Fork Ruby Client Library

Value Statement:
As a VANotify developer
I want to have the ability to make changes to the client library
So that VANotify can be updated for US users
Acceptance Criteria

Using the client library
WHEN vets-api is built
THEN it uses a VA-specific fork of notifications-ruby-client

Updates to the client
WHEN branches are merged into master of department-of-veterans-affairs/notifications-ruby-client
THEN a new version of the client is tested and published

No pushes to master, please
WHEN a developer tries to push to master of department-of-veterans-affairs/notifications-ruby-client
THEN the push is rejected - use PRs instead!

Checklist:

[ ] Fork the ruby client repo - see this issue for an example of how to request a fork. But specify Shane as the point of contact!
[ ] Add a Github Action workflow to run on every push, which:

[ ] Runs tests

[ ] Add Github Actions workflow to run on merges to master, which:

[ ] Runs Ruby client unit tests  (using the makefile!)
[ ] Installs dependencies (also using the makefile!)
[ ] Increments some version
[ ] Publishes the package to Github Packages (like this!)

[ ] In vets-api's Gemfile, change notifications-ruby-client to reference the new fork from Github Packages (like this!)

Assumptions:

We will use the same Ruby version

Additional Info/Resources:

Out of Scope:

Other client libraries

Open Questions:

Can the change we are making be supported by the existing library? If yes we don't have to play this story...we also aren't sure this is the best pattern to pursue.

@nathanbwright
Closing this, as it's not part of the API.


Support for | operator between strawberry primitives and None

Would be great to be able to do :
class User:
    created_at: DateTime | None

currently it returns error
TypeError: unsupported operand type(s) for |: 'ScalarWrapper' and 'NoneType'

That would be cool! Any idea how we can do it @fredgido?

I weeks ago tried writing a or for the strawberry types that makes it Union with the class next to it.
It worked but was very shortsighted as it only works with 2 types and None has to be on the write as to trigger the left class or.
I am really not sure this is a solid way. I guess None can be replaced by a custom NoneType from strawberry or something as last possibility.

An observation...
The optional type will work if define the scalar with the next construction:
@strawberry.scalar(
     serialize=lambda value: ",".join(value.items) if value else None,
     parse_value=lambda value: CustomList(value.split(",")) if value is not None else None
)
class CustomList:
     def __init__(self, items):
         self.items = items

Won't work if to use:
DateTime  = strawberry.scalar(
     datetime.date,
     serialize=lambda value: value.isoformat(),
     parse_value=datetime.parse_date
)

I'm also experiencing this issue with a custom scalar.
  File "/Users/jacob/code/saleor/api/api/payment/types.py", line 256, in PaymentInitialized
    data: None | JSONString = gql.field(
TypeError: unsupported operand type(s) for |: 'NoneType' and 'ScalarWrapper'


docs: Update section on commit bodies.

The existing documentation only referenced imperative voice in the
section on message subjects; also mention it in the section on message
bodies, as well.  At the same time, extend the section to give more
color to what else might belong in a message body.

(I have never been more conscious of how I was writing my commit message body than when writing the commit message for this commit)

Reading https://grammar.collinsdictionary.com/easy-learning/the-imperative as well, at least in my interpretation, its grammatically incorrect to use it when trying to describe something more in depth. It works as the subject because is the action we are taking in that commit.


Added Problems from Codechef Practice

Added Solutions of problems of Codechef Practice

Added Comments to all the questions.


New SGB Border.

-add my new SGB Border (not in code, i ad donly the image and meta)
-add in assets a zip with color variations (match to the camera shells) in 2 versions: "Studio" and "StudioNext"
-change the print frame to the new logo and add a "studio" version too

can you please move those into the assets/art/ rather than res/gfx/gb/backgrounds/ under the different names? i will put them into resources myself, maybe there will be more options for the SGB borders


PEP 541: mysql-connector-python  vs. mysql-connector

Hi,
I am leading the team at Oracle, which is developing our MySQL Connector for Python.
We publish our library on https://pypi.org/project/mysql-connector-python/
Due to some internal issues we didn't publish the 2.1 series on PyPi. An external person apparently tried to be helpful and registered https://pypi.org/project/mysql-connector/ serving some of the 2.1 releases, but didn't update any further for more than 1.5 years. This now seems to confuse people, as that page claims "Author: Oracle and/or its affiliates"
Is there, under PEP541, or some related policy, a way that mysql-connector is marked as abandoned or similar and forwards to mysql-connector-python?

Hi. Have you attempted to contact the author of that project,(e.g., at https://github.com/sanpingz/mysql-connector/issues) to see if they'd be willing to release the name ?

I sent out mails to the domain listed on the GH profile page without response. I now also filed https://github.com/sanpingz/mysql-connector/issues/10

I have deleted https://pypi.org/project/mysql-connector.

Referencing @johannes
A forwarding feature to a new package from an abandoned package would be quite nice with a warning  message containing a TTL before the old package is axed

@sanpingz Deleting mysql-connector broke our builds. From https://pypistats.org/packages/mysql-connector I can see that mysql-connector is still being downloads more than 10k times daily, so possibly it broke many people builds as well. Could you please restore deleted package? If possible, can you come up with smooth transition to mysql-connector-python package?

There's now a version 2.2.4 on [https://pypi.org/project/mysql-connector/#history]. There's no source code for that version at the github account. Is it a fake?

I uploaded again to smooth the transition, and put deprecation message. Let me know if anything else I can do to clear the confusion.

We had builds fail as well. Im not very familiar with the community, but is there a defined process of getting the word out before removing a package? Especially one as heavily used as mysql-connector?

My build broke as well. It seems to find the "new" mysql-connector uploaded today breaks our builds as well.
This is the error message:
  Running setup.py install for wrapt: started
    Running setup.py install for wrapt: finished with status 'done'
  Running setup.py install for pycparser: started
    Running setup.py install for pycparser: finished with status 'done'
  Running setup.py install for nltk: started
    Running setup.py install for nltk: finished with status 'done'
  Running setup.py install for mongoengine: started
    Running setup.py install for mongoengine: finished with status 'done'
  Running setup.py install for mysql-connector: started
    Running setup.py install for mysql-connector: finished with status 'error'
    Complete output from command /mnt/opt/util-software/conda/envs/utilkg/bin/python -u -c "import setuptools, tokenize;__file__='/mnt/tmp/pip-install-yo0pshal/mysql-connector/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /mnt/tmp/pip-record-u594bpde/install-record.txt --single-version-externally-managed --compile:
    usage: -c [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
       or: -c --help [cmd1 cmd2 ...]
       or: -c --help-commands
       or: -c cmd --help

    error: option --single-version-externally-managed not recognized

    ----------------------------------------

Needless to say, googling this error comes up with a lot of messages about updating setuptools and wheel, which are both up-to-date.  This error message still occurs though.
We've switched over to mysql-connector-python but I am sure glad that our production environment can tolerate this failure!  This is pretty serious.

Sorry guys, I don't even know it's being used by so many people, I'm trying to recover  2.1.x and 2.2.x to minimize the impact.

Any guidance from some pypi expert/admin how we can smoothen the transition? Could @sanpingz create a package which simply depends on the latest version of upstream or is there some redirect feature? Or a good way to point the right way when the user tries to install this package?

There is no redirect feature. The simplest thing (for users) would be to create a new, minimal distribution which depends on mysql-connector-python via install_requires, something like:
from setuptools import setup

setup(
    name='zippo',
    install_requires=['mysql-connector-python'],
    version='3.0.0',
    packages=[],
)

And possibly also adds some notice to the long_description (looks like this has already been done).
Unfortunately that won't fix the issue for users that have explicitly pinned to the now-deleted versions of mysql-connector, and it's not possible to bring back the deleted releases. Generally we don't encourage deleting releases like this, as it almost always leads to breakage.
It looks like @sanpingz is still the sole owner of mysql-connector, perhaps it makes sense to add @johannes and others as owners and then consider this issue as resolved as it's going to be?

@ewengillies curious, besides putting mysql-connector-python in requirements.txt what other changes did you need to make? I am running into a seg fault while trying to transition.

I only have mysql-connector 2.1.7 and 2.2.9 kept for the transition which roughly equal mysql-connector-python 2.1.6 and 2.2.2 with some fixes. The issue @ewengillies reported should be gone.

BTW, no need Protobuf dependency for these 2 versions.

hi @sanpingz looking at https://pypi.org/project/mysql-connector-python/#history do you mean 2.0.4 instead of 2.1.6?

Hi. Does anyone have an example of a Docker file that allows mysql-connector-python to work? I have:
In my Dockerfile I have:
RUN pip3 install -r /event_git/requirements.txt
In requirements.txt I have:
mysql-connector-python==8.0.15
Is there anything else I need to do?

hi @sanpingz looking at https://pypi.org/project/mysql-connector-python/#history do you mean 2.0.4 instead of 2.1.6?

No, here:
https://github.com/mysql/mysql-connector-python/releases

Thanks so much @sanpingz . Quick follow up. I got mysql-connector-python 8.0.5 working but I still have issues with 8.0.15 . You mentioned a protobuff dependency, do you recall how to fulfill that?

Folks, this is not the right place for this discussion. Please take it into the issue trackers for these projects.
@sanpingz, have you transferred ownership to @johannes? Can this issue be closed?

@sanpingz if you could transfer ownership to https://pypi.org/user/mysql_re_team/ as suggested by do that would be great, then we take it up to create some form of forwarding.
@EuclidHeron yes, protobuf is needed, but that should be installed as dependency, for issues please file a bug at bugs.mysql.com or reach out to me outside this specific issue here.

@johannes Don't know how to transfer but I just added mysql_re_team as owner.


HWiNFO data handling issue

I had an issue with liquidctl plugin, and had to go back to HWiNFO, and found out the real cause of this issue
The issue mentioned above was that logs.txt was constantly being written, I thought it was just HWiNFO being weird about it, but at the end, I think Fan Control has the issue receiving data from it.
As long as you first enable other sensor reporting from HWiNFO Gadget, and then enable liquid temperature sensor reporting so it's not indexed as "0" it will stop logging to logs.txt, so maybe it's the way the data is being handled? I don't know, but is a solid, I now have it working perfectly, I did change it back so index is 0 and indeed starts to write logs.txt every second, confirmed Index is the issue
With this index works perfectly
With this index it constantly writes to logs.txt
EDIT:
The issue seems to be with current implementation for NZXT Kraken AIO's of liquid temperature and pump rpm. Not related to FanControl or FanControl.HWiNFO plugin.
I found out that using liquidctl.exe from FanControl.Liquidctl with argument: initialize all (it doesn't require "-" or "--") fixes the sensors for Kraken X73, it doesn't gray out inside sensor status view in HWiNFO anymore and it doesn't delete the entry from regedit either.
I currently have it (liquidctl.exe) launching at system startup with task scheduler with the argument I mentioned, then HWiNFO starts at logon of any user, and FanControl also at logon of any user but with a delay of 10 seconds to give HWiNFO time to load and avoid : "System.Exception: HWInfo is not running or reporting to gadget is not enabled." to log.txt.

The interesting part is why it logs in the first place:
https://github.com/Rem0o/FanControl.HWInfo/blob/6ec922093d6e307d70dd929cfa1294944a814a4b/HWInfoPlugin.cs#L88
The app updates every second.
As you can see, it means that said sensor simply went missing in the registry during that update cycle, the value of that registry key could not be parsed to a float.  My guess is the parsing. The value must be something I'm not expecting/never seen before.  Would be practical to know what that is.

Just did a commit
https://ci.appveyor.com/project/Rem0o/fancontrol-hwinfo/builds/45979464
It will only log the failed sensors once you close the app or refresh the app.
I added more log information so we can take a look at the value it couldn't parse or if the sensor simply disappeared.

Just tried it and no luck, you will receive an email with a screen recording video in a few minutes so you can see how I have it setup and maybe there's something I didn't notice, also started logging sensors in HWiNFO and there wasn't any weird value in the csv

Did you use the binary from appveyor?

Did you use the binary from appveyor?

of course, I sent you a link to youtube with the recording, I created a copy of my Fan Control and unzipped the new contents there, 2 already existed and clicked replace so I had all the new 4 files from appveyor

From the video, you didn't put the DLL in the plugin folder, you put it in the FanControls root folder. Also, from the text log we see later, the message being shown is not possible if you used the latest binary: the message was deleted from the code.

oh, also, if you noticed from the recording, it grays out every 2 seconds in hwinfo and when it does that the registry key also gets deleted, and is the only sensor (that I use) that has that issue

@Rem0o caught this, in case it's of any help.
1/20/2023 01:20:28: HWInfo sensor failed momentarily during operation: HWInfo/NXZT Kraken X53/X63/X73/Liquid Temperature/°C - Missing
1/20/2023 01:20:28: Unhandled exception in FanControl v<IP_ADDRESS>
1/20/2023 01:20:28: System.Exception: HWInfo sensors failed: HWInfo/NXZT Kraken X53/X63/X73/Liquid Temperature/°C
   at FanControl.HWInfo.HWInfoPlugin.Update()
   at FanControl.Domain.BackendProviders.Plugin.PluginBackendProvider.Update()
   at System.Collections.Generic.List`1.ForEach(Action`1 action)
   at FanControl.Domain.ComputerAccessLayer.Update()
   at FanControl.Domain.ApplicationClock.DoActions()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

^ that means the sensor was undetected for more than 10 cycles straight, which at that point I would say it is probably not reliable enough to be used as a source.

at least now it isn't writing every second to log.txt with your fix :) thanks!

found out it is HWiNFO current implementation of NZXT Kraken devices they added in last version, edited the issue with a workaround until they fix it (if at all).


enhancement enhance visibility of flush thread pool

Proposed changes
added metrics for flush thread pool and flush tasks num

run buildall

run buildall

run buildall

run buildall


bug: palomad has peer issue after upgrading to v1.1.0

After upgrading palomad v1.1.0 i had peer issue. Downgraded to v1.1.0 and looks ok. Here you can see v1.1.0 logs(palomad)
https://gist.github.com/DasRasyo/82ca2a435823caf3e2f44f35ead09b5b

Discord Commentary: https://discord.com/channels/987730379182051408/1072137820405235762/1112073207420289064.

Closing. This is a semver rules issue. We need to upgrade via proposal for major number upgrades.


STORM-1573 (1.x): Add batch support for MongoInsertBolt

master: https://github.com/apache/storm/pull/1230

+1
Could you rebase these three commits? After rebasing I'll merge.

@HeartSaVioR Done. rebased both master and 1.x-branch.

+1 Thanks for the patch.


fix: Support bold ANSI escape sequence

The code to print bold text is currently commented out, even though it works fine. This PR uncomments it to add support for rendering bold text.
I also noticed that bold text wasn't very pronounced, so I changed the font-weight to bolder. If this is undesired, just let me know and I can remove that change.
Before:

After (with font-weight="bold"):

After (with font-weight="bolder"):

I also noticed the bold
Fixes #75

@meowgorithm Maybe my machine is rendering fonts strangely, but bold doesn't look very different than regular text, so I was trying to make the effect more obvious. Would you prefer bold? That might make more sense

Thanks for the insight, I hadn't thought about rendering handled when a 500 weight font wasn't available. For some reason PNG rendering doesn't work on my machine, so let me try it in a container. I'm curious to see the difference

Ok I went ahead and changed the weight back to bold since it sounds like that's the best solution. I tried generating PNGs, but the font weight looks exactly the same. I'm sure it isn't a testing issue because it still works when rendering to an SVG, so maybe the renderer isn't going to support bold text for some reason?
I don't think it's super helpful, but here's what I get:

If I render to an SVG, the font-weight: bold is definitely there:
<tspan xml:space="preserve" font-weight="bold">Bold text</tspan>


Add Function to allow users to addLike

Hey @OscarFMdev in this PR i finished Task #9
this task consist of allowing the users to add a new like, the likes are hidden until a user decided to like a movie
this is to prevent biased judgments when user's like a movie

since we're falling behind i'll go ahead and merge this branch


Adjust CI YAML headers

This PR adjusts the headers in a lot of files, we are getting rid of the misleading comment and instead adding a link to a (yet to be created aka.ms link).

Can one of the admins verify this patch?


Refactor Access Control

Few enhancements are required for the implemented access control functionality:

Access control configuration should include a parameter for the path at which the admin/permission endpoints will be hosted, e.g. the default could be /admin/access-control as it is now.
Further we need to move access control configuration as a part of the security configuration and the complete module must move to the security module.

Thinking about the first point some more, I don't think it is really required. It is very hard to provide a client for an arbitrary user-provided admin/permission API (after all, it's not just the base path, but the endpoint or endpoints, including semantics, that we would need to know), and I don't think it's worth going there. Instead we should probably focus on adding a client for the default API, and for that we know the routes. Users might still be able to reuse that client if they host a /permissions API at the expected base path (if all they are doing is modifying the behavior of the default endpoints). And if they wanna add a completely different API, then they would also need to implement the client along with it.
So rather than updating the model, I would rather create an issue to implement a client for the default permission endpoints, which then can be used in the controller logic of a FOCA app. A second issue would then be to use that client in the petstore example to automatically set the permissions of a newly created resource such that the creater becomes the owner of the resource, i.e., they should be allowed to read (GET /petsandGET /pets/{id}) and delete (DELETE /pets/{id}`) the resource. A third issue should focus on implementing what it takes to set owner permissions also for the permissions for that resource themselves in order to allow a user to, e.g., share the resource with another user.

Closing this issue and bifurcating it into separate small tasks will be easier to monitor.


Create Crossplane XRD for easily deploying the ServiceProxy operator across our clusters

[ ] Explore XRD concept
[x] Create DFDS crossplane provider
[ ] Write playbook to explain crossplane XRD concept
[ ] Implement XRD schema for inventa
[ ] Write unit test for provider

I spoke with Dan, he suggested that we base our provider of this repo @ https://github.com/crossplane/provider-template .

I've started looking at the DFDS Provider from the template:
Current WIP Source is here: https://github.com/dfds/crossplane-provider-dfds/tree/provider-from-template
Container is here: https://hub.docker.com/repository/docker/dfdsdk/provider-dfds
Can be installed in sandbox by adding it to the list of providers passed to the module: crossplane_providers = ["crossplane/provider-aws:v0.18.1", "crossplane/provider-azure:v0.16.1", "dfdsdk/provider-dfds:latest"]
It is currently in a non-working state and does nothing that the template doesn't do as standard
NAME                        INSTALLED   HEALTHY   PACKAGE                             AGE
crossplane-provider-aws     True        True      crossplane/provider-aws:v0.18.1     2d23h
crossplane-provider-azure   True        True      crossplane/provider-azure:v0.16.1   21h
dfdsdk-provider-dfds        True        False     dfdsdk/provider-dfds:latest         21h

I suspect because the template is out of date and not v2 as per this issue but not confirmed:
https://github.com/crossplane/provider-template/issues/5

I have managed to get the provider in a healthy state:
NAME                        INSTALLED   HEALTHY   PACKAGE                             AGE
crossplane-provider-aws     True        True      crossplane/provider-aws:v0.18.1     6d1h
crossplane-provider-azure   True        True      crossplane/provider-azure:v0.16.1   4d
dfdsdk-provider-dfds        True        True      dfdsdk/provider-dfds:latest         102m

We now have the provider controller in this docker repo: https://hub.docker.com/repository/docker/dfdsdk/provider-dfds-controller
The package is still in: https://hub.docker.com/repository/docker/dfdsdk/provider-dfds
Some instructions to follow are:
To build and publish:
For dfdsdk\provider-dfds-controller:
make build
make image
make image-push

For dfdsdk\provider-dfds:
cd package
kubectl crossplane build provider
kubectl crossplane push provider dfdsd/provider-dfds:latest

To install in cluster:
kubectl crossplane install provider dfdsdk/provider-dfds:latest

This will need putting into an azure-pipelines.yaml and an automated build setting up on dockerhub but I am not worrying about that for now.
Next step is to get some XRD CRDs' made after some investigation into how to do those.

Next step is to get some XRD CRDs' made after some investigation into how to do those and include them in the provider. We can either create some around some simple resources like RDS (including the SecurityGroup) or use Inventa to start

We have created and deployed an XRD today which involves a CompositeResourceDefinition(XRD), a Composition using that XRD, and a CompositeInstance which deploys that resource (aws rdsinstance). It deployed successfully, modified successfully with updated parameters and destroyed successfully so we have a rudimentary understanding we can build on. It was done using an AWS RDSInstance (postgres).
We have moved onto the task of putting this into our DFDS Provider but it seems more complicated to do this than it first appeared so we have a lot of reading and experimenting to do to make progress, given how little/no documentation there is

We now have a DFDS Provider package, a DFDS Configuration package, and a Composite resource working. We need to do some extra tasks to make this properly usable/presentable.

[ ] Use semantic versioning for the provider so we can have a dependsOn from the configuration
[ ] Add extra resources into our composite database resource (i.e we need to put a securitygroup in, vpc etc)
[ ] Rename our composite database resource to something better than compositepostrgesinstance
[ ] ProviderConfig - we need to decide on DFDS/AWS and how we do that. We are currently relying on 'default'. Is it possible to chain through the AWS provider?
[ ] We need an example deployment that shows the minimal requirement to deploy a resource
[ ] We need to look into RBAC around XRD so that we can do multi-tenancy single cluster appropriately (This perhaps should be another issue but it would be nice to get an understanding/example made)

Need to upgrade to 1.13 to enable CombineFromComposite property in composition files

To enable advanced policies like preventing switching engineVersion or prevent downgrading and displaying error messages directly to claim users, OPA Gatekeeper https://www.openpolicyagent.org/docs/latest/kubernetes-introduction/ watch https://www.youtube.com/watch?v=TaF0_syejXc for inspiration


Added deno_string_validator to third party list

Added a string validation library.

Thanks for the patch, but it must be green to be considered. Due to the high volume, I'm not reviewing red PRs, just closing to clear out the queue. I apologize if I haven't addressed something. Please open a new PR.


MainSideBar

Hi, Ivan:
Which one is the MainSideBar?

Do we expect to finish all our tasks on Asana by Thursday? Or only these tasks we are assigned to?

The side bar is the entire left side of the screen. Finish what you can by Thursday.

so this is the MainSideBar, combine each small components. Right?

Thanks!

Yup, that's correct.

Thanks!


Click login, but go no where

Describe the bug
When user try to make an appointment without login first. Then at appointment page click login, but no where to go
To Reproduce
Steps to reproduce the behavior:

Add service
Add staff
Click on proper time slot
Click login

Expected behavior
Website redirect to login page or register page for user.
Screenshots

Desktop (please complete the following information):

OS: FROM docker.io/python:3.12.5-slim-bookworm
Browser Chrome

Additional context
Add any other context about the problem here.
django==5.0.8
django-appointment==3.6.3
Possible Solution
No idea yet
Can you fix the bug?

[ ] Yes, I can fix the bug and will open a pull request.
[ ] No, I can't fix the bug, but I can assist in testing and providing feedback.
[ ] No, I can't fix the bug, but I can provide more information to help you fix it.
[x] No, I can't fix the bug, but I can open an issue to report it.

Hi @elcolie,
This isn't working at the moment, so it's normal that nothing happens when you click on it. The implementation depends on how the project handles the login process with Django, so it might take me a bit of time to sort it out. However, if you're up for it, I'd be happy to review a pull request if you'd like to take a crack at it.
Adams


Error during dialogue

bot_1            | 2022-07-10 17:53:19,497 - main - INFO - Saving
bot_1            | 2022-07-10 17:53:19,501 - main - INFO - Saved
app_1            | Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
app_1            | Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
app_1            | [2022-07-10 17:53:40 +0000] [9] [CRITICAL] WORKER TIMEOUT (pid:17)
app_1            | [2022-07-10 17:53:40 +0000] [17] [INFO] Worker exiting (pid: 17)
app_1            | [2022-07-10 17:53:41 +0000] [9] [WARNING] Worker with pid 17 was terminated due to signal 9
app_1            | [2022-07-10 17:53:41 +0000] [33] [INFO] Booting worker with pid: 33
app_1            | Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
app_1            | Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
bot_1            | /usr/local/lib/python3.7/site-packages/telegram/ext/utils/promise.py:96: TelegramDeprecationWarning: The @run_async decorator is deprecated. Use the run_async parameter of your Handler or Dispatcher.run_async instead.
bot_1            |   self._result = self.pooled_function(*self.args, **self.kwargs)
bot_1            | 2022-07-10 17:53:53,797 - main - INFO - Entering private toxification
app_1            | [2022-07-10 17:54:00,113] ERROR in app: Exception on /predict [POST]
app_1            | Traceback (most recent call last):
app_1            |   File "/usr/local/lib/python3.7/site-packages/flask/app.py", line 2073, in wsgi_app
app_1            |     response = self.full_dispatch_request()
app_1            |   File "/usr/local/lib/python3.7/site-packages/flask/app.py", line 1518, in full_dispatch_request
app_1            |     rv = self.handle_user_exception(e)
app_1            |   File "/usr/local/lib/python3.7/site-packages/flask/app.py", line 1516, in full_dispatch_request
app_1            |     rv = self.dispatch_request()
app_1            |   File "/usr/local/lib/python3.7/site-packages/flask/app.py", line 1502, in dispatch_request
app_1            |     return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)
app_1            |   File "/usr/local/lib/python3.7/site-packages/prometheus_flask_exporter/init.py", line 710, in func
app_1            |     return current_app.handle_user_exception(ex)
app_1            |   File "/usr/local/lib/python3.7/site-packages/prometheus_flask_exporter/init.py", line 708, in func
app_1            |     raise exception
app_1            |   File "/usr/local/lib/python3.7/site-packages/prometheus_flask_exporter/init.py", line 666, in func
app_1            |     response = current_app.handle_user_exception(ex)
app_1            |   File "/usr/local/lib/python3.7/site-packages/prometheus_flask_exporter/init.py", line 663, in func
app_1            |     response = f(*args, **kwargs)
app_1            |   File "/usr/local/lib/python3.7/site-packages/prometheus_flask_exporter/init.py", line 710, in func
app_1            |     return current_app.handle_user_exception(ex)
app_1            |   File "/usr/local/lib/python3.7/site-packages/prometheus_flask_exporter/init.py", line 708, in func
app_1            |     raise exception
app_1            |   File "/usr/local/lib/python3.7/site-packages/prometheus_flask_exporter/init.py", line 666, in func
app_1            |     response = current_app.handle_user_exception(ex)
app_1            |   File "/usr/local/lib/python3.7/site-packages/prometheus_flask_exporter/init.py", line 663, in func
app_1            |     response = f(*args, **kwargs)
app_1            |   File "/app/server.py", line 23, in predict
app_1            |     result = model.toxify(text, temp)
app_1            |   File "/app/Ebanko.py", line 21, in toxify
app_1            |     bad_words_ids=[[tokenizer.pad_token_id]],
app_1            | NameError: name 'tokenizer' is not defined
bot_1            | 2022-07-10 17:54:00,127 - telegram.ext.dispatcher - ERROR - A promise with deactivated error handling raised an error.

благодарю!


Use spaCy matchers to detect interaction

One possible way that we could detect interaction between characters is if there is the pattern 'CHARACTER VERB CHARACTER'. It is possible to detect this pattern using spaCy matchers.
Part of: #49.

This is outdated.


z3 instead of libz, added macos-14

I think I just need to install the latest version on the CI machines.


create single executable file

[ ] Use axios+cheerio instead of puppeteer
[ ] Replace node-canvas with pure-image.
[ ] Make a single binary with pkg.

If there is a demand, i will make it.

If there is a demand, i will make it
yes please.


cannot unmarshal !!str error when port-forwarding ports provided by environment variables

What happened?
I am using a configuration that contains the following lines:
version: v1beta7
vars:
- name: MY_SERVICE_PORT
  source: env
  default: 4000
[...]
dev:
   ports:
   - imageName: my-service
     forward:
     - port: ${MY_SERVICE_PORT}
       remotePort: 80

When I try to deploy any of my profiles, I get the error
cannot unmarshal !!str `4000` into int

If I comment out the dev section of my devspace.yaml, then the deployment is successful. Under Windows 10, on git bash, it works perfectly. Under Linux (especially alpine), I get the above error.
What did you expect to happen instead?
I would expect to have no error. It used to work properly on both Linux and Windows.
How can we reproduce the bug? (as minimally and precisely as possible)
Take the above simple snippet, just add an image to build and a deployment.
Local Environment:

DevSpace Version: 5.11.0
Operating System: linux
Deployment method: helm

Kubernetes Cluster:

Cloud Provider:  other
Kubernetes Version:

Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.8", GitCommit:"fd5d41537aee486160ad9b5356a9d82363273721", GitTreeState:"archive", BuildDate:"2021-04-01T13:29:43Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.8", GitCommit:"fd5d41537aee486160ad9b5356a9d82363273721", GitTreeState:"archive", BuildDate:"2021-04-01T13:25:36Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"linux/amd64"}

Anything else we need to know?

/kind bug

@zadigus thanks for reporting this issue! Could you try to update the DevSpace version to v5.12.1 and update the config version to v1beta10 and retry?

Looks good, thanks.


Change all static properties to class properties

Change all static properties to class properties
Allow PHP 7.2 +
Readme update with minimum requirement

Closing so as to target 1.x branch


Needs update for activity result for RN 0.37

Image picker is unstable in version 0.37, and crashes. This is due to a new interface change.
public void onActivityResult(final int requestCode, final int resultCode, final Intent data) {
}
Activity Result has a new interface:
public void onActivityResult(Activity activity, int requestCode, int resultCode, Intent data) {
}

@skyride99 did you find any quick solution or temporary hack for this?
I'm having the same issue.

I had several issues with upgrades with other components. Had to go back to RN 32 for the app to work correctly. Looking forward to the update on this.

I just updated the Example project to RN 0.38 and I'm not having any issues. We are handling the new Activity Result interface, so I'm not sure how you're running into this. I will push the changes to the Example project - can you run that?

It seems to be running now. Not crashing.  But now have an issue "Network Error". Lets close this issue.


Add cache for scores

Hi, This change adds a cache for scores. I noticed when optimizing 10 parameters for an xbgoost model that most scoring evaluations were duplicates. Below is output from the notebook example.
(There is another small change that is unrelated, sorry I could not find how to get github UI to create a separate pull request.)
cv = EvolutionaryAlgorithmSearchCV(estimator=SVC(),
                                   params=paramgrid,
                                   scoring="accuracy",
                                   cv=StratifiedKFold(y, n_folds=10),
                                   verbose=True,
                                   population_size=50,
                                   generations_number=10)
%time cv.fit(X, y)
Types [1, 2, 2] and maxint [0, 99, 99] detected
--- Evolve in 10000 possible combinations ---
Scoring evaluations: 5, Cache hits: 45, Total: 50
...
gen     nevals  avg             min             max
0       50      0.911823        0.895028        0.983425
...
Scoring evaluations: 146, Cache hits: 3304, Total: 3450
10      24      0.981547        0.895028        0.983425
Best individual is: {'kernel': 'rbf', 'C': 53366992.312063016, 'gamma': 0.0001232846739442066}
with fitness: 0.983425414365
Scoring evaluations: 147, Cache hits: 3313, Total: 3460
CPU times: user 11.1 s, sys: 69.4 ms, total: 11.2 s
Wall time: 11.3 s

Doesn't deap already provide a cache for these cases? is it necessary to have an external one?

I don't know details about deap. If it provides caching it would seem to either be disabled or busted. From output above from running the example, out of 3450 function evaluations only 146 were unique.
Scoring evaluations: 146, Cache hits: 3304, Total: 3450


fixed regex

FYI, this is because there is an ":" in the filepath under Windows.
https://github.com/jfcherng/SublimeLinter-contrib-iverilog/blob/4416b7ce97ccf6d9b1abab59cd5a404cf5bfe3e9/linter.py


Add a default content key for the service_kafka plugin

#1748

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

搞的有点乱了, 关了重新整理下.. 抱歉


Updated the installer file to use relative paths within the parent folder.

This is nice to have to be able to build the installer easily as everything is contained within the parent folder and not depending on some specific folder paths to exist.
The only change is that you only need to do on your side (i.e. on your local machine) is to create a FFmpeg folder and to put ffmpeg.exe into it for the installer to pick it up. I think that this folder can also be used as a source to copy the ffmpeg.exe into the build folder when building the solution, but I will look into this later.

much nicer

@shaked6540, just a kind reminder to merge this when you have the time and if there are not any issues. Thank you!


Handler API

Description
In this pull request the possibility of adding handlers to the pipeline without modifying the mediaStream file is introduced. The objective is to facilitate the processing of the audio and video streams inside the pipeline. To desing these new handlers they have to extend the CustomHandler interface created inside the Handler.h file, that has the same functions that the Handler class has plus a new function getPostion(), that returns the position where the handler has to be inserted inside the pipeline. This position is seleted by the handler designer and can not be changed. Handlers can be inserted into the pipeline into three different positions:

Beginning: At the start of the pipeline, before the rest of handlers.
Middle: In the middle of the pipeline.
End: After all the handlers.

These handlers are grouped into profiles that can be imported and selected through the API as documented in /doc.
To add handlers to the API they have to be copied to the erizo/src/erizo/handlers folder and add the name of the class and the name you want for them inside the API into the erizo/src/erizo/handlers/handler.py file. Then erizo has to be recompiled. Handlers are imported by the erizo/src/erizo/handler/HandlerImporter class which creates and stores pointer to the handlers. When running the ./installErizo script a python script modifies the HandlerImporter class to add the handlers selected in the handler.py file to the API. Once this is done the new handlers are available and can be selected by creating a series of profiles that can be later selected throught the API. This proccess of adding handlers and creating profiles is detailed in the doc file.
A test handler has been added which logs data from the packets received, as an example for handler development.
[] It needs and includes Unit Tests
Changes in Client or Server public APIs
Added the possibility of adding handler to the pipeline by creating profiles in the licode.config file and the selecting them through the API.
[Yes] It includes documentation for these changes in /doc.
How does it affect Erizo Client?
Erizo Client reads the new handlerProfile parameter, which contains the selected handler profile for the mediaStream, and sends it to the erizoJS
How does it affect Nuve?
It does not affect Nuve
How does it affect Erizo Controller / Erizo Agent / Erizo JS?
When creating a new webRtcMediaStream ErizoJS controller reads the config file to get the name of the handlers associated to the selected profile.
How does it affect ErizoAPI / Erizo?
The mediaStream recieves a new parameter, maps where each pair represents a filter and contains its name and parameters sended by the user. It has a default empty value for retrocompatibility.  Also when adding the handlers to the pipeline, in the three points specified before, a function calls pipeline->add_front function for all the handlers that have to be inserted in that position.
Erizo API has a new node addon that receives a vector of maps, containing the filter names and parameters and creates the handlers and inserts then in a  map that we then pass to the mediaStream.

I jsut changed the names that you suggested and fixed the indentation aswell as the documentation


DWU vs cDWU

Shouldn't this be cDWU instead of
"The service levels for Gen2 dedicated SQL pool (formerly SQL DW) are measured in data warehouse units (DWU), for example DW2000c." ?

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

ID: 73d18754-6ffe-e890-3ccb-3edfcb3f4b58
Version Independent ID: 35516444-7bb9-8420-dd7b-4b5b6a22194e
Content: Data Warehouse Units (DWUs) for dedicated SQL pool (formerly SQL DW) - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql-data-warehouse/what-is-a-data-warehouse-unit-dwu-cdwu.md
Service: synapse-analytics
Sub-service: sql-dw
GitHub Login: @mlee3gsd
Microsoft Alias: martinle

Thank you for your feedback! We will review and update as appropriate.

HI @D-Moonesinghe , These terms are defined by the product team and as this seems to be a product feedback, we would like to request you to please share your feedback in Azure Synapse Analytics user voice forum : https://feedback.azure.com/forums/307516-azure-synapse-analytics which is actively monitored and reviewed by engineering teams.
If you have any other feedback for this doc improvement please do let us know.
Thank you


[quick fix] is CTD api really used by BTE

check if "CTD api" is in the metaKG, and if not, remove it from the file.
This list of APIs is what BTE uses. It includes an api named "CTD API". However, the smartapi registry doesn't seem to have an api with this name (the automat ctd is something different).

We are removing this API from the list of APIs BTE ingests. This is being tracked in the other issue mentioned above...


[Enhancement] EnumToBoolConverter Support Enum ConverterParameter Type

Summary
Please provide a brief summary of your proposal. Two to three sentences is best here.
Support that ConverterParameter is Enum and not just String.
It can improve the flexibility for EnumToBoolConverter.
API Changes
Include a list of all API changes, additions, subtractions as would be required by your proposal.
EnumToBoolConverter
Intended Use Case
Provide a detailed example of where your proposal would be used and for what purpose.
<RadioButton Content="ON"
             IsChecked="{Binding EditType, Converter={x:Static conv:EnumToBoolConverter.Instance}, ConverterParameter={x:Static local:EditPanelValueType.Switch}, Mode=TwoWay, FallbackValue={markups:False}}" />

cant agree more with your point

Regarding MAUI? Its a shame…


[ERROR] server.cpp(281): accept failed! Software caused connection abort

Hello, I start load testing of ssdb for night with 10k rps. And I have two errors in logs - about 2 gb repetitive errors:
2015-01-10 02:37:43.977 [ERROR] server.cpp(281): accept failed! Too many open files
But I fixed it - set ulimit -s unlimited. Is it normal decision?
But what means a second?
2015-01-10 09:11:49.411 [ERROR] server.cpp(281): accept failed! Software caused connection abort
How I can find a reason? Logs are so curt.
I use freebsd 9.3.

The Too many open files error is described in this doc: http://ssdb.io/docs/config.html

@ideawu , thank you, I was careless, when reading docs.
And do you have any ideas about second error?

@zloy531 It will be helpful if you can provide a way to reproduce the second error.


fix polygon/polyline load fail points within the template

What does the pull request do?
use polygon in button's template,polygon invisible
i want fix polygon/polyline Unable to load points within the template.
What is the current behavior?

What is the updated/expected behavior with this PR?

polygon/polyline can load points within the template.
How was the solution implemented (if it's not obvious)?
Within the polygon constructor, “Points=new Points()”，BindingPriority is LocalValue；
When in the Template，loading points from xaml，BindingPriority is Template，so loading faild，the property points always the "new Points()";
Breaking changes
Obsoletions / Deprecations
Fixed issues

@dotnet-policy-service agree

It's not a correct fix.
With this PR PointsProperty will have single instance of the Points array as a default value. Meaning, the same array is going to be shared.
Probably, the better fix is to call SetValue(ProintsProperty, new Points(), BindingPriority.Template); from the ctor instead of Points = new Points.

I agree, your idea is better.

You can test this PR using the following package version. 11.0.999-cibuild0038746-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]


Feature suggestion: quickly jump to files / directories using the keyboard in lumina-fm

It would be very helpful if lumina-fm selected the first hit when a key is pressed.
For example in / pressing "u" selects "usr", "v" selects "var" etc.

Hmm, playing around some more it is already implemented. But it didn't work when I wrote this PR. ;)

The type-to-find functionality requires that the main tree/list widget have keyboard focus first (restriction caused by the multiple tabs/columns functionality). That is probably why you did not notice it originally.
:-)


AHI Audio via Raspi?

First of all, the a314 is a realy nice idea!
To use the trapdoor expansion slot in that way could bring complete new possiblitys to the Amiga500 :-)
As I read about piaudio the first idea was, why it is not the other way? For Amiga500 there is no modern soundcard available so I was thinking it would be very nice to have a ahi.device or something to play amiga audio through the soundcard of the raspi :-)

I am also in to trying to create an AHI-device with 14bit paula playback... Stereo mixer in the pi in other words.
Also a MHI-device for decoding mp3:s and such would be wonderful.
I will try to get my card working tonight, if I am lucky

Sounds like a great idea! 👍

@hgw77: Thank you for the kind words.
Before you do a driver to use the Raspberry Pi's audio output for the Amiga, you should consider the wide-spread criticism against the audio quality of Pi (https://hackaday.com/2018/07/13/behind-the-pin-how-the-raspberry-pi-gets-its-audio/ , the comments are especially interesting).
There is however a good reason for letting the Pi do the audio decoding, and that is if it was part of a RTG solution, where the HDMI would not only carry the video output from the Amiga, but the audio as well.
Even if the Amiga's "sound card" isn't modern, I would argue that it still produces audio with better quality compared to the output from the Raspberry's 3.5mm jack. But maybe it's just me being forgiving and nostalgic :) Perceived audio quality is such a personal experience.

@Eriond: thank you for your repsonse, and you are right the audio quality of the internal raspi is not that good but there are a wide range of USB-Soundcards with much better sound quality that you can use instead? :-)
By the way, I love the good old Paula and I am always impressed what this sound chip can do ;-) but on the other side I was always dreaming to have a better soundcard on my Amiga500 to play audio in 16bit quality.

@hgw77 you can get as much as 14bit stereo at 56kHz (with more exotic screen modes) thats where AHI mixing in the pi comes in, but then shuffle the mixed left/right as 14bit-hack sample data to Paula... it's an AWESOME upgrade without having to buy expensive audio hardware. :)

@aiobofh: yes thats what I mean with impressed. The Paula is more than 30 years old and can play audio in 14bit and 56kHz? Thats awesome and C= didn't know about that killer feature ;-)
But that feature comes at a cost, it needs a very powerfull CPU. With the installed Pi in the trapdoor maybe a standard Amiga500 (or with 68020/30) could play (with the help of the pi) an mp3 in 16bit quality ;-)

@hgw77 yeah. But with a314 allowed to write chipmem the CPU usage (for playback) is close to zero om the Amiga side.

A now I understand! the PI is doing all the prepartions so the paula only needs to play :-)
On the other side if you have an PI-AHI Device you can use any player on amiga side (like Eagleplayer or AmigaAMP) to play audio on the PI, but than the amiga needs to handle the encoding of an mp3 or playing a mod file by itself and send the audio data to the PI.

@hgw77 correct. And we could also have a MHI-device (the API used for stand-alone hardware mp3-decoders) too. just shuffle the mp3-data to the pi, let it decode it, and shuffle the 16bit wav back to the Amiga and then AmigaAmp or whatever shuffels it back to the PI using this AHI-device for 14bit stereo encoding. :) There could also be a short-cut raw-allocating paula directly with the MHI device, but it's not as system friendly, but would reduce the traffic between Amiga and Pi with that bidirectional raw 16bit stereo sample.

... Hm. On second thought. I think MHI is supposed to hijack Paula directly, since it's constructed for stand-alon output devices. But I have not studied the interace enough to be 100% sure.


synth_xilinx is discarding tristate logic

Steps to reproduce the issue
module top(inout pin, input dout, sel, output din);
assign pin = sel ? dout : 1'bz;
assign din = pin;
endmodule

yosys -p "synth_xilinx; dump" tristate.v
Expected behavior
Yosys either instantiates a tristate buffer that downstream tools can understand; or at least prints a clearer warning/error that the tristate is being totally discarded for synth_xilinx, rather than the ambiguous Yosys has only limited support for tri-state logic at the moment. warning.
Actual behavior
attribute \top 1
attribute \src "tristate.v:1"
module \top

  attribute \src "tristate.v:1"
  wire output 4 \din

  attribute \src "tristate.v:1"
  wire input 2 \dout

  attribute \src "tristate.v:1"
  wire inout 1 \pin

  attribute \src "tristate.v:1"
  wire input 3 \sel

  connect \din \dout
  connect \pin \dout
end

The bug happens in opt_expr -full.


BRM Card "Gang up" messes up Card Counter

If i "Gang up" a card from the opponent the 3 Copies are shown in the Tracker of the opponent (with the green Box)

I've had this happen to me, along with Gang Up showing the cards in my opponent's deck even though I cast it on my own minion, and in another case the Gang Up target not being added to any deck at all.

I decide to rewrite the entire log parser using the exact same code base as Hearthstone Deck Tracker because its app has few months and lot of users so he have a very solid code.
It will be a lot easier for me to get a solid base.... but it take some time...
I hope to finish this work very soon

The rewrite is mostly done, I have to test this on real games as well as #36

@JerkyGunner, @djlunte do you play in EU or NA ?
I still have a few test to make, but mainly for the "opponent" side and it's quite difficult to find an opponent which will play what I'm expect, so I would love some help to test some case scenario

EU.

EU - Free for testing!

Ok guys, could you add ben#2403.
I will have some scenario to test (will not be available tonight but will be tomorrow and sunday)

Done. If you see me online just ask if I'm available as I might not be at the PC.

Done. If you see me online just ask if I'm available as I might not be at the PC

Am 10.04.2015 um 20:28 schrieb Benjamin Michotte<EMAIL_ADDRESS>Ok guys, could you add ben#2403.
I will have some scenario to test (will not be available tonight but will be tomorrow and sunday)
—
Reply to this email directly or view it on GitHub https://github.com/bmichotte/HSTracker/issues/35#issuecomment-91645654.

Mit freundlichen Grüßen,
Marcus Sagafe
In de Böög 11
24997 Wanderup
Deutschland
Tel. +49 (0) 4606 9659160
Mobil: +49 (0)<PHONE_NUMBER> tel:+49 (0) 172 40 43 21 9
E-Mail<EMAIL_ADDRESS><EMAIL_ADDRESS>


R3.5, update to version 0.36.1, update recipe

Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy
[x] Ensured the license file is being packaged.

@bgruening ready to merge.


Refactor edge boundary, implement core vertex mode logic

Changes

Adds a new module, vertex.c, which will eventually hold vertex mode functions.
Implements vertexNumForDirection, which allows us to get the number (index, in the traditional sense) of the topological vertex in a given direction. The logic here is based on determining the number of rotations of the vertexes compared to a given "base" arrangement of direction-to-vertex. The base arrangement occurs when a cell is on the "home" face of its base cell. Additional rotations are required when the cell appears on a different face from its base cell's home face.
Most of the rotation information is included in faceIjkBaseCells, but that table is not optimized for this lookup, and there are additional rules for pentagons and polar pentagons. I've created a new lookup table to facilitate this operation, baseCellVertexRotations, along with a script to generate it.
Using vertexNumForDirection and a small refactor of faceIjkToGeoBoundary, allowing it to take a start vertex and sequence length, I've refactored getH3UnidirectionalEdgeBoundary to stop comparing geo coordinates and instead look up the vertex directly based on the direction.
Adds additional exhaustive tests down to res 6 to catch more edge cases.

Benefits

Fixes getH3UnidirectionalEdgeBoundary at resolutions > 12 (see changes in testH3UniEdge.c), which were broken due to inaccuracies in comparing geo coordinates.
Speeds up getH3UnidirectionalEdgeBoundary by about 3x:

Before
    -- getH3UnidirectionalEdgeBoundary: 58.733300 microseconds per iteration (10000 iterations)

After
    -- getH3UnidirectionalEdgeBoundary: 18.856600 microseconds per iteration (10000 iterations)

Future Work
Implementing vertexNumForDirection and a more efficient version of getH3UnidirectionalEdgeBoundary set us up nicely for Vertex Mode functions and a significantly more efficient version of h3SetToLinkedGeo.

Coverage decreased (-0.09%) to 99.042% when pulling bcc02f4ba01c57b0f78a56c1bbfff473ebe12705 on nrabinowitz:edge-boundary-refactor into ab05e375c756cc62637485e5e8f5ce2f814ab828 on uber:master.


Replace Meetup provider

Hi i want to propose to replace a outdated Meetup provider with my new Meetup provider.
Motivation: https://github.com/howlowck/meetup-oauth2-provider/issues/1

Hey I'm the original meetup-provider author (the one @wittestier is replacing).  Since I can no longer maintain the original meetup provider, the meetup provider could definitely be replaced.

Thank you for contributing! 🎉


das and bam

1.das provide same function as BAM?and it replaces BAM?
2.I want to archive cassandra data,but can not find the parameter of "external cassandra cluster" in BAM2.5,seems that it remove this param.
so I do confuse about the meaning of archiving cassandra data.we still store the data in this cassandra cluster.
by the way,how to deal with when you need to visit the archived data.

Hi,
Yes, DAS replaces the functionality given by BAM. For more info, check docs at https://docs.wso2.com/display/DAS301/WSO2+Data+Analytics+Server+Documentation.
And now, Cassandra is just one of the several data connectors we support in DAS, for Cassandra configuration, check https://docs.wso2.com/display/DAS301/Configuring+Data+Persistence+With+Cassandra.
Cheers,
Anjana.


Do ASM, as well as CLI and shadow

What it says in the title. Added a rudimentary CLI, an assembler for testing purposes, as well as shadowing for building jars.

It literally was just compiling on my machine...


Custom translation directory

I would like to keep translations in different directory than translations. I thought about changing inputPath and outputPath to something different. Unfortunately it doesn't work, because the default path is hardcoded in node_modules/ember-intl/app/instance-initializers/ember-intl.js

@estshy Right now, this isn't supported but in the future I will likely change that if you want to wait.  Otherwise, I'd welcome a PR if you are interested in tackling this.

IMHO changing the folder would be against the convention over configuration concept in Ember. What reason do you have for changing that folder?

@Turbo87 we can support both w/o much work IMO.  As someone who frequently bumps up against conventions, having an escape option is always nice.  But it would be nice to know the use-case, perhaps we've overlooked something.

But it would be nice to know the use-case, perhaps we've overlooked something.

that's all I'm saying. if there are good reasons to put them elsewhere we can support it, but I'm against supporting stuff just because we can if it increases complexity without much to gain.

Well. I would like to keep default language labels locally in the repository and fetch all remaining languages from the api with one little remark. In addition to that I would like to have them inside pods directory.
Why?  Also I don't want to have translations that are not used anymore, so when I am removing something from the pods directory I want to remove all related translation as well.
Default folder simply doesn't work well in large scale applications.

Well. I would like to keep default language labels locally in the repository and fetch all remaining languages from the api with one little remark.

This is already possible today https://github.com/jasonmit/ember-intl/blob/master/docs/asynchronously-loading-translations.md
As for pods, I would treat that entirely separately than a configurable outputPath.  Perhaps you want to open a separate issue (for tracking) for this.
I have plans to work on an addon that is library-agnostic that will work with ember-i18n and ember-intl to flatten yaml/json translation files.  This way you can put translations anywhere within your app which will address pods.  It likely will not deal with auto-namespacing, but will error/warn on key collisions.
/cc @alexlafroscia who has also expressed interest in this

@jasonmit that should probably target the new module unification RFC since pods are somewhat deprecated now

@Turbo87 what I had in mind wasn't targeting any specific app directory shape.  It would glob for **/*.(yaml,json) filter on everything that looks like a locale, merge by locale, and remove them from the app tree as they're merged together.
Lets chat over at #401

For what it's worth, our use case is such that we have an independent repo with the translations that's used for multiple projects (ember and non-ember).  Without the ability to specify the location this becomes cumbersome to integrate :(

@travisghansen were you planning to use submodules?  Curious the strategy so that I can support it.

@jasonmit well, our intent was to make a very 'dumb' repo with just the translations and include bare minimum per-language/toolkit packaging 'stuff'.  My hope was to add a simple package.json so we could version the translations and just include the npm package from the various js/ember projects.
Outside of npm (ie: package.json) we currently have a php app as well so we're dropping a composer.json in the translations repo and then simply include the composer package from the various php projects as needed as well.
High-level: translation repo -> ci -> when tagged build npm/etc/etc packages -> include in various packaging tools as a completely independent package

I should note for development purposes it would be ideal to point at any arbitrary folder outside the project structure.

Thanks for the info, I think I can get this working fairly easy.  I'll ping you once I have something.

@jasonmit thanks for the consideration!

Does the existing project have the ability to configure file naming conventions?  ie: en-us.json vs en_US.json etc?

@travisghansen it looks like I already added support for changing the inputPath but never made it part of the public API: https://github.com/jasonmit/ember-intl/blob/master/packages/ember-intl/tests/dummy/config/ember-intl.js#L63-L66
Can you verify this works for changing the translation path?  I'll look to make it public and update the documentation.

Does the existing project have the ability to configure file naming conventions? ie: en-us.json vs en_US.json etc?

Not yet, but something I can implement.

Okay, everything is on master right now.  The case insensitive translation file names along with the inputPath property made public.
https://github.com/jasonmit/ember-intl/commit/3935bf6b41d0538f1f8b43320a53c50f87d081b1
I plan to version tonight.

@jasonmit just to be clear, it's also _ instead of - so case insensitive alone won't do the trick on the convention side of things.

@travisghansen correct.  Your translation files can be named en_US.json or en-us.json or en_us.json.  ember-intl will normalize the the locale name to en-us at build time.
Also, I went ahead and versioned it as 2.21.0 so feel free to report back anything you find.
https://github.com/jasonmit/ember-intl/releases/tag/2.21.0

@jasonmit Thanks!  I'll have my team review to see how that works both from a dev and prod perspective and then provide feedback either way.

@jasonmit so far this looks like it will fit out need perfectly.  I would only suggest (not needed for my use case) that someone may want to supply an absolute path vs relative.  Also (forgive my ignorance), if I understand correctly if I'm going the npm route the dependency only needs to be a devDependencies as the translations are compiled into the build and not read at run time?

Correct on the dev dep (unless we are talking about an addon then the translation package would be a regular dep within the addon).
As for absolute paths, I believe it's possible.  I'll investigate

@jasonmit ok, thanks again for the consideration!  This was tremendously helpful!

@jasonmit It looks nice. Works almost exactly as I was expecting. The only thing which I dislike is repeating translation path in each translation file.
Structure:
app
-- pods
---- notifications
------list
--------en-US.yaml

File en-us.yaml:
notifications:
   list:
     title: 'Notifications'


Make a real CI pipeline for this project

I don't care if it's travis or jenkins. This real CI pipeline should be able to

create branch and merge jobs for each PR
build the projects
Test all haskell unit tests in all stack projects
test the rpc client tests against kvstore app baked into tendermint, see the README.
Run e2e test suite for simple storage against the docker-compose network
Run the e2e test suite for nameservice against the docker-compose network

it would be nice if the tests run in parallel.
By docker-compose network, I mean the one in the docker compose file. For example, here is the one for simple storage docker-compose file
There is a non-functioning dockerfile for each example app. It is an outstanding issue to fix it. I don't care if those e2e tests run the example app in docker, or if they run the app locally against the docker network that it depends on.
NOTE: The tests requiring a tendermint core docker image must wait until they push a 0.32.4 tag to dockerhub, or we must make this image ourselves and host it from our docker repo.
We can continue to run a merge travis job for hlint and stylish when the target branch is master.
Eventually there will probably be a nix component to this, but only if one of the future consultants can do most of the heavy lifting there.

NOTE: The tests requiring a tendermint core docker image must wait until they push a 0.32.4 tag to dockerhub, or we must make this image ourselves and host it from our docker repo.
Is this a hard requirement on patchlevel .4? I see there was an image of tendermint/tendermint v0.32.8 pushed yesterday, and there's been a v0.32.3 and v0.32.2 for 3 and 4 months respectively

Test all haskell unit tests in all stack projects
Is this covered by make test-libraries?


Component hierarchy sample

Can you please provide an example for a component hierarchy?  Something like:
<component-1 class="component-1">
    <component-2 class="component-2">
         <component-3 class="component-3">
         </component-3>
    </component-2>
</component-1>

NOTES:

I like to see an example where component-2 maintains a data list that component-3 iterates over and displays its items so that I can see an example of cross-hierarchy data access.  E.G.

  Data:

  Component-1 template:

      <component-2 content goes here>

  Component-2 template:

      <component-3 content goes here>

  Component-3 template:

{{#name.last}}, {{#name.first}}

I would like for the whole component hierarchy to be re-usable in an app page.

As given this can probably be simplified but the point of the exercise is to see how component insertion works in a hierarchy of components.
I have all this working in our Angular 1 with directives/transclusion but having a hard time migrating it to Angular 2.

Take a look at my treeview component. It makes a component hierarch of n levels. It's the same component, but recursive rendering makes it a tree structure

Hello @thelgevold I took a look at all your sample components already, including the treeview one.  I believe none of it uses the concept of insertion points().  In my case, components above(eg, component-2) define a place holder for components below(eg, component-3).  The components above (eg, component-2) can define its own template as well as transclude (or insert) its child component  (eg, component-3) onto itself.

I'm going to close this.  My Angular 2 beginners ignorance is probably leading me to think about this in the wrong way.  I'm digging deeper into Angular 2 and hopefully that will shed some light as to how best redesign my Angular 1 directives in Angular 2's component way of thinking.  Sorry for the troubles!

Ok, cool. I haven't had a chance to look into the < content > functionality in Angular 2.0 yet. Will post an update if I am able to put something together at a later time.


Cannot connect to restored database

This might actually be a database issue... I'm not sure.
It seems, however, that controller is never able to connect to a database that has been restored from backups.  With a fresh db, there is no problem.

@krancour how have you been testing this? just kubectl --namespace=deis delete pod <database pod>?

Nope... afaict, I have all components successfully using s3 for object storage now... so I've been going through scenarios where I helm uninstall the entire platform and then re-install.

Hmm. Perhaps you're hitting https://github.com/deis/postgres/issues/56. I just ran helm uninstall && help install without any drama, however I know that issue is still occurring today. I found my controller on a CrashLoopBackOff, but after deleting the pod the controller came up and is now able to communicate with the database.
That being said, there was no data in the database...
><> deis apps
Error:
401 Unauthorized
detail: Invalid token.

Let's route all stuff over to deis/postgres#56 and try to figure out why wal-e is being a pain.


10090 viewer seats tests bugs

This closes #10090.
@xavijam CR this, please.

@xavijam last CR after semistandard checks and fixes for tests to work, please.

One question, can a viewer apply a SQL?

@xavijam yes, they can (confirmed with @saleiva, cc @javisantana ). Read-only operations (as they can only get read permissions).
This is ready to merge after your +1 if nothing else comes to your mind, @xavijam .

:+1:

If Travis tests are ok, you are ready to go.


Str graphemes

This is mostly work done by @snprajwal in https://github.com/roc-lang/roc/pull/4286 who asked for some help with Str.graphemes on zulip.
discussion: https://roc.zulipchat.com/#narrow/stream/316715-contributing/topic/Missing.20Str.2Egraphemes
This pr reworks strGraphemes() and adds some passing zig tests.  I think more tests are needed but i'm not sure what types of tests to add.
The tests check these 4 cases currently:
1. "" -> []
2. "ab" -> ["a", "b"]
3. "éå" -> ["é", "å"]
4. "6🤔å" -> ["6", "🤔", "å"]

Great work @travisstaloch :rocket:
Note for GitHub: Closes #4286

That was quick! CI will fail though, bitcode.rs needs to match.

It's OK I managed to do it from the PR

It's OK I managed to do it from the PR

Oh neat. Didn't know you could make a commit like that. I didn't see you'd done that until after i pushed a fix. Hope I didn't mess anything up.

Oh yeah we require signed commits. Just noticed the "blocked" message now, didn't think to check.
Here are some links about it. I won't be around for the next few hours but you can ask on Zulip.
https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits
https://superuser.com/questions/397149/can-you-gpg-sign-old-commits

Oh yeah we require signed commits. Just noticed the "blocked" message now, didn't think to check. Here are some links about it. I won't be around for the next few hours but you can ask on Zulip.

Thanks. Wondered about that. I ran this command.  Will try to re-push and see if they're signed.
git config --global commit.gpgsign true

It would be good to squash the latest two fixup commits with your original commit to ensure git bisect doesn't break. Although if @brian-carroll will be squashing the PR itself, then it shouldn't really matter

Ok I think i figured out how to setup commit signing. Does the latest commit look ok?

Yep it is signed, but you seem to have not added your public key to GitHub. See https://docs.github.com/en/authentication/managing-commit-signature-verification/adding-a-gpg-key-to-your-github-account

Ok i just pushed again. Not seeing 'verified' so that seems to have worked. Thanks @snprajwal for the tip.

I also just squashed the commits. So i'm hoping that everything is signed now.

Ok thats better. I somehow lost the last two commits during squash. Sorry for all the noise.

One more thing i'd like to do is combine the tests for countGraphemeClusters() with the new tests i've added. I'd like to do this by adding a test helper which checks the count and the clusters, something like this:
test "graphemes: emojis, utf8, and ascii characters" {
    try testGraphemes("6🤔å🤔e¥🤔çpp", 10, .{"6", "🤔", "å", "🤔", "e", "¥", "🤔", "ç", "p", "p"});
}

It appears that @snprajwal's commit isn't signed and that's what is blocking merging.  I attempted to preserve his commit but i'm guessing this is blocked because that was before i had signing setup properly on this machine.
Does this make sense? Any suggestions?

Any suggestions?

git rebase -i HEAD~1
Add exec git commit --amend --no-edit -S after my commit
git push --force

Guessing this should fix the issue

I attempted to preserve his commit

Co-authored commits must be signed by the primary author, as a commit can be signed with only one GPG key

looks like they are both verified now.  thanks again for the assist with signing @snprajwal!

Thank you both for the great work on this! 😍

Closes #4193


emscripten time_t has wrong size

time_t is 64bit on emscripten since https://github.com/emscripten-core/emscripten/pull/17401
Currently the undersized struct definition here truncates the last 4 bytes of the timespec, I believe, so that Instant::now() etc. end up having no sub-second resolution on wasm32-unknown-emscripten.
Filed these issues with more details before I landed here:
https://github.com/emscripten-core/emscripten/issues/19872
https://github.com/rust-lang/rust/issues/113852
It is curious to note that time_t was only 32-bit for a short while, starting with https://github.com/emscripten-core/emscripten/pull/16966 (edit: according the comment in that PR, the size on wasm32 stayed the same at that point, so I presume it was indeed 64-bit for the first time from 17401)
although it has been 32-bit in this repo for 6 years. I am not seeing the same issue before 16966 that I am seeing after 17401, however. So there is probably another glitch or two that either masked the issue before 16966 or (in between 16966 and 17401) enabled the issue to surface as soon as the size became wrong again.
Cheers!

I think PR https://github.com/rust-lang/libc/pull/3569 would fix this.


Feature Request: SSAO

It would be interesting to see what SSAO could bring to OSRS. I imagine building interiors would look much better.
Has anyone tried this? Looking at SSAO implementations it seems easy, but I'm not experienced enough to be sure. I may try this some weekend, but it certainly won't be pretty! :laughing:

SSAO would be awesome! Since I've worked with SSAO before I've looked into it and I've considered two possible options. Since access to a depth buffer is a requirement for SSAO, the plugin would probably need to use a depth prepass for its current render pipeline. Another option would be to switch to deferred rendering which could in theory also help with rendering a lot of lights.
Either way would require quite a few changes to the current render pipeline and could impact performance.
If someone with more knowledge about this can think of other options I'd love to hear about it!

Either way would require quite a few changes to the current render pipeline

Darn, that probably makes it out of my skillset. :(


⚠ Bofrost.at has degraded performance

In d96f9b1, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1568 ms

Resolved: Bofrost.at performance has improved in e9721e9 after 9 minutes.


fix: Include dependency substitutions in dependency extractors

This addresses an issue with the affected-paths calculator where projects depending on included builds were not being returned when changes occurred in those included builds. This is becauseConfiguration.allDependencies used in dependency extractor was including artifacts from the included builds rather than the project dependency substitutions.
Details:
This updates DependencyExtractors.kt to include project dependencies from resolved artifacts.

The new method checks if the configuration can be resolved (e.g. compileOnly configuration cannot be resolved).
It retrieves dependencies from the resolution result and differentiates between direct and transitive dependencies.
It filters project dependencies and converts their paths so it will supported in the affected-paths logic.

Testing:
Published a snapshot version and used that against a gradle project with included builds. Verified that the new affected-paths accounts for projects dependent on included builds.

So looking further at what is being resolved here, this is actually solving an issue with dependency substitution of an included build. I believe this to be a good fix to the issue, although the concern I have is that it would increase the time spent by the Gradle daemon configuring, since now it is resolving dependencies. I wonder if having a flag for configuration resolution should be considered?

Please make sure to address @pablobaxter's comments. Thx!

Thanks for reviewing this. Yes, that has been addressed here https://github.com/square/affected-paths/pull/31/commits/2f95be9b1752f6bc5dfb6ae6778560228fc6e14e. I also discussed with Pablo this implementation last week and resolved https://github.com/square/affected-paths/pull/31#discussion_r1710733057.

Pulled in all commits and work into this PR: https://github.com/square/affected-paths/pull/32.


pip-tool: Add version 1.1.2

PiP-Tool: is a software to use the Picture in Picture mode on Windows.

https://github.com/LionelJouin/PiP-Tool#requirements

Microsoft Windows Vista or greater (64 bit).


[Feature] The ability to specify the map as an anchor to /map

It would be great if we could have the ability to specify an anchor in the URL that, when clicked, opens Pathfinder on a specific map. This means you could share maps by linking them to other users, useful when you have many maps available.
For example:
https://pathfinder.dev/map#mymap
would open Pathfinder on the 'mymap' tab.

:+1:

This is already implemented. There is even a "copy to clipboard" icon for deeplinks into a specific system on map


check for end iterator before trying to modify candidates table when …

…updating number of votes

Task linked: DEMZNE-1126 Investigate and fix "cannot pass end iterator to modify" when voting under certain circumstances


[Event Request] table 336 "Tracking Specification" - OnAfterSetNewTrackingFilterFromNewTrackingSpec

We would like to add a new event OnAfterSetNewTrackingFilterFromNewTrackingSpec and a new procedure:
procedure SetNewTrackingFilterFromNewTrackingSpec(TrackingSpecification: Record "Tracking Specification")
begin
    //A200F0000.Event.ns
    SetRange("New Serial No.", TrackingSpecification."New Serial No.");
    SetRange("New Lot No.", TrackingSpecification."New Lot No.");

    OnAfterSetNewTrackingFilterFromNewTrackingSpec(Rec, TrackingSpecification);
    //A200F0000.Event.ne
end;

Event Signature:
[IntegrationEvent(false, false)]
local procedure OnAfterSetNewTrackingFilterFromNewTrackingSpec(var TrackingSpecification: Record "Tracking Specification"; FromTrackingSpecification: Record "Tracking Specification")
begin
    //A200F0000.Event.n
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Path Variables in terraform commands is not substiting the correct variable ( path: ${CIRCLE_BRANCH} )

If we are adding the path as variable the orb fails
Path does not exist: ${CIRCLE_BRANCH}
The code which I am using is
version: 2.1
orbs:
  docker<EMAIL_ADDRESS>  terraform<EMAIL_ADDRESS>
jobs:
  terraforming:
    executor: terraform/default
    steps:
      - checkout
      - terraform/init:
          path: ${CIRCLE_BRANCH}

Found the solution, so closing the same.
version: 2.1
orbs:
  docker<EMAIL_ADDRESS>  terraform<EMAIL_ADDRESS>
jobs:
  terraforming:
    executor: terraform/default
    steps:
      - checkout
      - terraform/init:
          path: << pipeline.git.branch >>

workflows:
  terraform-init:
    jobs:
      - terraforming

Resolved.

Hello just wanted to share that your original request has been resolved in 3.1.0


ninja: disable line-clearing with TERM=dumb

Description of changes
TERM=dumb ninja is 1% prettier than the current solution ninja | cat
cat shows up as a separate process in my process tree
TERM=dumb is less visible
based on https://github.com/NixOS/nixpkgs/pull/149810#issuecomment-1030599299
ping @pennae
Background
https://github.com/ninja-build/ninja/blob/master/src/line_printer.cc
  const char* term = getenv("TERM");
#ifndef _WIN32
  smart_terminal_ = isatty(1) && term && string(term) != "dumb";

  if (smart_terminal_) {
    printf("\r");  // Print over previous line, if any.

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

if it works all the same then that's fine. should target staging though because it'll cause a mass rebuild.

lets set TERM=dumb as default?
for example, this ninja gives no output
https://github.com/NixOS/nixpkgs/blob/f72be3af76fb7dc45e2088d8cb9aba1e6767a930/pkgs/development/tools/build-managers/gn/generic.nix#L41-L45

nix-build largely passes TERM from the invoking shell to the builder, so patching ninja wouldn't help very much. setting TERM=dumb by default would also affect all other builds and all other commands. that might arguably be a good thing for reproducibility, but it'll mess with builders that can output pretty and useful logs right now :(

ninja could do getenv("NIX_STORE")

or similar

NIX_BINTOOLS=/nix/store/lvg99f3zni6zw4cvlci6wpmzlls0nsn4-binutils-wrapper-2.38
NIX_BINTOOLS_WRAPPER_TARGET_HOST_x86_64_unknown_linux_gnu=1
NIX_BUILD_CORES=3
NIX_BUILD_TOP=/build
NIX_CC=/nix/store/61zfi5pmhb0d91422f186x26v7b52y5k-gcc-wrapper-11.3.0
NIX_CC_WRAPPER_TARGET_HOST_x86_64_unknown_linux_gnu=1
NIX_CFLAGS_COMPILE= -frandom-seed=8cnrgjjflj
NIX_ENFORCE_NO_NATIVE=1
NIX_ENFORCE_PURITY=1
NIX_HARDENING_ENABLE=fortify stackprotector pic strictoverflow format relro bindnow
NIX_INDENT_MAKE=1
NIX_LDFLAGS=-rpath /nix/store/8cnrgjjflj3dyppz299w50l9yydgnqkp-x/lib64 -rpath /nix/store/8cnrgjjflj3dyppz299w50l9yydgnqkp-x/lib
NIX_LOG_FD=2
NIX_SSL_CERT_FILE=/no-cert-file.crt
NIX_STORE=/nix/store

if we do patch ninja we should do it in a form that can be upstreamed, looking at nix-specific environment variables doesn't sound like that would fit the case. maybe it's as simple as emitting line clearing only when outputting to a tty (which the builders do not)?

if we do patch ninja we should do it in a form that can be upstreamed

why?

nix-build -E 'with import <nixpkgs> {}; pkgs.stdenv.mkDerivation { name = "x"; unpackPhase = "if [ -t 1 ]; then echo term; else echo pipe; fi; exit 1"; }'

says term
feel free to post a better idea than getenv("NIX_STORE") ; )

why?

because carrying nix-specific patches to a build system just to make output a little prettier has a pretty bad cost/benefit ratio. this looks benign, but as soon as someone depends ninja doing what it does we'd have to patch that as well. (unlikely, but possible!)

says term

fd 0 is /dev/null though, so [ -t 0 ] says pipe as well. tty checks fd 0 to determine whether something is running on a tty or not as well, which is where our initial assertion came from. ninja checks fd 1 as you noted, a case could be made that it should check 0 instead?
in any case though: patching ninja for slight cosmetic changes is not the solution. we'd be better off patching nix to occasionally flush a line to the log reader once a certain buffer size limit is exceeded.

fd 0 is /dev/null though, so [ -t 0 ] says pipe as well

aah!
patch sent, fingers crossed : )

also when building ninja
building
bootstrapping ninja...
wrote build.ninja.
bootstrap complete.  rebuilding...
[36/36] LINK ninjainja.a.oKKx.o
installing

the 36 lines are line-cleared into one line

upstream said no, so lets just stick with TERM=dumb ninja

as soon as someone depends ninja doing what it does we'd have to patch that as well. (unlikely, but possible!)

yes. its not a bug, its a feature
when people call ninja directly in buildPhase, they will have to use TERM=dumb ninja too


Rewrite the data storage

Data should not be stored in .config.

Where do you think it would be better to store it in: .local or .cache? Or a $HOME/.dotodo?

I think it'd probably be best to store in $HOME/.dotodo, just to allow easier access to that data. I was thinking that we should probably let the user select where. But $HOME/.dotodo would be a nice default.

You know, the only opposition I have to $HOME/.dotodo is that, at least to me (and I've heard multiple people saying this), it's annoying how the home dir gets cluttered by "dotdirs". If we put the data in .local/share/dotodo it wouldn't contribute to such a situation.
Also, why do you think we should give users access to internal task storage? I mean, nothing wrong with that, but it still seems like something that maybe only power-users would be interested in. Besides maybe uses for synchronization, but as far as I understand, you're planning on implementing some sync in the app, so that shouldn't matter.

If we put the data in .local/share/dotodo it wouldn't contribute to such a situation.

That should be fine if it was mentioned somewhere in the README.md

Besides maybe uses for synchronization, but as far as I understand, you're planning on implementing some sync in the app, so that shouldn't matter.

It was more for allowing users to be able to backup their data locally if they wish to do so.

It was more for allowing users to be able to backup their data locally if they wish to do so.

Hm, okay, but I think just noting the location somewhere (e.g. `README.md) should be fine.


[FLINK-24978][build] Upgrade ASM to 9.2

Based on #18463.

Hi, when is it going to be possible to have a docker image with a release of 17 in it please?


[Staging & Production] S3: Access and Secret Key boxes remain up after successful submit

Reported by @ginavuu on the 0.27 Trello board
Steps

Go to user settings.
Connect to S3.

Expected
After typing in the keys, the secret and access key type box should go away if successful, and a success message should be displayed.
Actual
After success, both boxes remain up, and a success message is display below. Only when a user navigates away from the page and returns do the boxes disappear.

We should check how this functionality compares to the other add-ons -- do they all do this?

The other add-ons do not ask for username and password on the same page but instead goes to a different page.

https://github.com/CenterForOpenScience/osf.io/issues/3793 is the updated version of this issue.


Some screens get stuck when the app comes from the background.

The screen and many other screens get stuck when the app comes from the background. I think when the modal is open.

Duplicate of #4452.


⚠  Potential data loss within a working dir when using an argument . for the dest_path in convert function

I have not throughly tested CytoTable but I was just doing an install in docker env. I am yet to debug this but looks like I am unable to recover source path (directory and its recursive contents) if I follow convert function per tutorial.

Create a docker instance:

mkdir -p ~/dev && cd ~/dev
docker pull python:3.10.11
mkdir tmp
docker run -it -v tmp:/scratch python:3.10.11 /bin/bash

Within a docker container:

## install CytoTable
pip install git+https://github.com/cytomining/CytoTable.git

Successfully built CytoTable
Installing collected packages: duckdb, urllib3, typing-extensions, typeguard, tblib, six, setproctitle, pyzmq, pyjwt, pycparser, pyasn1, psutil, protobuf, numpy, jmespath, idna, google-crc32c, dill, cloudpathlib, charset-normalizer, certifi, cachetools, bcrypt, rsa, requests, python-dateutil, pyasn1-modules, pyarrow, isodate, googleapis-common-protos, google-resumable-media, cffi, pynacl, google-auth, cryptography, botocore, azure-core, s3transfer, paramiko, google-api-core, azure-storage-blob, google-cloud-core, globus-sdk, boto3, parsl, google-cloud-storage, CytoTable
Successfully installed CytoTable-0.0.1 azure-core-1.29.4 azure-storage-blob-12.18.0 bcrypt-4.0.1 boto3-1.28.46 botocore-1.31.46 cachetools-5.3.1 certifi-2023.7.22 cffi-1.15.1 charset-normalizer-3.2.0 cloudpathlib-0.13.0 cryptography-41.0.3 dill-0.3.7 duckdb-0.8.1 globus-sdk-3.28.0 google-api-core-2.11.1 google-auth-2.23.0 google-cloud-core-2.3.3 google-cloud-storage-2.10.0 google-crc32c-1.5.0 google-resumable-media-2.6.0 googleapis-common-protos-1.60.0 idna-3.4 isodate-0.6.1 jmespath-1.0.1 numpy-1.25.2 paramiko-3.3.1 parsl-2023.9.11 protobuf-4.24.3 psutil-5.9.5 pyarrow-12.0.1 pyasn1-0.5.0 pyasn1-modules-0.3.0 pycparser-2.21 pyjwt-2.8.0 pynacl-1.5.0 python-dateutil-2.8.2 pyzmq-25.1.1 requests-2.31.0 rsa-4.9 s3transfer-0.6.2 setproctitle-1.3.2 six-1.16.0 tblib-2.0.0 typeguard-2.13.3 typing-extensions-4.7.1 urllib3-1.26.16

docker image:: sbamin/cytotable_bug_issue102:0.0.1
cd /scratch
git clone https://github.com/cytomining/CytoTable.git
cd CytoTable
python

Run CytoTable convert function per turorial after giving ? a valid source_path argument. However, I am keeping a default  dest_path="." per tutorial.

from cytotable import convert

# using a local path with cellprofiler csv presets
convert(
    source_path="./tests/data/cellprofiler/ExampleHuman",
    source_datatype="csv",
    dest_path=".",
    dest_datatype="parquet",
    preset="cellprofiler_csv",
)

PosixPath('/scratch/CytoTable')

i.e., source_path /scratch/CytoTable directory (and all of its contents) are silently replaced by ? a parquet file, /scratch/CytoTable with md5: 219c76b59b7b2fb8fc37f5300ce50e31. I can not recover original source path, /scratch/CytoTable then after.
stat CytoTable

  File: CytoTable
  Size: 715142          Blocks: 1400       IO Block: 4096   regular file
Device: fd00h/64768d    Inode: 409909233   Links: 1
Access: (0644/-rw-r--r--)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2023-09-12 23:15:48.634381429 +0000
Modify: 2023-09-12 23:15:48.684381686 +0000
Change: 2023-09-12 23:15:48.684381686 +0000
 Birth: -

Now, in the same container, if I redownload CytoTable source code and rerun convert function but this time with a dest_path="testfile", instead of default dest_path=".", resulting ? parquet output is now at /scratch/ctable/testfile with an identical md5: 219c76b59b7b2fb8fc37f5300ce50e31 as above.

cd /scratch
git clone https://github.com/cytomining/CytoTable.git ctable
cd ctable
python

from cytotable import convert

# using a local path with cellprofiler csv presets
convert(
    source_path="./tests/data/cellprofiler/ExampleHuman",
    source_datatype="csv",
    dest_path="testfile",
    dest_datatype="parquet",
    preset="cellprofiler_csv",
)

PosixPath('/scratch/ctable/testfile')

file stats:
stat /scratch/ctable/testfile

  File: testfile
  Size: 715142          Blocks: 1400       IO Block: 4096   regular file
Device: fd00h/64768d    Inode: 143427352   Links: 1
Access: (0644/-rw-r--r--)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2023-09-12 23:17:20.450852099 +0000
Modify: 2023-09-12 23:17:20.499852350 +0000
Change: 2023-09-12 23:17:20.499852350 +0000
 Birth: -

and dir stats
stat /scratch/ctable

  File: /scratch/ctable
  Size: 4096            Blocks: 8          IO Block: 4096   directory
Device: fd00h/64768d    Inode: 143433934   Links: 9
Access: (0755/drwxr-xr-x)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2023-09-12 23:17:33.613919575 +0000
Modify: 2023-09-12 23:17:20.497852340 +0000
Change: 2023-09-12 23:17:20.497852340 +0000
 Birth: -

Thank you @sbamin for raising this issue and for the very thorough background! I plan to take a look into this and will follow up.

Just an update - I wanted to provide a background and proposed solution in reference to this.
CytoTable uses dest_path as a "work" directory where it prepares and delivers results. When using convert(..., join=True, ...)(a default) this directory becomes a single parquet file at the same location. The "work" directory of the same name is removed so as to perform "cleanup". The dest_path is used as a preparation location in contrast with temporary storage to avoid filesystem constraints (where data prepared there may be larger than allowed) and help with potential performance issues (where data transfer from temporary locations may be time-intensive).
I can see some roads towards resolving this and would be interested in your feedback (or additional options!):
a. We update the tutorial to not use . as a dest_path, also expanding upon dest_path usage there and within the docstrings associated with the work in Python.
b. We do the above and update CytoTable functionality to raise an Exception on encountering . as a dest_path, avoiding the . scenario entirely. I wonder if the same could be said of .. which may exhibit similar behavior.

Hi Dave,
Thanks for looking into this.

When using convert(..., join=True, ...)(a default) this directory becomes a single parquet file at the same location.

a. We update the tutorial to not use . as a dest_path, also expanding upon dest_path usage there and within the docstrings associated with the work in Python.

My take is to avoid such default and "cleaning" operation, especially when a directory is being replaced with a file object. User may run convert command from a parent working directory (where .git or other core files are - like myself), or worse case, home and then point a valid source_path and other arguments. I am concerned that .. could be even worse if say an user is running convert from ~/workdir/ or /projects/cytotable/ and that may end up replacing a level-up directory with a parquet file. This is unusual though given a traditional mv command should not move an existing directory with a file object. I suggest to have similar convention and let user decide about space constraints and explicitly state risk of replacing dest_path with a file object.

b. We do the above and update CytoTable functionality to raise an Exception on encountering . as a dest_path, avoiding the . scenario entirely. I wonder if the same could be said of .. which may exhibit similar behavior.

I suggest to raise exception for a dest_path if it is an existing directory and an user has not given an explicit true to let the directory (and all of recursive data) to be replaced by a parquet file.

Thanks @d33bs for fixing this promptly. I will wait until you close PR and put a new version. In meantime, I will take docker image offline sbamin/cytotable_bug_issue102:0.0.1

Thank you again @sbamin for your help in raising this issue along with the suggested approaches. With the merge of #104 I'm going to mark this as closed. If you think we should still address other aspects here please don't hesitate to mention it and/or raise additional issues in the future. Best wishes as you move forward in your research!

Wonderful and glad to be connected to you and team. Thanks and will be in touch.


Report tasks metrics to Prometheus

This PR adds Prometheus metrics to the sc_service::task_manager module.
There are four new metrics, each accepting a task name as label:

A counter incremented when we create a task.
A counter incremented when a task stops. The idea being to do started - stopped to know what's running.
A histogram with how long it took to run Future::poll. This can be used to figure out the CPU usage of the node.
A counter incremented every time we start polling a task. While this is kind of redundant with poll_duration_count, the idea is to do poll_started - poll_duration_count to detect tasks that are stuck.

I have consequently removed the futures-diagnose-exec tool, that was supposed to serve the same purpose but was totally undocumented and way more annoying to use on real nodes compared to Prometheus/Grafana.

Opened as a draft because this needs a Polkadot PR.

I was kind of worried about a performance degradation from having so many reports, but it seems fine.
Here's a CPU usage graph of my node. I deployed a new version of Polkadot with this PR around 18:40 UTC. You can't see any difference compared to the background noise.

Polkadot master seems broken. The error is unrelated to my changes, and the companion build was passing earlier.

unused imports let the CI fail.

icing while releasing.


Fixed procedure to mention volume selection

@JasonGerend
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Looks like the whole page needs a freshness pass, unfortunately. I'll take it over - let's close this PR.

Closing this PR as more changes are needed to this article.


KTX to TGA, PNG or DDS at least

Hello! I have a numerous of .ktx files, how i can convert them into tga or png or dds at least with kram?
00001459.txt
00001493.txt
They are actually .ktx, but for uploding to GitHub i had to rename them into .txt

Sure kram supports reading ktx/png/ktx2/dds and outputting dds/png.  I'm not going to deal with tga.  See the scripts directory, or run kram for cli help.