FileWatcher does not trigger for servers that have a symbolic link yaml-storage

Description
I have a second server that has a symbolic link of the folder yaml-storage for just permissions, that was fixed in 5.1.93, the slight issue is the only way to get the permissions to update on this second server once they are changed on the main server, is to run /lp sync Could we please have FileWatcher watch symbolic links too.
Expected behavior
For FileWatcher to automatically pick up changes to  symbolic link folders.
Environment details

Server type/version: Paper 1.12.2
LuckPerms version: 5.1.93

Any other relevant details
Please do not reply telling me to use MySQL instead I have already had that conversation in #2525

Unfortunately, I don't think this is something Java's file WatchService API supports this - not much I can do about that - sorry!

Hmm weird, I dont know Java but did some research just to see if I could find a workaround, looks like Symbolic links are transparent unless you specifically look for them, but they have a specific exception if the file is renamed or moved, no mention about checking for if its been updated.
https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/nio/file/package-summary.html

Many operating systems and file systems support for symbolic links. A symbolic link is a special file that serves as a reference to another file. For the most part, symbolic links are transparent to applications and operations on symbolic links are automatically redirected to the target of the link. Exceptions to this are when a symbolic link is deleted or renamed/moved in which case the link is deleted or removed rather than the target of the link. This package includes support for symbolic links where implementations provide these semantics. File systems may support other types that are semantically close but support for these other types of links is not included in this package.

I had a look on the WatchService API page for dependencies or OS based requirements but turned up nothing.
you can check for a symbolic link http://www.java2s.com/Tutorials/Java/Java_io/0980__Java_nio_Symbolic_Links.htm but I dont think this will help.
So if '/lp sync' is the right command to run, any way I can get my survival server to trigger this command for my other server? I have a wrapper that accepts console commands as parameters, but I would need to find a way to run a command/script when your file WatchService triggers?

Generally the path of least resistance would be to use one of the intended methods to sync your LP data such as MySQL, MongoDB, etc. These are designed for syncing data between multiple programs and you should use them as intended.
If you did want to jank-rig your own solution you would need something to run a console command on other servers, something like https://www.spigotmc.org/resources/commandsync.115/
You might actually be able to make use of LP's messaging services to do that but a) at that point just use a database anyway, b) the only one that works without a database is the pluginmsg option which only works when a player is online and c) not 100% sure if the messaging service gets called when LP isnt connected to a database.

Generally the path of least resistance would be to use one of the intended methods to sync your LP data such as MySQL, MongoDB, etc. These are designed for syncing data between multiple programs and you should use them as intended.
If you did want to jank-rig your own solution you would need something to run a console command on other servers, something like https://www.spigotmc.org/resources/commandsync.115/
You might actually be able to make use of LP's messaging services to do that but a) at that point just use a database anyway, b) the only one that works without a database is the pluginmsg option which only works when a player is online and c) not 100% sure if the messaging service gets called when LP isnt connected to a database.

I have already had to painfully explain in #2525 and I even wrote at the bottom of my ticket not to reply about MySQL, im more than fully aware of it all yet people still struggle to understand what a workflow is, please do not start this conversation again, Its already been covered in #2525 as linked in the original post.
Now forgetting about MySQL and carrying on with your other ideas, they both sound interesting.
c) I have split storage so users data is in MySQL but specifically just permissions separate as YAML, and yes the messaging system works on bungee.
How would I use the messaging system to do this?
That commandsync just runs commands on all servers on a bungee, so it would require a trigger to run it on the survival which means the file watch event still needs to trigger to run that command.
Alternatively I could just write a bash script that checks if the files been updated and checks it every X time.

The messaging service does not trigger for changes detected from configuration files.
Regarding the other replies above: Using a remote database to sync data between servers is a totally valid suggestion - it is literally what it was designed for. The flatfile storage was not designed with this in mind. You're running into problems (and are going to keep running into problems) because what you are trying to do is not particularly sensible or supported.
I have implemented setups like this before - not symlinks, but network shares / Git repos and special deploy scripts to sync config files between servers. It can work very well, but you need to realise that it is not something that plugin authors are going to jump to help you with. It is a non-normal setup, and you are going to run into quirks that you will need to sort yourself.
Back to the original issue - perhaps there is a way to get the WatchService API to play nice with symlinks, but I don't know without researching further. I will happily accept a good PR fixing the problem, but this is a very niche issue, and not a good use of my time at the moment.

The messaging service does not trigger for changes detected from configuration files.
Regarding the other replies above: Using a remote database to sync data between servers is a totally valid suggestion - it is literally what it was designed for. The flatfile storage was not designed with this in mind. You're running into problems (and are going to keep running into problems) because what you are trying to do is not particularly sensible or supported.
I have implemented setups like this before - not symlinks, but network shares / Git repos and special deploy scripts to sync config files between servers. It can work very well, but you need to realise that it is not something that plugin authors are going to jump to help you with. It is a non-normal setup, and you are going to run into quirks that you will need to sort yourself.
Back to the original issue - perhaps there is a way to get the WatchService API to play nice with symlinks, but I don't know without researching further. I will happily accept a good PR fixing the problem, but this is a very niche issue, and not a good use of my time at the moment.

Yup, I know. I have run the MySQL workflow where everything is in MySQL, I've done it. this is why I said it in my initial ticket to not take this ticket off topic because I knew people would go off on it, but nobody will actually read that the workflow is 100x slower.
I explained in the other ticket that the workflow for updating permissions (Not users) saved in MySQL is so poor in comparison to just being able to use VSCode to edit permissions is drastic, I came up with some solutions for improving the web UI to not need to have to login to the console/Minecraft to just push permission changes but these wont be acted on.
99.9% of the time, if you need to make a quick change to a permission, you don't want to have to go through the insane process of Starting Minecraft, waiting for it to load, logging into your server, being BOMBARDED by 100's of people that saw you come online, Run a command to generate a URL, copy that URL into a browser, use the web UI instead of a full Code editor, save your changes, copy that URL go back into game and paste it. (replace logging in with trying to copy a URl in a console thats moving the speed of light lol)
As opposed to the workflow of opening VSCode where all plugin configs are saved, make changes and just upload. Job done.
Its infuriating that nobody can see the difference in workflow here, why is there so much hostility towards me for wanting a better workflow.
Even the self hosted version of the web UI still requires this workflow and cant just connect directly to the MySQL database even though its on the same machine, that would solve it.
I dont know Java but I can ask the question about Symbolic links and the filewatcher in some Java communities to see if someone already has a solution and update this ticket.
I just dont want more people to lecture about converting to MySQL for everything just because thats what you are used to, I know, Ive done it.

There are a few things to point out when making a change to permissions regardless of setup/storage method.

You can always run commands from console. Being the one that can edit server files, I'm very sure you have console access... so I don't see the "insane process" being valid
You can also use commands for permission changes, such as lp group <group> permission set <node> to do a quick addition of permission node (and many others stuff using the relevant commands). You don't have to bring up the editor every time. (and note you don't have to open Minecraft and login in-game, see point 1)
Editor is still a good choice when you want to change many permission nodes. Sure you need to run /lp editor and then lp applyedits <code> (more steps) but the amount of time you are struggling with symbolic link probably already surpassed the simple additional steps of the editor. (and remember that you don't even have to do this if you are just doing small perm modification, see point 2)

There are a few things to point out when making a change to permissions regardless of setup/storage method.
1. You can always run commands from console. Being the one that can edit server files, I'm very sure you have console access... so I don't see the "insane process" being valid

2. You can also use commands for permission changes, such as `lp group <group> permission set <node>` to do a quick addition of permission node (and many others stuff using the relevant commands). You don't have to bring up the editor every time. (and note you don't have to open Minecraft and login in-game, see point 1)

