Download Documentation

Documentation is to short for me. In the sample you create a temp file. But how do I access this file and write bytes after download success? I am new to kotlin so it could be that I miss something.
The following works for me... but I think it should be possible to access the temp File directly. If I don't write bytes after download success the file is empty.
    Fuel.download(urlString).destination { response, url ->
        File("$myDir/$name.pdf")
    }.progress { readBytes, totalBytes ->
        runOnUiThread {
            val progress = readBytes.toFloat() / totalBytes.toFloat()
            Log.e("asd", "PROGRESS: $progress")
        }
    }.response { req, res, result ->
        val (data, error) = result
        if (error != null) {
            Log.e("fetchProfileImage", "error: ${error}")
        } else {
            result.fold({ bytes ->
                var file = File("$myDir/$name.pdf")
                file.writeBytes(result.component1()!!)
                Log.e("fetchProfileImage", "bytes --> ${bytes.size}")
            }, { err ->
                Log.e("fetchProfileImage", "error: ${err}")
            })
        }
    }

@vogttho firstly thank you for taking the time to raise an issue, I have started to look into this. I will update you when I have more information

@vogttho  I have looked at this and I am unable to replicate this
could you tell me what you get if you change
result.fold({ bytes ->
        var file = File("$myDir/$name.pdf")
        file.writeBytes(result.component1()!!)
        Log.e("fetchProfileImage", "bytes --> ${bytes.size}")
    }, { err ->
        Log.e("fetchProfileImage", "error: ${err}")
     })

to
result.fold({ bytes ->
        var file = File("$myDir/$name.pdf")
        Log.e("fetchProfileImage", "file bytes --> ${file.length}, response bytes -> ${bytes.size}")
    }, { err ->
        Log.e("fetchProfileImage", "error: ${err}")
     })

I get the following:
file bytes --> 0, response bytes -> 104587

what does this do?
  val file =  File("$myDir/$name.pdf")
  Fuel.download(urlString).destination { response, url ->
       file
    }.progress { readBytes, totalBytes ->
        runOnUiThread {
            val progress = readBytes.toFloat() / totalBytes.toFloat()
            Log.e("asd", "PROGRESS: $progress")
        }
    }.response { req, res, result ->
        val (data, error) = result
        if (error != null) {
            Log.e("fetchProfileImage", "error: ${error}")
        } else {
            result.fold({ bytes ->
                var file = File("$myDir/$name.pdf")
                Log.e("fetchProfileImage", "file bytes --> ${file.length}, response bytes -> ${bytes.size}")
            }, { err ->
                Log.e("fetchProfileImage", "error: ${err}")
         })
        }
    }


Tests related to process launching failing on Linux and Mac since refactoring

Turns out the ProcessLauncher wasn't killing the whole process tree which makes a difference on Linux/Mac systems.


Clean nuxt3 install fails on dev command

Environment
Nuxi 3.5.2                                                                                12:25:40 p.m.
12:25:40 p.m.
RootDir: /Users/samijaber/code/nuxt-app-pnpm                                                                                12:25:40 p.m.
Nuxt project info: (copied to clipboard)                                                                                12:25:40 p.m.

Operating System: Darwin
Node Version:     v18.13.0
Nuxt Version:     3.5.2
Nitro Version:    2.4.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction

run npx nuxi@latest init nuxt-app
cd nuxt-app
install (I tried with both yarn v3 and pnpm)
run yarn dev/pnpm dev

Describe the bug
[12:24:36 p.m.] [nitro] [dev] [uncaughtException] Error: listen EADDRINUSE: address already in use /var/folders/qr/md00ttgs1f93zww0lvtv2d6h0000gn/T/com.apple.WorkflowKit.BackgroundShortcutRunner/nitro/worker-41734-1.sock
    at Server.setupListenHandle [as _listen2] (node:net:1716:21)
    at listenInCluster (node:net:1781:12)
    at Server.listen (node:net:1880:5)
    at file:///Users/samijaber/code/nuxt-app-pnpm/.nuxt/dev/index.mjs:621:8
    at ModuleJob.run (node:internal/modules/esm/module_job:194:25) {
  code: 'EADDRINUSE',
  errno: -48,
  syscall: 'listen',
  address: '/var/folders/qr/md00ttgs1f93zww0lvtv2d6h0000gn/T/com.apple.WorkflowKit.BackgroundShortcutRunner/nitro/worker-41734-1.sock',
  port: -1
}

Additional context
I tried:

a yarn v3 monorepo
a simple yarn v3 repo
a simple pnpm repo

All of them fail for me with this same error. I'm on an M1 Mac, macOS version 12.1
Logs
No response

NOTE: this only happens in pnpm and yarn, but works for me if I use npm to install and run the modules.
It seems to be a global nitro issue, as it persists throughout all of my projects.


Expand the rpi_gpio component to support orangepi as well.

Description:
Instead of adding a new component for orangepi (which is a raspberry pi clone), let's just extend the existing component to support more than one family of boards.
I have tested this patch on a raspberry pi 3 and an orangepi prime, in both cases with a LED on a GPIO controlled switch. Note that the orangepi GPIO library requires access to /dev/mem, which I have resolved for now by running Home Assistant as root.
I'm holding off on a home-assistant.io pull request to match this until I get a feel for if this approach is acceptable to you.
This code was implemented to support a Home Assistant tutorial running at linux.conf.au 2019 in a couple of weeks, it would be super cool to be able to have this merged by then so that attendees don't need to carry private patches.
Example entry for configuration.yaml (if applicable):
rpi_gpio:
  board_family: orange_pi
  board: prime

switch:
 - platform: rpi_gpio
   ports:
     29: LED

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @mikalstill,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Ummm, I'm happy to sign the CLA, but https://www.home-assistant.io/developers/cla/ implies there should be a link above for me to do that thing, and I can't see it...

@mikalstill, it looks like your link is https://home-assistant.io/developers/cla_sign_start/?pr=home-assistant/home-assistant#19732
The message from the home assistant bot had it (the word here was hyperlinked to that url).

I swear that comment wasn't there before. Regardless, I'll sign the CLA now.

Hello @mikalstill,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

7fca000bf913d0af8818c2304ee5c3d19261ab4a - This commit has something that looks like an email address (miles@m0les.com). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Note that the orangepi GPIO library requires access to /dev/mem, which I have resolved for now by running Home Assistant as root.

To be honest, I not a big fan of suggesting to run HA as root. How was this issue solved for RPi.GPIO?

Note that the orangepi GPIO library requires access to /dev/mem, which I have resolved for now by running Home Assistant as root.

To be honest, I not a big fan of suggesting to run HA as root. How was this issue solved for RPi.GPIO?

Totally agreed. That said, its not an as-root requirement, its a with-permissions-for-/dev/mem requirement. That can be done with filesystem permissions if required.
My understanding is that Raspberry Pi deals with this by having a kernel driver which exposes the GPIOs in a more friendly manner. I'm not opposed to doing that for OrangePi, but it will take a little time to do.

The Rpi.GPIO library first tries /dev/gpiomem (which is Broadcom specific, and has perms granted via udev), then falls back to /dev/mem (which will require perms).
We can solve the access issue for /dev/mem with some udev rules.
Having said that, after some discussion around the office at Ozlabs, /dev/mem is quite dangerous, any process that can access that can effectively access the memory of any process on the system, including the kernel, and library devs who access that should be admonished^H^H^H^H taken aside and taught how to abstract access via kernel driver.
In this case, we should have access to /sys/class/gpio, which in provided by the pinctrl subsystem on the Orange Pi. This library access GPIO via that: https://pypi.org/project/OPi.GPIO/  It will need a patch to support the Orange Pi Prime, but that seems pretty straightforward.

This should not be part of the Raspberry Pi integration but instead be a new one. Over time, the integrations will diverge, and we will end up with a bunch of if…else statements. We don't want that.

This PR seems to have gone stale. Closing it.


Dask arrays have no dtype attribute

When I create a source from multiple (local) FITS files, I'm able to construct a Dask array using to__dask(). However, the resulting Dask array has no dtype attribute. It does have a shape attribute.
E.g.
import intake
>>> source = intake.open_fits_array('data/*.fits', ext=0)
>>> darr = source.to_dask()
>>> darr.dtype

gives
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-65-2fad671c8367> in <module>
----> 1 darr.dtype

~/anaconda/envs/aia-on-pleiades/lib/python3.7/site-packages/dask/array/core.py in dtype(self)
   1042     @property
   1043     def dtype(self):
-> 1044         return self._meta.dtype
   1045
   1046     def _get_chunks(self):

AttributeError: 'tuple' object has no attribute 'dtype'

Furthermore, darr._meta simply returns the shape attribute.
I've dug a bit into how _meta is set in dask.array.Array, but cannot seem to figure out where exactly the intake loader is missing this bit of information. Any pointers would be helpful! I can also supply a few example FITS files for debugging if needed.
If this is a bug in intake-astro or a feature that needs implementing, I'd be happy to open a PR to do so.

The call signature for dask.array.Array changed recently, and hasn't been updated here yet. You could downgrade Dask, or, I think the following change would do it:
--- a/intake_astro/array.py
+++ b/intake_astro/array.py
@@ -88,7 +88,8 @@ class FITSArraySource(DataSource):
                     dask[(name,) + bits] = (
                         _get_section, self.files[0], self.ext, slices, True
                     )
