wr.s3.read_excel() fails in AWS Glue Python Shell Job

Describe the bug
I have a simple script which reads a single sheet spreadsheet in .xlsx format from S3 into a dataframe. I'm running it using the latest version of Data Wrangler, 2.9.0.
The script runs fine when I run it in my IDE (Cloud9). But, when I run it as a Glue Python Shell Job it fails (with .whl file for Data Wrangler 2.9.0 configured in my Glue Job).
Error message is, ImportError: Missing optional dependency 'xlrd'. Install xlrd >= 1.0.0 for Excel support Use pip or conda to install xlrd.
Note, the Glue Job works fine if I regress to a wheel file for an earlier version of Data Wrangler, for example 2.3.0.
Environment
This is the output from pip freeze in my IDE (Cloud9). This is not necessarily the same as the environment that my Glue Job runs in.
asn1crypto==1.4.0
astroid==2.3.0
awswrangler==2.9.0
backcall==0.2.0
beautifulsoup4==4.9.3
boto3==1.17.86
botocore==1.20.45
certifi==2021.5.30
chardet==4.0.0
decorator==5.0.5
Django==2.0.2
et-xmlfile==1.1.0
git-remote-codecommit==1.15.1
idna==2.10
ikp3db==1.4.1
importlib-metadata==3.10.0
ipython==7.22.0
ipython-genutils==0.2.0
isort==4.3.21
jedi==0.18.0
jmespath==0.10.0
lazy-object-proxy==1.6.0
lxml==4.6.3
mccabe==0.6.1
numpy==1.20.3
openpyxl==3.0.7
pandas==1.2.4
parso==0.8.2
pbr==5.5.1
pexpect==4.8.0
pg8000==1.19.5
pickleshare==0.7.5
prompt-toolkit==3.0.18
ptyprocess==0.7.0
pyarrow==4.0.1
Pygments==2.8.1
pylint==2.4.4
pylint-django==2.3.0
pylint-flask==0.6
pylint-plugin-utils==0.6
PyMySQL==1.0.2
python-dateutil==2.8.1
pytz==2021.1
redshift-connector==2.0.881
requests==2.25.1
s3transfer==0.4.2
scramp==1.4.0
six==1.15.0
soupsieve==2.2.1
stevedore==3.3.0
traitlets==5.0.5
typed-ast==1.4.2
typing-extensions==<IP_ADDRESS>
urllib3==1.26.4
virtualenv==16.2.0
virtualenv-clone==0.5.4
virtualenvwrapper==4.8.4
wcwidth==0.2.5
wrapt==1.12.1
xlrd==2.0.1
zipp==3.4.1

To Reproduce

In Excel, create a simple, table like spreadsheet with a singe sheet. Save it as demo_excel.xlsx.
Upload the spreadsheet to <your bucket> in S3.
Run the script below as a Glue Python Shell Job, with Python library path configured to use, awswrangler-2.9.0-py3-none-any.whl. The job is confgured to use Python 3, and Glue Version 1.0.
The job fails with error message ImportError: Missing optional dependency 'xlrd'. Install xlrd >= 1.0.0 for Excel support Use pip or conda to install xlrd.

Script,
import awswrangler as wr
import pandas as pd
import boto3
import io

data_bucket = 'data-976763253578'
xlsx_object = 'demo_excel.xlsx'

xlsx_s3_df = wr.s3.read_excel('s3://' + data_bucket + '/' + xlsx_object)
print(xlsx_s3_df.head())

Could you please try:
xlsx_s3_df = wr.s3.read_excel('s3://' + data_bucket + '/' + xlsx_object, engine="openpyxl")

In short, it seems that pandas is not recognizing the file as a "xlsx" for some reason and defaults to the "xlrd" engine instead which is used for "xls" files. Previous versions had the engine hardcoded thus the difference.
Your Cloud9 instance already has "xlrd" installed which explains the difference in behavior.
My advice would be to specify the appropriate engine explicitly as much as possible to avoid relying on Pandas.

I tried adding the engine="openpyxl" parm that you advised. It has fixed my problem.
Many thanks. If OK with you, I'm happy to close this issue.


Mining not working

I've enabled mining in FRANZ, but it doesn't show any calculations per sercond.

Thanks for your submission, to assist us with dealing with this as promptly as possible please fully apply the issue template:
https://raw.githubusercontent.com/meetfranz/franz/v5.0.0-beta.14/.github/ISSUE_TEMPLATE.md
or the feature request template:
https://raw.githubusercontent.com/meetfranz/franz/v5.0.0-beta.14/.github/FEATURE_PROPOSAL_TEMPLATE.md

Ok...
In 5.14 there is no option for Mining anymore.


FIX: Background colour of textfield in survey screen is not updating.

Issue: https://github.com/apptentive/apptentive-ios/issues/263

Also sorry for the delay in responding. One more bit of busywork, if you haven't already filled it out, please fill out our [Contributor License Agreement])https://docs.google.com/a/apptentive.com/spreadsheet/viewform?formkey=dDhMaXJKQnRoX0dRMzZNYnp5bk1Sbmc6MQ#gid=0).
Thanks again for your contribution!

Also sorry for the delay in responding. One more bit of busywork, if you haven't already filled it out, please fill out our [Contributor License Agreement])https://docs.google.com/a/apptentive.com/spreadsheet/viewform?formkey=dDhMaXJKQnRoX0dRMzZNYnp5bk1Sbmc6MQ#gid=0).
Thanks again for your contribution!

Thank you for considering :)


[nginx ingress controller] config reload question

Im getting errors in logs like upstream default-someingressname-8080 does not have any active endpoints. Using default backend. Such ingresses was deleted long ago, but nginx din't know it. Is there way to reload config by hands?