3. Editor is still a good choice when you want to change many permission nodes. Sure you need to run `/lp editor` and then `lp applyedits <code>` (more steps) but the amount of time you are struggling with symbolic link probably already surpassed the simple additional steps of the editor. (and remember that you don't even have to do this if you are just doing small perm modification, see point 2)

Oh yeah, these are a given. At no point would I feel right trying to make permission changes based on just entering a command into the console.
Having a text editor open you can search, copy etc to see a quick overview of that entire rank you are editing is vital, you dont get any of that by just sending a command via console.
Commands in console have its own set of problems.

Feedback is posted with every other message being sent in chat, when you have a busy server with 50+ online this is impossible to keep track you have 0.1 of a second to read that feedback message.
You get no overview of a ranks existing permissions.
So I need to mass add permissions, great now I have to add them one by one.
I want to quickly move a permission from one group to another, I have to run 2 commands.
Where the hell is source control in all of this? At least as a file you have file history with whatever source control people use (Again I will re-iterate my users are MySQL for those that don't read) Im just talking about permissions.

Its not a workflow replacement, at all to having a full text editor.

3\. but the amount of time you are struggling with symbolic link probably already surpassed the simple additional steps of the editor.

Most likley, but once its fixed I don't need to have a special workflow just for permissions, compared to all my other plugins. Like I already said in the other ticket, 80+ plugins that are just all open and available  in VSCode.

Quick question: Do you have the base storage folder symlinked or everything inside it?

Quick question: Do you have the base storage folder symlinked or everything inside it?

No. Just the YAML folder, each server needs its own config.yml to tell each server what its called.
I thought of some more issues with MySQL/Commandline.

You have to be online to make any changes, you cant just work on anything offline. I found this out today on my laptop when I was out and about and wanted to make some quick changes to permissions, Opened up made them ready for when Im next online. You cant do this with MySQL permissioins (Not users) or Console Command.

There is really no point is giving the downside of one method, just so you can use your own method, it just annoys people.


Suggestion: support schema object "Title" property through existing SwaggerSchemaAttribute

Forgive me if I missed this somewhere, but I crawled through the docs, and even the code to see if there was some attribute or code for setting the title property of a schema object (as specified by the Open API spec), but it looks like this is not currently supported, except through a custom filter (which is what I'm using right now).
From what I can tell, this feature would be a trivial update to the AnnotationsSchemaFilter, however I wasn't sure if excluding title was a design decision or has deeper ramifications.
Would it make sense for the SwaggerSchemaAttribute to be enhanced with a title parameter?

See cb63055

Thanks for adding this! Much appreciated.


Initial integration with ExponentialFamilyProjection

This PR adds support for ExponentialFamilyProjection in RxInfer.jl. It introduces two new sections on running inference in non-conjugate models and handling factor nodes without explicitly defined message update rules.
This PR is accompanied by a corresponding PR to RxInfer.jl. Tests for individual rules are available in ReactiveMP, and this PR includes tests for the overall inference procedure.

Thanks for comments @albertpod , all of them have been addressed


Distributed training optimization

After playing with the current distributed training implementation for a while, I think it views each GPU as a separate worker.However, It is common now to have 2~4 GPUs in one box. Isn't it better to adopt the single box multi-GPU methodology to compute average gradients in single box first and then sync up across multiple nodes? This way it ease the I/O traffic a lot, which is always the bottleneck in data parallelism.

I think the example implementations might be one GPU per worker, but nothing prevents you from defining a worker having 8 GPUs, replicating the model across each GPU in a worker, averaging gradients before sending it to parameter servers.  TF is low-level enough that these are all possible.  To make the system easy to understand, the examples are simpler than what is possible.
I suspect higher-level libraries will be added over time that canonicalizes these types of model deployments that are becoming more popular.


unknown option '-p' and checksum do not match

§ nvm install stable
######################################################################## 100.0%
unknown option '-q'
options are
-c              to output the digest with separating colons
-d              to output debug info
-hex            output as hex dump
-binary         output in binary form
-sign   file    sign digest using private key in file
-verify file    verify a signature using public key in file
-prverify file  verify a signature using private key in file
-keyform arg    key file format (PEM or ENGINE)
-signature file signature to verify
-binary         output in binary form
-hmac key       create hashed MAC with key
-engine e       use engine e, possibly a hardware device.
-md5            to use the md5 message digest algorithm (default)
-md4            to use the md4 message digest algorithm
-md2            to use the md2 message digest algorithm
-sha1           to use the sha1 message digest algorithm
-sha            to use the sha message digest algorithm
-sha224         to use the sha224 message digest algorithm
-sha256         to use the sha256 message digest algorithm
-sha384         to use the sha384 message digest algorithm
-sha512         to use the sha512 message digest algorithm
-mdc2           to use the mdc2 message digest algorithm
-ripemd160      to use the ripemd160 message digest algorithm
Checksums do not match.
Binary download failed, trying source.

I got this error when trying to install 0.10.36 today, I'm using iTerm2 on OSX

What's your alias output? Specifically I'm looking for aliases to sha1sum, shasum, or sha (there shouldn't be any)

Oh, you're right. I had this line for alias alias sha1='openssl sha1'. So that's what causing the problem?

Yes - that shouldn't be there. However, I can bypass aliases, so I'll do that.

Cool, I've confirmed that things are all working again now. Thanks alot @ljharb
Just for curiosity, is there a way to keep this alias (well, I use it quite often) and not prevent nvm from working?

Yes - the commit I just pushed (https://github.com/creationix/nvm/commit/079294582452c7514e14635705c080295d7d3f81) which will go out with the next release I do, will allow you to keep your alias.
That said, it's an incredibly poor idea to override builtins with aliases, so I'd strongly recommend you do alias somethingElse=openssl sha1 and change your muscle memory :-)


Implement Event Registration Backend

This major PR provides the implementation of the Python backend for the Events Registration feature. This includes the SQLAlchemy Entities and Pydantic models, backend services, FastAPI routes, and test data.
This PR will ultimately serve to port over the ability for users to register for events, improving on the initial attempt from the A3 final project last semester.
Major Changes
Database and Model Changes

Added the EventRegistration SQLAlchemy Entity to model how registrations should be stored in the PostgreSQL database.
Established a many-to-many relationship between the user and events tables via the secondary join association table event-registrations.
Added the EventRegistration and NewEventRegistration Pydantic models to represent the shape of events data to be exposed by the APIs. The NewEventRegistration  model exists only for the creation of registrations by the component.

Scripts Changes

Added test data for event registrations. This data should populate when the test_reset.py script runs.

API Changes
Exposes the following APIs to the Angular Frontend via FastAPI:

POST api/events/register/:id: Adds a new event registration to the database.
DELETE api/events/register/:id: Deletes an event registration to the database.

NOTE: registrations are accessible through the GET APIs for users and for events.
Service Changes

Added the register() and unregister() functions to the EventService to manage event registrations.

Testing
These changes were thoroughly tested. New PyTests were added to thoroughly check the backend functionality for event registrations works. Test pass 100%.
Future Considerations

Frontend will be added in a later PR.
We can consider adding more fields and details such as timestamps to event registrations.
@KrisJordan, I think this can serve as a model for COMP 423 students to implement many-to-many relationships in their projects. We may be able to mention this on Monday and encourage them to pull from upstream/main if this looks good enough to serve as a model.

A few important changes we'll want to make to the design of this API and implementation:

We need to remove registrations from EventDetails. The reasoning is it leaks user data to all users (including unregistered) who are listing events.
We need to add an endpoint for getting the list of registrations for an event (guarded by permissions) since we're removing it from EventDetails.
We should roll back the changes to the UserAPI (including EventRegistrations) and establish an API endpoint for getting a user's registrations (is ideally going to have a similar pagination/history API as what top-level events will have, but if we want to launch with listing all, that's fine)
I'd like to try refactoring the EventRegistration models to avoid using event_id and user_id. These feel like they should be hidden behind the abstraction and are entity-level concerns. I can take a stab at this.

Going to take a stab at #1 and #2 next.

From #164, the scope of this backend API is starting to expand as some additional thought is put into use cases we need. Rather than merging into main before it's stable, and without a frontend, we'll demonstrate some concepts from this PR branch in lecture.
Some of the key pieces we need to resolve ahead of merging to main:

Flag for an Event determining whether users can register for it.
Max capacity of registrations for an Event.
Surfacing of registration count / seats available in API (and possibly denormalized count state in Event?) - better: advanced SQLAlchemy query that joins on an aggregate function. Count should not include event organizer registrations.
Logic for not allowing registrations beyond registration limit.
Logic for establishing who event organizers are (does the subject who creates an event automatically get listed as an organizer)?

One other nit, is the data modeling of Event correct with id: int | None. I know I did this in User, but I'm thinking this was a mistake in modeling. It leads to many places where we have to check for None, use the | bar, or suppress errors (such as in testing file).
I tend to wonder if we have something like an UnsavedEvent that has everything except an id, then Event extends with an id.

@KrisJordan re-requesting review! I have made an initial attempt for all of the bulletpoints mentioned and wrote tests for them (all pass atm)

@KrisJordan any updates on this, are we good to merge? I can also create a branch called feature/event-registration off of main as an intermediate step if needed here. Unsure if any migrations will be needed.

@KrisJordan, I am closing this ticket since there have been a lot of event registration backend changes on #212 !


run error

I ran command python ./dogfacenet/dogfacenet.py with environment ubuntu 18.04.2 LTS, python 3.6.8, and tensorflow 1.13.1.
And I got this error. I don't have any idea to fix this error and why this error show up.
Traceback (most recent call last):
File "./dogfacenet/dogfacenet.py", line 219, in
validation_steps=VALIDATION_STEPS)]
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py", line 1426, in fit_generator
initial_epoch=initial_epoch)
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/keras/engine/training_generator.py", line 177, in model_iteration
batch_data = _get_next_batch(output_generator, mode)
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/keras/engine/training_generator.py", line 258, in _get_next_batch
generator_output = next(output_generator)
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/keras/utils/data_utils.py", line 767, in get
six.reraise(*sys.exc_info())
File "/home/aji3rang/.local/lib/python3.6/site-packages/six.py", line 696, in reraise
raise value
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/keras/utils/data_utils.py", line 743, in get
inputs = self.queue.get(block=True).get()
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/multiprocessing/pool.py", line 644, in get
raise self._value
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/multiprocessing/pool.py", line 119, in worker
result = (True, func(*args, **kwds))
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/keras/utils/data_utils.py", line 680, in next_sample
return six.next(_SHARED_SEQUENCES[uid])
File "/home/aji3rang/dognet/DogFaceNet/dogfacenet/online_training.py", line 329, in online_adaptive_hard_image_generator
steps=int(np.ceil(len(subfilenames)/32)))
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py", line 1536, in predict_generator
verbose=verbose)
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/keras/engine/training_generator.py", line 171, in model_iteration
model.reset_metrics()
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py", line 1119, in reset_metrics
m.reset_states()
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/keras/metrics.py", line 460, in reset_states
K.set_value(v, 0)
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/keras/backend.py", line 2847, in set_value
get_session().run(assign_op, feed_dict={assign_placeholder: value})
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 929, in run
run_metadata_ptr)
File "/home/aji3rang/anaconda3/envs/tf3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1077, in _run
raise RuntimeError('The Session graph is empty.  Add operations to the '
RuntimeError: The Session graph is empty.  Add operations to the graph before calling run().

use tensorflow == 1.12.0.


Sports Betting - Place bet Use Cases

We've prepared the different use cases to show how a user place a bet.
For all cases, please take into consideration that

The initial state for the Wager field once the user includes a bet should be on focus.
$ is included by default, user can’t delete it.
To Win is a read only input.

Placing a single bet:
Desktop: https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7412%3A225993
Mobile: https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7477%3A263044
Placing multiple bets:
Desktop: https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7412%3A232657
Mobile: https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7477%3A263045
Transaction failed:
Desktop: https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7458%3A234838
Mobile: https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7502%3A258664
Odd changes
Desktop: https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7477%3A242607
Mobile: https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7502%3A262652
User tries to consume own order
Desktop: https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7412%3A225994
Mobile: https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7502%3A274589
Error examples: bet exceeds max amount
Desktop: https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7502%3A278736
Delete bet from betslip
https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=7502%3A281777

just change the best ask price once you have a bet up @matt-bullock

@phoebemirman tested this with Chwy and it didn't update


please update to 2.100.2 release

version 2.100.2 was released last month<EMAIL_ADDRESS>is it possible to get an update out to chocolatey

I just pushed 2.100.2 to chocolatey, sorry for the wait!


Deps diff

Codecov Report

Merging #60 (77cbe26) into main (ae8e58d) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main     #60   +/-   ##
=====================================
  Coverage   2.51%   2.51%
=====================================
  Files         30      30
  Lines        278     278
=====================================
  Hits           7       7
  Misses       271     271

Continue to review full report at Codecov.

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


Solidity Support features

Please try to include more solidity support features like autocompletions mostly especially for the most current Tools like Foundry

We try to keep VS Code lean and we think the functionality you're asking for is great for a VS Code extension. Maybe you can already find one that suits you in the VS Code Marketplace. Just in case, in a few simple steps you can get started writing your own extension. See also our issue reporting guidelines.
Happy Coding!


The current space is in the invalid state: RUNTIME_ERROR. Please contact the owner to fix this.

I am getting the error "The current space is in the invalid state: RUNTIME_ERROR. Please contact the owner to fix this." when running this code:
from gradio_tools import (StableDiffusionTool, ImageCaptioningTool, StableDiffusionPromptGeneratorTool,
TextToVideoTool)
from langchain.agents import initialize_agent
from langchain.llms import OpenAI
from langchain.memory import ConversationBufferMemory
llm = OpenAI(temperature=0)
memory = ConversationBufferMemory(memory_key="chat_history")
tools = [StableDiffusionTool().langchain, ImageCaptioningTool().langchain,
StableDiffusionPromptGeneratorTool().langchain, TextToVideoTool().langchain]
agent = initialize_agent(tools, llm, memory=memory, agent="conversational-react-description", verbose=True)
output = agent.run(input=("Please create a photo of a dog riding a skateboard "
"but improve my prompt prior to using an image generator."
"Please caption the generated image and create a video for it using the improved prompt."))

Same issue. https://huggingface.co/gradio-client-demos shows all spaces are in runtime error.


Update XcodeProj to 7.13.0

This has support for App Clips.

Thanks @brentleyjones. Looks like some of the ids changed in the update, so you’ll just need to check in the generated fixtures to get the tests to pass

@yonaskolb Tests are passing now 🙂


Update icon to the official one

So I originally used the material icon here on purpose, because Homebridge has fairly strict branding guidelines and I didn't think this would fit. Feel free to reach out to them if you'd like, I'd be happy to merge this if we get some kind of official blessing from them that it's ok.


on piper models we can find only one onnx file how to get the pairs from it

on piper models we can find only one onnx file how to get the pairs from it
thanks in advance for your great work

You can't. The pairs has to be exported from the original PyTorch checkpoint.

thanks a lot


ref(riak): remove riak

closes #68

@bacongobbler I think we should fix this instead of removing it. I haven't repro-ed the start failure, but Riak still does work for us, just not CS or Stanchion as of now.

right you are. Let's do that instead, but in the meantime I'll keep this open until we actually fix the problem upstream.

Sounds good!
Sent from my iPhone

On Feb 5, 2016, at 18:48, Matthew Fisher<EMAIL_ADDRESS>wrote:
right you are. Let's do that instead, but in the meantime I'll keep this open until we actually fix the problem upstream.
—
Reply to this email directly or view it on GitHub.


Roadmap

EDIT: @Salakar: See comment below; https://github.com/NodeRedis/node_redis/issues/1040#issuecomment-581418899

This roadmap is not sorted and has no fixed dates but is more a general mashup of things.
Main features to be implemented

[ ] Cluster
[ ] Sentinel
[ ] Input transformers
[ ] Output transformers
[ ] Native promise support
[ ] Offline queue limiter
[ ] Better script support / add individual commands
[ ] Better keep alive function / improve detecting dead connections

Other things that should be tackled

[ ] Document all code (JSDoc)
[ ] Make non documented API privat
[ ] Update README documentation to become usefuller
[ ] Fix windows spawn on appveyor
[x] Better stacktraces while not in production mode
[ ] Smoke tests

If you feel like there's something missing, please feel free to make further suggestions / open a feature request for it.

I'll be picking up:

[ ] Input transformers
[ ] Output transformers

👍

#1085 Do we have support for NX/XX flag support for commands ZADD

It's been a few months, is there any updated timeline on this front?

+1

Any chance cluster/sharding support will come soon? AWS has redis cluster support on ElastiCache and I would like to use it, but this lib needs to catch up to that feature set for that to really be viable :(

Any plans on replacing callbacks with Promises ?

This issue should be pinned IMHO
Also https://github.com/gosquared/redis-clustr might be a good enough solution for Cluster support?
An equivalent wrapper for sentinel would be great. Maybe https://www.npmjs.com/package/redis-sentinel but seems kind of dead (4 years since last publish).

Perhaps we should discuss the future of this repository here; the last publish to NPM of this package was over 2 years ago, there's fixes on master that haven't been published to NPM for almost 2 years, e.g. https://github.com/NodeRedis/node_redis/issues/1331;
Note that this isn't me complaining about @BridgeAR, he's actually doing a great job over in @nodejs, so understandably his time for this repo is limited.
I'm willing to pickup some of the maintenance burden but I'd like some thoughts on what course of action we could take for this given that NPM publish access is not available for the existing repository (have requested it several times over the years).
Some solutions I could think of (maybe vote with the reaction icons show below);

❤: deprecate this package and add alternatives to main readme, e.g. ioredis
👀: publish this package to a new NPM package name (probably scoped) as I and others do have some limited repository access
🎉: deprecate this repo (with a readme note) and move to a fork and work from there, new repository & NPM package, gives full control over the repo & NPM
😄: do nothing

👀 & 🎉 perhaps could just be short term for bug/security fixes, to keep it ticking over whilst people look at moving to alternatives?

@Salakar I'm also willing to help move this package forward. I'm not quite sure why we need to abandon the NPM package name ('redis' is powerful). Doesn't @BridgeAR control the NPM package and needs to transfer this to someone? There has been little action in many months, and I don't quite understand the logic of just sitting on it, to be honest.
I don't think we should deprecate this package - it's a good codebase that can be brought up to to date and huge number of other packages depend upon it.
The other thing I'd like to surface is the upcoming RESP3 / Redis 6 changes that will require significant alterations. I've looked into the ACL feature in Redis 6 which should be easy to support but we'll need some serious refactoring for node_redis. My job (at Redis Labs) would support my work on this module, but if we can't do a NPM release, it doesn't make sense investing the time.

Doesn't @BridgeAR control the NPM package and needs to transfer this to someone? There has been little action in many months, and I don't quite understand the logic of just sitting on it, to be honest.

Correct, but I've been requesting NPM publish access since Feb 2018, again in Feb 2019, and the most recent request in Sep 2019. I've had responses but not on the topic of requesting NPM publish access 🤷♂. https://github.com/NodeRedis/node_redis/issues/1402#issuecomment-490273744 might give some indication of intentions?
If the transfer isn't going to happen then I think it needs to be made clear so we can move on.
ioredis for example is real nice, and there was talks of consolidating the libraries back in the day (I worked on some of the underlying tooling for that such as the new parser, denque lib, cluster key slot calc etc): https://github.com/NodeRedis/node_redis#consolidation-its-time-for-celebration - which I still think should be the long term goal?

Off topic but I did start experimenting working on a new client a while back; https://twitter.com/mikediarmid/status/1074240036936318976 - but I stopped on it in the hopes of getting this one started again or helping consolidate;
Perhaps thats also a route 🤷♂

@Salakar have you talked to any of the other NPM owners (Matt, Ben, or Bryce)? If Ruben is MIA, then if we want the project to move forward I don't think a single person should be a roadblock. This sort of problem is why it was setup this way, I assume. I find the comment on 1402 troubling for an open source project, especially one that isn't tied (organizationally) to a single person.
I agree, ioredis is good but I don't think it's a one-size fits all solution. As far as consolidation, I thought the unified parser was the major goal that was already accomplished. I never considered that there would be fully a single module, just given the syntax differences.

@stockholmux: My job (at Redis Labs) would support my work on this module, but if we can't do a NPM release, it doesn't make sense investing the time.

Ditto also, we're willing to dedicate resource on this too @invertase, but if we can't publish its doesn't make sense for us either.

@stockholmux: @Salakar have you talked to any of the other NPM owners (Matt, Ben, or Bryce)?

This is a good point, I have not, I'll reach out to them shortly.

@stockholmux: I agree, ioredis is good but I don't think it's a one-size fits all solution. As far as consolidation, I thought the unified parser was the major goal that was already accomplished. I never considered that there would be fully a single module, just given the syntax differences.

Out of interest what is your requirements that the redis lib meets but ioredis does not? My requirements involved clustering & sentinel which redis currently does not support except though some third-party packages, some of which have also been abandoned.
Perhaps the underlying connection protocols between the two libraries could be shared also, then its purely just how you interface with each that is different?

@Salakar I like the modular approach to sentinel and clustering over the monolithic approach of ioredis (again, we need to deal with abandonware). Some people need this, others don't. The overall Redis ecosystem is getting bigger and modular is the way of supporting more without complexity, IMHO. Ioredis is a lot larger of a codebase (18,897 vs 7,038 lines of code), which may be more features, but I'd rather not have that much extra stuff I'm not using.
The place where I think node_redis has a big edge is Redis module support, which is easy with node_redis and rather a pain with ioredis.

I've reached out to @mranney via Twitter DMs and asked if he could grant both myself and @stockholmux owner access to the GitHub org and the NPM packages, so we'll see what comes from that.
I think myself & @stockholmux agree that maintaining and publishing everything from where they are currently is the best way forward - should we be able to. If not I guess we can look at the alternatives mentioned above, though I'd rather not.

A little suggestion. Maybe you should consider migrating source code to TypeScript.

Hey everyone, I've taken over as lead maintainer and have all the required access now 🎉 huge thanks to @BridgeAR for all the work he's done (and is doing) on this library and for letting me take over.
I've spent the last few days getting master ready for a release, and as of a few minutes ago I've just published v3.0.0 to NPM; https://github.com/NodeRedis/node-redis/releases/tag/v3.0.0 - which includes this change.
Expect regular releases - my first priority at the moment is making this project more contributor friendly to ensure that the project lives and continues to grow and is not blocked by any singular person's time. To do this I'd like to grow a larger set of shallow contributors. With this I hope to mitigate the previous problem of the project needing updates but there being no-one who has the power to do so. I'm working through the following on this;

[x] Contributing docs & Code of Conduct
[x] Setup an Open Collective and a contributors expenses policy

You'll notice the shiny new Sponsor button at the top of GitHub, I've also gone ahead and sponsored it myself and through my company to help kick start it for any future contributors

WIP: Release automation & semantic versioning (publishing to NPM, generating changelogs, etc)
WIP: Improve CI, e.g. Windows CI is super slow & flakey right now

After that I'll be switching my attention to modernising (e.g. promises, typescript) & clearing technical debt in the Node Redis codebase. @BridgeAR has already done a boat load of stuff for this, if you're curious check out the WIP v4 branch and its changelog.

@Salakar Congrats! I wanted to pick up the promises job (would love to ditch async-redis) but I assume it's mostly done by now. You guys have any idea of a timeframe? Are you accepting contributions on that front (eg.: do we have some sort of checklist)?

Hey @GCSBOSS, check out the 'v4' branch - its the in progress refactor, which has promise support, no eta / timeframe though yet sorry

@leibale Can you maybe share an update on the roadmap for 4.x? I would really appreciate it.

@sapkra Checkout this project or the v4 branch

@leibale Yeah, I have seen that you are actively working on it (that's the reason why I have mentioned you). Do you have an expected timeframe for the release. Even to know if Q2, Q3 or Q4 would be great.

@sapkra don't hold me on this, but I hope in Q3

@leibale Thanks, we all know how software development goes. It almost always takes longer.

@sapkra you can now install v4 alpha from npm npm install redis@next


fix: lint command execute error

miss chart.yaml.
When using ksbuilder create, the default Chart.yaml will be generated.
When using ksbuilder lint xxx, if the parameter is a directory, Chart.yaml will be updated based on extension.yaml.
value render error for extension.

fixed: #76 #77
fix: lint command execute error for ks-extension.

/lgtm
/approve

/lgtm


Text Render Issue

Installed via container. Text beneath countdown is overlapped by number countdown.
Can be seen here: https://i.imgur.com/UQNI6ba.png

Hello!
Could you tell me what browser you are using and what are your env values ?

Hello!
Could you tell me what browser you are using and what are your env values ?

Hi!
I’m using Safari, version 14.0.2. Find attached a screenshot illustrating
the environment variables. There were no adjustments/additions other than
the 'timer' values.
Thanks for the quick response.
On Sun, Jan 10, 2021 at 5:01 PM Timothee Boussus<EMAIL_ADDRESS>wrote:

Hello!
Could you tell me what browser you are using and what are your env values ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yooooomi/easy-countdown/issues/5#issuecomment-757551356,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASNLPKYR62BW7XQ4WJHBO3DSZIPRZANCNFSM4V4XFKQA
.

Hi!
I’m using Safari, version 14.0.2. Find attached a screenshot illustrating
the environment variables. There were no adjustments/additions other than
the 'timer' values.
Thanks for the quick response.
On Sun, Jan 10, 2021 at 5:01 PM Timothee Boussus<EMAIL_ADDRESS>wrote:

Hello!
Could you tell me what browser you are using and what are your env values ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yooooomi/easy-countdown/issues/5#issuecomment-757551356,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASNLPKYR62BW7XQ4WJHBO3DSZIPRZANCNFSM4V4XFKQA
.

As a quick follow up, I just tested using Chrome v.87.0.4280.141 and it
rendered normally. Safari appears to be the culprit.
On Sun, Jan 10, 2021 at 6:02 PM Dr Doom<EMAIL_ADDRESS>wrote:

Hi!
I’m using Safari, version 14.0.2. Find attached a screenshot illustrating
the environment variables. There were no adjustments/additions other than
the 'timer' values.
Thanks for the quick response.
On Sun, Jan 10, 2021 at 5:01 PM Timothee Boussus<EMAIL_ADDRESS>wrote:

Hello!
Could you tell me what browser you are using and what are your env values
?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yooooomi/easy-countdown/issues/5#issuecomment-757551356,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASNLPKYR62BW7XQ4WJHBO3DSZIPRZANCNFSM4V4XFKQA
.

As a quick follow up, I just tested using Chrome v.87.0.4280.141 and it
rendered normally. Safari appears to be the culprit.
On Sun, Jan 10, 2021 at 6:02 PM Dr Doom<EMAIL_ADDRESS>wrote:

Hi!
I’m using Safari, version 14.0.2. Find attached a screenshot illustrating
the environment variables. There were no adjustments/additions other than
the 'timer' values.
Thanks for the quick response.
On Sun, Jan 10, 2021 at 5:01 PM Timothee Boussus<EMAIL_ADDRESS>wrote:

Hello!
Could you tell me what browser you are using and what are your env values
?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yooooomi/easy-countdown/issues/5#issuecomment-757551356,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASNLPKYR62BW7XQ4WJHBO3DSZIPRZANCNFSM4V4XFKQA
.

Might be cause of Safari to be honest. I'll have a look tomorrow to see if there are compatibility issues.

Might be cause of Safari to be honest. I'll have a look tomorrow to see if there are compatibility issues.

This should be fixed now. Can you try and keep in touch to know if I can indeed close this issue ?

This should be fixed now. Can you try and keep in touch to know if I can indeed close this issue ?

I can confirm the issue is resolved. Thank you for making this available.
It’s exactly what I was looking for.
Any chance you will be making it more customizable in the future? Font,
size, colour, positioning, etc?
On Mon, Jan 11, 2021 at 9:04 AM Timothee Boussus<EMAIL_ADDRESS>wrote:

This should be fixed now. Can you try and keep in touch to know if I can
indeed close this issue ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yooooomi/easy-countdown/issues/5#issuecomment-757972074,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASNLPK554VRSS7PQITKEM2LSZMAPFANCNFSM4V4XFKQA
.

I can confirm the issue is resolved. Thank you for making this available.
It’s exactly what I was looking for.
Any chance you will be making it more customizable in the future? Font,
size, colour, positioning, etc?
On Mon, Jan 11, 2021 at 9:04 AM Timothee Boussus<EMAIL_ADDRESS>wrote:

This should be fixed now. Can you try and keep in touch to know if I can
indeed close this issue ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Yooooomi/easy-countdown/issues/5#issuecomment-757972074,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASNLPK554VRSS7PQITKEM2LSZMAPFANCNFSM4V4XFKQA
.

Yes I might have a look for more customization in the future. If you feel like you can add something to this project feel free to propose a merge request.

Yes I might have a look for more customization in the future. If you feel like you can add something to this project feel free to propose a merge request.


Track desired extras in project template

Description
Current situation

ballet quickstart causes ballet==x.y.z to be installed in the project
if the project maintainer wants to install any feature engineering extras, such as [all], then they must manually edit ballet.yml and setup.py
if they make these changes, every future invocation of ballet update-project-template will cause a merge conflict

Possible fixes:

prompt for extras in cookiecutter => a bit excessive
something else

Another possible fix is to move the extra feature engineering primitives to a namespace package that is release separately. Then ballet does not come with extras.


Tips for identifying the problem

Like many developers, I understand the gist of the message The LINQ expression could not be translated.. but it does not say which part of the expression doesn't work and the page also does not describe why some things work fine in C# but do not in VB.net.
For example, I have a where clause that does this (which works in C#)
.Where(m => m.Email.ToLower() == model.Email.ToLower().Trim() && m.Deleted == false)
(Deleted is nullable). When I try the same in VB.Net, even after removing the model.Email.ToLower().Trim(), I get the error about it not being able to run on the server. This is a table with 100s of 1000s of rows so I cannot run it in the client.
It would be nice to explain some of these issues in the article.

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

ID: fd35b912-445c-e801-7f70-de73839db80d
Version Independent ID: e558a081-5397-1e3f-1d58-abea3f72cad6
Content: Client vs. Server Evaluation - EF Core
Content Source: entity-framework/core/querying/client-eval.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @smitpatel
Microsoft Alias: avickers

@lukos translation failure exceptions generally do include the query fragment which caused an issue. We also progressively attempt to include customized messages for specific, common translation issues, to point people in the right direction (a good example is if you try to use the string.Equals overload with a StringComprison). We think these customized error messages are far better way than attempting to document all possible translations (and unsupported constructs) in an article; that would be impractical as LINQ is an infinite space.
However, you seem to have run into a specific translation failure with VB.NET. Can you please open an issue describing that - including a minimal, runnable code sample - on the main repo?

Sadly, I eventually found a bug which was already reported against EFCore in VB.Net where a basic string comparison is not correctly rendered in the expression and creates a call that will not work on the server.
It's annoying because it appears to be another, "we won't fix it except in .net 5", which is simply not possible for many of our projects. I have personally discovered various bugs in 5 and we can't afford to attempt to upgrade 50+ projects to .net 5 and hope for the best.
I have to work around this but I still think the page could provide some more pointers to what does and doesn't work and perhaps even "NOTE: There are some bugs in VB.net where your code will not work despite it working correctly in C#...."

I have personally discovered various bugs in 5 [...]

It would be good to have issues for these so that we can hopefully fix them for 6.0.

I have to work around this but I still think the page could provide some more pointers to what does and doesn't work and perhaps even "NOTE: There are some bugs in VB.net where your code will not work despite it working correctly in C#...."

I do understand that it seems that way, especially when you run into a specific bug or unsupported construct. But given how open LINQ is, a page detailing everything that isn't translatable would pretty much be a huge list that wouldn't be of much use to anyone.
Put another way... Given that we throw an exception when something isn't translatable, what would be the extra value of documenting that in some page? Isn't it much simpler to simply try executing a query with EF Core in order to know if it can be translated?

The bugs I found were all raised already but I don't think MS can use "Things will only be fixed in 5" at the same time as "there are some breaking changes which are not fixed yet"
Anyway, I don't think it necessarily needs loads of detail, there is already a page with supported and unsupported Linq constructs but specifically in this case, the structures are supposed to be supported but they don't work in VB and this was very confusing to me. If it wasn't for the reason that I knew the code worked in C#, I could have spent much more time trying to understand why it didn't work. A simple disclaimer that there are some known bugs in VB.Net doesn't seem excessive to me?

Sorry, to clarify, the bugs I have seen in 5.0 are not related to EF Core, just random different things that mean we can't simply upgrade everything.

A simple disclaimer that there are some known bugs in VB.Net doesn't seem excessive to me

As a matter of fact, they are no bugs for VB in EF Core directly. EF Core query pipeline deals with expression trees. Ideally compiler is supposed to create same expression tree for a query which is written in C# or VB if they are the same (just difference in language). But sadly this assumption is not maintained by compiler always because there are some random quirky behaviors. Some of those behaviors, unintuitive but they decided not to change to maintain backward compatibility. So task falls down to identifying the queries for which compiler generates different expression tree when coming from different language is simply not feasible at least until EF Core is fully done. In EF Core what we try to do is standardize those odd trees to C# one when we come across something like that.
So for disclaimer

If we put that different language can generate different trees, most users wouldn't understand what is happening underneath.
If we put that all queries won't translate for VB then same is true for C# also.
If we put that VB.NET in EF Core has bugs then it would be first misleading, and same is also true for C#.

So there is no additional value provided in putting disclaimer on this page.


Serial Port Configurability

I'm curious, from a code point of view what it would take to add configuration for data bits, parity and stop bits. Is there some hurdle to this being implemented? Asking before I start digging in for possible solutions. Sounds like you are open to contributions?

I am certainly open to pull requests.  I am not actively working on adding features, so most recent contributions have come from other people.
Configuration with more options should be relatively straightforward.  I would like to see:

Identical feature support on all platforms: linux, posix, osx, windows
Testing on several platforms including at least linux and windows
Backwards compatibility: if unset, the default should be for 8 bits, etc.
Implementation in the Config struct.  Some commented out starting points are here: https://github.com/tarm/serial/blob/master/serial.go#L77
Clean patches, which may mean rebasing your commits for bug fixes and after code reviews.

Several people have made forks and made changes to add those features but have never submitted pull requests.  I would start with those changes and either use them for inspiration or clean-up the patches and use them directly.  https://github.com/tarm/serial/network

@tajtiattila made a PR that adds this functionality and I just merged it to the master branch.
Please test and let us know if you have any problems.


Correcting wording for rule M2001

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[ ] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.

No modification for Python

No modification for Ruby

No modification for NodeJS

No modification for NodeJS

No modification for Python

No modification for Ruby

@amarzavery Looks like as i did change this on the docs itself so we are seeing push build. that'll cause the problem for push builds. Should i redo it offline to avoid push builds?

@vishrutshah - I would recommend closing this PR. Pull from upstream master and make a change in your fork and then send a PR.

@amarzavery Done with PR #854


Fixes for C++

C++ has bool type by default and utils.c throws undefined symbol errors without raylib.h :)

Yea, sure, if I remove #include "raylib.h", on compiling a C++ app I get these errors:
lib/libRaylib.a(textures.c.o): In function `LoadDDS':
textures.c:(.text+0x1663): undefined reference to `TraceLog'
textures.c:(.text+0x171c): undefined reference to `TraceLog'
textures.c:(.text+0x1734): undefined reference to `TraceLog'
textures.c:(.text+0x174c): undefined reference to `TraceLog'
textures.c:(.text+0x1764): undefined reference to `TraceLog'
lib/libRaylib.a(textures.c.o):textures.c:(.text+0x18e0): more undefined references to `TraceLog' follow
lib/libRaylib.a(core.c.o): In function `KeyCallback':
core.c:(.text+0x137): undefined reference to `WritePNG'
core.c:(.text+0x157): undefined reference to `TraceLog'
lib/libRaylib.a(core.c.o): In function `InitWindow':
core.c:(.text+0x543): undefined reference to `TraceLog'
core.c:(.text+0x701): undefined reference to `TraceLog'
core.c:(.text+0x75b): undefined reference to `TraceLog'
core.c:(.text+0x775): undefined reference to `TraceLog'
lib/libRaylib.a(core.c.o):core.c:(.text+0x78f): more undefined references to `TraceLog' follow
collect2: error: ld returned 1 exit status

Is it only when compiling under C++?
Yes!

Ok, got it! Without raylib.h there's no PLATFORM_DESKTOP defined for core.c file and it skips these functions. Is there a proper way to define it without including raylib.h?

Ah, I see, you adding platform definition globally in makefiles... I removed the include line!
The problem with bool type is still real :)


[FLINK-3771] [gelly] Methods for translating Graphs

Methods for translation of the type or value of graph labels, vertex values, and edge values.

Hi @greghogan,
I looked at the PR in more detail and I have some thoughts.
First, why do we need a Translate class and Translator interface? Can't we simply add methods in the Graph class that wrap a MapFunction or receive it as an argument like in mapVertices?
Second, translating values is already supported by mapVertices and mapEdges. Couldn't we simply extend these methods to also support changing the vertex key types?
Let me know what you think! Thanks.

Hi @vasia,
Comparing the APIs, Translate processes at the level of label or value whereas mapVertices/Edges process vertices and edges. What are use cases for the extra context (namely, the label) provided by mapVertices/Edges? Implementations of Translator can be reused across functions whereas the MapFunction provided to mapVertices/Edges are specific to that method. Would mapLabels require two new MapFunction or use an interface like Translator?
Translator is wrapped by a specific MapFunction for each Translate function.
Only one of the five current Translate functions operates on a Graph. tranlateGraphLabels could be moved to Graph, and there could be a translateVertexValues and translateEdgeValues also added to Graph. We don't always have a Graph, sometimes algorithms return a DataSet of vertices or edges, so having the static methods is helpful for these specialized Vertex and Edge operations.

Implementations of MapFunctions can also be reused :)
What I was thinking was to provide the translator simply as map functions, e.g. like Tuple2ToVertexMap. We can add them to org.apache.flink.graph.utils or create a subpackage for that. Then we can add the translate* methods to Graph. If someone wants to use the provided translator on a dataset of vertices or edges, they can simply do this with a mapper.
My concern is that we should try to be consistent with the existing Gelly API. e.g. something like graph.translateIds(new LongValueToStringValue()) is Gelly-like, while
Translate.translateGraphLabels(graph, new LongValueToStringValue()) is not. Also, I think that this feature is simple enough to be implemented as a collection of map functions instead of a separate utility.

By reuse I was referring to needing MapVertexValueLongToString and MapEdgeValueLongToString since the current Graph methods operate at the level of Vertex and Edge. And then to translate labels you need MapVertexLabelLongToString and MapEdgeLabelLongToString. In this PR we simply have a single LongToString which can be used to translate vertex labels, edge labels, vertex values, and edge values.
I agree that Translate methods taking a Graph as input would be better as methods on Graph.
And if we don't implement as MapFunction then we still need Translate to handle the TypeInformation.

How about we add the translate methods to the Graph class and implement translators as MapFunction<Old, New>? This way they can be used even if don't have a graph but a dataset of vertices or edges. There is no requirement to have the mappers operate on the vertex and edge level.
Does this sound like a good idea?

I think this is a very good idea, and we can wrap the user's MapFunction to function like a MapFunction.
Now we can start discussing algorithms :)