-            self.arr = da.Array(dask, name, chunks, self.dtype, self.shape)
+            self.arr = da.Array(dask, name, chunks, dtype=self.dtype,
+                                shape=self.shape)
             self._schema = Schema(
                 dtype=self.dtype,
                 shape=self.shape,

Would you mind trying and, if it works, submitting this as a PR? Note that the change is backward compatible.

That seems to have worked! See #5


Release latest versions to Rubygems

On rubygems.org:
https://rubygems.org/gems/geolexica-site

The latest versions are not available:
https://github.com/geolexica/geolexica-site/tags

We need to add a workflow to automatically release versions to rubygems.

This is done now through the automatic release flow.


Prepare for 0.3.0 release

Please specify the area for this PR
/releng
What does does this PR do / why we need it:
This PR updates the bundle dir for the release of 0.3.0 version of the registry-operator
Which issue(s) this PR fixes:
Partially fixes https://github.com/devfile/api/issues/1605
PR acceptance criteria:

[ ] Test Coverage

Are your changes sufficiently tested, and are any applicable test cases added or updated to cover your changes?

[ ] Gosec scans

Fix all MEDIUM and higher findings and/or annotate a finding per gosec instructions: https://github.com/securego/gosec#annotating-code to address why a finding is not a security issue

Documentation

[ ] Does the registry operator documentation need to be updated with your changes?

Testing changes
Running Unit Tests
Running Integration Tests

Special notes to the reviewer:

Do these changes go into main or a release branch? I just ask because the VERSION file is still at 0.1.4 and doesn't look like it has been being updated on main

As decided, in order to unblock a potential release automation we're going to:

Merge the current PR to release-v0.
Update the operatorhub & openshift-catalog sides.
Publish v0.3.0 and announce it.
Finally, we're going to merge the release-v0 branch into main making the v0.3.0 the latest release of the operator.

@Jdubrick @michael-valdron as the current PR had a few conflicts when tried to rebase it with main, I've closed it against https://github.com/devfile/registry-operator/pull/95 which tries to merge the v0.3.0 release updates directly to release-v0 branch.


Can the extensions be done i18n?

At this time, extensions here can not be translated

It'd be very nice to start localizing the extensions someday.

they can now


Unable to show learn profile

After logging in to https://docs.microsoft.com/de-de/learn/ the username and level progress is shown as intended at the  upper right Corner of the website. By clicking on the username the popup menu Shows the profile link. By clicking on the link (https://techprofile.microsoft.com/de-de/[USERNAME]) a 404 - Page not found error is shown.
Login is possible.
All progresses and completed modules are shown.

Hi,
I just wanted to follow up on this issue as I still cannot Access my techprofile / learn profile.
In case you need more Information please let me know.
Thank you in advance

@estelkay Report feedback at https://docs.microsoft.com/en-us/learn/support/troubleshooting#report-feedback


fix: move to etch and decaffeinate

I recreated the views in etch and decaffeinated the coffeescipt files to javascript so we can use jsx.
I'm not sure what we want to use for eslint config so I just used my standard. I'm ok with anything.
TODO

[ ] Write Tests
[ ] Agree on eslint config

@UziTech Thanks, for your work. I also had in mind to switch over to JavaScript maybe.
@sdinesh86 Is it okay for you, if we switch over from CoffeeScript to JavaScript?

We have to move to javascript if we want to use jsx. Atom's version of coffeescript (v1.12.7) doesn't support jsx.

@UziTech Would like to get an "okay" from @sdinesh86, because it's an great change.

I'm not sure what we want to use for eslint config so I just used my standard. I'm ok with anything.
I want to get all possible warnings so we can prevent bugs as good as possible.

@UziTech We realy need to write more tests. To be honest i am not familiar with Jasmine until now.

I added a few tests. We can add more later when we fuzz out how to best optimize the classes.
I removed the unnecessary eslint rules so it is only using the recommended settings and the react/jsx-uses-vars rule so we can just use the SelectList in jsx without eslint thinking it is an unused variable.

hello guys sorry for the late response.
@vivi90  I am completely ok with js, honestly I wanted to switch that long time back.

If we are changing commands we should add a notification on the old commands in case anyone uses that commmand on a toolbar. If we just change commands it will just error out without giving them a clue for what to change it to.

@UziTech Your right, my fault. Where to notice it? In the README file?

I would add the commands back and add a notification with atom.notifications.addWarning() saying something about deprecating that command and changing to the new command. and maybe in a month or two remove the command.

Before we change this too much can we merge this PR and add other changes as other PRs?

@UziTech How about addInfo?
Hm, let's also add it to the README file or documentation later.

So should I go ahead and merge the PR ?

Also guys you can take the liberty to merge the PR's as needed I really doubt I will get enough time in next few week's to start working on this.

@UziTech @sdinesh86 Okay so i will change some little things and then merge it.
Some details can discussed and changed later, if needed.

@UziTech @sdinesh86 I will increase the major version, because it might be API-breaking because of so much changes, i think.

Fixes #25


image-viewer

This PR:

adds helm chart and revproxy config for image-viewer

add a service.yaml for local-image-viewer  similar to how https://github.com/ACED-IDP/gen3-helm/blob/ohsu-develop/helm/fence/templates/service.yaml#L1-L15 and others are implemented so that don't have to manually run the expose command

exec into pod, build venv, run pytests. Pytests pass inside pod except test_base_url_from_env_file errors with
# Assert that the base_url comes from the .env file
        expected_base_url = 'https://env-file-url.com/objects/'
        assert response.status_code == 307
>       assert response.headers["location"].startswith(expected_base_url)
E       AssertionError: assert False

and [2024-10-08 23:32:33,236][WARNING] Unable to find refresh_file

@matthewpeterkort All tests now pass
root@image-viewer-deployment-57fb9cbcbc-qhvvm:/app/image_viewer# git log --oneline | head -1
3bdb3d7 include .env file for test
root@image-viewer-deployment-57fb9cbcbc-qhvvm:/app/image_viewer# pytest tests/
........                                                                                [100%]
8 passed in 6.38s

Working locally. Going to test this on cbds-dev

tested and running on cbds-dev LGTM


flag truncate_timestamps added to fetcharrow functions

#207
Arrow only allows timestamps with years between 1400 to 9999.
Adding the new boolean parameter truncate_timestamps to functions fetchallarrow and fetcharrowbatches to truncate the value to the valid range instead of throwing.

It looks like OSX if failing but I don't think it is related to this PR (I might be wrong though)

Hi @JorgeGarciaIrazabal - Could you please try rebasing to master and reopen the PR? OSX is fixed there.

Codecov Report

Merging #209 into master will increase coverage by 0.35%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #209      +/-   ##
==========================================
+ Coverage   98.68%   99.04%   +0.35%
==========================================
  Files         145      145
  Lines        3338     3130     -208
==========================================
- Hits         3294     3100     -194
+ Misses         44       30      -14

Impacted Files
Coverage Δ

python/turbodbc/cursor.py
98.97% <100%> (ø)
:arrow_up:

cpp/turbodbc/Library/src/time_helpers.cpp
100% <100%> (ø)
:arrow_up:

cpp/turbodbc_arrow/Library/src/python_bindings.cpp
100% <100%> (ø)
:arrow_up:

...pp/turbodbc_arrow/Library/src/arrow_result_set.cpp
98.5% <100%> (+1.81%)
:arrow_up:

...urbodbc_arrow/Test/tests/arrow_result_set_test.cpp
100% <100%> (ø)
:arrow_up:

...pp/turbodbc/Library/src/result_sets/result_set.cpp
100% <0%> (ø)
:arrow_up:

cpp/turbodbc/Library/src/field_translator.cpp
100% <0%> (ø)
:arrow_up:

cpp/turbodbc/Library/src/connection.cpp
100% <0%> (ø)
:arrow_up:

...c/Library/src/descriptions/boolean_description.cpp
100% <0%> (ø)
:arrow_up:

...python/Library/src/python_bindings/column_info.cpp
100% <0%> (ø)
:arrow_up:

... and 51 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 5556625...997f327. Read the comment docs.

Hi @JorgeGarciaIrazabal - Could you please try rebasing to master and reopen the PR? OSX is fixed there.

I have a different error after rebasing, I am not 100% sure why it did failed this time though.

@MathMagique running the job again, all the checks passed! let me know if you want me to change/add anything to the PR. Thanks in advance

This is not an issue with Arrow but with boost::gregorian::date in the date_to_days function. I would rather fix that than have this truncation workaround.

This is not an issue with Arrow but with boost::gregorian::date in the date_to_days function. I would rather fix that than have this truncation workaround.

Yeah, that makes more sense. Unfortunately, with my limited c++ experience, I can not think of a better alternative for the library. Are you guys planing on working on this any time soon?

@wesm, @xhochy  what would be the approrpriate changes to make in Arrow,  I took at look at the datetime parts of arrow-cpp but it wasn't clear to me what we can use instead of boost::gregorian ?


Use local chrome installed in image

Purpose
Reading more into the errors in the build stage it appears to be happening around the download of local version of chromium in the npm install with timeout / connection issues
The base image we're using comes with a stable version of chrome installed, so this change sets the env flags to skip the download (which is where the fails look to be coming from) and set it to use local version
Ran the npm tests on local and they pass without issue
Fixes UML-1309 (hopefully)
Checklist

[x] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] I have added welsh translation tags and updated translation files
[ ] I have run an accessibility tool on any pages I have made changes to and fixed any issues found
[ ] The product team have tested these changes

It's amusing that the upstream image specifically comments that they only include OS version of chrome cause it's the easiest way to install the dependencies needed for the puppeteer version.


Artifactory download hangs when lost network connection to server

when lost network connection to server, the download should fail with an error, but in our case, download just hangs there, the version we are using is 1.16.1.

I'm seeing similar issue in 1.14.0.


English - change all instances of "authentified" to be "authenticated"

A picture tells a thousand words
Hello again!
This one is an English correction. We wouldn't refer to something as "not authentified", the correct term would be "not authenticated". I've done a search and replace here to replace all instances. It all seems to work fine for me, but please do check it.
I realise this will also change one of the routes, but this is worth it as it's correct English. Hope this is ok!
Before this PR
Routes and props that were not authenticated were referred to as "not authentified".
After this PR
These routes and props are using the term "not authenticated".

Thanks @guillaumechervetaxa :)
If I can get a chance this week, I will try to do a few more things. I was thinking about looking at the docs to see if I could help out there a bit if you don't mind?
I will be working on a prototype during the week, so will do what I can when I get a chance. Thanks again for the quick merge!


Fix pull module code action fails when VSCode configured to an old ballerina.home

Purpose

Pull module code action fails sometimes whenever ballerina.home is set to an old version of ballerina and a latest version is already installed through the installers.

Tooling Pull Module code action, BBE page and LS indexer now uses ballerina/home received  through LSP didConfig change notifications.

This PR resolves https://github.com/ballerina-platform/ballerina-lang/issues/13788

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ba9b443). Click here to learn what that means.
The diff coverage is 36.36%.

@@            Coverage Diff            @@
##             master   #14338   +/-   ##
=========================================
  Coverage          ?   64.81%
  Complexity        ?      425
=========================================
  Files             ?     2182
  Lines             ?    96332
  Branches          ?    12862
=========================================
  Hits              ?    62433
  Misses            ?    28862
  Partials          ?     5037

Impacted Files
Coverage Δ
Complexity Δ

...ngserver/command/executors/PullModuleExecutor.java
7.89% <0%> (ø)
0 <0> (?)

...ballerina/example/BallerinaExampleServiceImpl.java
23.8% <0%> (ø)
0 <0> (?)

...lerinalang/langserver/BallerinaLanguageServer.java
87.34% <100%> (ø)
0 <0> (?)

...lerinalang/langserver/common/utils/CommonUtil.java
56.45% <42.85%> (ø)
0 <0> (?)

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 ba9b443...dd2dd95. Read the comment docs.

Ack. Seems like cause of the #13788 is another. I'll close this PR until then.


why not hook NSBundle infoDictionary function?

