GH-34882: [Python] Binding for FixedShapeTensorType

Rationale for this change
In the C++ the fixed shape tensor canonical extension type is implementated https://github.com/apache/arrow/pull/8510 so we can add bindings to the extension type in Python.
What changes are included in this PR?
Binding for fixed shape tensor canonical extension type.
Are these changes tested?
Yes.
Are there any user-facing changes?
No.

Closes: #34882

Thank you for summing it up Joris!
My vote is for option 2 ("Provide option to indicate if you want the permuted result or not") and I also do not think option 1 or 3 are good ways to go - for same reasons already mentioned.

One question is: are there good use cases for not wanting the automatic permutation? Because if so, always returning the permuted array (option 1) is also annoying because then it's less trivial to go back to the plain C array, instead of permuting the C array?

I think we would always want automatic permutation. The data being passed will always be passed zero-copy so users could always choose to interpret it with different strides/permutation, but the expectation is they would not want to do that in 99% of cases. Loosing strides/permutation information is a sharp edge imo.

An example of pyarrow -> numpy conversion that loses permutaion/strides:
import pyarrow as pa
from numpy.testing import assert_array_equal

tensor_type1 = pa.fixed_shape_tensor(pa.int8(), [2, 2, 3], permutation=[0, 1, 2])
tensor_type2 = pa.fixed_shape_tensor(pa.int8(), [2, 2, 3], permutation=[2, 1, 0])
storage = pa.array([[1, 2, 3, 4, 5, 6, 1, 2, 3, 4, 5, 6]], pa.list_(pa.int8(), 12))
arr1 = pa.ExtensionArray.from_storage(tensor_type1, storage)
arr2 = pa.ExtensionArray.from_storage(tensor_type1, storage)
narr1 = arr1.to_numpy_ndarray()
narr2 = arr2.to_numpy_ndarray()

assert_array_equal(narr1, narr2)
assert arr1.equals(arr2)

If arr2.to_numpy_ndarray() would throw an error saying this currently (we'd want to add it at some point) can't handle permuted conversion I think we'd be ok.

You don't necessarily "loose" that information, you still have it available in the extension type. It won't be embedded in the numpy array, but neither do the dimension names (of course, I know this is a bit different, as names can never be embedded in a numpy array). So in general you might still need to process the extension type attributes to ensure you correctly handle the resulting numpy array.
For example, assume you have channels-last array (logically NCHW, physically NHWC). This information could also be stored as "dim_names": ["H", "W", "C"], or only as permutation ([2, 0, 1]), or both. So if you convert such a array to numpy, you still need to check for those aspects as well, and also need to take into account that if you already got a permutated array by default from to_numpy_ndarray, the dim_names were not permuted.
(this is different compared to a Tensor, where the permuted dim names can be attached to the Tensor object)
Just to mention that there are various related aspects that can make this more complex. And so I still think it is a valid option to say for to_numpy_ndarray that it will just simply ignore all optional metadata (and thus always return the physical array), and it is up to the user to decide what to do with the optional metadata (permutation and/or dim_names).
Now, as mentioned above, I don't have a that strong opinion about it, so I am also fine if in the end to_numpy_ndarray permutes by default (in that case I think I would prefer adding an keyword to turn it off, since it's not that straightforward to convert the permuted array to its physical form).
On the short term, erroring for those ambiguous cases is also fine, that leaves all options open for finetuning this in a follow-up PR.

Just to be clear - pyarrow->numpy conversion should (imo) use permutation to create ndarray with correct strides while using the same physical layout (zero-copy) of the data buffer. I don't see any good reason to not do this in pyarrow but I might be missing something?

@jorisvandenbossche if u agree we can go ahead and merge this.

pyarrow->numpy conversion should (imo) use permutation data to create ndarray with correct strides while using the same physical layout (zero-copy) of the data buffer. I don't see any good reason to not do this in pyarrow but I might be missing something?

@rok see my comments above?
Potential reasons that someone might not want to have the permuted ndarray:

