We need releases for arm32 architectures! ✊

Please!!!

Please try compiling it for armv7a by yourself.
if you encounter errors, refer to the corresponding Android.bp file.

Please try compiling it for armv7a by yourself. if you encounter errors, refer to the corresponding Android.bp file.

I don't have a PC (so I can't use Android NDK😒) to build it for my armv7a architecture device. And the NDK you provide for phones are only for aarch64, that's also unsupported by my arch.😒

Please try compiling it for armv7a by yourself. if you encounter errors, refer to the corresponding Android.bp file.

I don't have a PC (so I can't use Android NDK😒) to build it for my armv7a architecture device. And the NDK you provide for phones are only for aarch64, that's also unsupported by my arch.😒

I guess you may need aapt2 to compile the Android App.

Please try compiling it for armv7a by yourself. if you encounter errors, refer to the corresponding Android.bp file.

I don't have a PC (so I can't use Android NDK😒) to build it for my armv7a architecture device. And the NDK you provide for phones are only for aarch64, that's also unsupported by my arch.😒

I guess you may need aapt2 to compile the Android App.

Yes, I don't need the NDK, I only need the build tools (like aapt, aapt2 etc..). But building these tools for my arch from a PC doesn't need the NDK? I told about that. I don'

I have compiled it for arm architecture, but I haven’t tested it, because I don’t have an armv7a device, you can try it, download from release.

I have compiled it for arm architecture, but I haven’t tested it, because I don’t have an armv7a device, you can try it, download from release.

Btw when did you add arm32 releases??! I didn't see it before so I opened this issue 🥴

I have compiled it for arm architecture, but I haven’t tested it, because I don’t have an armv7a device, you can try it, download from release.

Btw when did you add arm32 releases??! I didn't see it before so I opened this issue 🥴