[NSBundle aspect_hookSelector:@selector(infoDictionary) withOptions:AspectPositionInstead usingBlock:^(id aspectInfo) {
NSDictionary *dictInfoDictionary;
NSInvocation *invocationInfoDictionary = aspectInfo.originalInvocation;
[invocationInfoDictionary invoke];
[invocationInfoDictionary getReturnValue:&dictInfoDictionary];
NSMutableDictionary *dictInfoDictionaryNew =[NSMutableDictionary dictionaryWithDictionary:dictInfoDictionary];
dictInfoDictionaryNew[@"CFBundleIdentifier"] = @"com.naruto1.demo1";
NSLog(@"[%s,%d] dictInfoDictionary:%@",FUNCTION,LINE,dictInfoDictionaryNew);
[invocationInfoDictionary setReturnValue:&dictInfoDictionaryNew];
} error:NULL];
when I write this code:
NSLog(@"[%s,%d] dictInfoDictionary:%@",FUNCTION,LINE,[[NSBundle mainBundle] infoDictionary]);
console:
*** -[__NSDictionaryM retain]: message sent to deallocated instance 0x155ef1580

Aspects is open source - I welcome you to debug and provide a pull request for this.


mark things that need implementing still

Codecov Report

Merging #415 into experimental will decrease coverage by 0.04%.
The diff coverage is 0%.

@@               Coverage Diff                @@
##           experimental     #415      +/-   ##
================================================
- Coverage         72.89%   72.85%   -0.05%
================================================
  Files                30       30
  Lines              1734     1735       +1
================================================
  Hits               1264     1264
- Misses              470      471       +1

Impacted Files
Coverage Δ

vent/menu.py
59.09% <0%> (ø)
:arrow_up:

vent/menus/help.py
36.84% <0%> (ø)
:arrow_up:

vent/menus/main.py
11.17% <0%> (-0.07%)
:arrow_down:

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 2b83583...6cc2976. Read the comment docs.


Changing some properties like color, lineStyle after creating plottable with constructor, has no effect.

var hb = formsPlot1.plt.PlotScatter(
	Xs[brick].Values.ToArray(),
	highBands[brick][c].Values.ToArray(),
	color: Color.Green,
	lineWidth: 2,
	markerShape: ScottPlot.MarkerShape.none);

hb.label = "Brick_" + brick + "_HB_" + c;
hb.lineStyle = ScottPlot.LineStyle.Dash;
hb.color = Color.Blue;

if (myCurrentTemplate.Plottables.ContainsKey(hb.label))
	hb.visible = myCurrentTemplate.Plottables[hb.label].visible;
else
	hb.visible = false;

in above code snippet lablel and visibility will get modified but color and lineStyle will remain unchanged.

Thanks for mentioning this @Minu476. I solved the issue and will publish a new NuGet package this weekend.
I created a test which recreated the original problem
https://github.com/swharden/ScottPlot/blob/3687cd50295431764cce5bf0584634d494c49862/tests/Styling.cs#L13-L36
It now produces the correct output

Dear Scott,
Thank you for fixing both  issues. You are very smart and wonderful person!
When I get time; I share my story of lifetime trading system development
with you! There is a light at end of tunnel specially for individuals as
smart as you in that game.
Kind regards,
Nasser
On Wed, Nov 13, 2019 at 3:20 PM Scott W Harden<EMAIL_ADDRESS>wrote:

Thanks for mentioning this @Minu476 https://github.com/Minu476. I
solved the issue and will publish a new NuGet package this weekend.
I created a test which recreated the original problem
https://github.com/swharden/ScottPlot/blob/3687cd50295431764cce5bf0584634d494c49862/tests/Styling.cs#L13-L36
It now produces the correct output
[image: image]
https://user-images.githubusercontent.com/4165489/68812901-341ac800-0642-11ea-8943-793406eb3ac8.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/swharden/ScottPlot/issues/173?email_source=notifications&email_token=ADTE6IDP7KMTBVUSKXF5PRTQTSDUNA5CNFSM4JMYU2R2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEAAQ3I#issuecomment-553650285,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADTE6IEIUZ4XB23HD3JFBMDQTSDUNANCNFSM4JMYU2RQ
.


Help Wanted: maintainers wanted

I, @crccheck, don't use graphite these days. So I don't have the personal need to maintain this project anymore.
Ideas:

move this to a possibly new organization
add more people to the pypi project
add more people with push access

There are some basic levels of code quality I would like this project to meet that I would have put in (PR checks, contributers.md, code coverage, github templates) but I haven't had the time to do it yet.

If anyone is curious, I've been using Prometheus + Grafana.

As long as someone reviews my PRs before merge I'd be happy to even do the pypi work.  I think I'm reaching the feature end that I need for work.

Krux is planning on at least moderate use of cloudwatch-to-graphite for the foreseeable future and we would be happy to host it from our organization. I don't know the mechanics pushing to pypi off the top of my head, some help tuning that would be appreciated.

I would love to help. Let me know how I can help and maintain this project.

Thanks. I already gave you push access, @paul-krohn and I can get you push access to PyPI too.
@mkohn After you author a few pull requests, I can give you push access as well.


glibcCross: use a libgcc built separately from gcc

Summary
This PR completely and finally solves the gcc<->glibc circular buildInputs problem, for cross compilation.  The same technique can be applied to native builds in the future.
Closes #213453
Motivation
Prior to this PR, we had the following circular buildInputs problem:

gcc has glibc in its buildInputs

a compiled copy of glibc must be present before building gcc; if it isn't, gcc cripples     itself (inhibit_libc) and refuses to build libgcc_s.so

glibc has libgcc_s.so in its buildInputs

glibc dlopen()s libgcc_s.so in order to implement POSIX          thread cancellation.  For security reasons glibc requires that the path to libgcc_s.so is hardwired into glibc at compile time, so it's technically not a true dynamic link -- it just pretends to be one.

libgcc_s.so is built in the same derivation as gcc

libgcc_s.so is built as part of the gcc build          process

We must cut one of these three links in the loop.
Previous Attempts
Previously https://github.com/NixOS/nixpkgs/pull/238154 had attempted to cut link (1) by building gcc without glibc, and using the libgcc_s which emerges from that build.  Unfortunately this just doesn't work.  GCC's configure script extracts quite a lot of information from the glibc headers (which are a build artifact -- you can't just copy them out of the source tarball) and various ./configure-driven linking attempts.  If glibc isn't around at build time you wind up with a libgcc_s.so that is missing various unwinder features (see https://github.com/NixOS/nixpkgs/issues/213453 for the most problematic one).
Musl "cuts" link (2), or rather never creates it in the first place.  "Cancellation cleanup handling in musl has no relationship to C++ exceptions and unwinding... glibc implements cancellation as an exception".  IMHO Musl made the smarter decision here.  It is incredibly rare to find a codebase that uses both POSIX thread cancellation and C++ exceptions.  I have never seen a codebase that uses both and expects them to be aware of each other, and I would be astonished if one existed.  Glibc paid an immense cost in complexity for something nobody has ever used.
Changes Made
This PR cuts  link (3): instead of building libgcc_s.so as part of gcc, we build  it separately from gcc.  Now there is a strict acyclic graph of buildInputs:
 gccWithoutTargetLibc
 |
 +--->glibc-nolibgcc
 |    |
 |    v
 +--->libgcc
 |    |
 |    v
 +--->glibc
 |    |
 |    v
 +--->gcc

In other words, there's a simple linear buildInputs chain glibc-nolibgcc -> libgcc -> glibc -> gcc where all four packages are compiled by (and therefore have as a (native)BuildInput) gccWithoutTargetLibc.
gccWithoutTargetLibc and glibc-nolibgcc are strictly bootstrapping artifacts; nothing else has them as a buildInput and they shouldn't appear in the closure of any final deployment packages.  glibc-nolibgcc lacks libgcc_s.so, so it will segfault if you try to use it with POSIX thread cancellation.  Fortunately all we need from it is (a) its headers (lib.getDev) and (b) to use it in the ./configure script for libgcc.
When translated over to the native bootstrap, xgcc takes the place of gccWithoutTargetLibc, and the "first glibc" (we build two of them) takes the place of glibc-nolibgcc.  At that point our native and cross bootstrap have the same overall architecture, and it becomes possible to merge them (at last!)

@ofborg build tests.cross.mbuffer

@ofborg build tests.cross.mbuffer

@ofborg eval

Figured out how to do this without a rebuild of native stdenv; base has been switched to master.

Built:

[x] tests.cross.sanity

@Majiir, as the reporter of the bug which this fixes, would you mind verifying the fix?  If you can confirm that and approve this PR, I intend to merge it; then you won't need to carry a patch any longer.

Building pkgsCross.armv7l-hf-multiplatform.glibc fails (full build log):

Thanks for catching this, @Majiir.

