Add ability to export Node pointers as NodePaths

This PR implements:

A new hint: PROPERTY_HINT_NODE_TYPE for variant type OBJECT, which can take specific node types as hint string.
The editor will show it as a node path, but will set it as a pointer to a node from the current scene if you select a path.
When scene is saved, the node path is saved, then restored as a pointer.

This is the low level machinery required for doing something like:
@export(Button) var myvar

And be able to edit this as a node path from the inspector, so you can assign another node. This functionality has been very requested, but it was not done due to how difficult it actually is to implement.
NOTE: This is a proof of concept approach. It has the downside that it requires a pointer to be node to be kept during editor. If the node is deleted, it won't crash in GDScript because it checks for it to still be valid via de ObjectID inside of Variant, but this may be harder to avoid in Mono or C++ (which would need to keep the ObjectID around to ensure the pointer is valid and was not deleted).
Potential workarounds: A potential workaround may be for the editor to store this value (the node path), alternatively, in a metadata but this has other complexities and it still has the requirement that its set after the scene is done instantiating (to ensure the node will be found). The downside of this is that this is a far more complex solution.
In all, there does not seem to be an ideal way to implement this so far, so this PR will be left open until we can figure out the best way.

Another way we may be able to implement this in a more clean way is to make this a regular NodePath property that somehow points to another, hidden property. Example:
PropertyInfo(Variant::NODE_PATH,"which_node_path",PROPERTY_HINT_NODE_PATH_TO_NODE_PROPERTY,"which_node:Button");

This means that, during inspector editing, only the node path will be set via the which_node_path property (pointer could be set too, I don't know) but upon saving the scene, the extra special property will be saved (as the current code does) that will result in setting this value to which_node after the scene is loaded.
This script languages would need to expose this as an extra property and, if you want to use this in GDExtension, you have to also use it manually by exposing another property.

For completeness let me point out an existing workaround/hack in 3.x:
export(NodePath) onready var button = get_node(button)

One would "just" have to add a better syntax and static typing to this solution. Maybe it can serve as inspiration.

Another way we may be able to implement this in a more clean way is to make this a regular NodePath property that somehow points to another, hidden property. Example:

Yes I think this is the way.
I've always imageind this feature just as syntax sugar that would back the exported Nodes with a hidden NodePath variable.
eg) User writes:
@export(Button) var myvar

But Godot actually creates this:
@export(NodePath) var __nodepath_myvar
@onready var myvar:Button = get_node(__nodepath_myvar)

@jasonwinterpixel My most recent commit does this transparently now by using the newly implemented metadata system. It will probably need a good amount of testing once @vnen implements it in GDScript 2.0 but I am confident this is probably the simplest way for user (even if it required a bit of hack in the inspector and PackedScene).

Can the syntax be made in such a way so that we preserve actual type hints, not export hints? We just unified the mess we have in 3.x with both export hinted types and proper types.

@YuriSizov I think that is the case, I am just not used to writing modern GDScript so my example is likely wrong.

It has the downside that it requires a pointer to be node to be kept during editor.

I didn't quite understand this. Can you explain it a bit more?

If I understand this correctly, this implementation assigns the node properties when instantiating a packed scene rather than on _ready, right?

Ever since we got unique nodes the need for this with C# has been reduced drastically. It's still a nice feature to have but I just wanted to let you know about the unique nodes perspective of this.

It sets this on PackedScene instantiation, not ready.

Then I think this is much better than the "export NodePath & onready get_node" syntax sugar proposed above. It behaves just like any other exported property.

@neikeq Yes, it aims to be as transparent as possible to the user code, so it can be used from C# or C++ directly without language hacks. Just export a property to a node pointer and give the editor hint, then the editor will manage the nodepath internally and set the pointer on scene instantiation, all transparently.
Doing it from within GDScript would have been much easier, but this I something you want in the other languages too, which is why it took so long to figure out how to do it the best possible way.

I might be missing something, but I just tested this with GDScript in a new project and it's not working. It looks like these lines in the GDScript parser need to be changed to accept Nodes as a valid export type:
https://github.com/godotengine/godot/blob/68bf4eb100a6b49133157b0ecf8c55119b575be8/modules/gdscript/gdscript_parser.cpp#L3603-L3612

@AaronRecord The GDScript part is going to be implemented in a follow-up PR.

Thanks!


Change CommonName to use displayName when upgrading

When upgrading from config version 1.2 to 1.3 check if CommonName is read from Active Directory and the attribute Cn.
If so ask the admin if we should change to displayName.

Solved by #110

Solved by #110


Make bug on MacOS

My environment is MacOSX10.15
I met the following problem:
make[2]: *** No rule to make target `ext/yaml-cpp/src/yaml-cpp-build/libyaml-cpp.so', needed by `librl_engine.dylib'.  Stop.
make[1]: *** [src/CMakeFiles/rl_engine.dir/all] Error 2
make: *** [all] Error 2

I make this project successfully until i met this problem.
If more info is needed, just say it.
Thanks in advance.

I wonder whether it can work if i build the yaml-cpp lib seperately and mannually. I could clone it from github and build seperately but i don't know how to use it directly.

But stangely, i tried it on linux(Ubuntu), i succeeded in that step
[ 83%] Building CXX object src/CMakeFiles/rl_engine.dir/utilities/csv.cpp.o
[ 85%] Linking CXX shared library ../librl_engine.so
[ 85%] Built target rl_engine
Scanning dependencies of target rl_markets
[ 87%] Building CXX object src/CMakeFiles/rl_markets.dir/main.cpp.o
[ 88%] Linking CXX executable ../rl_markets

But when i was going to make it
[ 88%] Built target rl_markets
Scanning dependencies of target do_tests
[ 90%] Building CXX object test/CMakeFiles/do_tests.dir/main.cpp.o
test/CMakeFiles/do_tests.dir/build.make:62: recipe for target 'test/CMakeFiles/do_tests.d\
ir/main.cpp.o' failed
CMakeFiles/Makefile2:314: recipe for target 'test/CMakeFiles/do_tests.dir/all' failed
Makefile:94: recipe for target 'all' failed

very sad....

emmm,after tring it for several times, i make it zzzz.


Still working now?

I have just installed the extension, but didn’t see the data as in screenshot. Is it still working? Current GitHub layout may be different from the one last year.

Yeah the extension is currently in limbo and has not been maintained properly.  This extension will probably be obsoleted in the near future, but I'll keep the code up, since others might learn something from it.


How to disable SIP

I haven't found a way to get the VM into recovery mode so that I can disable SIP. Anyone any idea?
BTW: Great work and thank you, @KhaosT!!!

There is no public way to do this. In order to boot into recoveryOS, you'll need to invoke the private API to pass in the special configuration at the boot time.
Here are the API headers I found. https://gist.github.com/KhaosT/8167170d1452daec1ba292e37e5768c8

Works great, thanks!

Any chance to have a button added on the UI to boot from recovery?

Works great, thanks!

Hi, could you please elaborate how you managed to boot into Recovery in MacVM to disable SIP?


enable logging requests

HTTP.listen has an access_log argument.  not sure why, but passing it into HTTP.Servers.Listener doesn't work.
the other commit in this PR simply fixes the carriage returns, and replaces some tabs with spaces.

Thank you! Any idea what happened to the diff? Looks like only small changes, but the diffs are very large

the diff of the entire PR is large b/c every line of src/HTTPServer/implementation.jl had a ^M at the end, unlike all other files in this repo.  that might have been my fault from a previous PR.  better to look at the diff of the three commits individually.

Ok, thanks! Looks good to me!


Missing file 'get_all_titles_from_spoken_wikipedia.py'

I am currently looking into the building of the training dataset but it seems like the referenced file is nowhere to be found:
https://github.com/bene-ges/nemo_compatible/blob/27bce6d91a12b74f6e4f84f18998df2d80582470/scripts/nlp/en_spellmapper/dataset_preparation/build_training_data.sh#L23
./build_training_data.sh: 25: /nemo_compatible/scripts/nlp/en_spellmapper/dataset_preparation/NeMo/examples/nlp/spellchecking_asr_customization/evaluation/get_all_titles_from_spoken_wikipedia.py: not found

it's in /nemo_compatible/scripts/nlp/en_spellmapper/evaluation/get_all_titles_from_spoken_wikipedia.py
fixed the comment

That's working, thanks.
I am wondering though where the input_folder is supposed to be.
https://github.com/bene-ges/nemo_compatible/blob/45fdcead04a3bd7259e142fbbd4d76836908a45d/scripts/nlp/en_spellmapper/dataset_preparation/build_training_data.sh#L23

oh, it's the spoken wikipedia folder
it should appear after downloading and unzipping spoken_wikipedia, see this
code

Looks like the dataset is no longer available:
WARNING: cannot verify corpora.uni-hamburg.de's certificate, issued by ‘CN=GEANT OV RSA CA 4,O=GEANT Vereniging,C=NL’:
  Issued certificate has expired.
HTTP request sent, awaiting response... 500 Service unavailable (with message)
2023-11-14 11:24:59 ERROR 500: Service unavailable (with message).

Ok, I put spoken_wiki_titles.txt to the repo, should be sufficient for training
if you later need the full spoken_wikipedia for evaluation, and it still is unavailable, tell me, I will upload my copy to huggingface


Molecule should auto install tested collection during dependency step

Issue Type

Feature request

Desired Behavior
When the user creates scenarios inside a collection they also need to install the collection itself before running molecule or the tests would likely fail to access roles and modules from within the collection.
Until now users either manually added ansible-galaxy install the local collection with something like:
    sh -c "rm -rf dist/* .cache/collections/*"
    # build collection
    ansible-galaxy collection build -v -f --output-path dist/
    # install collection in custom location
    sh -c "ansible-galaxy collection install -f dist/*.tar.gz -p .cache/collections"

Clearly this is not something convenient and we want to have a seamless experience where molecule scenarios can be run right away without extra user interventions. The tool can easily detect the presence of a collection repository by cheking presence of the galaxy.yml file.
It should be noted that Molecule itself already knowns to isolated ansible collections and roles paths to scenario specific folders, so installing these should not pollute user folders.
At this moment dependency already knows to install dependencies but does nothing about local collections, the one being subject to the test.
Sadly Ansible does not provide any way to perform editable installs for collections, so we will need to install it each time forced.

Molecule already does this, closing.

@ssbarnea


OS4 Schema: agenda items have tags

I do not understand why agenda items have tags additionally to the tags that motions, topics and assignments already have.