(sdk-tools-static-arm.zip)[https://github.com/Lzhiyong/sdk-tools/releases]


chore: made styling colour for inspector consistent throughout extension

Current dependencies on/for this PR:

main

PR #143   👈

This comment was auto-generated by Graphite.


Remove some spec/support files

Closes #461

@TiuTalk @marcioj could you plz review?

@maurogeorge It seems to be ok! Just waiting for new build to merge it!

@rodrigopinto thanks for the review :star:

@maurogeorge It was on my plans, I've did it yesterday on a project. :( Thanks for PR.

Burn these files with :fire:


Fix bootstrap teardown

fixes Platform() in clustermetadata.go to return powervs.Name when deploying powervs. Bootstrap teardown seems to succeed with this.

Deployment is still running but it's getting further than it does without it. The installer says teardown was successful but haven't confirmed yet.

Deployment is still running but it's getting further than it does without it. The installer says teardown was successful but haven't confirmed yet.

Have confirmed that COS and bootstrap instance are destroyed


Design "mobile" search page

Design a wireframe for mobile search page

Aviel and I`ve created a wireframe for mobile version of search page.
a link and a png file for the prototype are shown below:
https://www.figma.com/proto/VjJhKU9ayU62rwczFyaUX0mt/FreeBMD-search-form-prototype?node-id=451%3A0&scaling=scale-down

Ready for feedback/review.

@neslihanu Is there potential to reduce the space taken up by the red banner and footer material?  E.g. reduce size of logos?

@PatReynolds  I`ll have a look into it. What would be the text in the red banner ? Has it finalised ? It would be good to see it.

@PatReynolds

I`ve reduced the text sizes on the red banner and footer paragraph. In general, font size needs to be 16 pixels for mobile websites. Anything smaller than that could compromise readability for visually impaired readers.

I`ve reduced the size of logos. We can say that they are also "buttons" where the user may click on them and be navigated to the other Free UK Genealogy websites. So, we should consider Finger-Friendly Design. This means that 40 pixels is the minimum.

Please have a look at the updated version.
Here is the link for demo;
https://www.figma.com/proto/VjJhKU9ayU62rwczFyaUX0mt/FreeBMD-search-form-prototype?node-id=451%3A0&scaling=scale-down
Here is the updated wireframe;

This looks good.  Thank you, @neslihanu.
Footer text should be as on the PC version of FreeBMD2

Discussed providing a link to the<EMAIL_ADDRESS>text rather than include it here - perhaps providing smaller icons for the other FUG project sites also.

@PatReynolds  to created new page for copyright info.

@PatReynolds @richpomfret Am not warming to the chopped up look of the nav bar. Has anyone considered giving what is left of the bar itself a white background (instead of the magenta)?

Goog point, @AlOneill .  What do you think, @neslihanu ?

@neslihanu @PatReynolds (I meant to add) or, keep the bar magenta and make the hamburger and "Menu" white?

Personally, I like magenta better than white

@PatReynolds @AlOneill
Aviel worked on this part, unfortunately we didn't have time to discuss and review this design together before his leave. So, I don't know what was his idea behind this menu design and also I didn't want to change it before you review.
I prefer a white hamburger menu icon on magenta background. I think when we put menu text + hamburger menu icon, users may click on the word “menu” than the hamburger menu icon.

We also need to define what is to be displayed from clicking the menu. Currently the apps display a vertical list of the actions contained in the Nav Bar. This is really part of # as it applies to all pages not just search results

@PatReynolds what would be the footer text ? I've checked the desktop version of the wireframes on Figma, both are the same. Can you please share it with me ?

Something different now needed for the red bar (text here is superceded by the text in the lowest, dark grey, bar.  I will make a separate story for that.
Text in dark grey bar on mobile needs to be changed to the same text as normal version (which is missing a space between the charity number and the next sentence ("1167484.We" needs to be "1167484. We" (I will add that typo to the main footer story).

Please checked the below design for updates:

https://www.figma.com/proto/VjJhKU9ayU62rwczFyaUX0mt/FreeBMD-search-form-prototype?node-id=451%3A0&scaling=scale-down

Design signed off: please implement @Vino-S


box86: init at 0.3.6

Description of changes
Closes #174113 (stale)
CC @Yeshey since you seem interested in box86
Heavily based on our box64 expression.
Second commit is maybe abit sketchy, so it's separate in case we don't want it.
Since box86 is only supported on 32-bit userland, its supported platforms can also only be 32-bit. But one may want to use it to run 32-bit x86 software on 64-bit non-x86 hardware as well. It's trivial to tell nix to cross-compile the package, but then the user would need to compile this on their own hardware since (AFAIU) the cache doesn't serve many 32-bit packages (especially on ARM I imagine?). This should work around this by mapping x86_64 -> i686 & aarch64 -> armv7l, which should hopefully let the cache be able to substitute this for ppl.
Things done

Built on platform(s)

[x] x86_64-linux (i686-linux cross)
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build box86 box86.passthru.tests

cool, @OPNA2608, hit me up if I could help with anything else

any chance for this to go through?


Caught exception when trying to load file of format [py]: init() missing 1 required keyword-only argument: 'executable'

Hello
I have tried to follow the instructions on the repo to install this package on ros2 galactic and am encountering the above error when am trying to launch the node.
[INFO] [launch]: All log files can be found below /home/user/.ros/log/2022-02-25-20-53-38-777999-user-desktop-793818
[INFO] [launch]: Default logging verbosity is set to INFO
Task exception was never retrieved
future: <Task finished name='Task-2' coro=<LaunchService._process_one_event() done, defined at /home/user/ros2_galactic/build/launch/launch/launch_service.py:226> exception=InvalidLaunchFileError('py')>
Traceback (most recent call last):
  File "/home/user/ros2_galactic/build/launch/launch/launch_description_sources/any_launch_file_utilities.py", line 53, in get_launch_description_from_any_launch_file
    return loader(launch_file_path)
  File "/home/user/ros2_galactic/build/launch/launch/launch_description_sources/python_launch_file_utilities.py", line 68, in get_launch_description_from_python_launch_file
    return getattr(launch_file_module, 'generate_launch_description')()
  File "/home/user/ros2_galactic/install/sllidar_ros2/share/sllidar_ros2/launch/sllidar_launch.py", line 48, in generate_launch_description
    Node(
TypeError: __init__() missing 1 required keyword-only argument: 'executable'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/user/ros2_galactic/build/launch/launch/launch_service.py", line 228, in _process_one_event
    await self.__process_event(next_event)
  File "/home/user/ros2_galactic/build/launch/launch/launch_service.py", line 248, in __process_event
    visit_all_entities_and_collect_futures(entity, self.__context))
  File "/home/user/ros2_galactic/build/launch/launch/utilities/visit_all_entities_and_collect_futures_impl.py", line 45, in visit_all_entities_and_collect_futures
    futures_to_return += visit_all_entities_and_collect_futures(sub_entity, context)
  File "/home/user/ros2_galactic/build/launch/launch/utilities/visit_all_entities_and_collect_futures_impl.py", line 45, in visit_all_entities_and_collect_futures
    futures_to_return += visit_all_entities_and_collect_futures(sub_entity, context)
  File "/home/user/ros2_galactic/build/launch/launch/utilities/visit_all_entities_and_collect_futures_impl.py", line 38, in visit_all_entities_and_collect_futures
    sub_entities = entity.visit(context)
  File "/home/user/ros2_galactic/build/launch/launch/action.py", line 108, in visit
    return self.execute(context)
  File "/home/user/ros2_galactic/build/launch/launch/actions/include_launch_description.py", line 130, in execute
    launch_description = self.__launch_description_source.get_launch_description(context)
  File "/home/user/ros2_galactic/build/launch/launch/launch_description_source.py", line 84, in get_launch_description
    self._get_launch_description(self.__expanded_location)
  File "/home/user/ros2_galactic/build/launch/launch/launch_description_sources/any_launch_description_source.py", line 53, in _get_launch_description
    return get_launch_description_from_any_launch_file(location)
  File "/home/user/ros2_galactic/build/launch/launch/launch_description_sources/any_launch_file_utilities.py", line 56, in get_launch_description_from_any_launch_file
    raise InvalidLaunchFileError(extension, likely_errors=exceptions)
launch.invalid_launch_file_error.InvalidLaunchFileError: Caught exception when trying to load file of format [py]: __init__() missing 1 required keyword-only argument: 'executable'

Can someone please look into what the problem is ? I am unable to figure out the issue.

Hi!
I suppose the error is related to the content of the launch file.
From galactic distro I think the labels "node_executable" and "node_name" have been deprecated as also reported here #1.
Modifying such labels to "executable" and "name", it should work.

Submitted a patch for the issue. Check PR #3

Would this close #1 and #2?

+1


Many-to-many search term bug

Summary Change Description
This change essentially re-filters search results in the template using the same query object that the original database .filter(Q(...)) performed server-side.
Details:
The derived classes in compositeviews.py now tracks whether each related table is related many-to-many with the root table.  If a many-to-many relation exists in the composite view, a refilter flag is set to true.  However, refiltering is only necessary if a field from the M:M table is included in the search terms, so in the template, a template tag is called named shouldKeepManyToMany.  This method sets a template variable named keep, which is used to decide whether to render the record.
I also fix the record count displayed at the top of the results.  That count was actually accurate for searches that used terms from the M:M table, but the number of rows were sometimes more.  But, the count was wrong in other cases when a search term was not from a M:M table.  I fixed every case by creating a counter in the template which is rendered (but hidden) at the bottom of the results and javascript is used to update the count at the top to match.
Ancillary small issues addressed:

Removed downloads from gitignore and added *~\$* to ignore Excel cruft.
I streamlined DataRepo/templates/DataRepo/search/results/display.html after moving the count to the format-specific templates.
Since the templates now uses the qry object, even in the initial browse mode (to obtain the template name for the title above the table), I had to add qry to the context in the addInitialContext method in that circumstance.

Oh yeah, I just remembered that I need to update DEVELOPMENT.md to reflect some of the new changes.
Affected Issue Numbers

Resolves #222

Code Review Notes
Please ignore changes to DataRepo/utils.py.  This is actually an edit copied from the other PR that was necessary to make the tests work correctly.  When I merge, I will resolve the conflict by accepting the code from the main branch.
Note, most of the template changes are just an extra indent.
Checklist

[x] All issue requirements satisfied (or no linked issues)
[x] Linting passes.
[x] Migrations created & committed (or no model changes)
[x] Tests implemented (or no code changes)
[x] All tests pass

Oh geez.  I was actually surprised the local run didn't complain with jscpd errors.  And let's see what flake8 is complaining about in the remote run...

Rebased after the other PR's merge.

Nothing jumps out at me as "needs fixing", although I have a gut feeling that if perhaps we are having to filter/execute a query twice, perhaps there is an alternative solution (unknown to me) that does the exact filtered-query we want the first time.

No.  Does not exist.  I have researched it thoroughly.  Django provides the ability to query through M:M related tables, but all it returns are root table records in the resulting queryset and the only way they provide to obtain related records is a left-join, without any sort of filtering.  When you get to a M:M table when compiling the results, all you can do, and in fact the only thing they support, is an independent query.  Their docs simply suggest appending .all in a nested loop in the template.  So if a root table record is queried using a search term from a M:M related table, and there exist related records that do not match the search term, there is no method provided to prevent getting those non-matching results.  There are 2 options: skip the unwanted records when rendering the template, or compile everything server-side in like a pandas table and skip them there.  There's no way around needing to apply an extra filter.  It's just a question of where you do it, and my gut feeling is that doing it in the filter is faster than compiling and sending a pandas df, which has its own drawbacks of needing special code to deal with cached properties.

    I.e. what is the precise issue?  Are you suggesting a different tag/function-name or are you saying the "stream-of-consciousness poetry slam" quality of the code is more broad in scope?  Should I be employing more encapsulation?

  I.e. what is the precise issue? Are you suggesting a different tag/function-name or are you saying the "stream-of-consciousness poetry slam" quality of the code is more broad in scope? Should I be employing more encapsulation?

I am not criticizing, I just find the template syntax confusing (tag/function chaining) since I have not personally used it. Also,  since I did not write the tags/functions myself, I don't have any specific intuition what pg is, or really any of the specific in's and out's of any of functions/tags.  It is just my unfamiliarity.

What is your suggestion/criticism here exactly?  I.e. what is the precise issue? Are you suggesting a different tag/function-name or are you saying the "stream-of-consciousness poetry slam" quality of the code is more broad in scope? Should I be employing more encapsulation?

I am not criticizing, I just find the template syntax confusing (tag/function chaining) since I have not personally used it.

I see.  Let me try to explain (that is, if you'd like to know what's going on...).  There's no chaining going on here.  shouldKeepManyToMany  is a function (decorated as a simple_tag) defined in DataRepo/templatetags/customtags.py.  It can be called as a tag because of the load of the customtags.py file at the top of the template.  You call it from a template like this: {% function_name space_delimited_arguments %}.  And you can assign the return value to a context variable by appending  as variable_name.  In this case, shouldKeepManyToMany takes 4 arguments:

pg
mm_lookup
qry
refilter

So the call with assignment is structured like this: {% function_name space_delimited_arguments as variable_name %}.

Also, since I did not write the tags/functions myself, I don't have any specific intuition what pg is, or really any of the specific in's and out's of any of functions/tags. It is just my unfamiliarity.

pg is the current peak group record, which is a variable defined in the for-loop.
mm_lookup is a dict that holds the current many-to-many records of any nested for loops that are used in the "re-filtering".
qry defines the user's search query, including the ands and ors involved.
refilter is a boolean sent into the template from its representation in views.py and it is only ever false in 2 cases (where re-filtering is not necessary): when in browse mode (i.e. there are no search terms) and when the composite view does not include a many-to-many related table.
If you think any of this would be useful as doc-strings, let me know.

I see. Let me try to explain (that is, if you'd like to know what's going on...). There's no chaining going on here. shouldKeepManyToMany  is a function (decorated as a simple_tag) defined in DataRepo/templatetags/customtags.py. It can be called as a tag because of the load of the customtags.py file at the top of the template. You call it from a template like this: {% function_name space_delimited_arguments %}. And you can assign the return value to a context variable by appending  as variable_name. In this case, shouldKeepManyToMany takes 4 arguments:

pg
mm_lookup
qry
refilter

So the call with assignment is structured like this: {% function_name space_delimited_arguments as variable_name %}.

Thanks, that clears it up for me.  As I said, I am unfamiliar with the syntax.  If I ever write a template, I will try to remember not to name any variables as, or presumably other Django reserved words.


Move videos based on music

Would be nice to add an additional module that would "push" or "jitter" frames of the video based on music. A closed source version of this can be seen here. Would love to have open souce option for doing this with stable diffusion 😄
I don't really have the bandwidth to do this myself right now, so help (even if its just a colab with minimal example) would be much appreciated on this!!
If anybody is creeping on this repo/working on their own project + ends up getting to this before me, please ping me here so I can play with it 😬

it's really a nice feature.
as a rough idea for the implementation I am thinking about detecting the frequency of the audio clip then changing the interpolation percentage relatively to the frequency changing speed, thus the number of the frames would increase for the low frequency audio.
I am planning to use some implementation of Fast Fourier Transform to detect the changing in the frequency.
any idea or suggestion before I try working on this ?

@mouhandalkadri thanks for your interest here! I honestly don't have a great idea on how to do this (and again, have limited bandwidth to think about it), but I did have some ideas. They might be bad haha, but I'll share what I was thinking.
Feel free to ignore anything here and experiment with something else, this is just one idea:

Basically, was thinking we keep the frames moving at a consistent rate over time, like we see below...

Then, we set a threshold (let's say -6db). If the audio goes above that threshold, we push frames faster and then come back to join the original rate. So, essentially, you're still moving at the constant rate from above, but you push to-and-from later frames as audio goes above a certain threshold.

Relevent: https://github.com/mikaelalafriz/lucid-sonic-dreams

Update - I've figured this one out. Will post the code when it's clean. Results are 🔥 and will share soon

Closed! Info in readme.


Alt+Arrows should scale the bottom/right edges of layers

Similar to arrow keys (and the Shift modifier) to nudge the position of selected layer(s), this feature should "nudge" (by resizing) the right and bottom sides of the selected layer(s). Left and right arrows resize the layer(s) by 1px on the right edge. Up and down arrows resize the layer(s) by 1px on the bottom edge. The Shift makes it 10px. This should happen in layer space, not viewport space, so regardless of viewport zoom the user should expect to see the dimensions in the Properties panel change by 1px or 10px in W or H. If the layer is rotated, it should scale along the axis of the layer, so the X and Y would change as expected but the W and H would not necessarily change by 1px/10px. Assuming it doesn't conflict with any other keys, Ctrl would be nice as a modifier that changes it to the Left/Top instead of Bottom/Right.

The winding order for rectangles created with the rectangle tool is arbitrary and doesn't change, and with the path tool it is based upon the order the points are added by the user, so I doubt this is the issue.

We probably need to enforce a standard winding order for all shapes, including those made with the Pen tool regardless of whether the user drew the shape in the CW or CCW direction. But it's worth doing some research to see if that's how it should be handled.

#990


[SPARK-43139][SQL][DOCS] Fix incorrect column names in sql-ref-syntax-dml-insert-table.md

What changes were proposed in this pull request?
This PR fixes incorrect column names in sql-ref-syntax-dml-insert-table.md.
Why are the changes needed?
Bug fix.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
N/A.

Merged to master and branch-3.4.


merge from template-arlac77-github

package.json

chore(deps): add ^2.2.10 remove ^2.2.9 (devDependencies.browser-ava)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
95

Relevant Lines:
95

💛  - Coveralls


Needs some feedback for first release candidate 1.0-rc1

I know have a stable ( I hope ) polished version for latest UI as https://github.com/KipK/openevse-gui-v2/releases/tag/1.0-rc1
please [@fhteagle ] [@jeremypoulter] can you give a test and reports back ?
Hope you'll like it.
To do for rc2;

1- Wifi Portal version
2- RAPI developper page done: https://github.com/KipK/openevse-gui-v2/commit/2a0b5012bfaf35083a8d9a8de9c700e10f34d364
3- check that passwords/keys are not sent on various forms when filled by "DUMMY_PASSWORD"
4- add more help pages whith doc when needed
5- add About section with OpenEVSE links + doc done: https://github.com/KipK/openevse-gui-v2/commit/5eef75469d3fe0ac3e071a04683d40ab607856d0
6- Add Max Current Soft setting ( forgot this one ^^ ) done: https://github.com/KipK/openevse-gui-v2/commit/01b03dfd11b993cfaab9cf6c8da3cc3971753451
7- clean-up the mess
8 - rewrite Monitoring page with tabs done: https://github.com/KipK/openevse-gui-v2/commit/b0773b84d02080fab68ec4e459bbe191597efb90
9 - host Fonts & Icons locally https://github.com/KipK/openevse-gui-v2/commit/72fd12006b30acb779d310f9546201faa372cd80
10 - write Ethernet config tab

Time & Charge Limits are disabled for now until proper API integration on EVSE firmware

[@jeremypoulter] what are the requirements for the Wifi Portal Version ?
I'm thinking about doing a step by step configuration process.
Also I'm thinking about pushing auto update to the UI, but we can't use github api directly as it requires https.
Could you setup a wrapper on openevse.com ?

Would be nice to have a sort of wizard that takes you through setting up the (hardware/install) max current, connects to the WiFi (or reports that ethernet is connected) and finally allows the firmware to be updated. Yes ideally this would be from GitHub, HTTPS shouldn't be a problem, it is just the redirects, need to ideally just handle these, either generically in ArduinoMongose or specifically for the update.

max_current_hard may not actually be implemented yet, but this is a write once value that sets the absolute maximum value, you can then adjust max_current_soft up to that value. This is mainly for the American market where the EVSE can handle up to 80A but will be installed with different wiring, etc and will likely not be able to supply the full 80A. For now we just set the soft value.
For wired it is just a case of detecting that the connection is wired and displaying that a message instead of the WiFi config.
I did do some testing of GitHub OTA and it works if you manually do the redirect, so just need to implement something to do that.

I did do some testing of GitHub OTA and it works if you manually do the redirect, so just need to implement something to do that.

Do you still have a bunch of code somewhere on
how you did that?

There are some examples in https://github.com/OpenEVSE/ESP32_WiFi_V4.x/blob/master/test/update.http, all I did to make it work was use cURL to get the actual URL

and for the ethernet module, what would be the condition to display the wizard ?
I see you use AP mode only for wifi modules.

I think max_currznt_hard is implemented , I remember I've fucked up my
openevse by setting it to 6A while testing, and then had to reflash to
Atmega firmware then. :)
Le ven. 16 déc. 2022, 19:55, Jeremy Poulter @.***> a
écrit :

max_current_hard may not actually be implemented yet, but this is a write
once value that sets the absolute maximum value, you can then adjust
max_current_soft up to that value. This is mainly for the American market
where the EVSE can handle up to 80A but will be installed with different
wiring, etc and will likely not be able to supply the full 80A. For now we
just set the soft value.
For wired it is just a case of detecting that the connection is wired and
displaying that a message instead of the WiFi config.
I did do some testing of GitHub OTA and it works if you manually do the
redirect, so just need to implement something to do that.
—
Reply to this email directly, view it on GitHub
https://github.com/KipK/openevse-gui-v2/issues/17#issuecomment-1355424253,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADXKVNFYVAGLHKLOZL5NU3WNS3LNANCNFSM6AAAAAAS4CZJKM
.
You are receiving this because you authored the thread.Message ID:
@.***>

@jeremypoulter , finally, there's no way I can do that on client side because of CORS issues with fetching binary from github. ( edited comments above )
I see in http_update.cpp line 30, the response code is catched here.
Can we catch the 302 redirect there and then get back to the
if(response->respCode() == 200)
?
I'm not used to this moongoose library, if you see the quickest way to handle that

It is something I need to sit down and think about. Will try and take a look, would be great to get it done.

Everything is ready on client side so if you have something to test, please share :)

So It now displays if there's an udpate available:

and then propose to downlaod the file or direct upgrade from github release using http_update

Wizard is ok.
It's handled on client side, and will display wizard mode if mode is "AP" or "STA+AP" with SSID not configured like current implementation.
After Wifi connection, it will redirect the window to the wizard  firmware update page using {hostname}.local url
I have not found a correct condition to show the wizard for ethernet connection

I have pushed rc2 https://github.com/KipK/openevse-gui-v2/releases/tag/1.0.0-rc2
I think we can try this one on the firmware now.


Fix 460: When sample animation mode is used, bake all bones

When sample animation mode is used, all bones are baked, not only bones with keyframes.
This will allow to export complex rigs, where deformation bones are not animated. This is how fbx export works, for example.
Tested on some various rigs, including a bouncing ball,and a simple character with IK leg / foot roll.
460.zip

Seems ready to be tested @emackey @donmccurdy

Unfortunately I'm not the right person to be reviewing skinning/bone/rigging code.  @donmccurdy or @pjoe or @Cyp, would you guys have any time to look at this?  Feedback from anyone else is welcome too of course.

I'll review it this week.
(and thank you @julienduroure for the PR and @Cyp for the review, much appreciated!)

From a quick read it looks pretty good to me :) I will try to do some tests with different files.
Btw. interesting with those python type annotations, though I do feel a bit weird that there is no actual type check - guess I have been doing to much TypeScript lately 😛

@julienduroure what export settings should I be using to check the IK leg example? With *Always Sample, or Always Sample + Bake Skinning Constraints, I'm not getting a correct output (although it looks closer than before.

Also – None of my skinning examples export correctly on the master branch. I'm happy to do reviews, but I'm also OK if we want to merge PRs more quickly, given that the master branch skinning export is already quite broken as far as I can tell. If there are any skinning examples that do work now, let's upload them somewhere – otherwise my testing things is not very effective. 😔

@donmccurdy Currently using only Always sample because bake sinning does nothing in code for now (see #485 for the discussion about that)
You have here 2 examples
460.zip
And here is a really complex rig (generate with rigify). Re-import in blender is correct (some glitches maybe linked to only 4 skin weights export ?), but is KO when viewing it in https://gltf-viewer.donmccurdy.com/ :(
rigify.zip

@donmccurdy :
In https://gltf.insimo.com/, animation is better, even if there is still a global orientation issue
And everything seems to be OK in https://sandbox.babylonjs.com/

Ok thanks! I've filed https://github.com/mrdoob/three.js/issues/16562 about the animation issue in three.js, we can look into that. We ready to merge this?

@donmccurdy Before merging, I want to be sure that this behaviour is linked to the right user option in UI.
My comments and solution proposal are in #485

Thanks for your feedback. I will manage merging myself later today


Using USB product ID and vendor ID (PID/VID) to find device (on Chrome 50+)

A regression in Google Chrome 45 has now partially been fixed in Chrome 50.
How it used to work: We pass the USB PID/VID to chrome.usb.getDevices, and only get back the devices that match that combo
What broke and the hotfix: Windows doesn't return any serial devices, so we removed the PID/VID filter and returned all serial devices.
How it's fixed now in this PR: On Windows Chrome 50+, chrome.serial.getDevices now returns the PID/VID combo as well, so we can use this to filter ourselves. OS X does not return the PID/VID combo, so we use chrome.usb.getDevices on Mac.

 <ux:0 dev:3 tst:3> [uploader] verify product ID and vendor ID for USB serial devices

LGTM to merge. Thanks!


Azure Functions V1 .NET Framework Version is Incorrect

The first table states 4.6 and the second states 4.7. At the moment, the actual version is 4.7.2.
Annoyingly there doesn't seem to be a Microsoft website anywhere with this information to hand, but it can be inferred from announcements.
Azure App Service—SameSite cookie handling and .NET Framework 4.7.2 patch and .NET Framework 4.7.2 update coming to Azure App Service apps
I think the safest thing to say would be ".NET Framework 4.x" with a link to a page with the current version, or at least these instructions.

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

ID: 1df5e2bd-489d-19b2-87d3-d993ebb82021
Version Independent ID: 9e9d9ded-8f40-22be-4654-48c87f2c59e8
Content: Azure Functions runtime versions overview
Content Source: articles/azure-functions/functions-versions.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hi @fowl2 Thanks for your feedback. We will review and update as appropriate.

@fowl2 PR has been created for this which will address this. Thanks so much for providing the feedback that help us in improving the documentation. Also for other items, I will create internal work item and work with content author. For now I will proceed with closure of this and If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@DixitArora-MSFT just a general process thing - it would be great if this issue could be updated when the PR was merged- thanks!


Request for Enhancement - Security

Client is under strict HIPAA and security requirements and needs some mechanism to guarantee containers require some form of handshake with their correct exports/LUNs.  This probably includes some form of intelligence to know which LUN to present to which container.  That functionality probably belongs higher in the orchestration layer but may require some adaptions in storage, so I’m submitting to raise visibility to the request.

Tracked internally by US2284

Multiple instances of Trident + CHAP (different user for each Trident instance) can do that (assuming two-way CHAP authentication - I haven't yet looked how CHAP is implemented, as one- or two-way auth).

As @scaleoutsean has pointed out Trident support CHAP for ONTAP-SAN, ONTAP-SAN-economy, and SOLIDFIRE-SAN drivers.


After flashing firmware via UART with the bootloader, BattOr output is corrupted

This bug was solved a long time ago in the process of fixing many other things.


Better CancellationToken support for ApiExplorer

Is your feature request related to a problem? Please describe.
I want to use Microsoft.AspNetCore.Mvc.Api.Analyzers and its DefaultApiConventions to analyze my apis. However, I use CancellationTokens in the controller actions. This means that the DefaultApiConventions won't match to my controller actions and I get an "API1000 Action method returns undeclared status code [...]" warning. Since CancellationTokens are not part the api contract per se, I don't think we should need to create duplicate conventions for controller actions with CancellationTokens and controller actions without CancellationTokens.
Describe the solution you'd like
CancellationToken parameters are ignored by the ApiConventionMatcher.
Describe alternatives you've considered
Creating duplicate conventions for controller actions with CancellationTokens and those without. eg:
        [ProducesResponseType(StatusCodes.Status200OK)]
        [ProducesResponseType(StatusCodes.Status404NotFound)]
        [ProducesDefaultResponseType]
        [ApiConventionNameMatch(ApiConventionNameMatchBehavior.Prefix)]
        public static void Get(
            [ApiConventionNameMatch(ApiConventionNameMatchBehavior.Suffix)]
            [ApiConventionTypeMatch(ApiConventionTypeMatchBehavior.Any)]
            object id)
        { }

        [ProducesResponseType(StatusCodes.Status200OK)]
        [ProducesResponseType(StatusCodes.Status404NotFound)]
        [ProducesDefaultResponseType]
        [ApiConventionNameMatch(ApiConventionNameMatchBehavior.Prefix)]
        public static void Get(
            [ApiConventionNameMatch(ApiConventionNameMatchBehavior.Suffix)]
            [ApiConventionTypeMatch(ApiConventionTypeMatchBehavior.Any)]
            object id,
            [ApiConventionNameMatch(ApiConventionNameMatchBehavior.Any)]
            CancellationToken cancellationToken)
        { }

@jklawrence thanks for your issue report. This sounds like a neat idea. One way we could add support for this is to allow optional parameters in the convention method e.g.
[ProducesResponseType(StatusCodes.Status200OK)]
[ProducesResponseType(StatusCodes.Status404NotFound)]
[ProducesDefaultResponseType]
[ApiConventionNameMatch(ApiConventionNameMatchBehavior.Prefix)]
public static void Get(
    [ApiConventionNameMatch(ApiConventionNameMatchBehavior.Suffix)]
    [ApiConventionTypeMatch(ApiConventionTypeMatchBehavior.Any)]
    object id,
    [ApiConventionNameMatch(ApiConventionNameMatchBehavior.Any)]
    CancellationToken cancellationToken = default)
{ }

which would match Get(id, CancellationToken token) as well as Get(id). In the meanwhile, one workaround to avoid duplicating your conventions might be to use to a trailing params argument.
[ProducesResponseType(StatusCodes.Status200OK)]
[ProducesResponseType(StatusCodes.Status404NotFound)]
[ProducesDefaultResponseType]
[ApiConventionNameMatch(ApiConventionNameMatchBehavior.Prefix)]
public static void Get(
    [ApiConventionNameMatch(ApiConventionNameMatchBehavior.Suffix)]
    [ApiConventionTypeMatch(ApiConventionTypeMatchBehavior.Any)]
    object id,
    params object[] any)
{ }

You do risk over-matching, since the convention could match methods with more than 2 parameters e.g this would match Get(int id, string name, CancellationToken cancellationToken)

@pranavkm thanks for taking a look. I like the idea of allowing optional parameters in the convention method. That seems like an elegant way to solve this without a special case for CancellationToken.
In my organization's application we use CancellationTokens in all of our ApiController actions, so I'm able to just set a CancellationToken parameter on each convention.


[GitHub Gist] Reduce GitHub authentication

Description

Reduce GitHub authentication permissions to access only GitHub Gist content
Fix the issue of Open in Ray.so command displaying garbled code
Add new Action: Save as Quicklink

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Also, for the PR, here's a screenshot of what the OAuth user prompt looks like with the superfluous scopes.


Account name is wrong in Ledger Bitcoin only mode

Account index on home page doesn't match that one from switch acc dialog

This was fixed in: https://github.com/leather-io/extension/pull/5571


Updating group name to scheduling.x-k8s.io

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #973
Special notes for your reviewer: N/A
Release note:

/easycla

❌ - login: @sriramg96 / name: Sriram Ganesan . The commit (e806d6c) is not authorized under a signed CLA. Please click here to be authorized. For further assistance with EasyCLA, please submit a support request ticket.

/easycla

/easycla

/ok-to-test

/retest

/test all

@k82cn I think the generated doc files need to be updated (such as here), but I don't see this hack/update-generated-swagger-docs.sh file that was used to generated the swagger docs. I was wondering if you knew if this was renamed or moved elsewhere, and if so how to regenerate the docs.

@k82cn I think the generated doc files need to be updated (such as here), but I don't see this hack/update-generated-swagger-docs.sh file that was used to generated the swagger docs. I was wondering if you knew if this was renamed or moved elsewhere, and if so how to regenerate the docs.

I'm not sure about that part. Maybe we can handle it later :)


[monitoring v2] grafana workloads dashboard shows memory in GiB, should be MiB

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

deploy a cluster
deploy monitoring v14.5.100
go to monitoring -> grafana
visit the rancher/workloads (pods) or similar dashboard

Result:
memory utilization is shown in GiB, but should be MiB

Other details that may be helpful:
also happens for rancher/etcd GRPC Client Traffic
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.5.8-rc9
Installation option (single install/HA): single

on v2.5.8-rc11:
everything appears to be on the correct scale now

other metrics (summaries, averages, etc.) are still using the correct scales


Error when using method="no_params" and GPU, because E and F incorrectly remain on CPU

When you create a Linformer() with method="no_params" and then load the model on your cuda device, you will get an error when trying to use the model. This is because the E and F matrices in the model accidentally remain on the CPU. When you call forward with an input, you will get an error at some point because the attention heads are trying to multiply the E matrix on CPU with another matrix on the GPU.
Basically, when you call Linformer().cuda() under this situation, the E and F matrices are not moved to the GPU.
(From what I've read so far, in order for them to be put on the GPU with a cuda() call, you also need to assign E to self.E in Linformer. However, this still doesn't fix it because of the lambdas in your Linformer initializer I think. The cuda() call can't track down the E and F inside the MHAttention objects inside the lambda call it seems)
My temporary fix is changing E_proj = get_EF(input_size, dim_k, method, head_dim) in the __init__ of Linformer to E_proj = get_EF(input_size, dim_k, method, head_dim).cuda(), but I think this would give an error if you do not have a gpu installed.

Hi @RaivoKoot!
I think the issue is simple to fix. I think all that we need to do is, in the forward call, we need to add a self.E = self.E.to(K.device) between these two lines
https://github.com/tatp22/linformer-pytorch/blob/0fe2c6206ac5a6a44cedb7fe97b773dad1320c47/linformer_pytorch/linformer_pytorch.py#L157-L158
And we need to add a `self.F = self.F.to(V.device) between these two lines:
https://github.com/tatp22/linformer-pytorch/blob/0fe2c6206ac5a6a44cedb7fe97b773dad1320c47/linformer_pytorch/linformer_pytorch.py#L177-L178
I'll make a new pull request right quick, I think this should fix it.

Just made it, let me know if it works for you. If it does I'll pull it in and up the version to 0.19.3

I merged this in, hopefully it works for your case as well

Works like a charm (1 GPU)! Thanks. Great work on the library btw! Will test everything out in a multi GPU setting as well!


Release v5.0.0

Merged PRs since v4.1.0:

#103
#104

A new version of the package @metamask/eth-token-tracker (5.0.0) was published at 2023-07-25T15:37:28.260Z from [redacted].
The shasum of this package is bfb91e6f161e36306a024b86ea8db344ad89ec29.


Add logs integration

What does this PR do?

Add logs instructions for RethinkDB.
Motivation

Additional Notes

Local forwarding of logs to staging works well -- see below (NOTE: logs parser not deployed yet, so logs are in raw format).
I restarted the server0 container to trigger log lines present in the screenshot:
$ docker container stop rethinkdb-server0
$ docker container start rethinkdb-server0

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Waiting a review from docs :-)


String added to With/From field in summary table even when nothing entered in the form

The summary view at the bottom added a string to the With/From field for a BP annotation even when I didn't enter anything in the form for that field.  See the second line in the table:

@vanaukenk @krchristie  ohh I published that code by mistake (was experimenting). But eventually it will solve this ambiguous  problem

In this case we don't know which entity is being annotated for cholesterol homeostasis. So we were gonna use with field to store gomodel id value for annotated entity (gp), but not show it to the user. I think we can make a new ticket for this issue and continue discussion there and @thomaspd can explain more


Update machine-controller to v1.59.0-beta.0

What this PR does / why we need it:
Update machine-controller to v1.59.0-beta.0
What type of PR is this?
/kind chore
Does this PR introduce a user-facing change? Then add your Release Note here:
Update machine-controller to v1.59.0-beta.0

Documentation:
NONE

/assign @kron4eg

/lgtm
/approve

/hold

/hold cancel

/retest


why do we need set causal_mask =True in clip

prompt is  a sentence ,we don't need to predict next token in prompt, is there a question to see the right tokens?
x = self.attention(x, causal_mask=True)

This my question as well.

I had the same query. This is the answer I found in the CLIP paper by OpenAI:
"Masked self-attention was used in the text encoder to preserve the ability to initialize with a pre-trained language model or add language modeling as an auxiliary objective, though exploration of this is left as future work."
Paper: https://arxiv.org/pdf/2103.00020


fix: MongoPlugin Update Command : parsing valid document or array of valid documents

Description
Fixes #11419
Based on the suggestion : https://github.com/appsmithorg/appsmith/pull/12241#discussion_r844997566

Hi @devnamrits , this approach has a drawback. If we do this, then we can never validate any input as valid or invalid. e.g. If this.update is "abc" then this will be parsed as a valid input which is not intended. Essentially this change converts any input into a valid input. However, what we want is to be able to detect if the user input is valid or not. Up until now we have only tried to check if the input is a valid Document or not, which in this case is not sufficient since MongoDB 4.2 onwards it also accepts an array of Documents (pipleine) as well. Hence, IMO, the correct approach here should be to parse for two things - (1) valid Document (2) valid array of Documents. To achieve this one good way would be create a new method parseSafely(fieldName, input., boolean) along the lines of parseSafely method that already exists and use the last parameter to check if we need to validate for array of Documents as well.

Added a method parseSafelyRawInput : parsing valid document or array of valid documents
Added junit to check the existing error on the existing parseSafely method
Added junits to check success and failures for the new method.

Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Added junit to check the existing error on the existing parseSafely method
Added junits to check success and failures for the new method.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

@sumitsum @mojtab23
Could you review and let me know of changes, if any?

cc @Nikhil-Nandagopal @mohanarpit

@amogh2019 thanks for raising this PR. I will be reviewing it. Can you please comment on the issue as well asking for the issue to be assigned to you, post which I can change the issue assignment to reflect your id.

@sumitsum

@mohanarpit @Nikhil-Nandagopal  Could you add more reviewers?
Sorry for pushing for this, but hacktoberfest will be ending on 31oct and then this contribution of mine won't be counted

@amogh2019 apologies for delay, I was OOO. Reviewing it now.

addressed review comments

@amogh2019 Approved it.

/ok-to-test sha=50204ee

@amogh2019 can you please rebase with release and update the PR ? - since this may improve changes of CI run passing.

Hi,
the branch is up-to-date with upstream release,
and is This branch is 4 commits ahead of appsmithorg:release.

Can we merge? @sumitsum

@amogh2019 can you please pull again from the remote release into your branch (rebase) and update the PR ? Some changes have been pushed in recently to fix CI failures. Also, would it be possible for you to provide my account with push / edit access to your PR branch so that I may do such kind updates if required (in case you are not accessible) ?

@amogh2019 seems like I am able to update the PR directly. I have updated the PR with the latest release. I will re-run the CI now.

/ok-to-test sha=db96cad

/ok-to-test sha=db96cad

not sure why ui-test-result is failing


Remove unnecessary HTML tags

https://docs.microsoft.com/en-us/windows/win32/http/add-iplisten
https://github.com/MicrosoftDocs/win32/blob/docs/desktop-src/Http/add-iplisten.md
#PingMSFTDocs

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


[FR] Vendor distutils stubs

What's the problem this feature will solve?
Type-checkers don't see distutils as available in Python 3.12+
This is already causing us much pain when it comes to type-checking, and is becoming a blocker.
One of the main issue is that importing from distutils, types are seen as Any, which breaks subclassing, removes tons of type-safety, and creates nasty inconsistencies when trying to type-check <3.12 vs >=3.12.
We also get some overloading issues when type-checkers don't understand we're using an updated distutils version vendored from pypa/distutils.
As long as _distutils_hack exists, we can't have type-checkers scan the vendored version instead. (if pypa/distutils was installed like a regular package, this wouldn't be an issue either)
Describe the solution you'd like
My suggestion is to vendor distutils-stubs, reflecting the types as the modernized pypa/distutils.
(concretely this means: Start with https://github.com/python/typeshed/tree/main/stdlib/distutils, then apply https://github.com/python/typeshed/tree/main/stubs/setuptools/distutils on top)
Those stubs should exist in the repo in a folder named distutils or distutils-stubs.
distutils-stubs should be installed in users' site-packages when installing setuptools (this can be locked behind an extra if you have concerns for vendors)
This also means we no longer need to wait on new mypy releases to get distutils typing fixes from typeshed AND we can be more accurate to what _distutils does.
Alternative Solutions

Make pypa/distutils a real package found in site-packages
Change how the hack works, and always import from ._distutils in setuptools instead.

Additional context

https://github.com/pypa/setuptools/pull/4504 (see failures for Python 3.12 and 3.13)
https://github.com/pypa/setuptools/issues/2345#issuecomment-1627561833

Code of Conduct

[X] I agree to follow the PSF Code of Conduct

Thank you very much @Avasam for having a look on this.
I will leave the final call on this to Jason.
But my first impression is that this is a bit too intrusive to be handled in this repo. (We don't even maintain distutils, so it would not make much sense to maintain distutils-stubs, if anywhere, pypa/distutils would be a best place for them).
Now the approach in https://github.com/pypa/setuptools/pull/4704 is slightly better, but it still introduces a lot of noise and some level of extra maintenance. It would be nice if mypy instead had a configuration that allowed us to point where the distutils package should be loaded from, to workaround mypy general limitation regarding meta path finders.
I wonder if there would be another approach...
Let's assume that we all are willing to move setuptools/_distutils to setuptools/_private/distutils or something similar. In this case, would simply adding the configuration mypy_path = $MYPY_CONFIG_FILE_DIR/setuptools/_private enough to solve the problem? (i.e. would mypy accept the setuptools/_private/distutils directory as a source of truth for the typings in distutils, or we would still need to add those .pyi files?).
(Moving setuptools/_distutils to a different directory is still something that we have to discuss and I don't know if Jason would be OK with that, but it is certainly better than adding 50-something extra .pyi files).

I agree that in the form of #4691, setuptools isn't the place to host these stubs. It is however, the best place for a PR to show the effects. (which probably means I should mark it as draft).
If going with that solution, I agree that it should live somewhere else (preferably with pypa/distutils, if it doesn't introduce distribution issues).

#4704 is different though, as setuptools is responsible for the distutils hack. From a separation of concerns pov, pypa/distutils shouldn't need to be aware that setuptools implements it using import hacks nor the exact folder name it lives in. In other words, setuptools is responsible for the distutils hack, and it should be responsible for the stub aliasing distutils to setuptools._distutils (or w/e folder location).
In the hypothetical that setuptools/_distutils is renamed to setuptools/_private/distutils, yes it means that locally for development, we no longer need a stub. But users of setuptools will still need one. And that's true as long as the distutils hack exists and as long as setuptools can't point to distutils as a regular dependency (so once Python 3.11 support is dropped ?). One advantage of the folder rename, is that the alias stubs could be generated on source install/build and included in wheels, so #4704 could potentially be updated to not have the stubs be part of the git source (it already includes the script to generate them)

In all cases, setuptools should be responsible to ship stubs-distutils, so that it gets installed (and maybe uninstalled) with it, even if it's only though an extra.


CSE: track when sourcing local dependencies

What this PR does / why we need it: Let's keep track of CSE pathways that use local files
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

CSE: track when sourcing local dependencies

@jackfrancis I added etcd + cc-runtime and removed the temp download URLs from the log statements

/lgtm

/lgtm


OCPBUGS-19834: use pull secret content for config hash generation in …

What this PR does / why we need it: use pull secret contents to create config hash for node pool, this allows for nodes to redeploy using the updated pullsecret
Which issue(s) this PR fixes
Fixes #  OCPBUGS-19834
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/jira refresh

/close


Can't bind to 'ngGrid'

Hello I have the following error

Can't

bind to 'ngGrid' since it isn't a known property of 'div'. ("
find my package.json
{   "name": "inspinia",   "version": "2.7.0",   "license": "Wrapbootstrap",   "angular-cli": {},   "scripts": {     "start": "ng serve",     "lint": "tslint \"src/**/*.ts\"",     "test": "ng test",     "pree2e": "webdriver-manager update",     "e2e": "protractor"   },   "private": true,   "dependencies": {     "@angular/common": "^2.4.9",     "@angular/compiler": "~2.4.*",     "@angular/compiler-cli": "~2.4.*",     "@angular/core": "~2.4.*",     "@angular/forms": "~2.4.*",     "@angular/http": "~2.4.*",     "@angular/platform-browser": "^2.4.9",     "@angular/platform-browser-dynamic": "~2.4.*",     "@angular/router": "^3.4.9",     "angular2-jwt": "^0.1.28",     "animate.css": "3.1.1",     "bootstrap": "^3.3.7",     "core-js": "^2.4.1",     "font-awesome": "^4.7.0",     "jquery": "^3.1.0",     "metismenu": "^2.5.0",     "rxjs": "^5.2.0",     "ts-helpers": "^1.1.1",     "zone.js": "^0.7.7"   },   "devDependencies": {     "@angular/cli": "1.0.0-beta.32.3",     "@types/jasmine": "^2.2.30",     "@types/node": "^6.0.42",     "codelyzer": "1.0.0-beta.1",     "jasmine-core": "2.4.1",     "jasmine-spec-reporter": "2.5.0",     "karma": "1.2.0",     "karma-chrome-launcher": "^2.0.0",     "karma-cli": "^1.0.1",     "karma-jasmine": "^1.0.2",     "karma-remap-istanbul": "^0.2.1",     "protractor": "4.0.9",     "ts-node": "1.2.1",     "tslint": "3.13.0",     "typescript": "~2.1.6"   } }
do you see any problem?

Have you included the NgGridModule in your app's module? That and set up the system.js config links? Are you using a regular Angular 2 app or the angular cli setup?

yes its included, I use the cli setup

@ybensacq did you find a solution?

@shadymostafa yes but I dont remember if the problem was on my side.


Add an option to publish to NuGet or GitHub Packages

This pull request add supports for publishing to GitHub Packages (or NuGet, either of those)

Codecov Report

Merging #28 (852e3de) into master (f28943a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #28   +/-   ##
=======================================
  Coverage   13.35%   13.35%
=======================================
  Files           4        4
  Lines         292      292
  Branches       32       32
=======================================
  Hits           39       39
  Misses        250      250
  Partials        3        3

Continue to review full report at Codecov.

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


Add LICENSE

Hi,
Thanks for this project.  Would you please add a LICENSE file?  I'm experimenting with using SIMD to speedup bsearch() in HarfBuzz, and like to borrow your measurement infrastructure into HarfBuzz as I experiment.
https://github.com/harfbuzz/harfbuzz/issues/566
Thanks,
behdad
cc @ebraminio

Do you prefer any particular license?  Would you mind sending a PR?

Thanks. I like "old MIT" or variants for their simplicity. Will send PR.
You also have me lots of ideas to experiment with. So far I implemented a 9ary search but have failed to see any speedup from it, so need to do proper benchmarking. But yeah didn't occur to me to combine that with a simd linear search.

Or there might be a point-and-click way for you on github website already...

Thanks for the nice work! Have a look at https://github.com/schani/linbin/community/license/new?branch=master&template=mit guess that should work, not sure if that is "old MIT", guess copying from https://github.com/harfbuzz/harfbuzz/blob/master/COPYING should work best also :)

Wow, I didn't know GitHub had a UI for that! :-)
If you're interested in fast searching, you should read this paper which goes beyond what I did here.

Thanks!  Will read that paper.
Our workload involves very small datasets so a lot of research doesn't apply.


No flow pathline with MPAS-O data set

Open VAPOR, import MPAS-O data: /little_glade/data/Source/MPAS/MPAS-O
Create a flow renderer, and enable it. Nothing gets rendered. Adjusting variable names doesn't seem to help.

This particular data set and variable upon opening the software has interesting field values that ain't technically missing values, but probably are. More specifically, there are field values as big as 5.14008662e+31 but the official missing values are inf. As a result, calculations based on values that big blew up.


chore(messaging-documentation) - Add GH page for Message Router

Changes:

Added GH page for Message Router

FYI I'm checking a local build with the changes at the moment.

@psmulovics @lilla28 This is looking good to me based on the local build:

@mimiflynn feel free to verify as well :)


DeleteMessage doesn't delete the message

DeleteMessage doesn't delete the message it Archives It..
anyway to actually delete it or move it to the trash folder then empty the trash??

FYI this project is dead. You should seriously consider switching over to MailKit. You are wasting your time continuing to use this library.


Adding the ability to pass dataloader related kwargs in SNPE, SNRE and SNLE

NOTE: The majority of the code changes are formatting related changes, which came up when I ran black and isort on the repo (as instructed in the contribution guidelines)
The real code changes are in the following files:

inference/snpe/snpe_base.py
inference/snre/snre_base.py
inference/snle/snle_base.py

In each of these files, I have added a **dataloader_kwargs parameter to be passed to the train method. I am imagining this to be "additional" args to be passed to the dataloader, over and above the ones that are already being used to build the train and validation dataloaders. In my specific case, this would be a collate_fn. I thought that it would be a waste to make the user specify all the args to dataloader here, as most of the default args being used right now are great - any additional args can be passed using **dataloader_kwargs.
I tried testing this code change with SNPE - however, snpe_c.py has a relatively complicated way of calling the train method of snpe_base using kwargs built here: https://github.com/mackelab/sbi/blob/main/sbi/inference/snpe/snpe_c.py#L150
So when I pass any dataloader related kwargs to snpe_c.train, those aren't recognized in the train method's signature. Since there's a complicated way in which the snpe_base.train is being called from here, I didn't want to mess with it before getting your opinion first.
Let me know how I should proceed, and if you have any other suggestions/questions about my approach!
closing #435

Many thanks for the PR, I will have a detailed look soon.
A first thing, I noticed a lot of formatting changes, which is great. Would it be possible to move all the formatting into a separate commit? Then we can see more clearly what was done when, where and why.

Hi there! Thanks for creating the PR! Just a quick heads-up: we just merged #431 which will generate merge conflicts. Please let us know if you need assistance in rebasing your branch :)
Michael

Thanks for the heads-up @michaeldeistler and for the comments @janfb - I will merge the current main branch from the sbi repo and update my changes in the next couple of days!

@janfb @michaeldeistler
I have been able to add in the dataloader_kwargs as a dict to the _base files of snpe, snre and snle, as well as the specific classes that inherit from those base classes (snpe_c, snle_a, etc). I overwrite the default train and val dataloader kwargs using dict here - this way the user can add a new kwarg to be passed to the dataloaders (or overwrite any of the default kwargs as well). I have no problem calling train with the additional dataloader_kwargs.
However, there are 2 additional issues that I now run into, despite being able to equip the dataloaders with a collate_fn. As I had explained earlier, my simulations consist of unequal length time-series, which have 5-D observations. I prepare the simulations as a 3-D tensor, with size (num_simulations x length of the longest series X 5). Observations in the shorter time-series in that tensor are filled with NaNs. Then, in my collate_fn, I take the longest time-series in each batch, cut all the other simulations in that batch to that length, and fill in any NaNs in the shorter time series with appropriate values. So at the end of the collate_fn, my x has a size (batch_size X length of longest series X 5) - the most important thing to note is that length of longest series naturally varies from batch to batch. My embedding net (a 1D CNN with Global max pooling) is designed to eventually produce a tensor of fixed size (batch_size X 32) from any such batch, irrespective of the length of the longest series in the batch.
Now, using this setup with the collate_fn, here are 2 persisting issues I have - both of them happen while constructing snpe_base._neural_net:

The neural net construction (here) takes the full tensor of simulations (in my case, the big consolidated initial tensor containing NaNs) to deduce the dimensionality of the embedding_net instead of just using a batch - thus, it ignores the collate_fn completely, which is a problem. It is an additional problem for me as my collate_fn actually permutes a couple of the axes of the simulations of the batch (to bring it to the shape needed for the 1D convolutions), and not taking that into account breaks the neural net creation process.

I am able to hack and fix 1 by putting my consolidated, NaN-containing initial tensor of simulations with the same order of axes that the collate_fn would have produced, but that in turn leads to problems while doing z_score_y here - the z-scoring, just like point 1 above, also works on the consolidated initial tensor (and not the simulations from a batch that have passed through the collate_fn) and is unable to deal as a result with the varying lengths of my time-series.

In the end, I was able to run the training by turning off z_score_y, but the training seems to fail without it (I get a best validation performance of -inf). The z scoring seems is probably important in my case as the values of the 5-D observations of my time-series can vary across a couple orders of magnitude, and the neural net is probably unable to deal with that kind of variability :|
Let me know how you think I should go about dealing with both these issues (and also, let me know if you have any other thoughts on why I might have gotten -inf log prob when I turned z_score_y off (of course, I couldn't train with z_score_y on in the first place)). Thanks again for all your help in advance!

Actually, I dug a bit more into the -inf issue, and it seems like it happens because snpe_base._loss (which in turn calls snpe_base._neural_net._log_prob) returns NaNs in my case :( I went and looked at the _log_prob in nflows, and the NaNs first come up here when calling _transform: https://github.com/bayesiains/nflows/blob/master/nflows/flows/base.py#L39
I checked that the output of the _embedding_net seems fine and I am not passing any NaNs by mistake in the batches, so I'm a little out of depth on what might be happening. Do you have any thoughts?

Hi @narendramukherjee and thanks for the update.
I see the problems, e.g., the build_maf function in sbi/neural_nets/flows will pass a single x through the embedding net to infer the final embedded shape of x without using the collate_fn.
At this stage, however, I think this is quite a special case which is unlikely to occur for too many users. So I suggest we find a special solution for your problem, not a general one in sbi.
I suggest to keep the changes you suggest in this PR, i.e., the possibility to pass the data loader kwargs. Thus, we will finish reviewing this PR and merge it and work on a solution for your problem separately. OK?

I can think of at least two options now.

you can pass your own build_neural_net function to sbi. Thus, you could just take our code from neural_nets/flows/ and change it such that it takes care of your case with the varying length of the time series.
The problem with the z-scoring and the NaNs is more difficult. You would have to turn off z-scoring because it will otherwise either ignore all data entries with NaNs or have NaNs in the transform.

Therefore, the other option might be better:
2) You embed the data yourself, not using the joint training in sbi. Giving that you have time series data, you could train a RNN architecture or an autoencoder unsupervised in order to learn an embedding of x. Once trained, you take, e.g., the encoding network of your autoencoder and pass all time-series through it to obtain the embedded_x. And then you use sbi without embedding net and pass as training data theta, embedded_x.
Does that make sense?

Hi @janfb : Thank you for those comments, yes, they do make sense. For now, I am also planning to just pad the whole set of simulations to a single fixed length, and that case works without a collate_fn. I need some results for a presentation next week, so I will just go with this approach for now and come back to the padding in batches with collate_fn idea in a few days time :)
So yeah, let's evaluate this PR and merge it if it looks okay to you for now.
Just a final q: in your experience, how important do you think the z-scoring of the simulations is and what are some pros/cons of doing it vs not doing it?

OK, sounds good.
to finish the PR I would ask you to rebase and squash your commits maybe into 2 separate commits, one with all the changes related to the data loader kwargs, and one with all the "unrelated" formatting changes?
regarding the z-scoring: under normal circumstances z-scoring never hurts, but only helps the NN with learning. But it can hurt when the data has many strong outliers, then it squashes potentially important differences in small values, or when the data contains NaNs like in your case, then it produces NaNs.

I had a quick look and tried to rebase your branch on main. But this turns out to be quite complicated.
Therefore, I think it would be easier, if you make a new branch from the current state of main and then basically redo your changes to add the data-loader kwargs on top of that.
Then you can also try to separate those changes from the formatting changes, i.e., make a first commit with all the formatting, and then a commit with the data-loader kwargs.
Would that be possible for you?


Update to SSPK Final Licensee List

@v-ryclap : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

Hi @v-ryclap - If you are a vendor at Microsoft. your GitHub account should be linked to your Microsoft account. The steps are described in the internal Microsoft Docs contributor guide. Let me know if you need information about how to do this; I can send it via email. Thanks.

Hello Kristine,
If you would be able to send that documentation over, I would appreciate it. Thank you for your help!
Best regards,
Ryan Clapp | IP Licensing Administration
Microsoft Corporation (TEKsystems) | ipla<EMAIL_ADDRESS>PlayReady | Components | Licensee Portal
[Microsoft]
From: Kristine Toliver<EMAIL_ADDRESS>Sent: Tuesday, September 29, 2020 9:51 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Ryan Clapp (TEKsystems Inc<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Update to SSPK Final Licensee List (#63442)
Hi @v-ryclaphttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fv-ryclap&data=02|01|v-ryclap%40microsoft.com|2c3c543adff54bb1096808d86497e0fc|72f988bf86f141af91ab2d7cd011db47|1|0|637369951348239231&sdata=iBPw%2F222k2lAYr4npU1kh62p6ZbB8HpPnkIla%2BB75vs%3D&reserved=0 - If you are a vendor at Microsoft. your GitHub account should be linked to your Microsoft account. The steps are described in the internal Microsoft Docs contributor guide. Let me know if you need information about how to do this; I can send it via email. Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F63442%23issuecomment-700842870&data=02|01|v-ryclap%40microsoft.com|2c3c543adff54bb1096808d86497e0fc|72f988bf86f141af91ab2d7cd011db47|1|0|637369951348239231&sdata=aJYztDXEAoIOiMKmre9iebEAWnfT2V4iBDtDhnT900g%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FARFXEH4LQW6PVJ6JS4QHYH3SIIGAFANCNFSM4R5EXPZQ&data=02|01|v-ryclap%40microsoft.com|2c3c543adff54bb1096808d86497e0fc|72f988bf86f141af91ab2d7cd011db47|1|0|637369951348249221&sdata=5zW7ufPPJW4rUtKZbeHlyX1hNMkUVj5gGDqGsbYako0%3D&reserved=0.

#sign-off


Learning DevOps

This PR has a conflict, please check

@Darshnadas You are committing 3 commits to the same pull request. Try making the commit one by one. Use git reset to reset the changes. Make a new pull request for each day. You can also use branching for each day and then create a PR from each branch.

@Darshnadas can you fix it ?

@kranurag7 Understood now, thank you for explaining

@saiyam1814 Raised this PR from map branch already that's why not removing it, from next time will create different branch everyday and raise a PR

@Darshnadas Always glad to help. Now the PR looks much better. Thank you for Restructuring.

LGTM


Error: A non-null value must be returned since the return type 'List' doesn't allow null.

🐛 Bug Report

Simply by importing the plugin I get this error message, anyone else experiencing this issue? :
"Error: A non-null value must be returned since the return type 'List' doesn't allow null."
Flutter Version: 2.10.5
Platform:

Android

Hi @lbesiche, thank you for reaching out!
This looks to be a NNBD issue. Are you experiencing this issue on Flutter 3 as well? Which version of the plugin are you using?


failure on installation

try to install spinningup in Ubuntu=18.04 Python=3.6 Anaconda3=-5.3.0, but following error occurs:
conda create -n spinningup python=3.6
conda activate spinningup
sudo apt-get update && sudo apt-get install libopenmpi-dev
git clone https://github.com/openai/spinningup.git
cd spinningup
pip install -e .

Then:
ERROR: gym 0.14.0 has requirement cloudpickle~=1.2.0, but you'll have cloudpickle 0.5.2 which is incompatible.
If I type pip install -e . again, then I actually see Successfully installed spinup
If I try to upgrade the cloudpickle to 1.2.0, then following absurd error occurs:
ERROR: spinup 0.1.2 has requirement cloudpickle==0.5.2, but you'll have cloudpickle 1.2.1 which is incompatible.
Then, try to run
python -m spinup.run ppo --hid "[32,32]" --env LunarLander-v2 --exp_name installtest --gamma 0.999
and see:
Illegal instruction (core dumped)
Appreciate it if anyone could help. Thanks.

I found that the issue of Illegal instruction (core dumped) is caused by my CPU. It seems it does not support the latest version of Tensorflow.


wifi, ble and javascript

This is a new article idea that I have been discussing with @shwetank. After scanning through the instructions, we decided to create a new issue here.
I am planning to complete this article by end February with the following outline:
Title: Wifi, BLE and JavaScript

Introduction

current landscape of interacting with sensors using wifi, ble and other protocols
pre-requisites required
brief outline of the project
list of hardware components required

Wifi with Spark

JavaScript SDK
Voodootiki Spark

BLE with TI Sensor Tag

various npm like noble

Conclusion

links to on-going specifications to lookout for e.g. web bluetooth
links to community initiatives such as NodeBoats or NodeBots

Throughout the article it will include pics (or gif) and videos and sectioned paragraph for better reading. Also, I will create the article instruction as a tiny project / demo for easier understanding. You can checkout some of my previous article samples here.
Let me know what you think :smiley:

Hey Sanayee,
The topic looks great.  Let's do this. Looking forward to reading the draft. :thumbsup:


Add stricter checking to "from ... import ..."

Currently token parsing on "from ... import ..." is rather loose — it sees the following invalid code as perfectly valid:
{% from "functions" import my_function, %}
{% from "functions" import, %}
{% from "functions" import %}

This is caused by the parser ignoring non-name values where there should be names, either as the first value or after commas.
This PR ensures only name values are allowed as the first value and any values after commas in the import section.
Sadly this may be a "semi-breaking" change, as it removes some functionality that should have never existed and hopefully nobody relies on.

Basically needs tests.

@jackwilsdon  are you still interested in your PR and willing to provide tests?


Added ENABLE_EMAILS env var to .env.example.

What this does
Resolves issue #186 by adding the ENABLE_EMAILS env var to .env.example.
Checklist

[x] Add the ENABLE_EMAILS env var to .env.example

How to test
Check the .env.example file.

Sorry about that. I just moved it so that it applies to both cases.


Can't upgrade to 4.11 and beyond

Issue Summary
When running ghost update I've gotten the same error after several attempts
To Reproduce

run ghost update
Get the following output:

✔ Fetched release notes
✔ Downloading and updating Ghost to v4.16.0
+ sudo systemctl stop ghost_benjaminbrandt-com
✔ Stopping Ghost
✔ Linking latest Ghost and recording versions
+ sudo systemctl start ghost_benjaminbrandt-com
+ sudo systemctl stop ghost_benjaminbrandt-com
✖ Restarting Ghost
A GhostError occurred.

Message: Ghost was able to start, but errored during boot with: alter table `oauth` add constraint `oauth_user_id_foreign` foreign key (`user_id`) references `users` (`id`) - UNKNOWN_CODE_PLEASE_REPORT: Referencing column 'user_id' and referenced column 'id' in foreign key constraint 'oauth_user_id_foreign' are incompatible.
Help: Error occurred while executing the following migration: 01-add-oauth-user-data.js
Suggestion: journalctl -u ghost_benjaminbrandt-com -n 50

Debug Information:
    OS: Ubuntu, v20.04.3 LTS
    Node Version: v14.17.6
    Ghost Version: 4.16.0
    Ghost-CLI Version: 1.17.3
    Environment: production
    Command: 'ghost update'

Technical details:
OS: Ubuntu, v20.04.3 LTS
Node Version: v14.17.6
Ghost Version: 4.16.0
Ghost-CLI Version: 1.17.3
Environment: production
Command: 'ghost update'

This seems like a bug in the migration,which is why I posted it.


feat: change some labels WIP

:tada: This PR is included in version 1.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Feature Request | Ability to add frame filtering to allow recording, monitoring and observation of audio

Hello,
The project promises to be awesome! Bravo!
This is not an issue as I have not tested yet in one of my projects, but a question.
The main question is:
How to achieve this (from streamingKit), as your framework is not yet documented:
audioPlayer.appendFrameFilter(withName: "CustomEQFilter") { (channelsPerFrame, bytesPerFrame, frameCount, frames) in
// Here I update my VU-Meter class for display.
}

Thanks,

Hey
This is not yet implemented, it's shouldn't be hard to implement though.
I'll explore the options, as this can be done as in StreamingKit or by installing a tap on the mainMixer of AVAudioEngine, the later might be a better option.

Hi @dimitris-c ,
I also use this feature for recording:
    func record() {
        let outputUrl = URL(fileURLWithPath: NSSearchPathForDirectoriesInDomains(.documentDirectory, .userDomainMask, true)[0]).appendingPathComponent("output.m4a")
        var audioFile: AVAudioFile? = nil
        self.audioPlayer.addFrameFilter(withName: "RecordFilter", afterFilterWithName: "CustomEQFilter") { (channelsPerFrame, bytesPerFrame, frameCount, frames) in

            let sampleRate: Double = 44100

            if audioFile == nil {
                let settings = [
                    AVFormatIDKey: kAudioFormatMPEG4AAC,
                    AVSampleRateKey: sampleRate,
                    AVNumberOfChannelsKey: channelsPerFrame
                    ] as [String : Any]

                do {
                    audioFile = try AVAudioFile(forWriting: outputUrl, settings: settings, commonFormat: .pcmFormatInt16, interleaved: true)
                } catch {
                    let error = error as NSError
                    print("Error: \(error.localizedDescription)")
                }
            }

            let format = AVAudioFormat(commonFormat: .pcmFormatInt16, sampleRate: sampleRate, channels: channelsPerFrame, interleaved: true)
            let buffer = AVAudioPCMBuffer(pcmFormat: format!, frameCapacity: frameCount)

            buffer?.frameLength = frameCount
            memmove(buffer?.int16ChannelData?[0], frames, Int(frameCount * bytesPerFrame))

            do {
                if let buffer = buffer {
                    try audioFile?.write(from: buffer)
                }
            } catch let error as NSError {
                print("Error: \(error)")
            }
        }
    }

Hi @dimitris-c ,
What about this enhancement? Do you think you will implement this feature soon?
Thanks,

Yeah I have this one planned will do once I get a bit free time

Ping

@iDevelopper See #18

@dimitris-c,
I see! If I understood correctly, I have to do like this:
    init() {
        player = AudioPlayer(configuration: .init(enableLogs: true))
        player.delegate = self

        configureAudioSession()
        registerSessionEvents()

        let filterEntry = FilterEntry(name: "NewEntry") { pcmBuffer, time in
            print(("buffer: \(pcmBuffer) - time: \(time)"))
        }
        player.frameFiltering.add(entry: filterEntry)
    }

I just need to find how to find the equivalent of the channelsPerFrame, bytesPerFrame, frameCount, frames of STKStreamingKit variables:
audioPlayer.appendFrameFilter(withName: "CustomEQFilter") { (channelsPerFrame, bytesPerFrame, frameCount, frames) in
// Here I update my VU-Meter class for display.
}```

Thanks!

The values you're referring can be retrieved from the output format of the main mixer node like this
let format = player.mainMixerNode.outputFormat(forBus: 0)
format.basicStreamDescription.mChannelsPerFrame
format.basicStreamDescription.mBytesPerFrame

@iDevelopper I see you're also using .pcmFormatInt16 as the common format.
Note that this has change the common format is now .pcmFormatFloat32 so be sure to use the floatChannelData value of the AVAudioPCMBuffer, if you need access to this.

If I replace .pcmFormatInt16 by .pcmFormatFloat32 in the record() function then there is a crash while:
            //memmove(buffer?.int16ChannelData?[0], frames, Int(frameCount * bytesPerFrame))
            memmove(buffer?.int32ChannelData?[0], frames, Int(frameCount * bytesPerFrame))

    func record() {
        let outputUrl = URL(fileURLWithPath: NSSearchPathForDirectoriesInDomains(.documentDirectory, .userDomainMask, true)[0]).appendingPathComponent(recordFileName)
        var audioFile: AVAudioFile? = nil
        self.audioPlayer.addFrameFilter(withName: "RecordFilter", afterFilterWithName: "CustomEQFilter") { (channelsPerFrame, bytesPerFrame, frameCount, frames) in

            let sampleRate: Double = 44100

            if audioFile == nil {
                let settings = [
                    //AVFormatIDKey: kAudioFormatMPEG4AAC,
                    AVFormatIDKey: kAudioFormatLinearPCM,
                    AVSampleRateKey: sampleRate,
                    AVNumberOfChannelsKey: channelsPerFrame
                ] as [String : Any]

                do {
                    //audioFile = try AVAudioFile(forWriting: outputUrl, settings: settings, commonFormat: .pcmFormatInt16, interleaved: true)
                    audioFile = try AVAudioFile(forWriting: outputUrl, settings: settings, commonFormat: .pcmFormatFloat32, interleaved: true)
                } catch {
                    let error = error as NSError
                    print("Error: \(error.localizedDescription)")
                }
            }

            //let format = AVAudioFormat(commonFormat: .pcmFormatInt16, sampleRate: sampleRate, channels: channelsPerFrame, interleaved: true)
            let format = AVAudioFormat(commonFormat: .pcmFormatFloat32, sampleRate: sampleRate, channels: channelsPerFrame, interleaved: true)
            let buffer = AVAudioPCMBuffer(pcmFormat: format!, frameCapacity: frameCount)

            buffer?.frameLength = frameCount
            //memmove(buffer?.int16ChannelData?[0], frames, Int(frameCount * bytesPerFrame))
            memmove(buffer?.int32ChannelData?[0], frames, Int(frameCount * bytesPerFrame)) **// What should I do here?**

            do {
                if let buffer = buffer {
                    try audioFile?.write(from: buffer)
                }
            } catch let error as NSError {
                print("Error: \(error)")
            }
        }
    }

The values you're referring can be retrieved from the output format of the main mixer node like this

Ok,
Can this values can be retrieved in pcmBuffer.format too?

If I replace:
            memmove(buffer?.int16ChannelData?[0], frames, Int(frameCount * bytesPerFrame))

by:
            memmove(buffer?.floatChannelData?[0], frames, Int(frameCount * bytesPerFrame))

No crash, but I try to play the recording with STKStreaminKit AudioPlay, it is inaudible. Only weird sounds.

Don't use int32ChannelData but floatChannelData, I suppose you've checked out the feature branch which includes the FrameFilter work.
Also note that you don't need to re-create the buffer and move use memmove...
The FilterCallback will return a AVAudioPCMBuffer so you can pass that in the write(from:) method of AVAudioFile
And yes the format value should be the same as the mainMixerNode.outputFormat(forBus: 0)

The branch name is feature/frame-filters, experiment with the changes from this one

I was talking about my actual code with STKStreamingKit.
Also I have checked the feature/frame-filters branch. But I have just tried for now the add filter feature not the recording one.
.pcmFormatFloat32 does not work with STkStreamingKit, perhaps because this library is out of date?

I will try to duplicate my project and replace STKStreamingKit by your framework...

Any code or changes I've mentioned are only related to this library, not StreamingKit...

Yes I have understand. Help, how to get the frameCount value, please? And the void *frames equivalent? You can see I am bad in Audio!...

I believe all the information can be retrieved from the buffer property
AVAudioPCMBuffer extends AVAudioBuffer in which you can find the audioBufferList from there you can extract the values you want
For the void *frames equivalent would be buffer.audioBufferList.pointee.mBuffers.mData
Mind if I ask why the need for frameCount?

Thanks,
I think I wouldn't need it anymore (frameCount). In fact for vu-meter? I use the FreeStreamer (FSFrequencyDomainAnalyzer) (https://github.com/muhku/FreeStreamer) which I modify a bit to show a graph. You can download my app or see the screenshots of my app on the App Store to see (https://apps.apple.com/fr/app/shout-radios/id1474971083#?platform=iphone). For recordings, I will also allow the user to modify the recording to trim, in a future release. I am using  EZAudioPlot for that, but deprecated. It would be nice to have a version in your awesome library, but that will be the subject of a new feature request.

In any case, thank you very much for this library and for your availability!

This is resolved by #18


Issue transforming Opta data

Bug report by @MKlaasman
Notebook to reproduce the bug: https://github.com/MKlaasman/error-kloppy/blob/main/error Kloppy 2.1.0.ipynb

Fixed in https://github.com/PySport/kloppy/commit/39942df6dbc482d2eab237e35cc967effc9451ca


vector search enabled MemoryDB cluster via terraform module

via aws-cli we can create the cluster with vector search enabeld by passing the parameter group default.memorydb-redis7.search.preview, as stated in:
To create a vector search enabled MemoryDB cluster, you can use the MemoryDB [create-cluster](https://docs.aws.amazon.com/cli/latest/reference/memorydb/create-cluster.html) command by passing an immutable parameter group default.memorydb-redis7.search.preview to enable the preview mode for vector search capabilities.

However, when I do that via terraform:

module "memory_db" {
  source = "terraform-aws-modules/memory-db/aws"

  engine_version            = "7.1"
  node_type                    = "db.r7g.2xlarge"
  num_shards                 = 1

  # Parameter group
  create_parameter_group = false
  parameter_group_family = "default.memorydb-redis7.search.preview"
 (...)
 (...)
}

it's not enabled.
I've confirmed it:
-ERR Command not enabled, instance needs to be configured for Public Preview for Vector Similarity Search

I have the same problem, can anyone help us, please?

I don't think you are following the docs correctly, I think you need to do:
module "memory_db" {
  source = "terraform-aws-modules/memory-db/aws"

  engine_version = "7.1"
  node_type      = "db.r7g.2xlarge"
  num_shards     = 1

  # Parameter group
  create_parameter_group = false
  parameter_group_name   = "default.memorydb-redis7.search.preview "
  ...
}

I don't think you are following the docs correctly, I think you need to do:
module "memory_db" {
  source = "terraform-aws-modules/memory-db/aws"

  engine_version = "7.1"
  node_type      = "db.r7g.2xlarge"
  num_shards     = 1

  # Parameter group
  create_parameter_group = false
  parameter_group_name   = "default.memorydb-redis7.search.preview "
  ...
}

I'm getting this error from terraform apply : InvalidParameterValueException: Parameter Group ID specified is not a valid identifier. Identifiers must begin with a letter; must contain only ASCII letters, digits, dots and hyphens; and must not end with a dot or hyphen or contain two consecutive dots or hyphens.

oh, there was an extra whitespace character in the snippet I provided - corrected

also, you shouldn't need parameter_group_family = "memorydb_redis7" - thats only used if you are creating a parameter group

awesome! closing this out then, thanks all!


mark.js is using 8.11.0 vs 8.11.1

General Support Request
Auro menu is using mark.js v 8.11.0 and we received an automated PR to update to 8.11.1 https://github.com/AlaskaAirlines/auro-menu/pull/153
The 8.11.1 update is failing to build.
Support request
Question: 8.11.1 was released 5 years ago, why did we use 8.11.0?
Review and address the build error
TypeError: Cannot set properties of undefined (setting 'Mark')

148 files changed for a patch change, but ok https://github.com/julmot/mark.js/compare/8.11.0...8.11.1

Duplicate of #189


Relations are not picked up if they are named "parent" or "children"

Since "parent" and "children" are reserved keywords in gatsby, this plugin automatically appends "strapi_" to the relation name. Since the relations are now named "strapi_parent" and "strapi_children" the code on this line doesn't work, https://github.com/strapi/gatsby-source-strapi/blob/2a60320b16ae6520617d743158c433d384dbd155/src/download-media-files.js#L117, since the relations on the strapi schema are named just "parent" and "children", but attributeName is "strapi_parent" and "strapi_children".
This problem is related to the other issue I have with relations: https://github.com/strapi/gatsby-source-strapi/issues/308, because without changing the code to schema.schema.attributes[attributeName.replace('strapi_', '')] the fix I illustrated in the other issue doesn't work because the fields generated from this plugin and the schema on strapi are different

@denisgoryaynov the parent and children names are forbidden in gatsby this is why we prefixed them with strapi_ same for the id field.

Hi @soupette, I understand that those names are forbidden in gatsby and that's why they are prefixed with strapi_, the problem is that this change is not reflected when getting the data from the strapi schema when downloading images.
Inside this line of code const attribute = schema.schema.attributes[attributeName]; the schema.schema.attributes has the children and parent fields (because it is fetched from the content builder api), but they are accessed using the prefixed fields strapi_children and strapi_parent, thus returning undefined every time


Added fail event management to dataflow

When the dataflow's mapper and reduce function returns:
{
  error: true
}

it will redirect the doc to the error db instead of to the sink and output db.
In summary:

When the sink returns error, in addition to the statusCode 201 the current service will returns a body with error and reasons without posting any of its docs to the output db.
When the map / reduce functions produces error, in addition to returning statusCode 201, the current service will returns a body with the error and reasons without posting any of its docs to the output db / to the sink. Instead it will post to error db.

For discussion:

The structure of the error object: (currently it is just a string.)
When the sink returns error, should the current service also post to its error db? (Each service has its own db)
An error dedicated service to manage errors instead of each service has its own db. (The set up would be similar to the sink. Each service will post to the error dedicated service instead of its own db)

Enhancement:

Add more endpoints that will allow operator to do more operations: example delete a doc after the operator fixes the issue, resubmit, or delete all.

Any feedback would be appreciated. Thanks.

Coverage increased (+0.3%) to 87.758% when pulling 1ddf000ad1c5c49e99d69801651918049df2a201 on KRuelY:dataflowErrorManage into b17cf152ebb2bc3f11a2c42499d194e128d87c43 on cloudfoundry-incubator:master.

Coverage decreased (-0.4%) to 87.13% when pulling f9a0fd72aa778836fe21af46345b9a91b3cc5b21 on KRuelY:dataflowErrorManage into b17cf152ebb2bc3f11a2c42499d194e128d87c43 on cloudfoundry-incubator:master.

For discussion:

The structure of the error object: (currently it is just a string.)

I'd suggest something similar to how Cloudant returns errors:

an error string field containing a keyword e.g. einvalidplan, eorgnoutfound
a reason string field containing a text description of the reason that caused the error

When the sink returns error, should the current service also post to its error db? (Each service has its own db)

Yes

An error dedicated service to manage errors instead of each service has its own db. (The set up would be similar to the sink. Each service will post to the error dedicated service instead of its own db)

Could be useful, but on top of the error db, as it'll be useful for each service to have its own db in distributed topologies, and posting to an error sink service could fail as well...

Enhancement:

Add more endpoints that will allow operator to do more operations: example delete a doc after the operator fixes the issue, resubmit, or delete all.

Yes, makes sense

Will submit a new pull request after making some clean up and adjustment due to commit f412896. Thanks!


Shortening a time duration annotation to a point of time annotation does not work

Trying to shorten a time duration annotation to a point of time annotation by dragging the end/beginning of the annotation in the timeline does not work. On the timeline the annotation appears as a point in time annotation but in the list of annotation the annotation has still the old start- and end-time indicated.
When pulling the start of an annotation to the end this start can be pulled further than the actual end of the annotation which ends in an point in time annotation with a time that is later in time that the initial end of this annotation. In the list of annotation this annotation is still indicated as time duration annotation with the initial start and end time.

Works as expected on SWITcast-Test.

Works almost as expected on SWITCHcast. Why almost: I expected the point-in-time annotation to appear with a anchor. See you answer here: https://github.com/opencast/annotation-tool/issues/299#issuecomment-580149602

I will look into it.

Okay I can reproduce this. In fact "converting" an annotation like this does not even seem to be persisted.

Looks fine on SWITCHcast-Staging. Thx!


Upgrade to Spine 4.x: Compatibility, Enhanced Features, and Runtime Coexistence

To fully leverage the latest technological advancements offered by Spine, and to enhance the quality and efficiency of spine  animations, we plan to upgrade to Spine 4.x.
This upgrade aims to ensure full compatibility with the latest version of Spine.
Benefits of Upgrading

Keep up with the latest version of Spine will help us taking advantage of its newest features and technological improvements.
2.Artists can now use the editor from version 4.x onwards to create Spine animations, benefiting from a slew of enhancements in the latest Spine update. Highlights include a simplified process for creating sequence frame animations, enhanced curve editing capabilities, and the addition of thrilling physics-based skeletal constraints! Additionally, we need to address the challenges associated with running multiple versions of the runtime simultaneously

done


[BUG] Netherite Tools and selecting enchants does not work

yeah i should probably fix that but i'm too lazy to do it rn so i'll do it later

did some testing on a separate branch and i couldn't replicate the bug, so i will need to do more testing on an actual game instance in order to see if this is a real bug or if i didn't see it right yesterday

yeah ok i'm blind lmao this is not a bug