Fix is in 322acc7
Regression test for armv7hf is in ac86ffc (so I don't miss this again)

36909cf and 1fea173 fail with infinite recursion encountered

I've squashed these two commits into the commit after them.
I originally separated them in order to try to change only one package per commit, but unfortunately that creates non-evaluable commits in the history which -- as you noticed -- is a pain when git bisecting.

I'm also trying to cross-compile my whole armv7l system on this branch

Let me know how it goes!  If you run into any problems that manifest for cross but not for native let me know; there's a good chance I've seen them before.  I mostly hang out in #tvl on hackint.org.

I get the arm-isa.h: No such file or directory error on armv6l and armv5tel; https://github.com/NixOS/nixpkgs/commit/322acc798714c06c64b4ca5bc4865de269a84581 probably needs to be expanded to cover all 32-bit ARM architectures.


When dragging into the margins of a connected list, the placeholder should not fall to the bottom

This can be the margins between the items, or it can be the margins on the sides of the lists. It is particularly bad when you are dealing with a large list and the placeholder is off the bottom of the screen. Once the placeholder is in the destination column, it behaves correctly. You can see this behavior on the demo page (I added a little padding to demonstrate the point):

@burtonjc  this will appear only a fraction of second., but this is a good catch will look in to this.

Thank you for the support, @a5hik! We are actually noticing this a lot. We have 15px of margin along the sides of our sortable containers and 15px in between each sortable item. If you are dragging quickly, it is fairly common for us to accidentally drop the sortable item at the bottom.

Looks like it's an old one, but it's a real UX wrecker.
Is there a fix or workaround for it?


Submitting form without submit button

Have a usecase similar to this:
https://stackoverflow.com/questions/77306898/how-to-submit-a-form-in-sveltekit-without-a-button
But I don't seem to be able to find any resources on how to get a reference to the actual form below, any idea how this might work?

In this situation, it's probably best to check out this part of the docs: https://www.formsnap.dev/docs/headless-usage
More specifically the part about Native form elements, which will let you access the form the way you want!


https://jira.desk.technology/browse/APIC-156

@eldark88 please review

@nartiles Have you verified that when we set etag header using DSAPIETagCache, cache policy NSURLRequestReloadIgnoringLocalCacheData doesn't clear it? Since, NSURLRequestReloadIgnoringLocalAndRemoteCacheData is not implemented, I assume it doesn't do anything with HTTP cache headers. But I have a feeling that NSURLRequestReloadIgnoringLocalCacheData might clear/modify cache headers. If it doesn't, than I'm totally fine with these changes and feel free to :shipit:

I confirmed with Charles Proxy that NSURLRequestReloadIgnoringLocalCacheData does not clear 'If-Non-Match' header when set from DSAPIETagCache.

@nartiles :shipit:


Added phaseDemolished parameter in RevitConverter

Description & motivation

Added phaseDemolished parameter
Identical implementation to previous issue #1390
Fixed Issue : #2215

Changes:

speckle-sharp\Objects\Converters\ConverterRevit\ConverterRevitShared\ConversionUtils.cs

To-do before merge:

Parameter can be null and is not included if the value is null (should this be changed?)

Screenshots:

Validation of changes:

Checklist:

[X ] My pull request follows the guidelines in the Contributing guide?
[X] My pull request does not duplicate any other open Pull Requests for the same update/change?
[X] My commits are related to the pull request and do not amend unrelated code or documentation.
[X] My code follows a similar style to existing code.
[X] I have added appropriate tests.
[ ] I have updated or added relevant documentation.

I wasnt aware my commits to my own branch would be automatically added onto the pull request, is it possible to remove the latest commit?

Hi @Dries-01,
It's not really possible to remove a commit (as far as I know), but you can reset your git branch to the previous commit and then force push to the branch. That process looks different depending on which tool you are using for git operations.
You'll want to be careful when resetting your branch that you do a soft reset and then commit those changes to a new branch if you want to keep those changes.
Git operations can be really confusing. If you still aren't clear on what to do, then you can probably look up "how to do a git reset using insert tool that you are using for git operations"

I did a revert on the latest commit, it should be fine now

Hey @Dries-01
Thanks so much for the PR. It looks perfect so we'll go ahead and merge it in. This functionality will be availible in our next release, 2.13.
Just curious, do you have a specific reason for wanting to set the demo phase for objects?

Hi @connorivy
I might cast some light to that.
Since we're working with LCA calcs an objects different states (existing, new or demolished) changes the different LCA Phases that we need to use when calculating the LCA. Until now we've been able to have the two states; existing and new, but we what to be able to automate the last option when an object gets demolished during construction.
Hope that helps. :)


Update deps & fix

Two main changes:

A fix spotted by @carsonfarmer in the Dispatcher.
Updating our dependencies. Turns out we had some important things quite outdated, so I guess we're getting multiple fixes/improvements for free.

May be relevant for @andrewxhill who has experienced some networking problems too.

👍


Feature request: Add explorer/API for bitcoin testnet ordinals

Very welcome for users who prefer to test their applications cheaply on testnet rather than costly on mainnet. Likewise for dapp developers who like to run a testing environment alongside their mainnet environment.

news?

@thewrlck I read this post from Sarala as: it is being prioritized but will take time.

Source: https://discord.com/channels/621759717756370964/909867922971820102/1151552712610164736
Perhaps @CharlieC3 or Rafael can confirm here and/or give a rough timeline.

Are there any updates on this?

since https://github.com/hirosystems/ordhook/pull/208 is delivered, this could be done with some effort right? @deantchi

I will bring this up for prioritization next week.

I will bring this up for prioritization next week.

any update on this?

@diwakergupta any update on when this will be available?

Bump.

When will this function be OK?

bump

we need this, more than a year has passed!

Any news?


SimpleMenu is broken with exactly 2 elements

MWE:
julia> using Term.LiveWidgets

julia> SimpleMenu(["a", "b"]) |> frame
"\e[1m\e[37ma  \e[22m\e[39m\n\e[2mb  \e[22m"

julia> SimpleMenu(["a", "b", "c"]) |> frame
a
b
c

julia> SimpleMenu(["a"]) |> frame
a

Thanks - I'm not currently actively maintaining this, but hopefully it will be fixed eventually


Updating the master branch

Hi @kuanfandevops, I'd like to keep master updated. Can you confirm & approve the merge? Thanks!

Pull request is reviewed, proceed to merge.

 Testing | create a BDD framework such that additional tests can be created/executed) |0|


setting the correct keep alive timeouts in grpc clients to reset the …

…idle timeouts over NLB

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


KeyError: 'frame_id'

Hi,
I encountered a problem in trying to pre train, that is, the steps of using training commands:
sh track_exps/crowdhuman_train.sh
When I use a single GPU or multiple GPUs, the following errors occur:
Traceback (most recent call last):   File "main_track.py", line 390, in <module>     main(args)   File "main_track.py", line 195, in main     sampler_val = DistributedVideoSampler(dataset_val, start_id=args.start_id, shuffle=False)   File "/root/autodl-tmp/TransTrack/datasets/sampler_video_distributed.py", line 36, in __init__     if img_info['frame_id'] == start_id: KeyError: 'frame_id' ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 2488) of binary: /root/miniconda3/bin/python3
There is no problem in the above steps. I don't know the cause of the problem.Thank you.

Hi~
Thanks for reporting this bug. We have fixed it just now.


Enhances EIA functionality

This PR enhances the EIA class functionality by

adding a list_facets function to help data discovery.

Does not duplicate existing functionality
Handles exceptions

adds a _facet_handler function to ensure facets are passed in a list (helpful for cases where users desire a single facet type).
in EIA.get_dataset

Apply _facet_handler
Add frequency parameter with default value hourly, updated the docstring
updates docstring to reflect new handlers

[Bug-fix] in _handle_region_data checks that the dataframe has a column, col before casting series values to float.
Adds handler for rto/region-sub-ba-data
Adds test for new handlers.

Unfortunately, I have not run the tests because I'm working from a windows machine. But the new test follows the pattern of other tests. Please let me know if I need to change anything in order to merge this PR!

Hey @samgdotson thanks for your contribution! It looks like the linting check is failing. Are you able to run linting, either with make lint-fix or poetry run ruff gridstatus/ --fix then poetry run black gridstatus/?

Thanks for the quick review @WillKoehrsen. My apologies for not fully reading the CONTRIB guide.
Updates to this PR:

runs blackify
adds test_region_subba_data to actually test the new functions.
adds a test_list_facets to test the other new function, which wasn't previously tested.
changes the start and end variables for a few tests in test_eia.py to provide a fixed and consistent time period.

Regarding that last point, there were a few tests that depended on pd.Timestamp.now() AND expect a dataset with no null values. In my view, this is undesirable because a reporting entity that has no nulls today might have nulls tomorrow (evinced by currently failing tests!). Let me know if you'd like this change reverted.


Memcached / Redis usability

Hello,
As I know, Laravel provide memcached / redis for many things like :

Session driver
Cache driver

At first see, the DotEnv don't have memcached / redis variables.
It will be a nice feature !
HerveM

look in config/database.php

For session driver, it's this file https://github.com/cachethq/Cachet/blob/master/config/session.php#L28
And for cache, it's this file https://github.com/cachethq/Cachet/blob/master/config/cache.php
But yes, redis config info must be set in config/database.php as said in Laravel doc
I think I will do a PR for adding this to .env file etc..

Thanks!

If this gets added into the .env file I can also add a Redis container to the docker-compose setup.

Done :)


Refactoring and fixed issue with different hash for the same package

Checklist before submitting

[ ] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
This PR does following -

Fixes the issue below:
Issue: Package name generated in conda env files has different hash than what the actual package is built with. For e.g.
opence-conda-env-py3.10-cpu-openmpi.yaml has libevent package name as - libevent 2.1.11.* h9983d05_4 whereas the package is built with name libevent-2.1.11-hbb84724_4.conda.
Cause: This difference is due to the addition of ppc_arch_variant in the conda config for libevent package during conda env file generation. But it was not being added when the package was built in build_feedstock.py
Fix: ppc_arch_variant is added in both the flows so that the package hash is same while writing conda env file as well as during the build. Also, the global variable named PPC_ARCH_VARIANT was not being set rightly. So, it is also corrected by adding opence_globals.py and defining that global variable there.

Refactoring of the code -
Created constants.py to have all the constant variables defined in it and utils.py to hold all the utility functions. Updated all the files to work with this separation of the code of utils.py.

Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

We can check if we can add a test to verify the env time hash and build time hash values of the packages in a subsequent PR.


Docker compose is failing on policy service buildstep

Problem description
I have attempted to pull down and run the latest Guardian code, from main branch (last tag v2.14.2)
Attempts have been made on three different systems/OS:

Digital Ocean (Ubuntu 20.04.2 LTS)
OSX iMac (13.4.1 - Ventura)
OSX iMac (12.x - Montgomery)

All systems have failed to run the docker build at Step 11/11 is failing for ERROR [policy-service policyservicebuilder 11/11] RUN yarn run build:prod
See screenshot below.
Step to reproduce
Following the documentation:

Configure GUARDIAN_ENV="develop" in .env
Add OPERATOR_KEY, OPERATOR_ID, and IPFS_STORAGE_API_KEY for web3.storage.
Run docker compose up -d --build

Expected behavior
Docker compose builds and runs the service.
Screenshots
I have listed two screenshots first from my local environment and second from my digital ocean droplet, both failing error messages are the same.
From my local system

From my digitalocean droplet

@prernaadev01 @anvabr flagging this for you both, as it breaking deployment of upgrading to the newest version of the guardian. 🕊

@mattsmithies Thank you for raising ticket. We are checking it asap.


refactor: improve member directory filter

Description ✏
Closes #539
Describe what this PR does.

Create two filter dropdowns for 'Company' and 'School'.
Clicking on a filter option automatically updates the displayed results in real-time, based on the selected criteria.

Type of Change 🐞

[ ] Feature - A non-breaking change which adds functionality.
[ ] Fix - A non-breaking change which fixes an issue.
[X] Refactor - A change that neither fixes a bug nor adds a feature.
[ ] Documentation - A change only to in-code or markdown documentation.
[ ] Tests - A change that adds missing unit/integration tests.
[ ] Chore - A change that is likely none of the above.

Checklist ✅

[X] I have done a self-review of my code.
[X] I have manually tested my code (if applicable).
[ ] I have added/updated any relevant documentation (if applicable).

I'm so confused why the company tag faces that issue but the school tags works well. I also noticed when I filter a company that is not in my work's experience the tag is not there at all.
I have tried using the loader function on the FilterDirectoryDropdown component to ensure the company filter is included in the search params alongside the other filters but it still doesn't work.

I am going to check out the offers page to see how the filters were implemented.

