Add basic manual configuration

Somehow I have issues with mDNS on my network and my Sync Box does not tell its there. This integration worked fine before but when I had to reinstall Home Assistant I was not able to set up my sync box.
With this workaround/feature I was able to configure my box. Maybe it is something you can use.

It works at least with my bridge, but there is no validation or error handling. What do you think?

It works at least with my bridge, but there is no validation or error handling. What do you think?

I was wondering about this too, validation on valid IP and at least a length check on the ID would be nice. Bbut I looked around a bit in HA code but for for IP address there does not seem to be validation anywhere...
When inputting wrong data it does show a message with "Unknown error", but at least it does not explode and it keeps the previous values in the input dialog so after fixing it you can just continue, so not perfect, but very usable.
If it is easy to add some validation/errorhandling it would be nice, but not mandatory.

HA code but for for IP address there does not seem to be validation anywhere

Validation in HA is 'Test if it works'. Because that validates if the IP is correct and working on one go.
Maybe we can add some regex for the ID, or not make it required at all if that is possible with HTTPS.

HA code but for for IP address there does not seem to be validation anywhere

Validation in HA is 'Test if it works'. Because that validates if the IP is correct and working on one go. Maybe we can add some regex for the ID, or not make it required at all if that is possible with HTTPS.

I ran into an errors because I forgot a dot in the IP, would have been nice to see the error before submitting, but I don't think HA does clientside checking (next to serverside) so it would not really help. Lets leave as is for now.

I merged it, any language updates can be done with separate PRs

FYI I finally made a release with these changes. Sorry it took so long, but I was working on some other things that I wanted to include, but I ran into some issues there so I finally decided to not delay further and release this feature.
Again thanks for the addition!

No worries! I’ve learned a lot of this integration to make the HomeWizard Energy integration. Great that I was able to do something back :)


Separate windows and unix code

Move unix specific code to unix only build files. Add stub implementation for Windows where needed. Remove syscall importing from platform-agnostic files.
This isn't perfect but support for devices and symlinks on Windows is still unclear to me. Also does not appear that there is any equivalent of UID and GID to set.

Can we add GOOS=windows make to .travis.yml?
@stevvooe please enable travis?

LGTM


Add as_curl

I borrowed as_curl method from 'yt' gem. and think it could be useful for debugging projects which use 'yt-core' gem. Thank you

Coverage decreased (-4.3%) to 95.726% when pulling caaf5c2f40f6936623044eb48692b20cfbdedfb9 on kangkyu:as_curl into 38c6892b8a9a853b50fc47ef748f7a64dfcbe882 on Fullscreen:master.

Coverage remained the same at 100.0% when pulling 6133e0193fdf6d0327bf7b7d542e85c896b443ca on kangkyu:as_curl into 38c6892b8a9a853b50fc47ef748f7a64dfcbe882 on Fullscreen:master.

closes in favor of #8


Bug 1731232: Report infrastructure provider metric from the operator

The infrastructure provider metric summarizes the desired type and
region (if any) to prometheus, based on the desired config. If type
is not recognized it is passed along, if it is empty it is set as
empty.

/assign @deads2k

/lgtm

/retest

/cherrypick release-4.2

/cherrypick release-4.1


[Agent] add models and message notebook

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (2961243) 70.70% compared to head (f33afa7) 70.70%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #249   +/-   ##
========================================
  Coverage    70.70%   70.70%
========================================
  Files           62       62
  Lines         3106     3106
========================================
  Hits          2196     2196
  Misses         910      910

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


nextjs 13

any plan to upgrade nextjs to version 13 ?

https://github.com/primefaces/sakai-react/pull/35

any plan to upgrade nextjs to version 13 ?

please look at the pull request, I've already pulled the request for changes in dependencies including next13 https://github.com/primefaces/sakai-react/pull/35

Has anyone fixed it for nextjs 13?

It is now available in Nextjs 13 as you can see. If you have any issues with Next 13 and Sakai template please open a new ticket. For now, I'm closing this."


Include Nikola app as a library user

Add Nikola App under the library users in the Readme file.

Coverage remained the same at 95.148% when pulling<PHONE_NUMBER>ecb20673f64199eb5966fbe8ada04d on geraldoramos:add-nikola-app into dafda514bc611b36d2a793b1e6b947973aba8821 on mseminatore:master.

@geraldoramos Thanks, happy to add it to the list!


[BUG] naive forecaster window error

Reference Issues/PRs
Fixes #1990.
What does this implement/fix? Explain your changes.
Increases padding width in the case that last_window wasn't the full window length. This occurs when the forecasting horizon starts in sample, as the window length is the full length of the data but there is not enough historical data to fill the window so I have added NA's in this case. There is precedent for this as NAs are added to satisfy a window length in the case that the window length isn't a multiple of sp.
What should a reviewer concentrate their feedback on?
Does this fix work theoretically. I am not sure if this breaks the functionality of the Naive forecaster.

@fkiraly are those tests now executed?

No, waiting for your review in #2022


Terminal icon is missing

Terminal icon from https://code.visualstudio.com/api/references/icons-in-labels is missing in this repo.

This is actually the same icon as console, it's only listed on our website as a reference from previous Octicon references:

Name
Icon
Links

console

Light / Dark

Let me know if this doesn't work for you.

Ah, thanks @misolori !


Upgrade quill to latest

There are a few new features and for me specifically a bug fix that I need, could you update to quill 1.3.0?
Thanks

Update to v2.2.6 please.


Fix NoSuchFieldException on <=ICS

kotlin has problems with inlining constants properly -> use numerical value

Thanks!

Released as v0.1.1.
http://search.maven.org/#artifactdetails|com.github.ksoichiro|androidpagecontrol|0.1.1|aar


Refactor command handling

Codecov Report

Merging #2 (e94c08c) into master (f61fc2e) will decrease coverage by 1.17%.
The diff coverage is 3.24%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
- Coverage   53.74%   52.57%   -1.18%
==========================================
  Files          26       32       +6
  Lines        1027     1048      +21
==========================================
- Hits          552      551       -1
- Misses        475      497      +22

Impacted Files
Coverage Δ

src/command/details.rs
0.00% <0.00%> (ø)

src/command/init.rs
0.00% <0.00%> (ø)

src/command/log.rs
0.00% <0.00%> (ø)

src/command/mod.rs
0.00% <0.00%> (ø)

src/command/refresh.rs
0.00% <0.00%> (ø)

src/command/submit.rs
0.00% <0.00%> (ø)

src/command/tasks.rs
0.00% <0.00%> (ø)

src/main.rs
2.94% <0.00%> (+2.09%)
:arrow_up:

src/error.rs
1.92% <20.00%> (+1.92%)
:arrow_up:

src/baca/details/language.rs
96.55% <83.33%> (-0.12%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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


Raw byte allocations via GC versus Pointer

The standard library sometimes uses GC.malloc and GC.malloc_atomic to allocate raw bytes, when Slice isn't appropriate. Some examples are:
https://github.com/crystal-lang/crystal/blob/0e821186d85c8a0142b496bbc884ab8d93ce36cd/src/log/metadata.cr#L35
https://github.com/crystal-lang/crystal/blob/0e821186d85c8a0142b496bbc884ab8d93ce36cd/src/string/builder.cr#L18
At other times it uses Pointer.malloc:
https://github.com/crystal-lang/crystal/blob/0e821186d85c8a0142b496bbc884ab8d93ce36cd/src/string/formatter.cr#L435
https://github.com/crystal-lang/crystal/blob/0e821186d85c8a0142b496bbc884ab8d93ce36cd/src/yaml/builder.cr#L37
The two may correspond to different allocators; an explanation is available in #12391. However, I believe this distinction is not that important because the two "different" allocators in this case are simply separate instances of the GC allocator. We should settle with one of those alternatives as a matter of consistency.
Personally I am in favor of the Pointer form, because IMO GC is an implementation detail and should be hidden away.
Note that Pointer(UInt8)#malloc is guaranteed to be atomic and Pointer(Void)#malloc is guaranteed to be non-atomic.

Note that Pointer(UInt8)#malloc is guaranteed to be atomic and Pointer(Void)#malloc is guaranteed to be non-atomic.

This however sounds quite a bit like a potential footgun. Would it make sense to make it more explicit?

I agree that all this duplicity seems unnecessary and confusing. Resolving this was previously proposed in https://github.com/crystal-lang/crystal/pull/12391#issuecomment-1217059181.
For moving to Pointer it would be nice to have a clearer API for atomic/non-atomic allocations. The implicit choice based on the element type is super useful for many use cases.
But I think in some use cases it can make sense to be explicit about it. You can force a different allocation method with Pointer(UInt8).malloc(sizeof(T) * size).as(T*) and Pointer(Void).malloc(sizeof(T) * size).as(T*), but that's not explicit at all.
Maybe adding just Pointer.malloc_atomic is good enough for that. It offers an easy way to explicitly choose atomic allocation. Of course it creates a difference between .malloc_atomic being always atomic and .malloc being either atomic or not depending on the type. This is a downside, but maybe acceptable?
Another downside is that it only allows explicit atomic allocation, but not explicit non-atomic allocation. I believe this variant is less relevant though, so it might be fine.
Another option could be adding a parameter to Pointer.malloc to determine whether allocation is supposed to be atomic. I would prefer different methods, though.

On the other hand, maybe Pointer.malloc is good enough as it is, and calling the lower level methods GC.malloc and GC.malloc_atomic directly is fine for a more explicit choice.
Maybe the actual problem is that GC.malloc and Pointer.malloc can end up using different instances of the GC.
If we ensure this doesn't happen, everything should be fine.

It's possible to have Pointer.malloc be auto atomic like now, Pointer.malloc_atomic to be always atomic, and Pointer.malloc_non_atomic to always be non-atomic.

If it is just about the interpreter, maybe we could expose Crystal::Type#has_inner_pointers? in the macro language instead. Then we could implement Pointer.malloc as a non-primitive method:
struct Pointer
  def self.malloc(size : UInt64)
    {% if T <= Reference && T.has_inner_pointers? %}
      GC.malloc(size * sizeof(T)).as(T*)
    {% else %}
      GC.malloc_atomic(size * sizeof(T)).as(T*)
    {% end %}
  end
end

I feel like I have either written or read that recently. But I don't recall. Maybe I had only intended to write about it...? 🤔
But yeah, there seems to be not much reason for Pointer.malloc to be a primitive when instead we can expose the reflection method that informs the allocation mechanism

I recall, the familiarity was from #13481.
Exposing has_inner_pointer? is a requirement for implementing .allocate in stdlib.


Include the newer Microsoft.PowerShell.ThreadJob module - fixes #19742

PR Summary
Fixes #19742  by updating the package reference file for modules pulled from the gallery to also include the renamed Microsoft.PowerShell.ThreadJob module
PR Context

PR Checklist

[ x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ x] Summarized changes
[ x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x ] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

looking into the failures now

Doing a quick search on GitHub there are no references to the module qualified ThreadJob\Start-ThreadJob.

The test failure is one that I've seen come up before and has nothing to do with this PR Set/New/Remove-Service cmdlet tests.Remove-Service can accept a ServiceController as pipeline input

I think there will need to be a minor update to docs for this PR

this pr renamed Microsoft.PowerShell.ThreadJob module ?
Will this cause duplicate command names In multiple versions of pwsh ?
What mechanism is there to ensure that 【Microsoft. PowerShell. ThreadJob】 will be used before 【ThreadJob】when automatically importing module?
What are the differences between the two modules?
Why are the two modules forked? Why not update on ThreadJob?

Maintainers would like the Committee to review this breaking change of renaming the module from ThreadJob to Microsoft.PowerShell.ThreadJob.

Currently waiting on this PR in the ThreadJob repo https://github.com/PowerShell/ThreadJob/pull/29 so can then push update to this PRT to resolve the conversation above by @adityapatwardhan

Still waiting on the committee to make a conclusion.

The @PowerShell/powershell-committee discussed this (6/28/23) and suggest that proper way to achieve this is as follows:

Create a new version of the ThreadJob module which proxies to the new Microsoft.PowerShell.ThreadJob  module

the proxy should include a warning message that the ThreadJob module will be deprecated and removed in PS 7.6

Include the new Microsoft.PowerShell.ThreadJob in the release

By taking step 1, we can avoid a breaking change and provide users a glide path to get off the old ThreadJob module which we would want to remove in PS7.6 (our next LTS after 7.4)
Implementation note:
The release YAML will need updating for the Microsoft.PowerShell.ThreadJob module for proper authorization
A new issue should be opened to track the removal of the ThreadJob module in PS7.6.

Just so that I am clear from the above
Currently we are waiting on a new ThreadJob release -> then can update this PR with that version number & inc the Microsoft.PowerShell.ThreadJob as currently proposed  & all's good for the time being?

@JamesWTruher - Just so that I am clear from the above
Currently we are waiting on a new ThreadJob release -> then can update this PR with that version number & inc the Microsoft.PowerShell.ThreadJob as currently proposed & all's good for the time being?

sort of.
We want to be sure that the current ThreadJob module is updated with a deprecation notice (and create a proxy from ThreadJob\Start-ThreadJob  to the Microsoft.PowerShell.ThreadJob\Start-ThreadJob cmdlet.
But essentially, we will ship the old ThreadJob module and the new Microsoft.PowerShell.ThreadJob module. If you must have the old behavior, you would be able to achieve this via: Import-Module -RequiredVersion x -Name ThreadJob but just typing Start-ThreadJob will auto-load the new Microsoft.PowerShell.ThreadJob module (which is why this is still technically a breaking change).
explicitly importing the new ThreadJob module should result in a warning message with the "this is going away, so you should use the Microsoft.PowerShell.ThreadJob module"

Thanks for clarifying @JamesWTruher that was how I understood it from your comment 🙂

@JamesWTruher A few questions:

but just typing Start-ThreadJob will auto-load the new Microsoft.PowerShell.ThreadJob module

How would we guarantee Microsoft.PowerShell.ThreadJob will be the one loaded when just typing Start-ThreadJob?

Who will be working on the proxy ThreadJob module? Is this PR blocked until the new proxy ThreadJob module is released? If so, I would rather to have this PR closed for now.

@JamesWTruher A few questions:

but just typing Start-ThreadJob will auto-load the new Microsoft.PowerShell.ThreadJob module

How would we guarantee Microsoft.PowerShell.ThreadJob will be the one loaded when just typing Start-ThreadJob?
Who will be working on the proxy ThreadJob module? Is this PR blocked until the new proxy ThreadJob module is released? If so, I would rather to have this PR closed for now given that we don't have an ETA.

Currently, autoloading loads the first module found, and Microsoft.PowerShell.ThreadJob will be found before ThreadJob
@SteveL-MSFT will have a more informed answer for this

Yes, this PR is blocked until someone can develop the ThreadJob proxy module (could be part of this PR as it's just a script module).

The proxy module shouldn't be developed in this PR, as the code should be in a different repo.
If this PR is blocked on that, then I will close this PR for now. Does it sound good to both of you?

James,
PowerShell will auto load Microsoft.PowerShell.ThreadJob because of the alphabetical order being above ThreadJob (M > T), correct?
Joseph Colvin
IT Coordinator
Warwick Public Schoolhttps://www.warwick.k12.nd.us/
School Phone:<PHONE_NUMBER>
My Cell:<PHONE_NUMBER>
My Office Phone:<PHONE_NUMBER> Ext. 328
“So you never know who you touch. You never know how or when you'll have an impact, or how important your example can be to someone else" ― Denzel Washington, A Hand to Guide Me
"When men speak ill of you, live so as nobody may believe them." ― Plato
“time, man....time is an illusion. The river flows at an uneven rate but we all drift along in it without noticing until we look back.” – John Gieser
Save a tree, think before you print.
@.?anonymous&ep=signature>                Book time to meet with @.?anonymous&ep=signature>

From: James Truher [MSFT] @.>
Sent: Monday, July 10, 2023 2:23 PM
To: PowerShell/PowerShell @.>
Cc: Joseph Colvin @.>; Mention @.>
Subject: Re: [PowerShell/PowerShell] Include the newer Microsoft.PowerShell.ThreadJob module - fixes #19742 (PR #19744)
@JamesWTruherhttps://github.com/JamesWTruher A few questions:
but just typing Start-ThreadJob will auto-load the new Microsoft.PowerShell.ThreadJob module

How would we guarantee Microsoft.PowerShell.ThreadJob will be the one loaded when just typing Start-ThreadJob?

Who will be working on the proxy ThreadJob module? Is this PR blocked until the new proxy ThreadJob module is released? If so, I would rather to have this PR closed for now given that we don't have an ETA.

Currently, autoloading loads the first module found, and Microsoft.PowerShell.ThreadJob will be found before ThreadJob

@SteveL-MSFThttps://github.com/SteveL-MSFT will have a more informed answer for this

—
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/PowerShell/pull/19744#issuecomment-1629588738, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZVVWZ7FWNU2KZZYV7PQRRTXPRJDNANCNFSM6AAAAAAY3GYGFI.
You are receiving this because you were mentioned.Message ID: @.***>

The proxy module shouldn't be developed in this PR, as the code should be in a different repo. If this PR is blocked on that, then I will close this PR for now. Does it sound good to both of you?

The proxy module is really only relevant as part of PS7 due to the breaking change of ThreadJob being included in PS7 prior to renaming to Microsoft.PowerShell.ThreadJob.  It's really only to provide compat for one (or two) release and provide a warning message then we can remove it.

But ThreadJob is a published module in PowerShell Gallery (v2.0.3 now). If we are going to have a proxy module for it, we'd better release it to PowerShell Gallery too, maybe as v2.1, so as formally deliver the message that this module is deprecating and Microsoft.PowerShell.ThreadJob should be one to use going forward.
Once the proxy module is published, PowerShell just need to pull it during the build, and then it's time to include Microsoft.PowerShell.ThreadJob module.

@daxian-dbw I agree it would make sense to have the ThreadJob module on PSGallery also indicate being deprecated and using Microsoft.PowerShell.ThreadJob instead.  In that case, I would also agree that proxy module would be done outside of this PR and repo since it's published to PSGallery.  It would make sense to have that proxy developed in https://github.com/powershell/threadjob

Opened https://github.com/PowerShell/ThreadJob/issues/30 to track the proxy module work. I will close this PR for now.


Adding bolt flag into the command line options

Currently bolt is setup as True by default, so Graph is built using bolt, and in Neo4j versions < 3.0 cycli stops working.
A new flag, -b --bolt has been added to the command line set as False by default.

cycli works in Neo4j < 3.0 because py2neo will just fall back to http. However I like the idea of making Bolt optional, because right now if you're on Neo4j >= 3.0 you're forced to use the protocol.

I definitely want it to default to true, though, so maybe change it to a --no-bolt flag?

Okay, I see now that it didn't work on Neo4j 2.0. cycli would start but would fail when you write a query. All that I needed to do was set the default bolt=None. py2neo will then auto-detect if it should use http or bolt.

I still think a --no-bolt flag would be useful, though, for people on Neo4j >= 3.0 who don't want to use bolt.

I've just submitted an alternative PR #66 with the changes @nicolewhite suggested

Enabled Bolt connection by default (with a --no-bolt opt-out)
Included a -B|--bolt-port flag that sets a non-default Bolt port when present


Ability to easily create Discord bot

User Story
A user should be able to use the Insitux CLI to quickly and easily create a Discord bot in a few steps.
Current workflow
At the moment one would have to extend the Insitux REPL with HTTP commands, and use sub-par dependency loading and management. There is no tutorials on how to do any of these things - the REPL is not documented.

please 🥺🥺

Partial HTTP support has been added, enabling the creation of basic Discord bots. Here's the first one ever: VSKB.
Further work:

fuller HTTP support (e.g. (HTTP :put {:headers {} :body {}})
ability to template projects with ix
a Discord library would be good


Pull req 105

Appears Jonathan has already submitted a pull request for this.  At any rate, there is another. -sk

Codecov Report

Merging #106 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #106   +/-   ##
=======================================
  Coverage   84.82%   84.82%
=======================================
  Files          31       31
  Lines        1858     1858
  Branches      264      264
=======================================
  Hits         1576     1576
  Misses        245      245
  Partials       37       37

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 e9d1ea5...a804a96. Read the comment docs.


Hyundai/EU user profile changes

Hi,
I am using this integration with a Hyundai IONIQ5.
Everything works fine, except that the user profile in the car is changed to the Bluelink user registered with the integration, every time it updates data from the car.
This doesn't happen when using the Bluelink app.
Is it possible to avoid user profile change, or is this just the way the integration works?

Would need sniffing of the traffic and comparing to figure this one out. Ideally have a dev from EU chime in on it.

Re-reading this when you say user profile do you mean language?  If so please try the "master" branch in HACS and let me know if it is solved.  It will need home assistant set to a language that the app supports.

No, with user profile I mean the selected Bluelink profile in the car that
holds settings for seat and mirror positions, phone connection etc.
ons. 21. des. 2022, 00:27 skrev cdnninja @.***>:

Re-reading this when you say user profile do you mean language? If so
please try the "master" branch in HACS and let me know if it is solved. It
will need home assistant set to a language that the app supports.
—
Reply to this email directly, view it on GitHub
https://github.com/Hyundai-Kia-Connect/kia_uvo/issues/360#issuecomment-1360462601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKIRVRQLZGDVKXJ4M3LRON3WOI6HFANCNFSM6AAAAAAQSO5PHU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Could you please clarify, where you see this profile information. If you have a debug log this would also help.


Can't connect to server

I cannot connect to server.
Server version: Opendistro 1.7.0
ODBC client version: 0.2.0
Error: Failed to Establish Connection to DB
Windows: Windows Server 2012R2

Log:
mylog.c[InitializeLogging]531:  es_odbc.c[DllMain]111: dlg_specif[CC_conninfo_init]452:    setup.c[ParseAttributes]575: dlg_specif[copyConnAttributes]145: dlg_specif[CC_conninfo_init]452: dlg_specif[getDSNinfo]224:    setup.c[ParseAttributes]575: dlg_specif[copyConnAttributes]145:    setup.c[test_connection]431: odbcapi30.[SQLAllocHandle]33:  environ.c[ESAPI_AllocEnv]44:  environ.c[ESAPI_AllocEnv]60: odbcapi30.[SQLSetEnvAttr]431: odbcapi30.[SQLAllocHandle]33: connection[ESAPI_AllocConnect]63: connection[ESAPI_AllocConnect]66:  environ.c[EN_add_connection]502:  environ.c[EN_add_connection]531: odbcapiw.c[SQLDriverConnectW]122: win_unicod[ucs2_to_utf8]126:  newlen=215 olen=215 utf8str=DRIVER=elasticsearchodbc;server=hostname.hostname.biz;database=elasticsearch;port=9200;UID=admin;PWD=admin;auth=NONE;region=;useSSL=1;hostnameVerification=0;logLevel=5;logOutput=C:\;responseTimeout=10;fetchSize=0;
    misc.c[make_string]115:     misc.c[make_string]117: dlg_specif[CC_conninfo_init]452:  drvconn.c[dconn_get_attributes]223: dlg_specif[getDSNinfo]224:  drvconn.c[dconn_get_attributes]223: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]121: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145:    mylog.c[logs_on_off]234:    mylog.c[logs_on_off]234: 	start_logging:Global.debug&commlog=5&5
   mylog.c[InitializeLogging]531: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: connection[CC_set_error_statements]553: connection[CC_log_error]641: connection[CC_log_error]644: connection[CC_log_error]647: connection[CC_log_error]650: es_communi[ESCommunication::LogMsg]474: connection[CC_log_error]641: connection[CC_log_error]644: connection[CC_log_error]647: connection[CC_log_error]650: odbcapi30w[SQLGetDiagRecW]197: es_api30.c[ESAPI_GetDiagRec]39:  environ.c[ESAPI_ConnectError]228: connection[CC_get_error]618: connection[CC_get_error]630:  environ.c[ESAPI_ConnectError]245:  environ.c[ESAPI_ConnectError]332: es_api30.c[ESAPI_GetDiagRec]60: win_unicod[utf8_to_ucs2_lf]244:  string=elasticsearch connection status was not CONNECTION_OK: Failed to establish connection to DB. ocount=92
odbcapi30.[SQLFreeHandle]219: connection[ESAPI_FreeConnect]207: connection[CC_Destructor]314: connection[CC_cleanup]413: connection[CC_set_error_statements]553: connection[CC_log_error]641: connection[CC_log_error]644: connection[CC_log_error]647: connection[CC_log_error]650: connection[CC_cleanup]428: dlg_specif[CC_conninfo_init]452: connection[CC_cleanup]494: connection[CC_Destructor]321: connection[CC_Destructor]332: connection[CC_Destructor]343: connection[ESAPI_FreeConnect]225: odbcapi30.[SQLFreeHandle]219:  environ.c[ESAPI_FreeEnv]69:  environ.c[EN_Destructor]442:  environ.c[EN_Destructor]465:  environ.c[EN_Destructor]477:  environ.c[ESAPI_FreeEnv]72:    setup.c[test_connection]431: odbcapi30.[SQLAllocHandle]33:  environ.c[ESAPI_AllocEnv]44:  environ.c[ESAPI_AllocEnv]60: odbcapi30.[SQLSetEnvAttr]431: odbcapi30.[SQLAllocHandle]33: connection[ESAPI_AllocConnect]63: connection[ESAPI_AllocConnect]66:  environ.c[EN_add_connection]502:  environ.c[EN_add_connection]531: odbcapiw.c[SQLDriverConnectW]122: win_unicod[ucs2_to_utf8]126:  newlen=216 olen=216 utf8str=DRIVER=elasticsearchodbc;server=hostname.hostname.biz;database=elasticsearch;port=9200;UID=admin;PWD=admin;auth=BASIC;region=;useSSL=1;hostnameVerification=0;logLevel=5;logOutput=C:\;responseTimeout=10;fetchSize=0;
    misc.c[make_string]115:     misc.c[make_string]117: dlg_specif[CC_conninfo_init]452:  drvconn.c[dconn_get_attributes]223: dlg_specif[getDSNinfo]224:  drvconn.c[dconn_get_attributes]223: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]121: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145:    mylog.c[logs_on_off]234:    mylog.c[logs_on_off]234: 	start_logging:Global.debug&commlog=5&5
   mylog.c[InitializeLogging]531: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: connection[CC_set_error_statements]553: connection[CC_log_error]641: connection[CC_log_error]644: connection[CC_log_error]647: connection[CC_log_error]650: es_communi[ESCommunication::LogMsg]474: connection[CC_log_error]641: connection[CC_log_error]644: connection[CC_log_error]647: connection[CC_log_error]650: odbcapi30w[SQLGetDiagRecW]197: es_api30.c[ESAPI_GetDiagRec]39:  environ.c[ESAPI_ConnectError]228: connection[CC_get_error]618: connection[CC_get_error]630:  environ.c[ESAPI_ConnectError]245:  environ.c[ESAPI_ConnectError]332: es_api30.c[ESAPI_GetDiagRec]60: win_unicod[utf8_to_ucs2_lf]244:  string=elasticsearch connection status was not CONNECTION_OK: Failed to establish connection to DB. ocount=92

@jordanw-bq @rupal-bq can you take a look.

I will update the log. It's the wrong one. One minute.

We don't support Windows Server 2012R2 as of yet. The only supported version for Windows is Windows 10
https://github.com/opendistro-for-elasticsearch/sql-odbc#supported-versions

does that make a big difference?

If I compile from source, would it work?

@coderReview We have not had a chance to verify the driver with the Windows Server platforms so far. If you provide the updated logs, we might be able to help provide a direction forward.
Could you also tell us what application (name and bitness (32 or 64 bit)) is using the driver?

Actually I was using ODBC admin tool test button.
This is the configuration.

Error

Log:
   mylog.c[logs_on_off]234: 	start_logging:Global.debug&commlog=5&5
   mylog.c[InitializeLogging]531:  es_odbc.c[DllMain]111:    setup.c[ParseAttributes]575: dlg_specif[copyConnAttributes]145:  es_odbc.c[DllMain]118:    mylog.c[logs_on_off]234: 	start_logging:Global.debug&commlog=5&5
   mylog.c[InitializeLogging]531:  es_odbc.c[DllMain]111:    setup.c[ParseAttributes]575: dlg_specif[copyConnAttributes]145: dlg_specif[getDSNinfo]224:    setup.c[ParseAttributes]575: dlg_specif[copyConnAttributes]145:  es_odbc.c[DllMain]118:    mylog.c[logs_on_off]234: 	start_logging:Global.debug&commlog=5&5
   mylog.c[InitializeLogging]531:  es_odbc.c[DllMain]111:    setup.c[ParseAttributes]575: dlg_specif[copyConnAttributes]145: dlg_specif[getDSNinfo]224:    setup.c[ParseAttributes]575: dlg_specif[copyConnAttributes]145:    setup.c[test_connection]431: connection[ESAPI_AllocConnect]66:  environ.c[EN_add_connection]531: win_unicod[ucs2_to_utf8]126:  newlen=216 olen=216 utf8str=DRIVER=elasticsearchodbc;server=hostname.hostname.biz;database=elasticsearch;port=9200;UID=admin;PWD=admin;auth=BASIC;region=;useSSL=1;hostnameVerification=0;logLevel=7;logOutput=C:\;responseTimeout=10;fetchSize=0;
    misc.c[make_string]115:     misc.c[make_string]117:  drvconn.c[dconn_get_attributes]223: dlg_specif[getDSNinfo]224:  drvconn.c[dconn_get_attributes]223: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]121: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145: dlg_specif[copyConnAttributes]145:    mylog.c[logs_on_off]234:    mylog.c[logs_on_off]234: 	start_logging:Global.debug&commlog=5&5
   mylog.c[InitializeLogging]531: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: es_communi[ESCommunication::LogMsg]474: connection[CC_set_error_statements]553: connection[CC_log_error]641: connection[CC_log_error]644: connection[CC_log_error]647: connection[CC_log_error]650: es_communi[ESCommunication::LogMsg]474: connection[CC_log_error]641: connection[CC_log_error]644: connection[CC_log_error]647: connection[CC_log_error]650: odbcapi30w[SQLGetDiagRecW]197: es_api30.c[ESAPI_GetDiagRec]39:  environ.c[ESAPI_ConnectError]228: connection[CC_get_error]618: connection[CC_get_error]630:  environ.c[ESAPI_ConnectError]245:  environ.c[ESAPI_ConnectError]332: es_api30.c[ESAPI_GetDiagRec]60: win_unicod[utf8_to_ucs2_lf]244:  string=elasticsearch connection status was not CONNECTION_OK: Failed to establish connection to DB. ocount=92
odbcapi30.[SQLFreeHandle]219: connection[ESAPI_FreeConnect]207: connection[CC_Destructor]314: connection[CC_cleanup]413: connection[CC_set_error_statements]553: connection[CC_log_error]641: connection[CC_log_error]644: connection[CC_log_error]647: connection[CC_log_error]650: connection[CC_cleanup]428: dlg_specif[CC_conninfo_init]452: connection[CC_cleanup]494: connection[CC_Destructor]321: connection[CC_Destructor]332: connection[CC_Destructor]343: connection[ESAPI_FreeConnect]225: odbcapi30.[SQLFreeHandle]219:  environ.c[ESAPI_FreeEnv]69:  environ.c[EN_Destructor]442:  environ.c[EN_Destructor]465:  environ.c[EN_Destructor]477:  environ.c[ESAPI_FreeEnv]72:  es_odbc.c[DllMain]118:    mylog.c[logs_on_off]234: 	start_logging:Global.debug&commlog=5&5
   mylog.c[InitializeLogging]531:  es_odbc.c[DllMain]111:    setup.c[ParseAttributes]575: dlg_specif[copyConnAttributes]145: dlg_specif[getDSNinfo]224:    setup.c[ParseAttributes]575: dlg_specif[copyConnAttributes]145:  es_odbc.c[DllMain]118:    mylog.c[logs_on_off]234: 	start_logging:Global.debug&commlog=5&5
   mylog.c[InitializeLogging]531:  es_odbc.c[DllMain]111: dlg_specif[getDSNinfo]224:  es_odbc.c[DllMain]118:    mylog.c[logs_on_off]234: 	start_logging:Global.debug&commlog=5&5
   mylog.c[InitializeLogging]531:  es_odbc.c[DllMain]111:    setup.c[ParseAttributes]575: dlg_specif[copyConnAttributes]145: dlg_specif[getDSNinfo]224:    setup.c[ParseAttributes]575: dlg_specif[copyConnAttributes]145:  es_odbc.c[DllMain]118:

I would like to use this with Power BI. Any hints on how to configure would help as well

Server response from Postman
{
  "name": "hostname",
  "cluster_name": "cluster",
  "cluster_uuid": "O7PI9F64QQSWIKmAYDUw8g",
  "version": {
    "number": "7.6.1",
    "build_flavor": "oss",
    "build_type": "rpm",
    "build_hash": "aa751e09be0a5072e8570670309b1f12348f023b",
    "build_date": "2020-02-29T00:15:25.529771Z",
    "build_snapshot": false,
    "lucene_version": "8.4.0",
    "minimum_wire_compatibility_version": "6.8.0",
    "minimum_index_compatibility_version": "6.0.0-beta1"
  },
  "tagline": "You Know, for Search"
}

@jordanw-bq let me know if you need more information

I am seeing the same behavior, but on Windows 10

@coderReview @nikersason Is the OpenDistro SQL Plugin installed on your server? The ODBC driver requires this in order to execute SQL queries against your OpenDistro data.
You can check this using the CAT API.
https://opendistro.github.io/for-elasticsearch-docs/docs/elasticsearch/catapis/#plugins

hi @jordanw-bq all plugins, including SQL, are installed. I've done it using YUM command.

@jordanw-bq I also confirmed using the cat api that I have the opendistro_sql v<IP_ADDRESS> component installed. I spun up a local Open Distro for Elasticsearch Docker instance to try to narrow down the issue and can't connect to that (same error). Thank you for looking into this, and let me know if you have any other troubleshooting advice or need more information.

@coderReview @nikersason It looks like the installer included as part of the <IP_ADDRESS> GitHub release is incorrect, since the driver and installer files are showing a pre-release version (0.2.0). We will investigate the cause of this and update you when the proper installers are available.

@coderReview @nikersason We have updated the installer packages, which should now install the correct version of the driver. Can you uninstall your current version of the driver, and try the updated installers?
https://github.com/opendistro-for-elasticsearch/sql-odbc/releases/tag/v<IP_ADDRESS>

@jordanw-bq - I've uninstalled the old version and installed 1.7, and unfortunately am seeing the same results. Any other ideas or things I should look at?
@coderReview - Have you had success?

@jordanw-bq - I am seeing the following in my Elasticsearch logs at the time of connection attempts:
Exception during establishing a SSL connection: io.netty.handler.ssl.NotSslRecordException: not an SSL/TLS record

I've tried testing this from Windows ODBC Data Source Administrator (checking the Enable SSL option), and on MacOS through iODBC Administrator as well as iodbctest on the command line with a connection string that includes UseSSL=1. Have tried these with and without HostnameVerification.
Any ideas based on this information?

Having the same problem as well @nikersason @jordanw-bq

@coderReview When you tested the Server response from Postman, What URL did you use?

hello @rupal-bq, I've used https://hostname:9200/
hostname => my server's name

Also, I am using Self-Signed certificates.

@coderReview Can you try setting DSN with host => https://hostname, port =>9200. I think there might be a problem with the SSL config.

@abbashus, Would you please create a Window version of installer for 1.8 version of odbc drivier?

@jzhou3 You can download from here

@abbashus Thank you for sharing the installer file. Our AWS authentication is AWS Signed V4 using KeyID and SecretKey.  I created the aws-auth connection property credentials files under C:\Users[user name].aws based on reading on (https://docs.aws.amazon.com/sdk-for-cpp/v1/developer-guide/credentials.html),  I am getting the error: "elasticsearch connection status was not CONNECTION_OK: Failed to establish connection to DB."  Any suggestions?

@jzhou3 Our documentation will be updated to include this note, but the credentials for AWS authentication with ODFE need to be in a specific profile in your credentials file, named elasticsearchodbc.
Example .aws/credentials file:
[default]
aws_access_key_id = xxx
aws_secret_access_key = xxx

[elasticsearchodbc]
aws_access_key_id=<ODFE KeyID>
aws_secret_access_key=<ODFE SecretKey>

@rupal-bq I have tried setting DSN with host => https://hostname, port =>9200 as you suggested to @coderReview , with no success. I no longer see "not an SSL/TLS record" in my ES logs but still get "Failed to establish connection" in my ODBC logs. Have you been able to connect successfully over SSL?

@nikersason I will try today as well.

@rupal-bq using https didn't work. Without https I am getting the same exception as @nikersason.

do you have a working example to provide?

@coderReview @nikersason I am not to reproduce the issue. Can you use the installer (https://github.com/opendistro-for-elasticsearch/sql-odbc/actions/runs/122823780) and set the log level to LOG_ALL and share logs. This will add more information in logs.
Also, please share the steps you followed to setup the server and use the driver, it would help to reproduce the issue.

@rupal-bq same problem. I am using a Self Signed Certificate. Can that cause any issue?

I will share logs ASAP.

It tries to connect to a database called elasticsearch. What is that?

that database option isn't getting used while connecting since elasticsearch server doesn't support multiple database/catalog. I am not entirely sure if Self Signed Certificate is causing any problem, logs will have a server response, which will help us identify the problem.

In my case, I am trying an OpenDistro ElasticSearch Docker container using instructions here: https://opendistro.github.io/for-elasticsearch-docs/docs/install/docker/

mylog_odbcad32_25160.log
I see an error in the logs indicating the SQL Plugin must be installed, but I confirmed opendistro_sql is installed using /_cat/plugins request.
I also still see io.netty.handler.codec.DecoderException: io.netty.handler.ssl.NotSslRecordException: not an SSL/TLS record in my ElasticSearch logs.

I am trying to get some info using SSLDUMP

I tried connecting to an AWS ElasticSearch Service Endpoint and was successful now, provided I include https in the hostname.
I wonder if @coderReview is onto something with the self signed cert question.

There is no SSL handshaking happening here. @nikersason are you using Self Signed certificates as well?

@coderReview - looks like we commented at the same time. I was able to connect successfully to an AWS endpoint. Indeed my local instance is using self-signed certificates

@coderReview @nikersason Thanks for sharing all information. I was able to reproduce the issue with it.

@rupal-bq for the tableau connection readme section, it says "Make sure the .tdc is copied to <user_home_directory>/Documents/My Tableau Repository/Datasources in both macOS and Windows." i only seeing a .tcd file in the source folder. Is that a typo? tcd vs. tdc

@jzhou3 It isn't a typo. The .tcd file is part of the tableau connector and the .tdc file you want here is the one available in your resources folder after installation. https://github.com/opendistro-for-elasticsearch/sql-odbc/blob/master/src/installer/Resources/odfe_sql_odbc.tdc

@coderReview @nikersason Disabling HostnameVerification isn't working on windows currently(https://github.com/aws/aws-sdk-cpp/pull/492). As the demo certs are not in the trust store, SSL certificate verification is failing.
We are working on a solution. Meanwhile, if you want to use SSL on windows, add certificates to the trust store.
Also if you are using SSL, you need to add https in the host. (eg. https://localhost)

@rupal-bq added certificate to host but still got error:
dlg_specif[copyConnAttributes]145: key='DRIVER' value='elasticsearchodbc'
dlg_specif[copyConnAttributes]145: key='server' value='https://HOSTNAME'
dlg_specif[copyConnAttributes]145: key='database' value='elasticsearch' not found
dlg_specif[copyConnAttributes]145: key='port' value='9200'
dlg_specif[copyConnAttributes]145: key='UID' value='admin'
dlg_specif[copyConnAttributes]121: key='PWD' value='xxxxxxxx'
dlg_specif[copyConnAttributes]145: key='auth' value='BASIC'
dlg_specif[copyConnAttributes]145: key='region' value=''
dlg_specif[copyConnAttributes]145: key='useSSL' value='1'
dlg_specif[copyConnAttributes]145: key='hostnameVerification' value='1'
dlg_specif[copyConnAttributes]145: key='logLevel' value='7'
dlg_specif[copyConnAttributes]145: key='logOutput' value='C:\'
dlg_specif[copyConnAttributes]145: key='responseTimeout' value='10'
dlg_specif[copyConnAttributes]145: key='fetchSize' value='-1'
   mylog.c[logs_on_off]234: mylog_on=7 qlog_on=7
   mylog.c[logs_on_off]234: mylog_on=7 qlog_on=7
	start_logging:Global.debug&commlog=7&7
   mylog.c[InitializeLogging]531: Log Output Dir: C:\
es_communi[ESCommunication::LogMsg]571: Initializing Aws API.
es_communi[ESCommunication::LogMsg]571: Verifying connection options.
es_communi[ESCommunication::LogMsg]571: Required connection option are valid.
es_communi[ESCommunication::LogMsg]571: Starting DB connection.
es_communi[ESCommunication::LogMsg]571: Attempting to establish DB connection.
es_communi[ESCommunication::LogMsg]571: Checking for SQL plugin
es_communi[ESCommunication::LogMsg]571: The SQL plugin must be installed in order to use this driver. Client error: 'Encountered network error when sending http request'.
es_communi[ESCommunication::LogMsg]571: Failed to establish connection to DB.
es_communi[ESCommunication::LogMsg]571: Dropping DB connection.
connection[CC_set_error_statements]553: entering self=00000001127F9250
connection[CC_log_error]641: CONN ERROR: func=LIBES_connect, desc='', errnum=202, errmsg='elasticsearch connection status was not CONNECTION_OK: Failed to establish connection to DB.'
connection[CC_log_error]644:             ------------------------------------------------------------
connection[CC_log_error]647:             henv=00000001127D7130, conn=00000001127F9250, status=0, num_stmts=16
connection[CC_log_error]650:             esconn=0000000000000000, stmts=00000001127FAD50, lobj_type=-999
es_communi[ESCommunication::LogMsg]571: Shutting down Aws API.
connection[CC_log_error]641: CONN ERROR: func=ESAPI_DriverConnect, desc='Error from CC_Connect', errnum=202, errmsg='elasticsearch connection status was not CONNECTION_OK: Failed to establish connection to DB.'
connection[CC_log_error]644:             ------------------------------------------------------------
connection[CC_log_error]647:             henv=00000001127D7130, conn=00000001127F9250, status=0, num_stmts=16
connection[CC_log_error]650:             esconn=0000000000000000, stmts=00000001127FAD50, lobj_type=-999
odbcapi30w[SQLGetDiagRecW]197: entering
es_api30.c[ESAPI_GetDiagRec]39: entering type=2 rec=1
 environ.c[ESAPI_ConnectError]228: entering hdbc=00000001127F9250 <512>
connection[CC_get_error]618: entering
connection[CC_get_error]630: leaving
 environ.c[ESAPI_ConnectError]245: CC_get_error: status = 202, msg = #elasticsearch connection status was not CONNECTION_OK: Failed to establish connection to DB.#
 environ.c[ESAPI_ConnectError]332: 	     szSqlState = '08001',len=92, szError='elasticsearch connection status was not CONNECTION_OK: Failed to establish connection to DB.'
es_api30.c[ESAPI_GetDiagRec]60: leaving 0
win_unicod[utf8_to_ucs2_lf]244: ilen=92 bufcount=512 string=elasticsearch connection status was not CONNECTION_OK: Failed to establish connection to DB. ocount=92
odbcapi30.[SQLFreeHandle]219: entering
connection[ESAPI_FreeConnect]207: entering...hdbc=00000001127F9250
connection[CC_Destructor]314: entering self=00000001127F9250
connection[CC_cleanup]413: entering self=00000001127F9250
connection[CC_set_error_statements]553: entering self=00000001127F9250
connection[CC_log_error]641: CONN ERROR: func=CC_cleanup, desc='', errnum=1, errmsg='Connection not open'
connection[CC_log_error]644:             ------------------------------------------------------------
connection[CC_log_error]647:             henv=00000001127D7130, conn=00000001127F9250, status=0, num_stmts=16
connection[CC_log_error]650:             esconn=0000000000000000, stmts=00000001127FAD50, lobj_type=-999
connection[CC_cleanup]428: after LIBES_disconnect
dlg_specif[CC_conninfo_init]452: entering opt=1
connection[CC_cleanup]494: leaving
connection[CC_Destructor]321: after CC_Cleanup
connection[CC_Destructor]332: after free statement holders
connection[CC_Destructor]343: leaving
connection[ESAPI_FreeConnect]225: leaving...
odbcapi30.[SQLFreeHandle]219: entering
 environ.c[ESAPI_FreeEnv]69: entering env=00000001127D7130
 environ.c[EN_Destructor]442: entering self=00000001127D7130
 environ.c[EN_Destructor]465: clearing conns count=128
 environ.c[EN_Destructor]477: leaving rv=1
 environ.c[ESAPI_FreeEnv]72:    ok
 es_odbc.c[DllMain]118: DETACHING odfesqlodbc.dll

@coderReview  Are you able to send a request from Postman successfully with SSL certificate verification ON and without adding a certificate?
Currently, Disabling hostname verification isn't working on windows which is equivalent to SSL certificate verification in Postman so it will be ON by default. Also, we can't specify a certificate while connecting to Elasticsearch yet.

Hi @rupal-bq does postman use the windows certificate manager? The same for the AWS library. I've added the self signed certificates to the windows trust store.

I will try tomorrow

Hello,
it seems I have the same issue there with :

Opendistro 1.9
Windows 10 odbc driver (64 bit)
The exact same error message @coderReview

@rupal-bq : hello, you talked earlier about a GET Postman request to try. Can you please tell me which request you want to try?
Best regards,

@vincent2mots Disabling hostname verification isn't working on Windows and the driver doesn't provide a way to specify the certificate so check if you are able to send a curl request from Verify the install without --insecure.
Alternatively, you can use Postman to send request but make sure File > Settings > General > SSL ceritificate verification is ON and File > Settings > Certificates doesn't have any certificate.
This will replicate the request which driver is sending to the SQL plugin.

I will try this tomorrow! I'll tell you if it works ☺

hi @rupal-bq Self Signed certificates won't work with postman even if added to windows certmgr. The only workaround I can see is disable HTTPs in Elasticsearch configuration

Also, @rupal-bq  can you check this code https://github.com/aws/aws-sdk-cpp/blob/74459ca66ab5867cab19458d9e70a7f6ced7f2ad/aws-cpp-sdk-core/source/http/windows/WinHttpSyncHttpClient.cpp#L160
Seems verification can be turned off. Don't know how to test it though.

@coderReview Thanks for finding this. I can probably test this with an independent program that uses the AWS SDK.

@rupal-bq any update? Also can we re-open the issue. thank you.

@coderReview Sorry for the late response. verifySSL wasn't working with the independent test, it was returning response code REQUEST_NOT_MADE.
The alternative fix will be Overriding HTTP Client and use curl instead of WinHTTP.
Please raise an issue on the new repository here

Done. https://github.com/opendistro-for-elasticsearch/sql/issues/686

if you can help me setting up the build environment I can help taking a look at this.

@coderReview Sure. That will be great.
All you need to do for setting up a build environment is

add dependencies
get source code
run script build_win_<debug/release><32/64>.ps1 from Developer PowerShell for VS. After this you can open global_make_list.sln in Visual Studio.
set dsn & dataset for running tests

Check Build Instructions for more details.
Let me know if you have any questions.


Does not render locally

Describe the bug
Hi,
It does not render on local SD installation. When I click "Render / Render Image" it renders using default EEVEE. No errors in both consoles.
Just in case "Render AI" tab also does not showup like in videos.
To reproduce
Just install as described in Documentation, make your prompt and run "Render / Render Image".
Error log
No logs
Environment

Blender version: 4.0
AI Render version: 1.0.1
Operating system: Windows

Screenshots/video
No response
Additional information
No response

Hey @CRYOMEN, sorry to hear that it's not working for you. Thanks for mentioning that there's nothing in either error log. That's definitely odd. Assuming that you still have the "run automatically" checkbox checked (under the "Operation" tab in AI Render), it should render or at the very least show an error.
Can you try switching to DreamStudio in AI Render's preferences and see if it gives you an error. (You don't even need to sign up for it or get a key. Just try rendering without it and you should at least get a message).

Hey @CRYOMEN, sorry to hear that it's not working for you. Thanks for mentioning that there's nothing in either error log. That's definitely odd. Assuming that you still have the "run automatically" checkbox checked (under the "Operation" tab in AI Render), it should render or at the very least show an error.
Can you try switching to DreamStudio in AI Render's preferences and see if it gives you an error. (You don't even need to sign up for it or get a key. Just try rendering without it and you should at least get a message).

Oh! "run automatically on render" was not checked. Now its works as expected. Thank you a lot!

Oh great - I'm glad it was an was an easy fix!


Is this still viable?

I'm a longtime Windows Mobile user, and I have a few 950/XLs I'd like to flash if this works well enough to be used as a phone OS...but I can't find any up-to-date info on compatibility. Does the LTE radio work reliably? What about SMS?

Yes, it's viable and up to date. Please refer to the driver project here and read more :) https://github.com/WOA-Project/Lumia-Drivers.
This project is the Deployer, that is the tool to deploy Windows On ARM 64 to the supported devices, on of which is the Lumias 950s.
Thanks for your interest!


Doy fix comment

Hi, I'm not absolutely certain about this change, but based on what I've read, I think I'm right...
The docs show:
http://momentjs.com/docs/#/i18n/
    week : {
        dow : 1, // Monday is the first day of the week.
        doy : 4  // The week that contains Jan 4th is the first week of the year.
    }

which, afaict means that the 4 as value for doy should correspond to the commented Jan 4th.
A significant portion of the locales get this right.
Sadly, the file that feels like a template doesn't match the documentation and a handful of locales kept that.
If you agree that the template like file is incorrect, I'm happy to @ mention the locale owners to get confirmation about fixing their comments. OTOH, if I'm absolutely wrong about what the comment means, I'd sooner not disturb your localizers.

Coverage decreased (-0.2%) to 94.449% when pulling 7f2cba43acc1ced0e410e8bb76bba495e9cb90cd on jsoref:doy-fix-comment into 2e2a5b35439665d4b0200143d808a7c26d6cd30f on moment:develop.

npm ERR! network request to https://registry.npmjs.org/arr-union/-/arr-union-3.1.0.tgz failed, reason: getaddrinfo ENOTFOUND registry.npmjs.org registry.npmjs.org:443
npm ERR! network This is a problem related to network connectivity.

This isn't my fault... I'm not sure how to beg travis to try again...

Thanks for this.


New feature: being able to execute plugins

I want Aqueduct being able to execute plugin-like code. This includes:

having infrastructure (GraphQL endpoints) to store and list plugins
having code to execute a particular plugin function (also, GraphQL endpoint).
having simple examples to implement my own plugins.

CTRL-2732


Move shared tools (eslint, babel, tslint, prettier) and base configs into tools package

This was originally a task in story #8185.

@justinbarry -- I think this problem was solved in a different way.
Re-open and lets schedule outside of this epic if you disagree.

@andrewdamelio It is possible that this can be done by moving the eslit/prettier rc files to the root on the mono repo.

Totally


Permission denied on package.json

I'm adding projen to my CDK construct library using the ConstructLibraryAws project. As my construct includes a NodeJsFunction, it fails during tests as it can't access the package.json file. If I change the permissions it works until they are resetted by npx projen.

EACCES: permission denied, open '.../package.json'

Solved it. The root cause was that I forgot the /src subfolder to be part of the entry file of the NodejsFunction. Because of that the parcel bundler thought it's in root and started to create it's temproary package.json file. That failed because the existing one was readonly. Now, the temporary package.json is created in /src folder where also the lambda code is located.

I solved it by moving my lambda code in a subfolder and adding a separate package.json. The problem is that NodejsFunction looks up the path for the next available package.json and modifies it for the bundling process. That's not possible with the package.json from root folder as it's write protected.
That means using a lambda function as described here is not possible.

@eladb yes, this is https://github.com/aws/aws-cdk/issues/9673#issuecomment-673999457
Would it be possible to provide an option so that projen doesn't make the package.json read-only?

I am changing package.json to be read/write as part of https://github.com/eladb/projen/pull/91


Optimize ByteArrayNodeFactory and CharArrayNodeFactory for "dense" trees

This request may also apply to other NodeFactory implementations, but so far I have only been using the ByteArrayNodeFactory and CharArrayNodeFactory implementations.
In my use case I build a tree for 1.5M+ mappings resulting in a RadixTree with 1.9M+ nodes. Of these nodes more than 33% have an incoming edge containing only a single character. Wrapping that single byte (or char) in an array incurs a rather big overhead: 4 bytes for the reference + 16 bytes for the array (12 object overhead, 1 byte payload, 3 bytes of padding). So a total of 20 bytes vs. a single byte (or two bytes for CharArrayNodeFactory). Depending on the padding required for the concrete node implementation I suppose one could save between 16 and 20 bytes of memory, when dealing with nodes with a single character in the incoming edge. For my particular tree that makes quite a difference.
Since I can supply my own NodeFactory implementation I can of course extend the mentioned implementations accordingly. I was just wondering if this improvement may be of general interest.

Hi Knut,
This sounds useful. If you could create a PR which adds the additional node type for each of the factories then it sounds like a useful inclusion!

Hi Niall,
DefaultByteArrayNodeFactory currently already returns one of six different implementations. This number would now go up to twelve. (Any more and I think it might make sense to generate the classes.) I am thinking of using the same naming scheme, but replace the prefix ByteArray with SingleByte. The alternative would be to have the prefix Byte only. Sounds good to you?
Would you want to also implement this for the DefaultCharArrayNodeFactory?
Detail: I noticed the DefaultByteArrayNodeFactory and its implementation classes refer to UTF-8 in the Javadoc. But AFAIK any code point beyond 0x9F needs two bytes when encoded using UTF-8. So I think referring to the ISO-8859-1 encoding would be more appropriate.

This has become quite a heavy proposal :/
I was originally thinking we might only have one new node implementation per type of NodeFactory, but I see now we'd need 12 new classes. It's a lot of code for a minor improvement.
I'm thinking that for a version 3.x of this project I might add support to store the trees on disk, or in off-heap memory. So this would reduce the need to optimize for every single byte of memory usage on the heap. Is the current memory usage a big problem, or do you think you could wait for off-heap/disk support in future?

I almost expected you to consider this change a bit heavyweight for what it achieves. That is perfectly understandable and not any problem for me, since I can easily supply my own NodeFactory as it is.
What you say with regards to off-heap memory sounds very interesting! Especially since the data I am currently indexing is already off-heap. I will be interested to see how you plan to organize the data in memory, because I have personally also already considered an off-heap implementation for RadixTree. Also, do you already have a target release date for 3.0?
If Java 9 comes with Value Objects (see e.g. https://www.sitepoint.com/what-java-might-one-day-look-like/) it will be interesting to see how many applications still actually benefit from off-heap storage.

So as discussed, I will close this issue for now.
Let's hope value objects will actually be useful in future!


Karate summary reports not showing all tested features

Hi Team,
Karate summary reports not showing all tested features, it is updated the latest executions in the karate-summary.html though all the executions are created each individual html files.
I have two features print1.feature and print2.feaure, when run the test with latest karate version 1.4.1, it is generating the karate-summary.html report with print2.feature only instead of both the features.
Same thing I have checked with karate version 0.9.6, it is capturing both the features report in karate-summary.html file. Please find the attachment. Please provide the solution, if it is available.
Thanks in advance.

@govindd06 please follow the instructions EXACTLY as given here to re-open: https://github.com/karatelabs/karate/wiki/How-to-Submit-an-Issue


Add a summary of the signed transaction

Print out a summary of the singed transaction form the wallet CLI

I will change a few things and make transactions generally printable. I converted the PR to draft.


ERROR: open file exefs/logo.darc.lz failed when trying to create cia

I'm trying to edit the paper mario ss cia file. I'm following the guide here. When I try to rebuild the exefs after extracting it, 3dstool throws out the error: ERROR: open file exefs/logo.darc.lz failed. I checked the directories and the logo never existed in a darc.lz file, it only exists in a bin file and a bmca.lz file.

3dstool version before v1.0.20, the logo file is logo.bcma.lz.
Starting from v1.0.20, this file is called logo.darc.lz.
I think the .bcma.lz extension is not reasonable, so I renamed it.

There is one more thing, some games, the logo is in the exefs, some are outside the exefs, but there is only one, don't mess up the position.
You only need to ensure that the extension of the file opened by 3dstool is the same as the actual file extension.

Alright, I replaced all instances of logo.darc.lz with logo.bcma.lz in the code and rebuilt it and it worked. Thanks for the help.


generating bold time series

on the following example,
https://github.com/ins-amu/vbjax/blob/main/examples/fc-opt.py
bold time series are generated offline using the firing rate output of the model rv_t[:,0],
where rv_t[:,0].shape = (samples, nodes)
# apply a bold monitor
# ok this api is not so convenient yet, but monitors are online,
# so there's a step function to consume some states at every step
# and a sample function to "read" a sample out of the monitor buffer
bold_buf, bold_step, bold_sample = vb.make_bold(shape=(SC.shape[0],),
                                                dt=dt, p=vb.bold_default_theta)
# we make it "offline" to process an array of states at once
bold_sample = vb.make_offline(step_fn=bold_step, sample_fn=bold_sample)
# now window the MPR r variable into windows 100 time points
windowed_r = rv_t[:,0].reshape((-1, 100, SC.shape[0]))
# scan that array of windows w/ bold
bold_buf, bold = jax.lax.scan(bold_sample, bold_buf, windowed_r)

why windowed_r is constructed that way and why is this the input to bold_sample function?

Yes yes


feat(web): plugin playground/reflect code editor to viewer

Overview
What I've done
I implemented the process to execute reearth.yml and other JS files and reflect the plugins in the visualizer.
https://github.com/user-attachments/assets/e6d77e4e-15c2-470e-a3d1-25965951f0eb
What I haven't done
How I tested
Which point I want you to review particularly
Memo

Great work! I've added small comments. Not sure if the PR is pointing to the right target. Ideally would be good to point all PRs to main. Let me know your thoughts @devshun .

@mulengawilfred
thanks for your review!

Not sure if the PR is pointing to the right target. Ideally would be good to point all PRs to main. Let me know your thoughts

This branch was branched off from #1216, so I changed the base branch to make the changes (files changed) easier to track. After merging #1216, I plan to merge this branch into main🙏

Thanks @devshun ! PR approved!


[ACR] [Do not merge] merge open PRs for apiview

Do not merge

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-container-registry


Projections loose references to the original dataframe

Here is a problem that is is currently documented in the doc for koalas.mlflow.predict: it is common to isolate a few columns of a dataframe, transform them into a column, and then add this column back into the first dataframe.
Examples:

featurization in this blog post
applying an ML model to predict variables
computing the mean of a few columns:

df = pd.DataFrame({"x": [1], "y": [2]})
df["z"] = df[["x", "y"]].mean(axis=1)

fixing NaNs for specific columns:

df = pd.DataFrame({"x": [1], "y": [2]})
df[["x", "y"]] = df[["x", "y"]].fillna(-1)

All these issues come from the fact that df[[...]] returns a new ks.DataFrame that has lost reference to the original df. The only solution currently is making a join, which does not work in all cases when there are duplicates (see  koalas.mlflow.predict )
I am creating this issue here mostly to trigger discussions, because this will most probably require some changes in the way we represent dataframes.

Note that the issue is not the the fact that we are adding a new column, but rather that we are unable to compare series from the original dataframe with the new dataframe. For example, this code to compute L1 error would currently fail:
df = pd.DataFrame({"output": [1], "feature1": [2], "feature2": [3]})
prediction = df[["feature1", "feature2"]].mean(axis=1)
mse = (df["output"] - prediction).abs().sum()

This was fixed by providing a workaround, option compute.ops_on_diff_frames. Now we support the operations on different DataFrames. We should still avoid to losing the original frame in this project in general though.


Shaded version of Jackson Dataformat affected by CVE-2020-28491

The version of jackson-dataformat-cbor(2.10.0) that is shaded within the bundle is affected by CVE-2020-28491  and is affecting security scans for components that embed the bundle jar.

Thank you for your report. This has already been fixed and will appear in our mid-March release.

Thanks for the update.

This issue has been resolved in 2.9.1.


Question of Training Time

In README, here is:

The model takes less than 10k steps (less than 1 hour on my GTX1080 GPU) of training to generate audio samples with acceptable quality

I clone this project, use the same dataset and parameters on GTX1080Ti (Two GPUs).  The log is:

It seems to need 4.2 days to complete. Is somewhere I made a mistake?

The code I changed the following two places:

https://github.com/ming024/FastSpeech2/issues/13#issuecomment-698263168
https://github.com/ming024/FastSpeech2/issues/14#issuecomment-678245971
Otherwise, I cannot run train.py.

Yes, you're right. The model takes less than 10k steps will have a pretty result.
After that, the improvement is tiny, especially after 50K (about three and a half hours). Sometimes the effect of individual words is worse.

I am very sorry to bother you.How about the quality of the audio you synthesized in the test phase.I use the model with Chinese dataset.When I set the pitch and energy normalization = False and use the frame level feature of them，the loss is much higher compared to you(pitch:~1500 energy:~150 in train and pitch ~8000 ~400 in validation).The audio quality is poor. There are many noises and incorrect pronunciations in the synthesized audio when i use the test data,but both the train and val audio  synthesized in trainning step is good.


[Question] size mismatch for mm_projector

Question
when i run cmd below, got size mismatch.
CUDA_VISIBLE_DEVICES=1 python -m llava.serve.cli --model-path liuhaotian/llava-v1.5-13b-lora --model-base liuhaotian/vicuna-13b-v1.5 --image-file ./images/view.jpg --load-4bit
raceback (most recent call last):
File "/apdcephfs/share_1157259/users/tttaozhang/tools/anaconda3/envs/llava/lib/python3.10/runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/apdcephfs/share_1157259/users/tttaozhang/tools/anaconda3/envs/llava/lib/python3.10/runpy.py", line 86, in _run_code
exec(code, run_globals)
File "/apdcephfs/private_tttaozhang/llm/LLaVA/llava/serve/cli_ttao.py", line 132, in
main(args)
File "/apdcephfs/private_tttaozhang/llm/LLaVA/llava/serve/cli_ttao.py", line 33, in main
tokenizer, model, image_processor, context_len = load_pretrained_model(args.model_path, args.model_base, model_name, args.load_8bit, args.load_4bit, device=args.device)
File "/apdcephfs/private_tttaozhang/llm/LLaVA/llava/model/builder.py", line 72, in load_pretrained_model
model.load_state_dict(non_lora_trainables, strict=False)
File "/apdcephfs/share_1157259/users/tttaozhang/tools/anaconda3/envs/llava/lib/python3.10/site-packages/torch/nn/modules/module.py", line 2041, in load_state_dict
raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for LlavaLlamaForCausalLM:
size mismatch for model.mm_projector.0.weight: copying a param with shape torch.Size([5120, 1024]) from checkpoint, the shape in current model is torch.Size([2621440, 1]).
size mismatch for model.mm_projector.2.weight: copying a param with shape torch.Size([5120, 5120]) from checkpoint, the shape in current model is torch.Size([13107200, 1]).

removed '--load-4bit', the code runs fine. However, how can I modify the code when using 4-bit quantization with lora weight?

I also have the same problem. I would like to believe that we didn't find the right model_base. But I can't find the instruction on the right one other than the information from the technical report.

removed '--load-4bit', the code runs fine. However, how can I modify the code when using 4-bit quantization with lora weight?

You can first create merged lora weights, then load with 4bit: https://github.com/haotian-liu/LLaVA/blob/main/docs/LoRA.md#create-merged-checkpoints
It seems that QLora does not support this either: https://github.com/artidoro/qlora/blob/main/examples/guanaco_7B_demo_colab.ipynb

removed '--load-4bit', the code runs fine. However, how can I modify the code when using 4-bit quantization with lora weight?

You can first create merged lora weights, then load with 4bit: https://github.com/haotian-liu/LLaVA/blob/main/docs/LoRA.md#create-merged-checkpoints
It seems that QLora does not support this either: https://github.com/artidoro/qlora/blob/main/examples/guanaco_7B_demo_colab.ipynb

Hello Haotian, Thanks for the reply. But I would like to confirm which  is the base model of liuhaotian/llava-v1.5-13b-lora? Because there is no clear documentation on this. I could also open a PR for the documentation if you can provide a way for me to validate the model_base of lora models manually.

Hi @LumenYoung The base model is Vicuna v1.5.

As a follow up, the model base is lmsys/vicuna-13b-v1.5, should anyone was uncertain about the it.

@haotian-liu Would you please check my problem when loading merged-lora checkpoint? It seems that the merged checkpoint has the same format as Vicuna-v1.5-7b and it has no information about the vision tower as well as the mm-projector. After I add configs about Vision tower and projector(non_lora_trainable.bin) to the config file, there would be an error saying 'ValueError:the weight is on the meta device, we need a 'value' to put in 0 ' when launching model worker.
But the code could run with the original llava projector(mm_projector.bin), I wonder if these two projector files has some differences?
How could I load the merged-lora checkpoint as well as the finetuned projector?

@haotian-liu Would you please check my problem when loading merged-lora checkpoint? It seems that the merged checkpoint has the same format as Vicuna-v1.5-7b and it has no information about the vision tower as well as the mm-projector. After I add configs about Vision tower and projector(non_lora_trainable.bin) to the config file, there would be an error saying 'ValueError:the weight is on the meta device, we need a 'value' to put in 0 ' when launching model worker. But the code could run with the original llava projector(mm_projector.bin), I wonder if these two projector files has some differences? How could I load the merged-lora checkpoint as well as the finetuned projector?
How did you solve this problem?


webdriver error: TypeError options must be an object

Thanks for your tool, first of all. It might be very helpful.
But when I tried to reproduce your example in the video on Ubuntu 14.04, I got the error: "There was a webdriver error: TypeError options must be an object".
Output of 'webdriver-manager' on startup:
"15:38:38.649 INFO - Java: Oracle Corporation 24.72-b04
15:38:38.649 INFO - OS: Linux 3.13.0-43-generic amd64
15:38:38.668 INFO - v2.44.0, with Core v2.44.0. Built from revision 76d78cf
15:38:38.755 INFO - Default driver org.openqa.selenium.ie.InternetExplorerDriver registration is skipped: registration capabilities Capabilities [{platform=WINDOWS, ensureCleanSession=true, browserName=internet explorer, version=}] does not match with current platform: LINUX
15:38:38.789 INFO - RemoteWebDriver instances should connect to: http://<IP_ADDRESS>:4444/wd/hub
15:38:38.790 INFO - Version Jetty/5.1.x
15:38:38.791 INFO - Started HttpContext[/selenium-server/driver,/selenium-server/driver]
15:38:38.792 INFO - Started HttpContext[/selenium-server,/selenium-server]
15:38:38.792 INFO - Started HttpContext[/,/]
15:38:38.809 INFO - Started org.openqa.jetty.jetty.servlet.ServletHandler@1a18b622
15:38:38.809 INFO - Started HttpContext[/wd,/wd]
15:38:38.816 INFO - Started SocketListener on <IP_ADDRESS>:4444
15:38:38.817 INFO - Started org.openqa.jetty.jetty.Server@357b2b99
"
After opening the Protractor API page with elementor, duplicating the tab and activation 'Protractor' as a event listener, elementor produces the following output when selecting elements:
"> Testing locators { byCss:
{ nodeName: 'div',
class: 'container protractor-container ng-scope',
'ng-view': '',
autoscroll: 'true' } }
Testing locator list [ { name: 'byCss',
locator: 'by.css('div.container.protractor-container')',
countExpression: 'element.all(by.css('div.container.protractor-container')).count()' },
{ name: 'byCss',
locator: 'by.css('.container.protractor-container')',
countExpression: 'element.all(by.css('.container.protractor-container')).count()' },
{ name: 'byCss',
locator: 'by.css('div[ng-view=""]')',
countExpression: 'element.all(by.css('div[ng-view=""]')).count()' },
{ name: 'byCss',
locator: 'by.css('div[autoscroll="true"]')',
countExpression: 'element.all(by.css('div[autoscroll="true"]')).count()' } ]
Sending locator results { 'by.css('div.container.protractor-container')': 'There was a webdriver error: TypeError options must be an object',
'by.css('.container.protractor-container')': 'There was a webdriver error: TypeError options must be an object',
'by.css('div[ng-view=""]')': 'There was a webdriver error: TypeError options must be an object',
'by.css('div[autoscroll="true"]')': 'There was a webdriver error: TypeError options must be an object' }
"
What went wrong?

Let me take a look. Which element did you click on? Did you go to http://angular.github.io/protractor/#/api?
Are both tabs pointing to the same url and are you using the second tab to open the developer tools?

Let me take a look. Which element did you click on? Did you go to
http://angular.github.io/protractor/#/api?

This is my output after starting elementor:
pedro@lms-dev:~$ elementor http://angular.github.io/protractor/#/api
util.puts: Use console.log instead
Type  to see a list of locator strategies.
util.puts: Use console.log instead
Use the list helper function to find elements by strategy:
util.puts: Use console.log instead
e.g., list(by.binding('')) gets all bindings.
util.puts: Use console.log instead
util.puts: Use console.log instead
IMPORTANT:
util.puts: Use console.log instead
The element explorer will not work when the dev tools window is open on
the first tab of the launched chrome browser. To use the dev tools
duplicate the first tab and inspect on the second tab.
util.puts: Use console.log instead
util.puts: Use console.log instead
Getting page at: http://angular.github.io/protractor/#/api
Listening on port 13000
The API page opens, I duplicate the tab, open the developer tools on the
second tab, select Protractor under 'Event Listeners', click an element in
the source tree and get the following result:

Are both tabs pointing to the same url and are you using the second tab
to open the developer tools?

Yes

Reply to this email directly or view it on GitHub:
https://github.com/andresdominguez/elementor/issues/1#issuecomment-69035018

I tried to reproduce the issue using my Linux box. This is my setup:

protractor 1.5.0 (for the webdriver manager)
<EMAIL_ADDRESS>(npm list -g | grep elementor)
node v0.10.33
Ubuntu 14.04

Can you check your node / protractor version? I will try to reproduce it with other versions.

My node version is 0.11.14. All other software is identical.
Am 07.01.2015, 18:25 Uhr, schrieb Andres Dominguez
<EMAIL_ADDRESS>
I tried to reproduce the issue using my Linux box. This is my setup:

protractor 1.5.0 (for the webdriver manager)
<EMAIL_ADDRESS>(npm list -g | grep elementor)
node v0.10.33
Ubuntu 14.04

Can you check your node / protractor version? I will try to reproduce it
with other versions.

Reply to this email directly or view it on GitHub:
https://github.com/andresdominguez/elementor/issues/1#issuecomment-69057131

I was able to reproduce it with 0.11.14. I will try to fix it tonight.

@letsmakesense Thank you for reporting the issue. Please update to 0.1.0.

Works with 0.1.0. Thanks a lot. The tool is a HUGE progress in writing e2e
tests with Protractor
Am 08.01.2015, 02:04 Uhr, schrieb Andres Dominguez
<EMAIL_ADDRESS>
@letsmakesense Thank you for reporting the issue. Please update to 0.1.0.

Reply to this email directly or view it on GitHub:
https://github.com/andresdominguez/elementor/issues/1#issuecomment-69120633


feat(celo-news): connect it to our API

Description
Connect the Celo news feed to our API.
Tested
loading + success
https://user-images.githubusercontent.com/57791/208414473-88673824-0bb2-42ed-abc8-062c0f1f4325.mp4
loading + error with retry
https://user-images.githubusercontent.com/57791/208414532-7256d792-392c-4247-832a-128e657398a2.mp4
Related issues

Fixes RET-519

Backwards compatibility
Yes

looks great!!
one thought i had was maybe we want to keep the header title and description around even when there is an error? i think it'd minimise the content moving around + add more context for the error, but just my opinion!

Yes I'm discussing this with Nitya / Reed, I didn't like how it looked with my initial attempt, but we'll come up with something nice I'm sure.

Merging now, I'll address changes to the error UI in a follow up PR.


Facebook Connect

How would you go about integrating facebook into adonis ?
We would like to use passportjs for facebook login but are unsure how we could implement this with adonis...

There is no official module to do this with Adonis. Passport is very tightly written for express.
So i personally feel you will have a hard time integrating it. The best way to do is use the official Facebook api and use that.

Rest you can follow the framework development lifecycle here. https://trello.com/c/wns5DzT3/28-authentication-provider
Authentication provider will have a sibling module to support the social authentication.


Rework await idle

Await idle can be stuck on waiting too long because of different reasons. Probe then is fully blocked as it is single threaded and we cannot even take a screenshot or stop the waiting, or modify waiting strategy in any way. Currently we made a temporary workaround allowing to disable await idle globally. The actual solution we want to end up with is to have await idle on driver side.
Probe should only be used to query the background task state. Wainting logic should be implemented in awaitIdle method in driver. BackgroundTasks.awaitNone and its usages should also be moved to driver side.
For specific endpoints we can use more sophisticated waiting mechanisms, like waiting for import to start and then wait for indexing to end instead of waiting for all tasks.
We can provide await limit and show tasks list and create a screenshot if it is exceeded.
Initial idea of implementing this is some trait WaitLogic { def await[A](f: Future[A]): Either[Error, Unit] } with multiple implementations that are composable, and make it a parameter for each endpoint that awaits by default (parameter. with a default value). Below some example code of how it could look like.
def invokeActionAsync(action: ActionRef, await = WaitLogic.Async) // implemented as no waiting
WaitLogic.ForFuture(atMost = 10.minutes) // just wait for future to complete
WaitLogic.NoBackgroundTasks(atMost = 10.minutes, checkDelay = 5.seconds, probeFrequency = 50.millis) // similar to current awaitIdle
WaitLogic.BackgroundTaskExists(task = "Importing", atMost = 10.minutes, checkDelay = 2.seconds) // wait for list to contain some task
WaitLogic.BackgroundTaskNotExists(task = "Indexing", atMost = 10.minutes, checkDelay = 2.seconds) // wait for list to not contain some task
def BackgroundTaskCompletes(...) = WaitLogic.BackgroundTaskExists(...) andThen WaitLogic.BackgroundTaskNotExists(...) // ensure that some background task appears and completes, example of composition
val OpenProject = WaitLogic.BackgroundTaskCompletes(task = "Indexing", atMost = 30.min) // some meaningful waiting stratgy based on default building blocks, await for opening project is created as expectation to indexing start and complete (though indexing might appear and dissappear couple of times so we might create more complex strategy)
probe.driver.openProject(path, await = WaitLogic.OpenProject.copy(atMost = 10.min)) // example of overriding project waiting logic to wait shorter than default.
WaitLogic.Condition(atMost = 10.minutes) { probe.driver.projectModel.name == "expected name" } // await for custom condition.

Fixed in 0.5.0


Could not build

I tried running on the current stable version of Lean and with Eigen installed.
I only installed Lean to build this project.
Could you please help me?
I get the following errors:
invalid import: data.list.set
could not resolve import: data.list.set
mnist.lean:8:0: error
invalid import: data.list.comb
could not resolve import: data.list.comb
mnist.lean:8:0: error
invalid import: data.list.comb
could not resolve import: data.list.comb
mnist.lean:8:0: error
invalid import: data.list.set
could not resolve import: data.list.set
mnist.lean:8:0: error
invalid import: data.hash_map
could not resolve import: data.hash_map
mnist.lean:8:0: error
invalid import: data.list.set
could not resolve import: data.list.set
mnist.lean:8:0: error
invalid import: system.time
could not resolve import: system.time

Sorry. I have now tried with your certigrad branch of Lean, which works.


Nested support

Thanks for your plugin, it save us tons of work! But I found that, if I put an instance into an instance of the same type, like below:
public class A {
private int mInt;
@Nullable private A mA;
}
and I have to apply this plugin twice to make it correct. Sometime someone forgot to apply it twice and make it crash or compile fail.

@kyze8439690 thanks for submitted issue.
To be honest I don't see what doesn't work for you. What's the difference between applying plugin once or multiple times? Could you explain what behavior do you expect?

Could you give an example of failing generated code? I tried your class structure and it worked fine. Generating a second time gave different code, but the differences were inconsequential. Here's my test. Granted, I ran against Robolectric and it could fail on the actual device.


Auto devpool pr

resolves #278

@pavlovcik @rndquu Ready for review

I need your input @pavlovcik before merging it  because your request seems not approved yet.

Theres at least one conversation I tagged you in for your input you need to reply to.

I need your input @pavlovcik before merging it  because your request seems not approved yet.

Still need your input you need to reply to.

Answered

Should this pull request be reverted? @0xcodercrane @Draeieg

Lets keep the PR and wait for @seprintour to create a hotfix pr if needed.

Lets keep the PR and wait for @seprintour to create a hotfix pr if needed.

Can i handle the QA?, This works so maybe i should handle the PR and test.. remember this will need 2 repos with the staging bot installed and one of them (directory) with a little bit more permissions to create pr, edit file and contents

@seprintour. yes you can definitely handle the QA. please follow the instructions here


Pure ruby support?

Can this be used with pure ruby? Or am I doing something wrong?
I managed to have it working with Rails, but not with ruby only.
All the following steps are reproductible in Rails (including the requirement failure) but for the last one.
Installation went ok
$ gem install ruby-saml
Fetching: ruby-saml-0.9.1.gem (100%)
Successfully installed ruby-saml-0.9.1
Parsing documentation for ruby-saml-0.9.1
Installing ri documentation for ruby-saml-0.9.1
Done installing documentation for ruby-saml after 1 seconds
1 gem installed

Requirement described in README.md did not work
$ irb
2.1.5 :001 > require 'onelogin/ruby-saml'
LoadError: cannot load such file -- onelogin/ruby-saml
	from /Users/rafaelgonzalez/.rvm/rubies/ruby-2.1.5/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /Users/rafaelgonzalez/.rvm/rubies/ruby-2.1.5/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from (irb):1
	from /Users/rafaelgonzalez/.rvm/rubies/ruby-2.1.5/bin/irb:11:in `<main>'

So instead I tried
2.1.5 :002 > require 'ruby-saml'
 => true

Now, following the usage example, it fails
2.1.5 :003 > idp_metadata_parser = OneLogin::RubySaml::IdpMetadataParser.new
 => #<OneLogin::RubySaml::IdpMetadataParser:0x007f7f6c35a780>
2.1.5 :004 > settings = idp_metadata_parser.parse_remote("https://my-tsta.unimelb.edu.au/fed/idp/metadata")
NameError: uninitialized constant OneLogin::RubySaml::IdpMetadataParser::Net
	from /Users/rafaelgonzalez/.rvm/gems/ruby-2.1.5/gems/ruby-saml-0.9.1/lib/onelogin/ruby-saml/idp_metadata_parser.rb:45:in `get_idp_metadata'
	from /Users/rafaelgonzalez/.rvm/gems/ruby-2.1.5/gems/ruby-saml-0.9.1/lib/onelogin/ruby-saml/idp_metadata_parser.rb:20:in `parse_remote'
	from (irb):4
	from /Users/rafaelgonzalez/.rvm/rubies/ruby-2.1.5/bin/irb:11:in `<main>'

Ok, this is simply resolved by requiring net/http. I would have expected the gem to do it.

Hey @rafaelgonzalez -- would you be willing to put together a PR to resolve this issue both for ruby and rails? Thanks for taking the time to look into it!

Sure, I'll try and get around it some time this week.

Thanks

I created a PR for the net/http issue.
Do you want me to make another PR to update the requirement statements I mentioned here as well?

I created another PR to reflect the requirement instructions in the README. #205

fixed via https://github.com/onelogin/ruby-saml/pull/204

Ah right, forgot to close it! Thanks


ACM-14557-Implement inferred ranges for cluster list advanced search

Regarding: ACM-14557
When input in the Advanced Search Field is incomplete the intuitive behavior (what is being implemented here) is that an assumed range of values could appear in the search results.
For example: "Distribution version = 4.", Here the table should display all values greater than or equal to 4.0.0 but less than 5.0.0
Before:

After:

/hold
Seeking feedback

/lgtm


$digest already in progress

Steps to reproduce

Unknown at this time. Observed crash via Rollbar

Expected behaviour
No $digest in progress when calling applyModel()
Actual behaviour
[$rootScope:inprog] $digest already in progress
http://errors.angularjs.org/1.5.5/$rootScope/inprog?p0=%24digest
1
File "../../bower_components/angular/angular.js" line 68 col 1 in [anonymous]
return new ErrorConstructor(message);
2
File "../../bower_components/angular/angular.js" line 19530 col 1 in m
return function(name) {
3
File "../../bower_components/angular/angular.js" line 19530 col 1 in $digest
return function(name) {
4
File "../../bower_components/angular/angular.js" line 19530 col 1 in $apply
return function(name) {
5
File "../../bower_components/angularjs-slider/dist/rzslider.js" line 1820 col 1 in applyModel
this.scope.$apply();
6
File "../../bower_components/angularjs-slider/dist/rzslider.js" line 1796 col 1 in positionTrackingHandle
this.applyModel();
7
File "../../bower_components/angularjs-slider/dist/rzslider.js" line 1493 col 1 in onMove
this.positionTrackingHandle(newValue);

What version?

2.13.0

Do you have a way to reproduce this error? Also could you please try to update to the latest version to see if you still get it?


feat: support --create-tag in update subcmd

Supporting the equivalent of the create --create-tag option in the update sub-command would allow a stack to be updated without forcing a local image build.

https://github.com/chanzuckerberg/happy/pull/208


Update INSTALL.md

Updated the install documentation to fix some formatting issues.
Copying and pasting commands beforehand would have resulted in some command line issues. This change should fix some of that.

This could use a bit more explanation of what each step is used for, and how differing installations might work (installing for testing vs installing for running a realization etc.).
If we dont have time for that at the moment I can just push this through.

More documentation is generally always better. This isn't in the scope of anything really, though - it's just something I went to edit when some copy and pasting didn't work out. The git submodule statements, for example, were rendering on the same line. Turns out git doesn't know what the argument git means.


Fix GatherV2 shape inference

Fixes #42522.
https://github.com/tensorflow/tensorflow/blob/8a002f2269310f513d326eaeff5c73f679c73f77/tensorflow/core/kernels/gather_op.cc#L92-L107

Sorry about the rollback, looks like it was totally spurious. I've rolled your PR forward, hopefully permanently this time (should show up in a few minutes).


How to keep backend server alive while error occurs for better DX

Hi,
First of all, this is not bug of vike-node, but I think some feature idea to improve for better DX.
On example code server/index.js, I add / for triggering error
import express from 'express'
import vike from 'vike-node/connect'

startServer()
/ // for triggering error

function startServer() {
  const app = express()
  app.use(vike())
  const port = process.env.PORT || 3000
  app.listen(port, () => console.log(`Server running at http://localhost:${port}`))
}

And it cause error and app exited with code 1
ReferenceError: startServer is not defined
    at /Users/forte/Downloads/express-react/server/index.js:4:1
    at instantiateModule (file:///Users/forte/Downloads/express-react/node_modules/vite/dist/node/chunks/dep-mCdpKltl.js:52650:5)
ReferenceError: startServer is not defined
    at /Users/forte/Downloads/express-react/server/index.js:4:1
    at instantiateModule (file:///Users/forte/Downloads/express-react/node_modules/vite/dist/node/chunks/dep-mCdpKltl.js:52650:5)
error: script "dev" exited with code 1

I have tried to use pm2 but still not get better DX
Any idea for better DX ?

Yes, I think I can catch these errors and make it log something like: Server crash, update a server source file or type r + enter to restart the server.

After install latest package. I got warning message in my app (fastify)
[2024-08-08 15:52:09] WARN: response will send, but you shouldn't use res.writeHead in stream mode
    reqId: "req-5"
[2024-08-08 15:52:10] WARN: response will send, but you shouldn't use res.writeHead in stream mode
    reqId: "req-5t"
^C


Divide users by role

Admin page should have the oportunity to filter users by role, in the same way user log is divided up.

Fixed in 907e4808f969b3aae54154e721cc96150ca1f504.


Recompile on permissions update?

Hi,
forgive the somewhat unrelated question, but as a newcomer to Elixir, I just pulled my hair out for way too long because my permissions weren't working ... until I realized that mix actually compiles the Map in config.exs directly into the @perms attribute, and that said compilation isn't automatically re-triggered when my configuration changes. In other words, I have to delete _build/guardian after updating my permissions or else my app won't work.
If this should happen automatically, then something must be missing in my setup, and I'd be grateful for a pointer to a working example.

oh that's a good point. The GuardianPermissions module is dynamically generated. I'm not sure how to watch for changes in the config and re-generate. That's an interesting challenge though.

Hm. I guess it's not that big of a deal in practice, but maybe a note in the docs would be helpful to save the next guy a few hours :)

Quick update on this. I've looked into getting some code reloading happening, and while it's very possible, it still doesn't help with partial compilations like tests. Only in dev. I'm going to take a stab at refactoring the permissions module to be a more run-time set of functions rather than compile time definitions.

Sweet, thank you! Looking forward to what you'll come up with. I'll keep the issue open until then.

It would be nice to have a notice about this in the section of the readme on permissions, I just had this same problem and couldn't figure out what was wrong for the life of me.


Pick dates in UTC

Since the UK ticked over from GMT to BST, the datepicker is now picking dates in BST, which on the surface, is fine.
However, if I then take those dates and turn them to UTC to store them, my "start of day" time becomes 11pm yesterday.
A bit confusing this one, but essentially I want to say that picking 1st of May 2019 is 1st of May 2019, and not 30th April 2019. I don't have any control over this with the current component.
Any ideas for how we can improve this?

I believe the date picker would be generating dates in the client's timezone.  If you convert this date to a different timezone/format then I think it would be expected that the time (and sometimes the date/day) would be different as well.  Converting it back into the client's timezone should yield the same date/time.
You could pull day, month, and year from the date object and use that to store the date in a format that is not timezone-specific, but I'm not sure that would suit everyone's needs.

Hey @antony - I'd like to resolve this for you, somehow.. maybe I could make the date attributes bindable?  Such that you could say something like <Datepicker bind:month bind:year bind:day /> and then you can assemble a date object using those integers regardless of timezone?

Hey @6eDesign - Thanks for thinking about this! I don't know if this is strictly necessary since for dd/mm/yyyy it's easy enough to pull the values off the Date object returned by the picker.
For now I think it may be best to leave it as is.


Feature request: Limit add-on width (custom setting?)

If you have a tab with long text in a tab group the add-on gets pretty wide when you open this group. It would be nice if users could customize the max width. Or limit the width in aspect to the browser window. (1/3rd, 1/4th or something...)

Request is already implemented in current version... Close and thanks! :)

Since v0.2.2: size is limited
In v0.2.3: you can switch between small and big window


Bugfix/focus whencollapse on body tap is false

Hi @jdayssol, your PR has some very good changes but just like I mentioned in your other PR, the minWidth to maxWidth is causing the body to shift, hence I'll change that bit to the previous code.
Thanks a lot for your contribution to this project.
I hope you'll continue to support this project and help other open-source projects as well. 😁