Timeout at file.download()/file.createReadStream()

Environment details

OS: Google Cloud Functions
Node.js version: 8.15.0
npm version: yarn 1.15.2
@google-cloud/storage version: all of them

Details
Hi,
While everything has been working fine for months, we recently (~15 days ago) started to encounter regular timeouts when using the Google Cloud Storage Node client in our application deployed on Google Cloud Functions with Node 8 runtime.
We use the file.download() method with the Promise overload to fetch files content.
We never had issues doing so, until recently. We updated our version of the Node client from 1.7.0 to 2.3.4 3 months ago, so I tried deploying our code with the latest 2.5.0 version and rollbacking to the 1.7.0 version, but the results are the same.
My first thought was that there were connections error and that the file was not downloaded completely, so I converted our use of file.download() to file.createReadStream() to check the different events being emitted, and the process actually stops at the last 'data' event, with the last part of the file being correctly downloaded. However, the 'end' event is never called. This issue seems to happen to certain files only, about ~95% of the time, but the rest of the files are correctly downloaded in functions running in parallel.
Here is a screenshot of what happens last in the Cloud Function, I just hid the function's name and execution ID, and illustrated the end of the file by ...end of text. to clarify that this is indeed the last part of the downloaded file.

You can clearly see that the function times out after 9min, the one we configured for this function.
Attached is another screenshot of a successful execution of the function for another file:

You can see that the 'end' event is correctly called right after the content has been downloaded and the process continuing.
Help would be highly appreciated on this since it has been disrupting our clients work for the last days.

After further investigation, it seems that this error was due to a compilation problem of our app that led to memory issues.
Sorry for the false alarm 🙏


Fix panic with cargo tree --target=all -Zfeatures=all

When cargo tree --target=all was used with the new feature resolver, this would cause a panic because the feature resolver doesn't know about the "all" behavior, and would filter out packages that don't match.
I don't feel like this is a particularly elegant solution, but I'm uncertain of how to make it better.
Closes #8109

r? @Eh2406
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+
Seems reasonable enough for now!

:pushpin: Commit d267fac22755d07bc1cbff34d10ec0d436d628dc has been approved by alexcrichton

:hourglass: Testing commit d267fac22755d07bc1cbff34d10ec0d436d628dc with merge 40d566d0aefd990c0d0cd41617c3c24c91c66514...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing 40d566d0aefd990c0d0cd41617c3c24c91c66514 to master...


Add a new article about sending media message

Write a story about how to use wechaty to send media message

Awesome! Thank you very much. :)


Support resolve override from open flag

Fix #14178

@LuQQiu Would you please take a look at this PR? Thanks.

alluxio-bot, merge this please


restore qos0 subscriptions

@mcollina , would this be a viable fix for Broker should store also subscriptions with QoS 0 ?

Can you send a PR to add a unit test to https://github.com/mcollina/aedes-persistence?


Meta update suggestions

Made a little bit of progress on implementing some of the suggestions. Still more to do, but sharing opening a pr for this first batch of changes.

Updated UI for readonly consensus meta
Refactored polling intervals

Let me know what you think about using the table for rendering consensus meta... definitely needs some refining.

Neat! Applying these updates to the working branch


Refactor module and docs

Create individual modules (camera, object, mesh etc)
Renamed folders from plural form singular
Switch API Documentation to atft library

Fantastic 🎆
PS: I should file an issue sometime -- the raycaster demo (story) doesn't work correctly in Edge. Seems to be off by some amount of 50-100 pixels.

the raycaster demo (story) doesn't work correctly in Edge. Seems to be off by some amount of 50-100 pixels.

Checked on my PC - works well. The only issue is that, if you move over SVG label (icon), then selection disappear and it looks like blinking selection. Will fix raycaster logic -> need to group multiple objects into group and select parent.


Add support for VPN

It would be nice to show the currently active VPN if any. Here is a command that does just that:
$ nmcli --get-values ACTIVE,NAME,TYPE connection show \
  | grep '^yes:.*:vpn$' \
  | sed -e 's/yes://' -e 's/:vpn$//'

see also https://github.com/Alexays/Waybar/issues/3583


Update documentation about --kes-period parameter

The documentation for creating an op-cert is misleading by specifying the --kes-period value as 0.
This should be a calculated value as per the documentation in https://github.com/input-output-hk/cardano-tutorials/blob/master/Shelley-testnet/solutions/Exercise-3-solutions.md#crating-keys-for-our-block-producing-node.

Thanks for noticing !!
I just added how to calculate Kes period in the tutorial itself, rather than directing to the solution. It is better to have it there, I guess.

👍 Thanks


[ARM] GLKitComplet images are missing

On the most recent master build - 20170331.3
Image on the Teapot is missing for these scenes -
Vertex Lit, Specularity, Texture
Solid Color, Textured w/ Decal
And -
Vertex Colors, Textured is black and white and the Text “Tea” is missing on the labels on the tea pot
Solid Color, Textured w/ Decal - is also black and white
This issues looks just like #2150 - however I'm only seeing this now on ARM, previously it was broken on ARM and x86

Do you know when this start to re-appear again?
I was trying to setup my phone and attempt to install the app with visual studio (also using the scripts).
Looks like the build I flashed on isn't being picked up by my computer :(.
I'll try it again in the morning

The previous master build had the same issue on both ARM and x86. This was supposed to be fixed with #2150. I haven't tried this with an ARM develop build that had the fix for 2150

I tried it with the current version of windows on phone, with master build 20170331.3 and was unable to reproduce the bug.
This may be related to D2D changes that occurred with the new version of windows #2284

I do not believe that the minimum precision changes that causes #2284 could be implicated in a complete loss of textures.