Forgot to mention I fixed the issue a few weeks ago.


Streaming SQL in Data Mesh / Netflix Blog

URL: https://netflixtechblog.com/streaming-sql-in-data-mesh-0d83f5a00d08
Review (1-2 sentences): May be, if you're writing a Data Platform in your company, there are not too many insights here. But I guarantee at least one: look how UI for Data Platform can look like

Published in Digest #34.


Destroy method needs to remove data

Currently have to add: "target.removeData('plugin_accTabs')" in order to completely destroy the plugin data.

When destorying the plugin, we may need to using .off to unbind the trigger when using existing elements as well as removeData.

There is a pull request for this: https://github.com/nomensa/jquery.accessible-tabs/pull/13


playermodel

i added playermodel ( mesh ) and now movement is glitching

please help

make sure the player model doesnt have a collider, or just disable the collider on yourself


Key binding 'h' overrides special-mode binding

srfi-mode is derived from special-mode; in the keymap of the latter mode, h (for "help") is bound to describe-mode. We should probably not override that binding in srfi-mode.
Ref 0e3ddd6

@arthurgleckler Since h is the standard help key, should we pick another binding for the landing page? For example, l (lowercase letter ell) is free.
Do you use the landing pages a lot even from Emacs with one-key shortcuts to the discussion and repo readily available? If they are special-purpose, we can also preserve the command without a key binding, in which case it needs to be run with M-x.

On Sun, Jun 28, 2020 at 10:06 AM Lassi Kortela<EMAIL_ADDRESS>wrote:

@arthurgleckler https://github.com/arthurgleckler Since h is the
standard help key, should we pick another binding for the landing page? For
example, l (lowercase letter ell) is free.

Good point.  That would be fine.

Do you use the landing pages a lot even from Emacs with one-key shortcuts
to the discussion and repo readily available? If they are special-purpose,
we can also preserve the command without a key binding, in which case it
needs to be run with M-x.

Yeah, I use it when checking that the landing page has been generated
properly, e.g. after adding See-also links.

Yeah, I use it when checking that the landing page has been generated properly, e.g. after adding See-also links.

Right. If you want the l key, please make a commit to that effect :)

Fixed in commit 041430e


bro the site is not working check it https://ptprashanttripathi.github.io/linkpe/?pa=6294303922@paytm&pn=MDAFTAB&amt=500

Last month, the Delhi cyber police requested me to shut down this website because scammers were using it to deceive people. Despite being an open source website, it was being misused for fraudulent activities, and it was necessary to take action to protect innocent victims.


FlushInterval not working for release build

Thank you so much for making this project.
But we have experienced an issue. We have use the configuration option com.mixpanel.android.MPConfig.FlushInterval in the AndroidManifest.xml. We have set it to 1.
It works fine for DEBUG build, but not RELEASE build - it seems like the value doesn't work for release build

I don't think we support arbitrary flush interval in production. https://github.com/mixpanel/mixpanel-android/blob/master/src/main/java/com/mixpanel/android/mpmetrics/MPConfig.java#L264
Though there must be a good reason behind this, I don't know the full context. However, what I do know is that setting it to 1 sec (assuming you didn't mean 1 milliseconds which is really bad) will lead to a high power/data consumption.

@changxiangzhong
I'll close this ticket as the behavior is expected. Please let me know if you do need this feature and explain your usage case.

It works on debug builds because the default flush interval on debug is 1 second https://github.com/mixpanel/mixpanel-android/blob/master/src/main/java/com/mixpanel/android/mpmetrics/MPConfig.java#L171
And it should work on non debug builds as well since this parameter is customizable through com.mixpanel.android.MPConfig.FlushInterval. Re-opening.

Well.. @patedit you are totally right (and I don't know what I was talking about).
@changxiangzhong I've tried it just now and it worked. If you add
        <meta-data android:name="com.mixpanel.android.MPConfig.FlushInterval"
            android:value="1000" />

to your AndroidManifest under the application block, it will work. I don't exactly know what you did, but don't put an expression there (eg. 1 * 1000).


getOptionKey Not Found In AutocomplateProps

I Try To Use AutocompleteProps And Call getOptionKey, But It's Not Found

Which version of Material UI are you using? It's available since version v5.14.19.
It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve.


Scan existing code

It'd be fantastic if Todo could scan existing code and open up issues for previously committed TODOs

No arguments here - however, I don't think it has a huge impact, since it'd only act once per repo. The thing is, Todo's current implementation specifically acts on diffs - this is for performance mostly. So things would have to change a lot for that to be viable.
That said, I'm open to a PR getting this into Todo! It'd have to listen for installation_repositories.added events, and deal with the Gitdata tree and blob APIs.

yes

yeah, shh #badbot :no_bell::robot::no_bell:

I don't mind if this issue is closed as wontfix or left open and neglected indefinitely, but I'd prefer if a bot did not ask if the most +1'd issue is still relevant every couple months

Yeah, not sure why it ignored the activity. But I'm inclined to leave it closed, because it's not something I can tackle, and because I don't think it's something that this app should be able to do. Due to how it scans for TODOs (via diffs), it'd be pretty hefty to instead scan all files; might be better for a GitHub Actions implementation of todo!


iotics: OAS header updates

adding header description & required annotation support to OAS generation.
leaving existing description default if not provided.

Great fork & additions to gnostic 👍

@clintberry @jnewmano


[LANG-1655] Add API canonicalCompositionAfterStripAccents (an issue where characters were separated in the NFD…)

https://issues.apache.org/jira/browse/LANG-1655

Coverage increased (+0.0003%) to 94.939% when pulling 74c0f0585b0f89e62cf6e7dfe04d8bb90bcac364 on dhmin5693:feature/LANG-1655 into d1e9e598c9bcbf91afa174fa9b6c2ef30bbc8157 on apache:master.

-1: Too niche IMO, more of a Commons Text method regardless.

Closing: No other opinions in over a month.


Improve "How to file a good bug"

See comments in https://github.com/google/WebFundamentals/pull/3680 ("Show outdated") for comments by @petele.
I volunteer Dru to make changes and me to review them.

Dru is @drufball

@petele, you made some changes already, is there more to be done still?

I'm happy with it, but if @drufball wants to take another swing that's fine. I believe @jpmedley was also doing an edit pass on it.


There is a bug where generation model textures are missing.

Describe the Suggestion
This model is missing textures on some devices
Water Wheel
Large Water Wheel
Crushing Wheel
Blaze Burner (empty included)
Valve Handle (All colors included)
Mechanical Roller
Flywheel
Additionally, the Fluid Pipe is operating normally, but the texture appears to be disjointed
The texture models of the above devices are displayed as shown in the attached photo below
Screenshots and Videos

Additional Context
No response

if you are using sodium you need indium for models to render properly

Thank you for quick response!!!
Solved as a result of using indium👍👍👍


Invalid site URL (ErrorCode: 0x0201000F)

Trying to migrate a Sharepoint 2013 on-prem site collection to a SharePoint online site and receiving an "Invalid site URL" message.  Verified I can navigate to the 2013 site by entering the site's url in a browser.
FatalError_R1.xlsx
Using the Sharepoint Migration Tool v. <IP_ADDRESS>

@dorseytm thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Set ESLint to use ES6 syntax

ESLint allows you to specify the JavaScript language options you want to support. By default, ESLint expects ECMAScript 5 syntax. You can override that setting to enable support for other ECMAScript versions as well as JSX by using parser options.

More on that here.
Adding this to the ESLint config should resolve the error:
"parserOptions": {
    "sourceType": "module",
    ...
},

/cc @arungalva

Cherry-picked commit b34d4fbb01fcdb936ad9f273c0ba07675fa08bc4 from arungalva/nyts/chore-es6-syntax. Thanks for the changes @arungalva :heart:

Cherry-picked commit b34d4fbb01fcdb936ad9f273c0ba07675fa08bc4 from arungalva/nyts/chore-es6-syntax. Thanks for the fix @arungalva :heart:
Fixed with 255285348a56d45329a63cfbe56c8c5f21406212.


Docs - Define random values in examples

This PR adds the definition of the random values when they are used in the constraints examples. This brings clarity and avoid differences with examples of others pages that define it when they use it.

Seems like after merging #350 we ended up with some merge conflicts. Could you rebase from the latest next?

Seems like after merging #350 we ended up with some merge conflicts. Could you rebase from the latest next?

Done :)


ISPN-6042 NPE when using avg on repeated property and one group does …

…not contain any value
jira: https://issues.jboss.org/browse/ISPN-6042

Integrated in 8.1.x.


ncvisual's set_yx function doesn't work

It'd be nice to have a minimal C example to compare against. This is a rust example:
use libnotcurses_sys::*;

const H: u32 = 10;
const W: u32 = 10;

fn main() -> NcResult<()> {
    let mut nc = unsafe { Nc::new()? };
    let mut stp = unsafe { nc.stdplane() };

    let buffer: Vec<u8> = vec![0x88; (H * W) as usize * 3];
    let visual = NcVisual::from_rgb_packed(buffer.as_slice(), H, W * 3, W, 255)?;
    let vopt = NcVisualOptions::builder()
        .yx(1, 2)
        .blitter(NcBlitter::Pixel)
        .parent(&mut stp)
        .build();
    let _plane = unsafe { visual.blit(&mut nc, Some(&vopt))? };
    nc_render_sleep![&mut nc, 1];

    // FIXME doesn't seem to do anything:
    visual.set_yx(0, 0, 0xFF0000FF)?;
    visual.set_yx(1, 1, NcPixel::new(0xFF0000FF))?;
    let _res = unsafe { c_api::ncvisual_set_yx(visual, 2, 2, 0xFF0000FF) };
    nc_render_sleep![&mut nc, 1];

    visual.destroy();
    unsafe { nc.stop()? };
    Ok(())
}

do i not have an example anywhere of usage?
is this just a bug in the wrappers, or core?

Yeah, I'm not sure where the bug is... the rust wrappers for that part are pretty minimal.
There's no standalone example in the C code, just one usage in the dragon demo. I can try to make a minimal ncvisual C example from tests and examples, but I'm very slow working with C.

and.... it's done! :D I forgot I had a barebones C code for a visual saved from more than a year ago...
It does work in C, so at least now I have that clear.

#include <notcurses/notcurses.h>

