Add pprof data to flare and disable zpages data

What does this PR do?
This PR adds the allocs, heap and profile profiles from the pprof extension to the flare. It also removes the zpages from the flare.
Motivation
OTEL-251
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Run otel-agent (inv otel-agent.build && ./bin/otel-agent/otel-agent run --config collector.yaml) + agent (inv agent.build && ./bin/agent/agent run -c bin/agent/dist/datadog.yaml) at the same time.
Generate a flare: sudo datadog-agent flare. If you see error invalid session token, you can return nil in this function for local testing purposes.
Ensure that the flare contains the pprof data:

and that you can open the profiles:

go tool pprof -http=: otel/otel-flare/pprof/dd-autoconfigured_debug_pprof_allocs.dat
go tool pprof -http=: otel/otel-flare/pprof/dd-autoconfigured_debug_pprof_heap.dat
go tool pprof -http=: otel/otel-flare/pprof/dd-autoconfigured_debug_pprof_profile.dat

Also ensure there is no otel/otel-flare/zpages folder.

/merge

=== FAIL: comp/trace/config  (0.00s)
panic: test timed out after 3m0s
running tests:
	TestDisableReceiverConfig (2s)

timed out, will attempt again.


Bump reaction

Bumps reaction to fix FormSwitcher and updates the static forms to designate isStatic and isMobile

Merge on green


bin/build-release.sh dockerImage should be passed environment variables

When I run STORM_RELEASE=0.10.0 bin/build-release.sh dockerImage the docker build command that gets run ignores my STORM_RELEASE setting when it runs.

Hmmph, yup. The bin/build-release.sh dockerImage isn't modifying the Dockerfile, and I'm not familiar enough with Docker to know what the "right" way to do that is.  i.e., is there some way to pass the fields when we invoke docker build such that we would set STORM_RELEASE and MESOS_RELEASE for the bin/build-release.sh invocation inside of the Dockerfile?

The workaround is to modify the Dockerfile by hand to have the STORM_RELEASE & MESOS_RELEASE environment variables set appropriately in the bin/build-release.sh line.  Might have to set them using an ENV Dockerfile command -- again, not something I've ever tried before.

Found this docker thread, seems that passing env variables to the Dockerfile is unsupported and would represent an anti-pattern.  That being said, I'm not sure what the right way to do it is.  Maybe have the bin/build-release.sh do a sed to replace fields in the Dockerfile before building?

https://github.com/docker/docker/issues/6822

I think I might have a solution using docker --build-args feature. I'll maybe have a PR later today...

oh, cool, thanks @drewrobb!


Modified PageRange to Range according to 0.16.0 release

@katmsft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


search: better responsive search form

What I did:
Browse http://invenio-demo-next.cern.ch in Chrome using Samsung Galaxy S4 emulation mode:

What happend:
Search field width is very small, meaning it's very difficult to see what is being typed.
What I expected:
A wider search field.
Possibly by moving Search tips, advanced search and search settings to out of the search field.

Issue moved to inveniosoftware/invenio-theme #59 via ZenHub


add WinitSettings resource in the examples

It fixes the problem when you run server and client on the same machine and only delta from the server is used. Without these settings you may see strange behaviors. For instance, client may have different velocity than server and etc.

Thanks for the PR! I already added the WinitSettings a while back, so I will close this one :)


Add NoResults Component

NoResults
Figma Desktop: https://www.figma.com/file/Lo1ZcBOO9t4QRbX2I0PwBQ/RBB?node-id=1155%3A11025
Figma Mobile: https://www.figma.com/file/Lo1ZcBOO9t4QRbX2I0PwBQ/RBB?node-id=1155%3A11093
When a user filters for Businesses or Allies, and their filter yields no results, they should be presented with a component that contains a Heading, Paragraph, and Airtable form embed via iframe src (use a placeholder for the Airtable form for now).
Styling should be handled through the use of ChakraUI components, and theme variables should be used where possible for things like font styles, colors, and spacing.
If possible, this component should be interchangeable between User Types (Businesses, and Allies).
If possible, through props, a Developer should be able to edit the heading, paragraph, and iframe src.
If reuse of this component isn't viable, two separate NoBusinessResults and NoAllyResults can be created, where the content is handled within the components themselves.

 [Business] - Add no results found component

 [AllyFeed] - Add no results found component


Migrate the source code to Git

For development, it would be helpful to have the history in Git rather than
Subversion.

Git would make it easier to contribute, track patches and maintain branches.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 25 Feb 2015 at 2:46

Done.


Update grads.py

Add the derivative of a determinant.

In the future, new grads should be accompanied by tests.

Thanks for the PR, though! Really glad to have folks contributing.


Engine Crashes when using a Custom Left Sustain

Describe your bug here.
Yes, I have tested this for a while in order to get one of my charts to work, I didnt want to just give up so I tested this extensively for like 40 minutes.
There is no problem with my code or anything. If you use a custom note0 of any length greater than 0 (at the time of testing, only on the opponents side not sure if it happens for the player as qell) the game will close when it hits the tail end. So far this only happened using the games mod packs but I'm not sure. If needed, I can send the code for my note after my classes are over. Thank you!
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Not at all. Only used the mod folder.

Wrong animation name for the left note end on the .XML file of your note asset.

@ShadowMario
<?xml version="1.0" encoding="utf-8"?> <TextureAtlas imagePath="NOTE_rushia.png"> 	<!-- Created with Adobe Animate version <IP_ADDRESS>714 --> 	<!-- http://www.adobe.com/products/animate.html --> 	<SubTexture name="blue hold end0000" x="48" y="454" width="60" height="58"/> 	<SubTexture name="blue hold piece0000" x="54" y="0" width="62" height="82"/> 	<SubTexture name="blue0000" x="0" y="82" width="158" height="180"/> 	<SubTexture name="green hold end0000" x="164" y="454" width="60" height="58"/> 	<SubTexture name="green hold piece0000" x="170" y="0" width="62" height="82"/> 	<SubTexture name="green0000" x="158" y="82" width="158" height="180"/> 	<SubTexture name="purple0000" x="316" y="82" width="180" height="158"/> 	<SubTexture name="purple hold end0000" x="280" y="454" width="60" height="58"/> 	<SubTexture name="purple hold piece0000" x="286" y="0" width="62" height="82"/> 	<SubTexture name="red hold end0000" x="396" y="454" width="60" height="59"/> 	<SubTexture name="red hold piece0000" x="402" y="0" width="62" height="82"/> 	<SubTexture name="red0000" x="0" y="262" width="180" height="158"/> </TextureAtlas>
Does this have any errors?


SelectEvent is unknown type in OL 5.2

How can I call dispatchEvent method and pass an instance of SelectEvent object when SelectEvent is unknown type?

@fredj Thank you for the support. I found a workaround and I do not need this pull-request any more. I am going to close the issue


Missing CIF config dependency in archetype v.25

Expected Behaviour
Project generated with archetype successfully deploys and works in AEM Cloud Service
Actual Behaviour
Project builds and deploys successfully but pages fail to open in AEM.
Reproduce Scenario (including but not limited to)
Generate a project with includeCommerce set to "y", deploy to AEM and open any page
Steps to Reproduce

Generate project from archetype - set includeCommerce to "y"
Run mvn clean install -PautoInstallSinglePackage
Navigate to site in AEM and select a page to edit

Platform and Version
Adobe Experience Manager 2021.1.4830.20210128T075814Z-201217
CIF Add-On 2021.02.01
Sample Code that illustrates the problem
Logs taken while reproducing problem
Caused by: java.lang.NullPointerException: null
at com.adobe.cq.commerce.core.components.internal.models.v1.storeconfigexporter.StoreConfigExporterImpl.initModel(StoreConfigExporterImpl.java:72) [com.adobe.commerce.cif.core-cif-components-core:1.7.0]
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:566)
at org.apache.sling.models.impl.ModelAdapterFactory.invokePostConstruct(ModelAdapterFactory.java:962) [org.apache.sling.models.impl:1.4.16]
at org.apache.sling.models.impl.ModelAdapterFactory.createObject(ModelAdapterFactory.java:761) [org.apache.sling.models.impl:1.4.16]
... 186 common frames omitted
11.02.2021 19:38:06.561 INFO [[0:0:0:0:0:0:0:1] [1613093886529] GET /libs/wcm/core/content/components.1613093603547.json HTTP/1.1] com.day.cq.wcm.core.impl.components.ComponentServlet provided components.
Resolution notes
We traced the issue to the fact that the following dependency was not included:
<dependency>
                <groupId>com.adobe.commerce.cif</groupId>
                <artifactId>core-cif-components-config</artifactId>
                <type>zip</type>
                <version>${core.cif.components.version}</version>
            </dependency>

Once this was added to the main pom.xml and the pom.xml in ui.apps and ui.all and embedded into the final package the error was resolved

@znikolovski this was fixed by https://github.com/adobe/aem-project-archetype/pull/587 which will be part of the next release.


[<IP_ADDRESS>] cannot add single backslash at the end of a tag value

I'm using InfluxDB <IP_ADDRESS> on Ubuntu. Sending line with tag that has backslash at the end of its value is failing with 'unable to parse'.
Line that I write:
t440753,label=http://a.b.c/path/?separator\=\ value=1178<PHONE_NUMBER>