External loadbalancer ip is changed every time then ingress pod restarts (example: cluster update). In case of "gce ingress" this can be fixed by writing annotation something like "keep ip permanent" (sorry, didn't remember configuration name exactly). Is there something similar in nginx ingress?

Thank you.

I also get weird messages in logs with not existing endpoints. @aledbf?


Pry-byebug 2.0 is not compatible with pry-remote

After updating to jazz_fingers, I got the following error while trying to work with pry-remote:
[pry-remote] Waiting for client on druby://<IP_ADDRESS>:9876
[pry-remote] Client received, starting remote session

From<EMAIL_ADDRESS>@ line 191 PryRemote::Server#teardown:

    182: def teardown
    183:   # Reset output streams
    184:   $stdout = @old_stdout
    185:   $stderr = @old_stderr
    186:
    187:   # Reset config
    188:   Pry.config.pager = @old_pager
    189:
    190:   # Reset sysem
 => 191:   Pry.config.system = @old_system
    192:
    193:   puts "[pry-remote] Remote session terminated"
    194:
    195:   begin
    196:     @client.kill
    197:   rescue DRb::DRbConnError
    198:     puts "[pry-remote] Continuing to stop service"
    199:   ensure
    200:     puts "[pry-remote] Ensure stop service"
    201:     DRb.stop_service
    202:   end
    203: end

From what I have found, this is the issue of the lack of compatibility between pry-byebug 2.0 and pry-remote - see issue deivid-rodriguez/pry-byebug#33 and Mon-Ouie/pry-remote#58.
Pry-remote works fine with pry-byebug 1.3.3, so maybe for now (until the author of pry-byebug solves the issue) the dependency can be weakend?

Ok @matfiz . I'm following both repos now to get updated on that.
Your proposal seems reasonable since pry-byebug 2.0.0 changelog says:

Improvements:

Compatibility with byebug 3
Now pry starts at the first line after binding.pry, not at binding.pry

Bugfixes:

'continue' doesn't finish pry instance (issue #13)

@matfiz Can you try version 1.0.0?
Thanks!

I confirm that with pry-byebut 1.3.3 it works fine :smiley:
Thanks for the prompt response to this issue :+1:


UI development wip

started work on the GUI with the redesign interface issue as a rough outline

wasn't sure if this was being submitted to be merged with main; reopen the pull request if you think it's ready and i'll have a look


Importing page seeds fails if any attached file path contains a white space character

Expected behavior
Running the comfy:cms_seeds:import rake task (e.g. rails 'comfy:cms_seeds:import[site-identifier]' should import all the seed layout, pages, files, etc., including any file that has correctly escaped special characters (incl. white space).
Actual behavior
If a page seed has an attached file path containing white space characters (esp. space), whether the special characters are escaped or not, it will fail with a Errno::ENOENT: No such file or directory @ rb_sysopen.
Steps to reproduce
If you have a db/cms_seeds/site-name/pages/some-page/content.html seed file that includes an attached file, for example:
[file slide_image]
file\ name.jpg

If the attached file exists, attempt to import it by running rails 'comfy:cms_seeds:import[site-name]', you'll get the following error:
Errno::ENOENT: No such file or directory @ rb_sysopen - /path/to/project/db/cms_seeds/site-name/pages/some-page/file

Note the file path is truncated at the white space character.
System configuration
Rails version: 5.2.0
CMS version: 2.0.16
Ruby version: 2.5.1p57

I believe that frag_content.split.collect should be frag_content.split("\n").collect in lib/comfortable_mexican_sofa/seeds/page/importer.rb:190.


Keeping track of issues filed at LLM+RL projects

This issue is just a place to keep track of issues filed at other github projects or HuggingFace hubs to ask for documentation

Mistral: https://github.com/mistralai/mistral-src/issues/9

Xwin-LM: https://github.com/Xwin-LM/Xwin-LM/issues/14

RedPajama: https://huggingface.co/togethercomputer/RedPajama-INCITE-7B-Instruct/discussions/9#64c7c30a527d763655434618
Now has an answer:

The instruction-tuned model was trained on instructions from the P3 and Natural Instructions datasets, which were decontaminated against HELM (you can find more details about the decontamination strategy in this blog bost).

You can find the resulting decontaminated dataset used to train the instruction tuned model here: https://huggingface.co/datasets/togethercomputer/RedPajama-Data-Instruct -- the metadata here contains a source field pointing to the task / dataset where the instance comes from.

Solar 70B: https://huggingface.co/upstage/SOLAR-0-70b-16bit/discussions/15

Boomer 1B: https://huggingface.co/budecosystem/boomer-1b/discussions/1#652be6c22aa5b27c77eab9c7

GenZ-70B: https://huggingface.co/budecosystem/genz-70b/discussions/1#6533ffd624c306369652e7d1

Erasmian Language Model (ELM): https://github.com/Joaoffg/ELM/issues/1 (and see #65)

Yi 34B Chat: https://huggingface.co/01-ai/Yi-34B-Chat/discussions/7

Gemma: https://huggingface.co/google/gemma-7b-it/discussions/64

Command R+: https://huggingface.co/CohereForAI/c4ai-command-r-v01/discussions/43#661fb1d3a3ce1d628c28aaa6

Nanbeige Chat: https://huggingface.co/Nanbeige/Nanbeige2-8B-Chat/discussions/2#6621e15a4d17641cf788cbd5


Powershell - Debug adapter process has terminated unexpectedly

VSCode Version: 0.10.11 , shell 0.35.6, Renderer 45.0.2454.85
OS Version: WIN10 Pro

Steps to Reproduce:

Have a working setup with VSCode 0.10.10 and powershell debug 0.5.0
Upgrade VSCode to 0.10.11.
Debug fail: The error says Debug adapter process has terminated unexpectedly
Reinstall VSCode 0.10.10 and powershell debug 0.50, issue remains

DebugAdapter.log.txt

@laurijsd The attached log shows that this is an error in your launch config. You are passing the string "-Verbose" where an array of strings is expected:

In VS Code this error is actually flagged as a problem:

This error occurs in VS Code 0.10.10 too, so this is not a regression:

I suggest to file a bug against the PowerShell extension because they should protect against these JSON parsing errors.

I had the same issue. But in my case this was problem with SVG Viewer extension. So, I've remove it, then also removed C# extension also.
After that, re-installed C# extension, then VS Code downloaded and installed .NET Core Debugger again.
And now, all are good,  even though, I've re-installed SVG Viewer extension.
[Note: Check Developer Console from 'Help' menu for your particular problem]


Honor default locale instead of hardcoded :en

At the fresh master (dfb0762) I've noticed here:
def with_locale(tmp_locale = nil)
  current_locale = Faker::Config.own_locale
  Faker::Config.locale = tmp_locale
  I18n.with_locale(tmp_locale) { yield }
ensure
  Faker::Config.locale = current_locale
end

that if I don't have :en locale in I18n.available_locales, enabled by Faker or Rails I18n.enforce_available_locales fails the app with:
$ ~/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/i18n-0.8.6/lib/i18n.rb:284:in `enforce_available_locales!': :en is not a valid locale (I18n::InvalidLocale)

Another simle fallback to hardcoded :en locale you may find here:
def translate(*args)
  opts = args.last.is_a?(Hash) ? args.pop : {}
  opts[:locale] ||= Faker::Config.locale
  opts[:raise] = true
  I18n.translate(*(args.push(opts)))
rescue I18n::MissingTranslationData
  opts = args.last.is_a?(Hash) ? args.pop : {}
  opts[:locale] = :en

  # Super-simple fallback -- fallback to en if the
  # translation was missing.  If the translation isn't
  # in en either, then it will raise again.
  I18n.translate(*(args.push(opts)))
end

I'm just curious, does faker intentionally has hardcoded :en locale scope in the modules? For example Faker::Internet module:
module Faker
  class Internet < Base
    class << self

      ...

      def user_name(specifier = nil, separators = %w(. _))
        with_locale(:en) do
          ...
        end
      end

      ...

      def domain_name
        with_locale(:en) { [Char.prepare(domain_word), domain_suffix].join('.') }
      end

      ...
    end
  end
end

I'd expect faker honors I18n default locale instead of hardcoded :en.
I'm OK to create PR with appropriate changes in case if community's sentiments the same.

If :en is not part of the available locales, faker triggers I18n to throw this:
i18n-0.8.6/lib/i18n.rb:284:in `enforce_available_locales!': :en is not a valid locale (I18n::InvalidLocale)

Configuration to recreate in a rails app:
class Application < Rails::Application
   config.i18n.available_locales = [:nb, :da, :es]
end

After adding factory_girl-rails + faker to the Gemfile, this is thrown when starting the console.
I do think the best would be to honor I18n.locale if possible.
If not perhaps :en should be pushed into available_locales?

My case is related to the next available locales – :'en-GB' (default locale), :'en-ZA'. But generally, I can imagine app that is intended to be exclusively in non-EN locales.

I do think the best would be to honor I18n.locale if possible.
That what I wanted to hear ^_^ So I'll try to find all possible places where faker has explicit :en locale declaration.

To answer your question about Internet, it uses :en specifically in order to solve a bug with not generating valid domain names and email addresses when a non-latin language is the default locale.


Admin page table goes beyond the browser bounds.

After performing a search in the admin page, the grid seems to go beyond the bottom of the browser page. This makes the last value of the table unreadable since more than half is below visibility.

Admin dashboard passed using Chrome.  FF17 the dashboard loaded blank.

Side issue:  Started up FF17 to check the issue with the Admin Grid, but it stays on "Loading Meridian" (localhost:3000).  However, when I checkout an older branch, I can load Meridian properly, somewhere there is a code delta between my older branch and this one that may be causing this blank screen issue.
I will work with whomever to pinpoint the problem more clearly.

Passed in FireFox 34. Chrome is not loading at all for me.

Unsure Tiff's local setup that explains her inability to load in Chrome.  For my setup, the code fix displays the grid correctly in both Chrome and FF37.  There is a code change elsewhere that does not allow me to load Meridian in FF17.

Update:  I am able to load meridian in FF17, but get these errors when trying to load the /admin screen in FF17, otherwise, works except FF17


Override return types of hooks

Currently, when the response type schema is not present for endpoints(Open API), the return type of the hooks is "any". It would be nice if we were able to override them.
Example:
const { data } = useFoo<MyType>();

PS: Thank you for maintaining this library! I use it in our day-to-day, and it has saved me a ton of hours.

Hi @nopitown,
I'm very pleased to hear that you use my library and that it's been helpful for your development. Thank you for your feedback and for your support through GitHub Sponsors!
I've just released v0.5.1. Now you can override the return type and errors.
const { data, error } = useFoo<[], MyType, MyError>();

An update:
In v0.5.2 we have changed the types, no longer using Omit and changed the order of the Type parameters.
In the following example, data's type will be a union of undefined and MyType.
const { data, error } = useFoo<MyType, MyError>();


Fix return type with prefix "?"

Fix return types prefixed with "?", example: "?string" or "string|null" in accessor methods.

Thanks! Merged


Failed to extract files from KutoolsforExcelSetup.msi

Describe the bug
Failed to extract files from KutoolsforExcelSetup.msi
To Reproduce
Steps to reproduce the behavior:

Download the KutoolsforExcelSetup.msi
Select All and Extract files from KutoolsforExcelSetup.msi
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: Win7 x64

Thanks for reporting this! I'll take a look. Usually these .cab file issues are either super trivial or super hard. 🤞 for the former.

Thanks for reporting this! I'll take a look. Usually these .cab file issues are either super trivial or super hard. 🤞 for the former.

@activescott
This doesn't look like it uses a .cab file. Here it stopped on a file for the chinese translation. Maybe this has to do something with encoding.

Based on the call stack it was almost certainly using a CAB originally. If the latest MSI doesn't have a CAB maybe it has changed since this was reported


Alarm control panel stopped working in 0.103

Just updated to the 0.103.b0 and realized that the alarm control panel is not working. The only thing that I see in the logs is:
2019-12-06 11:32:39 ERROR (MainThread) [homeassistant.components.alarm_control_panel] Setup of platform alexa_media is taking longer than 60 seconds. Startup will proceed without waiting any longer.

The media player works fine.
BTW, there was a similar error with alarm panel with another custom component (Aarlo) that had to be fixed.

I added the following to alarm_control_panel.py and it fixes the error.
    @property
    def supported_features(self) -> int:
        """Return the list of supported features."""
        """Make this non-dynamic later..."""
        try:
            c = __import__("homeassistant.components.alarm_control_panel.const",fromlist=['SUPPORT_ALARM_ARM_HOME', 'SUPPORT_ALARM_ARM_AWAY', 'SUPPORT_ALARM_ARM_NIGHT', 'SUPPORT_ALARM_TRIGGER'])
            _LOGGER.debug('supported: ' + str(c.SUPPORT_ALARM_ARM_HOME | c.SUPPORT_ALARM_ARM_AWAY | c.SUPPORT_ALARM_ARM_NIGHT | c.SUPPORT_ALARM_TRIGGER))
            return c.SUPPORT_ALARM_ARM_HOME | c.SUPPORT_ALARM_ARM_AWAY | c.SUPPORT_ALARM_ARM_NIGHT | c.SUPPORT_ALARM_TRIGGER
        except ModuleNotFoundError:
            _LOGGER.debug('not supported')
            return 0

Here's the updated file. I can create a PR if that is the best approach.

Thanks. Probably need to just do SUPPORT_ALARM_ARM_AWAY. It's a fairly simple fix.

I'm getting the same error
2019-12-12 10:36:28 ERROR (MainThread) [homeassistant.components.alarm_control_panel] Setup of platform alexa_media is taking longer than 60 seconds. Startup will proceed without waiting any longer.
I'm not using the alarm_control_panel however, it's just making my HA take an extra minute to start currently.


Xml comments are ignored in derived class

When using a complex object as a request parameter in GET the xml comments are ignored from the derived class.

Just wondering what type of comments are ignored. Does it look similar to this #499

I don't have my classes in a dependent assembly. It's the description I'm missing.

@PaulDotNet updated to latest version but still the same problem, Swashbuckle is ignoring my  <summary> notation in a model base class.

@k-ljung - can you please provide an exact set of repro steps (ideally starting from an empty project) so I can understand your exact issue.

Closing due to inactivity


react-query-devtools is missing style props from v4

Describe the bug
The react-query-devtools package is missing the styling props for customizing the devtools buttons and panels. I noticed these options were avaialble in v4 of react-query and also in the current version of tanstack/react-router. These props were extremely useful to integrate them into existing devtools UI for any app. Specifically, I'm referring to the panelProps, closeButtonProps, and toggleButtonProps on the ReactQueryDevtools component.
Is there a reason this was removed, and can adding them back be considered? It seems like it was meant to be a part of the Tanstack ecosystem since the router library had the exact same styling options as v4 did, but I'm not sure why it was removed for v5.
Your minimal, reproducible example
N/A
Steps to reproduce
Unable to customize styling for react-query-devtools buttons or panels.
Expected behavior
Should be able to customize styling for react-query-devtools using panelProps, closeButtonProps, and toggleButtonProps on the ReactQueryDevtools component.
How often does this bug happen?
None
Screenshots or Videos
No response
Platform
All
Tanstack Query adapter
None
TanStack Query version
5.51.9
TypeScript version
No response
Additional context
No response

Hello! Yeah the new devtools panel doesn't support passing inline styles. The reason for this being that different frameworks have different semantics for defining inline styles. For example fontSize in react as compared to font-size in solid-js. We however now prefix most of our DOM elements with classes prefixed with tsqd-*. You can target them with css and add your custom styles to each individual DOM element.


old combat mechanics

Info

Server version: spigot 1.13.2
OldCombatMechanics version: 1.6.6
Server log file:  https://pastebin.com/3h5mnNXm
OldCombatMechanics config file:  https://pastebin.com/VpJy6Nu8

Problem Description

Getting errors when players join the server

Please update Old combat mechanics. Have a look here and download the jar please.

this seems to fix the issue, thanks!

@gvlfm78 @Halfhand1 I guess this can be closed.


fix: update about section

Added a more descriptive and generic block of text to an "About" section of the blog.

LGTM!


Vim82

Merging upstream to MacVim. Most conflicts were because of comment styles.
Notable changes:

FEAT_TEXT_PROP renamed to FEAT_PROP_POPUP
Old platforms were removed

Merge remote-tracking branch 'vim/master'
Conflicts:
LICENSE
README.md
runtime/doc/gui.txt
runtime/doc/options.txt
runtime/doc/starting.txt
src/Makefile
src/beval.h
src/channel.c
src/edit.c
src/evalfunc.c
src/ex_getln.c
src/feature.h
src/gui.c
src/gui.h
src/gui_beval.c
src/keymap.h
src/main.c
src/mbyte.c
src/menu.c
src/mouse.c
src/option.c
src/os_unix.c
src/proto.h
src/screen.c
src/testdir/test_timers.vim
src/vim.h
closes #994

Really wish this pr could be merged.

Sorry for the delay. There were some issues with this PR (lingering merge markers, and README.md not copied over), and usually I do prefer to go through all the commits myself to make sure to hopefully catch gotchas. I merged from Vim upstream, and hence closing this PR. Also made https://github.com/macvim-dev/macvim/wiki/Merging-from-upstream-Vim which helps explain more of the upstream merging process.


Github backend requires full organization account access

Describe the bug
We want to keep a static site project in a private github repo inside our organization and use netlify-cms for content management. However, the OAuth backend requires full read/write access to all repos of that organization and a more fine-grained selection of grants is not possible.
To Reproduce

Create a new private repo inside a github organization with restrictive access (i.e. organization settings → third-party access → policy = access restricted)
Set up a Hugo/GatsbyJS/... site with netlify-cms in that repo as described in the guide
Configure the github backend for netlify-cms
Deploy site on Netlify
Create an OAuth2 app for Github authentication as described here
Visit the site's netlify-cms backend and click Login with Github

Expected behavior

One would expect that it's possible to restrict netlify-cms access to only the relevant repositories (just like it's possible when setting up a Netlify site from Github).
Screenshots

Applicable Versions:

Netlify CMS version: netlify-cms-app 2.12.22 / netlify-cms-core 2.30.6
Git provider: GitHub
Browser/OS: Chromium Version 85.0.4183.102 (Official Build) Arch Linux (64-bit)

Node.JS version: node v14.10.1 / npm 6.14.3

CMS configuration
backend:
  name: github
  repo: my-org/my-repo
  branch: master

media_folder: static/img
public_folder: /img

collections:
  - name: 'blog'
    label: 'Blog'
    folder: 'content/blog'
    create: true
    slug: 'index'
    media_folder: ''
    public_folder: ''
    path: '{{title}}/index'
    editor:
      preview: false
    fields:
      - { label: 'Title', name: 'title', widget: 'string' }
      - { label: 'Publish Date', name: 'date', widget: 'datetime' }
      - { label: 'Description', name: 'description', widget: 'string' }
      - { label: 'Body', name: 'body', widget: 'markdown' }

Additional context
I am aware that this issue might not be directly related to netlify-cms but to either the Netlify API or the Github OAuth API. Please let me know if I should report this upstream instead, thanks!
Our current workaround is creating a separate Github organization with a single private repo and granting the Netlify Auth Service full access to this organization.

Thanks @cbix, we are currently limited by the available scopes for OAuth apps provided by GitHub:
https://developer.github.com/apps/building-oauth-apps/understanding-scopes-for-oauth-apps/#available-scopes
It would be amazing to limit the access to a specific repo, but that's not available at the moment
if GitHub supports has some more information regarding this I would love to know about it.

Thanks @erezrokah for the quick reply. I had the same issue when evaluating forestry.io so I already considered it an issue with GitHub's API, however with their server-based solution there's the possibility of using deploy keys, which is probably not possible over the web-based API that netlify-cms uses...
Anyway, it's good to hear that you are aware of this and it's not the desired way to go ;)