int main(void){
	// init
	struct notcurses_options nopts = {
		.flags =
			NCOPTION_SUPPRESS_BANNERS
	};
	struct notcurses* nc = notcurses_core_init(&nopts, NULL);
	if(nc == NULL){
		return EXIT_FAILURE;
	}

	unsigned dimy, dimx;
	struct ncplane* stdn = notcurses_stddim_yx(nc, &dimy, &dimx);

	ncplane_putstr_yx(stdn, -1, -1, "press a key to continue");

	// VISUAL

	// allocate & fill the pixel buffer
	size_t height = 10; size_t width = 10;
	size_t buflen = height * width * 3;
	unsigned char* buffer = malloc(buflen);
	if (buffer == NULL)  {
		goto err;
	}
	for (size_t p = 0; p < buflen; ++p) {
		buffer[p] = 200;
	}

	// setup the ncvisual
	struct ncvisual* ncv =
		ncvisual_from_rgb_packed(buffer, height, width * 3, width, 0xFF);
	free(buffer);
	if(ncv == NULL){
		goto err;
	}

	struct ncvisual_options vopts = {
		.n = stdn,
		.y = 1,
		.blitter = NCBLIT_PIXEL,
		.flags = NCVISUAL_OPTION_CHILDPLANE
	};

	// modify the visual
	ncvisual_set_yx(ncv, 0, 0, 0xFF0000FF);
	ncvisual_set_yx(ncv, 1, 1, 0xFF00FF00);
	ncvisual_set_yx(ncv, 2, 2, 0xFFFF0000);

	struct ncplane* ncvp = ncvisual_blit(nc, ncv, &vopts);
	if(ncvp == NULL){
		goto err;
	}

	// render
	if (notcurses_render(nc)) {
		goto err_visual;
	}

	// wait for any input
	ncinput ni;
	do {
		notcurses_get_blocking(nc, &ni);
	} while (ni.id != (uint32_t)-1 && ni.evtype == NCTYPE_RELEASE);

	// exit
	if (notcurses_stop(nc)) {
		return EXIT_FAILURE;
	}
	return EXIT_SUCCESS;

err_visual:
	ncvisual_destroy(ncv);
	ncplane_destroy(ncvp);
err:
	notcurses_stop(nc);
	return EXIT_FAILURE;
}

OMG, I totally forgot blitting was necessary after changing the pixels. Solved.


plots: tooltip breaks when zoomed in

https://user-images.githubusercontent.com/2308172/214419295-15952a51-c551-4b82-bf5a-475610761759.mov

It got fixed by #3132
Simply needs to be released (will do that now).

This is what happens when you have too many product people 😄


Fail to send notification by email with STARTTLS with PlainText

What I'd like to do
send notification by email with STARTTLS with PlainText
What I did
send notification using openssl command and it makes success!! (executed command is below)
but, alertmanager failed to send notification with same credential information.
printf "%s\0%s\0%s" <user> <pass> | openssl base64 -e | tr -d '\n'; echo  ## to get Auth plain text
openssl s_client -connect <host>:<port> -starttls smtp -ign_eof
AUTH PLAIN <auth plain text>
MAIL FROM<EMAIL_ADDRESS>RCPT TO<EMAIL_ADDRESS>DATA
SUBJECT 'TEST MESSAGE'
MESSAGE 'THIS IS A TEST'
.
QUIT

Alertmanager Logs:

level=error ts=2018-03-05T08:48:52.555046949Z caller=api.go:803 msg="API error" err="bad_data: start time must be before end time; start time must be before end time; start time must be before end time; start time must be before end time"

level=error ts=2018-03-05T08:48:59.859725159Z caller=notify.go:303 component=dispatcher msg="Error on notify" err="starttls failed: x509: certificate is valid for xxxxx.com, not yyyyy.com"

Alertmanager configuration file:

global:
 smtp_smarthost: '<host>'
 smtp_from<EMAIL_ADDRESS> smtp_auth_username: '<user>'
 smtp_auth_password: '<pass>'
 smtp_require_tls: true

route:
  receiver: mail
  group_by: ['alertname','name','severity']
  group_wait: 30s
  group_interval: 5m
  repeat_interval: 5m
receivers:
- name: 'mail'
  email_configs:
    - send_resolved: true
      to<EMAIL_ADDRESS>
Why did alertmanager fail to send notification???

It seems that your server doesn't advertise a valid certificate:
err="starttls failed: x509: certificate is valid for xxxxx.com, not yyyyy.com"
Currently there's no way to bypass the verification so you'd need to fix the certificate.
There are a couple of PR opened to turn off the certificate verification or use a custom server certificate but all are stalled (see #1186, #1140 and #757).

Closed by #1528


docs(): enhance contributing guidelines

Add further information about how to retrieve the rep and contribute to it.
Add a section about commit messages guidelines.

Commit cherrypicked from fork


Move jQuery out of the bundle

Fixes #519
The difference between our custom build and the normal jQuery is 18KB, about 15% of our bundle.

I think this is to fix #519, not #439 which is already closed (even though it's related).

My bad. I always select the wrong issue when I type # and then select one from the list that appears.

I think we should use the official slim build of jQuery. 18 KB is not a lot, but it's still a waste.

Oh I didn't even know that there was an official slim version.


Backoff after Accept() errors

Bug report
System info: InfluxDB 1.0.0 on CentOS 6 x86_64
In the Graphite input (as well as other services, probably) there is a tight loop that Accepts client connections.
In the case of a temporary error (such as fd exhaustion on the system), InfluxDB will end up spamming Accept syscalls, and perhaps more significantly, will spam the log significantly. On a fairly slow system that single print was doing 70k lines/sec. The disk filled up fast.
I suggest having some kind of throttling in the error path, either before continuing the loop, or at least inhibiting the logging.
Thanks
Steps to reproduce:

Run out of file descriptors on system
Have hundreds of clients hammering at the door
Log goes nuts

I'm marking this as an enhancement since it's not really a bug even though it produces undesirable behavior at the moment. It seems to me like there should be some maximum backoff and that the maximum backoff can't be too large. Like maybe a maximum of 1 second or 5 seconds.
While we can avoid the 70k/s attempts while the system is exhausted, there will still be some spamming because the system is just not in a good place if this is happening.


NaN after large number of iterations on 200d GloVe training

Thanks for a great codebase!
When training a word-GloVe on a 100M tweets dataset with 30 minimum word-occurrence and resulting in a 1.2M effective vocabulary size, the GloVe trains great, albeit slowly. The error decreases at each iteration... and then after ~25 iterations, it suddenly goes NaN. Will follow up a log.
This happens for me every time, after slightly different iteration count. I don't think it is a memory problem. Rather, perhaps it's some kind of division by a very very small number, that becomes NaN. I'm thinking tiny, non-zero gradients, or taking the inverse of that gradient, resulting in a sneaky divide-by-zero.
If my theory had some merit, I would appreciate a code pointer, where I might look for the NaN in the gradient descent?
Again, it runs for hours, on a large 1.2M vocabulary. Great GloVe... then sudden a NaN. If I choose a much higher word occurrence cutoff or a smaller dimension like 50d, I can not reproduce the error. But with a 1.2M vocabulary and 15-word occurrence window... after running long enough, it happens every time.
Thanks! Love the GloVe.

./demo.sh
mkdir -p build
gcc src/glove.c -o build/glove -lm -pthread -Ofast -march=native -funroll-loops -Wno-unused-result
gcc src/shuffle.c -o build/shuffle -lm -pthread -Ofast -march=native -funroll-loops -Wno-unused-result
gcc src/cooccur.c -o build/cooccur -lm -pthread -Ofast -march=native -funroll-loops -Wno-unused-result
gcc src/vocab_count.c -o build/vocab_count -lm -pthread -Ofast -march=native -funroll-loops -Wno-unused-result
BUILDING VOCABULARY
Processed<PHONE_NUMBER> tokens.
Counted 22289076 unique words.
Truncating vocabulary at min count 25.
Using vocabulary of size 1193516.
COUNTING COOCCURRENCES
window size: 10
context: symmetric
max product: 13752509
overflow length: 38028356
Reading vocab from file "vocab.txt"...loaded 1193516 words.
Building lookup table...table contains 171365725 elements.
Processed<PHONE_NUMBER> tokens.
Writing cooccurrences to disk...........125 files in total.
Merging cooccurrence files: processed 924208891 lines.
SHUFFLING COOCCURRENCES
array size: 255013683
Shuffling by chunks: processed 924208891 lines.
Wrote 4 temporary file(s).
Merging temp files: processed 924208891 lines.
TRAINING MODEL
Read 924208891 lines.
Initializing parameters...done.
vector size: 200
vocab size: 1193516
x_max: 10.000000
alpha: 0.750000
iter: 001, cost: 0.159467
iter: 002, cost: 0.123287
iter: 003, cost: 0.103016
iter: 004, cost: 0.091396
iter: 005, cost: 0.084560
iter: 006, cost: 0.080124
iter: 007, cost: 0.076975
iter: 008, cost: 0.074624
iter: 009, cost: 0.072742
iter: 010, cost: 0.071216
iter: 011, cost: 0.069942
iter: 012, cost: 0.068856
iter: 013, cost: 0.067923
iter: 014, cost: 0.067107
iter: 015, cost: 0.066391
iter: 016, cost: 0.065768
iter: 017, cost: 0.065206
iter: 018, cost: 0.064694
iter: 019, cost: 0.064241
iter: 020, cost: 0.063825
iter: 021, cost: 0.063450
iter: 022, cost: 0.063102
iter: 023, cost: 0.062777
iter: 024, cost: 0.062470
iter: 025, cost: 0.062172
iter: 026, cost: 0.061888
iter: 027, cost: 0.061652
iter: 028, cost: nan
iter: 029, cost: nan
iter: 030, cost: nan
capital-common-countries.txt:
ACCURACY TOP1: 0.00% (0/506)
capital-world.txt:
ACCURACY TOP1: 0.00% (0/3629)
currency.txt:
ACCURACY TOP1: 0.00% (0/752)
city-in-state.txt:
ACCURACY TOP1: 0.00% (0/2467)
family.txt:
ACCURACY TOP1: 0.00% (0/506)
gram1-adjective-to-adverb.txt:
ACCURACY TOP1: 0.00% (0/930)
gram2-opposite.txt:
ACCURACY TOP1: 0.00% (0/702)
gram3-comparative.txt:
ACCURACY TOP1: 0.00% (0/1332)
gram4-superlative.txt:
ACCURACY TOP1: 0.00% (0/1122)
gram5-present-participle.txt:
ACCURACY TOP1: 0.00% (0/1056)
gram6-nationality-adjective.txt:
ACCURACY TOP1: 0.00% (0/1445)
gram7-past-tense.txt:
ACCURACY TOP1: 0.00% (0/1560)
gram8-plural.txt:
ACCURACY TOP1: 0.00% (0/1332)
gram9-plural-verbs.txt:
ACCURACY TOP1: 0.00% (0/870)
Questions seen/total: 93.17% (18209/19544)
Semantic accuracy: 0.00%  (0/7860)
Syntactic accuracy: 0.00%  (0/10349)
Total accuracy: 0.00%  (0/18209)

I can't offer a detailed explanation (I'm not 100% sure why that worked in a similar setting for me) but try setting a different (lower) alpha with the same number of iterations.