The switch to MapFunction means that a user can use a RichMapFunction with open, close, accumulators, etc.
Rather than add six new methods to Graph I implemented these with three GraphAlgorithm which use the Gellyish Graph.run(GraphAlgorithm)

Hi @greghogan,
I feel we're over-engineering something that should be way simpler... Translators are just a set of MapFunctions applied on vertex or edge ids and values. I think that the current API is way too complex for what it does, e.g.
graph.run(new TranslateEdgeValues<LongValue,LongValue,LongValue,StringValue>(new LongValueToStringValue())) seems too verbose to me for mapping an edge value.
It could be as simple as graph.mapEdges(new LongValueToStringValue()).
Also, I don't think we need 6 methods. We could either add 4 methods to Graph, i.e. mapVertexIDs, mapVertexValues, mapEdgeIds, mapEdgeValues or extend the mapEdges and mapVertices methods. In the latter case, we will have to provide specialized mappers as you note above, e.g. MapVertexValueLongToString, MapEdgeValueLongToString, etc., but these can internally wrap the same mapper.
Please let me know your thought on this! Thanks!

We can't translate vertex or edge IDs without modifying both as long as we want to return a Graph
Setting parallelism is critical to scaling Flink to large datasets which doubles the function count.
graph.run(new TranslateEdgeValues<LongValue,LongValue,LongValue,StringValue>(new LongValueToStringValue())) is verbose due to the parameterization and we should definitely try to find a way to improve that.