To tag agenda items independently from their content objects.


AttributeError: module 'itk' has no attribute 'PolyLineParametricPath'

I have installed all the dependecies, i'm running this on ubuntu 20.04 LTS, python version 3.7. Below you can find the whole error, I tried this on windows 10 as well, but I have the same error in windows also, the dependencies are as below for both OS
itk version 5.1.0
python version 3.7
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/itkwidgets-0.31.3-py3.7.egg/itkwidgets/trait_types.py in validate(self, obj, value)
    363                         geometry, dict) or 'vtkClass' not in geometry:
--> 364                     geometries[index] = to_geometry(geometry)
    365                 if 'metadata' not in geometries[index]:

~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/itkwidgets-0.31.3-py3.7.egg/itkwidgets/_transform_types.py in to_geometry(geometry_like)
    481         return geometry
--> 482     elif isinstance(geometry_like, itk.PolyLineParametricPath):
    483         vertex_list = geometry_like.GetVertexList()

~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/itk_core-5.1.0.post3-py3.7-linux-x86_64.egg/itkLazy.py in __getattribute__(self, attr)
     47     def __getattribute__(self, attr):
---> 48         value = types.ModuleType.__getattribute__(self, attr)
     49         if value is not_loaded:

AttributeError: module 'itk' has no attribute 'PolyLineParametricPath'

During handling of the above exception, another exception occurred:

TraitError                                Traceback (most recent call last)
<ipython-input-8-d900103ec6b1> in <module>
----> 1 reg.graph_source.view_mesh_eig_vec(eig_vec=eig_vec)

~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/cycpd-0.0.1-py3.7.egg/pyfocusr/graph.py in view_mesh_eig_vec(self, eig_vec)
    267         tmp_mesh = vtk_deep_copy(self.vtk_mesh)
    268         tmp_mesh.GetPointData().SetScalars(numpy_to_vtk(np.ascontiguousarray(self.eig_vecs[:, eig_vec])))
--> 269         plotter = Viewer(geometries=[tmp_mesh]
    270                          )
    271         return plotter

~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/itkwidgets-0.31.3-py3.7.egg/itkwidgets/widget_viewer.py in __init__(self, **kwargs)
    370             kwargs['interpolation'] = False
    371
--> 372         super(Viewer, self).__init__(**kwargs)
    373
    374         if not self.image and not self.label_image:

~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/ipywidgets-8.0.0a0-py3.7.egg/ipywidgets/widgets/widget.py in __init__(self, **kwargs)
    381         """Public constructor"""
    382         self._model_id = kwargs.pop('model_id', None)
--> 383         super().__init__(**kwargs)
    384
    385         Widget._call_widget_constructed(self)

~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/traitlets/traitlets.py in __init__(self, *args, **kwargs)
    995             for key, value in kwargs.items():
    996                 if self.has_trait(key):
--> 997                     setattr(self, key, value)
    998                 else:
    999                     # passthrough args that don't set traits to super

~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/traitlets/traitlets.py in __set__(self, obj, value)
    583             raise TraitError('The "%s" trait is read-only.' % self.name)
    584         else:
--> 585             self.set(obj, value)
    586
    587     def _validate(self, obj, value):

~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/traitlets/traitlets.py in set(self, obj, value)
    557
    558     def set(self, obj, value):
--> 559         new_value = self._validate(obj, value)
    560         try:
    561             old_value = obj._trait_values[self.name]

~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/traitlets/traitlets.py in _validate(self, obj, value)
    589             return value
    590         if hasattr(self, 'validate'):
--> 591             value = self.validate(obj, value)
    592         if obj._cross_validation_lock is False:
    593             value = self._cross_validate(obj, value)

~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/itkwidgets-0.31.3-py3.7.egg/itkwidgets/trait_types.py in validate(self, obj, value)
    367             return geometries
    368         except BaseException:
--> 369             self.error(obj, value)
    370
    371

~/anaconda3/envs/pyfocusr/lib/python3.7/site-packages/traitlets/traitlets.py in error(self, obj, value)
    623             e = "The '%s' trait must be %s, but a value of %r was specified." \
    624                 % (self.name, self.info(), repr_type(value))
--> 625         raise TraitError(e)
    626
    627     def get_metadata(self, key, default=None):

TraitError: The 'geometries' trait of a Viewer instance must be A data structure for rendering geometry in vtk.js consisting of points, verts (vertices), lines, polys (polygons), triangle strips, point data, and cell data., but a value of [(vtkmodules.vtkCommonDataModel.vtkPolyData)0x7f0773d042f0] <class 'list'> was specified.

What code are you running to get this error? It looks like the "geometry" trying to be viewed isn't a geometry. This could be because you are trying to view the geometry before it is created.
I will work on adding a feature that checks if the respective geometry exists before visualizing.

I tried to reproduce the error and never got this one, but noticed a few errors based on dependency versions. I have therefore updated the requirements.txt and added new installation instructions to the readme. These are all pushed to master.
Try installing in a fresh environment using Conda and then try running the example notebook in its original form and let me know if there are any issues.

now its workig on linux environment as well.. thankyou


Add filename_proc option to Factory generator

Add a filename_proc option to the Factory generator for better control over the generated factory filename.
Main motivation for this pull request is the ability to use the singularized model_name in the factory filename. For the model Book I want to be able to call the factory book_factory.rb, while with the suffix option only books_factory.rb can be achieved. I believe the latter filename is not semantically correct, because the file defines the data for a single book.
With the filename_proc option people could of course also add prefixes or suffixes to the filename. Therefore this option could potentially replace the suffix option, although it is also possible to support them both for legacy reasons.

@kreintjes can you rebase against master and push the update? I recently merged a fix to Travis in 8c16ae46e8a26d3415b16e8f086b675ac1d1284a

@joshuaclayton Done!

This has been merged in 5b17b5777b72028b2338c347e333a804d9adca9b


Integrated Comprehensive "Crypto Profit Calculator" Information Section to Maximize User Understanding and Utility

fixed #704

@MastanSayyad

@vanshchauhan21 remove the levels!

@vanshchauhan21 remove the levels!

Ok sir


Use Docker version provided by Travis

It would be nice to be aligned with the rest of the projects in the microservices-demo repo and currently there is a snippet in the .travis.yml file that upgrades Docker to the latest version:
sudo apt-get install -o Dpkg::Options::="--force-confnew" -y docker-engine

But the rest of the services just use the version provided by Travis.

Addressed in #5


Bad UI for liquid failing payment

The flush bar text should be better.
Not sure if there isn't an UX experience that can be better
Proposal: the user is not allowed to make the payment just like the minimum of 100 sats
https://github.com/user-attachments/assets/45c1b143-0f37-437f-98cf-339bbaf2d35c

We do dismiss payments below minimum payment limits & for those within limits but where the user does not have enough balance we hide "Approve" button and display an "Insufficient local balance" error message on the payment dialog.
I will change the error message on flushbar to "Insufficient local balance" but there appears to be an issue with the SDK as the user has enough local balance to pay the invoice.
CC: @ok300

I will change the error message on flushbar to "Insufficient local balance" but there appears to be an issue with the SDK, possibly with fee calculation, as the user has enough local balance to pay the invoice.

From what I see here, the InsufficientFunds error is justified and there's no issue with the SDK:

scanned invoice is for 2500 sats
sender (SDK) has 2533 sats available
sender has to be able to cover the fees

as per our docs ^1, the fees are:

~26 sat + ~14 sat + 0.1% service fee = ~40 sat + 0.1% service fee
these fees are more than the available 33 sats (2533 sat available balance - 2500 sat invoice amount)
therefore, the SDK throws InsufficientFunds

When I make a Liquid to Liquid payment the fees are ~250 sats - for lighting to liquid or reverse I never had a fee less than 42 sats

This case is about how we handle the insufficient funds case - I am intentionally not leaving enough for the fees

Opened an umbrella issue:

https://github.com/breez/misty-breez/issues/181

to better reflect fees & prepareSendPayment errors, which will be handled as part of send payment UI rework #179 .
Closing this by displaying a user friendly error message for now. #178


Page Not Found (404) for Every Page

For almost the past 2 weeks, ALL pages, other than the sign in page, return 404 Page Not Found, regardless of the D365 instance/portal it is pointed to. Nothing has changed on our end but none of our portals deployed via Azure App Service work anymore.
To make matters worse, when I run the code from VS, the same thing happens - 404 for every page, even for those D365 portal instances that are licensed via MS. But again, users are able to login but there is no content.
Any ideas????

Hi,
Is the portal schema and data installed in an online or on-premise instance? If online, how was the schema and data installed, via one of the packages supplied in the one-time release or via the online provisioning?

I have verified this issue occurs when the code is used with the Portals version 9.x solutions which this project is not compatible with.


[React Native][Thirdweb v5] Expo web support?

Running Thirdweb with Expo on web doesn't seem to work, I've been trying with the Expo Starter project, will it be supported in the future?

@ludwig-g-w currently the crypto/wallet libs required a not compatible with Expo web - that said i'd really like to make it work so we'll get to it at some point. Would love some contributions if you're interested!

@ludwig-g-w currently the crypto/wallet libs required a not compatible with Expo web - that said i'd really like to make it work so we'll get to it at some point. Would love some contributions if you're interested!

Ok thank you for your response. For sure, I'm willing to help out if I can :)


report bug in crf

In add_boundary_energy() with mask:
https://github.com/keras-team/keras-contrib/blob/5ffab172661411218e517a50170bb97760ea567b/keras_contrib/layers/crf.py#L401-L405
In this way the right_boundary will never be involved in optimization. Is there a mistake in this calculation of end_mask?
I think the correct calculation should be:
end_mask = K.cast(K.greater(mask, self.shift_left(mask)), K.floatx())

cc @lzfelix

I found the same error


Feature request: obtaining representation vector of a face

I'm looking for the API:
vec = deepface.represent('face.png')

This is already covered in the api: https://github.com/serengil/deepface/blob/master/api/api.py


Declutter the front-page cards

@Linedueo Let's create a task for decluttering the front-page cards.
Originally posted by @SjoenH in https://github.com/equinor/MAD-VSM-WEB/issues/306#issuecomment-993301770

Front page cards needs to be cleaned up to make sure the design is intuitive. We will add a "more" menu next to heart that will include sharing VSM and editing tags. All users will show on the cards (removing the +...) to easier be able to filter on users by clicking on them.