The documentation says that backslashes should not be escaped: https://influxdb.com/docs/v0.9/write_protocols/write_syntax.html#escaping-characters

In this case the backslash is escaping the following space character. That ends up making value=1178 part of the tag section, so the parsing fails.
It parses if there are two spaces there, but there's no way to get a single backslash at the end of the tag value:
> insert trailing_backslash,foo=ends-with-\  value=42
> insert trailing_backslash,foo=ends-with-\\  value=42
> insert trailing_backslash,foo=ends-with-\\\  value=42
> select * from trailing_backslash
name: trailing_backslash
------------------------
time			foo		value
1451331477814160860	ends-with- 	42
1451331480056996716	ends-with-\\	42
1451331482776764944	ends-with-\\ 	42

I understand what happens and why, it's pretty obvious. Either docs should explicitly state this as a limitation, or parser should somehow allow ending tag value with backslash.

@undera I was documenting this for the eventual bug fixer. This is a parser bug that should be repaired.

@beckettsean this seems like a very tricky thing to fix in the parser because it would involve determining if the parser is still looking at the measurement/tags or has moved onto the fields.
insert cpu\ value=1

It's not clear if the above is a measurement called cpu\ with a single field value=1, or if it's the beginning of the following point:
insert cpu\ value=1 value=2

as in, a measurement name with a space in it cpu value=1, and a single field value=2.
Because the parser parses points in one pass I think it might be impossible to determine if one wants to insert an escaped space \ , or a literal backslash \ and then some fields.
I'm assuming to fix this would require a change to the rules around escaping.

@e-dard that makes a lot of sense. I'd be quite happy to simply declare trailing backslashes illegal for identifiers.

@bekettsean I think it would be even better forcing to escape backlashes. I mean, there's a reason why every parser that accept special characters makes mandatory to escape backslashes, and it's avoiding this kind of ambiguity.
If the rules about escaping have to be changed, that's the one to change. Forbiding a character that is widely used in potential data sources for influxdb is not a good idea because using them or not is not always decided by the admin or developer. For example I'm getting disk data usage through SNMP, and the way to differenciate between disks is by using a description tag that is not decided by me or anyone who cares about influxdb and its parsing rules.
By the way, looks like the parsing also fails even if the next character is not a whitespace. I have a telegraf datasource and this in the log:
unable to parse 'Disk\\ usage,Disk=D:\\,agent_host=dservib01,host=dgrafana01.cyii.corp total=0i,used=0i<PHONE_NUMBER>000000000': invalid tag format
That's the telegraf log, that escapes backlsashes, so the actual input to influxdb is:
Disk\ usage,Disk=D:\,agent_host=dservib01,host=dgrafana01.cyii.corp total=0i,used=0i<PHONE_NUMBER>000000000
And the parser is failing with it.


Better recognition of local GitLab installations

GitLens Version: 6.2.0
VSCode Version: 1.18.1
OS Version: Windows 10

We run a GitLab CE server in our LAN and right now to open the remote I need to add this line to my settings (obviously example.com is replaced with the proper internal name ;-)
"gitlens.remotes": [{ "domain": "gitlab.example.com", "type": "GitLab" }]

But as you can see, the URL contains the part gitlab. It would be great if GitLens would automatically recognize the gitlab part in the URL and open the remote accordingly, so without the need to add the line to the settings.
And yes, when somebody chooses to use git.example.com they still would need to configure that manually.

Let me think about this some more. It feels too fragile to me right now. Another option would be if GitLens detects remotes that aren't currently "built-in" that it could prompt you to setup the remote with GitLens. What do you think about something like that?

That would help indeed, but I still think that when somebody put gitlab in the domain name of a remote it can be safely recognized.


pandas pickle

Implement https://pandas.pydata.org/docs/reference/io.html#pickling as a data saver/loader.
It should go into pandas_extensions.py, one class for the writer, on class for the reader, e.g. much like how this polars CSV reader and writer are structured.

Keen to take this on.


Feature: Back to Top

Add a fixed button on blogs to move to the top.

Hello, I would love to work on this

Hey @deeola, please create a fork and submit a PR for review.

https://docs.github.com/en/get-started/quickstart/fork-a-repo
https://docs.github.com/en/github/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork

Let me know if you face any issues in this process.

@deeola have you worked on this problem by any chance?

maybe this will be useful: https://pakjiddat.netlify.app/posts/adding-scroll-to-top-button-to-gatsby-website#client-side-script

https://github.com/pakjiddat/pakjiddat-website/blob/e513dd41d58f2c0e81b0cbbce50870cb9ff406c6/src/pages/articles/2020-04-26---Adding-Table-of-Contents-to-a-Gastby-website/index.md


couldn't find pc offset for node=119

Possible upstream bug, first time occurence. Can not reproduce.

@pedro93 closing, feel free to reopen.


Made it easier for less technical users to switch REDCap versions

@aldefouw, this is a great example of a tiny change that really simplifies things for less technical users.  Ellis is comfortable running commands but not as comfortable editing JSON configuration files.  I think this change would make it easier for users like him to change REDCap versions, and hopefully figure out how to do so on their own.  What do you think?

@mmcev106  - Thanks, Mark!  I agree this needs to be easier and can understand that some people do not want to deal with setting environment variables.
I'm definitely onboard with the idea of updatingthe cypress.env.json through a script - but I have a few thoughts.
If our overall goal is to make it easy for anyone to change REDCap version in Cypress ... would it possibly make more sense to write a "Change REDCap Version" script ... e.g. change_redcap_version.sh ... ?
Such a script would:

Ask you for the version you want to test against ... something like:
"What version of REDCap do you want to test in Cypress?"
Check to see if that specified REDCap version's source code is already present
If source code is NOT present, you could even prompt to download that version -
perhaps script could call download_redcap.sh in that condition?

To come full circle, we could conversely adjust download_redcap.sh to pair with change_redcap_version.sh in the opposite direction.
For example ... in download_redcap.sh ...

After downloading REDCap version, prompt user:
"Do you want to change the version of REDCap you are testing in Cypress?"
If so, run change_redcap_version.sh
If affirmative, we could say
"REDCap version is changed to X.Y.Z in cypress.env.json" or something like that.  Otherwise, we'd leave that version alone.

Here are some reasons why I think this makes sense as separate scripts:

offers flexibility to change a version without downloading REDCap (you might already have the REDCap version - so running download_redcap.sh - at least to me - is counter-intuitive )
offers possibility of downloading a version of REDCap without changing the version you are testing

I think having the verbs that are baked into the name of the script describe the behavior of what you're doing makes a lot of sense to me - but it's possible not to others.
That said, if you like the above proposal, feel free to create / adjust the scripts as described above in your feature branch and push them - I'd be happy to pull that modified request in.
Otherwise, if you think it should be left to just the download_redcap.sh like you are proposing, here's something I'd request: in that case, can we retain the instructions in README.md for changing REDCap version in the cypress.env.json?
I think if we don't have a script to change the version, we still need documentation indicating what drives the actual version of REDCap being tested in Cypress - so I wouldn't feel comfortable removing that without a specific script to change the version.
Let me know if you need any clarifications or have any questions.  Thanks!
Side note: Someone else actually recommended a very similar change in a different PR:
https://github.com/aldefouw/redcap_cypress_docker/pull/2/files
I was very close to pulling it in but I never got a chance to test it.  What I'm proposing is essentially a combination of what you're suggesting and what he's suggesting - with the nuance that I'm also proposing a new script that just changes REDCap version.

@aldefouw, one of my biggest goals here was to reduce the number of steps required.  Are you saying there is a scenario where someone would want to download a REDCap version without setting it as the version used to test?

@mmcev106 - That scenario is rare but it's a scenario that I use to update the redcap_source code in preparation for a new version that will be put under test.
If you want to ignore that scenario, fair enough ... I can work around it because I know the tools.
The scenario I'm more concerned about is the one where someone cannot download REDCap from the community site because they do not have an account.  This is a real scenario that many people encounter who work on this project.
If we're asking them to use download_redcap.sh to change the REDCap version and there is no documented method to do so outside of that file, isn't that problematic?
I'm fine with us using download_redcap.sh to adjust the REDCap version ... what I'm less certain about is the removal of documentation about how to change redcap_version outside of download_redcap.sh.

@mmcev106  - Thought about this last night after hours.
Here's how we'll proceed:

I will approve this request.  I understand your approach, I think it does reduce the steps, and I want you to be able to complete your tasks and help other people get onboarded with minimal effort.  I will prioritize your efforts and ideas ahead of others right now.

We do need to address how users who do not have REDCap community accounts download REDCap; however, I'll create a separate issue so we can circle back to it later.  Right now, we'll prioritize the efforts you are directing.

Thanks @aldefouw.  I really do believe prioritizing the happy path is important if we want to encourage less technical users to participate as much as possible.  We can of course have many side roads, but it will pay dividends to widen the main road, keep it clear, well marked, and generally prioritized.


go build github.com/vulkan-go/vulkan: invalid flag in #cgo LDFLAGS: -Wl,--allow-multiple-definition

if you are having this issue, its actually a problem with this repo
https://github.com/vulkan-go/vulkan
here is a solution
manually delete -Wl,--allow-multiple-definition from vulkan_windows.go line 8
reference from issue on other repo
https://github.com/vulkan-go/vulkan/issues/13#issuecomment-370299820