There's no reason we can't have both algorithms and methods on Graph which run the algorithm and ignore customization. I'll add this shortly.

In Java 8 this is {{graph.run(new TranslateEdgeValues<>(new LongValueToStringValue()))}}, see "Target Types" from https://docs.oracle.com/javase/tutorial/java/generics/genTypeInference.html.

One can now choose between:
graph.translateEdgeValues(new LongValueToStringValue()))
graph.run(new TranslateEdgeValues<>(new LongValueToStringValue()))
The latter is slightly longer but customizable with parallelism, etc.

My intention was to simplify the API, not have different ways to do the same thing. Anyway, if you think it's useful to have both the methods and the algorithms, I won't insist :)
Can you please add a section in the Gelly guide describing how/when to use the translators? Thanks!

I pushed the most recent changes. In order of things I care more about to things I care less about:
The indentation is as intended. I find method chaining on a single line quite difficult to follow. I find that algorithms are much easier to read if separate operators are easily identifiable.
Cramming implements, extends, and throws onto a single line is also difficult to parse, particularly for complicated nested parameters which may themselves extend interfaces.
There are empty lines throughout the code. The remaining two look fine to me.

Thanks for the updates @greghogan. Could you please also add the new GraphAlgorithms in the docs? And maybe explain when to use those instead of the Graph methods? Otherwise, it's good to merge imo.