@martabenthem @SjoenH

Figma: https://www.figma.com/file/IkHwmIQrsT0iR34f5R5UnZ/VSM?node-id=2946%3A115423

@Linedueo : Should it include renaming the process?

@knudsvik We can add this to the more menu. It makes sense that someone would like to easily change the title 👍🏻 We do have in mind though to later be able to edit the entire card from the front page- like in github. But for that we need to create a new task.

@SjoenH is this something we can add for this task?

Yes. Should be fine. 👍


Error in line 150

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ X] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

only run in a new environment (after follow instructions)

Any log messages given by the failure

... Python(v4.0)\Read_model\sample_analyze_read.py", line 150, in
from dotenv import find_dotenv, load_dotenv
ModuleNotFoundError: No module named 'dotenv'

Expected/desired behavior

that the file runs :-D

OS and Version?

Windows 10 22H2

Versions

Python 3.9 fails and 3.12 too

Mention any other details that might be useful

Thanks! We'll be in touch soon.

Please execute the command in the command line: pip install python-dotenv
Please refer to：https://pypi.org/project/python-dotenv/


Provide ability to encrypt the file for better security

We're currently not using this util because it doesn't support to encrypt the environment file as it contains sensitive information. Some our clients want to hide the information as much as possible for normal user. For now, we are using a helper class to load environment variables supports encrypt/decrypt the file based on the secret key (i.e. a .dat file stored somewhere that the app should know and read it contents) - using cryptoJS.AES.encrypt(input, key),  cryptoJS.AES.decrypt(encryptedStr,  key)
Hence, I would like to raise this issue to know other approaches and looking for this util can be updated and support that kind of feature.

Would the encryption be in a per field basis, or the full file at all? Where would you store the decription key and how?

@piranna  I would say it will be file encryption since all fields are sensitive information. In relation to the key, as mentioned above, where can provide a configuration property to allow user specify a key or a secret file (i.e. a .dat file).

You can always use git-crypt to encrypt .env within the repo but typically environment variables shouldn't be commited.

Sorry, I think this is out of scope for this library. If plain dotenv supported something like this I might consider it.


XPT2046 input device driver in LVGL 6.x

@amirgon:
First of all, thanks for keeping this repo updated and sorry for opening another issue about the XPT2046 driver!
After some time of absence I pulled the most recent version and found that your code in ussue #21 (to update a label with the current touch position) doesn't work anymore. As in LVGL 5.3, before compiling I had to set flags=0 to avoid SPI_HALF_DUPLEX errors; in contrast to LVGL 5.3, where I can get reasonable coordinates with identical wiring, I'm now only getting "-1:-1" all the time. Has there been any major change in syntax (additional to indev_next -> indev_get_next)? Thanks for your help!

Hi!
I have the same issue and am trying to solve it in #36 , but so far without success.

@gersch07, @sEpt0r Did you try the esp32_ili9341 with v.6 as I suggested on #36 ?
If this example works well, we can use it as a reference and try to eliminate the differences.
I would need your help because I don't have the hardware to test this.

I tried the esp32_ili9341. I am running the demo application and slide show is working, but the screen does not respond to touch.

@amirgon give the mailbox address, I can send a display for you.

slide show is working, but the screen does not respond to touch.

@sEpt0r esp32_ili9341 is maintained and aligned to LittlevGL v6.0.
It is part of the LittlevGL project and is unrelated to Micropython.
Originally I used it as a reference to create the Micropython XPT2046 driver, which is not working for you.
I suggest to first make it work on your setup, both display and touch.
After it works well, we'll know that the problem is related to the Micropython XPT2046 driver and take it from there.
If you are having trouble with esp32_ili9341, you can either post a question on LittlevGL forum, or open an issue on esp32_ili9341.
I think @kisvegabor and @C47D are the maintainers of esp32_ili9341, and they'll be happy to help you out!

Thank you! I did not have time to check this in more detail. I'll continue to test the esp32_ili9341.
My display and touch are worked with the Adafruit_ILI9341 and TFT_ILI9341 libraries.

Hi @sEpt0r,
We did some changes on the esp32_ili9341 example project so now the default kit is the ESP32 Wrover Kit v4.1, please open an issue on that repo if you need any help.

@C47D thanks for the explonation.
I have ESP32 DevKit v1 and I set the defines as described in the Readme, I also set TOUCH_SUPPORT=1.
I want to test other applications and will dig deeper to understand why it doesn't work.

Dear all,
here is a quick update regarding XPT2046: I've just finished testing the "touchscreen calibration" app (TPCAL) and it works perfectly with the pins defined as in the readme. Does this mean we have to use HSPI and VSPI for TFT and touchscreen, respectively?

here is a quick update regarding XPT2046

Thanks for the update @gersch07 !

By the way, is it possible to include the most recent commits from the "lv_examples" repository?

The Micorpython driver is a rewrite of the C driver, so it might not be that easy to merge commits from lv_examples.
The main difference between the C dirver and the Micropython driver is that in Micropython we want everything to be set on runtime, while on C driver you set it on compile time by "defines".
For example, PIN numbers and all configuration is a "define" in the C driver.
Please feel free to send us a PR when you make this work!

Dear all,
sorry for the slight delay in getting back to you! Today I modified the file "modXPT2046.c": basically I removed everything related to "spihost" and added the SPI initialization just like in "modILI9341.c" (I had to turn DMA for XPT2046 off). Following the wiring in the "esp32_ili9341" repository (where everything works), I should therefore reproduce the same conditions. However I was not able to obtain any coordinates; only the interrupt seems to do something (keeping the screen pressed anywhere pauses the continuous output of "-1:-1" in the console).
I'm a bit puzzled now...

@gersch07 When I have some time I'll review the code again.
Another option is to start over from the working C driver and convert it to Micropython driver. Most of the work is about changing the compile-time defines to runtime inputs (the wiring, SPI parameters etc.), but there is nothing "deeper" than that.
Actually, if you want to create an ad-hoc driver with fixed parameters for your own use, you don't even need that. Binding the driver to Micropython is very easy.
As I mentioned before, currently I'm not using the XPT2046 for my projects, so this is not a high priority for me right now - but if you want to try it yourself, I'll do my best to help.

Hi @gersch07 @sEpt0r @timstegeman!
You all expressed interest in XPT2046 touch driver for ESP32.
Just wanted to let you know that I've added XPT2046 driver and tested it with ILI9341, they work well together (even on the same bus).
On current settings they are both connected to the same SPI bus, 40MHZ for the ILI9341 and 5MHZ for XPT2046, everything is configurable through their constructors.
Both drivers are pure micropython drivers.
I found that it was much easier to develop and integrate python drivers than C drivers, and they give good enough performance on ESP32 (~30FPS easily, when the entire screen is refreshed).
Of course, they can be used as a reference if anyone wants to convert them to C and get the maximum performance / minimum power.
They are available on dev-6.1 branch, I've updated the README with a usage example.


Unable to open [object Object]: HTTP 404 attempting to load TileSource

I see this has been reported before but although the issue was closed it was not resolved as far as I could tell.

I've tried the following
tileSources: "../photos/_zoomables/21124/"
403 error
tileSources: "../photos/_zoomables/21124/ImageProperties.xml"
404 error
tileSources: "www.robgray.com/photos/_zoomables/21124/ImageProperties.xml"
404 error
If I just type the fully-qualified URL into the browser I get this
This XML file does not appear to have any style information associated with it. The document tree is shown below.
<IMAGE_PROPERTIES WIDTH="13037" HEIGHT="2926" NUMTILES="828" NUMIMAGES="1" VERSION="1.8" TILESIZE="256"/>

So the file exists and I assume it is in the correct format because I have many of these that used to work with Zoomify before I decided to swap over to Seadragon.
Rob

I think you may need the http://, like so:
tileSources: "http://www.robgray.com/photos/_zoomables/21124/ImageProperties.xml"
Assuming the HTML file is on www.robgray.com, this relative URL should also work:
tileSources: "/photos/_zoomables/21124/ImageProperties.xml"

Neither option helps unfortunately

Same result with both options.
Rob

Oh! I finally paid closer attention to the file at http://www.robgray.com/photos/_zoomables/21124/ImageProperties.xml... it's not in the correct format. I assume it's supposed to be a DZI? This is what a DZI should look like:
<?xml version="1.0" encoding="UTF-8"?><Image TileSize="254" Overlap="1" Format="jpg" xmlns="http://schemas.microsoft.com/deepzoom/2008"><Size Width="1000" Height="1000"/></Image>

Where did you get that file?

A year or two back I had working Zoomify images on the site. All the tiles and this XML were produced by the Zoomify app at the time.
It seems though that DZI is the going thing these days so maybe I should swap over. I would hate to have to redo all the tiles though.

@Graynomad
I haven't looked yet to see what's already there, but if necessary it should be relatively easy to make a tile source that'll work with your images without you having to change the images...

OK I seem to have it working, but only after much Googling.
I looked at the simple example on the SD web site but that points to a DZI file. I downloaded the DZI composer but there are so many options it's hard to figure out what to export. No option exports a DZI file.
Eventually I find a forum post where MS says "Oh didn't you know, the dzc_output.xml file is actually a dzi file.
So I point to that and things seem to work with a new test image.
http://www.robgray.com/test/test-seadragon.php
Now what about reusing all my old tiles? The old folders don't have the SparseImageSceneGraph.xml and scene.xml files, so I deleted them from the new test folder and everything still works. Are they required?
Am I correct in saying that if I create modified versions of dzc_output.xml and modify the old Zoomify folder structure to match that produced by the DZI composer those tiles should work?
OR
I don't mind recreating things, should I export as a single file and forget the tiles?
Rob

Oh, it's Zoomify! We support that format as well, but not in the way you were trying. Take a look at:
http://openseadragon.github.io/examples/tilesource-zoomify/
In your case it might be something like:
OpenSeadragon({
    ...
    tileSources: [{
        type:       "zoomifytileservice",
        width:      15870,
        height:     4829,
        tilesUrl:   "/photos/_zoomables/21124/"
    }]
    ...
});

I'm not sure if the tilesUrl is correct... you know your folder structure better than I.
With an image that big, I think it is better to go with a tiled system... 15000 pixels is a bit much to load as a single image.
I would say give a try to getting it working with Zoomify (as above). If it doesn't work out and you do want to work with DZI, there are a number of options:
http://openseadragon.github.io/examples/creating-zooming-images/
...and yes, you don't need SparseImageSceneGraph.xml and scene.xml... DeepZoomTools has a bunch of features OSD doesn't use.