Got this response from the GitHub Developer Support:

Thank you for contacting GitHub Support.
Providing more granular OAuth scopes is already the biggest blip on the API team's radar and it's something we'd love to do. However, we can't make promises about if and when this might be available -- the API team is rolling out additional scopes as they are completed.
The best way to keep track of changes is to follow the API blog.
https://developer.github.com/changes/
Improving this situation about scopes is something the team has been working on. You might want to check out the recently-announced GitHub Apps (formerly Integrations) feature:
https://docs.github.com/developers/apps
GitHub Apps allow per-repository access and more finely-grained scopes. If you have any other follow-up questions about the GitHub API, OAuth Apps, or GitHub Apps, we recommend checking out our new GitHub API Development and Support Board:
https://github.community/c/github-api-development-and-support/37
Please let us know if you need any further help.
[...]

I guess implementing a server-side GitHub app that provides API tokens to the client-side netlify-cms app means a significant amount of work. Also it seems like GitHub apps behave like a separate user so a big open question is if and how it's possible to a) authenticate a Github user towards the app and b) let the netlify-cms frontend create commits in the name of the authenticated user, without going through any server-side code and while still keeping the (very useful) possibility to restrict some users to only creating pull requests on the repo.

Great follow up @cbix, very good to know GitHub is moving on this.
I think there is a bigger pain here for any programmatic usage of GitHub's API since personal access tokens are also not limited to a specific repo, and creating a GitHub app just to have some GitHub automation is not ideal.

Netlify CMS still require full access to all repos, its not nice.

This is a huge issue, obviously. I can't grant Netlify access to every organization I'm a part of.

I can't grant Netlify access to every organization I'm a part of.

Hi @austinschrader, to clarify Netlify doesn't require access to every organization.
Netlify CMS (which acts as a GitHub client) requires repo access since it modifies repo files to manage content. That access can't be scoped to a specific repo due to the limitations specific https://github.com/netlify/netlify-cms/issues/4329#issuecomment-694826032 (those are on GitHub's side).
We'd be happy to change the CMS implementation once GitHub supports better scoping.

@erezrokah I saw few web apps asking for individual repo access (request access for repo on project basis).

@erezrokah I saw few web apps asking for individual repo access (request access for repo on project basis).

Hi @prakis, can you share those?

Ya sure.
https://Railway.app
Render.com
Tina.io
These are some apps which use only selected repositories.

https://Railway.app

I tested this one and it installs a GitHub application on your organization:

GitHub apps do provide better granularity for permissions, but I don't think they can be used in a headless environment like the CMS. See https://docs.github.com/en/developers/apps/getting-started-with-apps/differences-between-github-apps-and-oauth-apps

I can't trigger Netlify CMS to ask my github org for permission. We cannot publish articles but we can access the netlify CMS admin page no issue. Nothing is prompting for access in my Org settings

Still a problem for me ; I cannot let my private repositories accessed at Netlify login time, it would be too problematic in my case.
I wonder if offering an alternative with PAT (personal access tokens) which can now be restricted to specific repositories (https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/creating-a-personal-access-token) if I understand correctly, could be a nice solution.
At the moment, though, I cannot use Netlify on some important projects because of this limitation.

Until we get a working solution with PATs, it seems creating a new github account, and essentially using it as a "machine user":
https://docs.github.com/en/authentication/connecting-to-github-with-ssh/managing-deploy-keys#machine-users
is at least a workable option. Create a new user and then only invite that user to cms repos it should be able to access, and create the oauth credentials under that user as well-

Is there a reason we can't use GitHub Apps for this?

Until we get a working solution with PATs, it seems creating a new github account, and essentially using it as a "machine user":

Where is this tracked/worked on? In theory one could also use this to setup a login-system without people having to know about github at all by hiding tokens/config behind htaccess?

There is no way I could grant anything access to all my repos.   Moving on...

Is there a reason we can't use GitHub Apps for this? Edit: See https://github.com/decaporg/decap-cms/issues/7108

Should be fully possible, I solved it with a custom auth backend which uses a GitHub app and assigned it only to a specific repo. Works like a charm so I don't see why it can't be done for the built-in one with a few tweaks.

@PalmEmanuel would you be willing to give it a go with a pull request? Or maybe if you can share your custom setup, so we can take notes. Thanks!