Updated docs. Merging ...

Implemented in 08e4bf9440a566c874c2b8e74ae2127ff264c672


GetLegendGraphic with legendurl does not seem to work in non 1.3.0 versions

If a layer is configured with an (external) legendurl and the GetLegendGraphic request is used with any other version then 1.3.0 the request fails with the error message:
<?xml version="1.0"?>
<!DOCTYPE ServiceExceptionReport SYSTEM "http://schemas.opengis.net/wms/1.1.0/exception_1_1_0.dtd">
<ServiceExceptionReport version="1.1.0">
    <ServiceException>unknown WMS request type 'legendgraphic'</ServiceException>
</ServiceExceptionReport>

Hmm, looks like GetLegendGraphic has its first mention in WMS 1.1.1 (https://www.ogc.org/standard/wms/), so maybe you are trying to make something work which is not in the standard. I think this is not a bug


Javascript error in the console related to metric regex

Hello,
I was reworking a big panel which had 48 requests to Azure log analytics and causing us to hit the rate limit.
I went back to one single query with all the data, and started updating the rules.
The metric autocompletion works, and I can see the 48 differents metrics, and so choose the one I need.
After that when refreshing the dashboard I can see the following error in the js console :
module.js:2 Uncaught (in promise) TypeError: Cannot read property '1' of null
Related code :
                stringToJsRegex: function(t) {
                    if ("/" !== t[0])
                        return new RegExp("^".concat(t, "$"));
                    var e = t.match(new RegExp("^/(.*?)/(g?i?m?y?)$"));
                    return new RegExp(e[1],e[2])
                },

Error is on the return line.
What can I do to help you see what's the issue ?

Hi @morsok
Can u try todisable regex options if u don't use regex ?
BR

Hello,
Sorry for the late answer, I'm not sure I can deactivate the regex on this case, I'm speaking of the Apply to metrics field. I start typing, it correctly autocompletes with the list of 48 metrics, I choose the one I want (I disabled the 47 other rules so there's just this one active), and then I get the error and the graph does not refresh.
Would you need more screenshots / debugging steps / tests?

Hello,
Do you have some news about this pb ?
Best regards,
Charles

Cant reproduce, can you give me your regex ?


authorization endpoint to issue access tokens

When using the default Vaadin Services authentication endpoint I want to use an OAuth2-compliant Resource Owner Password Credentials authorization grant to obtain access tokens, so that I can access Vaadin Service methods securely and statelessly with an access token, and avoid passing the password credentials with every request.
DoD:

there is a /token HTTP endpoint compliant to the OAuth2 token endpoint requirements that returns an access token in response to valid resource owner password credentials
incorrect credentials should cause a 401 Unauthorized response code
the user details are supplied via an AuthenticationManager bean
the access token is a JSON Web Token that includes the following claims:

iss (issuer)
iat (issued at)
exp (expiration time)
sub (subject)
roles - an array of string role names, e.g. ['user', 'admin']

The common DoD applies.

Rather than directly using UserDetailsService and PasswordEncoder, it might also be feasible to use Spring's AuthenticationManager that combines those two classes and some additional convenience as well.

Agree. The ticket description was updated.


[FLINK-11326] [streaming] Fix forbidden negative offset in window assigners

What is the purpose of the change
Allow negative window offset in window assignment as the javadoc says.
Brief change log

Allow negative window offset in window assignment.
Throw IllegalArgumentException if offset is out of range for SlidingEventTimeWindows.of and SlidingProcessingTimeWindows.of.

Verifying this change
This change is already covered by existing tests and new test cases has been added to allow negative window offset.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Breaking changes

@PublicEvolving API SlidingEventTimeWindows.of and SlidingProcessingTimeWindows.of allows out of window offset previously, this merge request forbid this behavior. This way they behaves same as TumblingEventTimeWindows.of and TumblingProcessingTimeWindows.of. I think it is easier for caller to understand sliding windows.

@kezhuw The community is currently discussing to introduce labels, so that component owners can have a better overview over "their" pull requests.


icons should be white instead of black

With the current icons being black, then are near invisible on the W10 taskbar. Moving them to white would help with visibility.

Settings > Personalisation > Colours

Sure, it can be changed, but the default taskbar is black, and the default W10 systray icons are ALL white.

Interesting, ok...

Solution:
Just replace the PNG files in the directory. Here are the files that need to be replaced in the main folder.

Hope that it helped... however I figur that you already find out by yourself :smile:

Here are the files with inverted colors. I can see the icon now, but sadly the app doesn't work on my win 10 64bit pc.

Integrating icons in update


feat: partition into subintervals/squares adapted to an open cover

Also adds some useful instances and lemmas about the unit interval.
The subsquares version will be useful for Van Kampen.

[x] depends on: #7976

None of my comments above are intended to be blocking, this is outside an area I'm all that familiar with; they were intended as discussion points for (or to be ignored by) another reviewer who knows better.

bors r+


support ImageMinimizerWebpackPlugin

What problem does this feature solve?
Supports all hooks used by ImageMinimizerWebpackPlugin.
https://github.com/webpack-contrib/image-minimizer-webpack-plugin
Here is the list of unsupported hooks
compilation.hooks.moduleAsset
compilation.hooks.assetPath
compilation.hooks.afterSeal
compilation.hooks.statsPrinter
stats.hooks.print

we already supports https://www.npmjs.com/package/image-webpack-loader, so can  use it as workaround

image-webpack-loader uses imagemin
but image-minimizer-webpack-plugin can be used with sharp and svgo
surely this should be "closed this as completed"?

Bump

You can close it I managed to make it work with this: https://rspack.dev/guide/features/asset-module#using-optimizers-as-loaders

it work with this

Yes, but loader will only process those images that were imported. The plugin also processed those images that were simply copied by the HtmlWebpackPlugin plugin.
Therefore, in rspack, it was necessary to additionally manually configure optimization via transform in CopyRspackPlugin, like this:
transform: (input, absoluteFilename) => {
  if (absoluteFilename.includes('/affiliate/')) {
    return input;
  }
  switch (path.extname(absoluteFilename)) {
    case '.svg':
      return R.pipe(
        input.toString('utf-8'),
        content => svgo.optimize(content, svgoConfig),
        R.prop('data'),
      );
    case '.png':
      return sharp(input).png(sharpPngOptions).toBuffer();
    default:
      return input;
  }
},

So in an ideal world it would still be great to have support for image-minimizer-webpack-plugin as a plugin rather than as a loader.


CLOUD-741 fixed consul client creation

@keyki

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1122/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1123/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1124/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1125/


[Add Course] No Such Good resource for KOTLIN Course in Bengali

Lack of Quality Bengali Course for KOTLIN
There is no good resource available for learning Kotlin in Bengali. This is a problem because Kotlin is becoming increasingly popular in India and Bangladesh, and there is a growing demand for Android app developers in Bengali.

Proposed Solution:
I propose to add a new resource for a Kotlin course in Bengali to the list of resources.
The course covers the following topics:
1.Introduction to Kotlin
2. Kotlin syntax
3. Kotlin variables and data types
4.Kotlin functions
5. Kotlin object-oriented programming
6.Kotlin collections
7. Kotlin lambdas and higher-order functions
8. Kotlin Android development
The course is suitable for beginners and experienced programmers alike. It is a great way to learn Kotlin and start developing Android apps in Bengali.
Benefits:

This course will make it easier for Bengali speakers to learn Kotlin.
This course will encourage more Bengali speakers to develop Android apps.
This course will help to grow the Kotlin community in India.

Call to action:
I urge you to consider adding this new resource to the list of Kotlin resources. Thank you for your time and consideration.

Your suggestion is noted
If you're familiar with Git, feel free to submit a pull request. Ensure it aligns with the CONTRIBUTING.md guidelines
If you're not familiar with Git, please specify the link(s) of the resources you plan to add


RabbitMQ and Flower links not working in server status page

As an admin, RabbitMQ and Flower links are not working in server status page:

More info here: https://github.com/codalab/codabench/wiki/Administrator-procedures

In the production environment, it works only when using http instead of https.

In the production environment, it works only when using http instead of https.

Assuming on codabench-test it will also work with http. Creating a new PR with http

Is this fixed?

Is this fixed?

I think yes with https://github.com/codalab/codabench/pull/1153

I just deployed the last changes to the test server and it seems to not be working

Deployed on prod, it works

@Didayolo this can be closed


Fixes report

I have been trying to use this.  I found some fixes.  They are small, so just reporting for now.  They can be probably updated much faster by some else, then me forking / changing / and PRing.
In GIFEncoder.js:

line 148, should be sizeSet, not sizeset, causes exception when passing ImageData
The following lines need to be added to the end of download() to actually make it work:

		    var click = document.createEvent('MouseEvents');
		    click.initEvent('click', true, false);
		    templink.dispatchEvent(click);

I faced problem of download() not work on Mozilla Firefox.
I try to fix and make patch based on following article of Stackoverflow:
https://stackoverflow.com/questions/32225904/programmatical-click-on-a-tag-not-working-in-firefox
My implement is templink element append under document root. It's work fine on Firefox.
But @Palmer-JC 's code seems to more better. I hope to repository taking his code.


Automatically hide command buttons section if no commands are configured

Show only the screenshot

Released in https://github.com/maszaa/remote-desktop-software-controller/releases/tag/v0.5.1-alpha

Released in https://github.com/maszaa/remote-desktop-software-controller/releases/tag/v0.5.1-alpha


Remove logic that installs rdma-core package

Remove logic that installs rdma-core package to align the code for all platforms.
Before this commit the state is following:

Ubuntu - the logic is completely broken and newer executed.
Reason: buggy IsUbuntuCheck()
We use grep with --quiet flag and IsUbuntuCheck() returns true only if stdout is not empty. Because of the --quiet flag, stdout will be always empty. Broken for a long time, nobody reported an issue about this.
RHCOS - validation only logic from the begining
RHEL - package installation (partially broken in RHEL 8 and RHEL 9) Reason: This file Is available in RHEL 7, but not in RHEL 8 or 9

The operator currently uses quite dangerous logic to trigger loading of RDMA modules https://github.com/k8snetworkplumbingwg/sriov-network-operator/blob/ee40683567d8ac5d03365e3216592e74d47538b7/pkg/host/internal/kernel/kernel.go#L525
I think we don't want the operator to install the package and completely unload the driver, this can break multiple use-cases (NVIDIA driver in container, Externally managed PF and potentially more)
The commit changes logic for all OS to do only a validation of RDMA modules and print warning if they are not found.
@adrianchiris  @SchSeba

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

13 of 38 (34.21%)  changed or added relevant lines in 5 files are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage increased (+1.3%) to 45.238%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/sriov-network-config-daemon/service.go
1
2
50.0%

pkg/host/mock/mock_host.go
0
11
0.0%

pkg/host/internal/kernel/kernel.go
0
13
0.0%

Files with Coverage Reduction
New Missed Lines
%

controllers/sriovnetworkpoolconfig_controller.go
6
63.7%

Totals

Change from base Build<PHONE_NUMBER>9:
1.3%

Covered Lines:
6526

Relevant Lines:
14426

💛  - Coveralls

we should not unload mlx5_core driver from kernel, thats dangerous as primary NIC may become unavailable.
generally in NVIDIA we instruct customers to pre install rdma-core if inbox drivers are used, hence the install step is not needed from NVIDIA POV. (plus it seems it never worked :) )
+1 on just logging if something is wrong like we do in coreos. perhaps with a meaningfull msg like "rdma modules are not loaded, please install rdma-core package available from you package manager"

I am soooo happy to see this code going away from the operator!
never like it :)
Can you please just add in the read me information that the user must install the rdma package on the system? also you can add the in RedHat CoreOS that is not needed