@DHowett  it's not entirely that, but it looks like there were changes in the new version of windows that is causing this & the issue in your bug . They are not the same root cause, but it looks like there are changes in the new version of windows we need to look into, so same grouping.
This bug (as the other bug #2284) needs  further investigation.

I cannot reproduce this on ARM.


refactor(mode): rename compare static method

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
445

Relevant Lines:
445

💛  - Coveralls


Expose SLPJS completely

Some applications may need to access additional methods that are not wrapped by SLP-SDK.  For these use cases, it would be useful to expose the whole SLPJS.
I've seen other users have this same request, and I've come across this in consuming SLP-SDK myself.

What is the pro/con of including slpjs with SLP-SDK vs the developer requireing the slpjs separately in their application?

Pros:

Same version of SLPJS can be accessed, don't have to worry about type/feature mismatch with whatever version SLP-SDK is using
SLP-SDK is a Software Developers Kit and therefore should be the one stop shop for gaining access to all SLP related code (and recommended libraries associated with SLP)

Cons:

?

I'm also for exposing SLPJS completely.
A possible con is that WebPack Tree Shaking will have less code to eliminate https://webpack.js.org/guides/tree-shaking/
But seeing that a minified bundle.js of slp-sdk v4.4.0 which just exposed the SDK is already 1.28 MB (see code below), it seems to me there is little code to eliminate either way.
import * as SLP from "slp-sdk";
window.SLP = SLP;


[Twilight 2000 2.2] Changes to accommodate Dark Mode

Changes / Comments
Changes to accommodate Dark Mode
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[x] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Hi @Lockbox313
Just wanted to check, it looks like you've added two files called 'Kult' to this unrelated sheet. Was this intentional?

No it wasn't that willl teach me for trying to do 4 at once. Can the Kult
files be removed.?
On Thu, 3 Mar 2022 at 18:09, Nic Bradley @.***> wrote:

Hi @Lockbox313 https://github.com/Lockbox313
Just wanted to check, it looks like you've added two files called 'Kult'
to this unrelated sheet. Was this intentional?
—
Reply to this email directly, view it on GitHub
https://github.com/Roll20/roll20-character-sheets/pull/10251#issuecomment-1058337143,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACET77LCAOFFG2Q3NCBND2LU6D535ANCNFSM5P2OGL6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @Lockbox313
Just wanted to check, it looks like you've added two files called 'Kult' to this unrelated sheet. Was this intentional?
No it wasn't that will teach me for doing 4 sheets at once. Can the the Kult files be removed ?

Hi @Lockbox313 I've reverted the whole change, you'll need to move your changes to a new branch and create a new PR for this sheet.


Fix handling of None case

If you have a struct like:
my_struct = namedstruct.Message('example', [
  ('data', {
    this.thing: OtherSizedByte,
    this.other_thing: RegularByte,
    this.none_thing: None,
  }, 'data_id'),
])

When you try and use the this.none_thing, you'll get this error.
    def make(self, obj=None, **kwargs):
        """
            A utility function that returns a namedtuple based on the current
            object's format for the supplied object.
            """
        if obj is not None:
            if isinstance(obj, dict):
                kwargs = obj
            elif isinstance(obj, tuple):
                kwargs = obj._asdict()
        msg = self._tuple._make([None] * len(self._tuple._fields))
        for elem in self._elements.values():
            if isinstance(elem, ElementDiscriminated):
>               val = elem.format[kwargs[elem.ref]].make(kwargs[elem.name])
E               AttributeError: 'NoneType' object has no attribute 'make'

I think this fixes it by not calling make on the None object.

I was thinking the other thing you could do, instead of just checking if it was None, you might want to just check if it has make... but I'm not sure if there's any other situations where that would arise.
so the check would be:
if hasattr(elem.format[kwargs[elem.ref]], 'make'):
  val = ...
else:
  val = None

what I should do is just force each element to have a make method, then I can just call make for each element.  That way it can also handle taking elements with multiple valid input types (like the "numerical" elements) and force them into a single common value.  Then if an enum, or an integer, or a bytearray is passed in they would all generate the same "made" value...  I think I might make that change instead


chore: adapt node cmd to app/cosmos

After upgrade of #736, we need to adapt our cmd to be compliant with what app/cosmos wants
This is especially needed for our keys cmds for the current state
A separate issue to refactor out this part is needed
cc: @renaynay

Codecov Report

Merging #755 (c8b8117) into main (2d50db4) will decrease coverage by 0.14%.
The diff coverage is 60.33%.

:exclamation: Current head c8b8117 differs from pull request most recent head aba35e9. Consider uploading reports for the commit aba35e9 to get more accurate results

@@            Coverage Diff             @@
##             main     #755      +/-   ##
==========================================
- Coverage   52.63%   52.49%   -0.15%
==========================================
  Files         116      116
  Lines        6573     6585      +12
==========================================
- Hits         3460     3457       -3
- Misses       2751     2767      +16
+ Partials      362      361       -1

Impacted Files
Coverage Δ

core/client.go
100.00% <ø> (ø)

core/fetcher.go
51.78% <ø> (ø)

header/header.go
46.77% <0.00%> (ø)

service/state/core_access.go
28.33% <0.00%> (ø)

cmd/celestia/main.go
24.24% <5.00%> (-42.43%)
:arrow_down:

node/state/keyring.go
64.70% <70.00%> (ø)

node/testing.go
66.66% <71.42%> (ø)

core/testing.go
83.14% <78.57%> (ø)

header/testing.go
100.00% <100.00%> (ø)

... and 8 more

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 2d50db4...aba35e9. Read the comment docs.

Closing in favor of https://github.com/celestiaorg/celestia-node/pull/757 with agreement of @Bidon15. The https://github.com/celestiaorg/celestia-node/pull/757 has commit from this PR


CI: Create internal workflow

Background
We require a new fork of the current implementation that will allow us to work on upcoming major changes, such as the React 18 update. As there may be breaking changes involved, we do not want to use the existing unstable branch, as this could potentially block further development. Therefore, we need a separate fork that will allow us to experiment with these changes without affecting the existing codebase. This will enable us to test and refine the new features before integrating them into the main development flow, ensuring a smoother transition and minimizing the risk of disruption to ongoing development.
Solution
This PR introduces a new CI workflow that automatically deploys snapshots of the internal branch on demand. Similar to the unstable branch, every merge into the internal branch triggers the creation of a new set of versions that are published to NPM. A GitHub action monitors this branch and, upon detecting any changes, creates a snapshot release that publishes the changed packages as they exist at that point in time. Snapshot versions are identified by the format 0.0.0-internal-{{TIMESTAMP}} and are published to NPM with the internal tag. Packages published to this tag should not be consumed externally.
Checklist

[x] I have :tophat:'d these changes
[x] I have updated relevant documentation

Yup, just merge it


Allow make-style dependencies to be generated without fully compiling

At present, when using the -M flag, rgbasm fully compiles the given .asm file, and outputs a file containing its dependencies (e.g. due to INCLUDEs or INCBINs). This is great for many scenarios involving Makefiles.
However, if a file isn't found, compilation stops. This is good for the proper compilation process, as the files must be openable to correctly generate the object file. However, assume a snippet like so:
%.o: %.asm
%.o: %.asm $(DEPDIR)/%.d
	rgbasm -o '$@' -M $(DEPDIR)/$*.Td '$<'
	@mv $(DEPDIR)/$*.Td $(DEPDIR)/$*.d

%.2bpp : %.png
	rgbgfx -d 2 -o '$@' '$<'

As part of this, I convert PNGs to tile data using rgbgfx, and I'm treating the tile data as an intermediary. However, if I just add INCBIN "Tiles.2bpp" to an .asm file, and try the above dependency generation, it fails, because the .2bpp file doesn't yet exist. Instead, I'd have to manually add a test.o : Tiles.2bpp dependency to my Makefile.
To combat this, would it be possible to add a related (-m?) flag, which also generates dependencies in the same fashion, but doesn't (necessarily) compile the given file, and thus, not require that all files be present?
(I'm aware that GCC has the ability to generate dependencies without a full compile, but I've not tried it myself, so I don't know exactly how it behaves in comparison)

The problem is how to make the assembler to fail without printing a lot of messages related to missing definitions, for example... I don't think this should be the reason for a new flag, the old one should be enough, but the fix isn't trivial.

Fixed by #428 (-MG).
That said, the pret folk have expressed that it'd be a slow process. What could be done to optimize build times with this?

Maybe having two passes of the assembler would help. In the first one only macros are expanded, includes are handled, etc... and that generates the dependencies.

What was the reason for only having a single-pass assembler, again?

In the old two-pass assembler macros were re-expanded again, making assembly rather slow, and there wasn't really any reason to assemble in two passes when rgblink essentially serves as that second pass. Not to mention making the assembler single-pass simplified a lot of the logic.
I wouldn't make the assembler two-pass again, instead, disable whatever logic could be slow and redundant for the generation of dependencies.
That said, in my experience the expansion of the macros itself is rather slow (especially considering we use const a lot to simulate enums), and I'm not sure if anything other than code generation could be disabled/avoided to speed up the process.
The current issue in the pret projects is that all dependencies have to be known before anything can be built, so missing dependencies can be built in the correct order, and some files already take too noticeably long to assemble (order of 2-5 seconds depnding on machine/OS/optimization options), to double the time by using this feature on incremental builds. I don't consider macro-argument-based includes essential enough of a feature to sacrifice build speed, and listing all dependencies for conditional includes isn't particularly wrong, either. I understand why this feature is the "correct" way of doing things, and why people would use this (I would in my personal projects, as well), but it just doesn't scale very well.

That said, in my experience the expansion of the macros itself is rather slow (especially considering we use const a lot to simulate enums), and I'm not sure if anything other than code generation could be disabled/avoided to speed up the process.

If I remember correctly, the problem with macros is that the arguments can be expanded anywhere, and this means tokenization has to be redone on every call.
A possible solution would be introducing a type of macros with token arguments (more C-like, and perhaps ca65-like too), that would be more efficient. Of course, the downside then is having two types of macros.
Also, if the whole file wasn't reparsed up to the point the build failed, this could be a speed boost. But considering the complexity of implementing some kind of caching, I'm doubtful.

Plus included file contents could be modified by the build/update process, so scratch that last idea.

Implemented in 0.4.0 (-MG). While the process may be kind of slow, there's no way to speed it up without hacks of some sort, besides speeding up RGBASM itself.
Thus, closing as resolved; performance improvements are another problem...


adding rag evaluation tutorial

Adding the RAG evaluation tutorial using RAGAS with the Granite models in Python with watsonx.

First easy items before we look at the code will be

Add to tests
Add an entry in the README (and a colab button)

@AnnaGutowska can you tell me what version of Python you tested this with? It fails on Colab (Python 3.10) in the imports cell.

Please "sign" and "sign off" all commits.  This doc discusses both: https://github.com/ibm-granite-community/community/blob/main/CONTRIBUTING.md#signing-commits
This was also useful to me: https://stackoverflow.com/questions/13043357/git-sign-off-previous-commits


Cannot sign-in / publish to Bot Composer

Describe the bug
 I have signed in so many times but ended up having error "Sign in failed: failed to fetch" after updating to 2.1.1
Version
 2.1.1
Browser

[ ] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[X ] Edge

OS

[ ] macOS
[X ] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Sign in to BOT composer
Click on Publish
"Okay" cannot be clicked even if successful login

Expected behavior
 "Published successfully"
Screenshots

Additional context

Hi @maydaystella,
I'd like to get a bit more information from you so that we can try to solve your issue.

The name of your bot implies this is a corporate Azure account. Is this correct?
Are you having any issues with Active Directory, or reaching other services?
Does your Azure account have a subscription?
Are you able to create Azure resources within the Azure portal itself or though the Azure CLI?
Are you only having this issue on Composer 2.1.1? Does downgrading fix the issue?
Have you attempted any standard troubleshooting steps like reinstalling Composer and/or rebooting your machine?

Hi @anishprasad01
The name of your bot implies this is a corporate Azure account. Is this correct? Correct
Are you having any issues with Active Directory, or reaching other services? None
Does your Azure account have a subscription? Yes
Are you able to create Azure resources within the Azure portal itself or though the Azure CLI? Not sure. I am not an IT/SysAdmin of our company
Are you only having this issue on Composer 2.1.1? When I had it yesterday at Ver 2.1, it is already like that that's why I tried to update because it keeps on pushing me to update to ver 2.1.1. Usually the first thing that triggers is "update bot or Import as new"
Does downgrading fix the issue? Im not sure
Have you attempted any standard troubleshooting steps like reinstalling Composer and/or rebooting your machine? I rebooted my laptop but still the same

I assume you've been able to publish bots before? The error message No Azure Directories were found implies an access issue, though we can't rule out that there's a bug somewhere. I am able to create resources and publish bots to Azure, so thus far I haven't been able to reproduce this bug.
Let's try a few things. Start from the top, and keep going if it doesn't work:

Reinstall Composer 2.1.1
Uninstall Composer 2.1.1 and try Composer 2.0.0
Check to see if you are part of an Azure Directory. You can do this by logging in to Azure, clicking on your account name in the top right, and then clicking on Switch Directories. You should see at least one directory in this menu. The current one will be marked as such.

You can find Composer releases and installers here.
Once we know the results of these tests, we can try to hone in on what the problem is.

Hi @anishprasad01
I uninstalled and reinstalled composer 2.1.1 and I was able to successfully publish. However, I have to sign in again before I can publish. That means I have to sign in upon opening the app and sign in again before publishing.
Checked also and I am a member of Azure Directory

I too have to sign-in again every time I publish, so I'll need to double check to see if that's expected behavior and under what conditions, if any.
The more promising thing is that you're able to publish now.

@maydaystella,
I checked with the team, and they confirmed that having to sign-in again to publish is intended behavior due to the need to refresh access tokens. This being the case, there is no solution or workaround available. I am going to go ahead and close this issue, but should you have any new problem, please feel free to file a new issue.


[APIM] az apim api import throws a LinkedInvalidPropertyId error

Describe the bug
I am running a script which initially runs
az apim api versionset create --> This command is successfully creating a version set in the required resource group
Then I am running a loop which iteratively runs
az apim api import --> Here along with many other arguments I also specify --api-version-set-id
The behavior of this is very erratic, upon running it sometimes either successfully runs for all versions or it fails for a few or all versions.
Example: For a first few versions from v1 to v4 the api is successfully imported from a swagger json file but from v5 onwards it throws the following error
ERROR: (LinkedInvalidPropertyId) Property id 'mypropertyid' at path 'properties.apiVersionSetId' is invalid. Expect fully qualified resource Id that start with '/subscriptions/{subscriptionId}' or '/providers/{resourceProviderNamespace}/'.
Code: LinkedInvalidPropertyId
I have verified all the required paths and resources on their availability, and everything is correct.
Kindly assist me in resolving this issue ASAP.
Related command
az apim api import
Errors
ERROR: (LinkedInvalidPropertyId) Property id 'redmaniacVersionset-dev' at path 'properties.apiVersionSetId' is invalid. Expect fully qualified resource Id that start with '/subscriptions/{subscriptionId}' or '/providers/{resourceProviderNamespace}/'.
Code: LinkedInvalidPropertyId
Issue script & Debug output
ERROR: (LinkedInvalidPropertyId) Property id '/apiVersionSets/redmaniacVersionset-dev' at path 'properties.apiVersionSetId' is invalid. Expect fully qualified resource Id that start with '/subscriptions/{subscriptionId}' or '/providers/{resourceProviderNamespace}/'.
2024-02-12T02:03:31.5830869Z Code: LinkedInvalidPropertyId
2024-02-12T02:03:31.5831326Z Message: Property id '/apiVersionSets/redmaniacVersionset-dev' at path 'properties.apiVersionSetId' is invalid. Expect fully qualified resource Id that start with '/subscriptions/{subscriptionId}' or '/providers/{resourceProviderNamespace}/'.
2024-02-12T02:03:31.5831599Z "redmaniac-api-"dev"-v6-id"
2024-02-12T02:03:33.7762796Z ERROR: (LinkedInvalidPropertyId) Property id '/apiVersionSets/redmaniacVersionset-dev' at path 'properties.apiVersionSetId' is invalid. Expect fully qualified resource Id that start with '/subscriptions/{subscriptionId}' or '/providers/{resourceProviderNamespace}/'.
2024-02-12T02:03:33.7764597Z Code: LinkedInvalidPropertyId
2024-02-12T02:03:33.7765368Z Message: Property id '/apiVersionSets/redmaniacVersionset-dev' at path 'properties.apiVersionSetId' is invalid. Expect fully qualified resource Id that start with '/subscriptions/{subscriptionId}' or '/providers/{resourceProviderNamespace}/'.
2024-02-12T02:03:33.9638160Z "redmaniac-api-"dev"-v7-id"
2024-02-12T02:03:35.4886748Z ERROR: (LinkedInvalidPropertyId) Property id '/apiVersionSets/redmaniacVersionset-dev' at path 'properties.apiVersionSetId' is invalid. Expect fully qualified resource Id that start with '/subscriptions/{subscriptionId}' or '/providers/{resourceProviderNamespace}/'.
2024-02-12T02:03:35.4887274Z Code: LinkedInvalidPropertyId
2024-02-12T02:03:35.4888039Z Message: Property id '/apiVersionSets/redmaniacVersionset-dev' at path 'properties.apiVersionSetId' is invalid. Expect fully qualified resource Id that start with '/subscriptions/{subscriptionId}' or '/providers/{resourceProviderNamespace}/'.
Expected behavior
It should succesfully create an api version in the specified apim and must return its details..
Environment Summary
azure-cli                       2.55.0 *
core                              2.55.0 *
telemetry                      1.1.0
Dependencies:
msal                                        1.24.0b2
azure-mgmt-resource             23.1.0b2
Additional context
No response

Thank you for opening this issue, we will look into it.

I am running into the same issue, running az apim api import and specifying a --api-version-set-id and overwriting the specified version.  This is breaking our CI/CD pipelines.


Add ExecutionTotalTime

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

We can take DriverTotalTime of pieline to which RESULT_SINK belongs as the execution total time

[FE PR Coverage check]
:disappointed: fail : 0 / 17 (00.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/qe/Coordinator.java
0
16
00.00%

:large_blue_circle:
com/starrocks/common/util/RuntimeProfile.java
0
1
00.00%


add a CNAME file and set DNS for http://hack.british-hci2015.org/

cname added d36019e08050c2f4fb080bd9aacd05a193718886

url updated by 921c33616409d3e63d2b2ca6893ca20f5e573d40

url in readme updated by 453bdbc38e2e1237c54b774d598240993fa8575a
Closed


Update to latest Idris2 version

There was a breaking syntax change for name quotes ( `{{Name}} ->  `{Name}).
ParserFail now only has one constructor.

Thank you!


Flowcharts do not work in native IE11

When using native IE11, flowcharts do not render correctly (missing text, small diagrams)

Ok, thanks for reporting. Will take a look.

Any updates?

Sorry, for the slow response. I am currently very busy in another project.
It seems IE11 requires some xml info in the elements in the foreign objects tag. One can configure to use non foreign objects though.
xmlns="http://www.w3.org/1999/xhtml"

Not sure if thats all though.

Fixing this by updating the non-html labels. The non-html labels are improved with with background covering the edge for edge labels and possibility to use br to do row breaks.

I tested the latest commit and tested on IE11...
I see text if mermaid is initialized with htmlLabels:false, no text if htmlLabels:true. Is this intended behaviour?

@ivy88, yes it is! I am about to update the doc site to use the htmlLabels:false. With the new changes allowing background color and row breaks etc using non-html labels there is really no reasaon for the html labels anymore. Will keep them for backwards compatability though.

Perfect, thanks!

Is this change released ?

This change is working in the code
https://github.com/knsv/mermaid/tree/d6363e908df4f3f5cfa058c13d3b938347296efa
but not in the master.
Could you please check ?

Hi,
This change to initialize with `htmlLabels:false` does make the labels appear in IE11, which is great.

Is it possible you could add it to the live_editor demo site? At the moment something like this
http://knsv.github.io/mermaid/live_editor/#/edit/JSUgU3ViZ3JhcGggZXhhbXBsZQogZ3JhcGggVEIKICAgICAgICAgc3ViZ3JhcGggb25lCiAgICAgICAgIGExLS0gInRlc3QiIC0tPmEyCiAgICAgICAgIGVuZAogICAgICAgICBzdWJncmFwaCB0d28KICAgICAgICAgYjEtLT5iMgogICAgICAgICBlbmQKICAgICAgICAgc3ViZ3JhcGggdGhyZWUKICAgICAgICAgYzEtLT5jMgogICAgICAgICBlbmQKICAgICAgICAgYzEtLT5hMg
Still renders with no labels in IE11. (And the documentation site is the same, but you've mentioned that already).

Does not seem to work in IE if using the mermaid.init function to render graphs that aren't there when the page is first loaded (is fine in Chrome)
var renderMermaid = function(){
var config = {
startOnLoad:false,
flowchart:{
useMaxWidth:false,
htmlLabels:false
}
};
//mermaid.initialize(config);
mermaid.init(config,".algorithm")
}

I'm able to replicate the issue where HTML labels do not appear in IE. When setting htmlLabels to false, I am able to see the labels in IE, however, I require the use of labels so it's not a suitable fix. Are there any updates on what is causing this? Do we know if it's the way IE is rendering the SVG?

We've decided to give up IE due to lack of contributors. We do want to support MS Edge browser.


CI, release, publish-to-BCR automation

Most of this is copied from rules-template. The only difference is that the e2e/smoke stuff is removed.

@alexeagle is there a way to make the CI setup work with no actual tests? The current "test" I have is basically just some assertions in a BUILD file, so if bazel build //test:tests passes, the tests have passed. That's somewhat eccentric but was simple enough so I didn't spend more time on it. Any suggestions what's best here?


General code quality fix-1

This pull request is focused on resolving occurrences of Sonar rules
squid:S2864 - "entrySet()" should be iterated when both the key and value are needed.
squid:UselessParenthesesCheck - Useless parentheses around expressions should be removed to prevent any misunderstanding.
You can find more information about the issues here:
https://dev.eclipse.org/sonar/rules/show/squid:S2864
https://dev.eclipse.org/sonar/rules/show/squid:UselessParenthesesCheck
Please let me know if you have any questions.
Faisal Hameed

Thanks for the PR.
Just missing change from keySet() to entrySet().


expandAll() and expand() not working

Describe the bug
Mac app v2.8.1 expand() and expandAll() not working.
Versions

spatie/ray 1.41.2
mac app: 2.8.1

To Reproduce
Steps to reproduce the behavior:
$data = ['id' => 1, 'tags' => ['test1', 'test2']];
ray($data)->expandAll();

Expected behavior
When using expandAll() and expand() prior to v2.8.1, the app would automatically expand the tags in the UI.
Desktop (please complete the following information):

OS: macOS
Version 14.5

Additional context
Used to work before 2.8.1

I have the same issue, I'm on Windows and I'm using the Laravel Sail environment.

please check the issue on windows app. thanks

The same thing is happening with Laravel Ray version 1.37.1

Hi! I have the issue as well. MacOS Sonoma 4.6.1  and Ray 2.8.1.

@sebastienhenau any ETA on a fix for this? Would be helpful to have this working again ;)

I'd love to look at ray and just see the expanded arrays rather than moving to the app and manually doing it ever single time. At this point it's just as fast to use @dump()

meanwhile, as confirmed by the tooltip, "ctrl/cmd + click" on the main arrow does the work ... manually

also not working for me, but @camiant thank you so much for this tip! Saves me 50 clicks a day 😉

thank you so much for this tip!

my pleasure, @rderks88 ! 🤝

@raulp Sorry for the late response concerning this issue. At the moment we are working on version 3.0, where we try to solve most of the issues users are currently having with Ray. We'll take a look at this issue as well and we'll provide a fix to be released with version 3.0.


Make shadowOptions optional in workerOptions

What changed?
Make shadowOptions optional in workerOptions

Why?
We can provide a reasonable default value for ShadowOptions if users specifies the worker to run in shadow mode

How did you test it?
Unit test

Potential risks

Pull Request Test Coverage Report for Build 9102d347-5279-4b9b-8447-ba14d5e50d47

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 75.915%

Files with Coverage Reduction
New Missed Lines
%

internal/internal_task_pollers.go
2
75.03%

Totals

Change from base Build 377a3197-b85d-449f-a72c-7ee17a1e7859:
-0.02%

Covered Lines:
10231

Relevant Lines:
13477

💛  - Coveralls


feat(ct-metrics): add initial conntrack metrics for Prometheus

Description
Create initial metrics for conntrack:

packets_count_per_connection [gauge] see example below
bytes_count_per_connection [gauge]
connections_count [count]

Related Issue
#806
Checklist

[x] I have read the contributing documentation.
[x] I signed and signed-off the commits (git commit -S -s ...). See this documentation on signing commits.
[x] I have correctly attributed the author(s) of the code.
[x] I have tested the changes locally.
[x] I have followed the project's style guidelines.
[x] I have updated the documentation, if necessary.
[x] I have added tests, if applicable.

Screenshots (if applicable) or Testing Completed
This example networkobservability_packets_count_per_connection{src_ip="<IP_ADDRESS>"} includes <IP_ADDRESS>=prometheus-kube-state-metrics , <IP_ADDRESS>= prometheus-prometheus-kube-prometheus-prometheus and <IP_ADDRESS>=svc/kubernetes
In this example we see the direction="TRAFFIC_DIRECTION_UNKNOWN" since Prometheus was deployed before retina. Conntrack keeps track of source IP and destination IP, hence each time the connection is close the packets_count_per_connection goes to 0 before increasing again until the next TCP connection is closed.

This example shows a long-lived TCP connection between client src_ip="<IP_ADDRESS>" and server <IP_ADDRESS> via svc <IP_ADDRESS>

This example shows the bytes_count_per_connection where dst_ip=<IP_ADDRESS> which is the pod IP for kube-proxy

Additional Notes
I pushed this PR to create a discussion based on the above examples. Points to discuss:

Using gauge for packets_count_per_connection and bytes_count_per_connection
Reduce cardinality by removing labels?
Increase cardinality by adding labels like TCP port?

Known issues

GC for conntrack_metrics map
conntrack_metrics map update logic to be reviewed
direction="TRAFFIC_DIRECTION_UNKNOWN" for new connections to be reviewed

Please refer to the CONTRIBUTING.md file for more information on how to contribute to this project.

Did we think though around scalability, performance and stability when adding these metrics? Do we have any concerns? Do we have numbers to back this up?

after an offline discussion with @vakalapa and @nddq it was agreed to abandon this PR in favor of multiple PRs to break down the solution into:

data plane implementation
control plane Hubble
control plane legacy
advance metrics

It was also agreed to disable those metrics by default, with the use of a feature flag similar to enableConntrackMetrics
cc @MikeZappa87


Alibi explainer example not working due to old storage uri

/kind bug
What steps did you take and what happened:
[A clear and concise description of what the bug is.]
Alibi image explainer and  income explainer examples throwing module not found error. This is due to the fact that alibi version is upgraded to 0.9.1 by kserve/kserve#2832 while the model is using old version of alibi.
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/local/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/alibiexplainer/alibiexplainer/__main__.py", line 57, in <module>
    main()
  File "/alibiexplainer/alibiexplainer/__main__.py", line 43, in main
    alibi_model = dill.load(f)
  File "/prod_venv/lib/python3.9/site-packages/dill/_dill.py", line 272, in load
    return Unpickler(file, ignore=ignore, **kwds).load()
  File "/prod_venv/lib/python3.9/site-packages/dill/_dill.py", line 419, in load
    obj = StockUnpickler.load(self)
  File "/prod_venv/lib/python3.9/site-packages/dill/_dill.py", line 409, in find_class
    return StockUnpickler.find_class(self, module, name)
ModuleNotFoundError: No module named 'alibi.explainers.anchor_image'

What did you expect to happen:
What's the InferenceService yaml:
[To help us debug please run kubectl get isvc $name -n $namespace -oyaml and paste the output]
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Istio Version:
Knative Version:
KServe Version: 0.11.0-rc0
Kubeflow version:
Cloud Environment:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm]
Minikube/Kind version:
Kubernetes version: (use kubectl version):
OS (e.g. from /etc/os-release):

/assign


Fixes some outstanding Python 2/3 compatability issues.

LGTM.

Maybe you could squash the two commits though :)

Squashed.

@gwax Could you take a look my Google group post here: https://groups.google.com/d/msg/luigi-user/2UyGTJHFx_8/8n4TcGP9860J - the encode('utf-8') in line 315 seems to be causing the tmp_file write to fail for me. Just wondering if you could help. Thanks!

It looks to me like the problem is that, in some cases, we are doing an encode('utf-8') on lines 262 AND 315. I would suggest removing the .encode('utf-8') from line 262 and submitting a pull request.

Thanks! I tried that, but then I still get the error from line 315. Removing the .encode('uff-8') from that line is the only thing that works for me. I imagine that change would break other use cases, however. I'm afraid I don't understand enough about this commit to understand what other sorts of data I could try passing through it to find a solution that works for more than my particular file. :/


Nesterone/gh 82

do we have a task of the utils integration with the client or this is a part of the task?

do we have a task of the utils integration with the client or this is a part of the task?

I suppose that things from 'until-lang' would work as on the server as on the client, we have to make sure that it's defined in utils-lang/README.md
Let's cover it only with unit tests and later one (when needs) integrate with the client


Enable WebRTC

As README sates it's disabled.
Is it possible to enable WebRTC without re-building? If not is there an ETA to add this option or it is hard and I shouldn't hope for it anytime soon?

There's no way of enabling WebRTC right now without rebuilding.
I am planning on adding an option for it and other features. I will probably make them global options until we figure out a good and flexible configuration system to implement.
Related to #38.

I tried looking through the source code, but it isn't clear to me how I should attempt to add an option for this; WebRTC code is all over the place. I believe some Chromium forks have an option like this, but I don't remember their names or if they have source code. Any help will be appreciated.
Also, does anyone know what the WebRTC log uploader does in chrome/browser/media/webrtc/webrtc_log_uploader.cc?

I retrieved this:
https://chromium.googlesource.com/external/webrtc/+/master/webrtc/common_types.cc
by looking at mozilla wiki that linked to a similar common_types.cc

@Atavic I've gathered from your links that there's logging to aid in debugging applications that use WebRTC. But, I'm still not understanding how this relates to the WebRTC log uploader I mentioned earlier. Would you be able to clarify?
BTW, I've determined that it uploads to chrome::kUploadURL (which is https://clients2.google.com/cr/report) in WebRtcLogUploader::UploadCompressedLog

https://clients2.google.com/cr/report is also used for Crash Reports.
Here there are some comments.

@Atavic I can see what data they are capturing now, but I'm still not understanding why they are capturing the data. Perhaps it is for diagnostic purposes just like crash reports?

Paragraph 6.4.2 [Page 41] of http://www.ietf.org/rfc/rfc3550 is listed here.
What I have gathered is that proto is the log stored locally.

@Atavic Sorry, I made a typo in my original message. I mean to ask why they are uploading the logs to Google. I don't think this is something we can gather by just looking through code. I've tried looking online, but I haven't been able to find anything either.
BTW, Protobuf is described here. It's used in all Google projects involving some networking protocol as far as I've seen.

Wireshark could come handy.

@Atavic Thanks for the tip, but packet capturing will not be necessary. I will add a patch to disable log uploading altogether once a switch is implemented to turn on and off WebRTC support. (I should note that log uploading won't work anyway with domain substitution.)

This article has been recently brought to my attention: https://nakedsecurity.sophos.com/2017/05/31/chrome-bug-that-lets-sites-secretly-record-you-not-a-flaw-insists-google/
I haven't investigated the situation yet. If someone knows about the UI code behind this, that would be helpful to know.
I don't know much about WebRTC in Chromium, but perhaps we can add another feature to re-ask for permission to use WebRTC after it had stopped being used?

Bug report.

I have decided to change my stance on this issue. Chromium has a feature to enable WebRTC per-site, so there's really no need to have another switch for it. The log uploader will be patched out in the next stable version.
Also, I don't feel that the indicator light problem is enough to keep WebRTC disabled. If Google isn't doing anything about it, and Inox has WebRTC, then I see no reason why ungoogled-chromium should keep it disabled.

May I ask: Do I understand it correctly that WebRTC will be enabled on default in this browser?

@macandchief Yes, WebRTC will be enabled starting with version 62. All WebRTC runtime options will be the same as normal Chromium for now.

I see, thank you for the quick response. Somehow I had the understanding that being WebRTC removed is an advantage from a security point of view. As a comment under the "Chrome bug that lets sites secretly record you 'not a flaw'" says: "Disable WebRTC which honestly from a security and privacy standpoint you should have disabled anyway as it leaks information such as your real IP address and location even when using a VPN."
I liked it not being part of Eloston/ungoogled-chromium, but I guess it simply means that I will have to care about to disable it in the future.

@macandchief I've seen more people complain about WebRTC being disabled than leaving it enabled. Regarding your concern, uBlock Origin has a feature to mitigate it: https://github.com/gorhill/uBlock/wiki/Prevent-WebRTC-from-leaking-local-IP-address

Yes, I understand. Although those complains seem to be a bit strange to me as basically every other browser from FF over Chromium till Brave have WebRTC, if anyone needs it. While I considered it to be the unique feature of ungoogled chromium having basic functionality without all the junk (which makes it absolutely fantastic - and secure -, especially with the right add-ons being fast and easy on resources). Obviously it's ok too to add it and using something like uBlock Origin in the future. Anyhow thank you for the clarification.

@macandchief While it's true that features are removed or disabled, I don't believe it significantly improves performance (although I don't have much concrete to back this up). ungoogled-chromium pretty much does what its name implies.
And I'd like to emphasize that implication, because I think the major draw to this project is really just that. ungoogled-chromium hasn't really differed much from the status quo that Chromium and Firefox are setting (and that Brave is challenging) for the Internet. Significantly differing from this takes a lot more effort that I and some sporadic contributions can do. For now, I'll just ride along in a mostly similar path.

+1 on the macandchief.. there is ample oppurtunity to have a side browser with full functionalkuty and little privacy, even chromium/chrome normal.. changing your stance due to 'compaints' is the wrong way to go. The entire pointof privacy breaches is the reliance on the lazy masses.. you are now adding to that.
I just found this project as I have along time intended it myself but have so many other patching projects.. and the first thing I will have to do is fork this again to uncommit this? Wow, easier to just drawpatches again then and this becomes pointless.

Eloston.. having now built the (inox) , after patching it to my distro... I see where you are coming from. Although I stand by the unecessary unique machine id hash sharing and what not with this.. ("in the old days, one would manually grant stuff").. there is just so much metadata.. so ye.. the lan ip being blockable is a good thing.. that people who work on patching chromium to be non-google for safety, is great work.. and if it makes sense to keep webrtc enabled, and rather patch in the surrounding framework, then ok... but it is just generally how things are going which is so bad. I think maybe a solution to this, is integrating the chrome extension into the code.. It seemed Serge (from Canonical) linked the source code at some point and it seems to have been removed again.. not sure why, but as I will build (or try to build) this from an old version of yours, I can try to put that in.

Mind you, I have entirely different projects I am working on, so this might be a on and off thing to look into from my side.

WebRTC is another attack surface (security concern)

Firefox and Chrome have implemented WebRTC that allow requests to STUN servers be made that will return the local and public IP addresses for the user. These request results are available to javascript, so you can now obtain a users local and public IP addresses in javascript.
Additionally, these STUN requests are made outside of the normal XMLHttpRequest procedure, so they are not visible in the developer console or able to be blocked by plugins such as AdBlockPlus or Ghostery. This makes these types of requests available for online tracking if an advertiser sets up a STUN server with a wildcard domain. [1]

WebRTC leaks one's local IP address; WebRTC could leak data (privacy concern)

uBo is preventing the not-internet facing machines info to be sent (or leaked), while the machine connecting to the WebRTC service is still announced (that's needed for WebRCT to work).
WebRTC uses the RTCPeerConnection API and is based on a technology called Interactive Connectivity Establishment (ICE).
@Eloston The two above links may help in investigating this issue.
[1] https://github.com/diafygi

WebRTC is another attack surface (security concern);

Firefox and Chrome have implemented WebRTC that allow requests to STUN servers be made that will return the local and public IP addresses for the user. These request results are available to javascript, so you can now obtain a users local and public IP addresses in javascript.
Additionally, these STUN requests are made outside of the normal XMLHttpRequest procedure, so they are not visible in the developer console or able to be blocked by plugins such as AdBlockPlus or Ghostery. This makes these types of requests available for online tracking if an advertiser sets up a STUN server with a wildcard domain.

WebRTC leaks one's local IP address; WebRTC could leak data (privacy concern)

uBo is preventing the not-internet facing machines info to be sent (or leaked), while the machine connecting to the WebRTC service is still announced (that's needed for WebRCT to work).

@Atavic Insightful, thanks. But if a user is not behind a NAT, JavaScript can simply connect to a peer that can discover the IP address, right? Would the safer option be to disable all WebRTC JavaScript APIs?
Normally it would be better to disable WebRTC via GN, but it's not a tested configuration, and it breaks PepperFlash.

The CLI switch
--force-webrtc-ip-handling-policyhas some options that limit the interfaces opened to WebRTC communication.

@Atavic, @Eloston I'm not sure if I see any real issue here. The public IP is needed to initiate connections to other servers and is exposed even without WebRTC, independently of the user's connection method.
@Atavic I believe the information is mostly outdated, see [1][2].
1: https://github.com/diafygi/webrtc-ips
2: https://github.com/diafygi/webrtc-ips/issues/25

Well, exposing your public ip any browser does and has to... this was about internal local ip. This is a huge network discovery exploit.. and despite using stuff like tor, you will have e.g. 10.1.XX ranges identifying networks/users and so on. data collection is a very bad thing in this day and age, and no responsibilities upon 'corps', one needs to do it oneself unfortunately.. anyway. when it comes to the block ting from the internet, the source is by canonical 0.21, and is simply setting the media.peerconnection flag thing to disable. This should just be done in during building, so it is not enabled. with a button/option to enable it if needed. Like the extension just prebuilt imo. Apart form that ye.. as long as one can set it to not be active and not be circumvented, there is no issue with allowing it to be toggled on by a user should he or she wish to use it at some point. but local network info should absolutely be disabled by default. If it's an issue to integrate the extension, then sure, keep it as a manual extra install bit then.

@Atavic @xsmile I know of that flag, but WebRTC traffic can still pass through undetected by extensions, even if all site permissions are disabled (which are only media permissions), right? I don't think that flag is a guarantee no data will go through. If so, it's still a privacy risk.
I'm thinking it might be possible to use the flag's implementation as a reference for a new command-line flag that disables WebRTC.

 I guess it's your call... I haven't tested enough to know how swell only disabling the media.peerconnection is.. but I know when websockets webrtc first came out a long time ago and it oculd read my lan ip I totally lost it ,p As you said yourself, you are thinking of not disabling it entirely.. either it's ripped out.. or I thin for now, perhaps integrating the default connection value ot be false is the way to go? If over time it turns out it there are poc's showing vulernabilities , then patch then?

@Eloston Are you referring to this article [1]? They say that "a user first has to grant a site permission before it can access audio and video".
Chromium extensions have access to WebRTC IP handling settings, see [2]. Google's official WebRTC extension accesses them [3] and so does uBlock Origin [4]. From a privacy point of view the safest option for webRTCIPHandlingPolicy should be disable_non_proxied_udp which exposes the public IP only, like it should be.
Tests did not show my local IP address and could not even show the public one if I activated uBlock's WebRTC feature - with and without being connected to an OpenVPN server.
1: https://nakedsecurity.sophos.com/2017/05/31/chrome-bug-that-lets-sites-secretly-record-you-not-a-flaw-insists-google/
2: https://chromium.googlesource.com/chromium/src/+/master/chrome/common/extensions/api/privacy.json#40
3: https://chrome.google.com/webstore/detail/webrtc-network-limiter/npeicpdbkakmehahjeeohfdhnlpdklia
4: https://github.com/gorhill/uBlock/blob/master/platform/chromium/vapi-background.js#L199

The term interfaces used here refers to network cards.
For clarification, see: https://datatracker.ietf.org/doc/draft-ietf-rtcweb-ip-handling/?include_text=1

@Atavic The webRTCIPHandlingPolicy setting disable_non_proxied_udp corresponds to Mode 4 as defined in draft-ietf-rtcweb-ip-handling-04 and is activated by uBlock Origin.

I did some research, and this is my understanding thus far:

Setting disable_non_proxied_udp will prevent the computer's address from being visible, and also the ISP address when a proxy or VPN is used (except with an extension-based VPN or proxy)
Disabling site permissions for audio and video disables camera and microphone access in WebRTC

This implies the following:

Only data channels can be established, so no more types of data can be sent than what is possible with XmlHttpRequest.
Connections by WebRTC will not contain any more identifying information than other connections by the browser.

That means there is no privacy problem.
But, there is still a minor concern: WebRTC cannot be filtered by an extension like XmlHttpRequest. This is similar to @Atavic's point in https://github.com/Eloston/ungoogled-chromium/issues/179#issuecomment-345825985. Thus, having a flag to disable WebRTC could still be useful to someone that wants to micro-manage their connections (until extensions can intercept WebRTC connections, or an extension wraps the WebRTC interface). However, I'm not sure if there is an actual need for this.
Any thoughts?

6f209526c3a440d933c736d92406843a5119560c disables log uploading. Since there are no objections to enabling WebRTC, I'm closing this.

Cross-posting for those interested in how WebRTC works with custom tabs: https://github.com/bromite/bromite/issues/553


Add a sonos proxy to separate sonos call logic from skill logic

Calls to sonos should be made through a proxy. This helps protect the skill from any design changes to node-sonos-http-api. It also simplifies the logic in the skill.
e.g.
sonosProxy.setVolume('Living Room', 10);
instead of intents all building urls

I am happy to do this work but would like comments from others if they have them.


Problems with sam commands and CloudFormation template

Environment
SAM CLI, version 1.16.0
aws-cli/1.18.69 Python/3.8.5 Linux/5.4.0-1039-azure botocore/1.16.19
Issues

I was not able to deploy the CloudFormation template as-is, resulting in the error:

Error: Failed to create changeset for the stack: oidc-auth, ex: Waiter ChangeSetCreateComplete failed: Waiter encountered a terminal failure state: For expression "Status" we matched expected path: "FAILED" Status: FAILED. Reason: Template error: instance of Fn::GetAtt references undefined resource LogBucketName
Resolution: Removing the two sections in template.yaml referring to LogBucketName: lines 30-32 and lines 57-60.

Execution of sam package and sam deploy as per the shell script at https://aws.amazon.com/blogs/networking-and-content-delivery/securing-cloudfront-distributions-using-openid-connect-and-aws-secrets-manager/ failed with error:

botocore.exceptions.NoRegionError: You must specify a region.
Resolution: add the following regio specification to the sam package and deploy calls:
--region us-east-1

@MXClyde
I believe that the template expects the region to be defined in your ~/.aws/config file.
Now regarding the error about a wrong reference to the logging bucket ("Template error: instance of Fn::GetAtt references undefined resource LogBucketName"), I also ran against this issue. You can work around it without completey removing the logging feature. Indeed, Just correct the reference and make it point to the correct bucket reference (Use LoggingBucket instead of LogBucketName):
Logging:
  Bucket: !GetAtt LoggingBucket.DomainName


Responds to all OPTIONS requests as CORS preflights, even if they are not CORS requests

When enabling the CORS middleware, it inappropriately responds to all HTTP OPTIONS requests as if they were a CORS preflight request -- not just for CORS requests.
Per the CORS spec's "Resource Processing Model" section, requests without the Origin header present are not CORS requests.
This prevents us from responding to normal OPTIONS requests.
NOTE:
This bug could possibly be worked around by registering our OPTIONS route handler middleware ahead of the Swagger middleware but I haven't tested that.

Good point!  Thanks for the link to the W3C spec too, that's very helpful.
This is definitely a bug and will be fixed.

guys, what is the status of that issue?


Upbound CrossPlane addon upgrade

Issue #, if available:
Description of changes:
Upbound CrossPlane addon upgrade
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/do-e2e-tests

@shapirov103 Please review and merge this. Lets get this to release.

Nice work, appreciate the bias for action! please check a few comments/questions i left

@shapirov103 All comments are fixed, please check and run e2e.

@shapirov103 Please check te changes, currently doing local test with CXP. Will share details when im done testing.

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

/od-e2e-tests


I get the error "Reload already in progress, ignoring request" after the update.

After updating Flutter, VSCode constantly gives this error. It is very tiring to stop the project and start it again.
I cleaned the memory (flutter clean). The problem still continues.
I reinstalled VScode again, the problem was not fixed. I reinstalled Flutter (beta for the web). The problem has not improved.

After receiving this error, the screen stops working ("Syncing files to Device Chrome ...")
What is the reason of this problem?
in pubspec.yaml
`environment:
sdk: ">=2.7.0 <3.0.0"
dependencies:
flutter:
sdk: flutter
flutter_localizations:
sdk: flutter
The following adds the Cupertino Icons font to your application.
Use with the CupertinoIcons class for iOS style icons.
cupertino_icons: ^0.1.3
flutter_svg: ^0.17.4
html: 0.14.0+3
responsive_builder: ^0.1.5
settings_ui: ^0.2.0
flutter_map: ^0.9.0
latlong: ^0.6.1
extended_tabs: ^0.2.3
flutter_auth: ^0.0.1
dev_dependencies:
flutter_test:
sdk: flutter`
Flutter (Channel beta, v1.17.0, on Microsoft Windows [Version 10.0.18363.778], locale tr-TR)
VS Code (version 1.45.0)

Im not a flutter dev but there seems to be ALOT of issues with 1.17 personally I rolled back.

Hi @rizacantufan
Does this happen on a new project? Please a try a new project
can you please provide your flutter doctor -v and flutter run --verbose?
Also, to better address the issue, would be helpful if you could post a minimal code sample to reproduce the problem
Thank you

Without additional information we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Tiny cleanup: simplify getBlock0H in faucet

Overview

[x] Don't do unnecessary to/from hex conversions when defining these functions, by flipping the dependency of getBlock0H and getBlock0HText

Comments
#742 — Oops.

bors r+


Option to add component while defining a new variant in <Typography>

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
While adding a new variant in <Typography>, we can't yet specify a component type.
For example, I wanted to add a custom variant called inputLabel to my Typography configuration.

Now, while using this variant, I always want the component to be a label which I don't think I can do right now!?
But this seems like a very obvious case that when we have a custom variant, we should also define the component type.
Examples 🌈
No response
Motivation 🔦
After adding a new variant to Typography, I was repeatedly using component="label" that I could define while adding this new variant itself, which felt wrong.

You will need to cast the result to typeof Typography. See https://mui.com/material-ui/guides/typescript/#complications-with-the-component-prop

You will need to cast the result to typeof Typography. See https://mui.com/material-ui/guides/typescript/#complications-with-the-component-prop

Isn't this related to Typescript?
which I'm not using at all.


Regarding of paired-reads filename

Input S3 bucket (s3://bucket-name/inputs/)

In this case, it looks not work normally.

Yes there is a known limitation where the reads have to be named a certain way. It's in the documentation too:
Note that in this solution, the FASTQ files need to be named in the following manner:
<sample_name>_R1.fastq.gz
<sample_name>_R2.fastq.gz
This can be updated to your needs by updating the Python regex in start_workflow_lambda.py
You can also use example FASTQs provided here to test:
s3://aws-genomics-static-us-east-1/omics-e2e/test_fastqs/NA1287820K_R1.fastq.gz
s3://aws-genomics-static-us-east-1/omics-e2e/test_fastqs/NA1287820K_R2.fastq.gz


Code generation for Phase 1 state machines

Generate the C++ code for connecting externally-defined state machines to F Prime components.

[ ] Update the code generator
[ ] Add tests to fpp-to-cpp

Done in #450.


Make public static access to SPDX Converter

This makes some of the logic accessible to code outside the java library (e.g. the spdx-maven-plugin).

I'm going to go ahead and merge this since there is a project depending on the changes.
If any issues are found with the PR we can fix in a subsequent PR.


[WIP] #350 Bottom Drawer

Description
Add the bottom Drawer UI.

Toggle full height bottom drawer
Tab buttons for Drawer content
Icons + Buttons for the various features

Related Issue

#350

Checklist

[x] I read and understood the contributing guide as well as the code of conduct
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] I documented my code
[x] Review requested

Screenshots

Thanks for the review @lukepistrol I'll fix that asap

The status bar collapses when switching files.

Hi @lukepistrol I fixed this one

When no file is open and you try to drag up the status bar from the collapsed state the app freezes

I spent quite some time debugging this issue. When it's collapsed or almost collapsed it keeps jumping between 0 and a little expanded. Which in turn freezes the app. Any tips on making the drag gesture handle something like that better?
One thing I tried was using a VSplitView that makes the resizing way smoother but I didn't find a way to programmatically set the heights with that view. 🤔

@StefKors :  Could you post a video of what's happening right now?

https://user-images.githubusercontent.com/11800807/163961897-e4fe58f3-3081-46d2-860b-84a04b333d11.mov

@allcontributors add @StefKors for code

@StefKors : Some problems.
https://user-images.githubusercontent.com/20476002/164035159-43290e1e-2bc7-4173-8aca-89f136f89c14.mov
https://user-images.githubusercontent.com/20476002/164035034-c40effba-7212-454c-9a12-b9b945fec490.mov
https://user-images.githubusercontent.com/20476002/164035048-10568f91-7b3f-473f-8aa1-d29d4686abd1.mov

@Angelk90 Can you convert the above comment into an issue so it can be tracked?


Requirement for new codes.

This library is not exhaustive as it still needs several key algorithms and data structures which are widely used in Competitive Programming. The lack is partially due to my lack of knowledge about them or
my laziness to implement  and test them.
I'm listing some of the key algorithms here which this library needs.

A simple concise implementation of Centroid Decomposition, Bridge trees.
Knuth Morris Pratt (KMP), Manacher, Aho corasick : One can find an obselete code of kmp in obselete folder, I have used it several time but code is not concise.
A library for Computational Geometry including some key algorithms like finding convex hull of a set of points.

Any new cool completive programming template is also welcome ! Please report bugs in case u get one during using any of the template.
I would like to invite competitive programming enthusiasts to contribute to this library and let the knowledge grow. :)

i would like to take up this issue


Generate PTX artifacts

Hi.  Is there a way to use rules_cuda to generate PTX?  For example, this is useful for OptiX programming.

Sure.  Currently I'm using Buck, BTW, not Bazel, so concepts are close but not 1:1.  I have an offline process using a simple Makefile to build PTX output. via nvcc  Then I use filegroups to make those available to other targets, and I use that to set an environment variable for where to find the PTX files, which I then pass to OptiX functions.

This is quite reasonable usage, it just skip the stage1 compilation and only use the stage2 compilation. But this does not fit well into the actions.compile design. Becase it support generate multiple virtual archs.

Is that something you'd consider adding, either as a separate rule or as some kind of setting for an existing rule?  Agreed, it would be ideal to be also able to select the generated archs.

It seems that what we need is actions.ptxas, that is, a new rule that calls ptxas directly, instead of trying to mold it into actions.compile. If it is the case, then the problem will be much easier. Then all you need is to feed the generated objects into a cuda_library.
Maybe do it in the future.

Sounds great.  Thanks for considering it.

Oops, I took it as generate from PTX. But OP is requiring generate to PTX. That is, we need to extend actions.compile to support produce ptx. But as well as add actions.ptxas to support consuming it after all.


Add Rx Visualizer

Awesome work with RxVis @moroshko, love it.

@moroshko Checkout RxVis on Microsoft Edge, something is not right.


Hitboxes do not work properly with 'static' layers

I'm trying to create a dynamic HUD, where clicking on a button will expand a menu. I'm unable to because on a static layer i've created the hitbox of items in that layer moves with the viewport, while the position does not. As such, things like Crafty Mouse events are unreliable. There will be some points where the hitbox and position of the item line up, of course, but not always.
Steps to reproduce:
Have a scene with a movable viewport (e.g. follows a character)
Create a layer where the xResponse, yResponse and scaleResponse are set to 0
Create an entity in that layer, at some static x and y co-ordinate
Try to listen to Mouse events on that entity, such as clicking
Add the WiredHitBox component to see what the hitbox looks like
Outcome:
Mouse events are not fired when clicking the entity in the static layer
Hitbox moves with the viewport, becoming out of sync with the entity position
Expected:
Mouse events fire when clicking on an entity in the static layer
Hitbox mirrors the position of the entity

Add the WiredHitBox component to see what the hitbox looks like

Yes, this is broken right now -- the debug layer wasn't updated to handle the layer response parameters.  I think I see a pretty quick fix for that, so I'll try to get that merged in soon.

Mouse events are not fired when clicking the entity in the static layer

Hmm, this should work.  Does this occur only once the viewport has panned, or all the time?

Landed a fix for the wiredhitbox issue; the most recent nightly should now display that properly as the viewport pans.
I tested locally, and didn't have an issue with clicking on a static element, so it might be some more specific interaction.  Hopefully easier to figure out with a working debug layer!

Merry Christmas, and thanks for the quick response!
It happens all the time, but with your fix in, the problem is more apparent.

Seems likely to be a problem on my end, but a bit odd because inspecting the 2D component x and y properties return the expected value. All the static entity hitboxes seem to be offset by the same amount.
Here is the problem reproduced in a jsfiddle

It looks to me like there's some discrepancy between how the different layers are calculating locations -- WebGL and debug work one way, while DOM and Canvas another.  I'll update when I have a fix.
Thanks for the bug report!

This is hopefully fixed now in the develop branch --   WebGL, DOM, Canvas, debug , and click detection now all go through the same path, and so should be consistent.   But feel free to reopen if there are remaining issues.

Have been able to successfully implement everything I was looking to do when I raised the issue.
Will keep an eye out, but things look good. Thanks a lot :)


jacobian stiffening in MoveMesh is not lagged

fixed this when I updated mesh motion in mprans.MoveMesh.py


[AutoPR hanaonazure/resource-manager] Added HANA instance restart API

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3761

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/2605


Can't bind to 'mask' since it isn't a known property of 'ion-input'.

When i try use mask in runtime the error "Can't bind to 'mask' since it isn't a known property of 'ion-input'." is raised.
TS:
public cnpjMascara = '..*-';
public alterarTipo(evento: any) {
if (evento.detail.value == 'F') {
this.inputCNPJ.maxlength = 14;
this.cnpjMascara = '..-';
} else {
this.inputCNPJ.maxlength = 18;
this.cnpjMascara = '../***-**';
}
}
HTML:
<ion-input #CNPJ [mask]="cnpjMascara" [(ngModel)]="pessoa.cnpj">

Hey @danielbredasantos have you imported IonicInputMaskModule inside the component's module that you are working with?

@thiagoprz, yep.
My application is standalone.
Main:

Component:

Code in component:

HTML file:

I have never worked with standalone components, so I'm not sure if the issue may reside on how we import the module into this project but I see a observation about it on the following documentation for standalone Angular components: https://angular.io/guide/standalone-components#configuring-dependency-injection
Could you test using the importProvidersFrom  method?

Even with the suggested changes, the error persists


Update c++ SDK and fix network connections.

Network fixes From Jamie and some SDK updates from Dan.

Tested locally

@cianx I fixed per your feedback in separate commits for easier re-review. I'll re-rebase them in after you re-review.


-Zmiri-strict-provenance reports UB when using the address-exposing Strict Provenance APIs correctly

#![feature(strict_provenance)]
fn main() {
    let x = 0u8;
    let original_ptr = &x as *const u8;
    let addr = original_ptr.expose_addr();
    let new_ptr: *const u8 = core::ptr::from_exposed_addr(addr);
    unsafe {
        dbg!(*new_ptr);
    }
}

error: Undefined Behavior: dereferencing pointer failed: 0x27c02 is not a valid pointer
 --> src/main.rs:8:9
  |
8 |         dbg!(*new_ptr);
  |         ^^^^^^^^^^^^^^ dereferencing pointer failed: 0x27c02 is not a valid pointer
  |
  = help: this indicates a bug in the program: it performed an invalid operation, and caused Undefined Behavior
  = help: see https://doc.rust-lang.org/nightly/reference/behavior-considered-undefined.html for further information

  = note: inside `main` at /home/ben/.rustup/toolchains/miri/lib/rustlib/src/rust/library/std/src/macros.rs:315:13
  = note: this error originates in the macro `dbg` (in Nightly builds, run with -Z macro-backtrace for more info)

note: some details are omitted, run with `MIRIFLAGS=-Zmiri-backtrace=full` for a verbose backtrace

error: aborting due to previous error

I understand why this happens, but this is if nothing else incredibly surprising for users. The intuition is that the Strict Provenance mode in Miri should be compatible with the Strict Provenance APIs in the standard library. Could we make that happen? Or will this just be closed by the permissive provenance work?

Quoting the from_exposed_addr docs:

Using this method means that code is not following strict provenance rules.

So I think this is intended behavior.
However, after https://github.com/rust-lang/miri/pull/2059 landed I want to add a warning (or maybe even an error?) so Miri in strict provenance mode says, the first time from_exposed_addr or an int2ptr cast is used, that this operation is not supported.

Also see https://github.com/rust-lang/miri/issues/2133 which I literally wrote in parallel with you writing this. ;)

Is there a way to invoke miri such that the distinction between these two APIs is actually useful? i.e. such that it attempts to be strict but incorporates exposing as an operation? I forget, is angelic nondet a nightmare to actually support in an interpretter?

Not yet, but see https://github.com/rust-lang/miri/issues/2133 for the plan to make that happen. :D My plan is to make that the default, and make -Zmiri-strict-provenance a flag to basically promise that one doesn't use the exposed stuff.
Yes, angelic is a nightmare, so this mode will miss some bugs where you use a from_exposed_addr-ptr in multiple mutually incompatible ways. That's why we also have -Zmiri-strict-provenance which will not miss bugs but will also not support the angelic stuff.

I will close this in favor of https://github.com/rust-lang/miri/issues/2133. Currently this is intended behavior, and eventually the plan is that Miri would already error in the from_exposed_addr call and say that this is not supported with strict provenance enabled.


Bug Fix for collapsing Menu

Here's a quick patch for the menu collapsing not coming back.

Thanks @professorx00 . Seems like this was simpler than I thought :)

Not a problem.
On Fri, Feb 18, 2022 at 6:36 AM Medik @.***> wrote:

Thanks @professorx00 https://github.com/professorx00 . Seems like this
was simpler than I thought :)
—
Reply to this email directly, view it on GitHub
https://github.com/EddieEldridge/medik-foundry-ui/pull/6#issuecomment-1044379912,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG2FZ3VL3JOHH2UUQR6UIRTU3YVM3ANCNFSM5OWOC6LQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


Undefined method

When ever I use banking deposit,withdraw this error happens that it can't find inventory
org.jruby.exceptions.RaiseException: (NoMethodError) undefined method `inventory' for nil:NilClass
at RUBY.(root)(C:\Users\jesse\Documents\GitHub\apollo.\data\plugins\shops\shops.rb:174)
at org.jruby.RubyProc.call(org/jruby/RubyProc.java:271)
at RUBY.handle(C:\Users\jesse\Documents\GitHub\apollo.\data\plugins\bootstrap.rb:88)

@Major- any idea on maybe why since I can't figure this out

Even if you've fixed it, please don't close the issue until the fix is in master,


Error when trying to use master

Hi, I've been using the released version for quite a while, but now I see that the master version is quite advanced. However, when I try to use it in my project, I get this:
PS C:\ServizioDB> .\CreaRilascioNew.ps1
An error has occured!
Multiple ambiguous overloads found for ".ctor" and the argument count: "1".
At C:\Windows\system32\WindowsPowerShell\v1.0\Modules\Posh-SYSLOG\functions\Send-SyslogMessage.ps1:81 char:5
+     [OutputType($null)]
+     ~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodException
    + FullyQualifiedErrorId : MethodCountCouldNotFindBest

PS C:\ServizioDB> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      4.0
WSManStackVersion              3.0
SerializationVersion           <IP_ADDRESS>
CLRVersion                     4.0.30319.34003
BuildVersion                   6.3.9600.16394
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0}
PSRemotingProtocolVersion      2.2

By looking at line 81 I do not find anything resembling ".ctor".
Where is the problem? Am I missing something?
Thank you in advance.
Cris

I get the same error when using PS 4.0.  When I test on a machine that has PS 5.1, it works fine.  I think it has to do with the change to the enums in commit e6f4ed1ea2cfbeedad7e2ef3f880250aa5db4aba.  If I edit Send-SyslogMessage.ps1 and put the Add-Type -TypeDefinition lines back in, then I'm able to run it from PS 4.0 successfully.

Hi,
Thanks for reporting the issue.
I am guessing when you referred to "released version" you mean the PowerShell Gallery? That should match the GitHub version.
It certainly looks like the changes to the enums have caused an issue.
I have reverted the enum changes, and the Pester tests are now passing.

I have just pushed version 3.1 to this repository. Could you both test and confirm that this is now working as expected?
Once you confirm, I will push the new version to the PowerShell Gallery.

Hi @kjacobsen

I am guessing when you referred to "released version" you mean the PowerShell Gallery? That should match the GitHub version.

No, I was referring to v2.0.1 that you get when you click on the "release" header here on GitHub.
It is from 2015 and it is the last one released here.
I'll download the new master and I'll let you know.
Thank you!

Hi @Cris70,
I haven't been keeping on top of those, I will try to include those into my CI/CD processes from now on. I actually didn't realise anyone was using those!

I have created a release based upon this version to assist in testing.

@kjacobsen unfortunately I still get the same error.
I copied the files from v3.1 over the previous files. Do I need to do something to have powershell load the new files?
To me, it seems powershell is seeing the correct release of the code, because:
PS C:\ServizioDB> get-module Posh-SYSLOG

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     3.1.0      Posh-SYSLOG                         Send-SyslogMessage

Also the error is now on a different line, but the message is the same:
PS C:\ServizioDB> .\CreaRilascioNew.ps1
An error has occured!
Multiple ambiguous overloads found for ".ctor" and the argument count: "1".
At C:\Windows\system32\WindowsPowerShell\v1.0\Modules\Posh-SYSLOG\functions\Send-SyslogMessage.ps1:85 char:5
+     [OutputType($null)]
+     ~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodException
    + FullyQualifiedErrorId : MethodCountCouldNotFindBest

Cris

Why does that seem to make me thing it is an issue with the Output type, can you try removing the line [OutputType($null)] from Send-SyslogMessage.ps1?

Still error, but different now:
PS C:\ServizioDB> .\CreaRilascioNew.ps1
An error has occured!
Send-SyslogMessage : Multiple ambiguous overloads found for ".ctor" and the argument count: "1".
At C:\ServizioDB\CreaRilascioNew.ps1:109 char:3
+         Send-SyslogMessage -Server localhost -Message $msg -Severity $severity -Facili ...
+    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Send-SyslogMessage], MethodException
    + FullyQualifiedErrorId : MethodCountCouldNotFindBest,Send-SyslogMessage

Row 109 in my code is:
Send-SyslogMessage -Server localhost -Message $msg -Severity $severity -Facility user;
Cris

Hi,
How are you loading the module?

Sorry, I forgot to mention in my original message that I did have to comment out the "[OutputType($null)]" lines in most of the ps1 files.  That is the cause of the error that @Cris70 is getting.  I've not used the OutputType before, and I tried playing around with it, but haven't gotten it to work yet.  It does work on PS 5 though.
I also forgot that I was playing around with my username that night and didn't change it back before posting.  So you'll see this from my real username this time.
@kjacobsen Thanks for the great module!

@kjacobsen @bnapolitano after commenting out ALL the instances of "[OutputType($null)]" everything is now working!!
Thank you both for your help and for a great module!!

Thanks to both of you!!
As the output time isn't that important, I will remove it, and push out a new version shortly.

Quick update.
I have pushed version 3.1.1 out to both GitHub (https://github.com/poshsecurity/Posh-SYSLOG/releases/tag/3.1.1) and PowerShell Gallery.
I want to thank you both for your assistance with this one, it is very much appreciated.

Looks like this bug has reappeared from the quick analysis of #15

Hi,
This issue should now be resolved.


Core dumped when trying to render a site homepage

Using phantomjs-1.9.7-linux-x86_64, trying to page.open('http://www.aspectiva.com/'), got the following error:
phantomjs: ../3rdparty/harfbuzz/src/harfbuzz-shaper.cpp:484: void HB_HeuristicSetGlyphAttributes(HB_ShaperItem*): Assertion `glyph_pos == item->num_glyphs' failed.
PhantomJS has crashed. Please read the crash reporting guide at https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug report at https://github.com/ariya/phantomjs/issues/new with the crash dump file attached: /tmp/0f29188c-0469-75ff-7a86c9c3-244de9d1.dmp
Aborted (core dumped)
I was trying to extract the trace from the dump as described in the site, but it contains no addresses. The process is running on CentOS6.5

Please upgrade to PhantomJS 2. 1.9.x is no longer supported, and known to have many crasher bugs that are fixed in 2 version.


v0.1.3

0.1.3 (15.05.2024)

Fix bugs getting children

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

36 of 39 (92.31%)  changed or added relevant lines in 3 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage increased (+5.3%) to 91.775%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tests/test_html2object.py
25
26
96.15%

html2object/html2object.py
4
6
66.67%

Files with Coverage Reduction
New Missed Lines
%

html2object/html2object.py
5
84.62%

Totals

Change from base Build<PHONE_NUMBER>:
5.3%

Covered Lines:
212

Relevant Lines:
231

💛  - Coveralls


Add function type "identity" to color, radius, height stops

Problem
As a data scientist or developer, I want to build an application where a visualization's "height", "radius", or "color" is encoded directly into the source data as a property - for example, `"height": 2001.  This matters because I can create buildings that are the exact correct height in a 3D extrusion visualization.
Solution
Add identity function to https://github.com/mapbox/mapboxgl-jupyter/blob/6e9543462d149118fb81b0c9298bea1dd44a8be2/mapboxgl/viz.py#L589

@ryanbaumann Would this be similar to setting height_function_type='match' and specifying the height_property='height'?  It's a little clumsy, but just wanted to check if that's the same behavior you're looking for.  I like your idea of including height_function_type='identity'.  Then we can have the viz automatically specify the height property without the user having to edit two arguments.

@akacarlyann instead of using a height_function_type that was match, we want to create a Mapbox GL expression under the hood that sets the value from the data column directly equal to the value used in the map.  The identity keyword would indicate to pass ['get', 'myColumnName]` as the expression to the layer created by the Python Viz function.

@ryanbaumann cool cool. That's what I started in #161. Definitely nice to have this option.


Missing Item "Vindicator's Band of Triumph" in P3 Gearset

Hey,
i wanted to sim my PvP gear to determine some stat weights and noticed that the new S3 PvP ring is missing in the P3 Gearset, it only enables with P4.
https://tbc.wowhead.com/item=33919/vindicators-band-of-triumph

I fixed this in a new commit. We pulled item data from Wowhead, which unfortunately had a lot of the Phase information wrong as they were simply guessing which phase items come with. Let us know if you notice any other similar issues


core: Move to C++26 and use native_handle to CLOEXEC the debug fd

Moves hyprland to C++26 to use native_handle to CLOEXEC the debug fd and avoid it being passed to children (every process spawned...)
@fufexan CI
@jbeich will this work on BSD / clang / libc++? I can see it has support for it since clang18

I confirm, builds fine on FreeBSD -CURRENT with Clang/libc++ 18. For older versions (until EOL) downstream can temporarily revert or add workarounds.

alright, thanks!


get bounds for an offline tile source

this might be useful for offline tile users

this has been implemented with geopackage and with maps forge, however adding this feature to zip, sqlite or the gemf specification will be challenging since many people use mobac to create the archives. This would require them to updated their formats. Basically, we need to add a new table to the databases indicated the zoom levels and bounds that are included in the archive


Update connect-a-publishing-site-to-a-catalog.md

Changed the invalid link with https://docs.microsoft.com/en-us/sharepoint/create-and-manage-terms

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@JoanneHendrickson Changes look correct. Could you please review and merge? Thanks!


Version 3.8.1 as framework? (enable bitcode)

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
e.g. ios

Platform Version
e.g. 8.0-9.x

SDWebImage Version
3.8.1

Integration Method
manually

Xcode Version
Xcode 7.3.1

Issue Description and Steps
Please fill in the detailed description of the issue (full output of any stack trace, compiler error, ...) and the steps to reproduce the issue.
Trying to follow the guide of manually importing the version 3.8.1 of the framework:
Add the SDWebImage project to your project
Download and unzip the last version of the framework from the download page
Right-click on the project navigator and select "Add Files to "Your Project":
In the dialog, select SDWebImage.framework:
Check the "Copy items into destination group's folder (if needed)" checkbox
However, on the the downloadsite (https://github.com/rs/SDWebImage/releases) the latest download of a build framework is 3.6 (SDWebImage-3.6.framework.zip).
Is it possible to get a newer version of the framework to download? We are particularly looking at enabling bitcode for our project, so was hoping the newer version supported this.

+1

+1

Guys, I strongly recommend you don't download that version. To get the 3.8.1, you just need to follow the instructions https://github.com/rs/SDWebImage#installation. I recommend you use CocoaPods or Carthage, makes everything easier.
As for the archived frameworks, we stopped doing them from version 3.6 which is more than 1 year old. Forget about that.


Painfully Slow

Now, there's not much to explain, basically when loading minecraft and/or changing resource packs, I notice that MC takes forever to load. While minecraft was churning away, I decided to check logs and noticed that in all cases (I think) CIT Resewn was doing something to make it take forever to load.
However, I did tamper with the caching speed in the settings, so if I broke something or did something wrong please let me know.

Going to need the logs to see if there is anything out of the ordinary.

Right.
My bad.

I grabbed my logs for when minecraft was trying to load a resource pack.
They should be here.
If it may interest you, the CIT resource pack I'm using is called Hypixel+.

Yeah, I turned it up to 32 ticks from 1 tick.


Behavior or docs of immutable files is wrong

Actual behavior (the bug)
Any file with path starting with /immutable will be treated as immutable. For example, http://localhost:8080/immutable_example.png will be treated as an immutable file.
Expected behavior
https://javalin.io/documentation#caching

If you want to skip this check, you can put files in a dir called immutable,

Expectation: Any path starting with /immutable/ will be treated as immutable.
To Reproduce
Put a file in base static dir with name starting with immutable
Additional context
Implementation is here: https://github.com/tipsy/javalin/blob/8a02d5712ed4568582d48ac0932fea0dc8ec1208/src/main/java/io/javalin/staticfiles/JettyResourceHandler.kt#L66

Thanks @akunohomu, could you submit a PR?


Can't able to select last 2 elements.

Restricts me to scroll more towards right or left to select the boundary elements.  So unable to select the boundary numbers.

Hallo @shahwaiz90,
There is no error here, this is the normal behavior of the calendar to render 2 days before and after the range, as the selected date should be always centered on the screen.
Theses extra days are not selectable obviously as they are not in the specified range!
If you want to select these dates you must adjust your end and start range using HorizontalCalednar.Builder().startDate(Date start).endDate(Date end)