@PalmEmanuel would you be willing to give it a go with a pull request? Or maybe if you can share your custom setup, so we can take notes. Thanks!

For sure! It's open source at https://github.com/PalmEmanuel/decap-github-proxy and built with an Azure Function running PowerShell. No black magic really, just two endpoints following the Decap CMS custom auth proxy documentation. Have a look and feel free to reach out if there are any questions!


Getting error open GHOpenPR on litee/gh.nvim latest

I'm getting this error and am not sure what might cause it. I use the default setup() and tried to set the icon_set options to nil or "" but that didn't help:
Error executing vim.schedule lua callback: ...ker/start/litee.nvim/lua/litee/lib/tree/marshal/init.lua:66: attempt to index upvalue 'icon_set'
 (a nil value)
stack traceback:
        ...ker/start/litee.nvim/lua/litee/lib/tree/marshal/init.lua:66: in function 'marshal_node'
        ...ker/start/litee.nvim/lua/litee/lib/tree/marshal/init.lua:153: in function '_marshal_tree'
        ...pack/packer/start/litee.nvim/lua/litee/lib/tree/init.lua:306: in function 'write_tree'
        ...pack/packer/start/litee.nvim/lua/litee/lib/tree/init.lua:315: in function 'write_tree_no_guide_leaf'
        ...vim/site/pack/packer/start/gh.nvim/lua/litee/gh/init.lua:53: in function 'pre'
        ...ack/packer/start/litee.nvim/lua/litee/lib/panel/init.lua:129: in function 'toggle_panel'
        ...e/pack/packer/start/gh.nvim/lua/litee/gh/pr/handlers.lua:154: in function 'ui_handler'
        ...e/pack/packer/start/gh.nvim/lua/litee/gh/pr/handlers.lua:179: in function ''
        vim/_editor.lua: in function <vim/_editor.lua:0>

@marcomayer if you specify "default" does it work?

@marcomayer if you specify "default" does it work?

yes now it does and oddly I can't reproduce it anymore. Sorry, probably was some local config-cache issue!

Cool :)

Some issue... but using icon_set = "nerd" could not resolve this problem...
⣾ successfully fetched all PR state.
Error executing vim.schedule lua callback: ...ker/start/litee.nvim/lua/litee/lib/tree/marshal/init.lua:66: attempt to in
dex upvalue 'icon_set' (a nil value)
stack traceback:
        ...ker/start/litee.nvim/lua/litee/lib/tree/marshal/init.lua:66: in function 'marshal_node'
        ...ker/start/litee.nvim/lua/litee/lib/tree/marshal/init.lua:153: in function '_marshal_tree'
        ...pack/packer/start/litee.nvim/lua/litee/lib/tree/init.lua:306: in function 'write_tree'
        ...pack/packer/start/litee.nvim/lua/litee/lib/tree/init.lua:315: in function 'write_tree_no_guide_leaf'
        .../nvim/site/pack/packer/opt/gh.nvim/lua/litee/gh/init.lua:53: in function 'pre'
        ...ack/packer/start/litee.nvim/lua/litee/lib/panel/init.lua:129: in function 'toggle_panel'
        ...ite/pack/packer/opt/gh.nvim/lua/litee/gh/pr/handlers.lua:154: in function 'ui_handler'
        ...ite/pack/packer/opt/gh.nvim/lua/litee/gh/pr/handlers.lua:179: in function ''
        vim/_editor.lua: in function <vim/_editor.lua:0>

@ofseed you have the latest version of both "litee.nvim" and "gh.nvim" right?

litee.nvim and gh.nvim are what I just installed today, and the error occurs when I run GHOpenPR for the first time

@ofseed can you show me your config for both litee and gh.nvim? Ill try to reproduce

I separate the configuration file of every plugin into an individual file. This is the configuration file of gh.nvim, and this is how I call it. As you can see I copied almost the entire default configurationin the README.md.

@ofseed hey there, I just reproduced your config. I can't seem to reproduce the issue on my end.
I changed my config to this:
require('litee.lib').setup()
require('litee.gh').setup({
    icon_set = "nerd",
})

and opened a PR, nerd icons rendered fine with no error splat.
If you are using Packer, it does some module caching, maybe something is odd with this