@SchSeba  I updated the quickstart.md to mention that rdma-core packgage must be installed.


Masonry: Bug fix, "basic" layout not left aligning

Pull Request Template
Summary
Issue
When layout is set to basic, the docs state that the grid will be "Left-aligned, fixed-column-width masonry layout." This wasn't the case, due to gutterWidth pushing the items away from the left side. When gutterWidth is set to 0, it is appropriately left aligned, but then you have no gutter spacing between pins.
Additionally, center alignment was miscalculated due to the inclusion of an extra gutter (e.g. when you have 4 columns, you should only calculate 3 gutterWidths for the contentWidth)
What changed?
I've introduced a new optional prop justify which controls the horizontal alignment of items within the Masonry grid.
This change affects both Masonry and MasonryV2
Center Alignment
Center alignment was slightly off, due to an extra gutter being calculated

Before
After

Start Alignment
We were center aligning before, we are now properly start aligning when justify='start' is set

Before
After

End Alignment
End alignment didn't exist previously

Before
After

Did not exist

Why?
This change is needed for situations where we need the Masonry to be left aligned to match other Pin Grid formats (e.g. aligning Modules with PinReps with the Masonry).
This image highlights an issue we have, where the left aligned title looks disjointed from the Masonry grid.

Links

Jira
Figma