Please leave this issue open until this case is resolved/properly handled on the main repo so that newcomers can understand and resolve the issue they are having. Usability and support is the best way to bring in new users and contributers!


puppeteer_tests: Fix test_invalid_edit_bot_form, maybe

Commit 35960be510237513c9302011f1206b2260eca646 (#20815) disabled this test for being flaky.  Perhaps that was because the waitForFunction call was missing arguments:
     await page.waitForFunction(
         (cancel_button_selector: string) =>
             !document.querySelector(cancel_button_selector)?.hasAttribute("disabled"),
+        {},
+        cancel_button_selector,
     );

Replace it with a less janky waitForSelector call.
Cc @chdinesh1089

Hmm, still flaking locally.

I think that test hasn't flaked in some time and we can close this?

It hasn’t flaked in some time because we disabled it. The goal here was to re-enable it.


Correct duplicate named routes

Fixes #465.

FYI - the CI run is no longer AFAICT showing Deprecation warnings for these.


add white list

is't my pool ip. thank you!

https://github.com/XDagger/xdag/wiki/White-List
Please follow the step to add your IP to white-list.
Thanks for your cooperation!


more detail in user agent

@patflynn

And now I've just noticed that we're using the client's 'applicationName' model to set the user-agent.  Do we know this is safe?  How does One Platform treat application name?

https://developers.google.com/api-client-library/java/google-api-java-client/reference/1.19.1/com/google/api/client/googleapis/services/json/AbstractGoogleJsonClient.Builder#setApplicationName(java.lang.String)
plus we have a unti test

LGTM


Is this available in GCC?

[I have the license assigned to me, and enabled but I don't see the link or option to create a page.  We are not GCC High or DoD.]

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

ID: 7bad8864-5f08-90ae-a6d1-a23e58fc1843
Version Independent ID: 7bad8864-5f08-90ae-a6d1-a23e58fc1843
Content: Bookings with me
Content Source: microsoft-365/bookings/bookings-in-outlook.md
Service: bookings
GitHub Login: @kwekuako
Microsoft Alias: kwekua

@StrongDoBongSoon Thank you for your feedback. Unfortunately, Bookings with me is currently not available for GCC tenants.
Here is a URL link to share this situation with our relevant product developer team via Bookings · Community (Microsoft.com)
Hope this helps!
Thanks
Sri

@yogkumgit - Thanks for the response. I appreciate your time clarifying it.


Make optional file opener callback function

I'm thinking that if the user can supply a callback that's triggered on selection, what you do with it could be much more versatile, eg. Handing out certain filetypes to external programs.

Thanks for the suggestion.
Sorry. Due to my lack of understanding, could you please tell me what specific program you are going to pass it to?
I could not visualize how to use it.

xdg-open, or something similar that would do something based on the file/mimetype.

I think it is possible to use a function to get the information of the target file.
Please let me know if I'm wrong, as I may have missed something.
require 'lir'.setup {
    -- ['l']     = actions.edit,
    ['l'] = function()
      local ctx = lir.get_context()
      local current = ctx:current()
      if string.match(current.value, '[^.]+$') == 'mp4' then
        vim.fn.system('xdg-open ' .. current.fullpath)
        return
      end
      actions.edit()
    end,

    --...
}

yep, that's great.
Thanks!


Update grpc-netty to 1.63.0

About this PR
📦 Updates io.grpc:grpc-netty from 1.62.2 to 1.63.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.grpc", artifactId = "grpc-netty" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.grpc", artifactId = "grpc-netty" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #2902.


Emit files relative to entries

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
Consider dynamically created Webpack config entries:
...
compiler({
    target: 'node',
    entry: createEntries(...),
    output: {
        filename: '[name]/index.js',
        path: 'dist/'
    },
    module: {
        rules: [
            {
                test: /assets\/js\/.*\.js$/,
                loader: 'file-loader',
                options: {
                    ???
                    name: '[name].[ext]'
                }
            },
        }
    },
    ...
});

where the source folder structure:
src/components/comp-a/
    assets/js/index.js
    index.js
    package.json
src/components/comp-b/
    assets/js/index.js
    index.js
    package.json

and the resulting entry (the one calculated using createEntries function) is:
entry: {
    'comp-a-0.0.1': 'full/path/to/src/components/comp-a/',
    'path-b-0.0.2': 'full/path/to/src/components/comp-b/'
}

In all cases, where applicable, consider the paths in the above examples to be resolved properly, using path and such. They were omitted for brevity. Also 0.0.1 and 0.0.2 are versions extracted from corresponding package.json file that can be seen in source folder structure above.

As a result of the build configuration above, the following files are created:
dist/comp-a-0.0.1/
    index.js
dist/comp-b-0.0.2/
    index.js

What is the expected behavior?
I would have expected an ability to receive the following:
dist/comp-a-0.0.1/
    assets/js/index.js
    index.js
dist/comp-b-0.0.2/
    assets/js/index.js
    index.js

If this is a feature request, what is motivation or use case for changing the behavior?
Reasoning: in my case, I build node-targeted artifacts (the dynamic entries) that may carry static assets (those assets/js/index.js files). I want to be able to emit these files relative to the entry.
They are not going to be consumed using that path (so things like publicPath runtime adjustments are irrelevant) and are for organizational purposes (these are then published to NPM and such).
Currently, these are always relative to some single entity -- dist/, publicPath and such. outputPath doesn't help either, due to dynamically created entries and difference between source folder name and the corresponding entry name.
Please mention other relevant information such as your webpack version, Node.js version and Operating System.
webpack 2.4.1
file-loader 0.11.1
Node 7.9.0

@dnutels Seems it is out of scope of file-loader. Why don't use just CopyWebpackPlugin? If you want to handle js files using babel and other stuff, you can use multi-compiler. First compile dist/comp-a-0.0.1/index.js and copy from src/components/comp-a/assets/js/index.js to dist/components/comp-a/assets/js/index.js, second run babel-loader and other stuff. Thanks!

Thanks everyone! Duplicate https://github.com/webpack-contrib/file-loader/issues/114. Feel free to PR


Bar chart

Remove bottom line as in design

Add dark line as in design

Sorting labels seem off

For some reason the sorting - although all the settings are exactly the same - are different for the horizontal and the vertical bar chart

The grouped bar chart seems to have a fixed height, which makes the bars appear very small. Can we adapt the same behaviour here, as when we have a lot of labels and a scroll?

 They should be truncated

@squiles The error bar does not work on the bar chart. Do we need a design here first? @AnninaWalker
Dataset NFI: Topics by tree state (living/dead)

@squiles The error bar does not work on the bar chart. Do we need a design here first? @AnninaWalker Dataset NFI: Topics by tree state (living/dead)
How it looks for the column chart:

@florelina No I don't think a design is needed. It should be the same as for the vertical, but flipped

@squiles I've just reviewed again from my side and all the issues I've reported are fixed. I have just one question about the labels. Do we truncate them at a certain amount of characters?


 We should have a script in the repo that can be run to collect all that info and output it in an easy to copy and paste format, and then update the Bug Report template to have a section for this info.
First suggestion for what is should include:

gcc version (for Unit Tests or Middleware)
gcc-arm-none-eabi version and/or TI CCS version (for Firmware)
TI RTOS release number
Operating System
Github commit hash

Is your feature request related to a problem? Please describe.

Closed with merge of #191


Support Rechecking Suburbs

From some very rough estimations it seems that all the announced suburbs will be processed within 3 weeks.
Once that is done the oldest suburbs should get updated if they are X weeks old or more.
There isn't currently any such functionality for this. We could also scan suburbs which haven't been announced if that is of interest to people.

After all the missing suburbs are processed, maybe a second pass can look at the date of the last generation (now it's embedded), and if it's "too old" refresh it.

Hi - just had my address move from FTTP_NA to FTTP_SA over the weekend (along with all of my neighbours) but the map hasn't updated. https://lukeprior.github.io/nbn-upgrade-map/?suburb=kellyville&state=NSW
Is there any way to refresh it?

Idea: load the old output file, to create gnaf_pid -> loc_id cache.  This will halve the number of NBN requests for something that should be pretty-much fixed.

@shanelord01 suburbs should be reprocessed eventually, but there's a lot of addresses in Australia, it takes a long time (we haven't finished the first cycle yet - currently 68%)

I'll run it manually on my PC now

Hi @lyricnz @LukePrior if a "node" (ie an agent running separately from the main server that just ran distributed queries against the NBN API then reported back) was available that I could spin up in docker easily, I'd be happy to spin some servers up in Linode to help out.

We could do that, but (a) it's still pretty slow, (b) we don't want to DDoS the NBN API (or get their attention) - the process already does 10 threads in parallel.
My laptop+internet shut down yesterday while this was running (power was off for renovations). Will restart it now.

python ./code/main.py --suburb kellyville --state nsw --progress
2023-07-15 08:30:25,369 INFO MainThread Creating DB index...
2023-07-15 08:30:25,376 INFO MainThread Skipping index creation as already exists
2023-07-15 08:30:25,441 INFO MainThread Fetching all addresses for Kellyville, NSW
2023-07-15 08:30:25,661 INFO MainThread Fetched 10300 addresses from database
2023-07-15 08:30:25,699 INFO MainThread Loaded 9867 addresses from output file
2023-07-15 08:30:25,701 INFO MainThread Submitting 10300 requests to add NBNco data...
2023-07-15 08:31:59,074 WARNING nbn_9 Error fetching NBN data for 3 WAKEFIELD RISE KELLYVILLE 2155: Expecting value: line 1 column 1 (char 0)
Progress: |███████████████████████████-----------------------| 55.1% Complete

I think this can be closed


GetUserTopArtists returning forbidden

I used the same code on the Get User Top Artists example made here, and yeah. No custom code.

Did your access token expire?

Did your access token expire?

No

Did you use the user-top-read scope?


Contact us implementation

The default is to use php. First try Google sheets - a form to send the mail to sheets.

Done with Google sheets


Sassy CSS & Improved Gulp Tasks

Now using Sassy CSS!

Nevermind! Resolving conflicts before I reopen a PR.


Parsing root element as key-value

Hey! Thanks for this great library.
I have an xml file like that:
<soap-env:Envelope xmlns:soap-env="http://schemas.xmlsoap.org/soap/envelope/">
    <soap-env:Header/>
    <soap-env:Body>
        <n0:Z_CRM_KDK_OPT_GET_SOZ_DETAILSResponse xmlns:n0="urn:sap-com:document:sap:rfc:functions">
            <ET_ITEMS>
                <item>
                    <ADRESE_TESLIM_KM>0000000000</ADRESE_TESLIM_KM>
                    <ADRESTEN_IADE_KM>0000000000</ADRESTEN_IADE_KM>
                    <STATUS>E0001</STATUS>
                    <USER_STAT_PROC>ZKDK_KLM</USER_STAT_PROC>
                </item>
            </ET_ITEMS>
        </n0:Z_CRM_KDK_OPT_GET_SOZ_DETAILSResponse>
    </soap-env:Body>
</soap-env:Envelope>

And i'm parsing it like that:
function testXmlParse()
{
    $reader = new Sabre\Xml\Reader();
    $xmlResult = file_get_contents(APP_ROOT . '/sample.xml');

    $reader->elementMap = [
        '{http://schemas.xmlsoap.org/soap/envelope/}Envelope' => function(Sabre\Xml\Reader $reader) {
            return Sabre\Xml\Deserializer\keyValue($reader, 'http://schemas.xmlsoap.org/soap/envelope/');
        },
        '{http://schemas.xmlsoap.org/soap/envelope/}Body' => function(Sabre\Xml\Reader $reader) {
            return Sabre\Xml\Deserializer\keyValue($reader, 'urn:sap-com:document:sap:rfc:functions');
        },
        '{}item' => function(Sabre\Xml\Reader $reader) {
            return Sabre\Xml\Deserializer\keyValue($reader, '');
        },
        '{}ET_ITEMS' => function(Sabre\Xml\Reader $reader) {
            return Sabre\Xml\Deserializer\repeatingElements($reader, '{}item');
        },
        '{urn:sap-com:document:sap:rfc:functions}Z_CRM_KDK_OPT_GET_SOZ_DETAILSResponse' => function(Sabre\Xml\Reader $reader) {
            return Sabre\Xml\Deserializer\keyValue($reader, '');
        }
    ];
    $reader->XML($xmlResult);

    $result = $reader->parse();

    dump($result);
}

And the result is like that:
array:3 [
  "name" => "{http://schemas.xmlsoap.org/soap/envelope/}Envelope"
  "value" => array:2 [
    "Header" => null
    "Body" => array:1 [
      "Z_CRM_KDK_OPT_GET_SOZ_DETAILSResponse" => array:1 [
        "ET_ITEMS" => array:1 [
          0 => array:4 [
            "ADRESE_TESLIM_KM" => "0000000000"
            "ADRESTEN_IADE_KM" => "0000000000"
            "STATUS" => "E0001"
            "USER_STAT_PROC" => "ZKDK_KLM"
          ]
        ]
      ]
    ]
  ]
  "attributes" => []
]

Envelope element itself is the root and i want it to be a key-value pair too. like that:
array:1 [
  "Envelope" => array:2 [
    "Header" => null
    "Body" => array:1 [
      "Z_CRM_KDK_OPT_GET_SOZ_DETAILSResponse" => array:1 [
        "ET_ITEMS" => array:1 [
          0 => array:4 [
            "ADRESE_TESLIM_KM" => "0000000000"
            "ADRESTEN_IADE_KM" => "0000000000"
            "STATUS" => "E0001"
            "USER_STAT_PROC" => "ZKDK_KLM"
          ]
        ]
      ]
    ]
  ]
]

Sorry for the big delay. I haven't really been monitoring this as much.
Deserializers can only work on the 'content' of an element, not the element itself that's why it doesn't work on that level.
However, you are completely free in using all the apis as you see fit. For example, Sabre\Xml\Service::expect also reads an xml document and reformats the result:
https://github.com/sabre-io/xml/blob/master/lib/Service.php#L138
So I'll just ask the obvious question... why don't you build a function that does exactly what you want?
function foo($input) {
   $server = new Sabre\Xml\Service();
   $result = $server->parse($input);
   return [$result['name'] => $result['value'];
}

However, the expect function might be even a bit better for your use-case.


Problems with JS

TL Classic without problems, changing to Turbolinks 5 with the following applications. js leads to JS only executed on first page loaded:
//= require jquery
//= require jquery_ujs
//= require plugins
//= require jquery.turbolinks
//= require functions
//= require turbolinks

https://github.com/kossnocorp/jquery.turbolinks/issues/56


fix(search): removes input clear after search

Closes: FX-2591
I noticed this bug this morning when trying to use the mobile site. The go button (which in mobile Safari simulates <enter>) clears the input without searching.

This was introduced here as a feature not a bug: https://github.com/artsy/reaction/pull/3217
I think it makes sense not to clear the input because frequently one wants to edit the query to refine it — this is a pretty common pattern in most searches/search engines. I couldn't find any specific accessibility guidelines on this particular point, but it makes an intuitive kind of sense. So this PR just disables the clearing of the input, rather than fixing the submission with the addition of the input clear.
We also already populate the query when linking to search results directly, anyway.

I think what maybe the original behavior was getting at is that you want to clear the query once you leave the search results page?

I think what maybe the original behavior was getting at is that you want to clear the query once you leave the search results page?

Going to merge this later unless y'all have any comments @mzikherman / @damassi

Going to merge this later unless y'all have any comments @mzikherman / @damassi


aerial on Sierra

The screensaver no longer works after upgrading.  Is there a new release on the way that is Sierra compatible?

The screensaver works in Sierra. There is a longstanding bug in OSX/macOS. And only the preview is affected.
Select Aerial, close System Preferences and reopen System Preferences. The preview will now work.


Wouldn't it be useful to have an iterable which is low cost for zero usages and for many usages?

Consider the following iterable:
var b = false;

Iterable<int> foo() sync* {
  for (int i = 0; i < 2; ++i) {
    for (int j = 0; j <<PHONE_NUMBER>; ++j) { // Pretend that we have to work hard ..
      if (b) print('Hello!');
    }
    yield i; // .. before we can yield the next element.
  }
}

This iterable is characterized by being costly to iterate, and this means that it will be a significant performance benefit to avoid running the body of foo at all if the resulting elements are never needed.
This means that we cannot use the advice "never use foo(), use foo().toList() instead", unless we know statically that the elements will be used at least once.
However, it is even worse if we execute foo() at some point and obtain an object v, and v.toList() is executed many times. This could occur because the resulting elements are needed many times, in many different locations in a large software system, and v is passed there in various ways (as a parameter, stored in a data structure, etc.), but there is no single location which is reached early after the creation of v where it is known that this iterable will definitely be used at least once.
This issue is a proposal that we might benefit from having a class similar to the following class widely available:
import 'dart:collection';

class LazyList<E> extends IterableBase<E> implements Iterable<E> {
  Iterable<E> _iterable;
  List<E>? __list;

  LazyList(this._iterable);

  List<E> get _list {
    var list = __list;
    if (list == null) {
      __list = list = _iterable.toList();
    }
    return list;
  }

  Iterator<E> get iterator => _list.iterator;
}

The concrete implementation is completely minimal, but it allows for the desired effect: We can replace an expression like foo() that yields an expensive iterable by the expression LazyList(foo()), and the resulting object will perform toList() on demand, when the first iterator is obtained from it.
This means that we will avoid toList() entirely if the iterable is never iterated (we can pass it around just fine, we just never actually iterate it), and we can pass it around to multiple clients, and we will still only perform toList() once if multiple clients make the choice to iterate it.
Obviously, there may be a need to do many other things in order to obtain a good LazyList implementation: Presumably, we'd want an unmodifiable list as the value of _list, and we'd want to return that list from LazyList.toList(), and there could be many other important details. But the core idea is already present in the very simple LazyList above.
One assumption is crucial to know for users of LazyList, so that must be part of the documentation:
It must be appropriate to invoke toList() just before an iteration starts; in particular, a LazyList cannot be used on an iterable that delivers an infinite sequence of values, and it should not be used on an iterable which is intended to iterate over just a few elements and then stop the iteration.
Here is an example run:
class Timer {
  final start = DateTime.now().millisecondsSinceEpoch;
  int get time => DateTime.now().millisecondsSinceEpoch - start;
}

void run(Iterable<int> iter) {
  var timer = Timer();

  void speak(String message) {
    print('  ${timer.time}: $message');
  }

  // Use `iter` to obtain the elements, twice.
  for (int k in iter) speak('Reached $k');
  speak('Done');
  for (int k in iter) speak('Reached $k');
  speak('Done');
}

void main() {
  print('Using the object from `foo()` directly:');
  run(foo());
  print('Using a LazyList:');
  var ll = LazyList(foo());
  run(ll);
}

which prints:
Using the object from `foo()` directly:
  1869: Reached 0
  3802: Reached 1
  3802: Done
  5448: Reached 0
  7076: Reached 1
  7076: Done
Using a LazyList:
  3274: Reached 0
  3274: Reached 1
  3274: Done
  3274: Reached 0
  3274: Reached 1
  3274: Done

I see two or more approaches here.

A lazily initialized list: Lazy init, then immediately doing full toList. Good if you have an expensive initial computation, then cheap iteration.
A lazily populated/cached iterable and/or list: Lazily build up a cache of elements, with each element only computed once, up to the watermark that we have so far seen. (Doing length would immediately fill it up). Good if each element computation is expensive.

Both are potentially costly in memory when the cached values aren't needed any more. If it's a list, that's less surprising than if it's an iterable.

Almost exactly this sort of LazyList pattern became dominant in dartdoc, though unfortunately a refactor to condense it into a class has not happened.  Wish I had thought of that!
I can confirm @lrhn 's statement that it is definitely costly in memory when the cached values aren't needed anymore, if you use this idea extensively in your program.  The challenging bit is determining when they won't be needed.  I could in theory measure and predict for my use case which declarations generate instances iterated over 1, N, or sometimes even specific values of N times. This tends to be consistent between runs of my application.   Measuring this is challenging though unless I build a tool to do so specifically, so my in-practice default for a batch application has tended to be "cache them all and let the profiler sort it out if people complain about memory usage".  Not exactly a nice outcome.
A really smart LazyList could have a debugging version that logs how many times each instance derived from a declaration is iterated over and help the programmer not waste memory by suggesting a parameter to an optimized version that knows to either never build a list, or discard it after a certain number of reads.  Or it could check in with a manager singleton that responds to memory pressure, or other possibly over-complicated ideas to help users make better tradeoffs.
Worth thinking about though if the cache / no-cache decision on Iterables is an important cause of waste -- CPU or memory -- in typical user code.

For comparison, an incrementally caching list could be:
class LazyList<T> extends ListBase<T> with UnmodifiableListMixin<T> {
  List<T> _elements = [];
  int? _length;
  Iterator<T>? _sourceIterator;
  Iterable<T> _source;
  LazyList(Iterable<T> elements) : _source = elements;

  int get length => _length ?? _computeAll();

  bool _upTo(int index) {
    var length = _length;
    if (length != null) return length >= index;
    var iterator = _sourceIIterator ??= _source.iterator;
    while (_elements.length <= index) {
      if (iterator.moveNext()) {
        _elements.add(_source.current);
      } else {
        _length = _elements.length;
        return false;
      }
    }
    return true;
  }

  bool get isEmpty => _upTo(1);
  bool get isNotEmpty => !_upTo(1);
  T get first => _upTo(1) ? _elements.first : (throw StateError("No element"));
  T get last {
     _computeAll();
     return _elements.last;
  }

  T operator[](int index) {
    _upTo(index);
    return _elements[index]; // Throws the right error if not long enough.
  }

  int _computeAll() {
    var length = _length;
    if (_length != null) return length;
    while (_source.moveNext()) {
      _elements.add(_source.value);
    }
    return _length = _elements.length;
  }
}

Cool stuff!
Note that it's trivially possible to reset __list to null in the LazyList that I showed, and it would then just be recomputed when needed. It could be interesting to have a global priority queue of weak references to the LazyList objects whose __list is non-null, and then allow us to get some references from that priority queue and go out and reset the __list. Ideally, it would be the GC that did this, based on a heuristic about the frequency of GCs or somesuch.
This would introduce another precondition for usage:
A LazyList(iterable) should never be created if it is inappropriate to use iterable to iterate over all its values repeatedly, potentially each time iterator is called.

See also here.

Oh!  I'm just seeing CachingIterable for the first time.  Thanks for this link @leafpetersen!

I think that could be taken as a sign that we should have CachingIterable (or whatever variant turns out to work best) in a core library. It is in fact not specific to Flutter.

By the way, I think the incremental approach (CachingIterable and this one) may be slower than the variant that relies on invoking toList() when the first iteration occurs: With an incremental approach we never avoid the indirection (for instance, we're probably relying on an iterator that uses operator[] during the iteration, and operator[] would be implemented by additional code in the caching iterable-or-list.
In contrast, if we use toList() of the underlying iterable and iterator of the resulting list then it should be possible for an implementation to detect that the given iterator is an iterator of a system provided list type, and then it could presumably run something like a traditional C-style for loop to do the iteration. (Yes, today we probably just check the iterable, not its iterator, but that shouldn't be too hard.)
So, in the case where it isn't crucial to avoid running toList() just before the first iteration, this could yield a better overall performance.

Interesting.
One advantage of the late final with in-line closure call approach that I've been moving toward for dartdoc is that it sidesteps the need for indirection after the first call.  (I haven't measured how much a difference this makes... maybe not enough for the complication?).  This method is also generically applicable to any data structure that you want cached.  And it is applicable for any sort of deferred, cached data structure scenario without any underlying implementation support required.  A dartpad-able example:
Iterable<String> stringGenerator() {
  print('building');
  return ['hello', 'there'];
}

class A {
  late final Iterable<String> cachedStrings = (() => stringGenerator().toList()) ();
}

void main() {
  var x = A();
  print('A initialized');
  print(x.cachedStrings);
  print(x.cachedStrings);
  print(x.cachedStrings);
}

The downside is some boilerplate of parens/braces and a bit of headscratching when you first see it.

Yes, a late final with an initializing expression is actually quite nice (and safe), and if you don't need to write any statements then you could of course just omit the function literal:
class A {
  late final Iterable<String> cachedStrings = stringGenerator().toList();
}

The original argument seems a bit of a straw argument. You can't do anything with the Iterable without incurring the expense, not even ask .isEmpty. It is better not to call foo() at all if is not needed, and if it is needed, then the expense is unavoidable.
I have come to the conclusion that our lazy Iterables are error-prone and should only be used in a linear way: An Iterable may be iterated once, or passed to one place that also has the responsibility to iterate once. If you need to iterate twice, you need to copy to something that supports efficient re-iteration. This property could be checked by static analysis.
There was a bug in the protocol buffer libraries that was responsible for several seconds of jank in an important application. The code to clone a message would create a new message object and add elements of a repeated field using PbList.addAll. addAll has this structure:
void addAll(Iterable<E> collection) {
  collection.forEach(validate);
  _internalList.addAll(collection);
}

This was called, in a different file, using:
  newMessageRepeatedField.addAll(oldMessageRepeatedField.map((item) => item.clone()));

The two iterations of the lazy mapped Iterable caused clone calls exponential in the depth of repeated structures.
One way† to fix this would be:
void addAll(Iterable<E> collection) {
   final copy = List.of(collection);
   copy.forEach(validate);
   _internalList.addAll(copy);
}

CachingIterable could have been used too, but I think the performance would not be adequate. Once we have a system list in our hands, which can be determined trivially at compile time for List.of, the compilers can generate code where a loop is a handful of instructions, rather than a handful of calls. Forcing the representation to a system list makes subsequent operations and order of magnitude faster.
† What is unfortunate about allAll's list copy is that it is unnecessary when collection is already a system list, which is most of the time. The copying cost was measurable and unwelcome on these paths, so the final fix was to add .toList() after .map() and leave the performance hazard in place.

I would find it useful to have a family of idempotent factory constructors that copy an Iterable if it is not a system List, but return the argument if it already is suitable system List.
List.ensureUnmodifiable(Iterable<E> input)
Returns input if it us already a List that might have been returned by List<E>.unmodifiable(input), otherwise returns a copy that is guaranteed to be unmodifiable. I would use this to store a local copy of an input to a multi-step algorithm to ensure it cannot be changed between calls into the steps, but avoid a copy if the input is already unmodifiable (e.g. const []). A simple example is storing the children in an unmodifiable tree structure, so client code can walk the tree but not change it.
List.ensure(Iterable<E> input)
Returns input if it is a system list with interface List<E>, otherwise copies to some kind of system List. The intended use is like the call to List.of in the addAll example - the result is not modified by the method and the extend of the potential copy is just the call to the method.

I would totally support a lint detecting when you iterate an argument iterable more than once. It's a great way to have unexpected side-effects.
For the addAll, I'd probably prefer doing:
void addAll(Iterable<E> collection) {
  var preLength = length;
  for (var e in collection) {
    if (validate(e)) {
      _internalList.add(e);
    } else {
      _internalList.length = preLength;
      return;
    }
  }

(assuming the internal list is just a plain SDK list).
I'm not too psyched about making "system lists" special. You should be able to treat every List as if it was a system list. If the object doesn't have efficient lenght and operator[], it should't be a List.
(On the other hand, you should treat every Iterable as horribly inefficient and not iterate it more than necessary. For experts only: Don't even expect it to be finite :grin:).

What I'm hearing is that the desired behavior of the lazy/caching iterable depends on the use-case, and either can be dangerous if misused because they cache values after they're actually needed (and both cost extra allocation when not actually needed).
So, the solution would be to provide both, a CachingIterable and a LazyList, and recommend only using them only locally in a single function which knows whether and how it uses it.
(And LazyList is probably unnecessary because you can just do lazy var lazyList = iterable.toList(); or List<T>? lazyList; ... lazyList ??= List.of(iterable).)
If you know you're going to do multiple partial iterations, and want to cache the values, then CachingIterable does make sense, you just need to know when you're done using it so it can be GC'ed. I'm not sure how often that use-case occurs - if you need to iterate more than once, you usually iterate the same number of elements each time (assuming the iterable hasn't changed between iterations, in which case you definitely don't want to cache). How that number is found is likely different between use-cases, so I'm not sure it's something we can easily support generally.

@lrhn The suggestion for addAll makes my point that we need something better. The suggestion is horribly coupled to the use of the data from the Iterable, and incorrect since validate throws, and sometimes less efficient due to inserting one element at a time rather than as a batch. You would need to do something completely different if the underlying was a Set. A version of addAll that uses 'copy-if-needed' is more clearly validating-then-adding, and the code is uncoupled from the inner addAll, so could be reused (either the actual code, or the simple pattern) in a Set implementation.
Unmodifiable/const Lists are important tool because you can reason about the elements being invariant. This should be better supported by the language and libraries.
You might not like to "make" system lists special, but they already are special if your application has constraints on performance, either runtime or memory.
It is difficult to get into the efficient zone, and some mechanisms for getting there, like using an unmodifiable List, have other uses, like protecting against modification.
I can reasonably fake
var items = List.ensure(collection);

with
List<E> items = collection is List<E> ? collection : List.of(collection);

There is still a performance hazard because the compilers often can't tell that items is monomorphic (try passing in "".codeUnits and UnmodifiableListView([]), and time your program again).
I don't know how to fake List.ensureUnmodifiable.
On 'my' app, I could reduce memory below a hard limit by scattering List.unmodifiable(...) all over the place, but there was copying of lists that were already unmodifiable, so further reductions could potentially be available if the corelib was more helpful.


Build Failure: https://travis-ci.org/ManageIQ/manageiq/jobs/59257084

Commit: https://github.com/ManageIQ/manageiq/commit/d5c66377b68f1720182a9b16027c11f910c3b07f
Build: https://travis-ci.org/ManageIQ/manageiq/jobs/59257084
Error:

$ bundle install --without development:metric_fu:qpid --path=${BUNDLE_PATH:-vendor/bundle}
Updating git://github.com/ManageIQ/rails.git
error: packfile ./objects/pack/pack-74f9e38eab9043b49d3740f62cd6ae9b51c72832.pack does not match index
warning: packfile ./objects/pack/pack-74f9e38eab9043b49d3740f62cd6ae9b51c72832.pack cannot be accessed
error: packfile ./objects/pack/pack-74f9e38eab9043b49d3740f62cd6ae9b51c72832.pack does not match index
warning: packfile ./objects/pack/pack-74f9e38eab9043b49d3740f62cd6ae9b51c72832.pack cannot be accessed
error: packfile ./objects/pack/pack-74f9e38eab9043b49d3740f62cd6ae9b51c72832.pack does not match index
warning: packfile ./objects/pack/pack-74f9e38eab9043b49d3740f62cd6ae9b51c72832.pack cannot be accessed
error: packfile ./objects/pack/pack-74f9e38eab9043b49d3740f62cd6ae9b51c72832.pack does not match index
warning: packfile ./objects/pack/pack-74f9e38eab9043b49d3740f62cd6ae9b51c72832.pack cannot be accessed
fatal: bad object 5b3f7563ae1b4a7160fda7fe34240d40c5777dcd
error: git://github.com/ManageIQ/rails.git did not send all necessary objects
Retrying git fetch --force --quiet --tags 'git://github.com/ManageIQ/rails.git' "refs/heads/:refs/heads/" due to error (2/3): Bundler::Source::Git::GitCommandError Git error: command git fetch --force --quiet --tags 'git://github.com/ManageIQ/rails.git' "refs/heads/*:refs/heads/*" in directory /home/travis/build/ManageIQ/manageiq/vmdb/vendor/bundle/ruby/2.0.0/cache/bundler/git/rails-9af6475ceda5ce0ac29345bc84230d32c46f7b56 has failed.
If this error persists you could try removing the cache directory '/home/travis/build/ManageIQ/manageiq/vmdb/vendor/bundle/ruby/2.0.0/cache/bundler/git/rails-9af6475ceda5ce0ac29345bc84230d32c46f7b56'
error: packfile ./objects/pack/pack-74f9e38eab9043b49d3740f62cd6ae9b51c72832.pack does not match index
warning: packfile ./objects/pack/pack-74f9e38eab9043b49d3740f62cd6ae9b51c72832.pack cannot be accessed

(same error/warning lines repeated hundreds of times before dying)

Found this https://bitbucket.org/site/master/issue/7295/error-on-cloning-repository-pack-does-not.
Seems to be either a permission issue with ssh based git cloning or corrupted indexes.  Either way, it's not clear why it's happening sporadically.

Perhaps it's a git version thing?

Closing as it hasn't reappeared and seems to be a git specific thing.


[Git Sync]: Check if there's a better way to push changes to avoid rate limits

Some users report that they're hitting rate limits due to their big number of token sets. When using multi-file sync and you have a lot of token sets, we trigger multiple API calls to GitHub on a push or a pull, even if only 1 file was changed.
Ideally we'd either

trigger less api calls so that only the changes are pushed that are required instead of every set (which I think is what we do, but still we first check if there's a change)
or introduce some kind of limiting / slowdown so that we do not run into the problem of rate limits (try creating 300 sets, you'll likely hit a rate limit)

Marking this as a bug as it's causing some users to not be able to push changes if their token sets exceed a certain number.

@six7 Just trying to get some updates on when this bug is going to be worked? This is currently blocking us from using the plugin to update, create, and remove tokens. Thanks!

Migrated to featurebase

@akshay-gupta7 can you check if this is still needed or is this already covered in another issue/PR?


Broken timestamps

Required Info

Camera Model
T265

Operating System & Version
Win 10 1809

Platform
PC

SDK Version
2.19.2 / 2.20.0

Language
C++

Hello!
After patch 2.19.2 method frame::get_timestamp() returns broken timestamps.
There is code of obtaining frames:
void RealSenseDevice::RSCallback(const rs2::frame& frame)
{
	if (rs2::frameset frameset = frame.as<rs2::frameset>())
	{
		std::lock_guard<std::mutex> lock(_visualMutex);
		rs2::video_frame left = frameset.get_fisheye_frame(LEFT);
		rs2::video_frame right = frameset.get_fisheye_frame(RIGHT);
		_stroPairWasUpdated = left && right;
		if (_stroPairWasUpdated)
		{
			_curStereoPair.leftFrame = RealSenseUtils::frame_to_mat(left);
			_curStereoPair.leftFrame.timestamp = left.get_timestamp();
			_curStereoPair.timestamp = left.get_timestamp();
			_curStereoPair.rightFrame = RealSenseUtils::frame_to_mat(right);
			_curStereoPair.rightFrame.timestamp = right.get_timestamp();
		}
	}
	else
	{
		rs2::motion_frame motion = frame.as<rs2::motion_frame>();
		auto motionProfile = motion.get_profile();

		std::lock_guard<std::mutex> lock(_inertialMutex);
		if (motionProfile.stream_type() == RS2_STREAM_ACCEL && motionProfile.format() == RS2_FORMAT_MOTION_XYZ32F)
		{
			auto accelerationVec = motion.get_motion_data();
			std::memcpy(&_curPose.vecAcc.a[0], &accelerationVec, sizeof(rs2_vector));
			_curPose.timestamp = motion.get_timestamp();
			_poseWasUpdated = true;
		}
		if (motionProfile.stream_type() == RS2_STREAM_GYRO && motionProfile.format() == RS2_FORMAT_MOTION_XYZ32F)
		{
			auto gyroVec = motion.get_motion_data();
			std::memcpy(&_curPose.vecVelocity.a[0], &gyroVec, sizeof(rs2_vector));
			_curPose.timestamp = motion.get_timestamp();
			_poseWasUpdated = true;
		}
	}
}

The timestamp for both visual and inertial data has value -2147483648 always. Note that, the data is correct, but timestamps are broken.
With patch 2.19.1 all works fine.
I suppose that this problem appeared after PR #3453. Do I need to install any T265 drivers for enabling system timestamps or is it bug?
Thanks!

Hey. Have the same issue. WIn 10 x64

@elevir hi,
The code fragment seem to be legit, and yet we couldn't find a way to reproduce it so far.
Do you see it with the standard realsense-viewer tool?
If so - in which versions 2.19/2.20/2.21 ?

I'd suggest to start by putting cout s next to query to see that the data is not altered elsewhere in the code:
	rs2::video_frame left = frameset.get_fisheye_frame(LEFT);
	rs2::video_frame right = frameset.get_fisheye_frame(RIGHT);
	if (left) std::cout << left.get_timestamp();
	if (right) std::cout << right.get_timestamp();

@ev-mp, hello!
I took screenshots of viewers since version 2.19.1. For versions above 2.19.1 timestamps have warning that the hardware timestamps are unavailable.

I'd suggest to start by putting cout s next to query to see that the data is not altered elsewhere in the code:

I've already checked, they were broken.
2.19.1 -

2.19.2 -

2.20.0 -
2.21.0 -

@elevir  hi,
At least int the provide 2.19.2 screenshot it is clear that the IMU and Fisheye timestamps are not identical:

In other screenshot this in either trimmed or obscured so I couldn't confirm it, but it seem that the system timestamp are within the expected range.
Can you provide some 5-10 sec record file of the streaming?

Note also that the value that you've mentioned in the beginning  -

-2147483648

which is obviously 0x80000000, is definitely different from the timestamps captured in the screenshots

@ev-mp

Can you provide some 5-10 sec record file of the streaming?

Unfortunately, I will able to provide recorded file just tomorrow.

Note also that the value that you've mentioned in the beginning -

-2147483648

which is obviously 0x80000000, is definitely different from the timestamps captured in the screenshots

That's true, but I don't know how is realized timestamp capturer in RealSense viewer. Can you confirm that there is no any substitution of hardware timestamps by software timestamps?

@ev-mp hello!
The recorded file is here

@elevir, I checked the record and the timestamps for Fisheye and IMU streams are fully functional:

Both timestamps and frame numbers update as expected.
There is a problem I encounter with the playback functionality that does not show pose frame metadata (frame number, timestamp, fps) - but that's a separate issue.
Please double check the code you use - e.g. things like frame-frameset casting.
The Viewer's code in this regards is written in a straight-forward manner though as overall not that easy to follow:

frame_number = f.get_frame_number();
timestamp_domain = f.get_frame_timestamp_domain();
timestamp = f.get_timestamp();

Therefore I'd recommend to check rs-data-collect tool with T265 , inspect the timestamps and review the tool's code. That code base is much smaller and provides similar functionality.

@ev-mp ok, thanks!
I'll try to implement in the same way as in this tool, and get back to you as soon as possible.
But anyway in framework with version 2.19.1 the code I showed was working.

@elevir
Have you been able to verify this?  Can the issue be closed?

@RealSenseCustomerSupport I'm sorry, I didn't verify this yet. I will try to do this tomorrow.

@ev-mp hello!
I've executed the code below with different versions of SDK:
void omw::device::RealSenseDevice::RSCallback(const rs2::frame& frame)
{
	if (rs2::frameset frameset = frame.as<rs2::frameset>())
	{
		std::lock_guard<std::mutex> lock(_visualMutex);
		rs2::video_frame left = frameset.get_fisheye_frame(LEFT);
		rs2::video_frame right = frameset.get_fisheye_frame(RIGHT);
		_stroPairWasUpdated = left && right;
		if (_stroPairWasUpdated)
		{
			_curStereoPair.leftFrame = RealSenseUtils::frame_to_mat(left);
			_curStereoPair.leftFrame.timestamp = left.get_timestamp();
			std::cout << "left timestamp: " << _curStereoPair.leftFrame.timestamp << std::endl;
			_curStereoPair.timestamp = left.get_timestamp();
			_curStereoPair.rightFrame = RealSenseUtils::frame_to_mat(right);
			_curStereoPair.rightFrame.timestamp = right.get_timestamp();
			std::cout << "right timestamp: " << _curStereoPair.rightFrame.timestamp << std::endl;
		}
	}
	else
	{
....

....
        std::cout << "Real sense initializing..." << std::endl;
	rs2::config cfg;
	cfg.disable_all_streams();
	cfg.enable_stream(RS2_STREAM_FISHEYE, LEFT, IMG_IN_WIDTH, IMG_IN_HEIGHT);
	cfg.enable_stream(RS2_STREAM_FISHEYE, RIGHT, IMG_IN_WIDTH, IMG_IN_HEIGHT);
	cfg.enable_stream(RS2_STREAM_ACCEL, RS2_FORMAT_MOTION_XYZ32F);
	cfg.enable_stream(RS2_STREAM_GYRO,  RS2_FORMAT_MOTION_XYZ32F);
	rs2::pipeline_profile profile = _pipe.start(cfg, std::bind(&RealSenseDevice::RSCallback, this, std::placeholders::_1));
....

Output with patch 2.19.1:

Well, as we can see timestamps are correct.
Output with last patch (2.21.0)

Also, I've noticed, if I build library with parameter -DBUILD_WITH_TM2=OFF then camera is not detected, but if I leave this parameter with default value (I suppose that default value is ON) then camera is detected successfully, but timestamps are incorrect. Is this a coincidence?

@ev-mp  Oh, my god. Timestamps are double :D


Information not saving to database?

Hi there,
After uploading this template to a hosting service, I am able to login to accounts that have already been made/inserted in to the database, however when making a new account, it appears as if the account creation was successful, the new user appears to be logged in, however no information is saved to the database.
I've double checked my config settings (within the template) as well as created a MySQL user with all permissions (all permissions for testing purposes) and linked it to the torrent database. People can login and upload torrents just fine, however when creating a new account no data is inserted in to the MySQL database.
Any idea's on why this would be the case? Do I need any particular PHP version or .ini settings?
Try register at torrents.azukachan.com, logout and then try log back in using the account you just made. You will notice there is no user found.

Figured it out;
For anyone having a similar issue to as described above; login to your phpmyadmin and navigate to your torrent database, select the users table and then select "structure" from the navigation bar at the top of the screen. From here you will need to set the default value of "uploaderstatus" to 0 instead of NULL.


Data does not have a time field

actually i was using a time series  data  after using my query i am getting data into table but unable to fetch it on time series graph it keeps on showing as data doesnot have a time field.

Same issue, I tried a CSV files with a few different date formats, but getting the same message: "Data does not have a time field".

Seems that time needs to be in this format: "2021-08-19T14:42:40.583563Z"

Check your time format is supported. Here is a simple format to try with:
2021-06-01 11:32:59
Then ensure your model fields are setup correctly. For example, if your time/date column is called "Date" then ensure you have added a Field called Date, with the Type set to Time.
In my attached example I am using a Date field for the timestamp, and have a few other columns with names such as Render(msec), Physics(msec), etc...

That should do the trick and you won't see the "Data does not have a time field" error anymore.

Thank you @Shogan, my mistake was that I didn't select Field Date as type Time. Works now.


Update style guide for annotation standards

See #85. Don't implement until #85 has been completed.

Duplicates #377.


any ideas around table schema to support this?

did you implement?
We have likes in app / but it seems quite complex to support multiple types of responses.
or maybe could be like table with a type....
need to refine the query to get number of likes to number of responses of type.....

Hello @8secz-johndpope ,
Yes i implemented it in one of my project.
You can implement this in tableview also as this is just View.
Can you please specify your requirements using visual approach so i can help you achieve your goal?

so you would have multiple like tables / then collapse into one view? or just add more columns...

I have the same problem I am using this functionality in table view like for the first time if I long-press on button then the view of all reaction are appear just above the button but if at the same time if I long press on the same button but different index path the reaction view appears on the top.


files to download new merged splits updated, small changes i made for…

… most recent experiments

minor pr with some change that were handy for experiments. links for checkpoints updated. gdrive link for merged splits was also updated (though the script in data/MERGED/huge_data did not need to change)


Deprecation Notice

PHP 7.4, Composer version 1.10.10, Symfony 5.1,
Deprecation Notice: Class VK\Actions\Enum\GroupsAccess located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsAccess.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsSubject located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsSubject.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsAgeLimits located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsAgeLimits.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsAudio located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsAudio.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsWiki located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsWiki.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsVideo located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsVideo.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsWall located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsWall.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsDocs located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsDocs.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsMarketCurrency located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsMarketCurrency.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\StoriesLinkText located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/StoriesLinkText.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsRole located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsRole.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsWorkInfoStatus located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsWorkInfoStatus.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsPhotos located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsPhotos.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201 Deprecation Notice: Class VK\Actions\Enum\GroupsTopics located in ./vendor/vkcom/vk-php-sdk/src/VK/Actions/Enums/GroupsTopics.php does not comply with psr-4 autoloading standard. It will not autoload anymore in Composer v2.0. in phar:///home/polyanin/bin/composer/src/Composer/Autoload/ClassMapGenerator.php:201

@aplab While waiting fixes, I've made a small fix on two issues
https://github.com/VKCOM/vk-php-sdk/compare/master...sizeg:patch-2?expand=1
U can use temporary my branch, while they not fixed it
"require": {
   "vkcom/vk-php-sdk": "dev-patch-2 as 5.101.0"
}
"repositories": [
  {
    "type": "vcs",
    "url": "https://github.com/sizeg/vk-php-sdk.git"
  }


change some in Javascript and css and html

Related Issue #306
[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
Description
[Please include a brief description of the changes or features added]
Type of PR

[x] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[ ] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[ ] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

Kindly don't raise a PR without proper description and your assigned issue number. Also this issue has already been resolved and closed.


Image url

Hello. I have a issue about image url in live activity. I show image with url in simulator it work. but on device real not work. You can help me? Handle issue this?. Thank you!

Hello?


@kanaabe => News rss default meta img

Addresses GROW-585
In the RSS feed, uses default meta image for enclosures on 'news' articles rather than thumbnail_img.

This was the only place visible to me passing a meta image, so hopefully it does it!

It looks like enclosure is recommended for thumbnail images so hopefully these consumers know how to read it.


Single Page Structure

I am rendering my pages from NestJS Controller. Is there any way to do it by keeping the singe page structure so everytime a page is switched the window is not refreshed, instead the browser URI is changed, as its done in react-router or Next SSR?
Details:
I have followed the basic example from here: https://github.com/kyle-mccarthy/nest-next/tree/master/examples/basic and created  an app with exact same configuration, everything is working fine, but in my app single page app structure is broken. The views folder is not generating in next directory. Any idea why is this happening? Any help will be appreciated so much.

Worked with it and found out the issue. The issue was using Link from next as proposed:
                href={{
                    pathname: '/views/category/[slug]',
                    query: { slug: slug },
                  }}
                  as={`/category/${slug}`}
            >
```. Missed that part.


[SelfHosted]RestClientCE no attribute 'auth_controller' TB version 3.3.4, TB-Client versin 1.3.2

Having a error using scripts from official documentation.
OS: Windows 10  Python: 3.10.1
Traceback (most recent call last):
  File "D:\Python projecs\pythonProject\main.py", line 37, in <module>
    user = rest_client.get_user()
  File "C:\Users\User\AppData\Roaming\Python\Python310\site-packages\tb_rest_client\rest_client_ce.py", line 316, in get_user
    return self.auth_controller.get_user_using_get()
AttributeError: 'RestClientCE' object has no attribute 'auth_controller'

Some times get error with ssl certification
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)
When I start script sever die for 30-45mins

having the same issue.
Additionally rest_client.login(username=username, password=password) freezes the script.

ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)
Resolved with changing ssl certificate to chain certificate. Site used for checks https://www.sslshopper.com/ssl-checker.html of certificates
'auth_controller' still returns same error as said above, but doesn't kill server.

Hi @Daoma1, update your rest client to the newest version via the master branch and let us know if it helps.


reading default settings from gmond.conf

It would be very useful to read all the default settings from the gmond.conf file if it can be found in one of the usual locations.
It may also be possible to inspect the list of open sockets of any running gmond process to discover configuration details.

FWIW https://github.com/addthis/metrics-reporter-config uses the local gmond.conf but does so rather a pile of regex hacks.


[libdjinterop] add new port with version 0.14.6

What does your PR fix?
Add new port for libdjinterop

Which triplets are supported/not supported? Have you updated the CI baseline?
all

Does your PR follow the maintainer guide?
yes

x64_linux CI failed but
Publishing build artifacts failed with an error: Not found PathtoPublish: /agent/_work/1/a/failure-logs

It works for me locally on x64_linux...

LGTM, thanks for the new port!

Thank you for the quick review and merge.


Add Presets

Closes #610

Changed title because there's 2 additional flag presets.

I'll look at this later once I have enough time

hey, so what are the fflags you added, i want to enable them. also, @pizzaboxer  when this gunna get merged?

@user1126s this is closed


Create SetupSSL.md

SSL setup manual for self-signed certificate

@avarabyeu can we accept in now?

Fixed in 88bacb9


BUG: fix special.hyp0f1 to work correctly for complex inputs.

The current implementation computes hyp0f1 using two relations with
Bessel functions, but it applies those relations incorrectly. This
fixes that issue and restructures the code a bit to avoid calling the
Bessel functions more than is necessary. Closes gh-5764.

@@            master   #5777   diff @@
======================================
  Files          235     235
  Stmts        43287   43290     +3
  Branches      8163    8164     +1
  Methods          0       0
======================================
+ Hit          33615   33618     +3
  Partial       2598    2598
  Missed        7074    7074

Review entire Coverage Diff as of 2ae592f

Powered by Codecov. Updated on successful CI builds.

Thanks, merged.


chore: Add coverage

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ce8f9c4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #71   +/-   ##
=========================================
  Coverage          ?   70.15%
=========================================
  Files             ?        3
  Lines             ?      258
  Branches          ?       48
=========================================
  Hits              ?      181
  Misses            ?       70
  Partials          ?        7

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 ce8f9c4...3c77167. Read the comment docs.


Add example usage of mergeInMap

Changes
Add examples directory
It will eventually contain examples for every function in the library. It currently only contains examples for mergeInMap.
The examples are numbered. The mergeInMap examples look like so:
01-set-property.ts
02-set-multiple-properties.ts
03-set-property-deep.ts
04-compute-property.ts
05-compute-multiple-properties.ts
...

The idea is for the examples to be progressively more advanced. To see very basic usage, go to example number 1. Go to the end for advanced usage examples.
Add examples of mergeInMap
There are 7 examples which show the main features of the function.
Add mergeInMap to README
It exists in a new "Functions" section, which will eventually include every function.

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


python310Packages.jdatetime: 3.8.0 -> 3.8.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.jdatetime is: "Jalali datetime binding for python"
meta.homepage for python310Packages.jdatetime is: "https://pypi.python.org/pypi/jdatetime"
meta.changelog for python310Packages.jdatetime is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.8.1 with grep in /nix/store/wgmxmiad7qd193bpz348s0zc5i3kqv4g-python3.10-jdatetime-3.8.1

found 3.8.1 in filename of file in /nix/store/wgmxmiad7qd193bpz348s0zc5i3kqv4g-python3.10-jdatetime-3.8.1

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
python310Packages.jdatetime
python39Packages.jdatetime

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/wgmxmiad7qd193bpz348s0zc5i3kqv4g-python3.10-jdatetime-3.8.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.jdatetime https://github.com/r-ryantm/nixpkgs/archive/38e16d7c76761055096b0dd7e8353e6401f75571.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/wgmxmiad7qd193bpz348s0zc5i3kqv4g-python3.10-jdatetime-3.8.1
ls -la /nix/store/wgmxmiad7qd193bpz348s0zc5i3kqv4g-python3.10-jdatetime-3.8.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.jdatetime
python39Packages.jdatetime

Maintainer pings

Result of nixpkgs-review pr 155761 run on x86_64-linux 1

2 packages built:

python310Packages.jdatetime
python39Packages.jdatetime


How to Install?

How would I go about installing this plugin to be able to play with rift s modders on quest 2?

Hello!
Sorry for the wait here. You can get help in our discord at https://discord.com/invite/gezGrFG4tz


Discussion: unwrap tracker

Just opening this PR for discussion, related to the changes in https://github.com/huggingface/accelerate/commit/38fd30e764ea87ef86e7d69fcba559c3605925b1
Observations:

According to https://github.com/huggingface/accelerate/blob/38fd30e764ea87ef86e7d69fcba559c3605925b1/docs/source/usage_guides/tracking.mdx#accessing-the-internal-tracker, getting the tracker should expose that tracker's methods. In the example, the user is able to invoke log_artifact, which is specific to the wandb tracker. However, the same appears not to be working for us with the tensorboard tracker when we attempt to invoke add_images: https://github.com/huggingface/diffusers/issues/2468.
The unwrap argument to get_tracker() appears not to be in use.

My question is, is the user expected to pass unwrap=True (and, therefore, there's a slight API change here), or is this being caused by some other issue? How is unwrap meant to work? The change in this PR makes it work for me, but I'm not sure if this is the intended way. Happy to dig into this further once the expectations have been clarified :)

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Yes, it's likely unwrap should be True by default, but the goal is to entice users to use the returned Accelerate tracker API instead as it handles everything. unwrap=True and the if...else... should be used if you want to grab the internal tracker, which is an API change.
Let me stew on this though, there may be something else I can do.

However, your solution Is actually perfect so going to go ahead and merge it :)


Frameworks issues

I am trying to implement using numpy 1.18, I am getting error. So, I deprecated NumPy version to 1.16, a bit of code is executed and throws an error. Can you please share all the environment details like pip list. So, it would be helpful for my minor thesis project.

What's the error you get?

What's the error you get?

Thanks for the mail.
The error got fixed out,
With regards,
Krishna Sai Chandrapati,
+61-423338671
On Fri, Jan 15, 2021 at 2:29 AM XintianHan<EMAIL_ADDRESS>wrote:

What's the error you get?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/XintianHan/ADV_ECG/issues/1#issuecomment-760269828,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMCFU5IASEAFQJJCZK76Y2TSZ4EUJANCNFSM4PMIHMVQ
.

Thanks for the mail.
The error got fixed out,
With regards,
Krishna Sai Chandrapati,
+61-423338671
On Fri, Jan 15, 2021 at 2:29 AM XintianHan<EMAIL_ADDRESS>wrote:

What's the error you get?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/XintianHan/ADV_ECG/issues/1#issuecomment-760269828,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMCFU5IASEAFQJJCZK76Y2TSZ4EUJANCNFSM4PMIHMVQ
.


chore: fix task.json for older tags

This is to override the old major version setting in older tagged versions of the code.

No longer required