@ofseed For me, it was probably a cache issue, I did PackerCompile a couple of times, reinstalled the packages, and then the issue was gone. Also might want to try to set
require("litee.lib").setup({icon_set = "nerd"}
explicitly.

I tried everything in this thread but nothing worked.
For some reason, litee.lib is not set up when gh.nvim is loaded and so icon_set is not set and is nil.
Couldn't figure out why, maybe it's a but with packer.
Anyway, what I did was load the config lazily by doing this:
  use({
    "ldelossa/gh.nvim",
    cmd = { "GHOpenPR" },
    config = function()
      require("dlvhdr.gh-nvim")
    end,
    requires = { { "ldelossa/litee.nvim" } },
  })

I also had to remove the pcall calls and do a straight up require.
With these 2 things - it works.

Seems like folke/noice was the culprit.
It probably overrides vim.notify and screws things up.


2.2.2 release seems not correct

Hi there,
on the tags page there is a release listed as 2.2.2, however the code that it contains appears to not be up to date (e.g. the the PR: https://github.com/jazzband/django-auditlog/pull/420 is not included).
Am i missing something, should this release be re-released ? Sorry if i'm missing something obvious.
thanks and regards

Hi,
Yes, the latest release is 2.2.2 and the code in master is not released yet. we are preparing a major release V3.
Here is the V2 branch


Reflection: use std::strtol instead of std::atoi

What's in this pull request?

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

Address post-commit review comments from Harlan.  Use std::strtol as it can
deal with hex and octal inputs.

We don't care about hex and octal digits here, the code above makes sure the digits are only 0...9.  This code is parsing mangled names.

Thats what I thought to (which is why I opted to use atoi).  @harlanhaskins asked for this change though.

I suggested it before seeing the validation beforehand. I suppose atoi is alright, but the lack of failure reporting in atoi raises a red flag

@harlanhaskins okay, so whats the plan on this?  Are we happy with the code as is and this can be abandoned or do we want to pursue some changes here?

I'm hesitantly okay with this now. Sorry for stirring the pot.

I see the benefit in using strtol as long as we actually check the error code.

@gribozavr updated with the error handling, but Im really not sure how much it buys us.  I did also remove an allocation, which can be nice, but, really, Im not convinced that it really is an improvement.  Note that it previously didnt have any error handling either.

@swift-ci Please test and merge


Enable GitHub merge queue

Enable the Github merge queue. Merge queues prevent semantic conflicts, thus providing an additional safeguard against breaking the main branch.

Woah, didn't know about this! Absolutely.
Looks like we need to update our CI to support it first:

Feel free to put up a PR. If not, I'll take a stab at this at some point.


Workspaceone: Validate config early on the start-up

/cc @kylesmartin

I think this makes sense for sure! @kylesmartin thoughts?


[Story]: Hide metrics when project is kserve enabled

Goal
Improve hiding of metrics features not supported by kserve when a project is kserve enabled.
Dependency issue
No dependencies
Itemized goals

Hide "Install TrustyAI" widget when project is kserve enabled
Hide "Show server metrics" kebab item on project detail page when project is kserve enabled
Model server component on project detail page should not show links to model metrics when project is kserve enabled
Global model serving page should not show links to model server when kserve is enabled on the project
When project is not marked as kserve or MM, warn user that selecting kserve reduces features (TBD with UX)

Aspects continued elsewhere
No response
Mocks

No mocks, discussions with UX needed around warning user kserve does not support bias or perf metrics and change is not reversible.

Replaced by https://issues.redhat.com/browse/RHOAIENG-248 - moving to closed


Icon on Taskbar can't be shown correctly

VSCode Version: Code 1.16.1 (27492b6bf3acb0775d82d2f87b25a93490673c6d, 2017-09-14T16:38:23.027Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

vscode-instant-markdown
dba
1.3.0

python
don
0.7.0

cpptools
ms-
0.12.4

java
red
0.10.0

vscode-icons
rob
7.13.0

(1 theme extensions excluded)

Steps to Reproduce:

Reproduces without extensions: Yes/No

@Blazero Can you please be specific about what icon and where ? Screenshots would be very much helpful.

Of course. As I said, the icon on Taskbar.
<EMAIL_ADDRESS>The one in red circle is VSCode, it was not shown correctly.
Thank you for your time.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Sandeep<EMAIL_ADDRESS>Sent: 2017年9月25日 22:01
To<EMAIL_ADDRESS>Cc: LU<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [Microsoft/vscode] Icon on Taskbar can't be shown correctly (#34930)
@Blazerohttps://github.com/blazero Can you please be specific about what icon and where ? Screenshots would be very much helpful.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vscode/issues/34930#issuecomment-331889856, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANsmYRXuB2yFX3Xf3IN0uiWkbsYbdjXzks5sl7IMgaJpZM4PiP8z.

@Blazero looks like the screenshot is not coming through?

I’ll send you a new picture. In case you can’t see it, I also add it as an attachment.
<EMAIL_ADDRESS>This problem happened after I uninstall 32-bit VSCode and install the 64-bit one.
And the VSCode has been auto-updated for some times after installed, this problem still exists.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Benjamin<EMAIL_ADDRESS>Sent: 2017年9月26日 20:56
To<EMAIL_ADDRESS>Cc: LU<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [Microsoft/vscode] Icon on Taskbar can't be shown correctly (#34930)
@Blazerohttps://github.com/blazero looks like the screenshot is not coming through?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vscode/issues/34930#issuecomment-332189018, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANsmYe8raNjiOy4XVXh1W7x9US1Am61Pks5smPR6gaJpZM4PiP8z.

@Blazero I still don't see the image. Please reply in github, not through email.

I've re-installed VSCode and it's solved. Thanks.


Java error

When I'm trying to launch the toolkit, I'm getting "A Java Exception has occured".
I have Java 17.0.9
How I can fix this?
Thanks

I tried the new version, It's still crashing. I have jdk-20.

Can't particularly say without any more information.
Is that the only error displayed? Multiple installs of Java on your system?

Yeah it's the only error displayed... The title of the error dialog is "Java Virtual Machine Launcher".
On this computer I just have Java JDK 20 and jre-1.8 (latest version of java 8)

Does the Java 8 runtime have precedence over the 20 JDK then?
Running with the wrong version generally seems to be the cause of that specific issue.
Try checking the version of Java that's being used in the path, or launching through command line.

Ok when I type java -jar toolkit-0.1.jar it's working.
And when type java -version i have java version "20.0.1".
I thought `java -jar toolkit-0.1.jar was the same result has double clicking on the .jar file. Weird but at least it's working like that. Maybe double clicking uses Java 8?
Just a question, any plans about PSP version support?

Yeah, seems so.
As for PSP support, it's not currently planned at the moment, although I've considered it.

Ok, thanks for responding.


Crafter's cheaty trick

Take a Crafter, define recipe with 1 damaged Tinker's Construct high-leveled tool and corresponding ingot to make a recipe of repaired tool. Then support that ingot and tool from same material and with same id:meta value, but low-leveled. We gets repaired, high-leveled (full of bonuses) tool from low-leveled tool!
Maybe ignoring NBT-data when crafting is universal, but it allow bad people to do bad things...
Version: 3.21.

Not sure I fully understand what you mean here. Do you have a screenshot perhaps to clarify what exactly that you are doing?

https://youtu.be/eGtE48fysWk

Thanks for this video. Very weird. I have to see what is going on there.

I'll try to describe: first you set up recipe, and Crafter remembers the final result with all NBT-data of result. But it is not check NBT of items we supply for craft, so we can trick it - to supply a cheaper items.

I could reproduce it and will fix in the next release


Github actions: update comment rather than re-create

Mode
Other
Feature description
Use case
Lost Pixel currently posts a new comment for each force push, which creates some noise for projects that extensively use Git in this way. Some comments could event get lost as GitHub collapses activities when there are too many.
I would love to have a single Lost Pixel comment at the top that gets updated based on CI status.
Screenshots

What are other tools doing?
Vercel deploy preview

Posts one comment when opening the PR

Upon force push, the comment gets updated until the new build finishes

Codecov
Lets you configure between updating (default) or deleting and then re-creating: https://docs.codecov.com/docs/pull-request-comments#behavior.

This is actually pretty awesome idea to update single comment instead of always posting a new one! Should be possible 👍🏼
Lost Pixel is really not having the best synergy with force pushes as we calculate the baselines in the git history 🥲 do you think this flow is still usable all together?

Our need is to make sure that there's no regression between the pull request and main; we don't care that much about diffs between individual commits on the same PR. The fact that we'll have to re-approve the diffs is not ideal, but not blocking either - we can definitely live with it.
Out of curiosity, I just looked at how Vercel does that and it looks like they're adding a hidden token in the comment to identify it (I suppose vc stands for Vercel Comment):

I wish there was a nicer way on our platform to work with force pushes :( you did not have this issue with chromatic, right?
I did not fully get the meaning of hidden comment 🙂 you mean they use it to identify what to update when searching trough issue comments?

you did not have this issue with chromatic, right?

The CI logs are expired, so I can't double-check, but from what I remember chromatic used to remember validated diffs even after a force push.

you mean they use it to identify what to update when searching trough issue comments?

Correct, I suppose that they're doing that.

We will try to think how to avoid the duplicated approvals on force pushes as well. Thanks for the idea of improving the comments!

As a workaround, a setting to turn off comments on PRs would be great. On PRs with multiple force pushes, we often end up with dozens of comments that blotter the thread, while the CI job status would be just enough.

@Betree we currently don't have an option on the platform to just disable the gh integration :/ miss on our side, would it help if I disable it for you unit we have a better way of handling this(via updating the comment as agreed)?

@Betree we currently don't have an option on the platform to just disable the gh integration :/ miss on our side, would it help if I disable it for you unit we have a better way of handling this(via updating the comment as agreed)?

Ideally, we would just disable the Github comments (while keeping the CI job). If not feasible then yes, disabling the Github integration altogether would work for us. Thanks!

of course GH integration would continue to work, it's misleading on the platform, what you've enabled is purely for posting comments!

Done, GH comments are gone, check is still present!


Nested definitions

(splitting issue #190 and merging with #186)
This issue is about supporting nesting definitions, e.g.:
return [
    Db::class => DI\object()
        ->constructor(
            \DI\object(DbConfig::class)
                ->constructor(
                    \DI\env('DATABASE_HOST')
                )
        ),
];

Implemented in 5.0


SELECT statement executes for more than 30s when meausurement does not exist

Describe the bug（Bug 描述）
The select statement executes for more than 30s when measurement does not exist.
To Reproduce（Bug 复现步骤）
No response
Expected behavior（期望结果）
When the measurement does not exist, the select statement execution immediately returns a relative failure error.
Screenshots（屏幕截图）
No response
Logs（完整的错误日志）
No response
Additional context（其他的一些补充内容）
No response

Thanks for the bug report. Please mention the openGemini version number and the SELECT statement. e.g.

show measurements
name: measurements
name

sensor

select * from not_exists
<reported an error after 30s or just stopped here without outputing anything?>


Homepage on Github Pages empty

Git repository URL: https://github.com/niritum/niritum.github.io
GitHub Pages hosted: https://niritum.com
My homepage is empty on Github Pages despite it working without any errors on local PC.
Used "Gem-based method".  What did I do wrong?

Gem-based method does not "automatically work" on GitHub Pages. You need to setup some CI building or use remote-theme method (which is GH Pages-compatible out-of-the-box).


Load waveform but defer loading of audio file until Play is clicked

Is it possible to render the waveform from JSON file or data object, but defer loading of the actual MP3 audio file until you're actually ready to play the file?
My use case is that I have an infinite-scrolling page of players, each with a play button and waveform.  I obviously don't want to abuse my users' bandwidth by loading each and every audio file they scroll past.  I only want to actually load the MP3s that they play.  Buffer time is not a concern as my files are voice, not music.
I suppose I could render the waveforms into a div myself, then replace that with the player once play is clicked.  I worry that would cause flickering, or that I wouldn't be able to exactly replicate the look of your waveform.

Is it possible to render the waveform from JSON file or data object, but defer loading of the actual MP3 audio file until you're actually ready to play the file?

No, this isn't possible currently, but it wouldn't be too much work to allow for this. Currently the load() function does both things, but I can implement two new additional function loadWaveform() and loadAudio() for separately handling the (initial) waveform drawing and loading the audio file.
Another option is to make the preload attribute of the HTML audio element configurable. Currently this is hardcoded to 'auto', but it would make more sense to allow the user to determine the value for this, and perhaps the default should even be 'metadata' instead of 'auto' come to think of it. However, it also says that:

The browser is not forced by the specification to follow the value of this attribute; it is a mere hint.

That is kind of vague to say the least, but solely relying on the value of the preload attribute probably isn't sufficient.
So I will add both things in the next release. Not much time this week, but probably can do it somewhere next week.

loadWaveform() and loadAudio() sound like they would do the trick!

Just published v1.2.0 which contains these new features.

That's awesome!  I'll test it out in the next few days.  Thanks!


Cordova with iOS Google Map

Hi ,
First of all, I wanna know that now a days apple allowed google maps or not while upload my app in the app store.
I tried to use this nice plugin in my ios Cordova project But I can't able to proceed. I had a Cordova latest version installed in my machine Cordova @6.1.0 and Cordova iOS version is @4.1.1.
After installing the plugin using CLI the plugin was installed but while build my application I'm getting the below error as
** BUILD FAILED **

The following build commands failed:
CpResource HelloMap/Resources/GoogleMaps.bundle build/emulator/HelloMap.app/GoogleMaps.bundle
(1 failure)
Error: Error code 65 for command: xcodebuild with args: -xcconfig,/Users/Test/Shiv_Suthan_M_Drive/Suthan_Drive/Cordova_Dev/HelloMap/platforms/ios/cordova/build-debug.xcconfig,-project,HelloMap.xcodeproj,ARCHS=i386,-target,HelloMap,-configuration,Debug,-sdk,iphonesimulator,build,VALID_ARCHS=i386,CONFIGURATION_BUILD_DIR=/Users/Test/Shiv_Suthan_M_Drive/Suthan_Drive/Cordova_Dev/HelloMap/platforms/ios/build/emulator,SHARED_PRECOMPS_DIR=/Users/Test/Shiv_Suthan_M_Drive/Suthan_Drive/Cordova_Dev/HelloMap/platforms/ios/build/sharedpch

So, I can't able to run the application please help me in this to use your plugin properly ASAP.

Please past all error logs, not only a few lines.

Hi - I think the plugin will work well for the combination of Cordova CLI 5 series and the Cordova iOS of 3.9.0 like this series i can able use the plugin but newer version i can't able to proceed.
@wf9a5m75 thanks for your msg - for more information while using the newer version errors please find below
while build using cordova CLI
Getting below error

** BUILD FAILED **
The following build commands failed:
CompileC build/HelloCoredova.build/Debug-iphonesimulator/HelloCoredova.build/Objects-normal/i386/GroundOverlay.o HelloCoredova/Plugins/plugin.google.maps/GroundOverlay.m normal i386 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)
Error code 65 for command: xcodebuild with args: -xcconfig,/Users/Test/Shiv_Suthan_M_Drive/Suthan_Drive/Cordova_Dev/HelloCoredova/platforms/ios/cordova/build-debug.xcconfig,-project,HelloCoredova.xcodeproj,ARCHS=i386,-target,HelloCoredova,-configuration,Debug,-sdk,iphonesimulator,build,VALID_ARCHS=i386,CONFIGURATION_BUILD_DIR=/Users/Test/Shiv_Suthan_M_Drive/Suthan_Drive/Cordova_Dev/HelloCoredova/platforms/ios/build/emulator,SHARED_PRECOMPS_DIR=/Users/Test/Shiv_Suthan_M_Drive/Suthan_Drive/Cordova_Dev/HelloCoredova/platforms/ios/build/sharedpch
ERROR building one of the platforms: Error: /Users/Test/Shiv_Suthan_M_Drive/Suthan_Drive/Cordova_Dev/HelloCoredova/platforms/ios/cordova/build: Command failed with exit code 2
You may not have the required environment or OS to build this project
Error: /Users/Test/Shiv_Suthan_M_Drive/Suthan_Drive/Cordova_Dev/HelloCoredova/platforms/ios/cordova/build: Command failed with exit code 2
at ChildProcess.whenDone (/usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/cordova/superspawn.js:131:23)
at ChildProcess.emit (events.js:110:17)
at maybeClose (child_process.js:1015:16)
at Process.ChildProcess._handle.onexit (child_process.js:1087:5)
eMO:HelloCoredova Test$

While build directly in xcode getting below error log :

Well, I (and other people) use Cordova 6.1 with this plugin (th master branch).
And the CDVJSON.h must be included by Cordova framework, not this plugin.
So I guess the error reason is somewhere elese.
http://stackoverflow.com/questions/10714600/cdvplugin-h-file-not-found-in-cordova-as-component-cleaver
Try to install the master branch.

Hi
Uninstall plugin and use this
cordova plugin add https://github.com/phonegap-googlemaps-plugin/cordova-plugin-googlemaps --variable API_KEY_FOR_ANDROID="YOUR_ANDROID_API_KEY_IS_HERE" --variable API_KEY_FOR_IOS="YOUR_IOS_API_KEY_IS_HERE"
You might want to user<EMAIL_ADDRESS>if that does not work

I'm also having a similar issue running on iOS.
I was successfully building on 5.4.1 but had to upgrade for phonegap-plugin-push.
I upgraded to Cordova 6.1.1 and tried<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I'm not seeing the CDVJSON issue in Xcode 7.2, only the self.webView issues.
CompileC build/YoRally.build/Debug-iphonesimulator/YoRally.build/Objects-normal/i386/GoogleMaps.o YoRally/Plugins/plugin.google.maps/GoogleMaps.m normal i386 objective-c com.apple.compilers.llvm.clang.1_0.compiler
    cd /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios
    export LANG=en_US.US-ASCII
    export PATH="/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/usr/bin:/Applications/Xcode.app/Contents/Developer/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin"
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang -x objective-c -arch i386 -fmessage-length=211 -fdiagnostics-show-note-include-stack -fmacro-backtrace-limit=0 -fcolor-diagnostics -std=c99 -fobjc-arc -fmodules -gmodules -fmodules-prune-interval=86400 -fmodules-prune-after=345600 -fbuild-session-file=/var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/C/org.llvm.clang/ModuleCache/Session.modulevalidation -fmodules-validate-once-per-build-session -Wno-trigraphs -fpascal-strings -O0 -Wno-missing-field-initializers -Wno-missing-prototypes -Wno-implicit-atomic-properties -Wno-arc-repeated-use-of-weak -Wduplicate-method-match -Wno-missing-braces -Wparentheses -Wswitch -Wunused-function -Wno-unused-label -Wno-unused-parameter -Wunused-variable -Wunused-value -Wempty-body -Wuninitialized -Wno-unknown-pragmas -Wno-shadow -Wno-four-char-constants -Wno-conversion -Wconstant-conversion -Wint-conversion -Wbool-conversion -Wenum-conversion -Wno-shorten-64-to-32 -Wpointer-sign -Wno-newline-eof -Wno-selector -Wno-strict-selector-match -Wundeclared-selector -Wno-deprecated-implementations -DDEBUG=1 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator9.2.sdk -fasm-blocks -fstrict-aliasing -Wprotocol -Wdeprecated-declarations -mios-simulator-version-min=8.0 -g -fvisibility=hidden -Wno-sign-conversion -fobjc-abi-version=2 -fobjc-legacy-dispatch -iquote /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/YoRally-generated-files.hmap -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/YoRally-own-target-headers.hmap -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/YoRally-all-target-headers.hmap -iquote /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/YoRally-project-headers.hmap -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/emulator/include -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/emulator/usr/local/lib/include -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/UninstalledProducts/include -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/UninstalledProducts/iphonesimulator/include -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/emulator -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/DerivedSources/i386 -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/DerivedSources -F/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/emulator -FYoRally/Plugins/com.googlemaps.ios -FYoRally/Plugins/cordova-plugin-facebook4 -include /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/sharedpch/YoRally-Prefix-cwpwcqwyfbhasbhkhnbfkmxbohku/YoRally-Prefix.pch -MMD -MT dependencies -MF /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/Objects-normal/i386/GoogleMaps.d --serialize-diagnostics /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/Objects-normal/i386/GoogleMaps.dia -c /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMaps.m -o /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/Objects-normal/i386/GoogleMaps.o
/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMaps.m:24:30: warning: incompatible pointer types assigning to 'UIWebView *' from 'UIView * _Nullable'
      [-Wincompatible-pointer-types]
    self.pluginLayer.webView = self.webView;
                             ^ ~~~~~~~~~~~~
/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMaps.m:28:45: warning: incompatible pointer types assigning to 'UIWebView *' from 'UIView * _Nullable'
      [-Wincompatible-pointer-types]
    self.pluginScrollView.debugView.webView = self.webView;
                                            ^ ~~~~~~~~~~~~
/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMaps.m:34:30: warning: incompatible pointer types assigning to 'UIWebView *' from 'UIView * _Nullable'
      [-Wincompatible-pointer-types]
    self.pluginLayer.webView = self.webView;
                             ^ ~~~~~~~~~~~~
/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMaps.m:167:30: warning: incompatible pointer types assigning to 'UIWebView *' from 'UIView * _Nullable'
      [-Wincompatible-pointer-types]
        self.mapCtrl.webView = self.webView;
                             ^ ~~~~~~~~~~~~
/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMaps.m:176:59: error: no known instance method for selector 'initWithWebView:'
        Map *mapClass = [[NSClassFromString(@"Map")alloc] initWithWebView:self.webView];
                                                          ^~~~~~~~~~~~~~~
/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMaps.m:186:38: warning: incompatible pointer types assigning to 'UIWebView *' from 'UIView * _Nullable'
      [-Wincompatible-pointer-types]
            self.pluginLayer.webView = self.webView;
                                     ^ ~~~~~~~~~~~~
/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMaps.m:245:68: error: no known instance method for selector 'initWithWebView:'
                pluginClass = [[NSClassFromString(className)alloc] initWithWebView:self.webView];
                                                                   ^~~~~~~~~~~~~~~
/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMaps.m:321:19: error: no visible @interface for 'UIView' declares the selector
      'stringByEvaluatingJavaScriptFromString:'
    [self.webView stringByEvaluatingJavaScriptFromString:jsString];
     ~~~~~~~~~~~~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMaps.m:404:34: warning: incompatible pointer types assigning to 'UIWebView *' from 'UIView * _Nullable'
      [-Wincompatible-pointer-types]
        self.pluginLayer.webView = self.webView;
                                 ^ ~~~~~~~~~~~~
6 warnings and 3 errors generated.

CompileC build/YoRally.build/Debug-iphonesimulator/YoRally.build/Objects-normal/i386/GoogleMapsViewController.o YoRally/Plugins/plugin.google.maps/GoogleMapsViewController.m normal i386 objective-c com.apple.compilers.llvm.clang.1_0.compiler
    cd /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios
    export LANG=en_US.US-ASCII
    export PATH="/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/usr/bin:/Applications/Xcode.app/Contents/Developer/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin"
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang -x objective-c -arch i386 -fmessage-length=211 -fdiagnostics-show-note-include-stack -fmacro-backtrace-limit=0 -fcolor-diagnostics -std=c99 -fobjc-arc -fmodules -gmodules -fmodules-prune-interval=86400 -fmodules-prune-after=345600 -fbuild-session-file=/var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/C/org.llvm.clang/ModuleCache/Session.modulevalidation -fmodules-validate-once-per-build-session -Wno-trigraphs -fpascal-strings -O0 -Wno-missing-field-initializers -Wno-missing-prototypes -Wno-implicit-atomic-properties -Wno-arc-repeated-use-of-weak -Wduplicate-method-match -Wno-missing-braces -Wparentheses -Wswitch -Wunused-function -Wno-unused-label -Wno-unused-parameter -Wunused-variable -Wunused-value -Wempty-body -Wuninitialized -Wno-unknown-pragmas -Wno-shadow -Wno-four-char-constants -Wno-conversion -Wconstant-conversion -Wint-conversion -Wbool-conversion -Wenum-conversion -Wno-shorten-64-to-32 -Wpointer-sign -Wno-newline-eof -Wno-selector -Wno-strict-selector-match -Wundeclared-selector -Wno-deprecated-implementations -DDEBUG=1 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator9.2.sdk -fasm-blocks -fstrict-aliasing -Wprotocol -Wdeprecated-declarations -mios-simulator-version-min=8.0 -g -fvisibility=hidden -Wno-sign-conversion -fobjc-abi-version=2 -fobjc-legacy-dispatch -iquote /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/YoRally-generated-files.hmap -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/YoRally-own-target-headers.hmap -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/YoRally-all-target-headers.hmap -iquote /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/YoRally-project-headers.hmap -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/emulator/include -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/emulator/usr/local/lib/include -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/UninstalledProducts/include -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/UninstalledProducts/iphonesimulator/include -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/emulator -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/DerivedSources/i386 -I/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/DerivedSources -F/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/emulator -FYoRally/Plugins/com.googlemaps.ios -FYoRally/Plugins/cordova-plugin-facebook4 -include /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/sharedpch/YoRally-Prefix-cwpwcqwyfbhasbhkhnbfkmxbohku/YoRally-Prefix.pch -MMD -MT dependencies -MF /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/Objects-normal/i386/GoogleMapsViewController.d --serialize-diagnostics /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/Objects-normal/i386/GoogleMapsViewController.dia -c /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMapsViewController.m -o /Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/YoRally.build/Debug-iphonesimulator/YoRally.build/Objects-normal/i386/GoogleMapsViewController.o
/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/YoRally/Plugins/plugin.google.maps/GoogleMapsViewController.m:10:9: fatal error: 'Cordova/CDVJSON.h' file not found
#import <Cordova/CDVJSON.h>
        ^

** BUILD FAILED **

The following build commands failed:
	CompileC build/YoRally.build/Debug-iphonesimulator/YoRally.build/Objects-normal/i386/GoogleMaps.o YoRally/Plugins/plugin.google.maps/GoogleMaps.m normal i386 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)
Error: Error code 65 for command: xcodebuild with args: -xcconfig,/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/cordova/build-debug.xcconfig,-project,YoRally.xcodeproj,ARCHS=i386,-target,YoRally,-configuration,Debug,-sdk,iphonesimulator,build,VALID_ARCHS=i386,CONFIGURATION_BUILD_DIR=/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/emulator,SHARED_PRECOMPS_DIR=/Users/jshredder1/YoRallyAuto/ChatApp_0.1/YoRally/platforms/ios/build/sharedpch

@hoffination you need to upgrade using the master version. Delete old plugin version and reinstall. Howto is on first-page wiki

That worked! Thanks!


LNK1181  error with Hello World exmaple

I have .net-core 2.1 installed and I have VS2017 15.7.4
this is the error I got

H:\project\corertTest\HelloWorld>dotnet publish -r win-x64 -c release
Microsoft (R) Build Engine version <IP_ADDRESS>72 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

  Restoring packages for H:\project\corertTest\HelloWorld\HelloWorld.csproj...
  Restore completed in 917.17 ms for H:\project\corertTest\HelloWorld\HelloWorld.csproj.
  HelloWorld -> H:\project\corertTest\HelloWorld\bin\release\netcoreapp2.1\win-x64\HelloWorld.dll
LINK : fatal error LNK1181:  [H:\project\corertTest\HelloWorld\HelloWorld.csproj]
C:\Users\h6hfhrht\.nuget\packages\microsoft.dotnet.ilcompiler\1.0.0-alpha-26623-01\build\Microsoft.NETCore.Native.targets(234,5): error MSB3073: The command "link  @"obj\release\netcoreapp2.1\win-x64\native\link.rsp"" exited with code 1181. [H:\project\corertTest\HelloWorld\HelloWorld.csproj]

I followed everything on README.MD in Building a Hello World console app with CoreRT
any ieda what is causing this error?
Folder:

Nuget.config:

LNK1181 means "cannot open input file": https://docs.microsoft.com/en-us/cpp/error-messages/tool-errors/linker-tools-error-lnk1181
Could you please run the failing command "link  @"obj\release\netcoreapp2.1\win-x64\native\link.rsp" directly on the command line? It should show more details about the error.

Solved, was just my local env variable problem.

Hi i've the follower error:
C:\Users\andre\.nuget\packages\microsoft.dotnet.ilcompiler\1.0.0-alpha-27222-01\build\Microsoft.NETCore.Native.targets(302,5): error MSB3073: uscita dal comando "link  @"obj\release\netcoreapp2.2\win-x64\native\link.rsp"" con codice 1181. [C:\Users\andre\Desktop\Nuova cartella\HelloWorld\HelloWorld.csproj]```

(it's italian but simply it says that it's impossible to open the 'kernel32.lib')

@andreakarasho Try to compile with verbose log (/v:d command line switch). These kind of errors are typically caused by bad environment variables, e.g. your environment can have variable pointing to Windows SDK instance that was uninstalled and does not exist anymore.

Which sdk version should i need?
From verbose log error appers after thus line:
Attività "Exec"
link  @"obj\Release\netcoreapp2.2\win-x64\native\link.rsp"

@jkotas
C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\bin\Hostx64\x64

Basically some time ago i added this variable for another application that didn't find the LINKER.
Not let's try to understand how write a rd.xml file :D


Include build flags to remove 'softdevice' from the build

The softdevice premade binary adds signficant size to the generated builds, and is frequently not needed by users so @finneyj has constructed a branch without the softdevice additions (see nosd branch), but we should really have a build switch to configure this behaviour, as removing softdevice does eliminate support for many radio functions.
This would allow high-level-languages to decide if they need this support, or would rather have the flash space, but needs significant work with CMake et.al to achieve.
NB. Putting this on v0.2.45 for now, but this is not a final decision, but just as a placeholder

Merged in to staging, and looks good so far, so on-track for the next tag.

Are you pushing the staging branch? Can't really identify it looking at the current branches in this repo.


fix spatial alignment key_added

the dataset description appears on mouseover so i only clarified the difference in spatial or not

View entire conversation on ReviewNB


Fix incorrect code sample

getInitialState and componentDidMount were nested inside the render method.

Nice catch, thank you! :)


Gum plugin failure

The gum plugin fails when trying to install the following versions of gum:

0.14.1
0.14.0

> proto use
Error: proto::execute::missing_file

  × Unable to find an executable for Gum, expected file ~\.proto\tools\gum\0.14.1\gum.exe does not exist.

❯ ls -Recurse C:\Users\user2\.proto\tools\gum\0.14.1

    Directory: C:\Users\user2\.proto\tools\gum\0.14.1

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
d----          2024/07/16    13:59                gum_0.14.1_Windows_x86_64

    Directory: C:\Users\user2\.proto\tools\gum\0.14.1\gum_0.14.1_Windows_x86_64

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
d----          2024/07/16    13:59                completions
d----          2024/07/16    13:59                manpages
-a---          2024/07/16    14:10       27823616 gum.exe
-a---          2024/07/16    14:10           1080 LICENSE
-a---          2024/07/16    14:10          12718 README.md

    Directory: C:\Users\user2\.proto\tools\gum\0.14.1\gum_0.14.1_Windows_x86_64\completions

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
-a---          2024/07/16    14:10          26012 gum.bash
-a---          2024/07/16    14:10          18998 gum.fish
-a---          2024/07/16    14:10          14978 gum.zsh

    Directory: C:\Users\user2\.proto\tools\gum\0.14.1\gum_0.14.1_Windows_x86_64\manpages

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
-a---          2024/07/16    14:10           5099 gum.1.gz

The following version of the plugin seems to work, even for gum v0.13.0:
name = "gum"
type = "cli"

[resolve]
git-url = "https://github.com/charmbracelet/gum"

[platform.linux]
archive-prefix = "gum_{version}_Linux_{arch}"
download-file = "gum_{version}_Linux_{arch}.tar.gz"
checksum-file = "checksums.txt"

[platform.macos]
archive-prefix = "gum_{version}_Darwin_{arch}"
download-file = "gum_{version}_Darwin_{arch}.tar.gz"
checksum-file = "checksums.txt"

[platform.windows]
archive-prefix = "gum_{version}_Windows_{arch}"
download-file = "gum_{version}_Windows_{arch}.zip"
checksum-file = "checksums.txt"

[install]
download-url = "https://github.com/charmbracelet/gum/releases/download/v{version}/{download_file}"
checksum-url = "https://github.com/charmbracelet/gum/releases/download/v{version}/{checksum_file}"

[install.arch]
x86 = "i386"
arm = "armv6"
aarch64 = "arm64"

Thanks, I've merged your fix.


Add a new service the clear's the stray DMA transactions

In the current state , the DMA infrastruture written in pldm
is blocking IO, due to pldm operating in blocking mode there
are cases where pldm hangs in read/write system calls due to
multiple reasons like :

Stray DMA message in mctp buffer
When DMA operations in progress with PHYP and it goes down
abruptly, the DMA command of PHYP could be stored inside
the mctp buffer and that can be delivered to Hostboot(which does
not have pcie support) leading to a pldm hang.

PHYP crashes while a DMA transaction is in progress
xdma driver on bmc does not have a way to know if host had gone
down due to crash , so the outstanding DMA operation that is
currently running would be left hanging, leaving pldm to hang
as well.

Added new service pldm-clear-dma.service which would send
signals (SIGSTOP, SIGCONT) signals to pldm while the host
is going down, so that the stray DMA operations are cancelled
and pldm daemon can be recovered from the system call hangs.
Signed-off-by: Manojkiran Eda<EMAIL_ADDRESS>

Critical fix for SW553469. Must-fix approval in process.


SyncScript Inheritance hiding base class member with "new" keyword broken.

Release Type: Official Release
Version: <IP_ADDRESS> (or .7)
Platform(s): Windows
Describe the bug
When implementing SyncScript on a base class, a child class using "new" keyword to hide base class fields does not hide/overwrite the base fields at runtime.  This could be due to integration with Game Studio. I am able to cache references/propagate fields for the child script in Game Studio but at runtime base class fields are null.
You can work around this by assigning base fields to child fields during Start().
To Reproduce
Steps to reproduce the behavior:

Create a base SyncScript (i.e. "Character").
Create a child class and inherit from base (i.e. "Ninja : Character").
Create protected or public fields/properties in Character.
Create public fields/properties in Ninja (use the "new" keyword).
Cache references in Game Studio (assign a prefab to field/property in Ninja).
Compile and run.

Expected behavior
I would expect this to work, but I'm fairly amateur and could be missing something. I would expect Script components to support inheritance and still be able to cache references through Game Studio.
Screenshots
If applicable, add screenshots to help explain your problem (use GitHub drag & drop)
Unfortunately I moved on and have no screenshots, there are no errors though, just null base class members.
Log and callstacks
If any, please attach here any log or callstack (preferably in a .txt file using GitHub drag and drop)
Additional context
Add any other context about the problem here.
In a large game I would expect script components would support inheritance and there would be modularity in inheritance and interfaces, etc.

It is not a bug in Xenko, just how c# works. using the new keyword on members is bad and just be avoided at all cost.
The new keyword only hides the members in the context of that class, not from the base class point of view.

Awesome, thanks. I don’t generally use the feature and misunderstood it’s implementstion.

Welcome.
It's by doing mistake that we learn. You are on the right track.


train_large.sh with --useGPT

Hello, I added --useGPT in train_large.sh, but I encountered an error when trying to train with Whisper Large and GPT-2 together. (but it seems normal in train.sh)

Could you tell me how to solve this problem? Thanks!!!!!

Hi. Thank you for your message. --useGPT option is not supported when you use the Whisper large model. Whisper models such as base.en and medium.en have the same set of tokens as GPT2, whereas Whisper large does not. Therefore, you will have indices that do not belong to the GPT2 token embedding matrices hence causing this index overflow error.

Hi. Thank you for your message. --useGPT option is not supported when you use the Whisper large model. Whisper models such as base.en and medium.en have the same set of tokens as GPT2, whereas Whisper large does not. Therefore, you will have indices that do not belong to the GPT2 token embedding matrices hence causing this index overflow error.

Thanks!!! So, could you tell me how the implementation of Whisper large + TCPGen + GPT-2, as mentioned in the paper, was done?

Hi. This was done by first generating the N-best list using Whisper large, and then rescoring them using GPT-2. In this process, you just treat each hypothesis as text and calculate language model scores using GPT-2. Then you add that language model scores to the Whisper output scores of each hypothesis (with an LM scaling factor of 0.05 in my case) and re-rank them to find the new best one.

Hi. This was done by first generating the N-best list using Whisper large, and then rescoring them using GPT-2. In this process, you just treat each hypothesis as text and calculate language model scores using GPT-2. Then you add that language model scores to the Whisper output scores of each hypothesis (with an LM scaling factor of 0.05 in my case) and re-rank them to find the new best one.

I see!!!!! Thank you very much!!!!!!!!!!!!!!!!!!!!!!


Calibrate Glitch's DriveTrain

Get Max Low Gear Speed
Get Max High Gear Speed
Set PIDF
Determine what artificial cap will be placed on MaxSpeed
Graph velocity/acceleration curves
Determine upshift velocity
Determine downshift coast velocity

Largely done. Will need to be tweaked when we have the final weighted version of the robot


Add Drawer Shadow

IT'd be awesome if you create a method to easily set the drawer shadow on the drawer. Regardless this is great work and progress so far!! keep it up

I am using this for setting shadows
mDrawerLayout.setDrawerShadow(R.drawable.drawer_shadow, Gravity.START);

I'll add this today to the custom DrawerLayout implementation and then I'll release it


Cleanup: IO and formatting

Since open by default in read mode I removed 'r' arguments, also made some minor cleanups.
Thanks.

✨🍰✨

✨Yuhuuu thank you✨


Executor should call sendModelingCommand with pathToNode

We are currently using sourceRange as our pointer to a specific place in the code, and backwards engineering where this is in the AST, the problem with this is the sourceRanges in the ast go stale as the only way to get accurate sourceRanges in the AST is by parse(recast(ast))ing, so after multiple code-mods we have to do this extra step which is an ongoing source of bugs and is inefficient.
pathToNode is a much more AST native pointer into the code.
Will require the executor to know where it is in the tree as it's executing nodes, so that it can pass the path through to sendModelingCommand

I was thinking about this yesterday, and it occurred to me that we modify the AST during normal use, not just the program text. Those modifications can invalidate a path-to-node because the structure of the AST changes.
If we're going through the work to change this, would we be better off giving every AST node an ID, and using that? Then when modifying the AST, new nodes that would have been at the same path can inherit the same ID. Or not. Maybe it moves to a different path. It depends how we use path-to-node, but an ID could serve the same purpose with even more flexibility.
A drawback is that we'd need to maintain a mapping from ID to node.

If we went with IDs, the other metadata we'd need to store is given an ID/node, what is its parent?

We talked about this as seemed to reach a working consensus, we can move forward with ids, that doesn't preclude pathToNode though, but instead we might use more of a UUID-path.


fix: adapt 11.3 api to create layouts in project

I am sure you fixed that perfect and certain better than I could do!


Fixed AR-113 and added regression tests

Scope & Purpose
Previously, the geo-index-optimizer rule could move expressions into the index which used variables which aren't available yet at the index node.

[X] :hankey: Bugfix (requires CHANGELOG entry)
[X] :book: CHANGELOG entry made

Backports:

[ ] No backports required
[ ] Backports required for: (Please specify versions and link PRs)

Related Information

[X] GitHub issue / Jira ticket number: Fixes AR-113.

Testing & Verification

[X] The behavior in this PR was manually tested
[X] This change is already covered by existing tests, such as aql-optimizer-geoindex.
[X] This PR adds tests that were used to verify all changes:

[X] Added new integration tests

http://jenkins.arangodb.biz:8080/job/arangodb-matrix-pr/13147/

http://jenkins.arangodb.biz:8080/job/arangodb-matrix-pr/13195/

http://<IP_ADDRESS>:8080/job/arangodb-matrix-pr/13212/

http://jenkins.arangodb.biz:8080/job/arangodb-matrix-pr/13216/


PermissionError when ./setup/ros_setup.bash at first

my IAM accont has admin permission, when I execute in Cloud9,
git clone https://github.com/aws-samples/multi-robot-fleet-sample-application.git
cd multi-robot-fleet-sample-application
./setup/ros_setup.bash
it has error:
File "/usr/local/lib/python3.6/dist-packages/colcon_core/command.py", line 528, in verb_main
rc = context.args.main(context=context)
File "/usr/local/lib/python3.6/dist-packages/colcon_bundle/verb/bundle.py", line 143, in main
dependencies_changed)
File "/usr/local/lib/python3.6/dist-packages/colcon_bundle/verb/_archive_generators.py", line 103, in generate_archive_v2
workspace_tar_gz_path)
File "/usr/local/lib/python3.6/dist-packages/colcon_bundle/verb/_overlay_utilities.py", line 32, in create_workspace_overlay
_CONTEXT_VAR_BASH
File "/usr/local/lib/python3.6/dist-packages/colcon_bundle/verb/_overlay_utilities.py", line 156, in _generate_template
with open(script_location, 'w') as file:
PermissionError: [Errno 13] Permission denied: '/usr/local/lib/python3.6/dist-packages/colcon_bundle/verb/assets/v2_workspace_setup.bash'