Thanks Igor, but this does not address my question. Where in the code could I be getting a NaN? Of course I can avoid it with hyper parameter selection. But I would like to find and fix the problem if possible, so I could train massive GloVes in high dimensions, on sparse-ish data, for a long period of time.
And perhaps so that others could do the same.
Sincerely,
Nikolai

On Feb 18, 2016, at 3:25 AM, Igor Brigadir<EMAIL_ADDRESS>wrote:
I can't offer a detailed explanation (I'm not 100% sure why that worked in a similar setting for me) but try setting a different (lower) alpha with the same number of iterations.
—
Reply to this email directly or view it on GitHub.

Absolutely! Much better to actually fix it rather than avoiding it with parameters!

I would like to find and fix it. Do you have any pointers in the code, where to look?
Thanks,
Nikolai

On Feb 18, 2016, at 10:15 AM, Igor Brigadir<EMAIL_ADDRESS>wrote:
Absolutely! Much better to actually fix it rather than avoiding it with parameters!
—
Reply to this email directly or view it on GitHub.

The code throwing the NaNs is certainly in src/glove.c. See the train_glove() function on line 228 in particular. I recommend you reduce the number of threads to 1 before debugging, and perhaps even replace the pthread calls for standard function calls before debugging. It should be possible to include assertion statements to the effect that no NaNs are produced in various places and then debug with gdb. Given the fact that this indeed likely affects many other users, if you (or anyone else with this problem) are able to provide a corpus, I'll take a look at what's going on.

Thanks! I was thinking something similar so good to hear this from you.
Going to take a look tonight and respond in some details. Not sure I can share this corpus... at least not easily. Let me see about that also.
Best,
Nikolai

On Feb 18, 2016, at 11:51 AM, Russell Stewart<EMAIL_ADDRESS>wrote:
The code throwing the NaNs is certainly in src/glove.c. See the train_glove() function on line 228 in particular. I recommend you reduce the number of threads to 1 before debugging, and perhaps even replace the pthread calls for standard function calls before debugging. It should be possible to include assertion statements to the effect that no NaNs are produced in various places and then debug with gdb. Given the fact that this indeed likely affects many other users, if you (or anyone else with this problem) are able to provide a corpus, I'll take a look at what's going on.
—
Reply to this email directly or view it on GitHub.

Hey @Russell91, suppose I had a theory that line 106 could in some case get into a NaN.
https://github.com/stanfordnlp/GloVe/blob/master/src/glove.c#L106
I'm not quite the C-wizard. What's the easiest way to test that this computation does not yield NaN, and if it does, printf() some stuff? Will figure it out if I don't hear back, but thought maybe can save a bunch of time. If I can catch and replicate it, will do my best to make it reproducible for you, as well. Currently, it takes a couple of hours to fail, but pretty consistent at which point it fails when it does.
My theory is that something falls over and creates a numerical divide by zero, when dividing by a very small, non-zero value. I've see this before in a similar adaptive gradient descent problem.
Good point about running in one thread to test. Thinking about how to change parameters to try to get a fail to happen sooner.
Best,
N

Pretty sure I solved it. Running the 7-hour job on my dataset now to be sure.
Will send a review tonight if it's confirmed. Basically just need to check for isnan() and isinf() before applying the gradient update. If error, just skip the update and keep going.
For a smaller set, I even introduced random NaNs into the gradient, and it trained to the same number.

Wow, sounds very useful if it indeed works.

So it does appear to work. Adding the checks, I'm able to train for more iterations, and to a lower loss in the co-occurence metric. I also tested it by faking bad gradients, and the network is able to skip those updates and train to the same number, as well.
Performance does not seem slower, but I did not carefully benchmark it.
Would you like me to send my change a code review?

Yea, a significant speed decrease would have been unexpected from adding an inline if ... then ... check. Please do send a pull request. If you haven't done that before, just 1) hit the fork button on the glove repo 2) add your own repo locally with git remote add moscow https://github.com/moscow25/ or such, 3) push your changes locally e.g. git push moscow master, and then come back to stanfordnlp/glove and hit the create pull request button.

Excellent. I'll do that tonight. Have sent pull requests before... but it's
been a while :-)
On Sun, Feb 21, 2016 at 5:36 PM, Russell Stewart<EMAIL_ADDRESS>wrote:

Yea, a significant speed decrease would have been unexpected from adding
an inline if ... then ... check. Please do send a pull request. If you
haven't done that before, just 1) hit the fork button on the glove repo 2)
add your own repo locally with git remote add moscow
https://github.com/moscow25/ or such, 3) push your changes locally e.g. git
push moscow master, and then come back to stanfordnlp/glove and hit the
create pull request button.
—
Reply to this email directly or view it on GitHub
https://github.com/stanfordnlp/GloVe/issues/15#issuecomment-186963885.

Pull request created. Thanks! Let me know if you want style changes, or modify them yourself as needed. Thanks for a nice repo.

Looks great! The issue should be addressed by https://github.com/stanfordnlp/GloVe/pull/16.


Duplicate Images - Viewport Change

I have noticed that the script visually works well but when the user changes the viewport the same image is download again and again.
Example when changing the width of the demo page by a few pixels:

I have seen something similar to this but the image was cached. But this is just showing a 304 status of "Not Modified", does this image use bandwidth or am I missing something?
Needless to say Chrome seems to be waiting on the image. I am using the latest version.

This one should be fixed with the 2.3 version. @Ne-Ne can you verify and close this one?

Assuming this has been fixed with 2.3. If not, @Ne-Ne, let us know and we'll reopen this issue.


the color of selected text in the variable preview debug window is similar to background window and that's why I cannot diferentiate the selected text.

the color of selected text in the variable preview debug window is similar to background window and that's why I cannot differentiate the selected text.

The text in popup window is selected but I cannot differentiate.
Thanks.

Related to #55

Hi @felipearpa :wave:, thanks for your contribution :+1:
Thanks for your patience, there are tons of tasks in the Nord project and my free time is limited to work on the massive amount of issues/PRs.
Anyway, I guess this should change when #55 gets resolved, but I'll leave this issue open since it is also related to another background color than the editor. We can check if the regression still exists when the selection style has been changed.


Add Contacts From ObjectDetailView's return url should redirect to the contacts tab instead of the main tab.

Proposed Changes
After adding/assigning a contact/team from ObjectDetailView, the web page should be redirected to the contacts tab of the ObjectDetailView instead of the main tab.

Justification
The user can see immediately that the add/assign operations are successful from the contacts tab. Now the user has to click an extra button to navigate to the contacts tab.

Implemented by #5358


swarm start option support

Can run all, --min, or specific set of services:
# run all the services
$ ./swarm start

# run the minimum required services for dev
$ ./swarm start --min