All working well now. I gave up on using the Zoomify tiles and swapped over to DZ. All good now.
What I would like to do now is launch an image into full screen mode by clicking on another image. Is there some example JS code and an API call to do this?

You should be able to launch the viewer into fullscreen with viewer.setFullScreen(true).

This doesn't work,

which was a quicky to see if anything happened (nothing did).
But anyway I want to invoke the view from another image's onClick() function.
I've Goggled 'till I'm blue in the face and can't find an example of doing this.

Looks like you don't need an onClick() function, I have a working test file, here is the relevant code for future searches.

Right on. That definitely works. I think the reason the other one wasn't working is that the browser requires user interaction before going fullscreen. Calling the fullscreen function from within a click handler satisfies that requirement.


Contribution page of AML documentation broken

In GitLab by @NicolasDenoyelle on May 29, 2019, 11:11
The link format fails, and the page of contribution guidelines isn't here.

In GitLab by @perarnau on May 30, 2019, 10:03
I'm looking at it, and it looks like it's only possible if we convert the contributing file to rst first, and then including it from a dummy page.
The problem is that we want to keep the contributing file at the top of the repo too.

In GitLab by @NicolasDenoyelle on May 30, 2019, 17:07
Maybe you can just put an URL link to the gitlab page?
The contribution guidelines are always the latest guidelines, aren't they?
Therefore we can link to the master branch contribution file.

In GitLab by @perarnau on May 31, 2019, 10:25
That's an easier fix for sure. I'll go ahead an do that.

In GitLab by @perarnau on May 31, 2019, 10:36
mentioned in commit ba48903a32ef4093fcbf5c482ccbe917f8dd1996

In GitLab by @perarnau on May 31, 2019, 10:37
mentioned in merge request !62

In GitLab by @perarnau on Jun 3, 2019, 10:28
closed via merge request !62

In GitLab by @perarnau on Jun 3, 2019, 10:28
mentioned in commit db305dffb728e5c96def9fa08bdacaf08caf2083

In GitLab by @perarnau on Jun 3, 2019, 10:28
closed via commit ba48903a32ef4093fcbf5c482ccbe917f8dd1996

In GitLab by @perarnau on Jul 1, 2019, 12:28
mentioned in commit 13bb4b12c77304dfd8a1a99050ecefe230720d6c

In GitLab by @perarnau on Aug 6, 2019, 13:51
mentioned in commit c7864c50d3718c7bb8e87ab7da9f4e21ef563b03


2024.11.0 HA update

 Component won't start after updating to 2024.11.0
Version of the custom_component
 5.4.1
Configuration

Add your logs here.

Describe the bug
A clear and concise description of what the bug is.
Debug log

Add your logs here.