Checklist

[x] Added unit and Flow Tests
[ ] Added documentation + accessibility tests
[x] Verified accessibility: keyboard & screen reader interaction
[ ] Checked dark mode, responsiveness, and right-to-left support
[x] Checked stakeholder feedback (e.g. Gestalt designers, relevant feature teams)

@CalvinChhour sorry for the short notice, but I am planning on landing https://github.com/pinterest/gestalt/pull/3567 this week for TypeScript migration, and this PR will conflict with that. Can you check with @liuyenwei and determine if you'll be able to merge this PR tomorrow (5/16)? If so, I will wait and apply the same changes to the TS branch.


Buffering multiple can

I believe the buffer behavior works as indicated in the "usage", but it is NOT ready for release. Why? The code has instrumentation 'fprintf()' in place to help me confirm behavior.
I'm releasing it to allow you to confirm the behavior is what you expect.
Usage: acf-can-talker [OPTION...]
            [ifname] dst-mac-address/dst-nw-address:port [can ifname]

acf-can-talker -- a program designed to send CAN messages to a remote CAN bus
                  over Ethernet using Open1722. The default behavior is to
                  send one CAN frame per Ethernet frame. However, a buffer can
                  be used that gives the opportunity to pack more than one.

OPTIONS

      --count=COUNT          Set count of CAN messages per Ethernet frame
      --timeout=TIME         Set time to wait for CAN messages to arrive
  -t, --tscf                 Use TSCF
  -u, --udp                  Use UDP
  can ifname                 CAN interface (set to STDIN by default)
  dst-mac-address            Stream destination MAC address (If Ethernet)
  dst-nw-address:port        Stream destination network address and port (If
                             UDP)
  ifname                     Network interface (If Ethernet)
  -?, --help                 Give this help list
      --usage                Give a short usage message

BUFFERING

  acf-can-talker, by default, packs only one CAN frame into an Ethernet frame.
  Use the --timeout and/or --count options to change this behavior.

  --timeout <time>   where <time> is an integer followed by units,
                     e.g., '1s', or '400us'. Allowable units: 's', 'ms', 'us'.
                     The Ethernet frame will be sent <time> after arrival
                     of the first CAN message.

  --count <count>    where <count> is the max number of CAN frames allowed in
                     an Ethernet frame.

  If both buffering options are presented, then whichever occurs first will
  trigger the sending of the Ethernet frame.

  In all cases, when the Ethernet frame is full it is sent.

EXAMPLES
  acf-can-talker eth0 aa:bb:cc:ee:dd:ff

    (tunnel transactions from STDIN to a remote CAN bus over Ethernet)

  acf-can-talker --count 10 eth0 aa:bb:cc:ee:dd:ff

    (as above, but send Ethernet frame as soon as we have 10 CAN frames)

  acf-can-talker --timeout 400ms eth0 aa:bb:cc:ee:dd:ff

    (as above, but send Ethernet frame when 400 msec have passed since
     arrival of first CAN frame)

  acf-can-talker -u <IP_ADDRESS>:17220 vcan1

    (tunnel transactions from can1 interface to a remote CAN bus over IP)

  candump can1 | acf-can-talker -u <IP_ADDRESS>:17220

    (another method to tunnel transactions from vcan1 to a remote CAN bus)

Hi John,
thanks for all the work! I'll probably need a few days to review your code and test it a little bit more. On first glance, the way the timeout works seems to be correct.
However I've also encountered a few bugs, Unfortunately I did not have time to do a deeper investigation yet. I tried to run cangen vcan0 -g 500 and cangen vcan0 -g 750 in parallel (generating CAN frames in cycles of 500ms and 750ms) and in the talker I set a timeout of 500ms acf-can-talker lo aa:bb:cc:dd:ee:ff vcan0 --timout 500ms. The expected behavior would be that half the Ethernet frames would contain two CAN frames and the other half only one CAN frame. Here I've encountered some strange behavior, probably because some resources are accessed at the same time from normal program flow and from an interrupt context without using a mutex.
I'll attach wireshark traces or screenshots later.
Regards,
Adriaan

Thanks for your feedback! It help me show an issue by creating dropped messages in my test environment. I demonstrated dropped frames for two cangen's talking at 50ms and 75ms.
I've fixed the dropped messages by reordering the steps in my main loop for packing Ethernet.  But to be clear, the fix was only proven out in my test environment, which, admittedly, is not the same as what you suggested in you note. For example I use candump piped into the talker and I'm using UDP.
Tomorrow I should be able to recreate your exact suggestion.