I found /usr/local/lib/python3.6/dist-packages/colcon_bundle/verb/assets/v2_workspace_setup.bash is not exist

This issue should be resolved with https://github.com/aws-samples/multi-robot-fleet-sample-application/pull/11
Can you confirm @qqhzg

Confirmed from @qqhzg  that this issue is resolved. closing issue.


feat: monitor hooks, connection send hooks

- What I did
Implemented the following hooks for monitor connection:

pre/post notification decryption (perfect for locking the sync client, no race conditions!)

I did a very minimal implementation for monitor, because the other use-cases are not well defined or clear at this time. I went with what I needed and no more than that.
Implemented the following hooks for atclient_connection_send:

pre/post send
pre/post receive

I did a full implementation with mostly strong type definitions for the hooks, since I don't foresee any other places we would want them at the connection level. Anywhere else in the connection lifecycle in which the end developer might want intermediate control, they wouldn't need hooks for.
- How I did it
- How to verify it
- Description for the changelog
feat: monitor hooks, connection send hooks

Upon approval, we should write up a ticket to write tests for this feature

I would like to propose this flow for initializing hooks and setting them to a client.
/* 1. Initialize clients */
atclient monitor_client;
atclient_init(&monitor_client);

atclient main_client;
atclient_init(&main_client);