Logger: homeassistant.setup
Source: setup.py:334
First occurred: 09:18:57 (1 occurrences)
Last logged: 09:18:57
Setup failed for custom integration 'frigate': Unable to import component: No module named 'hass_web_proxy_lib'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/setup.py", line 334, in _async_setup_component
component = await integration.async_get_component()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/loader.py", line 1031, in async_get_component
self._component_future.result()
File "/usr/src/homeassistant/homeassistant/loader.py", line 1011, in async_get_component
comp = await self.hass.async_add_import_executor_job(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/loader.py", line 1071, in _get_component
ComponentProtocol, importlib.import_module(self.pkg_path)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/util/loop.py", line 200, in protected_loop_func
return func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/importlib/init.py", line 90, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1387, in _gcd_import
File "", line 1360, in _find_and_load
File "", line 1331, in _find_and_load_unlocked
File "", line 935, in _load_unlocked
File "", line 995, in exec_module
File "", line 488, in _call_with_frames_removed
File "/config/custom_components/frigate/init.py", line 57, in
from .views import async_setup as views_async_setup
File "/config/custom_components/frigate/views.py", line 11, in
from hass_web_proxy_lib import (
ModuleNotFoundError: No module named 'hass_web_proxy_lib'

Same issue here. I rolled back to 5.4.0 via HACS and things are working again.

Rolling back to 5.4.0 worked for me too :)

Duplicate: https://github.com/blakeblackshear/frigate-hass-integration/issues/762

I'd be  curious if upping/downing your HA instances fixes this issue, please chime in on the other issue linked on whether or not that worked, as dependencies should install automatically.


pick up and follow me across dimensions is a disaster area

If you ask a meecreeps (or several) to "pick up a chest and follow me" and then cross into a new dimension (tested with @aroma1997's mining world) the meecreeps kinda derp out a lot - getting stuck in terrain and dying and stuff. The chests are either randomly deposited in the original dimension, or are lost entirely.

I will check it out

Will be fixed next release. I think I managed to solve all the kinks and weird things


[BUG] React or Web SDK not working with NextJS when building project

Observed behavior
I am trying to use an internal tool that uses the React SDK and Web SDK as dependencies along with the LaunchDarkly client provider. Running the app in the local dev server that comes out of the box with NextJS works as expected, but once I try to build the project I run into this:
TypeError: Cannot read properties of null (reading 'Symbol(@openfeature/web-sdk/api)')
    at io.getInstance ([redacted]/nextjs/.next/server/app/page.js:18:171732)

Weird thing (or maybe it's not weird?) is that the code is literally trying to read from null as it seems that is what globalThis is being set to based on this snippet of built code from the next server within page.js
it = Symbol.for('@openfeature/web-sdk/api'),
          io = class extends null {
            constructor() {
              super('client'),
                (this._events = new o9()),
                (this._defaultProvider = o8),
                (this._createEventEmitter = () => new o9());
            }
            static getInstance() {
              let e = null[it];
              if (e) return e;
              let t = new io();
              return (null[it] = t), t;
            }

Does the web or React SDK support NextJS?
Expected Behavior
No property read error to occur.
Steps to reproduce

Create a package that exports a React component which uses the react SDK's Provider component and hooks.
Import said package into a NextJS application
Try to build the NextJS app

Hey @juanparadox, the React SDK is a work in progress, but @toddbaert is actively working on it. We haven't tested it with NextJS yet but we plan on supporting it.
It looks like the Spotify team was able to get OpenFeature working in NextJS but they're using their own React implementation. Perhaps there's an issue in our React SDK implementation. FYI @toddbaert
What version of NextJS are you using? Did you customize the bundling configuration?

@beeme1mr To clarify, it works fine when imported directly into the project and used as directed, just not when used with a library that uses the OpenFeature SDKs (sorry if that wasn't immediately apparent). I am getting this error with version 13.3.1 and latest (currently at 14.0.3). No bundling configuration has been modified.
@weyert Yes, I am using the app router.

I'm going to try removing the React SDK from the library to see if the build issue goes away. Will update shortly.

Interesting, thanks. The extends null part looks very odd to me. Would you happen to have an example repo available?

@beeme1mr I wish I did, but it's a private repo and the library is also private 😭 If removing the React SDK doesn't solve the issue (I will know shortly), I can try and recreate the NextJS app and a small library repo to provide for replication.

Okay so I fixed the issue and it's existence is due to the web-sdk being built for server side use instead of client side use. I was importing a file like this:
import { OpenFeature } from '@openfeature/web-sdk';
import { LaunchDarklyClientProvider } from '@openfeature/launchdarkly-client-provider';

const launchDarklyClientProvider = new LaunchDarklyClientProvider('1234', {});

function initializeOpenFeature() {
  return OpenFeature.setProviderAndWait(launchDarklyClientProvider);
}

export default initializeOpenFeature;

which was being imported into a NextJS file being used server side. Moving the initialization into a client side component fixed the build issue.
It feels like when implementing the web-sdk into a NextJS app we need to make sure it's used in a client side component (which feels obvious in hindsight), but I'm just not sure how one might try to initialize/set a provider server side for use on the client (not sure if that's even a supported feature anyway).
TLDR: I tried to initialize the provider server side with NextJS (not in a use client component) and it was causing build issues. The issue is not apparent when running the local NextJS dev server, only when building the app.

Glad you were able to figure it out. NextJS definitely starts to blur the lines between the server and the client. We'll have to keep that in mind as we build out our NextJS support.
Since you have it working, I'm going to close this issue.


Hash table added in data structure readme

Heyy shhossain i found that in your data structure readme file , hash table was missing , so i was helping you with some update. Please accept my pull request. If you like my work , then i can also add the rest of the contents.
Thank You.

Merging


Configure timeouts for the distributor

This PR is just the beginning of the task titled "Ensure proper timeouts are configured for each service". At this point, it only implements timeouts for the distributor.
I did not want to advance further to check this is the correct way to implement proper timeouts

This PR is now complete. I implemented the timeouts for the distributor and query-frontend. I will create another PR for the ruler and alertmanager when #6 is merged

I messed up this branch. So, I opened another PR in a separate branch #18. Closing this one.


Add flag to hide TOC header

This update should address https://github.com/ekalinin/github-markdown-toc.go/issues/9.
I wanted to leave the default behavior to include a TOC header, but the
way https://github.com/alecthomas/kingpin#boolean-values work make it
difficult, so I head to create a negated flag.
I wanted to create a header flag, but I was not able to setup a
default value for a boolean flag. I was not able to discern a missing
flag from a negative value.
I thought of calling the flag no-header, but kingpin defines a
--no-* flag by default, so it would have ended up including a
--no-no-header, so I chose --hide-header instead.

Agreed. Thanks!


Removed use of struct tm and fixed extra line highlight bug

The math in the long if condition was refactored as it was causing the bug.
time(0) returns the number of seconds since 01/01/1970 as a time_t, which at least for Macs is just a typedef for long

I also watched the most recent stream and have a better suggestion:
Rely on ctime() and simply parse its output. Starting from offset 11, an exact ASCII string representation of the local time is returned (though you only need the first 8 characters [11,19) to determine the digit). Furthermore, since ord(':') == ord('9') + 1, you can simply subtract by '0' to get the zero-based index of the character, even if it's a colon.
Example code:
time_t t = time(0);
char* a = ctime(&t) + 11;
for (int i = 0; i < 8; ++i) {
  int index = a[i];
  /* do what you need to with index */
}

Here's my playground in cling, a C++ interpreter:
****************** CLING ******************
* Type C++ code and press enter to run it *
*             Type .q to exit             *
*******************************************
[cling]$ #include <time.h>
[cling]$ time_t t = time(0);
[cling]$ char* a = ctime(&t);
[cling]$ a
(char *) "Sat Jun 15 18:41:17 2024
"
[cling]$ a + 11
(char *) "18:41:17 2024
"
[cling]$ a[19]=0;
[cling]$ a + 11
(char *) "18:41:17"


repology having problems with extracting package names

Issue description
https://github.com/repology/repology/issues/854
repology is unable to reliably split our package names to pname and version
Many packages have no explicit pname and version, only name which is tricky to split
Nix's own builtins.parseDrvName uses a simple heuristic which is unreliable

How many packages need manual intervention (in the sense that "automatic tools get a wrong result")?

builtins.parseDrvName isn't really a heuristic per se.  It's the only way that nix-env uses AFAIK, i.e. it's how nix defines version comparison.

How many packages need manual intervention (in the sense that "automatic tools get a wrong result")?

As a human I am not sure how to split for example "wmii-hg-2012-12-09".
Even after looking at the introductory commit titled wmii-hg: init at 2012-12-09 but
https://github.com/NixOS/nixpkgs/blob/c0b710fc68352d672ff1c450010b35d9992154af/pkgs/applications/window-managers/wmii-hg/default.nix#L6
The automated function which would mark the packages with need manual intervention would not be easier than the automated function which do the split.

In such cases, I'd probably "rename" them.  (Regardless of improving the tooling in some ways.)

Also the recommended format for unreleased version version = "unstable-YYYY-MM-DD" is not compatible with builtins.parseDrvName's logic, it will include "unstable" to the pname

The manual says

Also append "unstable" to the name

i.e. it is consistent with the splitting rule.  The rule is actually older than pname attributes in stdenv's mkDerivation.  I'm not really convinced this part is usually a good practice.

Perhaps we should start an RFC to (re)define name, drvname, pname, version, status of words like "unstable" and "hg" (package name suffix or version prefix) ...

in a consistent way
compatible with 3rd party tools (such as repology).

The issue of "using HG/GIT/CVS/wth" in the name was always polemical. About that, I think the "unstable" should be part of version attribute.

I think putting unstable in pname is so that nix-env -u doesn’t automatically upgrade to the newer version. This is confusing though either way. IMO we shouldn’t carry unstable or unreleased versions of the same package at all, instead allow overrides and overlays to provide a more unstable package set.

I think putting unstable in pname is so that nix-env -u doesn’t automatically upgrade to the newer version.

If adding -unstable suffix to the pname is the only way to prevent automatic upgrade, then there is something fundamentally wrong with nix-env
For example, perldevel (technically being unstable) has exactly the same pname "perl" as perl528, will nix-env perform the  automatic upgrade?
Or will it automatically upgrade postgresql 9 to 10 ?

You can mark such packages lowPrio.

BTW, this is the function to split name to pname and version which works only for perl-packages.nix
https://github.com/volth/cpan2nix/blob/68b4e6c000c5321415255e70f57ca9fe9641975e/Cpan2Nix.scala#L310-L338
eight regexps + list of exceptions

IMO we shouldn’t carry unstable or unreleased versions of the same package at all, instead allow overrides and overlays to provide a more unstable package set.

Problem is, sometimes the upstream code doesn't release very often. An "unstable" version includes some valuable modifications, as the TCL_CONFIG environment on OpenMSX

Repology started to publish its logs at
https://repology.org/repositories/updates
The latest log for nixpkgs-unstable is:
https://repology.org/log/1282366
There are a bunch of errors because they cannot get the version of a derivation or cannot extract the version because of multiple hyphens.


Parts Unlimited Refresh Work

Upgrade and test source code from .Net Core RC2 to RTM
Update .NET core base documentation
Port existing HOLs to work with the .Net Core code
VSTS Quick start HOL

Hey Steven,
Thanks for the PR!
I see that Getting Started on Linux and OSX hasn't been updated and is deprecated.
Do you plan on refreshing it?

Hi William:
How are you?
I will need to confirm this with @julienstroheker .
Updating this document is out of scope for us.
Cheers
Steven


in helper_code.py ,NameError: name 'draw_circle' is not defined

NameError: name 'draw_circle' is not defined
where is your draw_circle?
thank you!

@wojiaobarnet I fixed this issue can you double-check it.


bug: streaming TPC-H Q21 outputs empty result

Describe the bug
The MView shows 0 results, while the batch query gives 1 rows.
To Reproduce
After #5292 being merged, just uncomment the Q21 in the e2e test
Expected behavior
No response
Additional context
No response

Looks like there's some issue with anti/semi join. Could you please help to take a look? @yuhao-su


README: Prereqs

In https://github.com/knipknap/better-tool-library?tab=readme-ov-file#prerequisites we see:

Better Tool Library (BTL) is compatible with FreeCAD 0.19, 0.21, and 0.22.

No mention of 1.0.0 ?
Perhaps it's better to state it this way and therefore omitting the need to specify all subsequent versions since 0.19

Better Tool Library (BTL) is compatible with FreeCAD ≥ v0.19

Thanks for pointing it out! It's fixed.


remove partOf from workHeader section

Who is this for?
https://github.com/wellcomecollection/platform/issues/5525
What is it doing for them?
Now that partOf is in the body of an item's layout, with links, the "part of" line in the header is redundant.

Should this be removed from search results too?
https://github.com/wellcomecollection/wellcomecollection.org/blob/dfdb8cc95e6b7abfc02b52da7018aa1dd1068932/catalogue/webapp/components/WorksSearchResult/WorksSearchResult.tsx#L151


OPF Spine doesn't matched with NCX entries

Even though Apple checking the validation for "ERROR_EPUB_NCX_NAVPOINT_CHILD_ORDER" and it's not listed in all the ePub checker versions [Old - Latest].
Example Error message from Apple:
Asset validation error for file XXXXXXXXXXXXX.epub:
ERROR_EPUB_NCX_NAVPOINT_CHILD_ORDER: Found error in NCX file
(OEBPS/toc.ncx): The following entries are invalid in the given order:(Chapter navPoint-4 and navPoint-5 are in a different order than in the  of the OPF-file)

Please provide a sample ePub file.

Sample ePub is uploaded.
Zip package is not uploaded. So I've changed the extension, please change the extension .txt to .zip
Sample1.txt

You'll find that the order of content documents in the spine is not the same as the order in the NCX (contents and copyright are reversed).
Whether this should be an error is somewhat ambiguous, so both interpretations could be considered correct. What OPF says is that playOrder, when used, is supposed to match the linear order of the spine, so there should at least be a warning in this case:

Although playOrder attributes are not used to sequence SMIL content, if present, they should contain valid values that reflect the linear document reading order.

But since OPF 2 doesn't require the playOrder attribute, and doesn't say anything about the implied order of entries in the NCX matching the spine order, it's kind of open for interpretation what is valid or invalid. The DAISY specification has nothing to say on the subject, since playOrder was a required attribute.
EPUB 3 states that the toc nav entries have to follow the spine order and markup order within each document, so you can probably infer that's what OPF intended, too. Apple's chosen to take this approach, it seems.

EPUB 3 states that the toc nav entries have to follow the spine order and markup order within each document, so you can probably infer that's what OPF intended, too. Apple's chosen to take this approach, it seems.

Thanks for having a look, Matt!
From my daily work I can report that errors which appear only in the distribution phase (when sending to Apple, Amazon, etc.) which can't be reported by previous tools are very annoying for users...
So I would suggest to implement this behaviour also in EpubCheck, what do you think?

If you're asking about this specific test, then, yes, I'd agree that it should be in epubcheck. I can't see that having the ncx entries not match the spine order was the intended of removing the playOrder attribute.
In general, though, unless it's a test that can be confirmed by the specifications, I'd say no. Epubcheck should validate to the specifications by default.
If there were a way of including vendor-specific tests for those who want to enable them, that's another issue. It seems like it could be a maintenance headache to get into if you have to make space for anyone who wants to include their own tests, so I leave those kinds of decisions to the epubcheck team.

Hey Matt,
no no, I'm totally with you! I was asking about this specific test!
We shoulnd't include vendor-specific tests, nobody can maintain that...

This is why we created FlightDeck. Retailer-specific requirements are constantly changing, and some require specialized testing tools like Apple Transporter. I don't think it makes sense for EpubCheck to include all of those tests.

Related to #353


Add Explore Academy

https://www.explore.academy/ is the website it is located in ABQ NM

@kiwi334334 explore.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


[1.9] Review metadata keys for all Azure Storage components

What content needs to be created or modified?
In <= 1.8, Azure Storage components use an inconsistent set of metadata keys for the same things. In 1.9 that is being standardized to keys such as accountName, accountKey, etc (old keys are still supported as aliases for backwards-compatibility but are now deprecated).
The docs for these components need to be updated:

bindings.azure.storagequeues
bindings.azure.blobstorage
state.azure.blobstorage
state.azure.tablestorage

Describe the solution you'd like
Standardize the names of metadata keys so values are consistent.
The associated pull request from dapr/dapr, dapr/components-contrib, or other Dapr code repos
https://github.com/dapr/components-contrib/pull/1837

/assign

Resolved


Failed at the<EMAIL_ADDRESS>install script.

Unable to install the node. Raspberry Pi with Raspbian.
See console.log.txt for installation output messages.
2021-10-28T15_37_48_361Z-debug.log
console.log.txt

It's not working anymore with the latest raspi kernel. Not much I can do about it :(
bluetooth-serial-port is not maintained anymore

Installs and works on raspberry pi running ubuntu mate 32 bit 20.04.1 LTS, kernel 5.4.0-1050-raspi, bluetoothd 5.53, node.js 12.22.9, node-red 1.2.9.
I had to force older node-red install with switches:
bash <(curl -sL https://raw.githubusercontent.com/node-red/linux-installers/master/deb/update-nodejs-and-nodered) --nodered-version="1.2.9" nnnode12.
Sorry to hear this node is not maintained further, by far the best and easiest to work with Zebra DS3678 bluetooth scanners set to serial mode.

Hi, had the same issue, I created a pull request that fixes that.
In the meanwhile I'm using my fork

Thank You!


[BUG] Create/Import rule | Detection field not validated

What is the bug?
Users can create/import a rule without validating detection field content
How can one reproduce the bug?
Steps to reproduce the behavior:

Go to the Rules page
Click on "Import rule" or "Create new rule"
Scroll down to the Detection field
Enter any invalid YAML content
Fill in all required field
Click on Create button
There is no warning that the Detection field is invalid and Rule can be created

What is the expected behavior?
There should be a warning saying the Detection field is invalid and the Rule should not be created
What is your host/environment?

OS: Ubuntu
Version 22.10

Fixed as part of PR

Closing as already fixed with PR


Bring excluded specs offline.

Based on issue #9952.
I was able mock the payload except for Scene/GlobeSurfaceTileSpec.js if you have a suggestion on how to un-exclude this this let me know and I'll try to fix it.
The other tests work offline. I captured the payloads from successful transactions on main, and cached them in a Mock directory.

Thank you so much for the pull request @TheMagicNacho! I noticed this is your first pull request and I wanted to say welcome to the Cesium community!
The Pull Request Guidelines is a handy reference for making sure your PR gets accepted quickly, so make sure to skim that.

:x: Missing CONTRIBUTORS.md entry.

Please add yourself to the contributors file!

:heavy_check_mark: Signed CLA found.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.

:x: Missing CONTRIBUTORS.md entry.

Please add yourself to the contributors file!

My name is already in the contributors file.

Thanks @TheMagicNacho. If your name is not being detected in CONTRIBUTORS.md, perhaps you need to update this branch by merging in the latest main branch.

CI is reporting some test failures. These tests all should be passing before we can accept the change. You can run npm run test to run the tests on the command line locally.
Additionally, CI is reporting eslint failures which signal some errors in the code. These will need to be fixed as well. You can run npm run eslint to see the errors locally.

Thanks for the PR @TheMagicNacho. To keep things tidy, I'm going to close this PR due to inactivity. But if you get the chance to take another look at this, feel free to reopen!


Missleading error log when no HTTPRoutes are set

Do you want to request a feature or report a bug?
Bug
What did you do?
Install Traefik 2.4.8 with the Gateway Provider enabled and do not have routes deployed yet.
What did you expect to see?
No errors in the log
What did you see instead?
Errors in the log :)
Output of traefik version: (What version of Traefik are you using?)
2.4.8

What is your environment & configuration (arguments, toml, provider, platform, ...)?
      --global.checknewversion
      --global.sendanonymoususage
      --entryPoints.traefik.address=:9000/tcp
      --entryPoints.web.address=:8000/tcp
      --entryPoints.websecure.address=:8443/tcp
      --api.dashboard=true
      --ping=true
      --providers.kubernetescrd
      --providers.kubernetesingress
      --providers.kubernetesgateway
      --experimental.kubernetesgateway
      --pilot.dashboard=false

If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)
time="2021-03-30T10:02:17Z" level=error msg="an error occurred while creating gateway status: 1 error occurred:\n\t* Cannot fetch HTTPRoutes for namespace \"default\" and matchLabels map[app:traefik]\n\n" providerName=kubernetesgateway gateway=traefik-gateway namespace=default

Closed by #8023.


tracker.getVideoDuration returns 0.0

Hi there,
i am trying to retrieve the video duration
but the function returns 0.0
that's my code
 youTubePlayerView = rootView.findViewById(R.id.lesson_video);
        getLifecycle().addObserver(youTubePlayerView);
        youTubePlayerView.setEnableAutomaticInitialization(false);
        youTubePlayerView.addYouTubePlayerListener(new AbstractYouTubePlayerListener() {
            @Override
            public void onReady(@NotNull YouTubePlayer youTubePlayer) {
                super.onReady(youTubePlayer);
                youTubePlayer.cueVideo("UHHIFb14OiY",0f);
                youTubePlayer.play();

                YouTubePlayerTracker tracker = new YouTubePlayerTracker();
                youTubePlayer.addListener(tracker);
                duration = tracker.getVideoDuration();
                Toast.makeText(getContext(), String.valueOf(duration), Toast.LENGTH_SHORT).show();

            }

            @Override
            public void onVideoLoadedFraction(@NotNull YouTubePlayer youTubePlayer, float loadedFraction) {
                super.onVideoLoadedFraction(youTubePlayer, loadedFraction);
            }
        });

Please Override onVideoDuration method
@Override
            public void onVideoDuration(com.pierfrancescosoffritti.androidyoutubeplayer.core.player.YouTubePlayer youTubePlayer, float duration) {
                super.onVideoDuration(youTubePlayer, duration);
                Log.e(TAG, "onVideoDuration:   "+duration );
            }

You can do as @mmahmoudothman  said.
But this is what's wrong with your code: you should add the tracker before calling cueVideo.

Look at the source code to see why.


player join game error

Timings link
None
Expected behavior
None
Observed/Actual behavior
After the error is reported, the server may be shut down, or it may not be shut down
Steps/models to reproduce
Start the server, join the game, then the following error will be generated, no plugin installed
bat script
@echo off title Server 1.19 "C:\Users\Konsheng\Desktop\Server\jdk-17.0.3+7\bin\java.exe" -Xmx4G -jar purpur-1.19-1647.jar -nogui pause
Purpur version

version
[21:21:26 INFO]: Checking version, please wait...
[21:21:27 INFO]: Current: git-Purpur-1647 (MC: 1.19)*

You are running the latest version

Agreements

[X] I am running the latest version of Purpur available from https://purpurmc.org/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] My version of Minecraft is supported by Purpur.

Other
crash-2022-06-11_21.18.50-server.txt
latest.log
2022-06-11-15.log.gz

Known Pufferfish issue. We'll get the fix merged from upstream as soon as they fix it.


Does this trigger if HDMI or input changes

Does this trigger if HDMI disconnects or is changed? It isn't triggering for me and want to know if I'm doing something wrong (web UWP app).
            var hdmiDisplayInformation = Windows.Graphics.Display.Core.HdmiDisplayInformation.getForCurrentView();
            hdmiDisplayInformation.addEventListener("displayModesChanged", onDisplayModesChanged);

onDisplayModesChanged doesn't seem to be triggering for me. Am I getting hdmiDisplayInformation correctly? onDisplayModesChanged isn't triggering for me. Also, is there a way in JS to view the stringified object? I tried stringifying but the object is blank (Using JSON.stringify).
Thanks!

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

ID: e0118b1a-6521-e0e6-e3c9-8bc5a1b7d859
Version Independent ID: 6a7dc1c2-8bee-471e-1111-db97b8c1ef7a
Content: Core.DisplayModesChanged Event (Windows.Graphics.Display.Core) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.graphics.display.core.hdmidisplayinformation.yml&version=GBlive&_a=contents
Service: unspecified
GitHub Login: @wdg-dev-content
Microsoft Alias: wdg-dev-content

Note that this app is deployed to xbox and we were changing inputs on the TV and disconnecting/reconnecting the HDMI cable

Any update on this? It is relatively time-sensitive

@joannaleecy Any update on this issue?

Answered in UWP forum


Feat(eos_cli_config_gen): add SNMPv3 hashed user passphrases support

Change Summary
As of now the eos_cli_config_gen module only support the following syntax:
snmp-server user <USER> <GROUP> v3 [auth <AUTH_ALGORITHM> <CLEARTEXT_AUTH_PASSPHRASE> [priv <PRIV_ALGORITHM> <CLEARTEXT_PRIV_PASSPHRASE>]]

(though the documentation indicates "encrypted" passphrases for auth and priv, this is not accurate as the passphrases will be rehashed with the local engineID when configured on the switch.)
This feature request intends to allow the configuration of:
snmp-server user <USER> <GROUP> v3 localized <ENGINE-ID> [auth <AUTH_ALGORITHM> <HASHED_AUTH_PASSPHRASE> [priv <PRIV_ALGORITHM> <HASHED_PRIV_PASSPHRASE>]]

Related Issue(s)
Fixes #1720
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes
Modification of the snmp-settings.j2 template to:

Add the support for local engineID (remote can be added in the future as it is not needed for this feature)
Add the capability to detect the presence of the localized keyword that indicate that the engineID is

Documentation template has been updated to indicate the configured Local Engine ID.
README.md updated
CAVEATS:

if the snmp_server.engineid.local is not configured the current behavior (which can be changed) is that the template will revert to non localized syntax (the one is use currently in the repo). However passing hashed passphrases to the non localized syntax will eventually fail as the passphrases will be rehashed with the device engineID (the default one if none is configured) at configuration time. This faulty behavior is already allowed in the current AVD version - and the passphrases in this version MUST be passed in cleartext (contrary to what the documentation indicates).

How to test
Run molecule
Checklist
User Checklist

N/A

Repository Checklist

[X] My code has been rebased from devel before I start
[X] I have read the CONTRIBUTING document.
[X] My change requires a change to the documentation and documentation have been updated accordingly.
[X] I have updated molecule CI testing accordingly. (check the box if not applicable)

So the changes are changing some outputs in some molecule:

eos_cli_config_gen_v4.0
evpn_underlay_ebgp_overlay_ebgp

this is because of the change in documentation - is the process to regenerate all these molecules as well?

Yes, we need to update molecule on all scenarios.

Yes, we need to update molecule on all scenarios.

this has already been done in the latest commit


For code review

It seems that the unit test configuration is not right.

I think you can try this code snippet in your package.json file:
"scripts": {
    "test": "jest --testEnvironment jsdom --testPattern='*.test.js'",
}

kindly request that check the configuration and also try to install the @babel/preset-den i hope the importing will work after that


Blank Page When Navigating from view to another

When my app launches and I try to navigate to a page from the MainView, I get a blank page. I am using an empty <controls:InteractiveContainer /> in the MainView and loading from the code-behind OnLoaded.
Code:
public static class NavigationService
{
    public static async Task<bool> Navigate<T>(bool preventBack = false, double delay = 0) where T : UserControl, new()
    {
        await Task.Delay((int)(delay * 1000));

        UserControl page = new T();
        MobileNavigation.Push(page, preventBack);
        return true;
    }
}

public partial class MainView : UserControl
{
    public MainView()
    {
        InitializeComponent();
        Loaded += OnLoaded;
    }

    private void OnLoaded(object? sender, RoutedEventArgs e)
    {
        _ = NavigationService.Navigate<LoginScreen>(true);
    }
}

public partial class LoginScreen : UserControl
{
    public LoginScreen()
    {
        DataContext = new LoginScreenModel();
        InitializeComponent();
    }
}

Even if I use
<controls:InteractiveContainer>
	<views:LoginScreen />
</controls:InteractiveContainer>

inside MainView and try to load another page from LoginScreen, I get an error:
System.NullReferenceException: 'Object reference not set to an instance of an object.'

on MobileNavigation.Push(page, preventBack); from NavigationService with details:
System.NullReferenceException
  HResult=0x80004003
  Message=Object reference not set to an instance of an object.
  Source=Avalonia.Base
  StackTrace:
   at Avalonia.VisualTree.VisualExtensions.<GetVisualDescendants>d__13.MoveNext()

But it navigates to HomeScreen only after 3 seconds if I put a delay of 3, but not less. NavigationService.Navigate<HomeScreen>(true, 3); else it shows a blank page.
public partial class LoginScreenModel : ViewModelBase
{
    [ObservableProperty]
    public string _username;
    [ObservableProperty]
    public string _password;
    [ObservableProperty] private bool _isLoggingIn;

    private readonly HttpClient _httpClient = new();
    private readonly IPreferences _preferences;

    public LoginScreenModel()
    {
        _preferences = App.Current.Services.GetService<Preferences>();
        OnScreenLoaded();
    }

    public async void OnScreenLoaded()
    {
        if (DateTime.TryParse(_preferences.Get("authExpiration", ""), out DateTime expireDate) && DateTime.Now < expireDate)
        {
            IsLoggingIn = true;
            IsLoggingIn = !await NavigationService.Navigate<HomeScreen>(true);
        }
    }
}

Is it platform specific ?
I had something similar on android one day but never on desktop.

Is it platform specific ?
I had something similar on android one day but never on desktop.

it's on windows only.
I just tested on android and got this and and debugging stops while trying to load the HomeScreen from LoginScreen without the 3 seconds delay
System.Exception
  Message=You are trying to use a InteractiveContainer functionality without declaring one !

I have tested with your code
public partial class MainView : UserControl
{
    public MainView()
    {
        InitializeComponent();
        Loaded += OnLoaded;
    }

    private void OnLoaded(object? sender,RoutedEventArgs e)
    {
        _ = NavigationService.Navigate<MenuControl>(true, 0);

    }

}

public static class NavigationService
{
    public static async Task<bool> Navigate<T>(bool preventBack = false, double delay = 0) where T : UserControl, new()
    {
        await Task.Delay((int)(delay * 1000));

        UserControl page = new T();
        MobileNavigation.Push(page, preventBack);
        return true;
    }
}

But I can't reproduce the bug at all on windows, neither on android emulator. However I know that this problem happened few times on my real android device so I know there is something to find.
I update the nuget package to 11.1 beta, maybe it did something actually. I will retry later when I will be able to debug with my android device.

Maybe it's a problem on my end. I've also encountered issues with MAUI navigation. In the end, I switched to Flutter.


page.waitForNavigation never resolves

Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.8.0
Platform / OS version: MacOSX High Sierra
URLs (if applicable):
Node.js version: 10.0.0

What steps will reproduce the problem?
const http = require('http')
const puppeteer = require('puppeteer')
const PORT = 3000

/**
 * Dummy HTTP Server
 */
http.createServer((req, res) => {
  if (req.url === '/there') {
  	console.log('Responded')
    res.writeHead(200, { 'content-type': 'text/plain' })
    res.end('reached there')
  } else {
    res.writeHead(200, { 'content-type': 'text/html' })
    res.end(`
      <a href="/there"> Redirect </a>
    `)
  }
}).listen(PORT)

/**
 * Puppeteer script
 */
async function run () {
	const browser = await puppeteer.launch()
	const page = await browser.newPage()
	await page.goto('http://localhost:3000')
	await page.click('a')
	await page.waitForNavigation()

	const text = await page.evaluate(() => document.body.innerText)
	console.log(text)
}

run().then(console.log).catch(console.log)

What is the expected result?
To see the text on console
What happens instead?
page.waitForNavigation is stuck and then times out after 30 seconds

Or you can use this way:
await Promise.all([page.click('a'), page.waitForNavigation()])

Thanks, and that does work
Is there any better way to avoid this race condition, without wrapping all calls with redirection inside Promise.all?
Also I believe, the race condition will be true for other waitFor functions as-well? For example: page.waitForSelector

Also I believe, the race condition will be true for other waitFor functions as-well? For example: page.waitForSelector

The waitForSelector is about page's state rather then some page events, so there's no race condition there.
The reason waitForNavigation is racy is because there's no direct connection between page.click and the navigation it causes.

Is there any better way to avoid this race condition, without wrapping all calls with redirection inside Promise.all?

This race condition is inherent to the scenario; the Promise.all approach is just a nice trick to handle this kind of race conditions. You can avoid using Promise.all and waitForNavigation using waitForSelector.
Hope this helps.

how is there a race condition. would love some explanation.

nvm https://github.com/GoogleChrome/puppeteer/issues/1412#issuecomment-345482273

What to do if it's unknown whether click will result in navigation?  Ideally any started network requests would be detected and if there are none waitForNavigation would return immediately

not saying this is ideal solution but you could wrap await in try and catch and if your code is in catch that's running meaning it timed out so you can take steps from there.

I cannot get this to work
            await Promise.all([thePage.click(searchButton), thePage.waitForNavigation()])
            // await thePage.waitForNavigation({ waitUntil: 'load' })
            // thePage.click(searchButton)
            // await navigationPromise

going to try waitForSelector

do this
let wait = thePage.waitForNavigation();
await thePage.click(searchButton);
await wait;

@vsemozhetbyt Unfortunately that code also results in race conditions. Promise.all does not guaranty that the order of the array of promises is also the order of execution. So the waitForNavigation can be executed after the click and still result in a timeout. Learned it the hard way. :)

In my specific case I didn't get to resolve the issue using solution suggested in this thread. However I have came up with another approach that worked well for my case:
let requests = 0;
let timestamp = Date.now();

await page.setRequestInterception(true);

page.on('request', (event) => {
  event.continue();

  requests++;
});

page.on('requestfinished', (event) => {
  requests--;
  timestamp = Date.now();
});

page.on('requestfailed', (event) => {
  requests--;
  timestamp = Date.now();
});

// I haven't tested lines below, more investigation is required as we don't want to have reduce number of requests twice per one request
page.on('requestservedfromcache', (event) => {
  requests--;
  timestamp = Date.now();
});

async function allRequestsToComplete(minDelay = 250, maxDelay = 500) {
  do {
    await new Promise(resolve => setTimeout(resolve, Math.max(minDelay, maxDelay - (Date.now() - timestamp))));
  } while (0 < requests);
}

await yourElement.click();
await allRequestsToComplete(); // Works like a charm

@lu4 thanks for sharing this! As you said works like a charm

Also I believe, the race condition will be true for other waitFor functions as-well? For example: page.waitForSelector

The waitForSelector is about page's state rather then some page events, so there's no race condition there.
The reason waitForNavigation is racy is because there's no direct connection between page.click and the navigation it causes.

Is there any better way to avoid this race condition, without wrapping all calls with redirection inside Promise.all?

This race condition is inherent to the scenario; the Promise.all approach is just a nice trick to handle this kind of race conditions. You can avoid using Promise.all and waitForNavigation using waitForSelector.
Hope this helps.

I have the similar issue here. The purpose of the code is to generate pdf file from html by either passing html or url.
export const html2pdf = async (html = '', url = '', format = 'Legal') => {
let myBrowser = await getBrowser();
const page = await myBrowser.newPage();
const loaded = page.waitForNavigation({
waitUntil: 'load'
});
if (html) {
await page.setContent(html, { waitUntil: 'networkidle0' });
} else if (url) {
await page.goto(url, { waitUntil: 'networkidle0' });
}
// await page.setViewport({ isMobile: true, width: 375, height: 667 });
await loaded;
const pdf = await page.pdf({ format: format });
await page.close();
await myBrowser.close();
return pdf;
};
Is there a race condition? how do you resolve it if there is?
Thanks


v1.0.0-rc2 throws error on React Native Expo

Error message
addEventListener and attachEvent are unavailable

Other info
I was working on updating deps and adding deps.edn support for projects created with this create kit.
v1.0.0-rc2 throws the error above when the app first starts. v0.12.0 works fine.

Thanks for the report @jgoodhcg, you've indeed found a bug 🐛 👍
This error is occurring here in the google closure library due to my unconditional use of the goog events api here.
The fix will be to first check that the environment (e.g. react native expo) supports the event APIs prior to calling it. We'll get this done in 1.0.0-rc3.

This might be fixed by #615 so I have released 1.0.0-rc3 ad left this issue open for feedback @jgoodhcg

It looks like there is still a problem.

The problem stills on 1.0.0-rc5

In 1.0.0-rc6 we have added a try/catch that ignores the exception on environments that do not support addEventListener via the goog.events/listen API (e.g. React Native).
This fixes the issue but does mean that in React Native :loaded? in settings will never be set to true, so you'll continue to get superfluous duplicate event handler warnings when hot reloading code (e.g. via shadow-cljs, figwheel etc).
If you have any suggestions on the following please let us know @neural-works @jgoodhcg

What is a good way for re-frame to detect that it is in the React Native environment ?
Is there an alternative in React Native to goog.events/listen on js/self (i.e. the global object, usually js/window) to listen for the "load" event ? Or in other words, to execute a fn when all scripts have loaded ?

Reopened awaiting response to above. Any feedback on above two questions @neural-works @jgoodhcg on how to detect being in React Native ?

I have not had the to develop a library that runs on any platform but a quick google search found this option for detecting environment.
I've not seen anything for executing a function when scripts load.

Updated to 1.1.1 today and it seems to be working :smile:. Thanks!


Clang format linter

Trying to get a linter to make formatting consistently easier. Starting with clang-format, but if there are other suggestions I'm open to that.
This adds a .clang-format in firmware/ec in the first commit, and then applies it to all .c and .h files.
Not sure if this is what we want, but it is based on the linux kernel .clang-format file with (I think) all differences modified based on coding_style_guide.
My plan is to include the linter in the Jenkins pipeline, so that we can see on each PR what does/doesn't match coding style.

@mdlewisfb can you also add clang execution as Makefile tag?

Just for reference, here is the example code provided in the coding guidelines:
Example.txt
and here is how the same code would look after running clangFormat with the current config file: ExampleClang.txt I would have attached as .c and .h files, but GitHub doesn't allow that.
Would it be helpful to have these (as ExampleModule.h/.c) checked into the repo under an "examples" directory or something (regardless of this specific clang-format stuff) so that there can be an easy reference point if guidelines are ever modified in the future?


Convert Solana Ed25519 Keypairs for Use in TweetNaCl Box Encryption

Solana utilizes Ed25519 keypairs for its signing mechanism. While these keypairs are useful for signing and verifying transactions, they are not directly compatible with the TweetNaCl.js library for asymmetric encryption (box) due to differences in the underlying cryptographic schemes.
By leveraging the ed2curve-js library, we can convert the Ed25519 keys to Curve25519 keys, which are compatible with the TweetNaCl.js box encryption functions and made for Diffie Helmann exchange. This would allow wallets to utilize their Solana keypairs for encryption and decryption without needing to generate separate encryption keys.
If adding an additonal dependency causes a problem to a wallet's maintainers, ed2curve conversion can also be done using only tweetnacl.
Please review the changes and let me know if there are any concerns or suggestions for improvement.
c/ @jordansexton

Alternatively, noble has native Ed25519 https://github.com/paulmillr/noble-ed25519


view update - export to xlsx - first gantt compone

added support sap.m.table export to xlsx
new gantt chart components added (work in progress for the demo)

Thank you!


Fixed a couple of mistakes related to Lightrun

VSCode is supported. It can also be self hosted on-premise.
I didn't touch "Supported libraries" but it's unclear as a section.

Thanks for the contribution. I reverted the on-premise change back to the phone icon. I use it to indicate that it's available but you need to get in contact with the company to get it. Looking at the website, that appears to still be the case.
As for the supported libraries, I'll probably be changing them soon. The idea was to show how many frameworks are automatically observed via the technology (https://github.com/apache/skywalking-java/blob/main/docs/en/setup/service-agent/java-agent/Supported-list.md). For example, using Source++ on a project with a PostgreSQL driver it will automatically monitor SQL queries and whatnot.


feat: support cast_precision for RawBlock

Part of TD-29125

Codecov Report
Attention: Patch coverage is 15.78947% with 32 lines in your changes are missing coverage. Please review.

Project coverage is 69.46%. Comparing base (226bad9) to head (82b3300).

Files
Patch %
Lines

taos-query/src/common/raw/mod.rs
0.00%
13 Missing :warning:

taos-query/src/common/timestamp.rs
38.46%
8 Missing :warning:

taos-query/src/common/raw/views/timestamp_view.rs
0.00%
6 Missing :warning:

taos-query/src/common/raw/views/mod.rs
16.66%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #317      +/-   ##
==========================================
- Coverage   69.52%   69.46%   -0.07%
==========================================
  Files         115      115
  Lines       35764    35798      +34
==========================================
+ Hits        24865    24867       +2
- Misses      10899    10931      +32

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Provide training scripts

Hi, I was going through recent denoising papers and found yours.. I see you've provided pretrained model on BSDS. Could you provide training and evaluation scripts for inpainting? Thanks.

Right. You can definitely train a new model yourself on a new dataset. @gji1 can you shed more light on specific advice for training... maybe including some code?
AFAIK, the recommended high-level procedure is:

Extract lots of 8x8 patches from your dataset
Use our BNPy package (github.com/bnpy/bnpy) to fit a DP mixture model with zero-mean full covariance Gaussian likelihood (obsmodel = ZeroMeanGauss). Use birth moves / merge moves to make sure the model escapes local optima and finds good data-driven structure.
If you want a more powerful model, use the DP model from step 2 to initialize an HDP model and then fine-tune that fancier HDP model.
Apply trained model (HDP or DP) to inpainting, using code like in demo.py

Thanks, I'll try it. Does Inpainting task take such considerable time(1.30 h in my case) for each of the testing images? Also I see boundary pixels can't be restored. Is that the limitation of the algo or implementation?

Thanks for your interest, George. In our implementation the inpainting task generally takes more time to run than the denoising task because it requires more iterations. That said, you are also very welcome tune the annealing schedule to better fit the specific data you have for better performance or faster speed. The settings we use right now are directly adopted from the public EPLL code so as to make a fair comparison with that method.
Test image doesn't have to be from BSDS. Actually the example image we showed in our paper is not from BSDS either. In theory, boundary pixels can also be restored using our algorithm. But the current implementation doesn't support that yet.

Hey,
So I'm trying to understand this paper and https://arxiv.org/abs/1209.5019
for my presentation at uni. I want to try to train the HDP inpainting model
of my own. Do I have to train a DP model first and then use it to
initialize the HDP model? Or I can just directly train HDP model?
Is it this birth and merge script of BNPy:
https://bnpy.readthedocs.io/en/latest/examples/04_bars_one_per_doc/run-05-demo=topic_model_vb+births-model=hdp_topic+mult.html#sphx-glr-examples-04-bars-one-per-doc-run-05-demo-topic-model-vb-births-model-hdp-topic-mult-py
Or this merge one:
https://bnpy.readthedocs.io/en/latest/examples/04_bars_one_per_doc/run-04-demo=topic_model_vb+merges-model=hdp_topic+mult.html#sphx-glr-examples-04-bars-one-per-doc-run-04-demo-topic-model-vb-merges-model-hdp-topic-mult-py
Or just HDP-HMM:
python -m bnpy.Run /path/to/dataset.csv HDPHMM DiagGauss memoVB --K 8
Or any other scripts as in how do I use bnpy? And also,if I want to find
PSNR and SSIM values for inpainting of this paper and EPLL, how can I do
that?
Thank you for all your help.
With Regards,
George.
On Tue, Apr 24, 2018 at 3:49 AM, Geng Ji<EMAIL_ADDRESS>wrote:

Thanks for your interest, George. In our implementation the inpainting
task generally takes more time to run than the denoising task because it
requires more iterations. That said, you are also very welcome tune the
annealing schedule to better fit the specific data you have for better
performance or faster speed. The settings we use right now are directly
adopted from the public EPLL code so as to make a fair comparison with that
method.
Test image doesn't have to be from BSDS. Actually the example image we
showed in our paper is not from BSDS either. In theory, boundary pixels can
also be restored using our algorithm. But the current implementation
doesn't support that yet.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bnpy/hdp-grid-image-restoration/issues/2#issuecomment-383741105,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkqehBOcgymfjdjHWuhE-jZsCfjjAOjtks5trlNegaJpZM4TYJxM
.

You don't have to train a DP model first. We did that mainly because we found this would provide a better initialization. Instead, to directly start from HDP, here's the backbone script you need to build things upon:
python -m bnpy.Run /path/to/dataset HDPMixtureModel ZeroMeanGauss memoVB --K 10 --nLap 10 --nBatch 10 --moves birth,merge,shuffle
Depending on the type of your data, for best performance you may need to tune and add the optional parameters, as in the examples you mentioned. The default settings are stored in folder bnpy/bnpy/config. I suggest that you try our trained model first if your data is also just natural images.

Hey, so I'm diving into the paper and you've written that you trained the model on 400 training and validation images from BSDS. But I see there are only 200 training and 100 validation images and the other 200 are testing images in BSDS-500. Which ones did you train on? Also what values did you choose for various hyperparameters like concentration parameter(alpha)? It'd be very helpful if you could list them out with the values you chose.
Thanks :)


[Multi Actions] Feature flag and/or command to "switch" a workspace to multi-actions

"Switching" a workspace means all their assistants start behaving as multi-actions assistants. Their "single action" builder UI is no longer accessible.
This has to be entirely reversible.

Transitioning workspaces from single action world to multi action world
We won't keep the single action builder. Once you switch, you switch.
All your assistants, including existing ones, start behaving as multi action agents.
The migration is entirely reversible (we keep your forceUseAtIteration  settings, so if we switch you back, all assistants that can be used as legacy agents will start behaving that way again)

Meditating for a bit

Ok with your plan to kill it with fire and have the flag drive the code path we use.
Agreed we need to rename to maxSteps
One thing we could consider is replacing the forceUseAtIteration by a simple boolean forceUse which would force the use at Step 1. But I don't think it's worth it if we believe the agentic route to be the future here, and I think we do.

One thing we could consider is replacing the forceUseAtIteration by a simple boolean forceUse which would force the use at Step 1. But I don't think it's worth it if we believe the agentic route to be the future here, and I think we do.

I would be in favor of dropping for simplicity. I think we'll be fine, but we'll try on our own workspace first. If it turns out to not be fine, then we can re-introduce as a bool easily before shipping

Tots


Higher current HP than max HP on some pals with the added scaling data causes pals to sleep forever

I don't know if it's just on my file but upon testing with the added scaling data breaks a lot of Pals showing higher current hp than their max hp, said pals also won't do any work and sleep forever once they do so when night time comes and will do so until their hp is lowered. After lowering their hp through any means such as combat, using the PalEdit again will break their current hp once more.

This should be fixed now, I believe


Sketch-Measure is not working in Sketch 52 or 52.1

Not working for me either. Is anyone working on a fix?
Can anyone suggest another plugin that has similar functionality?

Temp fix available on my fork

I gave it a try to fix it here:
Fix for Sketch v52 and v52.1 - With Inspector / Mark Property (and more) working
You can get the file from the fork. Give it a try and see if it works for you:
Repository
Download the zip file from the fork
I hope it helps someone out there :)

Please note, that there are now 11 Issues that basically say the same thing: Sketch Measure is broken with the latest releases of Sketch.
Posting yet another issue saying exactly the same thing IS NOT HELPFUL. You can simply add your own experience, or frustration, by leaving a comment in one of the many existing issues.
These issues are #468, #469, #472, #473, #475, #476, #477, #479, #481, #482 and this one.
Sketch Measure developers: please review the fix from @ellunium, which might enable you to close these 11 issues at once.


feat(jobs/pingcap-inc/enterprise-extension): add plugin job

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

@purelind thanks, they are solved.


Added Temperature, Humidity & Contact Sensor Post/Call Back to SmartThings

I added Temperature, Humidity & Contact Sensor Post Back to SmartThings.  Modifying the SmartApp was so much simpler than any other solution. 10 lines of code but world of difference with these 3 capabilities and their read-only limitations in SmartThings.
Ultimately, being able to post these back to ST were big missing pieces for me. @stjohnjohnson I'd appreciate my enhancement pull becoming part of your release and respect your work very much! As an FYI, most of my code is a subset of pull #194 + a few more lines. However, the pull is just more narrow is scope and thus less code to verify. @gandazgul thanks for that pull.

Thanks for this, I needed it!!!   I had to make a small change, not sure if it is a bug or not, but the code assumes the temperature is an int and at line 775 in the smartapp  you have 'device.setTemperature(value as int),   I simply replaced this with 'device.setTemperature(value)'
Thanks again