For a specific use case, someone might not care about the permutation and just wants the physical C-contiguous array
Because you want to do the handling of permutation / dim_names yourself, and therefore first want to get the numpy array as the physical array (see https://github.com/apache/arrow/pull/34883#issuecomment-1503146111 above for a possible reason for this, eg if you need to handle the potential combination of both permutation and/or dim_names, this might be easier to just all do yourself)

For a specific use case, someone might not care about the permutation and just wants the physical C-contiguous array

Wouldn't that not be possible with tensor_ext_arr.storage.to_numpy()? Or would that not be zero-copy?

Because you want to do the handling of permutation / dim_names yourself, and therefore first want to get the numpy array as the physical array (see GH-34882: [Python] Binding for FixedShapeTensorType #34883 (comment) above for a possible reason for this, eg if you need to handle the potential combination of both permutation and/or dim_names, this might be easier to just all do yourself)

How about we have pyarrow pass the permutation/strides information by default and have users use lower level API for manual handling of permutation / dim_names? I'm assuming underlying buffer doesn't get actually permuted in either case of course.

Short interruption of the continuation of the thread 😊
Thank you Joris and Rok for all the help and reviews! 🙏
The docs for this PR are ready for review: https://github.com/apache/arrow/pull/34957

For a specific use case, someone might not care about the permutation and just wants the physical C-contiguous array

Wouldn't that not be possible with tensor_ext_arr.storage.to_numpy()? Or would that not be zero-copy?

to_numpy() gives a 1D object-dtype array of arrays, so that's not convenient to work with if you want the nD array (it can actually be "zero-copy", but you can't get the zero-copy nD array from it)

How about we have pyarrow pass the permutation/strides information by default and have users use lower level API for manual handling of permutation / dim_names?

What do you mean exactly with "pass the permutation/strides information"? (I suppose you mean returning the permuted/logical array by default?)
Users could indeed do this conversion to an nD array manually (the to_numpy_ndarray implementation isn't that long), but to me that feels a bit the point of providing this method ;) (it could be a keyword argument to the function, to indicate if you want the permuted logical array or not)

What do you mean exactly with "pass the permutation/strides information"? (I suppose you mean returning the permuted/logical array by default?)

I think so yes :D.

Users could indeed do this conversion to an nD array manually (the to_numpy_ndarray implementation isn't that long), but to me that feels a bit the point of providing this method ;) (it could be a keyword argument to the function, to indicate if you want the permuted logical array or not)

I suppose having this as an opt-in behavior (with the keyword) would work yeah. Just want to make sure this is not on a default path as that would be surprising.


limit messagebox size for responsive on mobile

Please makes sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's Contributing Guide.
[ ] Make sure you are merging your commits to dev branch.
[ ] Rebase before creating a PR to keep commit history clear.
[ ] Add some descriptions and refer relative issues for you PR.

see #1561

Thanks for contributing, but Element is not for mobile web apps, so the changes here may seem a little weird. Maybe you can just override its CSS in your own project.


Query state of server/database

Some times, but only some times, OrientDB is forcibly terminated. When it is started up again AND a connection is made to a database, it will do, what I think is, some validations and integrity checks (right here it is checking and filling holes/indexes, apparently quite many). The connection will then only proceed once this process is done. In other words, connecting to the database takes really long.
Would it be possible to do something along the lines of:
(- allow fast connecting (return session id) (#1447))

query for a status (enum READYFORCOMMANDS, STARTINGUP, SHUTTINGDOWN)

Already available with the new HA STATUS commands.


start Lane detection by reading images.

This is your first task,
You can use iPython notebook
first read the images using opencv and then show that image.
Make your PR for that.

Can I take this?

what are you asking?

can I take up this issue?

Yes ofcourse!

Great work @bharath-acchu

Is this pr is closed now or we can contribute?

Done by @ashwanisng


Tests run with wrong Django version

For example, on the latest (python 3.6, django 3.0) tests run against master branch,
https://github.com/django-hijack/django-hijack/runs/3459457910
If you look in the "Install Django 3.0" section of the output you can see
Collecting django~=3.0
...
Successfully installed asgiref-3.4.1 django-3.2.6 pytz-2021.1 sqlparse-0.4.1 typing-extensions-<IP_ADDRESS>

its actually installing Django 3.2.6. This is because you are specifying django~=3.0.
So the tests aren't actually testing all the django versions they claim they are.

@LincolnPuzey Thank you. Good catch.


Support OSXFUSE 3.x, fall back to 2.x if needed

Also supports custom bundled OSXFUSE installs.
The homebrew cask osxfuse version is now 3.5.0: https://github.com/caskroom/homebrew-cask/blob/master/Casks/osxfuse.rb#L2.
This code is basically transferred from bazil.org/fuse.

First, I would like to apologize for delayed response.
Second, you are completly right in wanting to have references for these changes; stellar documentation is why I chose this library in the first place. I must have spend two days going through the commits just to verify everything that you are asking.
As for the answers:

All the variables are indeed needed.
Yes. They have changed the osxfuse file system library paths and options, even during development of osxfuse 3.x.
At first OSXFUSEPaths were internal, but someone might (at some point) have the need for custom osxfuse build, and this way it is easy to do so.

I will attempt to provide updated pull request as soon as I am able to.
Also, if you think some things should be changed, I won't mind to do so.

Thanks! Sorry for my slow response; I've been away on holiday. As for what needs to be done here:

Could you please make all of the new symbols unexported again? We can export them later if a need truly arises. I would rather have the library just do the right thing.
It sounds like you did a bunch of research on this. Could you please link (in the code comments) to your sources?

It would be best if you could do these as new commits in the pull request rather than overwriting the existing one. I may wind up pulling this as-is if osxfuse 3 support becomes urgent, so would like to be able to cleanly make these fixes afterward.

Just as I started to commit changes, you already finished.
Sorry again, this was really the only free time I've had in previous weeks.

No worries, thanks for the fix. I'm still very interested in getting references from you if you have them.


New APIs for VipSwap2.0 with slot identification for CS-ES

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
January, 2022
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
February, 2022
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[x] PowerShell
[] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that label “WaitForARMFeedback” will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[x] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.
This PR contains following changes:

New APIs for VipSwap2.0 on CS-ES - There are two new APIs(GET/PUT)  which has been added to support VipSwap through EV2.
There is new property called slotType added to cloud service schema to support slot identification on cloud services.

@msyyc please help check the python breaking changes.

❌ 1022 - RemovedAdditionalProperties	The new version removes the 'additionalProperties' element.
New: Microsoft.Network/stable/2022-01-01/network.json#L228:9
Old: Microsoft.Network/stable/2021-05-01/network.json#L219:9

If this change is intentional, please follow the Action in above comment to request breaking change approval.

❌ 1022 - RemovedAdditionalProperties	The new version removes the 'additionalProperties' element.
New: Microsoft.Network/stable/2022-01-01/network.json#L228:9
Old: Microsoft.Network/stable/2021-05-01/network.json#L219:9

If this change is intentional, please follow the Action in above comment to request breaking change approval.

I have done this change because i got a validation error that I cannot have inherited anonymous properties and I should move all such properties inside  definitions section. So I move the additionalProperties schema to definitions. This property already existed and I have not added it. I updated it because I was getting merge validation failure because I have other changes in the same file.


QA - Site - My Acc - View Order - Need to Proper Alignment of View Order Page.

QA - Site - My Acc - View Order - Need to Proper Alignment of View Order Page.

Click on search button and go to search page.
Click on any product and go tot detail page.
Place One Order.
Now Click on View Order Page.
Click on View Button From Order List.

Done.
Looking Fine Now.


num pad keys not indicated

I noticed that num pad keys are not displayed differently than their main keyboard counterparts<PHONE_NUMBER>=/*-+,).
Did I overlook something?

I don't know if there's already a consensus about this.
Keyboard Maestro uses this ⊞ Squared Plus Unicode Symbol (U+229E), which I would be fine with.
I guess prepending "num" would be too verbose.

There are some specific characters to represent the number pad keys when used for navigation. I'll probably add those in #296

I checked the new version, num pad key characters are still the same, i.e. not different to regular?


Add filter, add/remove date to the UG. Demo picture is not done.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
see 8 files with indirect coverage changes


E-mail and in-app text changes

User Story
As a Product Owner,
I want to update our automated e-mail messages and in-app text,
So that the messaging is consistent across the app and mailings.
Acceptance Criteria
Given that I am using FTL,
When I view the Sign In modal and various success and error messages,
Then Sign In modal and messages should appear as in the Google Doc linked above.
Given that I am receiving a Sign In e-mail,
When I view the e-mail,
Then it should be from "Follow the Learners"<EMAIL_ADDRESS>and the text should appear as in the Google Doc linked above.
Given that I am receiving a first learners assigned e-mail,
When I view the e-mail,
Then it should be from "Follow the Learners"<EMAIL_ADDRESS>and the text should appear as in the Google Doc linked above.
Given that I am receiving a 50% learners assigned e-mail,
When I view the e-mail,
Then it should be from "Follow the Learners"<EMAIL_ADDRESS>and the text should appear as in the Google Doc linked above.
Given that I am receiving a 100% learners assigned e-mail,
When I view the e-mail,
Then it should be from "Follow the Learners"<EMAIL_ADDRESS>and the text should appear as in the Google Doc linked above.

We are waiting for #354 to be rolled in officially before we change the text on the 50% and 100% learner reached and close this task.


How to make a custom LeRobotDataset with v2?

Hi folks, thanks for the amazing open source work!
I am trying to make a custom dataset to use with the LeRobotDataset format.
The readme says to copy the example scripts here which I've done, and I have a working format script of my own.
https://github.com/huggingface/lerobot/blob/8e7d6970eaf5a64b8af6ec45586d201b8ca9ef16/README.md?plain=1#L323
but when it comes time to create the dataset, the push_dataset_to_hub.py uses LeRobotDataset.from_preloaded which is no longer supported in dataset V2
https://github.com/huggingface/lerobot/blob/8e7d6970eaf5a64b8af6ec45586d201b8ca9ef16/lerobot/scripts/push_dataset_to_hub.py#L216
So I'm just wondering what the proper way of loading your own custom local dataset is?
Thank you in advance for your help!

okay so I've found a work around for now. I initialize an empty dataset and add the frames to it, and then I can load it after calling dataset.consolidate(). If this is a proper way to do it, pls lmk and I'll make a PR with updates to the docs.
Otherwise please let me know what the right way to do this is. Thank you! I'll update this with my code once I've cleaned it up.

I encountered the same issue.

@aliberts i also got the same issue, the documentation on how to generate custom dataset is not up to date now (the code doesn't run anymore). could you please up the instruction and relevant scripts for custom dataset generation? thanks

Hey there,
Yes, all the push_to_hub script are deprecated in favor of scripts in examples/port_datasets (just one for now).
Basically, you need to create a new empty dataset using LeRobotDataset.create(), then add individual frames using add_frame(), then save the added frames into an episode using save_episode() (which actually saves data).
Then at the end you need to call the consolidate() method to handle a few more things (we will try to get rid of this step in future iterations) before finally calling the push_to_hub() method.
You can find more info about the changes of this new api in the PR (#461)
We will remove push_to_hub.py scripts in the future after adding more equivalent scripts like the one mentioned above in the examples section. Hope this helps!

Will update the Readme soon!


Add ADR-0070 for new recommendation on cypress test structure

Summary
As part of PO7 A-Team decided on a new way to organize tests for large features. This PR adds an ADR to describe that choice and facilitate discussion about it and hopefully settle on a new pattern to ensure our tests are well organized.
this slack thread has some details.
Setup to Run Your Code
No steps, just an ADR to read
Verification Steps for Author
These are to be checked by the author.

[X] Request review from a member of a different team.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 9c3b0771a1c57025b9e77abd3c2be494c2b8bd15

A-Team plans to update the PPM tests to meet this standard, but I'm not sure about the others. It does sound like a good topic for the engineering check-in meeting.


70 automatically signin user who creates signin

change user to be the person executing command, should be something like i.Member.User.ID
change span context to match execution block (use span instead of span_signinSlug)

I will look closer when I get home. But looks like you did not change existing interaction response, so it will
cause an error because they are trying to respond to the same interaction.

Yeah, I am aware of that. Was going to communicate with you about that, just confused about it.


pycairo error

fix for pycairo error based on https://github.com/mesonbuild/meson/issues/2273#issuecomment-420412230
Thanks for contributing to manim!
Please ensure that your pull request works with the latest version of manim.
You should also include:

The motivation for making this change (or link the relevant issues)
How you tested the new behavior (e.g. a minimal working example, before/after
screenshots, gifs, commands, etc.) This is rather informal at the moment, but
the goal is to show us how you know the pull request works as intended.

pkg-config and cairo lib are only needed when the wheel installation from pypi fails. This env export is only specific to macos

The docs in master branch has been updated to a renew one, so this is outdated, I'm closing this.


[coherence] Update common files for branch master

Update common files

Java 8 not supported in next release.


Style/GuardClause suggests inefficient rewrite of my method

When I have 2 to-level unless in my function with each block having more
then 1 line, rubocop complains to use guard clause instead. While that is
possible, it would mean calling the function in the condition twice, slowing
everything done without any benefit. Or leading to some pretty ugly code.
My code looks like this:
def foo(f)
  unless File.exist?(f)
    $l.error { 'Does not exist' }
    raise 'Does not exist'
  end
  unless File.directory?(f)
    $l.error { 'Not a dir' }
    raise 'Not a dir'
  end
end

rewriting this code using guard clause is possible (but triggers other warnings)
def foo(f)
  ( msg = 'Does not exist'; $l.error { msg }; raise msg ) unless File.exist?(f)
  ( msg = 'Not a dir'; $l.error { msg }; raise msg ) unless File.directory?(f)
end

but don't tell me it actually improves the code quality.
Other way to rewrite it would be
def foo(f)
  $l.error { 'Does not exist' } unless File.exist?(f)
  raise 'Does not exist' unless File.exist?(f)
  $l.error { 'Not a dir' } unless File.directory?(f)
  raise 'Not a dir' unless File.directory?(f)
end

But that means accessing filesystem twice instead of just once.
The rule this cop references ( https://github.com/bbatsov/ruby-style-guide#no-nested-conditionals )
talks about nested conditional, I don't believe I have any of those in my code so I'm not entirely sure why it is complaining.

Expected behavior
To not complain when more then 1 top-level unless/if is used assuming
the unless/if in question has more than one line.
Actual behavior
Rubocop complains even when there are 2 top-level unlesss used.
Steps to reproduce the problem
[wolf@devel rubocop_test]$ cat test.rb
class Foo
  def bar
    unless File.exist?('/xxx')
      $l.error { 'File does not exist' }
      raise 'Does not exist'
    end
    unless File.directory?('/xxx')
      $l.error { 'File is not a dir' }
      raise 'Not a dir'
    end
  end
end
[wolf@devel rubocop_test]$ rubocop test.rb
Inspecting 1 file
C

Offenses:

test.rb:1:1: C: Style/Documentation: Missing top-level class documentation comment.
class Foo
^^^^^
test.rb:3:5: C: Style/GuardClause: Use a guard clause instead of wrapping the code inside a conditional expression.
    unless File.exist?('/xxx')
    ^^^^^^
test.rb:4:7: C: Style/GlobalVars: Do not introduce global variables.
      $l.error { 'File does not exist' }
      ^^

1 file inspected, 3 offenses detected

RuboCop version
[wolf@devel rubocop_test]$ rubocop -V
0.54.0 (using Parser <IP_ADDRESS>, running on ruby 2.5.0 x86_64-linux)

But that means accessing filesystem twice instead of just once.

It actually changed 2 filesystem accesses to 4. You can write this method with one filesystem access, btw:
class Foo
  def bar
    unless File.stat('/xxx').directory?
      $l.error { 'File is not a dir' }
      raise 'Not a dir'
    end
  rescue Errno::ENOENT
    $l.error { 'File does not exist' }
    raise 'Does not exist'
  end
end

Rubocop doesn't complain about a guard clause with this version, but this isn't a general solution to "double unless" style code in your initial example.

I believe this is still an issue in current rubocop:
$ gem list | grep rubocop
rubocop (0.59.1)

# noop
class Foo
  # noop
  def bar
    unless File.exist?('/xxx')
      l.error { 'File does not exist' }
      raise 'Does not exist'
    end
    unless File.directory?('/xxx')
      l.error { 'File is not a dir' }
      raise 'Not a dir'
    end
  end
end

$ rubocop
Inspecting 1 file
C

Offenses:

test.rb:9:5: C: Style/GuardClause: Use a guard clause instead of wrapping the code inside a conditional expression.
    unless File.directory?('/xxx')
    ^^^^^^

1 file inspected, 1 offense detected


Add 0.16.2 RPC docs

Preview: http://dg0.dtrt.org/en/doc/0.16.2/
First commit is auto generated using the contrib/doc-gen/generate.go script and should be deterministic.  Second commit just updates a version number.
CC: @karel-3d

Tested ACK 471fd33d281672bd74764d18373d4cdc3393a10e
I verified that the first commit is the result of running generate.go, and that the second commit correctly redirects to the 0.16.2 rpcs. I didn't manually check all of the text.


Use python2 binary, since python can be an alias to python3

would have been fixed in #911

I'm having issues trying to compile stuff
gyp ERR! configure error
gyp ERR! stack Error: Python executable "/opt/local/bin/python" is v3.3.6, which is not supported by gyp.
gyp ERR! stack You can pass the --python switch to point to Python >= v2.5.0 & < 3.0.0.
gyp ERR! stack     at failPythonVersion (/Applications/User/Atom.app/Contents/Resources/app/apm/node_modules/node-gyp/lib/configure.js:454:14)
gyp ERR! stack     at /Applications/User/Atom.app/Contents/Resources/app/apm/node_modules/node-gyp/lib/configure.js:443:9
gyp ERR! stack     at ChildProcess.exithandler (child_process.js:204:7)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:172:7)
gyp ERR! stack     at maybeClose (internal/child_process.js:827:16)
gyp ERR! stack     at Socket.<anonymous> (internal/child_process.js:319:11)
gyp ERR! stack     at emitOne (events.js:77:13)
gyp ERR! stack     at Socket.emit (events.js:169:7)
gyp ERR! stack     at Pipe._onclose (net.js:477:12)
gyp ERR! System Darwin 16.0.0
gyp ERR! command "/Applications/User/Atom.app/Contents/Resources/app/apm/bin/node" "/Applications/User/Atom.app/Contents/Resources/app/apm/node_modules/.bin/node-gyp" "rebuild"
gyp ERR! cwd /Users/samuel/.atom/packages/autocomplete-ctags/node_modules/ctags
gyp ERR! node -v v4.4.5
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok

Python defaults to 3.x but python2.6 and python2.7 are available. ??????????

that’s OK. PEP 394 says:

python refers to python3 on at least Arch Linux […] so python should be used in the shebang line only for scripts that are source compatible with both Python 2 and 3.
in preparation for an eventual change in the default version of Python, Python 2 only scripts should either be updated to be source compatible with Python 3 or else to use python2 in the shebang line.

#911 would fix node-gyp to do this right and use python2 if available, but still falls back to python if it’s not available.

This allows you to work around the issue for app:
apm config set python $(which python2.7)

Perhaps it works with npm too.

Just FYI:
https://groups.google.com/forum/#!topic/gyp-developer/FpZiX88f5ZU
https://groups.google.com/forum/#!topic/gyp-developer/S75A6C2e3eA
Hopefully this will be fixed upstream too, soon.

I'm going to close this because I don't think we'll change the current behavior.  node-gyp goes to some lengths to find a python2 binary with a fallback to just 'python'.  IOW, this should work most of the time and when it doesn't you can pass --python <path> explicitly (or set it with npm config.)

when this was created, it certainly didn’t thought 😉

It did, actually.  The algorithm was tweaked a little, though.

am i missing something?
look at #911’s diff (my PR from back then): i don’t see any attempt at the previous code to use anything but the “python” binary.


Fix changing focus with Tab key for Button

How was the solution implemented (if it's not obvious)?
This is the gif of how Button behaves before fixes:
http://g.recordit.co/fJEMyNmBqb.gif
This is the gif of how Button behaves after fixes:
http://g.recordit.co/TO3nXhqhqD.gif
The current solution matches WPF and UWP behaviour. The pressed state is being re-set on Tab key press, click event is not invoked, flyout is not opened.
Fixed issues
fixes #6654

Also, I would like to ask, does it make sense to handle keypress here?

Yes, I think so. e.Handled is set to stop the event bubbling. Imagine if the button was embedded in a text box. If the user presses space on the embedded button, the key event should not bubble to the containing text box because that would insert a space as well as clicking the button.

I think if user subscribes to OnKeyDown on his Button he should receive all info about the pressed keys

Kinda makes sense, but that's not how other XAML frameworks work either... This causes a lot of confusion but I think we'd cause more confusion by changing the behavior.

In WPF it seems that Enter key is being skipped also, Space key kinda works, but strange, with some delay.

Not sure what you mean - WPF handles Space and Enter here: https://github.com/dotnet/wpf/blob/main/src/Microsoft.DotNet.Wpf/src/PresentationFramework/System/Windows/Controls/Primitives/ButtonBase.cs#L604

@grokys Check this video. It would describe to you what do I mean by "kinda works".If I will press the Space key on the button for some time event will be fired, but with some delay.
https://recordit.co/5FUBiBmiCC


Release mutex anyway despite autocommit option

Now every mutex driver must release mutex manually if option autocommit set on true.
This decision not only adds difficulties in implementing the new driver, but also does not eliminate problems when managing the mutex.
Imagine that you have some acquired mutex:
$fileMutexFactory = new FileMutexFactory(__DIR__ . '/var/mutexes', false);

$mutex = $this->mutexes->create('some_mutex');

// some actions

$this->doSomething(); // But there will be exception

$mutex->release();

Such mutex will not be released because autocommit turn off.
I think, mutex releaser must be required and do not rely on the user's actions. Every mutex implementation must release mutex anyway in they destructor's.
The following code will guarantees that the mutex will be released anyway on the script shutdown, and it also does not require drivers to implement it themselves.
abstract class Mutex
{
       /**
     * Acquires a lock.
     *
     * @param int $timeout Time (in seconds) to wait for lock to be released. Defaults to zero meaning that method
     * will return false immediately in case lock was already acquired.
     */
    abstract public function acquire(int $timeout = 0): bool;

    /**
     * Releases a lock.
     */
    abstract public function release(): void;

    abstract public function released(): bool;

    final public function __destruct()
    {
       if (!$this->released()) {
           $this->release();
       }
    }
}

After that, we can delete the call to the register_shutdown_function function and the `autocommit' option.

The flag was removed. Having abstract class instead of an interface has its drawbacks.

The flag was removed. Having abstract class instead of an interface has its drawbacks.

You don't need an interface or an abstract class, but you go your own way as always.


Handle subclasses of dicts in DataFrame constructor, with tests

Minor change to allow passing subclasses of dict into DataFrame constructor.  I assume there is no way to pass these directly to cython, we have to make a copy first?

I think the solution is just to use the regular pure Python alternative to fast_multiget. I'll make a quick patch to avoid the copy-- surprised I don't have an initialization from collections.defaultdict somewhere already in the codebase

alright, played some tricks to not have to copy the dict and added a defaultdict test also for good measure


Json output not formatted correctly when message contains json

Describe what happened:
We're running DataDog agent on kubernetes and have enabled DD_LOG_FORMAT_JSON:                        true to output DataDog's log in json format for further processing by FluentD.
We are getting json output, but some records are invalid json due to missing encoding of json inside the message string.
An example event:
{"agent":"core","time":"2019-08-07 11:43:07 UTC","level":"ERROR","file":"pkg/collector/runner/runner.go","line":"294","func":"work","msg":"Error running check prometheus: [{"message": "HTTPConnectionPool(host='prometheus-server', port=80): Max retries exceeded with url: /metrics (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f8ae60c7290>: Failed to establish a new connection: [Errno -2] Name or service not known',))", "traceback": "Traceback (most recent call last):\n  File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/base.py\", line 884, in run\n    self.check(instance)\n  File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/prometheus/base_check.py\", line 109, in check\n    ignore_unmapped=True,\n  File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/prometheus/mixins.py\", line 398, in process\n    for metric in self.scrape_metrics(endpoint):\n  File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/prometheus/mixins.py\", line 362, in scrape_metrics\n    response = self.poll(endpoint)\n  File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/prometheus/mixins.py\", line 533, in poll\n    endpoint, headers=headers, stream=False, timeout=self.prometheus_timeout, cert=cert, verify=verify\n  File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py\", line 75, in get\n    return request('get', url, params=params, **kwargs)\n  File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py\", line 60, in request\n    return session.request(method=method, url=url, **kwargs)\n  File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py\", line 533, in request\n    resp = self.send(prep, **send_kwargs)\n  File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py\", line 646, in send\n    r = adapter.send(request, **kwargs)\n  File \"/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/adapters.py\", line 516, in send\n    raise ConnectionError(e, request=request)\nConnectionError: HTTPConnectionPool(host='prometheus-server', port=80): Max retries exceeded with url: /metrics (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f8ae60c7290>: Failed to establish a new connection: [Errno -2] Name or service not known',))\n"}]"}
Describe what you expected:
Valid json output that can be processed further.
Steps to reproduce the issue:

Deploy latest cluster agent
Make log events happen that have json in their message (no clear repro)

Additional environment details (Operating System, Cloud provider, etc):
We're running datadog/agent:latest-jmx with DD_LOG_FORMAT_JSON enabled. Container is running in kubernetes on AWS EKS.

Hello @ncrothe, thanks for reporting this issue. We're working on fixing this, apologies for the inconvenience in the meantime.

Hi @ncrothe,
I opened a PR to fix this here. This should be available in the next release 6.14.0.


Fix example & cherry-pick server::conn::Parts.service to master

The first commit fixes cargo test for me.
The second commit backports forwardports the fix to issue #1471 to master. Hopefully it was just forgotten and not omitted intentionally, because otherwise implementing server upgrades is more difficult.

Oh wow, thanks for noticing this! It definitely was just forgotten, and it would have been unfortunate, cause bringing it back would have been a breaking change.


ImportError: No module named http_client

When I type from gcloud import datastore
them
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "gcloud/datastore/__init__.py", line 53, in <module>
    from gcloud.datastore.batch import Batch
  File "gcloud/datastore/batch.py", line 24, in <module>
    from gcloud.datastore import helpers
  File "gcloud/datastore/helpers.py", line 27, in <module>
    from gcloud._helpers import _datetime_to_pb_timestamp
  File "gcloud/_helpers.py", line 34, in <module>
    from six.moves.http_client import HTTPConnection
ImportError: No module named http_client

How did you install gcloud-python?
You could try pip install --upgrade six since it looks like you may be using an older version of `six.
Let me know if that changes anything!

If you are on Mac OS X you may not be able to upgrade six

Yes, i am on Mac os X,but i have already upgrade six package，it still not working,it is great on linux

Apple both locks down the system installed six package and puts that same package on your import path in front of where you've installed it. Run
$ python -c 'import six; print(six.__file__)'

to see where it is actually getting imported from and you'll see the problem


Support changeName and changeDescription actions

Description
It looks like here we do not support other actions than addAttributeDefinition.

Hello @wizzy25 .. please check this issue. If you find it not important please close it.

@junajan This involves adding new features, so personally I think it should be closed


How to get chat_id on a Pipeline (not a filter)

Question :point_up:
{'stream': False, 'model': 'spot', 'messages': [{'role': 'user', 'content': '### Task:\nGenerate 1-3 broad tags categorizing the main themes of the chat history, along with 1-3 more specific subtopic tags.\n\n### Guidelines:\n- Start with high-level domains (e.g. Science, Technology, Philosophy, Arts, Politics, Business, Health, Sports, Entertainment, Education)\n- Consider including relevant subfields/subdomains if they are strongly represented throughout the conversation\n- If content is too short (less than 3 messages) or too diverse, use only ["General"]\n- Use the chat\'s primary language; default to English if multilingual\n- Prioritize accuracy over specificity\n\n### Output:\nJSON format: { "tags": ["tag1", "tag2", "tag3"] }\n\n### Chat History:\n<chat_history>\nUSER: hello\nASSISTANT: Hi there! How can I help you today?\nUSER: hello again\nASSISTANT: Hello again! If there\'s anything you need, feel free to ask.\n</chat_history>'}], 'user': {'name': 'User', 'id': '4e25310a-7db0-40a4-b5b6-7f0c1a830506', 'email': 'admin@localhost', 'role': 'admin'}}

We only get something like that

read this https://github.com/open-webui/pipelines/issues/168

thanks


org-mode refile causes Emacs on MAC crash

Here is the error report I filed on emacs mac build:  issue report.
I am not sure whether it is an emacs issue or due to wrong configuration.

I now have the same thing, and also sparse-tree match crashes too, but only if I run the windowed app. If I run emacs from the terminal, then it works.

Are you still experiencing this issue? If so, please include your M-x doom/info.

Refiling works fine now for me.


support channel policies in LN edges

Closes https://github.com/bitcoin-dev-project/warnet/issues/227
Taking into account https://github.com/bitcoin-dev-project/warnet/pull/303 as well and using hostnames instead of ipv4 addresses where applicable.
I think this may have already been done but in the graph file now bitcoin_config uses command line argument syntax (i.e. -key=value  not bitcoin.conf style key=value\n)
Pushing up commit by commit to burn CI minutes

You may want to cherry-pick #319 instead of "ci: force docker compose 2.24" (if it works)


Specification: https://www.w3.org/TR/html52/interactive-elements.html#the-dialog-element

I'll include it, thanks.


Make figure of insect & all hosts (approximate) ranges

Create some range maps, perhaps using some 95-98% density envelopes.

A version of this is in f5cc03a.


PR1: Update to pipeline

Module update to take 10 glyphs for font creation.
added module to extract already present Tibetan glyphs

there is branch conflict, please resolve the conflict first and then send PR


Self-organized DC workshop request form

Data Carpentry now has guidelines for self-organized workshops, and we'd like to have a separate self-organized workshop registration form for instructors to register a self-organized workshop and confirm that instructors will meet the self-organized workshop criteria
http://www.datacarpentry.org/self-organized-workshops/
I drafted a form that has the necessary fields. For fields that are the same as in the existing form, that same field is fine. I added a few fields that are just relevant for self-organized workshops and took out some that weren't relevant.
https://docs.google.com/forms/d/1SHctDIQYLEDy139NNTKkgi0ctbMeh2zdE927hsWh1Ew/viewform
Can we make this a form in Amy, so the information goes in to the Amy system? I imagine some discussion will be needed to make sure the questions can match with the database structure. Let me know if a call would be helpful.

@tracykteal the form you proposed differs from the current similar form for SWC (https://amy.software-carpentry.org/workshops/submit/). The existing form is rather intended for people to tell AMY that they've been running self-organized workshops, however I don't think anyone used that form ever :cry:
@maneesha please decide if you want to stick with the existing form or implement something similar to what @tracykteal proposed for DC.

Yes, this new form has a slightly different motivation, in that we want to make sure people are prepared to run self-organized workshops and that we get them registered. This is partly in response to the challenges @maneesha has seen with self-organized workshops. It also matches the process approved by the DC Steering Committee for self-organized workshops http://www.datacarpentry.org/self-organized-workshops/

Any update on this? Anything we can do to help?

@tracykteal just to make sure: this new form is meant to replace https://amy.software-carpentry.org/workshops/submit/, correct?

No, it would be an additional form. We want to have one for hosts to request workshops and one for self-organized workshops, because they have different sets of questions.

We want to have one for hosts to request workshops and one for self-organized workshops, because they have different sets of questions.

We currently have 2 different forms for requesting a workshop:

https://amy.software-carpentry.org/workshops/dc/request/
https://amy.software-carpentry.org/workshops/swc/request/

We also have 1 form for informing admins about a possibly self-organized workshop that already has a webpage: https://amy.software-carpentry.org/workshops/submit/
I'm asking if the last one should be replaced with the new form since they both cover self-organized workshops and the existing form does not seem to be used.

Sorry, I was thinking about the regular form. The current 'Tell us about your workshop' form is the one that SWC I think is using, although I'm not sure if they're using it. @maneesha @jduckles?
I think SWC and DC do need different self-organized forms though. SWC is requesting the information either before or after the workshop, but DC is asking that people fill in this form and check the boxes that their workshop meets these requirements before they run the workshop.

@tracykteal question about the form's field "Domain for the workshop". Is it attendees' domain of interest or is it Data Carpentry data set the organizer wants to use?
Question about "I confirm that I will handle registration for this workshop *" and "I confirm that I will distribute the Data Carpentry survey to participants *" fields - do they need to have an "other" as an answer?
I also need the reasons for the fee waiver, if it needs to be a choice list.
Thanks!

@tracykteal is "Data wrangling" a different lesson than shell?

Thanks @pbanaszkiewicz I update the form at https://docs.google.com/forms/d/1SHctDIQYLEDy139NNTKkgi0ctbMeh2zdE927hsWh1Ew/edit
to address these questions.
I also added a question about 'Instructor status' (4th question) and the categories for Self-organized workshop fees. For the workshop waiver section, can we just leave it as free response? I want people to just address the fee waiver with a couple of sentences.

@tracykteal thanks! I left the waiver section as a text box.

Great, thank you!

Closing because #836 was merged

Thanks @piotr. I realize I haven't yet updated our website with the new AMY self-organized workshop form that you implemented here. What's the link?

@tracykteal it's https://amy.software-carpentry.org/workshops/dc/request_selforganized/

Great, thank you!


Make the TLSv1.3 check more robust and not depend on the Java version…

… to detect if its supported or not
Motivation:
TLSv1.3 is not strictly limited to Java11+ anymore as different vendors backported TLSv1.3 to Java8 as well. We should ensure we make the detection of if TLSv1.3 is supported not depend on the Java version that is used.
Modifications:

Add SslProvider.isTlsv13Supported(...) and use it in tests to detect if we should run tests against TLSv1.3 as well
Adjust testcase to work on latest JDK 8 release as well

Result:
Correct detection of TLSv1.3 support even if Java version < 11.

This is related to having TLSv1.3 shipped as part of OracleJDK 1.8u262:
https://java.com/en/jre-jdk-cryptoroadmap.html

Addressed all comments.... PTAL


Add ChartVersion to helm install when running tests

What this PR does / why we need it:
Fixes issue with ChartVersion not being set as a flag when calling helm install in the e2e tests, preventing the correct setup of kind pods and therefore testing.
Release note:

NONE

Thanks for digging into this one 😄 😅

/cherrypick release-0.8


v1.0.12 breaks asset paths in addon CSS

I verified that everything works if I force it to 1.0.11.
But at 1.0.12, I get the following issue:
We have an ember-cli app that uses ember-cli-bootstrap-colorpicker. When we do a production build, the vendor css contains rules that look like this:
//my-assets-url/assets/../img/bootstrap-colorpicker/saturation-668d0a0e7a0189f62898bcc57ab8370c.png
Note the /../ in the middle of the url - that causes it to resolve to the wrong url and 404. Again, when I forced broccoli-asset-rev to use v1.0.11 of broccoli-asset-rewrite, we no longer see this issue and we get proper urls in the css like:
//my-assets-url/assets/img/bootstrap-colorpicker/saturation-668d0a0e7a0189f62898bcc57ab8370c.png

Could you write a failing test case for this?

Ugh, I can try but I'm not sure how. I'll take a stab.

Pinging @john-kurkowski

OK, that wasn't so hard. I think I got it right.

Thanks! Taking a look...


LeNet1989 with Commentary

Description
Jax/Flax implementation of LeCun 1989's CNN. Since you asked for a notebook, I assume you would like to optimize for readability. As a result, I based my work on Flax's official Annotated MNIST notebook, which includes a lot of comments and is very comprehensive. I have also added some sanity checks to demonstrate the shape of the parameters are correct.
assert state.params['H1w']['kernel'].shape == (5, 5, 1, 12)
assert state.params['H1b']['bias'].shape == (8, 8, 12)
assert state.params['H2s1w']['kernel'].shape == (5, 5, 8, 4)
assert state.params['H2b']['bias'].shape == (4, 4, 12)
assert state.params['H3']['kernel'].shape == (4*4*12, 30)
assert state.params['H3']['bias'].shape == (30,)
assert state.params['Out']['kernel'].shape == (30, 10)
assert state.params['Out']['bias'].shape == (10,)

I have not added the modern adjustments yet to avoid bloating the notebook, but I can include some if that's desirable. In fact, I have experimented with making all units in the convolutional layers share their bias in addition to the weights, and using minibatch instead of SGD. Both adjustments don't seem to have a huge impact, though.
Colab link

https://colab.research.google.com/drive/1N5qvJfmJnHgmfSdw8frU5C9J7gS_NgKX?usp=sharing
Issue

convert LeNet1989 from Torch to JAX
Checklist:

[X] Performed a self-review of the code
[X] Tested on Google Colab.

Potential problems/Important remarks
Apart from commentary and coding styles, there are no substantial differences between my implementation and the one in https://github.com/probml/probml-notebooks/pull/54, despite we worked independently.

LGTM. However Please remove layer_norm_jax.ipynb from this PR.
And make the colab public so I can check it out.

I have removed the domain restriction. Please check again if you can access the notebook on Colab.
https://colab.research.google.com/drive/1N5qvJfmJnHgmfSdw8frU5C9J7gS_NgKX?usp=sharing
I have also deleted the file layer_norm_jax.ipynb, so that you can merge my branch now.

Yes, I can read it now. This looks like a great introduction to flax. Thanks!


For revolute joints allow a "revolutions" as well as degrees and rads unit type

New option in prefs dialog


Support for the experimental syntax 'jsx' isn't currently enabled

I get this when trying to use it with Vite + Solid starter.
Support for the experimental syntax 'jsx' isn't currently enabled (8:14):

Add @babel/preset-react (https://github.com/babel/babel/tree/main/packages/babel-preset-react) to the 'presets' section of your Babel config to enable transformation.
If you want to leave it as-is, add @babel/plugin-syntax-jsx (https://github.com/babel/babel/tree/main/packages/babel-plugin-syntax-jsx) to the 'plugins' section to enable parsing.

If you already added the plugin for this syntax to your config, it's possible that your config isn't being loaded.
You can re-run Babel with the BABEL_SHOW_CONFIG_FOR environment variable to show the loaded configuration:
	npx cross-env BABEL_SHOW_CONFIG_FOR=index.jsx <your build command>
See https://babeljs.io/docs/configuration#print-effective-configs for more info.I

I see, thanks. I've settled on stylex in the meanwhile, but good to know if there is a fix.


Tooltip from Editor Description field should propagate immediately

Godot version
4.1.2rc1
System information
Windows 10
Issue description
When the user updates the Editor Description field, the update does not reflect as a tooltip until the node is deselected.

While this might seem irrelevant, it impacts the discoverability of the feature.
My immediate assumption was that the field was to be displayed in a tooltip, but upon testing it didn't work until I changed scenes. This might make new users dismiss the feature.
Steps to reproduce

Open any project and click on a node
Find the Editor Description field in inspector and type something into it
Hover the node in the tree hierarchy with your mouse. Observe that it lacks a description
Deselect and select the node again to see the updated description in the tooltip

Minimal reproduction project
N/A

While I do agree, to achieve the update in real time would probably require updating the entire tree all over again, if not handled the most ideal away.

The tooltip is built per _add_nodes instead of everytime editor description field changes.

This was partially fixed by https://github.com/godotengine/godot/pull/66266, but you need to select another node for the tooltip to update.


Mermaid Diagrams fail to generate when viewing documentation via github pages

Problem
Currently, our GitHub Pages site is encountering an issue where it fails to generate Mermaid diagrams. Mermaid is a popular JavaScript library for generating diagrams and flowcharts. It is an essential tool for visualizing complex information and improving the overall user experience on our site.
Problem Solution

Investigate and identify the root cause of the failure in generating Mermaid diagrams on the GitHub Pages site.
Fix the issue to ensure that Mermaid diagrams are correctly rendered and displayed on the site.

Re-opening given that a bug currently exists with Jekyll that results in mermaid still not properly rendering:
https://base-org.github.io/pessimism/architecture/etl


Mac OSX 10.9.5 local development gives SSL certificate problem

On Mac OSX 10.9.5 a call to the API gives an SSL error
MessageBird\Exceptions\HttpException: SSL certificate problem: unable to get local issuer certificate

I did not find out why the MessageBird Cert is not trusted. But the error does not seem to occur on other os (linux etc)
One can switch off verification in Development by using this hack in vendor/messagebird/php-rest-api/src/MessageBird/Common/HttpClient.php:106
curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, 0);
curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, 0);

Maybe it is possible to add a function to skip SSL verification in the high level API?

This issue is commonly caused by a CA certificate nog being present on the system that is making the request. You'll find the certificate that is needed here https://www.digicert.com/CACerts/DigiCertSHA2SecureServerCA.crt.


feat: controller rumble on damage taken

Couple things:

First few commits don't look related to the scope of the MR at all. If you want to merge both things in a single MR, I'd appreciate if you could explain in the description what was done for each "task" so we know where to look.
As for the rumble, I don't like the idea of having that code script-side; the fact that we use SDL is an implementation detail that the scripts should not need to know or deal with. I'd encapsulate that code in a method in ModuleInput, which will also avoid repeated code.


Add support to load audio data

Feature request: Support to load audio & video files into documents would be cool 🤗 . Similar to the LangChain Audio transcript loader integration: https://python.langchain.com/docs/integrations/document_loaders/assemblyai/

Love the idea! 🚀 I'll definitely put it on the feature list

DONE 🥳


Add application specific/development codecs

For dev purposes, it is good to have a range of valid codecs that can be used for developing support for new protocols. I think we had something like that in the past but I can't find it in the table right now.

Just came up regarding IPLD resolvers as well, when you want to have a custom codec. We should have a range people can chose in between and be sure we want add something to the table to overwrite it.
cc @vmx

Duplicate of #39


Added install instructions for WSL users - fixed #889

I just hit https://github.com/rust-lang/rust-www/issues/889 as well. I guess it becomes pretty normal for Windows developers to use WSL, so I think this change helps here to get started easily. ❤

thanks so much!

Thank you!


cannot discover the tests immediately after update the configuration in "Test" tab for unittest

VS Version: 16.4.0 Preview 2.0 [29316.151.master]
Python version: 16.4.19255.2
Steps to Reproduce

Create a python application
2.Add two unittest file such as "test_1.py" "test_2.py"
3.Set "Test framework" to "unittest"
Open "Test Explorer" to make sure the tests are discovered
Open "Property->Test", Set "Pattern" to "test_1.py" and then press "Save" button
Check "Test Explorer" panel

Expect Result
It should discover the test "test_1.py" immediately after change the configuration for unittest
Actual Result
It cannot discover the tests "test_1.py" immediately after change the configuration for unittest

I just tried. Couldn't repro.
VS Version: 16.4.0 Preview 2.0 [29316.153.master]
Can you try again?

I tried on 16.4.0 Preview 2.0 [29317.151.master] and always repro. below is my environment, you can repro this issue on my live env if you needed
[Test machine info](https://microsoft.sharepoint.com/teams/CloudPlatformToolsDevOpsCTI/_layouts/OneNote.aspx?id=%2Fteams%2FCloudPlatformToolsDevOpsCTI%2FShared Documents%2FTeam Notes%2FDevOps CTI Wuxi&wd=target(04 PTVS%2FVS%2FTestResults.one|63BE2640-375F-4969-9861-C50F944DE029%2FTest Machine info|2EA9D36B-00A9-455B-BD8A-DF5A977DF565%2F)
onenote:https://microsoft.sharepoint.com/teams/CloudPlatformToolsDevOpsCTI/Shared Documents/Team Notes/DevOps CTI Wuxi/04 PTVS/VS/TestResults.one#Test Machine info&section-id={63BE2640-375F-4969-9861-C50F944DE029}&page-id={2EA9D36B-00A9-455B-BD8A-DF5A977DF565}&object-id={6D3B00EE-A535-0A08-3F98-80A81BB080D7}&10)


Broken with the latest "boost" version

When using boost version 1.72.0, lgogdownloader will report:
error while loading shared libraries: libboost_system.so.1.71.0: cannot open shared object file: No such file or directory
The only solution is to downgrade both boost and boost-utils to version 1.71.0 (in Arch).

Rebuilding lgogdownloader fixed the issue, closing.


Destroy Instance not working

The terraform command to destroy instances seems not to work, as it does not apparently cancel the subscription

Great work!


If you don't take any issue in consideration please be fair and write it in the readme ...

Like that people can choose an other project

Hey sorry. I just built a new lib called informed as a replacement for this one and was going to update the readme this week.

Well started months ago I can't change lib like that ... Why a complete rewrite instead of an progressive update ?

This is no different than a Major version bump.. this was never fully my project. So i went and rewrote a version that u could personally support. The OG creator of react form and I talked it through and figured it would be best for me to take over the project.

I see ... Is react-form will be deprecated ?
Le lun. 25 juin 2018 à 16:49, Joe Puzzo<EMAIL_ADDRESS>a écrit :

This is no different than a Major version bump.. this was never fully my
project. So i went and rewrote a version that u could personally support.
The OG creator of react form and I talked it through and figured it would
be best for me to take over the project.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/react-tools/react-form/issues/337#issuecomment-399979049,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAb2L0cnvlyFWJOB7TvqAnziYrcVmHK2ks5uAPiGgaJpZM4U2M0L
.

React form is deprecated and we are now going to support informed https://www.npmjs.com/package/informed

Got it ! Thx for answering my questions !
Le lun. 25 juin 2018 à 18:34, Joe Puzzo<EMAIL_ADDRESS>a écrit :

React form is deprecated and we are now going to support informed
https://www.npmjs.com/package/informed
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/react-tools/react-form/issues/337#issuecomment-400015066,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAb2L9RnSOKV872-pLVeGd56JSAuTIUrks5uAREAgaJpZM4U2M0L
.


Bump CodeQL version to 2.7.5

Merge / deployment checklist

[ ] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the changelog has been updated if necessary.

Shall we update this PR to run the integration tests on the new 2.7.5 release candidate so we can avoid any nasty surprises?

Yes we should. I'll modify it appropriately.


Split wrappers

Closes #1450
Used https://github.com/potherca-bash/git-split-file to preserve the commit history so blame still works for each file. That's the reason for so many commits, it creates a branch to rename __init__.py to each split file.
werkzeug.wrappers still exports all the classes, the documentation remains the same. The understanding is that the documented API was the publicly importable API. However, none of the incidental imports are exposed at the top level any more, so anything that was behaving badly and importing werkzeug.wrappers.datetime, etc. will need to update.

Any way to s/Changes/Change/ in those commit messages that are probably created by a script?

... ... yeah


support multiple ego vehicles

add support for publishing bboxes + lidar_overlay when multiple ego vehicles are spawned

@zoq I figured that try block was necessary there incase the lidar doesnt exist for that parent(ego vehicle)


Presets: Add ramp=yes to highway=stairs

The preset for highway=stairs https://github.com/openstreetmap/iD/blob/develop/data/presets/presets/highway/steps.json does not have information about the ramp-key https://wiki.openstreetmap.org/wiki/Key:ramp.
The key is "in use" with 30k+ uses.
Unfortunatelly the values that are documented on the wiki page dont allow for an easy UI. Maybe a first step would be to have a dropdown with fixed values: ramp=yes|separate|no|NULL/Unknown. The other, special ramp types could be added later, IMO.

PS: https://github.com/westnordost/StreetComplete/issues/2036 is thinking about this key as well.

@tordans Supporting ramp=yes/no sounds okay. But I find ramp=separate confusing since these are mapped as distinct ways anyway. It has less that 1% of the uses, so I'm inclined to leave it off.

@quincylvania thanks. For reference, see also https://github.com/westnordost/StreetComplete/issues/2036#issuecomment-703252986 with thought about the separate-value.


Player freezing after ad

Hi there,
there is an ad from AdX which is freezing the player once it ends.
Ad Id: "d59884ee-1445-4d8e-b66e-d39e7bcdcb25"
Ad Query Id: "307699625"
Media URL: "https://s1.adform.net/banners/scripts/video/VPAIDWrapper.min.js?cache=14"
Can the freeze be prevented?

Just wanted to check in. Is the provided ad and ad query id enough to track down the issue?

Hi there,
We can't troubleshoot any serving issues with your AdX ad. Do you have an example page where the freeze happens?

It is only this AdX ad that is freezing the player. Other players are not freezing when this ad plays. how would you go about debugging it if it is only available programatically?

Are you getting a javascript error when the freeze happens? You can try a proxy to log all the requests and responses to see if anything strange is happening there. Otherwise I would recommend checking in the the DFP support forums which handles AdX: https://productforums.google.com/forum/#!forum/dfp

It's not throwing any errors.
The other players (where it is working fine) are using the ima sdk as well, how can the DFP support help in this case?

If it's an issue with the ad, which appears to be a VPAID, you can ask DFP support. It could also be an issue with the player. You can try other VPAID ads, like this sample to see if your player plays VPAID normally.

Hello,
We are seeing a similar effect, when a pre-roll is played over a streaming content, the Preroll -> ContentPlayback event is never fired.
I've checked, and found that, while PlayerWrapper.prototype.onAdBreakStart has a this.vjsPlayer.pause();, PlayerWrapper.prototype.onAdBreakEnd doesn't have a corresponding this.vjsPlayer.play();, so adding it solves this issue.
Check if this is your case.

Hi all,
Looking into this it looks like an issue with the ad-tag.
@damianvila work around presents a potential issue.
PlayerWrapper.prototype.onAdBreakEnd = function() {   this.vjsPlayer.on('contentended', this.boundContentEndedListener);   if (this.vjsPlayer.ads.inAdBreak()) {     this.vjsPlayer.ads.endLinearAdMode();   }   this.vjsControls.show(); };
The call to this.vjsPlayer.ads.endLinearAdMode(); should restart the player, but it is only called if this.vjsPlayer.ads.inAdBreak() evaluates true.
I think that the issue might be that related to this.vjsPlayer.ads.inAdBreak() evaluating false. I am going to look into replacing that to this.vjsPlayer.ads._inLinearAdMode because that seems more related to then calling this.vjsPlayer.ads.endLinearAdMode();
I will follow up on this issue soon.

Hey all,
Take a look at this pr https://github.com/googleads/videojs-ima/pull/840
I think the change might be able to solve the issue, but because I am not able to reproduce, I cannot confirm that.
Let me know if it works for you.

Hey there, I'm not convinced this will fix the issue. I am currently trying to implement ad breaks aswell and the problem is related to some misbehavior of the ima plugin according the contract between this ad-plugin and videojs-contrib-ads.
First of all, I'm pretty sure the problem in this issue happens because the media ressource is a live stream. And as far as I have seen, there were other issues about playing live streams with video js and the ima plugin. It just can't deal with live streams and I dug really deep into the ima and the contrib-ads plugin to figure out, what is happening.
I have already seen shawnbuso mention the problem here in #629 in May 2018 and it seems to be still not fixed. Based on what shawnbuso said, it's pretty obvious, that the ima plugin doesn't stick to the contrib-ads plugin contract. If the content is a Live-Stream, it should not be paused, since contrib-ads will expect the media ressource to keep running, mutes it and hide it by adding the vjs-ad-playing class. Since ima pauses the content player no matter if it's live or not, it won't get resumed automatically by contrib-ads, because all it does is removing the class again and unmute the content player.
It's even worse, the contrib-ad plugin does not store a snapshot of the content player, if the content is live, and thus won't restore the content player when the ad ends even if you would fix the freeze issue by calling play in PlayerWrapper.prototype.onAdBreakEnd. This would cause the stream to start from the beginning after an ad break and is not the correct behavior.
I think the problem is, that this plugin was never ment to work with live streams, but it definitly should do. A simple fix for this issue would be to introduce the new option liveContent, which default to false to not break anything, but wrap the hard vjsPlayer.pause() call in an if statement to not pause if the option is set to true. Of course it would be great, if you could detect dynamically if the content is Live, but this simple option would be enough as a quick fix.

Hi @broadcoder ,
Thanks for the update.
I see, if this is related to https://github.com/googleads/videojs-ima/issues/629 it looks like it may require the cuePoints feature to be implemented into the videojs-ima plugin.
I will continue to investigate the issue and the potential of adding that feature.

@Kiro705 Hi, is there an ETA for a fix for this? I'd like to use this for live playback.

Hi @malini ,
This feature has not yet been prioritized, so there is no ETA available.

Workaround that worked for me is this:
playerInstance.on('adend', () => playerInstance.play())

Hi all,
Looking into this issue, I am having a tough time reproducing the issue. Could someone share how they are able to reproduce it? (sample content and sample AdX ad)

Are you getting a javascript error when the freeze happens? You can try a proxy to log all the requests and responses to see if anything strange is happening there. Otherwise I would recommend checking in the the DFP support forums which handles AdX: https://productforums.google.com/forum/#!forum/dfp

Hi all,
Looking into this it looks like an issue with the ad-tag.
@damianvila 's work around presents a potential issue.
PlayerWrapper.prototype.onAdBreakEnd = function() { this.vjsPlayer.on('contentended', this.boundContentEndedListener); if (this.vjsPlayer.ads.inAdBreak()) { this.vjsPlayer.ads.endLinearAdMode(); } this.vjsControls.show(); };
The call to this.vjsPlayer.ads.endLinearAdMode(); should restart the player, but it is only called if this.vjsPlayer.ads.inAdBreak() evaluates true.
I think that the issue might be that related to this.vjsPlayer.ads.inAdBreak() evaluating false because of something specific to the ad tag. I am going to look into replacing that to this.vjsPlayer.ads._inLinearAdMode because that seems more related to then calling this.vjsPlayer.ads.endLinearAdMode(); and that might fix the issue
I will follow up on this issue soon.

But it is throwing  this.vjsPlayer.ads._inLinearAdMode is not a functionCan any one help me to resolve this


Add all citations at CITATIONS.md

Description of feature
Clustalo, tcoffee etc, all main tools used

Closed by #111 .


CLI log tailing should retry on connection error

If connection to master is cut when tailing a log (kontena service logs -t), CLI will terminate. Suggestion is to retry if I EOFError is raised.

should be fixed with #624 as they are pretty much same thing

fixed in #835


TWAP: Add overflow test

Closes: #2412
What is the purpose of the change
This PR adds tests regarding overflow in accumulator within the TWAP module, specifically a test proving that it takes 2 ** 128 nano sec(over 292years) with max spot price for a pool to overflow our current accumulator.
This PR tests with testing field of sdk.Dec instead of time.Duration, because it becomes impossible to test overflowing testing scenarios since time.Duration would always automatically cap to time.Duration.MaxDuration, without erroring or panicing if we're trying to create a duration significantly big enough
Brief Changelog

Adds overflow test for TWAP Accumulator

Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
How is the feature or change documented? (not applicable   /   specification (x/<module>/spec/)  /  Osmosis docs repo   /   not documented)

Nice job!


fix(console): policy routes

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


drive push failing silently

We are trying to upload large files ~1.5TB in size to Google drive and failing silently.  Even when drive says that operation is complete 100% and  when I look at the files in Google Drive they are not complete - the file sizes are different, as are the md5 sums. (here we are talking about 1.5 and 1.3TB files showing up as 1TB and 98GB).
We have tried multiple times and get different sizes every time.
I hope some one could help me solving this issue.  Thanks in advance.

I have the same kind of issue when trying to push files inside a folder which does not exist remotely, whatever their size: the message states that
drive push -verbose "$file" 2>&1
Resolving...
Everything is up-to-date.

Only a drive stat can pinpoint the error:
drive stat "$file"
... remote path doesn't exist


A claim of type 'sub' was not present on the provided ClaimsIdentity.

I'm using the MembershipReboot assemblies to create a membership manager for my IdentityServer3 based authentication/authorization system.  I started with one of the samples and had great success at turning it into a system that fits my requirements.  After adding an AuthorizationManager class and  [ResourceAuthorize(...)] attributes to my controller methods, I started getting the error:
"A claim of type 'sub' was not present on the provided ClaimsIdentity"
The post at  https://github.com/IdentityModel/AuthorizationServer/issues/61  is right on target to my issue.  However, I don't see where or how I add a "Sub" claim to the token.
Any help I can get with this would be greatly appreciated.  Thank you.

Sorry -- was traveling and this issue fell off my radar.
IdentityServer needs a sub claim and you'd map that from the UserAccount.ID property of MembershipReboot. I'd suggest looking at this code to get an idea how to implement the user service in IdentityServer:
https://github.com/IdentityServer/IdentityServer3.MembershipReboot/blob/master/source/MembershipRebootUserService/MembershipRebootUserService.cs

Also, this is not really an issue with MembershipReboot, so I'll close it. If you have more questions, open an issue in the more appropriate repo.


Support order constraints across objects

This example passes, but it should fail:
describe "something" do
  it "does something" do
    one = double
    two = double

    one.should_receive(:foo).ordered
    two.should_receive(:foo).ordered

    two.foo
    one.foo
  end
end

I'm referencing my gist here (https://gist.github.com/1704783) as I'd like to get a general idea of how people feel about the use of explicit sequences, using either of the syntaxes in that gist (my preference is the block form as it seems more readable to me).

Closed by pull request #106.


Log spam of "InvalidValueError" for lat/long

InvalidValueError: at index 0: not a LatLng js?key=AIzaSyDSTJMwyUK9K5Vd0I1MWtg2Gb14DylAhpE:66 InvalidValueError: at index 0: not a LatLng or LatLngLiteral with finite coordinates: in property lat: not a number
InvalidValueError: setPosition: not a LatLng js?key=:66 InvalidValueError: setPosition: not a LatLng or LatLngLiteral: in property lat: not a number
These two lines repeat ad nauseam. I don't have lat/long set. Attempted to use Forester for the first time and imported a 5k+ QSO ADIF generated by WSJT-X. App itself is basically unusable and is getting killed.
OS: Ubuntu 22.04
Browser: Chrome 101.0.4951.54

Thanks for the report. The log spam is probably coming from the map, right now there's no guard to stop it from trying to render QSOs without coordinates. I should be able to fix that up quickly.
(Sorry for the mess, there aren't many users yet so I'm not as disciplined about keeping things as pristine as I should!)

Hopefully this is fixed in a23894e66b9361ff6254a0a8ecc5806158f86a89.

Looks good, thanks!


troubleshooting tips for performance issues

Add a troubleshooting section in the docs to help users to identify the performance (throughput, latency etc) problems.
Occasionally users run into situation where there is decent throughput host-to-host and but may see degraded/terrible performance from pods/containers manged through Weave net. Very likely it could be due to wrong tuning of parameters of test suite used (e.g iperf3) , non-optimal MTU size etc. There is no single answer/approach to troubleshoot this kind of performance issues and need significant involvement.
Its good to have prescribed steps to evaluate the performance and troubleshooting tips.

I agree with this.


Update the webex notification provider and markdown

Signed-off-by: ahothan<EMAIL_ADDRESS>I have tested this against the latest webex API portal and verified I could get notifications in the targeted webex space.
Requires 2 additional parameters to make it work: a room ID and a Webex bot access token (both can be retrieved using the
webex developer web site after registering).
I have also enhanced the formatted markdown.
Example of webex space output:

Thank you for this, if you are looking to add an example webex notification provider, the file where provider-specific examples are drawn from is: spec/v1beta1/provider.md
The example would land in the fluxcd docs under here:
https://fluxcd.io/docs/components/notification/provider/#example
(The current supplementary docs from each controller repo are pulled into the fluxcd/website repo on each deploy)

checking if this markdown format would be better:

source: //

| = other kv pairs in manifests (all but summary)

@ahothan can you please post a print screen of how it looks now?

@stefanprodan
I wanted the summary to be first row as it would show where the notification.
Depending on whether the alerts have a summary or not, the notification will use one or another layout.
The comment for the CreateMarkdown function describes how the markdown looks like based on summary present or not,
Here is a snapshot of a mock up (not real notification so dont pay attention to the exact severity)
In this snapshot, the Alert summary was set to "cc-k8s cluster notification" for the first 2,
thelast 2 is without a summary field.

Please provide any suggestion...

not sure what to make of this error in the "kind" job:
Run kubectl -n notification-system get providers -oyaml
error: the server doesn't have a resource type "providers"
Error: Process completed with exit code 1.

Snapshot from new flux bootstrap with latest version:


Failed to Build image, Exit 100

[merith@archlinux .ssh]$ sail run  merith-tk/sail
2020-03-05 09:25:37 INFO	running docker build --network=host -t merith-tk_sail -f /home/merith/Projects/merith-tk/sail/.sail/Dockerfile /home/merith/Projects/merith-tk/sail --label com.coder.sail.base_image=merith-tk_sail
Sending build context to Docker daemon  9.485MB
Step 1/8 : FROM codercom/ubuntu-dev-go:latest
 ---> bba480e31f87
Step 2/8 : SHELL ["/bin/bash", "-c"]
 ---> Using cache
 ---> 53edc1ef0766
Step 3/8 : RUN sudo apt-get update &&   sudo apt-get install -y htop
 ---> Running in 53e3737a0672
sudo: unable to resolve host archlinux: Invalid argument
Ign:1 http://security.ubuntu.com/ubuntu cosmic-security InRelease
Err:2 http://security.ubuntu.com/ubuntu cosmic-security Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Ign:3 http://archive.ubuntu.com/ubuntu cosmic InRelease
Ign:4 http://archive.ubuntu.com/ubuntu cosmic-updates InRelease
Ign:5 http://archive.ubuntu.com/ubuntu cosmic-backports InRelease
Err:6 http://archive.ubuntu.com/ubuntu cosmic Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Err:7 http://archive.ubuntu.com/ubuntu cosmic-updates Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Err:8 http://archive.ubuntu.com/ubuntu cosmic-backports Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Reading package lists...
E: The repository 'http://security.ubuntu.com/ubuntu cosmic-security Release' does not have a Release file.
E: The repository 'http://archive.ubuntu.com/ubuntu cosmic Release' does not have a Release file.
E: The repository 'http://archive.ubuntu.com/ubuntu cosmic-updates Release' does not have a Release file.
E: The repository 'http://archive.ubuntu.com/ubuntu cosmic-backports Release' does not have a Release file.
The command '/bin/bash -c sudo apt-get update &&   sudo apt-get install -y htop' returned a non-zero code: 100
2020-03-05 09:25:45 FATAL	failed to build image: failed to build: exit status 100

this is what i get when i try to edit my fork of sail (so that i can add a feature that i and im sure a few others would use)

output with -v
[merith@archlinux ~]$ sail -v run merith-tk/sail
\2020-03-05 09:27:54 DEBUG	verified Docker is running
2020-03-05 09:27:54 INFO	running docker build --network=host -t merith-tk_sail -f /home/merith/Projects/merith-tk/sail/.sail/Dockerfile /home/merith/Projects/merith-tk/sail --label com.coder.sail.base_image=merith-tk_sail
Sending build context to Docker daemon  9.485MB
Step 1/8 : FROM codercom/ubuntu-dev-go:latest
 ---> bba480e31f87
Step 2/8 : SHELL ["/bin/bash", "-c"]
 ---> Using cache
 ---> 53edc1ef0766
Step 3/8 : RUN sudo apt-get update &&   sudo apt-get install -y htop
 ---> Running in 52ceeec54aae
sudo: unable to resolve host archlinux: Invalid argument
Ign:1 http://security.ubuntu.com/ubuntu cosmic-security InRelease
Ign:2 http://archive.ubuntu.com/ubuntu cosmic InRelease
Ign:3 http://archive.ubuntu.com/ubuntu cosmic-updates InRelease
Err:4 http://security.ubuntu.com/ubuntu cosmic-security Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Ign:5 http://archive.ubuntu.com/ubuntu cosmic-backports InRelease
Err:6 http://archive.ubuntu.com/ubuntu cosmic Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Err:7 http://archive.ubuntu.com/ubuntu cosmic-updates Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Err:8 http://archive.ubuntu.com/ubuntu cosmic-backports Release
  404  Not Found [IP: <IP_ADDRESS> 80]
Reading package lists...
E: The repository 'http://security.ubuntu.com/ubuntu cosmic-security Release' does not have a Release file.
E: The repository 'http://archive.ubuntu.com/ubuntu cosmic Release' does not have a Release file.
E: The repository 'http://archive.ubuntu.com/ubuntu cosmic-updates Release' does not have a Release file.
E: The repository 'http://archive.ubuntu.com/ubuntu cosmic-backports Release' does not have a Release file.
The command '/bin/bash -c sudo apt-get update &&   sudo apt-get install -y htop' returned a non-zero code: 100
2020-03-05 09:28:02 FATAL	failed to build image: failed to build: exit status 100

I'm also running into this, it appears it might be related to Ubuntu 18.10 being a non-LTS release and only having a 9 month support window:

https://fridge.ubuntu.com/2019/07/19/ubuntu-18-10-cosmic-cuttlefish-end-of-life-reached-on-july-18-2019/

https://superuser.com/questions/1527250/apt-update-error-with-ubuntu-18-10-cosmic-version

Not sure if there's an easy fix to upgrade in the .sail/Dockerfile or if it's best to fix the base image:

https://hub.docker.com/r/codercom/ubuntu-dev-go

We could see the difference between a base image and their image, and try to recreate it?

This hack downgrades the .sail/Dockerfile to Ubuntu 18.04 LTS to avoid that error, but now I'm getting a new error in the Node code (maybe that needs to be pinned to a specific version too, it appears its an error on a deprecated method):
https://github.com/dougnukem/sail/blob/997bff4de75b992f34706d53d39a431090ed2eeb/.sail/Dockerfile#L4-L11
I think the real fix might be to downgrade the base docker image to an LTS version  https://hub.docker.com/r/codercom/ubuntu-dev-go to Ubuntu 18.04LTS
Node error:
Step 10/11 : RUN installext peterjausovec.vscode-docker
 ---> Running in 6a69967454c3
(node:7) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
INFO  code-server v1.1119-vsc1.33.1
ERROR (node:18) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
ERROR
ERROR Extension 'peterjausovec.vscode-docker' not found.
ERROR Make sure you use the full extension ID, including the publisher, eg: ms-vscode.csharp
ERROR
ERROR Failed Installing Extensions: peterjausovec.vscode-docker
ERROR

maybe remove that command from the docker image? it looks like it just cant find that extension name,

Looks like some of these underlying problems are reported as separate issues:

https://github.com/cdr/sail/issues/254
https://github.com/cdr/sail/issues/250

Hmm it looks like sail is downloading the latest linux release which now includes arm64 and amd64 architectures, so it's incorrectly downloading the arm64 AARCH one resulting in the error:
code-server DOWNLOAD URL: https://github.com/cdr/code-server/releases/download/2.1698/code-server2.1698-vsc1.41.1-linux-arm64.tar.gz

/lib/ld-linux-aarch64.so.1: No such file or directory

open a new issue?

I think I've got a work around for all the issues, but it will require a new docker base image to be pushed

https://github.com/cdr/sail/pull/262

Successfully tagged dougnukem_sail:latest
2020-03-10 15:40:03 INFO	using repo image dougnukem_sail
2020-03-10 15:40:03 DEBUG	host home dir: /Users/douglasdaniels
2020-03-10 15:40:03 INFO	writing sail proxy logs to /var/folders/ht/gwk63jw55_b_kpkk3jf599r40000gn/T/sailproxy_dougnukem_sail209919673
2020-03-10 15:40:03 DEBUG	started container
2020-03-10 15:40:04 DEBUG	code-server online
2020-03-10 15:40:04 INFO	please visit http://<IP_ADDRESS>:62686

Just a side note, when upgrading to your version of sail, i did have to prune my docker (not an issue, i can just re-download them as i am on my dev machine, not my production) due to a binary execution error, probably because of the issue you fixed

also i noticed your path is windows? /Users/douglasdaniels ?? Can you
check if the folder C:\var was created??
—Merith
On Tue, Mar 10, 2020 at 2:02 PM Merith<EMAIL_ADDRESS>wrote:

Just a side note, when upgrading to your version of sail, i did have to
prune my docker (not an issue, i can just re-download them as i am on my
dev machine, not my production) due to a binary execution error, probably
because of the issue you fixed
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cdr/sail/issues/261?email_source=notifications&email_token=ACPQOXRDVOWY3FKTM2PHKG3RG2TIFA5CNFSM4LCO2HTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEONEOVI#issuecomment-597313365,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPQOXQLXDBURNLIEPEWINTRG2TIFANCNFSM4LCO2HTA
.

@Merith-TK my host machine was MacOS, and my changes in that pr should only impact the docker virtual machine which is ubuntu linux based so i dont think there should be any issues on windows.

alright, that makes sense,


won't start

It just won't start on windows. Perhaps because I don't know where to put the "cuda" folder, since the installation doesn't say how to add that part to the rest?
S:\new\DeepStyle\neural_artistic_style-master>neural_artistic_style.py --subject image.jpg --style s
tyle.jpg --output output.jpg
CUDArray: CUDA back-end not available, using NumPy.
At that point, I get a windows error:
Problem signature:
Problem Event Name:	APPCRASH
Application Name:	python.exe
Application Version:	<IP_ADDRESS>
Application Timestamp:	56abcaee
Fault Module Name:	ntdll.dll
Fault Module Version:	6.1.7601.19160
Fault Module Timestamp:	56bcd74c
Exception Code:	c0000005
Exception Offset:<PHONE_NUMBER>04a604
OS Version:	6.1.76<IP_ADDRESS>.256.1
Locale ID:	1033
Additional Information 1:	3935
Additional Information 2:	39350178b1f7dc4d15d5471cf2a5a09f
Additional Information 3:	6976
Additional Information 4:	697614677dd37f475b8d30bce4aa474e
Seems like perhaps an exception is nto being caught?

As an update, I got things this far, but am still stuck (I am ATI/AMD so NumPy is fine, I don't care about speed, I just want to get this working):
CUDArray: Failed to load CUDA back-end.
Traceback (most recent call last):
File "neural_artistic_style.py", line 7, in
import deeppy as dp
File "build\bdist.win-amd64\egg\deeppy_init_.py", line 1, in
File "build\bdist.win-amd64\egg\deeppy\dataset_init_.py", line 2, in
File "build\bdist.win-amd64\egg\deeppy\dataset\infimnist.py", line 6, in
File "build\bdist.win-amd64\egg\deeppy\base.py", line 1, in
File "C:\anaconda\lib\site-packages\cudarray-0.1.dev0-py2.7-win-amd64.egg\cudarray_init_.py", li
ne 20, in
from .cudarray import *
File "C:\anaconda\lib\site-packages\cudarray-0.1.dev0-py2.7-win-amd64.egg\cudarray\cudarray.py", li
ne 2, in
from .wrap.array_data import ArrayData
ImportError: No module named array_data
What am I doing wrong? Definitely something!

Can you try to make sure the backend is set to Numpy and not Cuda. The error message says that it's trying to load the Cuda libraries
http://andersbll.github.io/deeppy-website/installation-guide.html

Thanks! I'm not quite sure how to do that, but if I unset CUDARRAY_BACKEND, my test fails completely - windows error, no console output at all.  Same thing if i set it to numpy.  If I set it to cuda, I get the error message from my comment above (ArrayData import fail).

The result of "import cudarray; print(cudarray._backend)" is "numpy".

Did you run that command to install Cudarray?
python setup.py --without-cuda install

I'm running it again!

(Because I've tried the CUDA install a few different times now.)
Anyway, that setup did not create any setup error messages.  But I still get an error when I try to generate an image. And I'm possibly making things worse by trying to fix this in various ways and not remembering which things I might have undone.  I think I've spent 4 hrs.
Where I'm at now:
verify-backend.py:
import cudarray
print(cudarray._backend)
output:
CUDArray: Failed to load CUDA back-end.
Traceback (most recent call last):
File "verify-cuda-backend.py", line 1, in
import cudarray
File "C:\anaconda\lib\site-packages\cudarray-0.1.dev0-py2.7-win-amd64.egg\cudarray_init_.py", li
ne 20, in
from .cudarray import *
File "C:\anaconda\lib\site-packages\cudarray-0.1.dev0-py2.7-win-amd64.egg\cudarray\cudarray.py", li
ne 2, in
from .wrap.array_data import ArrayData
ImportError: No module named array_data

What strikes me as interesting about my situation:

Cudarray would not even build unless I ran setup.py with the "--without-cuda install "

neural_artistic_style.py ONLY runs (produces console output) if I set the environment variable CUDAARRAY_BACKEND=cuda ... Everything else is a full on windows exception.

How weird that 1= "no cuda" and 2="cuda". This does not seem right. But it's the only combination that's even given me output to even get the error message about ArrayData.
And for the life of me, I can't google or find anything about ArrayDatay in conda.exe or anywhere.

I've faced a lot of installation issues on Ubuntu (I'm a big Windows guy...) but in the end I made it work. It took me a while too.
On the very first error, do you have more logs? Maybe in the Event Viewer?
Did you install Python 64bits?

I have a hard time with things like these, even though I'm a command-line
guy, because I live in my own proprietary world, haha.  Windows+TakeCommand
for 20+ yrs + Cygwin.
I think this might have been the error:
Faulting application name: python.exe, version: <IP_ADDRESS>, time stamp:
0x56abcaee
Faulting module name: ntdll.dll, version: 6.1.7601.19160, time stamp:
0x56bcd74c
Exception code: 0xc0000005
Fault offset: 0x000000000004a604
Faulting process id: 0x5010
Faulting application start time: 0x01d180d7154c6a4b
Faulting application path: C:\anaconda\python.exe
Faulting module path: C:\Windows\SYSTEM32\ntdll.dll
Report Id: 53ca1384-ecca-11e5-a8a3-001167c2c87c
and when I reproduced it, this is what popped up:
Problem signature:
Problem Event Name: APPCRASH
Application Name: python.exe
Application Version: <IP_ADDRESS>
Application Timestamp: 56abcaee
Fault Module Name: ntdll.dll
Fault Module Version: 6.1.7601.19160
Fault Module Timestamp: 56bcd74c
Exception Code: c0000005
Exception Offset:<PHONE_NUMBER>04a604
OS Version: 6.1.76<IP_ADDRESS>.256.1
Locale ID: 1033
Additional Information 1: 3935
Additional Information 2: 39350178b1f7dc4d15d5471cf2a5a09f
Additional Information 3: 6976
Additional Information 4: 697614677dd37f475b8d30bce4aa474e
Thanks :)

but anyway, the error that i'd actually like to try to fix is this one:
ImportError: No module named array_data
...Because that's the situation where DeepStyle is the closest to running -- it actually produces STDOUT at least :)

I'd let you remote into my machine, too, if it would help.  That might be a nightmare though.  It seems like the error is simply that some module called array_data is not being loaded. It's weird because I can't find reference to array_data anywhere.
I really wanted to start running this on permutated sets of images...


Print exception origin in crash messages

We use PrintExceptionContinue in many places and we pass crash origin to it as a param but we never use it. Also, change the param name to better match its current meaning.

backported in https://github.com/dashpay/dash/pull/3670


Plugin broken in 16.11

Remove use of C4:Branches in EDS.pm
Remove use of C4 Branches in eds-search.pl
Replace
my $branches = Getbranches();
with
my $branches = { map { $->branchcode => $->unblessed } Koha::Libraries->search };
Remove use of C4::Ratings in eds-search.pl

Many thanks - that worked mostly; I get the following error when trying to get the branches. I'll take a look at what other Koha core files are doing to fix this.
Software error:
Bareword "unblessed" not allowed while "strict subs" in use at /var/lib/koha/library/plugins/Koha/Plugin/EDS/opac/eds-search.pl line 392.
Execution of /var/lib/koha/library/plugins/Koha/Plugin/EDS/opac/eds-search.pl aborted due to compilation errors.

In eds-search.pl:
use Koha::Libraries;
then make that line above:
my $branches = { map { $->branchcode => $->unblessed() } Koha::Libraries->search() };

my $branches = ''; did the job... Able to get the opac record without branch details.
Thanks for raising this.


logpush job deleted outside of terraform causes unexpected error

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.1.7
on darwin_arm64
+ provider registry.terraform.io/cloudflare/cloudflare v3.27.0

Affected resource(s)

cloudflare_logpush_job

Terraform configuration files
resource "cloudflare_logpush_job" "http_requests" {
  enabled             = true
  zone_id             = var.zone.id
  name                = "full-edge-logs"
  logpull_options     = "fields=RayID,OriginIP,<and-more>&timestamps=unixnano"
  destination_conf    = "s3://<bucket>/logs/..."
  ownership_challenge = data.aws_s3_object.challenge_file[0].body
  dataset             = "http_requests"
}

Link to debug output
https://gist.github.com/vroy/dbc0b94ab34ff3c21c820a96a0f54ba9
Panic output
No response
Expected output
For the provider to correctly handle 404 errors as "resource does not exist".
Actual output
│ Error: error reading logpush job '..' for zone (<ZONE_ID>): job not found (1000)
│
│   with module.cloudflare_logs[0].cloudflare_logpush_job.http_requests,
│   on ../../../modules/cloudflare/logs/main.tf line 135, in resource "cloudflare_logpush_job" "http_requests":
│  135: resource "cloudflare_logpush_job" "http_requests" {

Steps to reproduce

Create logpush job using Terraform Provider
Delete logpush job outside of Terraform (we do this for reasons)
Remove resource from Terraform Configuration
terraform plan errors on not being able to find

Additional factoids
This used to work in Cloudflare provider version 3.10.1.
The error handling here looks for "404" in the error message: https://github.com/cloudflare/terraform-provider-cloudflare/blob/8ae0ac9d9905a070eee372d4b58d33c39efc5a98/internal/provider/resource_cloudflare_logpush_job.go#L109-L114
It appears error handling in the go SDK was updated between these two provider versions.
The error handling in getLogpushJob does not wrap and perhaps leads to "404" not being found in the error message anymore? Specifically this line: https://github.com/cloudflare/cloudflare-go/commit/7c5d7f8629a118b9991303c541866f478501d943#diff-a351491bfb6362cd854fe0e7e1193665f9462282e406f7f0c5241affacd07705L412
References
No response

I linked to a gist that contains the relevant information 🤷🏻♂


Use the wording 'rayon job' for with_min/max_len

Because it depends on scheduling and job size how many jobs are
processed per thread, it seems more consistent to say rayon job here.

Makes sense, thanks!
bors r+


audio feature extraction using librosa

The inputted 'mp3/wav' file should be converted to 'numpy' array for feeding to model.

why numpy?
spectrogram analysis alleda?

Numpy to reduce complexity

Spectrogram indakiyekana aa numpy array vech aada,


calculate-min-fee with signed transaction body

The calculate-min-fee command should support the following scenarios:

The user submits an unsigned tx body specifying the number of byron/shelley witnesses they desire.

In the cli we construct dummy witnesses (byron if needed or shelley or both) to construct a "bogus" tx and call getMinFeeTx. We return only the fee in this case otherwise the users will have to remove the bogus witnesses if we returned the resulting tx. Constructing the dummy witnesses will allow us to avoid implementing something like calculateByronWitnessFees.

The user submits a signed tx or an unsigned tx body along with the signing keys they intend to validate the tx with.

In this case we should return the re-signed transaction. The transaction has to be re-signed after the minimum fee is updated and the user will receive a valid tx they can immediately submit.
Consider returning an error if the number of witnesses in the signed tx does not equal the number of signing keys (0 supplied signing keys is fine and is actually the third scenario).

The user submits a signed tx without the signing keys needed to validate the tx. We simply return the fee as we can't return a valid tx.

As referenced in this comment on PR#534
The first scenerio is currently being worked on/completed. This issue only pertains to the second and third scenerio.

Not sure what we want to do with this one. The thread that started this stream of work got quite deep: https://github.com/IntersectMBO/cardano-cli/pull/534
cc @Jimbo4350


Write a script to put initial conditions and photolysis rates in MICM driver format

NOAA to handle further testing