I tried your setup and the talker program seems to work correctly. (I actually have two VM's talking over a private ethernet network).
The version of talker that I last posted had errors in the loop. That is fixed. It also has a cleaner output (again, just for debugging) that shows how many CAN frames are loaded into each Ethernet frame.
You may like to use:
$ cangen vcan0 -g 500 -e -I f004 -L 8 -D i
It provides incrementing data values, making log files easier to examine for missing frames. (it also is 29-bit CANID's like in J1939)

Thanks! I'm going to test it again tomorrow with your new commits.

Hi, looks good! Everything seems to work now as expected. Are you planning on doing more changes to the code or is the feature now done from your perspective?
Maybe you've notice, in the meantime we did some changes to the main branch in parallel like adding support for CAN-FD so there are some conflicts now. But that's not really a problem, I can resolve these conflicts for you as long as your branch is in a working state. To do that I would branch your branch so that the commit history stays intact.

I’m done with the features. Thanks for asking.
(For what it is worth, I did enjoy digging into the Unix interrupt scheme and understanding how to make the timeout work. I very much appreciate that you had interest in adding these features.)
On Aug 20, 2024, at 4:01 AM, Adriaan Nieß @.***> wrote:
Hi, looks good! Everything seems to work now as expected. Are you planning on doing more changes to the code or is the feature now done from your perspective?
Maybe you've notice, in the meantime we did some changes to the main branch in parallel like adding support for CAN-FD so there are some conflicts now. But that's not really a problem, I can resolve these conflicts for you as long as your branch is in a working state. To do that I would branch your branch so that the commit history stays intact.
—
Reply to this email directly, view it on GitHubhttps://github.com/COVESA/Open1722/pull/32#issuecomment-2298346178, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABEGGWDQF2LV5TQY3JNOFCDZSMAYDAVCNFSM6AAAAABMEQY5L6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJYGM2DMMJXHA.
You are receiving this because you authored the thread.Message ID: @.***>


connecting keycloak to a local DB

Hi every body,
i am new to kubenetes. I have recently tried to connect the keycloak containers ( pods )  to a local DB ( postgres ). But the connection will be refused.
Do you know how can i connect the pod to a postgresql DB on the host ( on the host works every thing fine )?
Thanks alot !

Hi @mazbeh,
I'm closing this pull request because it's not related to the project. As you are new to Kubernetes you need to go through a few tutorial to better understand it. In short: it's tricky to connect to any services running on localhost from Kubernetes. In this situation I highly recommend that you follow the instructions in the README.md to install both Keycloak and PostgreSQL in Kubernetes.

ok, thanks for replying .


Add and integrate an action for USC for setup-snfoundry

Which components does the task require to be changed? (think hard pls)
snforge
Description
Please try using composite actions in order to achieve that

You can use composite actions to integrate the setup-USC action with setup-foundry action https://docs.github.com/en/actions/creating-actions/creating-a-composite-action


Import repositories from broken gitea

Gitea version (or commit ref): cbfc7f5
Git version: 2.11.0
Operating system: Debian Linux
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:
Wiped logs as they were 22 GB in size

Description
Earlier today my gitea server went offline, when i checked at home the server had run out of space, looking in to it; XORM had created a 22 GB log file breaking the database and making the system very unresponsive.
I deleted all the logs and tried to use the database but as stated, it was corrupted.
How can i import the repositories from the old gitea version to the "fresh" version?
Since i've had to remove the database.
I have the gitea-repositories folder.
...
Screenshots

Copy gitea-repositories, data, custom and dump your databases and paste to your new machine.

Can’t dump database, it is corrupted

Git repositories are stored on the file system, not in the database. Still the database is used for displaying information on the web interface.
I had a similar issue a few weeks ago. I moved Gitea to another server and forgot to adjust the database configuration. The old server was shut down and my database was lost.
I still had all my repositories on the file system and restored the database by creating a new repository via the web interface and restored the old repository files. It took me some time to figure out that Gitea uses Git hooks to retrieve the changes from commits and show the files on the web interface.
At least that worked for me, running Gitea in a Docker container...
I went into the folder with the repositories and did this for each repository I wanted to restore:

Backup repository folder: mv my-repository.git my-repository.git-bak
Create a new repository with the same name via the Gitea GUI
Remove the folder that is created: rm -rf my-repository.git
Restore the saved repository: cp -R my-repository.git-bak my-repository.git
Set correct permissions: chown -R git:git my-repository.git
Set execution permissions for Git hooks: chmod -R +x my-repository.git/hooks/*
As of right now you should still see no files on the repository page on Gitea
Clone the repository: git clone git@my-gitea.com:user/my-repository.git
Make a change to the local repository and push it: touch test.txt && git add . && git commit -m "Test" && git push -u origin master
Check the repository page on Gitea. The files should show up.
Revert the last commit by changing “pick” on the second line to “d”: git rebase -i HEAD~2 && git push -f

I've decided to switch back to github for now, though thanks for the suggestion.

I've decided to switch back to github for now, though thanks for the suggestion.

nice.. but that not a solution, that only a workaround... sad


[FLINK-20107][network] Make InputChannel.statePersister immutable

What is the purpose of the change
Make InputChannel.statePersister immutable to

prevent possible errors (forgetting to set the field when converting from UnknownChannel
and visibility problems (write by task thread, read by network thread.

Verifying this change
This change is a trivial rework without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? no

@flinkbot run azure

@flinkbot run azure

In general, I would have liked to make the statePersister immutable from the get-go. However, you also managed to make it immutable only for the Local/RemoteInputChannel - all other channels, DI still happens through setter.
I'm not sure what the value of this intermediate solution is. Now, for some channels DI happens through ctor and for some through the setter.
Certainly the threading model is much more unfortunate with a setter-based DI. But it's not solved by this PR as for all setter-based input channels, the issue still prevails (What happens if I created UnknownChannel in thread1, set channel persister in thread2, and then convert to local channel in thread1?)

I mentioned it in JIRA (and now pulled to PR description):

This change leaves statePersister mutable in channels that will be converted after requesting partitions. There is no race condition here because requesting partitions and conversion is done by the same (task) thread.

So it does solve the threading problem.
Another problem it solves is that UnknownInputChannels weren't aware of statePersister and weren't passing it to "known" channels upon conversion. I didn't enforce setting statePersister on UnknownInputChannels though because it's not relevant for all the cases (e.g. batch).

Thanks for merging!

Known channels are fully initialized including state persister in task thread before being passed to network thread

Before this PR, the fields in known channels weren't final (and there is no explicit fence). So even being fully initialized, the network thread might see them un-initialized


Bluetooth Ear Buds causes eqmac to stop responding

I have two different pair of ear buds and both cause eqmac to stop responding.

+1

+1

Hi, Is this still happening? I have several models of AirPods and they all work fine. As this issue is quite old, I would close it for now. If this bug still happens to you, please reopen this issue and comment, I would be here to help.


Apex chart tooltip not working for internet explorer.

Bug report
Codepen
https://codepen.io/lazyboy1801/pen/xxZgqMG
Apex chart tooltip not working for internet explorer 11.
Thank you

I am having the same problem. The tooltips don't show at all in IE11.  Unfortunately, I am stuck with that browser as it is being used in an embedded browser in a .NET windows form.

I am having the same problem. The tooltips don't show at all in IE11.  Unfortunately, I am stuck with that browser as it is being used in an embedded browser in a .NET windows form.

Just a heads up on this. I followed including the Symbol polyfill, since it seemed to be returning an error related to that issue. On a wild hare, I went ahead and included all of the ES6 Array polyfills from https://polyfill.io/v3/url-builder/ and it started working immediately. I suppose if you have the time, it would be better to remove array polyfills until landing on the ones you actually need, but this should get you started on solving the issue. Good luck.
Here's the URL it generated for me:
https://polyfill.io/v3/polyfill.min.js?features=Symbol%2CArray.prototype.%40%40iterator%2CArray.from%2CArray.isArray%2CArray.of%2CArray.prototype.forEach%2CArray.prototype.flat%2CArray.prototype.keys%2CArray.prototype.flatMap%2CArray.prototype.map%2CArray.prototype.sort%2CArray.prototype.reduce%2CArray.prototype.indexOf%2CArray.prototype.lastIndexOf%2CArray.prototype.includes%2CArray.prototype.findIndex%2CArray.prototype.find%2CArray.prototype.filter%2CArray.prototype.fill%2CArray.prototype.every%2CArray.prototype.entries%2CArray.prototype.copyWithin%2CArray.prototype.values%2CArray.prototype.some%2CArray.prototype.reduceRight
Caveat - you have to open it with the browser you want to polyfill. I tried opening this in Chrome on my Mac and it returned a blank document because my browser didn't actually require any of those polyfills. However when I opened it in IE 11 on the machine my customers would be using, it created every single one of them. I hope this helps to resolving these kinds of unfortunate issues with IE.

Just a heads up on this. I followed including the Symbol polyfill, since it seemed to be returning an error related to that issue. On a wild hare, I went ahead and included all of the ES6 Array polyfills from https://polyfill.io/v3/url-builder/ and it started working immediately. I suppose if you have the time, it would be better to remove array polyfills until landing on the ones you actually need, but this should get you started on solving the issue. Good luck.
Here's the URL it generated for me:
https://polyfill.io/v3/polyfill.min.js?features=Symbol%2CArray.prototype.%40%40iterator%2CArray.from%2CArray.isArray%2CArray.of%2CArray.prototype.forEach%2CArray.prototype.flat%2CArray.prototype.keys%2CArray.prototype.flatMap%2CArray.prototype.map%2CArray.prototype.sort%2CArray.prototype.reduce%2CArray.prototype.indexOf%2CArray.prototype.lastIndexOf%2CArray.prototype.includes%2CArray.prototype.findIndex%2CArray.prototype.find%2CArray.prototype.filter%2CArray.prototype.fill%2CArray.prototype.every%2CArray.prototype.entries%2CArray.prototype.copyWithin%2CArray.prototype.values%2CArray.prototype.some%2CArray.prototype.reduceRight
Caveat - you have to open it with the browser you want to polyfill. I tried opening this in Chrome on my Mac and it returned a blank document because my browser didn't actually require any of those polyfills. However when I opened it in IE 11 on the machine my customers would be using, it created every single one of them. I hope this helps to resolving these kinds of unfortunate issues with IE.


🐞 [BUG] - Benefits section are not curated properly.

Issue Verification

[X] I have verified that this is a new bug report.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Describe the bug
The Benefits section under the Testimonials section is not curated properly.

@jfmartinz Please assign me this issue under the Gssoc label as I will fix it by styling it properly.

Currently the website is still in development (redesigning). Read this #998