Block orientation not respected when disassembling a ship

(I checked briefly to see if this issue has already been posted, but I only used the search bar, so it's possible this is a duplicate. Apologies if so.)
When a ship is assembled with a block that has an 'orientation' (such as the ship helm or a bed), turning the ship 90 degrees and disassembling it will result in all oriented blocks returning to their original orientation, which no longer matches up with the ship.
As an example: The two individual blocks of a bed will remain facing north-south if originally placed this way, even if the ship is rotated to face east-west. This can be fixed by reassembling the ship and rotating it again, but that seems a little bit silly.
The expected behavior would be the blocks being rotated to match the change in the ship's change in orientation from its original assembly point.
[This is usually represented as a 'facing' or 'axis' block tag. This doesn't appear to be an issue for logs, for whatever reason, but this is still an issue with stairs, for example. This does not seem to affect things that face upwards or downwards.]

Bizarrely, I cannot seem to replicate this now that it's happened. I've rebuilt my ship to the exact same specifications but for the life of me I can't get it to work.
It may be because I disassembled before aligning, or might be something to do with a movement key being input to rotate the ship as it disassembled.

Some basic experimentation later and it seems like it might have something to do with the original chunk the ship was in being unloaded, but this is HEAVY speculation, I really have no clue.

very-old and long-known issue
planned on being fixed "eventually" aka: when they have nothing better to do

If you assemble pointing, for instance, north, and then at some point later on, disassemble pointing east, it will bug out and rotate blocks like stairs, fences, and what not
if you do that again, disassembling pointing south since you assembled pointing east (90° clockwise/to the right) will actually fix itself!


:broom: Migrate to k8s api 0.22

Changes

:broom: Migrate to k8s api 0.22

/kind cleanup
Closes #137

/approve
/lgtm
cool, thanks!


build(deps): upgrade npm dependencies

:tada: This PR is included in version 5.0.0 :tada:
The release is available on:

v5.0.0
GitHub release

Your semantic-release bot :package::rocket:


permissions: records always in community feature

:heart: Thank you for your contribution!
Description
Closes inveniosoftware/product-rdm#183
This PR adds an option to prevent deleting all communities in a record except for the last one, as our org requires at least one community per record. This feature is off by default and can be activated in your instance by adding:
RDM_ENSURE_RECORD_COMMUNITY_EXISTS = True

Then you will still be able to remove communities, but not the last one.

This PR is connected to this package we create, it prevents submitting any record without selecting a community from the API level. and prevents users from creating communities unless they have certain privileges that we set for the theme programmatically.
Future plans are to implement this as well with the master branch.
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[x] I'm aware of the code of conduct.
[x] I've created logical separate commits and followed the commit message format.
[x] I've added relevant test cases.
[x] I've added relevant documentation.
[x] I've marked translation strings (for relevant code).
[x] I've followed the CSS/JS and React guidelines (for relevant code).
[x] I've followed the web accessibility guidelines (for relevant code).
[x] I've followed the user interface guidelines (for relevant code).
[x] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[x] I've NOT included third-party code (copy/pasted source code or new dependencies).

Third-party code
If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

This https://github.com/CERNDocumentServer/cds-rdm/issues/88 will close this PR once resolved.

Finally, closing in favor of https://github.com/inveniosoftware/invenio-rdm-records/pull/1813


A lot of PBI Screen shots are not visible in this page

[Enter feedback here]
A lot of PBI Screen shots are not visible in this page. Using edge. only brokens icons with description what you are supposed to see...

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

ID: 744f5d32-18ed-9018-3003-c5f9bcb3d1ff
Version Independent ID: e2123a9f-edef-5170-05c6-98d42bc9fd35
Content: Find content from Power BI service Home - Power BI
Content Source: powerbi-docs/consumer/end-user-home.md
Service: powerbi
Sub-service: pbi-explore
GitHub Login: @mihart
Microsoft Alias: mihart

Thanks for your comment, @ehsf973. This issue has been assigned to the doc author for review.


New rule: spacing around bangs

Allow us to lint the spacing around a bang (before and after).
Scsslint currently do this with one rule, bangFormat, but I feel we should continue in the current style of whitespace rules and have two, space-before-bang and space-after-bang. Thoughts?
Happy to do this either way  if it gets the thumbs up.

Space before/after bang works for me


Extension host terminated unexpectedly in vs code 1.41.1

Hi there, I'm using vs code 1.41.1. Recently i'm facing this problem "Extension host terminated unexpectedly". After googling this problem, most suggestions was to uninstall Live Server. So i tried to uninstall & disable the extension, but the message was "Cannot uninstall/disable extension live server". Please give hints to solve this problem.

Sorry, this two links are not helpful to solve my problem. They are suggesting to uninstall extensions, which is not working in my vs code.

This may help you: https://github.com/ritwickdey/vscode-live-server/issues/285
/causedByExtension


Render map on Tizen (watch)

We try render .mbtiles offline map on TizenXaml Application.
We use Xamarin.Forms for front page. Like in this link (https://github.com/Mapsui/Mapsui/blob/master/docfx/documentation/Mapsui.Forms.md) , we try use Mapsui.UI.Forms, and get from there MapControl. From back we set NameMapControl.Map.Layers.Add(OurTileLayer).
In debug we shure that it creates, but on front page dont render MapControl.
So how can we render this control? Any solutions or advise for it?

I am not aware of any user on Tizen. Perhaps the best first step would be to use the OpenStreetMap layer, like in this sample:
https://github.com/Mapsui/Mapsui/blob/master/Samples/Mapsui.Samples.Common/Maps/OsmSample.cs


Allow passing Parameter instances to Param pane

This is a bit of convenience to easily add single parameter widgets, i.e. this code:
 pn.pane.Param(parameterized, parameters=['parameter'], show_name=False)

becomes:
pn.pane.Param(parameterized.param.parameter)

Codecov Report

Merging #303 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #303      +/-   ##
=========================================
+ Coverage   89.26%   89.3%   +0.03%
=========================================
  Files          63      63
  Lines        6504    6528      +24
=========================================
+ Hits         5806    5830      +24
  Misses        698     698

Impacted Files
Coverage Δ

panel/param.py
92.12% <100%> (+0.1%)
:arrow_up:

panel/tests/test_param.py
99.66% <100%> (+0.01%)
:arrow_up:

Continue to review full report at Codecov.

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

From the code, it looks like this would also work?
pn.Column(parameterized.param.param1, parameterized.param.param2)

That seems like a more common pattern than pn.pane.Param(parameterized.param.param1).  But then there would be no label for each widget?

Sounds good!


Multiline strings get indented

This refactoring, similar to what I actually used:
import ast
import refactor

class WrapF(refactor.Rule):
    def match(self, node: ast.AST) -> refactor.BaseAction:
        assert isinstance(node, ast.Constant)

        # Prevent wrapping F-strings that are already wrapped in F()
        # Otherwise you get infinite F(F(F(F(...))))
        parent = self.context.ancestry.get_parent(node)
        assert not (isinstance(parent, ast.Call) and isinstance(parent.func, ast.Name) and parent.func.id == 'F')

        return refactor.Replace(node, ast.Call(func=ast.Name(id="F"), args=[node], keywords=[]))

refactor.run(rules=[WrapF])

produces this:
 def f():
-    return """
-a
-"""
+    return F("""
+    a
+    """)

This changes the value of the string.
Possibly related is https://github.com/isidentical/refactor/issues/12, but I couldn't reproduce an equivalent problem with just ast.unparse:
import ast

source = '''
def f():
    return """
a
"""
'''

tree = ast.parse(source)
node = tree.body[0].body[0].value
call = ast.Call(func=ast.Name(id="F"), args=[node], keywords=[])
ast.copy_location(call, node)
ast.fix_missing_locations(call)
print(ast.unparse(node))  # '\na\n'
print(ast.unparse(call))  # F('\na\n')

Interesting and not thought about in: https://github.com/isidentical/refactor/pull/66, which partially solve the issue with a line, but because it is f-string'd with parenthesis, still applies the indent for """ line - Looking into a workaround to improve the PR


Preserve file creation meta data on the file itself

Description
Currently, if you download your media using xbox2local, all files will be named for their creation timestamp, but all have essentially the same file creation date history on the OS... It is possible to modify these values using python, it would be a nice bonus if xbox2local could preserve this data in a way that keeps your media sorted based on when it was actually created (outside of the file name), which would enable OS level file searching, filtering, to be usable then as well for managing backups.
Can you help develop and maintain this feature?
Maybe - Python isn't my primary language, so I am having to relearn things trying to make this work, if I solve it, I will absolutely create a pull request, but right now, I am about 0 / 5 on attempts at anything that doesn't throw new errors.

I have a working example (no feature or CLI flag -- its just enabled full time -- honestly I dont see a reason to not use it for data accuracy). Will clean it up for a PR and wait to discuss?

Hi @ktiedt! Sorry for the slow response. My day job keeps me very busy and I only get to work on random projects sporadically. I completely agree that storing the metadata pulled from the API calls is a good idea and I can give that some thought. Thanks for taking a first stab at it with #23.
One note: your screenshot/clip filenames should reflect the capture date/time—which is the time the screenshot or clip was recorded on your Xbox as reported by the API call—not the time that the file was written to your local device.

Now that I'm looking into this more carefully, could you describe exactly what you're hoping the filenames will look like after this change? For example, I have the following rows in my history.csv file (spacing added for readability):
id,                                   game,                           type,       capture_dt,          download_dt,         xboxnetwork, width,  height, sdr_filesize, hdr_filesize
9fd619e2-2b6b-4820-a4a0-a67a2f5201f9, Assassin's Creed IV Black Flag, screenshot, 2015-09-06T07-38-59, 2023-01-02T22-48-23, True,        1920.0, 1080.0, 3696843,      0
0b6e1fbc-881f-404c-b2fc-34eed3d6f95f, Batman™ Arkham Knight,          gameclip,   2021-02-26T04-39-11, 2023-01-02T23-02-06, False,       1920.0, 1080.0, 103053018,    0

The corresponding files on my filesystem are: Assassin's Creed IV Black Flag/2015-09-06T07-38-59.png and Batman™ Arkham Knight/2021-02-26T04-39-11.mp4, respectively, following the capture_dt variable as expected. What would you prefer these filenames look like? And what extra information would you want stored in the history.csv file that isn't there yet?

Sorry if I didn't describe it well. My goal wasn't to modify the file name or the meta data file but to modify the files creation dates to enable file system sorting and searching features to work.
I am syncing my clips to google photos and without this change all clips get synchronized with the same date which is the download time (so essentially all identical). With this update, they get assigned to their original date that they were taken (at least on OSX - admittedly I didn't think to test on any other OS).

Thanks! The PR does achieve it, most definitely a better way I imagine, but I've been running this since it made the PR and it hasn't failed yet.

Right, I'll build off of what you've done in the PR and see if I can get a simpler but correct implementation.


Add ng-annotate loader

Added ng-annotate loader. This will resolve #61.
Also added ng-strict-di directive to ensure that ng-annotate handles sources correctly.

LGTM


Feature/#25 avl tree

Coverage decreased (-3.5%) to 92.034% when pulling 4c5afa352fc34add70bf045422df27a77a016430 on feature/#25_avl_tree into 4fb4545e92b32a844413560f3efef06a85a04128 on master.


The argument "region" is required, but was not set.

Describe the Bug
using version 0.23.2
a provider region is required but not set.
my providers have all got region set correctly.
│ Error: Missing required argument
│
│ The argument "region" is required, but was not set.

Expected Behavior
the following runs
module "dynamodb_user" {
  source = "cloudposse/iam-system-user/aws"
  # Cloud Posse recommends pinning every module to a specific version
  version     = "0.23.2"

  namespace   = "production-eu"
  stage       = "production"
  name        = "dynamodb"

  policy_arns_map = {
  }

  inline_policies_map = {
    dynamodb = data.aws_iam_policy_document.dynamodb_policy.json
  }

  providers = {
    aws = aws.production-eu
  }
}

disregard just found
https://github.com/cloudposse/terraform-aws-iam-system-user/releases/tag/0.23.0


directly synchronize S3 buckets in OpenTUNA

Is it possible to synchonize & provide service directly through S3? It would be useful for repos such as hugging-face-models and dart-pub that are hosted on S3 originally.

There is a solution to sync s3 buckets across partitions.


osc~ playing at the wrong pitch

Windows 8.1.
Chrome  42.0.2311.90
Pd 0.46.5
[osc~ 440] in the patch is playing at 478.9Hz in the browser. Possibly a sample rate issue? The ratio is the same as 48000/44100. Windows runs at 48KHz by default.

Does this only occur in Chrome? Does the same problem exist in Firefox? IE? (I dunno even if it runs in IE!)

The Owl runs at 48KHz as a default, so Windows and browser should be congruent. My Mac runs on 44.1KHz. So i have to make sure to loadbang the samplerate~ object to distribute current sample rate to tabread~ etc not to run into pitch problems

This is still open right?

@woldhek I wasn't able to figure out what's different about it running in Chrome on Windows, doesn't seem to make sense to me. Do you still have this problem?

yeah still the same.
On Wed, Feb 8, 2017 at 7:52 PM, Joe White<EMAIL_ADDRESS>wrote:

@woldhek https://github.com/woldhek I wasn't able to figure out what's
different about it running in Chrome on Windows, doesn't seem to make sense
to me. Do you still have this problem?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/enzienaudio/heavy/issues/37#issuecomment-278425051,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFlq4-EOHcfW4VPSu8Aas7XkwBbwI-3Rks5rag8GgaJpZM4EL9L4
.

@woldhek this was a silly one, can you try refreshing your page and see if you still have the same problem? 🤕

@ntkeep very sorry about the 2 year wait on this :/
It's kinda like how you find your keys in the last place you look. And I can't really say it's due to analogue drift or anything...
Anyway fixed now so you can enjoy perfect pitched pipes everywhere!

No worries! There was nothing critical about it. Thanks for the fix!


Coding question

I really apologize a lot about asking here but I have been banned on the discuss.px4.io .So I don't know where else can I ask , this might be my last question also .So please can you guide me on how the velocities are given to each motor or in other words how's the motors controlled . Where can I find the code handling the velocity of motors ?

https://dev.px4.io/en/concept/architecture.html


UDP Socket on Loopback address <IP_ADDRESS>

Hello,
I am trying to bind a UDP Socket on loopback address <IP_ADDRESS> using ofp_socket and ofp_bind.
I have created the local interface by calling:
ofp_config_interface_up_local(0, 0, inet_addr("<IP_ADDRESS>"), 8);
and then I call ofp_scoket and ofp_bind
I receive an error: "Cannot assign requested address"
Am I missing something?
Regards,
Neetika

Hmm... I may need this feature. I'll have a look.

Do you have SP enabled?
So far, my best guess is that is waiting for a message from netlink (ofp_netlink.c) before setting the address ...and for some reason is not arriving.

Yes, I have SP enabled, and from the traces it seemed to me that the IP
Address gets plumbed OK
But it refuses to let me create a UDP socket on the same.
On Wed, Aug 19, 2020 at 4:05 AM Bogdan Pricope<EMAIL_ADDRESS>wrote:

Do you have SP enabled?
So far, my best guess is that is waiting for a message from netlink
(ofp_netlink.c) before setting the address ...and for some reason is not
arriving.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenFastPath/ofp/issues/257#issuecomment-675753275,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQB7SBSKRDEIWVD7N56KPTLSBL63VANCNFSM4PP2CEGQ
.

The logic around loopback interface is a little bit odd: probably netlink was responding differently in the past.
Logic is like this:

add an interface
wait for netlink to inform you about link up and new address being set on linux 'lo' interface.

Now, 'link up' does not arrive (interface index under Linux is not set) and 'new address' does not arrive for <IP_ADDRESS> as it is already set (by Linux).
The best workaround so far:

flush <IP_ADDRESS> address from linux 'lo'
ip addr flush  dev lo
Modify code to set the 'correct' (in my case) linux interface index
ofp_config_interface_up_local()
{
...
data->linux_index = 1;
ofp_update_ifindex_lookup_tab(data);
...
}
configure through ofp cli file the loopback interface
$ cat ./ofp_lo.cli
loglevel set debug
ifconfig fp0 <IP_ADDRESS>/24
ifconfig lo0 <IP_ADDRESS>/8

Now, I can see address <IP_ADDRESS> set on the lo0 in the CLI ... don't know yet about bind.

Thanks Bogdan,
I will try these :-)
Could you suggest me if we should be using the development branch of OFP?
We are currently using the release 3.0.0 which as per the details got published in Dec 2018.
We are facing a few issues in our IPv6 testing and we feel that it could be since we are using a very old code base.
Is there any stability issues in the development branch.
Also, is any new release expected soon?

Hi Neetika,
I am trying to find complete fix for this problem but basically I don't have enough time.
Please take a look at final commit from https://github.com/bogdanPricope/ofp/tree/loopifs
Basically it gets the interface index (as reported by linux) and set it on the interface.
You still need to:

flush 'lo' before starting ofp application
ip addr flush dev lo
configure the ip on the interface in cli:
ifconfig lo0 <IP_ADDRESS>/8
or with ofp_config_interface_up_local(...)

With that, you should be able to bind and send/receive packets.

I am (from the beginning of 2020) an  independent contributor to ofp... so all opinions are my own and not 'official'.
My opinion is that current version of development branch is better (fixes, etc.) but it requires an update of odp / dpdk. I know that not all the project owners are willing to do that.
IPv6 was not requested by many customers... so was not fully developed or tested.
I have no idea when the next release will take place (Nokia & friends  can answer to this question).

I am contributing to the OFP (and to my own forked version of it) in my spare time but if your company is willing to pay for support and development we can discuss.


token_lifespan issue

Version: 1.1.2

Setting config.token_lifespan to less than 2.weeks or less than 24.hours cause headers to disappear. Also setting config.max_number_of_devices to any other value and config.token_lifespan to less than 2.weeks cases the same issue

I couldn't get the headers less than 2.years

So, after changing  token lifespan to for example 10 minutes (lesser then previous), what happened after that is we've old generated tokens with lifespan 2 months (old value) associated with the user, and we have a concept in that max_devise and limit is 50, whenever we generate a new token, it deletes the token based on expiry, so in our case, after sorting all the tokens based on expiry it is deleting the new one because the new token has an expiry of 10 minutes.
To make it work this we need to delete all the previous tokens associated with a user, which means we need to logout all the users
https://github.com/lynndylanhurley/devise_token_auth/blob/6d7780ee0b9750687e7e2871b9a1c6368f2085a9/app/models/devise_token_auth/concerns/user.rb#L238
 def clean_old_tokens
    if tokens.present? && max_client_tokens_exceeded?
      # Using Enumerable#sort_by on a Hash will typecast it into an associative
      #   Array (i.e. an Array of key-value Array pairs). However, since Hashes
      #   have an internal order in Ruby 1.9+, the resulting sorted associative
      #   Array can be converted back into a Hash, while maintaining the sorted
      #   order.
      self.tokens = tokens.sort_by { |_cid, v| v[:expiry] || v['expiry'] }.to_h

      # Since the tokens are sorted by expiry, shift the oldest client token
      #   off the Hash until it no longer exceeds the maximum number of clients
      tokens.shift while max_client_tokens_exceeded?
    end

I've also experienced this behaviour when reducing token lifespan. The new token gets instantly deleted and auth headers are not returned. An option would be to delete tokens by the last updated field (on last token) instead of the expiry field

@AndresAbed @rohmanitsavirus @ankitkhadria1 @daniel-koudouna curious if any of y'all has gone with a better workaround or solution to this issue?


[docs] Add description for max topics per namespace

Add description for https://github.com/apache/pulsar/pull/8942.

@Anonymitaet I am not sure about where to add this parameter, so I added it at the end of each section. Please let me know if I should remove it to the middle. Thank you.

@sijia-w thanks for your contribution, you can ask help from the author as well. @hangc0276 could you please help review? Thanks

@sijia-w some checks were not successful, you can comment /pulsarbot run-failure-checks to rerun the tests.

/pulsarbot run-failure-checks


test(e2e): remove outdated bluewallet test

why remove it and not just use a different lndhub URI?

why remove it and not just use a different lndhub URI?

Ah, I didn't know about this. Was just wondering why it started to fail. Thanks, updated. All good.


Writing to flash, crashes dvi

Hi Wren6991!
I want to know, if there is a way, to store data in the flash, whitout crashing the app, i'm using a temporary register to store interruptions, do write data cycle, and return interruptions, but dvi crashes, reboot pico, and the data is correctly store, but i need to reboot pico manually.

Auto responding me:
Do an auto fisic reset.


How do I know what listings i am able to fulfill

As a marketplace owner for v2 storefront, how can i fetch information about what listings i can fulfill or not?
The list of addresses that can fulfill an listing is not exposed and there is not method to check if I am allowed to do so or not.

As a marketplace owner for v2 storefront, how can i fetch information about what listings i can fulfill or not?
The list of addresses that can fulfill an listing is not exposed and there is not method to check if I am allowed to do so or not.

If I understand correctly what you wanna say here -  Is how does the marketplace know which listing it should facilitate? If yes then during the creation of the listing marketplace can pass an identifier so they can easily filter those listings that they created and show them on their UI and other listings (those are not created from their platform) they can facilitate if they wanna earn sweet commission and keep an eye which listing offering the commission and whether there address is listed init or not.
Does my interpretation of your question right?

Let say that .find in our NFT detail page want to know what listings we can fulfill or not. From my understanding it is impossible to do this using the listing resource on chain. I can check if the given NFT for the given user has a listing, but there is no way for me to know if I can fulfill it or not, or if I cannot to link to the origin site where it can be fulfilled.

it is not possible, you need to access :
https://github.com/onflow/nft-storefront/blob/6a3982ebc55d053542c25cdf98780e4450a2bb95/contracts/NFTStorefrontV2.cdc#L261

I think it would be useful if a marketplace wants to use this limitation to create private sales that they should also provide a link to where this sale item can be fulfilled.

@satyamakgec  to clarify, for listing that make use of  "an optional list of marketplace
receiver capabilities. In this case, the commission amount must be transferred to one of the capabilities in the list..."
We should consider exposing something in the listing event, perhaps a flag, to allow the community to know if, or if not, they are able to capture the commission on a given listing, at a glance (we should avoid having to run a script to check every listing) @bjartek did I capture the issue you raised here? Do you have any thoughts for how we could address this?

@satyamakgec, I think some business logic I am missing.

an optional list of marketplace receiver capabilities.

I don't know if there are any storyboards for StoreFront, but this didn't add up for me, what is the use case here?
For example bloctoBay on listing will add Viv3 ? Or TopShot marketplace will add bloctobay there?

Yeah this is very tricky, marketplaces would like to keep their inventory. ( or they die )
Not sure if any sane marketplace would like to share.

Marketplaces could make use of saleCuts to ensure they are paid. It seems to me that the reason to use the marketplacesCapabilitiy is in case the commission is abused. The idea initially was that Dapps, not marketplaces, would use the commission as a way to entice marketplaces to list an item for sale. And in that case, it might make sense to limit the ability of capturing a commission to a list of known marketplaces.


CI configurations are inconsistent

appveyor.yml doesn't contain rake features while circle.yml does.

Problem is in running rake features in Appveyor. This not work, problem find executable file, some windows path specific problems. We need someone with windows to solve this problem.

Maybe @XVincentX could help?

I'll try to have a look.

I tried to fixed this. Unfortunately no luck. I tried it with RDP to Appveyor, but even I added all paths manually it did not work. I also tried to use relative paths to binaries but again it did not solve this problem.


gparyani: Watch<PHONE_NUMBER>

gparyani requests the watch of the watch_number<PHONE_NUMBER>. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Ryan M in Charcoal HQ


Re-engagement of OP immediately after disengagement, sometimes only actvates stock ACC

Describe the bug
After pressing the gas pedal to merge into traffic and hitting the resume button again to get OP to take over, VWs stock ACC system engaged, however OP was still disengaged.
How to reproduce or log data
I can share the data privately, please PM me Semi-Lucent on Comma.ai Discord.
It seems that comma explorer doesn't show the disengagement in the timeline, however it did indeed disengage with the gas pedal. I tried to re-engage afterwards and the stock ACC system engaged instead of OP, I had to cancel ACC and then resume again for OP to work again. This can be seen as the small brake in engagement on the timeline after 22:38:00, or 1000 on cabana on date Tue, 13 Oct, 2020.
Expected behavior
OP to have fully disengaged, and been aware that it disengaged, then re-engaged fully afterwards when the resume button was pressed.
Device/Version information (please complete the following information):

Device: Comma 2
Version:   openpilot v0.7.9-VW-release release2
Car make/model: 2019 Volkswagen Golf R

Additional context
Afterwards, I noticed that the "Max speed" display wasn't showing what the ACC was set to, while OP was working. I had to reboot my C2 to get that readout to work again, though the rest of OP seemed to be working. Not sure if this issue is related at all or not. I noticed afterwards, but it very well could have been happening before and gone un-noticed by me and be totally unrelated to the above.

Sorry I haven't looked at Issues in awhile. This concern is very old, and probably not applicable to the version of openpilot you're running now. We've spoken on Discord recently, just let me know if this is still a repeatable concern.

Yeah, this weirdness has been gone for a while. I probably should have circled back too. Thanks as always James! :)


Compile, js func split() error

I guess the split() js function tries to run on an empty logfile or sth.
Error is below:
Uncaught TypeError: logfile.split is not a function
at Object.success (request_game_assepile.js?ver=4.8.2:108)
at i (jquery.js?ver=1.12.4:2)
at Object.fireWith [as resolveWith] (jquery.js?ver=1.12.4:2)
at y (jquery.js?ver=1.12.4:4)
at XMLHttpRequest.c (jquery.js?ver=1.12.4:4)

Yes fixed now


To fix CPU tensor issue

use the tensor containing scalar instead of 1-D tensor. this would avoid "one of the tensors on CPU" error.
What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

Seems a number of models use prepare_decoder_attention_mask, so ran a few regressions to ensure transformers work alright with this change, following pass:

fast_tests
fast_tests_image_classifications
fast_tests_feature_extraction
fast_tests_image_segmentation
fast_tests_feature_extraction
fast_test_videomae
slow_tests_image_to_text_example

Yet to run: slow_tests_text_generation_example

@regisss , please take a look

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


not comptiable with nginx 1.15.2

installed with 13.9 as dynamic module.
after upgrade to 15.2 by command
apt upgrade
got this error
nginx -t
nginx: [emerg] module "/etc/nginx/modules/ngx_pagespeed.so" version 1013009 instead of 1015002 in /etc/nginx/nginx.conf:6
curl -I https://sarkariresultz.in
Date: Mon, 06 Aug 2018 12:13:08 GMT
X-Page-Speed: <IP_ADDRESS>-0
compiled in 1.13.9 and using in 1.15.2

again complied with nginx 1.15.2 problem fixed.


[WIP] delete a bunch of deprecated resource_type code

Pull Request Checklist:

[ ] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

[Enter positive test case here]

retest this, please

METRIC
VALUE

NEW DUPLICATED LINES DENSITY
5.32258064516129

NEW CODE SMELLS
11

NEW BUGS
10

NEW VULNERABILITIES
0

NEW SECURITY HOTSPOTS
1

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-3841


preserving layout

Hi,
I was curious as to if you tried to keep layout when OCRing the image.
I find information on the internet that gives me hope that it's possible
I don't need this myself I think but I can imagine people asking for this.
I would gladly do a PR if I managed to pull it off but I can't make your addon work for now.

Thanks for this, definitely a stretch goal as this addon was originally intended just gather text for card searchability. That said, it would definitely make the addon more useful.
I don't have a huge amount of time, hopefully, we can get your install working if you want to have a play.

Gladly!


Fails on Windows 10

I traced it down to this line where filter_var returns false:
https://github.com/theofidry/cpu-core-counter/blob/3a4682016af125f1f41d125798544641d49b79ae/src/Finder/ProcOpenBasedFinder.php#L91
The content of the $process variable was:
NumberOfLogicalProcessors
8

This was my testing script:
use Fidry\CpuCoreCounter\CpuCoreCounter;

echo (new CpuCoreCounter())->getCount();

Hi thanks for the report! You could tell me on which version this happened? I would expect it to have been fixed with https://github.com/theofidry/cpu-core-counter/pull/83 in 4.0.0

Hi, it was version 0.4.0 according to composer.lock.
Basically I found out about this PR to PHPStan https://github.com/phpstan/phpstan-src/pull/2047 and I used the same version to test after noticing it's become much slower for me after a recent update as it was defaulting to running only one subprocess.
Please let me know if I can do anything else to help.

I see, then I don't think it will be too hard to fix. I may be able to take a look at it next week or if you're keen you can also check #83 which should give enough examples on what to touch to fix & test it.


Use docker for OLM test in nightly pipeline

Many thanks for submitting your Pull Request :heart:!
Please make sure your PR meets the following requirements:

[x] You have read the contributors' guide
[ ] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[ ] Pull Request contains a link to the JIRA issue
[ ] Pull Request contains a description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

we should cherry-pick in 1.3.x as same problem occurs there

Cherry-pick: https://github.com/kiegroup/kogito-cloud-operator/pull/761

having a quick run to check olm tests are now passing and then I will merge that PR


Feature/Config file w parameter description

[x] Unit test
[ ] Test in car

Just need to test this one in car, and rebase once radarVIN fix is merged in. @BogGyver


Final: combined conversion from java to Typescript

Great typo time.

Ignoring @codacy-production's Codacy Static Code Analysis, our website project now uses TypeScript for its development.


chore: added unit test file for inspector

Current dependencies on/for this PR:

main

PR #134   👈

This comment was auto-generated by Graphite.


On startup, plugin opens "Acquisition / Reconstruction" tab instead of "LC Calibration" tab

Annoyingly, Qt Creator sets the startup tab focus to the last tab you were editing when you save the .ui file, so when I edit the .ui file I sometimes accidentally change the tab focus.
The current behavior is an example of this problem: on startup the plugin opens to the "Acquisition / Reconstruction" tab instead of the expected "LC Calibration" tab.
@ziw-liu would you mind adding a couple lines to the main_widget constructor to fix the tab focus?

Problematic line:
https://github.com/mehta-lab/recOrder/blob/ecca6ffc53752c45e0eaf03fe2a9621cc2da0faa/recOrder/plugin/gui.py#L1128
Changing From (1) to (0) solves the issue.
@talonchandler Do we want to modify it in gui.UI_Form or override in main_widget.MainWidget?


Add BAB

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

we are leaving ERC20 and migrating to TRC10
Tronscan
https://tronscan.org/#/token/1002918

I sent an email asking to replace the Basecoin ERC20 for TRC10 and so far this has not occurred. Could you help me?
I don't know how to remove the BAB ERC20
Would you help me?
this is the email sent:
- Please type your reply above this line -
Your request (3342746) has been updated. To add another comment, please reply to this email.
Andrew (Trust Wallet)
Jun 20, 2020, 4:35 PM GMT + 8
You need to submit a Pull Request for adding a new token.
Just comment that this is for replacement of the ERC20 one.
Dev
Jun 20, 2020, 12:52 PM GMT + 8
I did not find the tutorial for deleting a token
I just found it to add.
A message had already been sent to kolya182 to remove Basecoin's erc20 token and update TRC10 but there was never a return in more than 60 days.
Can't you do the BAB update and removal yourself (erc20)?
Sent from Email for Windows 10
From: Trust Wallet
Sent: Wednesday, June 17, 2020 10:47 PM
To: Dev
Subject: [Binance] ： Basecoin portfolio change
Andrew (Trust Wallet)
Jun 18, 2020, 9:47 AM GMT + 8
Hi,
Thanks for contacting Trust Wallet Support.
For adding or updating tokens, this has to be submitted to our Github repo.
You can follow this guide:
https://community.trustwallet.com/t/how-to-submit-a-new-token-asset/3863/2
Same guide can be used to request removal of tokens.
Once our devs verify that it complies with the standards then it will be added to the codebase.
Stay Decentralized,
Trust Wallet Support Team
Visit our Help Center for FAQs and Guides about Trust Wallet.
Dev
Jun 18, 2020, 12:20 AM GMT + 8
We are currently migrating the Basecoin token from the ETH network (ERC-20) to Tron (TRC-10)
I wish they could delete Basecoin's ERC-20 portfolio and keep TRC-10. this is causing confusion for users.


Showing wrong date in iPhone and iPad devices

I have used the plugin in woocommerce platform. Please visit https://www.indiadresswala.com/product/apple/. When we choose the start date the end date should be 4 days+ from the current date. For an example suppose I am choosing 22nd December as a start date the end date will be 25th December.
The end date showing the current date it is not changing respective of the current date. The other devices is working fine. Please suggest what is the exact issue.

Please follow the bug report guide: https://github.com/amsul/pickadate.js/blob/master/CONTRIBUTING.md#bug-reports


Unable to watch a clip back - Error with media stream contents

Checklist:

[ X ] I updated to the latest version available
[ X ] I cleared the cache of my browser

Release with the issue: 4.0.0
Last working release (if known): N/A
Browser and Operating System: Debian 11/Windows 10 with Chrome

The live stream works fine, but when trying to click to playback any clip/recording it just shows: Error with media stream contents.
I've checked the troubleshooting but it didn't yield anything. The stream plays fine in the Frigate UI (both JPEG and MSE using go2rtc).
What could it be?
Thanks

Is the clip being recorded with H264 and AAC? Frigate docs mentions that this is a must.

Is the clip being recorded with H264 and AAC? Frigate docs mentions this.

The stream is H264 with no audio.

Have you checked the clip (not the stream) itself? Try to open it in VLC and inspect the codecs.

I meant downloading the clip and playing locally in VLC

But anyway if it works in Frigate UI, then probably the codec is correct.
Let's wait for @dermotduffy

Suggest: Lets get https://github.com/dermotduffy/frigate-hass-card/issues/1010 working first (same reporter), and then we can debug here if there are still issues.

@badnbusy Is this still an issue for you? The new (not yet released codebase) included fixes that made the underlying HLS media player less fragile and more resilient to temporary stream errors. I'm wondering if it has helped?

Yes, I still can't watch clips back within HA, it spins forever as if it can't play the file...

Do they play in the HA media browser (not the card)?

@badnbusy Thoughts on the above key question? If the answer is 'Yes' your problem is not with the card but something with your Frigate setup.
You could also try with the latest build v5.0.0-beta.1 (link) which requires integration version v4.0.0-rc1 and frigate version v0.12-rc2 or later.

@dermotduffy  Thanks. Upgraded everything to the latest and still the same issue.
However, as you've said, I can't watch the recordings back in the Media > Frigate section in HA either, so I don't know what's going on there? I can stream both the substream and HD stream using go2rtc inside HA so what/how do the recordings differ?
Thanks

@badnbusy OK -- so the issue is not with the card. I am guessing it is some class of video codec issue with the recorded media, and your browser cannot play them back (at least in the stream returned for Home Assistant). Suggest opening an issue over in the Frigate repo, as the issue almost certainly is not with the integration either -- but perhaps the way in which you have Frigate itself configured.

Thanks @dermotduffy


Prevent exit when help not found

I have the CLI inside a small shell loop. When urfave/cli finds a command to execute or help for a command, the shell loop continues as usual providing another prompt. When no help is found, the shell loop breaks to the normal unix prompt, with this message "2020/08/18 14:45:43 No help topic for 'gg'". What is it doing differently when it cannot find help and is there a way to stop that?
Thanks, enjoying the package.
Mark

Ok, so I was probably a little lazy before, I can see what it is doing:
"if ctx.App.CommandNotFound == nil {
return Exit(fmt.Sprintf("No help topic for '%v'", command), 3)
}"
So I guess the question is whether there is a way to override it, or if I just need to modify the source. I am guessing the latter.

The way it's laid out is a bit confusing, but you shouldn't have to modify the source. Exit is a function that returns an exit error, where the default behavior for a cli.App is to call os.Exit, but can be overridden.
Take a look at the ExitErrHandler field. As an example, this would disable the exiting behavior completely:
app.ExitErrHandler = func(*cli.Context, error) {}

Thank you for the suggestion. For now, I commented out the fatal log error on the app.run(). I was not able to get the ExitErrHandler to create a different outcome.
//app.ExitErrHandler = func(*cli.Context, error) {
//  fmt.Println("Got to errorHandler")
//}
err := app.Run(args)
if err != nil {
fmt.Println(err)
//log.Fatal(err)
}
This is good for now.
Thanks
Mark

Not sure what the rest of your program looks like, but log.Fatal always exits an application. Glad you got something work.


⚠ Bofrost.de has degraded performance

In 94006bf, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1161 ms

Resolved: Bofrost.de performance has improved in ecd6902.


Dentistry footer added

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fixes
Test URLs:

Original: https://www.sunstar-foundation.org/dentistry

Before: https://main--sunstar-foundation--hlxsites.hlx.live/dentistryfooter

After: https://dentistry-footer--sunstar-foundation--hlxsites.hlx.live/dentistryfooter

[ ] New Blocks introduced in this PR
If yes, please provide details below

Block Name
Documentation

For e.g. cards
doc-link

[ ] New variations introduced in this PR

Variation Name
Documentation

For e.g. cards (grid)
doc-link

The original URL on sunstar-foundation.org shows nothing.
Also I see no difference between the Before and After URLs...
Are those URLs correcrt?

The original URL on sunstar-foundation.org shows nothing.
Also I see no difference between the Before and After URLs...
Are those URLs correct?

Only Footer is different, you can check the after url footer

Ah now I see it. Thanks! 👍

Merging this PR. Thanks @bosschaert for the review.


Document vm.parseJson

See https://github.com/foundry-rs/foundry/pull/2293

FWIW the cheat interface is not yet in forge-std, I'm planning to try to PR that in today or tomorrow but we should probably make sure that gets in first so the info in the docs actually works

Generally agree but I don't think the book should block Forge Std and the other way around until we do more coordinated releases :smile:

Fixed in #584


Remove taggy things from manuals

This PR is basically a revert of https://github.com/alphagov/specialist-publisher/pull/532. That PR changed the publishing flow so that this app would fetch the topics/specialist sectors from the content-api before publishing to rummager.
This was needed at the time, because otherwise the payload sent to rummager wouldn't contain the tags set in panopticon, and the manual wouldn't be displayed on a /topic page.
Since then, we've started indexing the links from the publishing-api in rummager. As part of this we stopped sending specialist sectors to rummager (https://github.com/alphagov/specialist-publisher/pull/723).
At the time of that PR we didn't realise that the only reason for the fetching of tags from content-api was now gone. This PR removes the last (now basically unused) code, which means that we are no longer dependent on content-api.
✌

👏  👍


Cannot mount symlink as volume unless owned by root

Output of docker version:
Client:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.7
 Git commit:   23cf638
 Built:        Fri Aug 19 02:03:02 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.7
 Git commit:   23cf638
 Built:        Fri Aug 19 02:03:02 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 17
 Running: 2
 Paused: 0
 Stopped: 15
Images: 42
Server Version: 1.12.1
Storage Driver: btrfs
 Build Version: Btrfs v4.7
 Library Version: 101
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: host bridge null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.7.2-1-ARCH
Operating System: Arch Linux
OSType: linux
Architecture: x86_64
CPUs: 6
Total Memory: 15.59 GiB
Name: bs
ID: Z7DX:2XOG:LKMA:JLP3:P3DJ:IKYS:T7LY:LFH4:G4G4:VOBX:HT6O:FQSN
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Insecure Registries:
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
Archlinux desktop
Steps to reproduce the issue:
0. su user

ln -s /home/user /tmp/tmp
docker run -it -v /tmp/tmp:/home/user container

Describe the results you received:
docker: Error response from daemon: mkdir /tmp/tmp: file exists.
Describe the results you expected:
Running container.
Additional information you deem important (e.g. issue happens only occasionally):
Everything works fine as long as the symlink is create as a root. It stops working when it's created as a regular user.

sorry, duplicate of #21055


Test the file path with Chinese characters.

@lovell
I encountered a problem on Windows:
Once a image file path contains Chinese characters, sharp will can't handle the image on Windows. But it works well on Mac.
This PR is a test case for reproduce the problem. Could you please solve the problem? Thanks very much!

Coverage remained the same at 99.029% when pulling bf03bd92d0b45abad6dd510a16776a83915231d1 on natural-law:test-chinese-path into 1051fcd278188c5ffe2c0ac4969ba82c94713764 on lovell:master.

Hello, libvips v8.4.0+ is required for Windows unicode support - see https://github.com/jcupitt/libvips/blob/master/ChangeLog#L57
sharp v0.17.0 will provide pre-compiled binaries for libvips v8.4.2 (or later).

@lovell Thanks for your reply.

sharp v0.17.0 will provide pre-compiled binaries for libvips v8.4.2 (or later).

Could you please tell me when the sharp v0.17.0 will be released? Or whether there is a way/document can guide me to update the libvips myself? Thank you very much!

https://github.com/lovell/sharp/milestone/25
The work-in-progress branch is unstable but can be used for early testing:
npm install lovell/sharp#quill


Installation of the package on Google Colab was cumbersome

Follow these steps:
`!pip install --ignore-installed sector
!python -m spacy download en_core_web_lg
!python -m nltk.downloader stopwords
!python -m nltk.downloader wordnet
!python -m nltk.downloader averaged_perceptron_tagger_eng`

Thanks for the details. Yes in collab one has to manually download the dependent models. Let me try to include a example as well if someone wants to try in colab
In local machines I have included download_models.py file to download
Thanks again for pointing it out


Adding a timeout

Hello,
I would like to add a timeout of 2 seconds to all my requests (using NodeJS). I tried modifying the o.js file at line 1450 like this:
var Xhr2 = require('xhr2');
xhr = new Xhr2();
xhr.timeout = 2000;

to no avail. Is there some way I can remedy this issue?
Thank you.

Hi,
xhr2 is an external lib we implemented for node.js support. Normaly your approach should work. If not, it might be an issue with the xhr2 lib implementation. Please open an issue there.
Not quite sure but maybe you can try:
xhr = new Xhr2({ timeout: 2000 });

Regards
Jan

Thank you for the fast answer. I believe you are right, it is working now.
Good day.


Remove support for the HTTP API

The Bolt driver now does everything I need, and I'm not aware of anybody else actually using it.
Remove as cruft.

Removed. Now let us never speak of this again.


Task :react-native-webview:compileDebugJavaWithJavac FAILED with LATEST REACT-NATIVE VERSION: 0.69.0 (ANDROID)

buildscript {
ext {
buildToolsVersion = "31.0.0"
minSdkVersion = 21
compileSdkVersion = 31
targetSdkVersion = 31
"dependencies": {
"react": "18.0.0",
"react-native": "0.69.0",
"react-native-webview": "^11.22.3"
},

For a temporary fix:
update android/build.gradle  to
17 line classpath("org.jetbrains.kotlin:kotlin-gradle-plugin:1.6.21")
25 line classpath("org.jetbrains.kotlin:kotlin-gradle-plugin:1.6.21")

Same issue.
After updating @iamiota temporary fix, I receive this message
❯ yarn android
yarn run v1.22.19
$ react-native run-android
info Running jetifier to migrate libraries to AndroidX. You can disable it using "--no-jetifier" flag.
Jetifier found 968 file(s) to forward-jetify. Using 8 workers...
info JS server already running.
info Installing the app...

6 actionable tasks: 2 executed, 4 up-to-date

FAILURE: Build completed with 2 failures.

1: Task failed with an exception.
-----------
* Where:
Build file 'C:\Users\alysonvn\dev\new_app_corretor\android\app\build.gradle' line: 18

* What went wrong:
A problem occurred evaluating project ':app'.
> Could not find method classpath() for arguments [org.jetbrains.kotlin:kotlin-gradle-plugin:1.6.21] on project ':app' of type org.gradle.api.Project.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.
-----------
* What went wrong:
A problem occurred configuring project ':app'.
> compileSdkVersion is not specified. Please add it to build.gradle

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
==============================================================================

* Get more help at https://help.gradle.org

BUILD FAILED in 9s

error Failed to install the app. Make sure you have the Android development environment set up: https://reactnative.dev/docs/environment-setup.
Error: Command failed: gradlew.bat app:installDebug -PreactNativeDevServerPort=8081

FAILURE: Build completed with 2 failures.

1: Task failed with an exception.
-----------
* Where:
Build file 'C:\Users\alysonvilela\dev\myapp\android\app\build.gradle' line: 18

* What went wrong:
A problem occurred evaluating project ':app'.
> Could not find method classpath() for arguments [org.jetbrains.kotlin:kotlin-gradle-plugin:1.6.21] on project ':app' of type org.gradle.api.Project.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.
-----------
* What went wrong:
A problem occurred configuring project ':app'.

@alysonvilela
The updated file path is ${your project path}/node_modules/react-native-webview/android/build.gradle
NOT ${your project path}/android/build.gradle

Any update?
What went wrong:
A problem occurred configuring project ':react-native-webview'.

Could not determine the dependencies of null.
Could not resolve all task dependencies for configuration ':react-native-webview:classpath'.
Could not resolve org.jetbrains.kotlin:kotlin-gradle-plugin:1.6.0.
Required by:
project :react-native-webview
Could not resolve org.jetbrains.kotlin:kotlin-gradle-plugin:1.6.0.
Could not get resource 'https://repo.maven.apache.org/maven2/org/jetbrains/kotlin/kotlin-gradle-plugin/1.6.0/kotlin-gradle-plugin-1.6.0.pom'.
Could not GET 'https://repo.maven.apache.org/maven2/org/jetbrains/kotlin/kotlin-gradle-plugin/1.6.0/kotlin-gradle-plugin-1.6.0.pom'.
The server may not support the client's requested TLS protocol versions: (TLSv1.2, TLSv1.3). You may need to configure the client to allow other protocols to be used. See: https://docs.gradle.org/7.3.3/userguide/build_environment.html#gradle_system_properties
PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target

@rajeevverma076  same issue 😭。。Does anyone have a solution？？？


Add missing ellipsoid @params to TileBoundingVolumes functions

In TileBoundingVolumes, the functions that now accept an ellipsoid parameter were missing a doc comment about the new param. This PR adds the missing comments.

Thanks @j9liu!


Timeout at downloading files

Hello, every time I try to download media I receive this error:
Error in function downloadFile (Timed out after 500000 ms). Command: return WAPI.downloadFile('https://mmg.whatsapp.net/d/f/AkcWNQuVF54U77EjCV2UhctLA61kR38Nc44no52EwGRp.enc', arguments[0])

Did you fix the problem?


make package error

Making all for tweak BJUtils…
==> Compiling BJLocation.m (arm64)…
==> Compiling BJSBAlertItem.m (arm64)…
==> Compiling BJServer.m (arm64)…
BJServer.m:99:175: error: property 'name' not found on object of type
'CLPlacemark *'
...target="blank">(%f, %f)", targetPlacemark.name, targetPlacemark.l...
^
1 error generated.
make[3]: *** [/Users/macos/Downloads/BJUtils-master/.theos/obj/debug/arm64/BJServer.m.62349286.o] Error 1
make[2]: *** [/Users/macos/Downloads/BJUtils-master/.theos/obj/debug/arm64/BJUtils.dylib] Error 2
make[1]: *** [internal-library-all] Error 2
make: *** [BJUtils.all.tweak.variables] Error 2

This is due to Theos built-in headers
Thank you for opening this Issue, I overlooked the change I made to my personal copy of Theos.
The Theos header is read before the target SDK. I will be opening an Issue on theos/headers to resolve this conflict in some way, as my project is the not the only one affected by these poorly stubbed out classes.
This issue will not be closed until my Issue is closed on the Theos repo(s)

That header definitely shouldn’t have existed, probably a super ancient relic of mine from when I used the iPhone OS 3.1 SDK (!). Removed in theos/headers@4ba3a95.


fix saving ruleset stagename

for issue https://github.com/joaorb64/TournamentStreamHelper/issues/277

Thanks!


[SPARK-46517][PS][TESTS][FOLLOWUPS] Reorganize IndexingTest: factor out test_iloc*

What changes were proposed in this pull request?
factor out test_iloc* and add the missing parity test
Why are the changes needed?
for test parity and parallelism
Does this PR introduce any user-facing change?
no, test-only
How was this patch tested?
ci
Was this patch authored or co-authored using generative AI tooling?
no

Thanks, merged to master

thanks @yaooqinn and @HyukjinKwon


0.3.0 -> 0.4.0

Codecov Report
Base: 72.60% // Head: 72.60% // No change to project coverage :thumbsup:

Coverage data is based on head (9585a7e) compared to base (b05cf3a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage   72.60%   72.60%
=======================================
  Files          11       11
  Lines         460      460
=======================================
  Hits          334      334
  Misses        126      126

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


DeprecationWarning: with timeout() is deprecated, use async with timeout() instead

Have been looking to get this going for our org for a while now and finally got all the pieces in place only to be stumped by what appears to be a deprecated function.
Wondering if you or anybody else has come across this and has a quick fix?

So everything still works, its just a warning.  Have seen it on Python3.9 and 3.10.

Can you fix the formatting in the output above? Also can you share the xml for the script? I don't think this is an issue with the timeout based on the output posted above

Can you fix the formatting in the output above? Also can you share the xml for the script? I don't think this is an issue with the timeout based on the output posted above

The output is kind of messy and I'm not sure I can fix the formatting more than just adding a few spaces to space it out. I've attached the XML
script.xml.txt

Sorry for the bump but was wondering if you have any ideas as to why this isn't working?

Apologies for the delayed response, I no longer have a jss instance to test against so it might be touch and go for a bit :( Can you maybe try an older version of python and let me know if it works? When I originally developed this it was against 3.6 - just trying to narrow this down to a python version, jamf api issue, or malformed script issue

No worries on the delay, I will specify the original python version and let you know the outcome.

If it detects the script, ensure you have the proper permissions enabled in Jamf for the API user


python DeltaTable.schema().to_pyarrow() does not support data type void

Environment
pip datalake==0.14.0
Environment:

Cloud provider: /
OS: Linux

Bug
What happened:
I wanted to read the schema from a delta table in pyarrow format via
from deltalake import DeltaTable

file_uri = "..."
deltaTable = DeltaTable(file_uri, storage_options=...)
pyarrow_schema = deltaTable.schema().to_pyarrow()

It failed with error:
Traceback (most recent call last):
[...]
  File "my_python_file.py", line 63, in __
    pyarrow_schema = deltaTable.schema().to_pyarrow()
Exception: Schema error: Invalid data type for Arrow: void

What you expected to happen:
pyarrow has a null data type. I would expect that in case of a void type in delta, the pyarrow schema returns the column with null data type instead of throwing an exception.
How to reproduce it:
More details:

p.s i tested this with deltalake 0.11.0 and had there the same issue. So I guess this has never been implemented.

This is some info on void that I can find in the delta protocol definition:

Note: Existing tables may have void data type columns. Behavior is undefined for void data type columns but it is recommended to drop any void data type columns on reads (as is implemented by the Spark connector)

https://github.com/delta-io/delta/blob/master/PROTOCOL.md

The delta protocol actually has something to say about void columns :)

Note: Existing tables may have void data type columns. Behavior is undefined for void data type columns but it is recommended to drop any void data type columns on reads (as is implemented by the Spark connector).

SO i guess to not throw any errors, we could drop all void fields ...


retdec: 3.0 -> 3.2

Motivation for this change
retdec has been outdated for a while. That is probably because it is a pain to upgrade. I've opened https://github.com/avast-tl/retdec/issues/412 for that.
What I've ended up with here won't win any beauty contests, but it is the best I could do. @dtzWill what do you think?
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

Failure on x86_64-darwin (full log)
Attempted: retdec
Partial log (click to expand)
Try 'libtool --help' for more information.
libtool:   error: unrecognised option: '-static'
make[5]: *** [lib/Support/CMakeFiles/LLVMSupport.dir/build.make:1495: lib/libLLVMSupport.a] Error 1
make[4]: *** [CMakeFiles/Makefile2:124: lib/Support/CMakeFiles/LLVMSupport.dir/all] Error 2
make[3]: *** [Makefile:106: all] Error 2
make[2]: *** [deps/llvm/CMakeFiles/llvm-project.dir/build.make:113: external/src/llvm-project-stamp/llvm-project-build] Error 2
make[1]: *** [CMakeFiles/Makefile2:258: deps/llvm/CMakeFiles/llvm-project.dir/all] Error 2
make: *** [Makefile:130: all] Error 2
builder for '/nix/store/0717dnhxhj4mj5007k5i85mzc6id7qi8-retdec-3.2.drv' failed with exit code 2
error: build of '/nix/store/0717dnhxhj4mj5007k5i85mzc6id7qi8-retdec-3.2.drv' failed

Failure on aarch64-linux (full log)
Attempted: retdec
Partial log (click to expand)
[  FAILED  ] ByteValueStorageTests.Get10ByteWorks (0 ms)
[  PASSED  ] 349 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ByteValueStorageTests.Get10ByteWorks
 1 FAILED TEST
FAILED (return code 1)

warning: module 'colorama' (https://pypi.python.org/pypi/colorama) not found, running without color support
builder for '/nix/store/dvpfhgk4mh1sd6jvivc1q0rm9jav7sy3-retdec-3.2.drv' failed with exit code 1
error: build of '/nix/store/dvpfhgk4mh1sd6jvivc1q0rm9jav7sy3-retdec-3.2.drv' failed

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: retdec
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: retdec
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Timed out, unknown build status on x86_64-linux (full log)
Attempted: retdec
Partial log (click to expand)
[ 45%] Built target retdec-dwarfparser
Scanning dependencies of target retdec-loader
[ 45%] Building CXX object src/loader/CMakeFiles/retdec-loader.dir/utils/range.cpp.o
[ 46%] Building CXX object src/loader/CMakeFiles/retdec-loader.dir/utils/overlap_resolver.cpp.o
[ 46%] Building CXX object src/loader/CMakeFiles/retdec-loader.dir/utils/name_generator.cpp.o
[ 46%] Building CXX object src/loader/CMakeFiles/retdec-loader.dir/image_factory.cpp.o
[ 46%] Building CXX object src/llvmir2hll/CMakeFiles/retdec-llvmir2hll.dir/llvm/llvmir2bir_converters/new_llvmir2bir_converter/llvm_type_converter.cpp.o
[ 46%] Building CXX object src/loader/CMakeFiles/retdec-loader.dir/loader/pe/pe_image.cpp.o
building of '/nix/store/vwf4xmsw23rm5s8cqcxcf0r9vz6x9xcy-retdec-3.2.drv' timed out after 1800 seconds
error: build of '/nix/store/vwf4xmsw23rm5s8cqcxcf0r9vz6x9xcy-retdec-3.2.drv' failed

I removed the two debugging comments, so if you have no further suggestions this should be good to go. Thank you for the review :)

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: retdec
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: retdec
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Great, thanks for tackling this!

Success on x86_64-linux (full log)
Attempted: retdec
Partial log (click to expand)
retdec-tests-loader
[  PASSED  ] 65 tests.

retdec-tests-unpacker
[  PASSED  ] 68 tests.

retdec-tests-utils
[  PASSED  ] 350 tests.
warning: module 'colorama' (https://pypi.python.org/pypi/colorama) not found, running without color support
/nix/store/19ac1lp618yw5y9wx5c0yz2yi4p8bqcf-retdec-3.2


Add glm.save and read_glm

Closes #116
Implement a method to write GLM results to disk and read them back.
Handles the wrangling of nested types.

[ ] Confirm the approach taken is reasonable
[ ] Is there a way to test if two objects are the same? Currently I test if the resulting dataframe is identical.
[x] Also implement for ContrastResults
[x] Add more tests specifically checking types:

[x] ContrastResults, OLSModel

[x] Add tutorial

Codecov Report

Merging #337 (e0b0994) into main (439d998) will decrease coverage by 0.21%.
The diff coverage is 84.61%.

@@            Coverage Diff             @@
##             main     #337      +/-   ##
==========================================
- Coverage   96.11%   95.89%   -0.22%
==========================================
  Files          49       49
  Lines        1671     1707      +36
  Branches      212      221       +9
==========================================
+ Hits         1606     1637      +31
- Misses         27       28       +1
- Partials       38       42       +4

Impacted Files
Coverage Δ

mne_nirs/io/snirf/tests/test_snirf.py
96.92% <0.00%> (ø)

mne_nirs/statistics/__init__.py
100.00% <ø> (ø)

mne_nirs/statistics/_glm_level_first.py
86.50% <84.37%> (-0.49%)
:arrow_down:

mne_nirs/statistics/tests/test_glm_type.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

@larsoner could you please give this a review? (The numpydoc failure is existing issue). There is quite some data wrangling, so maybe an easier solution exists.

Here is the tutorial render as i never solved the linking issue https://1580-254026779-gh.circle-artifacts.com/0/dev/pull/337/auto_examples/general/plot_80_save_read_glm.html#sphx-glr-auto-examples-general-plot-80-save-read-glm-py

The approach seems reasonable to me, we do something similar for saving and loading dipy classes in MNE-Python

Appreciate you looking @larsoner. I'll check out the dipy code you mention. And then push on with this.

That code lives in mne/morph.py. we also use a state-based approach in csd.py I think


Support Reboot & Resume

When a chocolatey package requires a reboot have the option to let the reboot happen and then resume the setup process post reboot.

This has been implemented in e9f5543. Cupboard will detect reboots and abort the run unless the resource has IgnoreReboots set to true or --ignore-reboots have been passed.


Fixes #3462 - Change file name casing to fix builds on linux

Fixes

Fixes #3462

Proposed Changes/Todos

[X] Rename files for case-sensitive systems

Pull Request checklist:

[x] I've named my PR in the form of "Fixes #issue. Terse description."
[ ] My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
[ ] My code follows the Terminal.Gui library design guidelines
[x] I ran dotnet test before commit
[ ] I have made corresponding changes to the API documentation (using /// style comments)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any poor grammar or misspellings

Literally all I did.

[ ] I conducted basic QA to assure all features are working

Ready to go whenever. 🤦♂

Critical, for project members to do ASAP
Adding an important note from the issue over here, as well, for everyone to please do ASAP, or at least BEFORE they pull this to their local copies once it's merged (may as well just do it now and be done with it):
git config core.ignorecase false
Without that, you may end up with duplicate files because of casing and the way windows works, and they might even make it back into the repo if certain sequences of not-uncommon actions are performed.
If you do it in your global config, you won't lose the setting if you re-clone on that machine, so it may be a good idea as well:
git config core.ignorecase false --global

Critical, for project members to do ASAP
Adding an important note from the issue over here, as well, for everyone to please do ASAP, or at least BEFORE they pull this to their local copies once it's merged (may as well just do it now and be done with it):
git config core.ignorecase false
Without that, you may end up with duplicate files because of casing and the way windows works, and they might even make it back into the repo if certain sequences of not-uncommon actions are performed.
If you do it in your global config, you won't lose the setting if you re-clone on that machine, so it may be a good idea as well:
git config core.ignorecase false --global

This was all working fine before, for years.
Now, after doing this, if I try to checkout another branch, I get

Please advise.

Because that branch has the wrong file names and different settings.
Case sensitivity mismatches SUCK.
Those are the extra files mentioned in the comment above and quoted in the reply.
Delete them. They are extras, because the case was wrong before.

Basically, any way that you take care of it, it will be taken care of for good unless you've got some weird sort of diamond inheritance or something going on that can't be auto-resolved.
But then you still can just delete the wrong-cased files, which are the ones with lower-case build and lower-case shell in Powershell.
Basically, if it says it's untracked, it doesn't care about that file any more. It's just trying to help you in case you actually meant to keep them around.
As for a root cause analysis?
It's my fault, and I think I can trace it back to almost an exact time when I screwed it up, because it was a last-second rename of stuff for consistency, but I missed especially that s vs S in a string vs CLI file listing. My bad.
 As long as you don't git rm a file that IS in the tree, you're not deleting anything the repo cares about. 👍
And it's honestly pretty annoying that the rename doesn't work like it's supposed to, even though I did explicitly use the git cli to do it in the end, just in case it could make windows behave. But no. Windows gonna Windows.

And if you ever are in a situation with any file anywhere untracked and you're unsure and it's blocking you, just stash them. Then you can carry on and address it later when someone can answer/help/etc.


Add missing grid keys

None of these keys seem to warrant new features. The grid-auto-flow keys have incorrect Safari data, but will likely have the same support as the other added keys once that is resolved.

@mirisuzanne Can you verify that none of the new keys in the dist are features that are separate from what's already in the grid feature?


After running my flutter I get this error message

Launching lib\main.dart on Android SDK built for x86 in debug mode...
Initializing gradle...
Resolving dependencies...

Error running Gradle:
ProcessException: Process "C:\android\flutterprojects\world_holidays\android\gradlew.bat" exited abnormally:
Starting a Gradle Daemon (subsequent builds will be faster)

Configure project :app
Observed package id 'platform-tools' in inconsistent location 'C:\android\Sdk.temp\PackageOperation04' (Expected 'C:\android\Sdk\platform-tools')

FAILURE: Build failed with an exception.

Where:
Build file 'C:\android\flutterprojects\world_holidays\android\app\build.gradle' line: 46

What went wrong:
A problem occurred evaluating project ':app'.

For input string: "3.0"

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 4m 50s
Command: C:\android\flutterprojects\world_holidays\android\gradlew.bat app:properties
Finished with error: Please review your Gradle project setup in the android/ folder.

The Bellow is my Fluttter Doctor
C:\android\flutter\bin\flutter.bat doctor --verbose
[√] Flutter (Channel stable, v1.7.8, on Microsoft Windows [Version 10.0.18362.476], locale en-US)
    • Flutter version 1.7.8 at C:\android\flutter
    • Framework revision d51fd86cdb (5 months ago), 2019-06-21 22:31:55 -0400
    • Engine revision d004bcd4d6
    • Dart version 2.4.0

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\android\Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[√] Android Studio (version 3.5)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 41.1.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[√] VS Code (version 1.40.1)
    • VS Code at C:\Users\Michael Odafe\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.6.0

[!] Connected device
    ! No devices available

! Doctor found issues in 2 categories.
Process finished with exit code 0

Hello @michaelodafe !
Thank you for posting an issue.
Did you review line 46 in your build.gradle file? This looks like a problem with configuration file and not with the Flutter itself.


[WIP] Respect canonical import comments

What does this do / why do we need it?
This adds capability to pkgtree to extract canonical import path comments from the packages. That will later be used as additional constraints when solving the dependencies (will come in a separate PR).
(And a couple unrelated typo fixes and code cleanups along the way)
What should your reviewer look out for in this PR?
Do you need help or clarification on anything?
I was wondering, at the very end of ListPackages, where the package gets added to the Packages map, should I also add a copy with a canonical path key like blow? So that solver could see both versions available?
if pkg.CommentPath != "" {
	ptree.Packages[pkg.CommentPath] = PackageOrErr{
		P: pkg,
	}
}

Also, is there a way to develop from my fork? Can't even get tests running:
$ go test $(go list ./internal/gps/pkgtree/...)
# github.com/rtfb/dep/internal/gps/pkgtree
internal/gps/pkgtree/pkgtree_test.go:20:2: use of internal package not allowed
FAIL	github.com/rtfb/dep/internal/gps/pkgtree [setup failed]

It works fine on the main golang/dep repo, but then transplanting pieces of code to another repo before committing is tedious.
Which issue(s) does this PR fix?
This is phase 1 for issue #902.

I signed it!
On Tue, Aug 15, 2017 at 10:47 PM, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If your company signed a CLA, they designated a Point of Contact who
decides which employees are authorized to participate. You may need to
contact the Point of Contact for your company and ask to be added to the
group of authorized contributors. If you don't know who your Point of
Contact is, direct the project maintainer to go/cla#troubleshoot.
In order to pass this check, please resolve this problem and have
the pull request author add another comment and the bot will run again.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/dep/pull/1017#issuecomment-322569079, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAaBZEBDCv9dJbAtma0GMaHgRCuZJznfks5sYfXSgaJpZM4O4BPx
.

--
Vytautas Šaltenis
On the Web: http://rtfb.lt/

sorry sorry it took me so long!

No worries, I haven't been able to spend significant time on it either :-). Just enough to address your feedback...
AppVeyor tests are failing at something that doesn't seem to be related. Can this be due to my branch falling too far behind?  I was travelling when you last commented, and came around to sit down with it couple days back, only to find out that I was locked out of my GH 2FA during phone transition. You know, little joys of life :-)
I'll try to push an update later today.

The build failed only on 1.9.x, with a lint error, while on others it passed. Weird :-/

no worries - thanks for keeping at this! and yeah, we only run the linters on the 1.9 travis build. it should just be that one issue, then: https://travis-ci.org/golang/dep/jobs/284321611#L547 - the newline byte slice var wasn't used.
i've no idea why the CLA bot is misbehaving. it's been doing this for a week or so now :(

I apologize for the CLA problems - we are working on resolving the issues. I've rescanned this PR and it is now cleared.

@gmlewis oh, i appreciate the update - and i didn't even have to go looking for anybody to ping about it 👍 😄


fix: buffer not available in browser environments, fix #1272

We recently remove all Node polyfills. Unfortunately, we still need a polyfill for Buffer, which is not available in browser environments. :(
See #1272

@geoffgscott did u just copy paste my comment? 😆

OMG, totally forgot about btoa

OMG, totally forgot about btoa


Incompatibility with MRI 2.4

I got following error when upgrading ruby 2.4
/Users/lokenmakwana/.rbenv/versions/2.4.0/lib/ruby/gems/2.4.0/gems/polyglot-0.3.5/lib/polyglot.rb:65:in require': cannot load such file -- xmlrpc/client (LoadError)`
There are some games which are still in development and depends on it, eg. deface

This is not a fault with polyglot. The xmlrpc/client cannot be loaded; that's why it got a LoadError.


Fix handling of PYPRIMEMESH_LAUNCH_CONTAINER environment variable during launch

Fixes #475 .
This change also adds functionality to launch containers in parallel

Codecov Report

Merging #549 (58843b3) into main (1c6f118) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #549      +/-   ##
==========================================
+ Coverage   77.39%   77.43%   +0.04%
==========================================
  Files          31       31
  Lines        2199     2203       +4
==========================================
+ Hits         1702     1706       +4
  Misses        497      497

Impacted Files
Coverage Δ

src/ansys/meshing/prime/internals/launcher.py
28.15% <100.00%> (ø)

src/ansys/meshing/prime/internals/utils.py
65.03% <100.00%> (+0.87%)
:arrow_up:


fix: prioritize runtime deps envars

Fixes https://github.com/cashapp/hermit/issues/368. See individual commit messages for details.

I can't think of a situation where this would be problematic, so this seems reasonable to me. Thanks Kats.

Oh and thanks for the very thorough test in the new test system!

Thanks @alecthomas ! It looks like the change needs to be tagged in order to have a new release published, could you please do that too?

Done!


Add probe hook prototype

We would like to validate the generated data. This probe class could help us to dump data at the end of the march, or more precisely, post-loop of a hook.
Besides, a helper script view-p.py is added as well. This is a script to use paraview library to visualize the data in a easier way. It is not related to the probe feature implementation.
The TubeProbe class will be enhanced soon to meet the requirement of issue #128, including:

dump data of any desired coordination point for each inner and outer loop.
write the dumped data as a vtu so we could visualize it later by paraview.

The is the first step of issue #128.
Test This PR
Issue ./go run in solvcon/sandbox/gas/tube
Expected Result
Specified types of the data will be showed at the end of the whole march, like:
*** *** Probe result at Pt/poi#1164(0,0,0)601:
*** *** - rho = 1.000
*** *** - p = 0.400

Update the PR as the comments of review. This PR is ready to review again.

Merged.  Thanks for the fix.


[Core] useless-supression fix for next-pylint

On next-pylint there is now a new warning that can occur that will indicate that there are useless disables for pylint. This pr resolves those useless-supression warnings by removing disables that aren't needed
A few of the warnings being removed:

broad-except: no longer raised when except Exception: raises the error
protected-access: checks if outside the class or a descendant of the class where it's defined (self._response defined in the class)
raise-missing-from: more granular - doesn't trigger on try: raise

API change check
API changes are not detected in this pull request.

/check-enforcer override


Added integration-test directory with all the Sdk support from the p8e-integration-tests project

Codecov Report

Merging #13 (f10fbc8) into main (403724c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #13   +/-   ##
=========================================
  Coverage     38.80%   38.80%
  Complexity       31       31
=========================================
  Files             9        9
  Lines           688      688
  Branches         70       70
=========================================
  Hits            267      267
  Misses          395      395
  Partials         26       26

Continue to review full report at Codecov.

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