/* 2. Initialize connection hooks */
atclient_connection_hooks my_hooks;
atclient_connection_hooks_init(&my_hooks);

/* 3. Set hooks to clients */
atclient_connection_set_hooks(&(monitor_client->atserver_connection), &my_hooks);
atclient_connection_set_hooks(&(main_client->atserver_connection), &my_hooks);

/* 4. Do stuff ... */

/* 5. Free everything */
atclient_free(&monitor_client);
atclient_connection_hooks_free(&my_hooks); // it doesn't matter when hooks is freed in relation to its connections, since it is allocated independently from the connection contexts themselves
atclient_free(&main_client);

Hooks has to be part of the memory of the atclient_connection struct, otherwise we have to pass around a new object.
The reason there is only an init and not a free, is that we don't want hooks to be using more memory than otherwise necessary. So the hooks init is a second init on the atclient_connection struct which additionally allocates memory for the hooks struct. This should still all be freed by atclient_connection, so that you never have to worry about previous state of an atclient_connection in order to free it.

I would like to propose this flow for initializing hooks and setting them to a client.
/* 1. Initialize clients */
atclient monitor_client;
atclient_init(&monitor_client);

atclient main_client;
atclient_init(&main_client);

/* 2. Initialize connection hooks */
atclient_connection_hooks my_hooks;
atclient_connection_hooks_init(&my_hooks);

/* 3. Set hooks to clients */
atclient_connection_set_hooks(&(monitor_client->atserver_connection), &my_hooks);
atclient_connection_set_hooks(&(main_client->atserver_connection), &my_hooks);

/* 4. Do stuff ... */

/* 5. Free everything */
atclient_free(&monitor_client);
atclient_connection_hooks_free(&my_hooks); // it doesn't matter when hooks is freed in relation to its connections, since it is allocated independently from the connection contexts themselves
atclient_free(&main_client);

Hooks has to be part of the memory of the atclient_connection struct, otherwise we have to pass around a new object.
The reason there is only an init and not a free, is that we don't want hooks to be using more memory than otherwise necessary. So the hooks init is a second init on the atclient_connection struct which additionally allocates memory for the hooks struct. This should still all be freed by atclient_connection, so that you never have to worry about previous state of an atclient_connection in order to free it.
Should I call it enable hooks instead?

Yes, enable hooks makes more sense to the flow we have going on in the C SDK.
If we call it init, then it tells the caller that they have to at some point call free. But calling it enable_hooks solves that problem.