$ run specific set of services (services must be defined in the `swarm` script
$ ./swarm start etcd influxdb ...

This Correctly only start the specified service now

LGTM


update generated diagnostic messages for isolated declarations

In investigating adopting isolated declarations (#59097), I found that the generated Diagnostics needs a type annotation. This updates the script so we're able to provide a type annotation for it.
(this PR does not turn on the flag but once I make the PR that turns on the flag, I'll link it here: ____)

The duplication is unfortunate; I do wonder if there's a better way but I think this is the minimally invasive way.
The only other thing I can think of is like:
export const Diagnostics = {
    Foo: diag() as Diagnostic,
}

But not sure if ID accepts that.

closed in https://github.com/microsoft/TypeScript/pull/59635


Sky130 ff libs missing table templates

Description
STA warns about missing template. Reading ff libs bombards the logs with these warnings
table template ccsn_ovrf not found.
table template ccsn_ovrf not found.
table template ccsn_ovrf not found.
table template ccsn_ovrf not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_pnlh not found.
table template ccsn_ovrf not found.
table template ccsn_ovrf not found.
table template ccsn_ovrf not found.
table template ccsn_ovrf not found.

Expected Behavior
No warning
Environment report
Kernel: Linux v6.1.20-060120-generic
Distribution: ubuntu 22.04
Python: v3.10.6 (OK)
Container Engine: docker v20.10.22 (OK)
OpenLane Git Version: 54bfaa3d0e826fe64bdc7e6e1049f3600fc28942
pip: INSTALLED
python-venv: INSTALLED
---
PDK Version Verification Status: OK
---
Git Log (Last 3 Commits)

54bfaa3d 2023-05-11T21:01:58+03:00 [BOT] Update openroad_app (#1774) - Openlane Bot -  (HEAD -> master, origin/master, origin/HEAD)
587fe72b 2023-05-11T14:51:16+03:00 [BOT] Update PDK (#1712) - Openlane Bot -  ()
5a291005 2023-05-10T22:30:22+03:00 Integrate back OpenROAD STA (#1784) - Kareem Farid -  (tag: 2023.05.11)
---
Git Remotes

donn	https://github.com/donn/openlane.git (fetch)
donn	https://github.com/donn/openlane.git (push)
fork	git@github.com:kareefardi/OpenLane.git (fetch)
fork	git@github.com:kareefardi/OpenLane.git (push)
origin	git@github.com:The-OpenROAD-Project/OpenLane.git (fetch)
origin	git@github.com:The-OpenROAD-Project/OpenLane.git (push)

Reproduction material
N/A
Relevant log output
line 619, table template ccsn_ovrf not found.
line 625, table template ccsn_ovrf not found.
line 633, table template ccsn_ovrf not found.
line 639, table template ccsn_ovrf not found.
line 647, table template ccsn_pnlh not found.
line 654, table template ccsn_pnlh not found.
line 661, table template ccsn_pnlh not found.
line 668, table template ccsn_pnlh not found.
line 675, table template ccsn_pnlh not found.
line 682, table template ccsn_pnlh not found.
line 689, table template ccsn_pnlh not found.
line 698, table template ccsn_pnlh not found.
line 705, table template ccsn_pnlh not found.
line 712, table template ccsn_pnlh not found.
line 719, table template ccsn_pnlh not found.
line 726, table template ccsn_pnlh not found.
line 733, table template ccsn_pnlh not found.
line 740, table template ccsn_pnlh not found.
line 789, table template ccsn_ovrf not found.
line 795, table template ccsn_ovrf not found.
line 801, table template ccsn_ovrf not found.
line 807, table template ccsn_ovrf not found.
line 815, table template ccsn_ovrf not found.

Is there a particular resolution for this?

@kareefardi I'm having the same issue, did you find any solution to fix this?

FYI, OR doesn't support CCS currently.  Perhaps this is for use in another tool?

@maliberty These are just coming from the PDK without them targeting a specific tool. However, I do think that the warning by OpenROAD is correct and it should be fixed in the PDK file.
@delaram-mz No fix a the moment. They are just warnings and don't cause errors. However, they are noisy and we can attempt to handle them.


chore(vote) change webpack.vote to benefit from latest commits and in…

…stall it from git vs npm (we dont have rights to publish)

mostly said in title, don't have control over package on npm. As we move it will now be fetched from git with ^v0.3.0

Who has control?

I need to fix export of the library, yarn run start locally works but production build fails with
ERROR in TypeError: Super expression must either be null or a function, not undefined
    at evalmachine.<anonymous>:104:18230
    at evalmachine.<anonymous>:104:18313
    at Object.n.exports (evalmachine.<anonymous>:104:19429)
    at s (evalmachine.<anonymous>:104:15848)
    at Object.n.exports (evalmachine.<anonymous>:104:40516)
    at s (evalmachine.<anonymous>:104:15848)
    at Object.n.exports (evalmachine.<anonymous>:104:24780)
    at s (evalmachine.<anonymous>:104:15848)
    at n.exports (evalmachine.<anonymous>:104:16648)
    at Object.<anonymous> (evalmachine.<anonymous>:104:16657)

Which can have multiple causes, i guess it occured during the webpack upgrade v3 -> v4 in the vote app repo

Which can have multiple causes, i guess it occured during the webpack upgrade v3 -> v4 in the vote app repo

Hmm that's unusual, worked well for me when I was working with it. Should I take a look at it?

@anshumanv thank but i know what it is, its the SSG run target: node that's killing it, i have a fix for it already but want to upgrade react to 16 first

@anshumanv thank but i know what it is, its the SSG run target: node that's killing it, i have a fix for it already but want to upgrade react to 16 first

Awesome, please let me know if I can help somewhere. :)

@montogeek I think it’s behind Tobias as he built the voting bits.

I think its on @skipjack you can see this on npm: https://www.npmjs.com/package/webpack.vote

@montogeek want to have another look ?

I think its on @skipjack you can see this on npm: https://www.npmjs.com/package/webpack.vote

You guys need me to cut / publish a new release? Also, maybe I can add one of your npm accounts to that package so you can publish yourself next time?

Its fine we can use github url now so no worries


2.0.1 htmlcoind Error: -daemon is not supported on this operating system

whenever I try to run htmlcoind on Windows 10, I get this error.
C:\apps\htmlcoind-2.0.1-win32\htmlcoind-2.0.1-win32>htmlcoind Error: -daemon is not supported on this operating system
I don't have this issue on Windows 7.
The command prompt usually is locked to this process and the window remains in task bar.
htmlcoind just gives this error and exits. Not sure if this is a new issue from Windows 10 update?
I tried running compatibility and changing it to Windows 8 and Windows 7. Neither did anything different.

taking htmlcoin.conf out of %appdata%\HTMLCOIN seems to have removed the behavior

Was daemon=1 specified in the htmlcoin.conf file?


[Extension] Allow users to use custom icons over sets?

Hi there,
I’ve published an icon set (https://marketplace.visualstudio.com/items?itemName=emmanuelbeziat.vscode-great-icons). I had a feature request by a user: https://github.com/EmmanuelBeziat/vscode-great-icons/issues/46
Basically, he asks me if it would be possible for him to define custom icons that would override my list. I’m giving it a real thought, because I think it would be a great improvement.
I now that it wouldn’t be possible to just define some custom file and folder, as the update process of vscode is to remove the old extension folder and create a new one.
I don’t see if there’s a way to use (contributes.configuration)[https://code.visualstudio.com/docs/extensionAPI/extension-points#_contributesconfiguration] and define custom files for some icons, and then kindda "import" theses adds in my script? At this point, I’m just wondering if this is possible in any way or not.
The user suggested this:

I guess another option is to have a piece of code that runs offline to merge the new and old versions. Is there any way to run code on installation, ala Squirrel?

I need confirmation about this, ’cause I didn’t find anything relevant on the doc: is this possible to run a script before the installation / update of a plugin?
Thanks for your thoughts :)

Extending or customizing an icon theme is currently not possible.
duplicate of #14662 or #11556


Error with S1C

Hi Tomer,
I have HASS 0.69.1 and broadlink 0.9.
I received this error:
File "/home/pi/homeassistant/custom_components/sensor/broadlink_s1c.py", line 98, in async_setup_platform
for i, sensor in enumerate(raw_data["sensors"]):
TypeError: 'NoneType' object is not subscriptable
I would appreciate your help.
Thanks in advance,
Gabriel

sorry for the delayed response,
is this issue still relevant?


ARROW-11935: [C++] Add push generator

A push generator has a producer end which pushes values to a queue, and a consumer end (the generator itself) which yields futures that receive the values pushed by the producer.

@westonpace I would welcome your input on this.

Note the generator could perhaps be made reentrant if there's some use for that.

Also, in regards to reentrancy.  I don't think there would be any advantage to doing so here because there is no backpressure / connection with the producer.

@westonpace I turned the API on its head so that PushGenerator is really a generator. Also I added comments and the ability to early-close the queue. Can you take a look again?

Travis-CI build: https://travis-ci.com/github/pitrou/arrow/builds/220827493

CI failure is unrelated, will merge.


feat: add support for NavigatorObservers

Description
Currently, it is not possible to use NavigatorObservers with duck_router. I mentioned this once in a previous issue, but did not pursue it.
However, this limitation stayed with me, as in other apps (where we do not use duck_router) we do rely quite heavily on these observers. So this PR is my attempt to support these observers and remove that limitation.
You'll notice that unlike the navigatorObservers property in MaterialApp, my solution requires a builder function to be added. So the callback is really meant to create new instances of the observers.
PS: I know this topic can become quite complex, they did not achieve a satisfying result yet in go_router either, but I believe we can get away with this solution for now.
Related Issues
Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

Hey there. I have recently left Onsi, and this package is in a bit of transition as a result. It will be transferred to my personal account. Please allow me to come back to it next week!

This will need a rebase, apologies (should be smooth though).
 My decision making around this functionality till now has indeed been based on experience using these in go_router and other Flutter routers: they take a lot of effort to work with as soon as you get into stateful territory. There's a lot of gotchas there. So I decided to actively not support them, I just don't think they API considered all the edge cases. Instead, we offer onNavigate. I get the want to still support this though, so I just have one question:
How can we prevent users of this API to assume that it will work great in all cases, when we know that to not be true? Your documentation in the PR already indicates it, there's a bunch of gotchas here.

Rebase done, and thanks for the feedback!
I'm afraid we will never truly be able to force the user to use it correctly, we can only point it out as much as possible in the documentation and the readme.
I've already stated in the documentation that observers should not be shared; do you think this is enough information, or should I add more info (link to example, stating they should be stateless...)? Also, it might be worth to repeat this information in the readme?

I've already added a section to the readme, does this ease your concerns?

@JaspervanRiet  I'm having some trouble getting past the pipeline checks (I didn't change anything in those files), could you maybe have a look?

Not sure where that came from but https://github.com/JaspervanRiet/duck_router/commit/233a5d6c53a764db413985f92bd4e7b6a4843e5b fixed it.


⚠ Bofrost.be has degraded performance

In ba6519a, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1200 ms

Resolved: Bofrost.be performance has improved in 0f0599f after 37 minutes.


sol-pin-mux: Add Intel Edison mux description

Also add rules to detect Intel Edison platform.
Tested and the platform was detect and the mux rules successfully applied!

As I told you last PR, that's intended, so the schema will allow it.

Merged


images: Add images.Overlay filter

This allows for constructs ala:
{{ $overlay := $img.Filter ($logo | images.Overlay 50 50 )}}

Or:
{{ $logoFilter := ($logo | images.Overlay 50 50 ) }}
{{ $overlay := $img | images.Filter $logoFilter }}

Which will overlay the logo in the top left corner (x=50, y=50) of $img.
Fixes #8057

There is one missing piece in this PR that needs to be done before I merge.
Looking at the above, I yet again get insecure about the argument order:
{{ $logoFilter := ($logo | images.Overlay 50 50 ) }}

Which is "pipe friendly" -- but I suspect it would read better as:
{{ $logoFilter := images.Overlay $logo 50 50 ) }}

As in "overlay logo at x=50 y=50" ... ?
/cc @onedrawingperday @moorereason @regisphilibert etc.

I'd go for the second option. Pipe is great, but not everyone uses it and I don't think it should necessary dictate the order of arguments.

I use the pipes syntaxall the time. It' reads better.
Well... dunno maybe flip a coin and see what happens? LOL
Heads for the 1st option
Tails for the 2nd

I'll go for the second option. Piping does not make much sense here.

Done. This is a feature that have a good usefulness/time spent ratio.
This is the docs, if anyone wants to protest about the signature:

Is it possible to add overlay logo to right bottom of an image ?

@adityatelange Yes, this is possible. Please use the forum (https://discourse.gohugo.io) for questions and troubleshooting. We prefer to use GitHub for verified bugs and vetted enhancements.

@adityatelange Yes, this is possible. Please use the forum (https://discourse.gohugo.io) for questions and troubleshooting. We prefer to use GitHub for verified bugs and vetted enhancements.

Thank you! I've figured it out though..
I'm using the following code with examples provided in docs https://gohugo.io/functions/images/#overlay
{{ $image := $image.Filter (images.Overlay $logo (sub $image.Width $logo.Width) (sub $image.Height $logo.Height) ) }}


Errors in the documentation

add_file() with just these two params is not defined. It requires the 3rd verbosity param.
// Put every log message in "everything.log":
loguru::add_file("everything.log", loguru::Append);

Here, loguru::INFO doesn't work but loguru::Verbosity_INFO does.
// Only log INFO, WARNING, ERROR and FATAL to "latest_readable.log":
loguru::add_file("latest_readable.log", loguru::Truncate, loguru::INFO);

Thanks for the library!

Thanks, fixed in 4d6b54b1787c1eb51d9719714b2eecc284ee787a!


SyntaxError: invalid syntax on setup

$ sudo sh virtualenv-burrito.sh
throws:
File "", line 1
import platform;print ".".join(platform.python_version().split(".")[:2])

First off, this is not meant to run under sudo. It's meant to run in a standard user account. Are you intentionally trying to install this for the root user?
Second, this is a BASH script, but you're running it under sh instead of bash. Probably, sh doesn't understand the syntax.

Hi, this is not a bash/sh related issue. The element
import platform;print ".".join(platform.python_version().split(".")[:2])
is included in the line:
pyver=$(python -c 'import platform;print ".".join(platform.python_version().split(".")[:2])')
Hence the error comes form the python environemnt. More specifically from the print impementation in the Python 3. In Python 3, print is a function, you need to call it like print(".".join(platform.python_version().split(".")[:2])).
@sselvaggi should check additionally if there was Python 2 or 3 installed.