Editing JobInfoChips

Discussed in https://github.com/bullhorn/career-portal/discussions/471

Originally posted by 930turborob May 16, 2022
Hi, I've gotten the code to fix the JobInfoChips overlapping the Job Title (as well as other chips) from here, but am using WordPress to edit the Bullhorn OSCP plugin code. Problem is, in the plugin files section, there are no root folders that contain any of the files I need to actually change the code. Is there an actual Bullhorn desktop app or other application I need to use in order to access the job-list.component.scss file? Thanks in advance.

Hi, I wanted to follow up with you to see if you were able to find a solution? I am struggling to find an answer as well.

Hi, I haven't yet. I'm neck deep in code doing trial and error until BH provides a solution. If I get one, I can let you know. Problem seems to be embedded in the JavaScript coding for mobile devices.
[cid:5bad4b04-39d8-41d6-8acb-26a89bba022a]
Robert Crocker
Digital Marketing
T: (tel:410-910-3330410) 910-0248
M: (tel:805-279-6620805) 807-9163
E<EMAIL_ADDRESS>[X]https://www.youtube.com/channel/UC7rCGMxXb8ZWX4ZkX2tq4EQ  [X]
http://www.kryptauri.com/

From: tren10jam @.>
Sent: Wednesday, June 1, 2022 2:51 PM
To: bullhorn/career-portal @.>
Cc: Robert Crocker @.>; Author @.>
Subject: Re: [External] [bullhorn/career-portal] Editing JobInfoChips (Issue #472)
Hi, I wanted to follow up with you to see if you were able to find a solution? I am struggling to find an answer as well.
—
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_bullhorn_career-2Dportal_issues_472-23issuecomment-2D1144176403&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=lSnxEHQNXfi__8POvDeUq9tWLExTOBsw-ld5kl_u2xw&m=y2GOQwlv9GB4bUZNnAgtbvwmanqQLByRU-i6V2F8EGE&s=gj-Dy1tGOKCdZBfTTlP83CBP7rU3rlmguoT44xqxL_c&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AZGGJRN2XH3VYXUY3HY2RITVM7LPXANCNFSM5WF452LQ&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=lSnxEHQNXfi__8POvDeUq9tWLExTOBsw-ld5kl_u2xw&m=y2GOQwlv9GB4bUZNnAgtbvwmanqQLByRU-i6V2F8EGE&s=NPa9r2SrUqffgoCkqxPnmSvTDBvPq0InDU_mySteoK8&e=.
You are receiving this because you authored the thread.Message ID: @.***>

Hi Guys,
There are other solutions to display jobs on WordPress site.
Check the Bullhorn market place for the options https://www.bullhorn.com/marketplace/?subcategory=career-site
There are solutions that will make it easier to brand your as well fix SEO issues that OSCP has.
Paul

Thank you for the reply. I will give it a look.
Best,
[cid:bb781656-36f0-42bd-8cc0-d951fbfc7c8b]
Robert Crocker
Digital Marketing
T: (tel:410-910-3330410) 910-0248
M: (tel:805-279-6620805) 807-9163
E<EMAIL_ADDRESS>[X]https://www.youtube.com/channel/UC7rCGMxXb8ZWX4ZkX2tq4EQ  [X]
http://www.kryptauri.com/

From: Paul Bearne @.>
Sent: Monday, June 6, 2022 1:07 AM
To: bullhorn/career-portal @.>
Cc: Robert Crocker @.>; Author @.>
Subject: Re: [External] [bullhorn/career-portal] Editing JobInfoChips (Issue #472)
Hi Guys,
There are other solutions to display jobs on WordPress site.
Check the Bullhorn market place for the options https://www.bullhorn.com/marketplace/?subcategory=career-sitehttps://urldefense.proofpoint.com/v2/url?u=https-3A__www.bullhorn.com_marketplace_-3Fsubcategory-3Dcareer-2Dsite&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=lSnxEHQNXfi__8POvDeUq9tWLExTOBsw-ld5kl_u2xw&m=9yDQWDF5wG9Djg4XMssu44C1_Egtem3jbBIv00L7UFU&s=K6LJADSWMkprjlcEmghy9nDGGqaL_Fl8f-hKXQJU0Bc&e=
There are solutions that will make it easier to brand your as well fix SEO issues that OSCP has.
Paul
—
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_bullhorn_career-2Dportal_issues_472-23issuecomment-2D1147168323&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=lSnxEHQNXfi__8POvDeUq9tWLExTOBsw-ld5kl_u2xw&m=9yDQWDF5wG9Djg4XMssu44C1_Egtem3jbBIv00L7UFU&s=pALs3gfHSKv6gQ4RtQeCN_60hHiwexozEWD9hlxGU1o&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AZGGJRMCW2IMFPOFBNBFRVLVNWWTRANCNFSM5WF452LQ&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=lSnxEHQNXfi__8POvDeUq9tWLExTOBsw-ld5kl_u2xw&m=9yDQWDF5wG9Djg4XMssu44C1_Egtem3jbBIv00L7UFU&s=bxVKXdStbfaLsjYrHM0w96NZMmZCT3xiWl5WXN4eb6I&e=.
You are receiving this because you authored the thread.Message ID: @.***>


Enable cataloging unlogged postgres tables

Describe the bug
We use postgres 12.4.1 as a datawarehouse. During the ingestion processing, we use unlogged tables to speed up writing data into tables. Those tables are in a schema dedicated for storing raw data. We then transform this raw data using dbt.
We use the command below to generate the dbt documentation :
dbt docs generate --project-dir <MY_DBT_PROJECT> --profiles-dir <MY_PROFILE_DIR>

When looking at the dbt documentation, none of the unlogged tables have their columns information filled up.
We checked the dbt code source and found that dbt does ignore unlogged and temporary tables when getting columns information :
https://github.com/dbt-labs/dbt/blob/749f87397ec1e0a270b2e09bd8dbeb71862fdb81/plugins/postgres/dbt/include/postgres/macros/catalog.sql#L39
Steps To Reproduce
Create a postgres unlogged table, initialize a dbt project then run :
dbt docs generate

Expected behavior
In the dbt documentation, i can see postgres unlogged table columns information
Screenshots and log output

System information
Which database are you using dbt with?

[x] postgres
[ ] redshift
[ ] bigquery
[ ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
installed version: 0.20.2
   latest version: 0.20.2

Up to date!

Plugins:
  - bigquery: 0.20.2
  - snowflake: 0.20.2
  - postgres: 0.20.2
  - redshift: 0.20.2

The operating system you're using:
PRETTY_NAME="Debian GNU/Linux 10 (buster)"
NAME="Debian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
ID=debian

The output of python --version:
Python 3.8.8

Additional context
Add any other context about the problem here.

@ollivierj Good call! It sounds like the fix here is really quite minor, by changing the linked line to:
      and tbl.relpersistence in ('p', 'u') -- [p]ermanent table or [u]nlogged table. Exclude [t]emporary tables

It doesn't look like we have any integration tests for the unlogged config, so now is as good a time as any to add one. I think this could be as simple as:

Defining a model with {{ config(materialized = 'table', unlogged=True) }}
Running it
Executing a query (or dbt test!) that hits pg_class to confirm that the table is indeed unlogged
Generating docs
Confirming that the table appears in catalog.json

Would you be interested in contributing that fix? :)

@jtcohen6 Hi Jeremy,
Thank you for getting back to me.
I will be available from December 11st for working on this issue.

Sounds good! If someone else wishes to contribute the fix for this sooner, I'd welcome a PR for it.


Feature request: official support for form-floating and input-group

There are quite a few issues in regards to using form-floating with input-group, and the proper markup for such:
https://github.com/twbs/bootstrap/issues/34275
https://github.com/twbs/bootstrap/issues/34198
https://github.com/twbs/bootstrap/issues/33871
https://github.com/twbs/bootstrap/issues/33741
https://github.com/twbs/bootstrap/issues/33721
https://github.com/twbs/bootstrap/issues/33341
Bootstrap's reply is "not supported at this time". Makes sense. I really wanted to have this work, so went through various ways to make this happen. I think it can be supported fairly easily with just a few rule adjustments. So I'm leaving my findings here in hopes of discussion to support this in the main branch.
From my findings, there are 4 problems with using input-group with form-floating:

What is the proper markup? Some people are combining input-group with form-floating in the same element, which causes problems when you have prefixes to the input-group such that the form-control isn't the first element in the group. The benefit to this approach is that it (sort of) fixes some of the border-radius issues. However, IMO the floating issues would be very difficult to solve cleanly. So, form-floating used on a child element of input-group works best here.

z-index focus/hover issues. Solved by CSS rule.

border-radius issues. Solved by CSS rule.

form-control flex-grow issues. Solved by CSS rule.

My findings are most easily seen in the codepen provided below. I may be overlooking some cases, obviously. But, it seems to work for my purposes. Do you have any feedback?
Associated codepen:
https://codepen.io/nefiga/pen/VwbpdVw
Anyway, feel free to close this if these types of requests are driving you nuts, just figured I'd at least provide feedback on what I assembled over a few hours.

Doing a little more testing, in some cases the form-floating input would break with a button funny as seen in the codepen above on small width screens (mobile).
I've addressed this fix by using an existing rule in bootstrap and just adding .form-floating to the definition
See
https://codepen.io/nefiga/pen/BaRRQrK

How does this work when you don't use extra large form elements, such as inputs and buttons?

Made a PR #34527 34527
The above codepen (only important parts) with the PR:
https://codepen.io/nefiga/pen/dyWRWXK

Wow, not sure how I missed following up here. Sorry!
Does this look a little odd to anyone else? Something about it feels a little wrong with the label inside the field with a value and the vertically centered at sign.

I compared it to bootstraps default floating labels and I see no difference.

I can see where it looks a little off center, but I looked at it for so long I'm used to it. I mostly wanted it for alignment purposes with other .form-floating elements. I also use it for input-groups containing dropdowns and buttons.
Looking around, there are various options which become possible with this fix. For example, I see this, which uses a different layout you could achieve by changing some border styles:

I updated this pen real quick for a rough example (last input):
https://codepen.io/nefiga/pen/KKXapZG
My point isn't that bootstrap should directly support the "borderless" input, just that more things become more easily possible with this fix, too.

@mdo @ffoodd
Hey guys this is a feature I could really use, one I think the community seems to want, and think it should be merged. It's been almost a year with a PR I keep updating in the hopes it will get merged, I honestly see no drawback to the merge, it works great, fixes numerous "input group doesn't work with form-floating" issues that seem to get posted every few months and linked to the PR https://github.com/twbs/bootstrap/pull/34527
If you don't want to support it and just want to kill the PR, that's fine, but some response would help me at least know if this has any chance of getting merged so I don't keep pestering if it isn't in alignment with the project direction. Thank you sincerely for your time and effort in maintaining such an awesome repo.

So sorry! Let me try to merge it today, I've seen enough issue requests to justify it :).


(NB) Token generator has issue on rendering json

Duplicate of https://github.com/pondersource/sciencemesh-php/issues/189


FinTS::DialogError with ING-DiBa

Hi!
This is basically the same issue as https://github.com/schurig/ynab-bank-importer/issues/64.
require 'ruby_fints'
FinTS::Client.logger.level = Logger::DEBUG
f = FinTS::PinTanClient.new('50010517', 'my_username', 'my_password', 'https://fints.ing.de/fints/')
f.get_sepa_accounts

Gives me the following output with a FinTS::DialogError 😕
I, [2019-04-25T07:48:39.917712 #29]  INFO -- FinTS::Client: Initialize SYNC
D, [2019-04-25T07:48:39.918193 #29] DEBUG -- FinTS::Client: Sending SYNC: HNHBK:1:3+000000000379+300+0+1'HNVSK:998:3+PIN:1+998+1+1::0+1:20190425:074839+2:2:13:@8@00000000:5:1+280:50010517:REDACTED:S:0:0+0'HNVSD:999:1+@216@HNSHK:2:4+PIN:1+999+6820269+1+1+1::0+1+1:20190425:074839+1:999:1+6:10:16+280:50010517:REDACTED:S:0:0'HKIDN:3:2+280:50010517+REDACTED+0+1'HKVVB:4:3+0+0+1+ruby_fints+0.0.4'HKSYN:5:3+0'HNSHA:6:2+6820269++REDACTED''HNHBS:7:1+1'
I, [2019-04-25T07:48:39.918269 #29]  INFO -- FinTS::Client: Sending Message
D, [2019-04-25T07:48:39.918622 #29] DEBUG -- FinTS::Client: << HNHBK:1:3+000000000379+300+0+1'HNVSK:998:3+PIN:1+998+1+1::0+1:20190425:074839+2:2:13:@8@00000000:5:1+280:50010517:REDACTED:S:0:0+0'HNVSD:999:1+@216@HNSHK:2:4+PIN:1+999+6820269+1+1+1::0+1+1:20190425:074839+1:999:1+6:10:16+280:50010517:REDACTED:S:0:0'HKIDN:3:2+280:50010517+REDACTED+0+1'HKVVB:4:3+0+0+1+ruby_fints+0.0.4'HKSYN:5:3+0'HNSHA:6:2+6820269++REDACTED''HNHBS:7:1+1'
D, [2019-04-25T07:48:42.141609 #29] DEBUG -- FinTS::Client: >> HNHBK:1:3+000000000211+300+000006IJEV7V3RF87QRBNDEIIB2UV4+1+000006IJEV7V3RF87QRBNDEIIB2UV4:1'HIRMG:2:2:+9800::Der Dialog wurde abgebrochen.+9010::Ungültiger Signaturaufbau?: Fehler im Segmentaufbau.'HNHBS:3:1+1'
Traceback (most recent call last):
        7: from /usr/local/bin/irb:23:in `<main>'
        6: from /usr/local/bin/irb:23:in `load'
        5: from /usr/local/lib/ruby/gems/2.6.0/gems/irb-1.0.0/exe/irb:11:in `<top (required)>'
        4: from (irb):5
        3: from /usr/local/bundle/gems/ruby_fints-0.0.4/lib/fints/client.rb:19:in `get_sepa_accounts'
        2: from /usr/local/bundle/gems/ruby_fints-0.0.4/lib/fints/dialog.rb:39:in `sync'
        1: from /usr/local/bundle/gems/ruby_fints-0.0.4/lib/fints/dialog.rb:83:in `send_msg'
FinTS::DialogError ({"9800"=>"Der Dialog wurde abgebrochen.", "9010"=>"Ung\u00FCltiger Signaturaufbau?: Fehler im Segmentaufbau."})

Apparently this is working fine for others. Is there anything I can do to further debug this issue? I can successfully authenticate to my account via MoneyMoney using the same credentials.
Thanks for any help in advance!

I tested with my ING-DiBa account and it's working for me... Sorry that it's not really of much help. I'm having a similar problem with Targobank (see #7), but only for some operations. I have a feeling banks aren't implementing things consistently, but really no clue about the whole FinTS shenanigans...


remove unnecessary console warning about acroform classes

?

meant to target my own forked version, not upstream, so i closed this.


Request: Add observer to conditional properties

Currently, you can pass in a string property to an if or unless conditional. However, that validator is not run whenever the dependent property updates. It would be nice to have an option to allow the validator to be run again whenever a conditional property changes.
I am currently handling this in a custom validator. If you specify and if or else conditional with a string property, run this.dependentValidationKeys.pushObject(condition);
Any thoughts on the validity of this use case?

this is being planned out right now. Basically when any of the values for a validation changes, the validator should recalculate

Awesome, glad that's in the pipeline

Hey @green-arrow could you give an example of how you handled this? I'm running into the same issue.
Cheers

@bcardarella Any indication for when this one will be resolved?
Happy to help out where I can

+1, same issue here.

@davidpike7 the solution I came up with was for a PresenceValidator that listened for changes to the property it was dependent on.
app/validators/local/conditional-presence.js
import Ember from 'ember';
import PresenceValidator from 'ember-validations/validators/local/presence';

export default PresenceValidator.extend({
  init: function() {
    this._super.apply(this, arguments);

    var condition;

    if(this.conditionals) {
      condition = this.conditionals['if'] || this.conditionals['unless'];
    }

    Ember.assert('You must specify "if" or "unless" in the options for the conditional presence validator.', !!condition);

    if(Ember.typeOf(condition) === 'string') {
      // Condition based on another property. Make sure to watch that property
      this.dependentValidationKeys.pushObject(condition);
    }
  }
});

Thanks @greenarrow. That works great.
I've just attempted to create a conditional numericality validator using the code you've provided but I don't fully understand how it works. What would need to be tweaked in your presence validator to get it working?

@davidpike7: I believe that there is already a numeric validator provided with ember-validations, you could use that as your base instead of PresenceValidator and work from there.

I am going to close this issue for now, feel free to reopen this issue if you feel like it's still relevant and we should take a look at it.


ciao-controller: API version discovery

ciao-controller API needs to have a version discovery as   https://wiki.openstack.org/wiki/VersionDiscovery  defines

@kaccardi  with the new layer idea for api compatibility should we close this issue then, right ?


Use central package management to ensure package version consistency

The larger repo after #16 will have package versions spread all over the various csproj files. Move to using a root-level Directory.Packages.props to make package upgrades much easier. See https://learn.microsoft.com/en-us/nuget/consume-packages/Central-Package-Management for documentation on central package management.

Create Directory.Packages.props in repo root with ManagePackageVersionsCentrally set to true.
For each PackageReference in each csproj, create a corresponding  item in Directory.Packages.props at the latest of all version numbers across the projects. Remove the version number in the csproj.
Ideally at this point upgrade all packages to latest from nuget.org.

Combining repos not happening per closure of #17. Core repo only has 2 projects with non-overlapping package refs.


Added The Beaconsfield School  (beaconsfield.txt)

Added The Beaconsfield School
The Beaconsfield School
Wattleton Road,
Beaconsfield,
Buckinghamshire
HP9 1SJ
UK

@tobyporter First stage of verification passed, official domain for The Beaconsfield School is beaconsfield.school as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@tobyporter Thank you for your patience while waiting for a review and processing your request. Pull request merged.


[Broken] ERROR: [HiDive] Unable to extract login form;

Checklist

[X] I'm reporting a broken site support
[X] I've verified that I'm running yt-dlp version 2021.09.02
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones

Verbose log

PASTE VERBOSE LOG HERE
[debug] Command-line config: ['--verbose', '--all-subs', '-u', 'PRIVATE', '-p', 'PRIVATE', '-v', '--cookies', 'cookies.txt', 'https://www.hidive.com/stream/why-the-hell-are-you-here-teacher/s01e001']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] yt-dlp version 2021.09.02 (zip)
[debug] Python version 3.8.10 (CPython 64bit) - Linux-5.4.0-84-generic-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4
[debug] Optional libraries: keyring, pycryptodome, sqlite
[debug] Proxy map: {}
[HiDive] None: Downloading webpage
ERROR: [HiDive] Unable to extract login form; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 582, in extract
    self.initialize()
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 490, in initialize
    self._real_initialize()
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/hidive.py", line 45, in _real_initialize
    form = self._search_regex(
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 1147, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
yt_dlp.utils.RegexNotFoundError: Unable to extract login form; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 582, in extract
    self.initialize()
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 490, in initialize
    self._real_initialize()
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/hidive.py", line 45, in _real_initialize
    form = self._search_regex(
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 1147, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
yt_dlp.utils.RegexNotFoundError: Unable to extract login form; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1214, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1239, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 600, in extract
    raise ExtractorError(
yt_dlp.utils.ExtractorError: [HiDive] Unable to extract login form; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.

Description

WRITE DESCRIPTION HERE
From what I can tell hidive changed there login screen breaking the existing script.

Was a corrupt cookie.txt file.  Once I removed to old cookie.txt file it got past that error.


Can start the game to input illegal race number

e.g. input "4" for selecting races, we can start games

Thanks for the big report!!


Pull request: fix branch for Issue-244

Original thread: https://github.com/SheffieldML/GPyOpt/issues/244
This merge fixes on
i.  Receiving model for AcquisitionOptimizer
ii. Correcting typo "thompsom_sampling"
iii.Unifying the key of the map the class receives
travis_test.py is taken, all passed except a common error that also observed in the master branch

Please refer to Issue-256: https://github.com/SheffieldML/GPyOpt/issues/256

Codecov Report

Merging #257 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #257      +/-   ##
==========================================
+ Coverage   77.58%   77.58%   +<.01%
==========================================
  Files          97       97
  Lines        4621     4622       +1
==========================================
+ Hits         3585     3586       +1
  Misses       1036     1036

Impacted Files
Coverage Δ

GPyOpt/methods/bayesian_optimization.py
88.88% <100%> (+0.12%)
:arrow_up:

GPyOpt/optimization/acquisition_optimizer.py
94.82% <100%> (ø)
:arrow_up:

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 107e96b...e9b298e. Read the comment docs.


Favicon

To find the tab in the browser, it would be easier with a favicon

Issue is reported to our private repository.


fix : error thrown if onChangeText is undefined in validationInput

Hi, i am using ui-kitten in a javascript project.
While I was creating a javascript version of component validationInput, i encountered an error when onChangeText was undefined in validationInput.
I realize that this might not be the usual case to use an input field without using it's value, but still.
Steps to reproduce:
<ValidationInput validator={EmailValidator} />

That's a good point. Thanks for the PR :)


Remove handling of 101 Status code

In order to handle internet timeouts inside Iran just removed the checking 101 status code

if we remove this line, how to make sure the payment has done successfully?

Both 100 an 101 are set when the payment has done successfully.


Performance issues with Mac Catalyst

There is significant performance issue on Mac Catalys since commit 3d6dfc5988432f153e9cd128edaf4d697373d528. It also consumes more memory. Its probably same with other platforms. @symbiogenesis  can you have a look at that?

Made a lot of extra performance-related changes since 2.0.0 including:

https://github.com/akgulebubekir/Maui.DataGrid/pull/77
https://github.com/akgulebubekir/Maui.DataGrid/pull/76
https://github.com/akgulebubekir/Maui.DataGrid/pull/74
https://github.com/akgulebubekir/Maui.DataGrid/pull/68
https://github.com/akgulebubekir/Maui.DataGrid/pull/63
https://github.com/akgulebubekir/Maui.DataGrid/pull/51

I have also fixed some issues such as:

https://github.com/akgulebubekir/Maui.DataGrid/issues/70
https://github.com/akgulebubekir/Maui.DataGrid/issues/55
https://github.com/akgulebubekir/Maui.DataGrid/issues/53

I have another queued up change to set colors less. MAUI itself also has some major issues that they are trying to resolve:
This pull request should help when it is merged.

The memory usage of the CollectionView has been very bad going back to Xamarin.Forms, and they never fully fixed it. There are other issues beyond that recent pull request.
There are multiple collection view projects and branches that try to improve things. Unfortunately, most such projects don't work on every MAUI platform.
See also:

https://github.com/dotnet/maui/issues/12130#issuecomment-1422570083
https://github.com/dotnet/maui/issues/12039#issuecomment-1424888066

Deploying a .NET 7.0 version to nuget may help a little, since only the .NET 6.0 version made it over.

I found a way to reliably reproduce and test a case with bad performance.
My findings so far:
Reverting the ContentView wrapper: No visible improvement
Reverting to the DataTemplateSelector: No visible improvement
Reverting the WeakReference wiring for events: No visible improvement
Reverting the LineBreakMode: No visible improvement
Removing the IsVisible binding: No visible improvement
I also tried moving the _headerGrid inside of the HeaderTemplate of the CollectionView. This had no positive impact, but made the header no longer sticky at the top, and thus was worse.
My opinion is that MAUI has internal bugs, and that it potentially even worsened recently, such as with their recent Grid calculation changes.

I alerted the MAUI team in that memory leaks issue posted above. The developer Jonathan Peppers from the MAUI team is looking into this right now, and found an issue.


pass string as parameter? C ++ module

If I can not have "const char *" parameters in the functions, how would I do if I need to pass a string as a parameter?
Example:
#In my.h
myFunction(const char *s, int i=0);
#In my.cpp
MyClass::myFunction(const char *s, int i)
{
//Here you would use the parameters
}

#Calling myFunction with gd script:
myFunction("I exist?", 2)

Have you tried myFunction(String s, ...)?

yes, Also does not work

Try adapting from label set_text
void Label::set_text(const String &p_string) {
void Label::_bind_methods() {
    ClassDB::bind_method(D_METHOD("set_text", "text"), &Label::set_text);
    ClassDB::bind_method(D_METHOD("get_text"), &Label::get_text);

    ADD_PROPERTYNZ(PropertyInfo(Variant::STRING, "text", PROPERTY_HINT_MULTILINE_TEXT, "", PROPERTY_USAGE_DEFAULT_INTL), "set_text", "get_text");

I will try here
But what have you done in this line?
ADD_PROPERTYNZ(PropertyInfo(Variant::STRING, "text", PROPERTY_HINT_MULTILINE_TEXT, "", PROPERTY_USAGE_DEFAULT_INTL), "set_text", "get_text");

It's for property. If your method isn't setter or getter, just ignore it.

But I still can not understand why I can not pass a "const char *", what happens?

This is where String is defined https://github.com/godotengine/godot/blob/master/core/ustring.h
If I understand correctly, it has more features, like unicode support etc. GDScript uses String (Variant). You can convert char * to String, but not the other way around.

But I can convert string to char *:
string s = "my string";
char *my_char = new char[s.length()+1];
memcpy(my_char, s.c_str(), s.length() + 1);

Godot String != string

I understand, I thought I was talking about c ++ XD I'll sleep, tomorrow I'll try with string, and more, thanks for the help

Use String, I've used it on a module and it works fine, you can check it here for reference: oamlGodotModule

I do not understand how you're using string without including any godot file for this, or do you just need the "reference"?

Yeah, just include "reference.h" and then you can use String, and to convert to const char * use .ascii()

That's what I needed. Compiling here now. I thank you.

I am getting "undefined reference to" for Enet functions, am I doing it correctly on my SCsub?
# SCsub
Import('env')

module_env = env.Clone()
module_env.add_source_files(env.modules_sources,"*.cpp")
module_env.Append(CXXFLAGS='-lenet')

The last line hould be module_env.Append(LIBS=['enet'])

I have changed and I continue to receive the same.

Try this patch, it compiled fine for me now.
compile.patch.zip

Thank you, compiled, but this is not necessary:
-void unregister_simplep2p_types () {
+ Void unregister_SimpleP2P_types () {

Because the name of my folder is in tiny anyway.

I try to do this:
var simple = SimpleP2P.new()
But I get this error:
SCRIPT ERROR: GDScript::reload: Compile Error: Identifier not found: SimpleP2P
   At: built-in:6.
ERROR: reload: Method/Function Failed, returning: ERR_COMPILATION_FAILED
   At: modules/gdscript/gd_script.cpp:522.

By clicking the button:
ERROR: emit_signal: Error calling method from signal 'pressed': 'Button::_on_Button_pressed': Method not found.
   At: core/object.cpp:1162.

If it says "var simple = SimpleP2P.new ()" the button works, when otherwise I get this error. This is happening when I try to compile into other modules as well.
But the error only appears when I export, when text in the editor works normally.

That's because you have to compile the export templates with the custom module. The default godot export templates don't have the module you're trying to use.

I do not understand, but if I compiled godot, I thought it included my module. That's at least what I understood in the documentation.

You compiled the godot editor not the godot export templates, you have manually compile the export templates you need to use and then replace the default ones. This is always needed when you add a custom c++ module. If you're using Windows read this: http://docs.godotengine.org/en/stable/development/compiling/compiling_for_windows.html#creating-windows-export-templates

I'm debian, I'll see here.

This link then: http://docs.godotengine.org/en/stable/development/compiling/compiling_for_x11.html#building-export-templates

I think I understood, I have to compile the editor, compile the export models, open the editor and put the models in the "custom binary", but I'm still a bit confused of what is done, Godot takes all the c ++ and Create a template? Why do I already think and have more than one module?

I saw it here and I'm getting "ERR_ALREADY_IN_USE" when trying to create the client to connect to the server.

Closing as it's not a bug but more a support thread, but feel free to continue discussing :)


Improve PP

Fix #141

bors d=by-cloud

I think codecov reports are essentially random 🤣

bors r+


Update wrapper to 8.5 RC2

@bot-gradle merge

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.


[Feature] Support Hbase catalog

Describe the feature
Support Hbase catalog
Motivation
Our company needs to manage HBase.
Perhaps I might go and implement it.
I open an issue first.
Describe the solution
No response
Additional context
No response

closed due to https://github.com/apache/gravitino/issues/6019


Write Radius GitHub workflow FAQ/ getting started guide

Engineering Improvement
We should write a FAQ/ developer documentation for getting started with GitHub actions. This should include:

What we use GitHub workflows for
Do's/Don'ts
etc.

AB#10757

Adding release-improvements label since this issue came up during release PIR and would help folks making changes release workflows.


[Bug]: Seed issue with Pipeline Parallel

Your current environment
v0.5.1
🐛 Describe the bug
OpenAI API specifies that you can provide a seed: https://platform.openai.com/docs/api-reference/chat/create#chat-create-seed
This allows reproducibility for example with non-zero temperature parameter.
Currently, any state information is stored/advanced on the driver process only. We need to extend this to the worker actually doing the sampling.

1, got openai.InternalServerError: Internal Server Error when I specified seed in the completion request

@binxuan do you have any other debug information? That shouldn't be expected

Below is my error trace, not sure if this is the root cause.

Traceback (most recent call last):
File "python/ray/_raylet.pyx", line 919, in ray._raylet.prepare_args_internal
File "/home/binxuan/.conda/envs/vllm_dist/lib/python3.10/site-packages/ray/_private/serialization.py", line 519, in serialize
return self._serialize_to_msgpack(value)
File "/home/binxuan/.conda/envs/vllm_dist/lib/python3.10/site-packages/ray/_private/serialization.py", line 497, in _serialize_to_msgpack
pickle5_serialized_object = self._serialize_to_pickle5(
File "/home/binxuan/.conda/envs/vllm_dist/lib/python3.10/site-packages/ray/_private/serialization.py", line 444, in _serialize_to_pickle5
raise e
File "/home/binxuan/.conda/envs/vllm_dist/lib/python3.10/site-packages/ray/_private/serialization.py", line 439, in _serialize_to_pickle5
inband = pickle.dumps(
File "/home/binxuan/.conda/envs/vllm_dist/lib/python3.10/site-packages/ray/cloudpickle/cloudpickle.py", line 1479, in dumps
cp.dump(obj)
File "/home/binxuan/.conda/envs/vllm_dist/lib/python3.10/site-packages/ray/cloudpickle/cloudpickle.py", line 1245, in dump
return super().dump(obj)
TypeError: cannot pickle 'torch._C.Generator' object

@andoorve yes this is the known issue, I should hopefully have time to fix it tomorrow.

Hi @njhill - Hope you have had some chance to fix this issue. Is it available now in the latest version (0.5.2)?

@sekh77 it's not in 0.5.2. I am working on it right now and it should be ready today.

Ok, thanks. Will this also be available for people who are on 0.5.1?

@sekh77 yes, it will be available to everyone! You'll just have to upgrade to the latest version (hopefully will be in a new release in the next day or two).


Combine active faults and component apis

Merging the two active apis into one harmonious one, as well as adding the 'extension' data as per #18

Change list for the new api needs to added in here

This has been added


Documentation issues for vcpkg_install_msbuild

In
https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/vcpkg_install_msbuild.md
the first line says the latest version of the doc lives in the vcpkg repo. Clicking the link for that takes you to a 404.
Then there are two examples listed for projects that use vcpkg_install_msbuild - xalan-c and libimobiledevice - neither uses vcpkg_install_msbuild afaict.

Thanks for fixing. There was another issue in here (the two examples listed for using vcpkg_install_msbuild) that no longer use vckpg_install_msbuild. Should I file a separate issue for that?

@aggieNick02  Yes please.


[enhancement] On Details page we need some mechanism to go to topmost story in that section

We have similar item on list view, is there a way to get that on detail view?

Added Up arrow in bottom bar. Available in release <IP_ADDRESS> / 1.0.2 for testing.

Tested in version <IP_ADDRESS>. working properly.


Error when running npm run deploy

After running sudo npm run deploy (Amazon Linux), I get the following error output:
><EMAIL_ADDRESS>deploy /home/ec2-user/aws-lambda-image
> claudia create --profile $npm_package_config_profile --region $npm_package_config_region --version dev --handler index.handler --no-optional-dependencies --timeout $npm_package_config_timeout --memory $npm_package_config_memory --policies policies/*.json

Source directory is the Node temp directory. Cowardly refusing to fill up disk with recursive copy.

npm ERR! Linux 4.9.32-15.41.amzn1.x86_64
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "run" "deploy"
npm ERR! node v6.11.1
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>deploy: `claudia create --profile $npm_package_config_profile --region $npm_package_config_region --version dev --handler index.handler --no-optional-dependencies --timeout $npm_package_config_timeout --memory $npm_package_config_memory --policies policies/*.json`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>deploy script 'claudia create --profile $npm_package_config_profile --region $npm_package_config_region --version dev --handler index.handler --no-optional-dependencies --timeout $npm_package_config_timeout --memory $npm_package_config_memory --policies policies/*.json'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the aws-lambda-image package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     claudia create --profile $npm_package_config_profile --region $npm_package_config_region --version dev --handler index.handler --no-optional-dependencies --timeout $npm_package_config_timeout --memory $npm_package_config_memory --policies policies/*.json
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs aws-lambda-image
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls aws-lambda-image
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/ec2-user/aws-lambda-image/npm-debug.log

What am I doing wrong?

I'm getting the same error
Windows 10
node v6.11.2
npm  v3.10.10

This might help you. You have to set TMPDIR env var as described here: https://github.com/ysugimoto/aws-lambda-image/issues/125

Hi, thanks for the response. I was able to run Claudia without setting TMPDIR, like so:
"deploy": "claudia create --profile $npm_package_config_profile --region $npm_package_config_region --version dev --handler index.handler --no-optional-dependencies --timeout $npm_package_config_timeout --memory $npm_package_config_memory --policies policies/*.json"
But it seems that I have a different problem, it seems that $npm_package_config_VARNAME vars are undefined, so currently "npm run deploy" breaks because claudia create has no arguments. My package.json file is identical to the latest repository version
Thanks for any insight on this

I have this problem using migrating from node 14 to node 16


New Samples Not Copy-Pastable

This may beg for a lengthier discussion somewhere else, but I'm filing it as a bug here just in case it counts as one.
All the code samples in /samples now use await, but await is only valid in async function, which means that the samples can't be run if someone just copies and pastes the code. This is different from what we are striving for across all our samples.

I hope to have this finalized in the coming days.


fix savelayer in example

Do we need to fix the draft?

Do we need to fix the draft?

Ideally yes. I'll do that now.


WIP! (PUP-5475) Refactor windows service provider to connect directly to windows API

Neither net.exe or sc.exe really satisfy our management needs for windows services, so this PR adds a new Util::Windows::Service ruby module to puppet's utilities that implements service control on windows directly with the windows API via FFI.
Subsequently, this PR updates the service provider to re-implement the various functions of the provider API using the Windows::Service utility rather than win32-service + net.exe

jenkins please test this on windows2008r2-64a

This PR is still VERY WIP.
It's up now for PR testing and early discussion

CLA signed by all contributors.

Thanks for putting this up, Sean! I spent some time looking into this ticket so that I have some context on what's going on here. I have my notes pushed up here if they'd be useful to you: https://github.com/ekinanp/puppet/commit/PUP-5475. Some questions/comments:

Have you looked into the Win32::Service codebase? If so, could we use some of its API calls here? From what I've seen, I don't think so b/c they use the ANSI system calls (E.g. OpenServiceA) while we're using unicode (OpenServiceW). Plus, I kind of like your approach of just creating a separate FFI module. Gives us greater flexibility.

We don't have any acceptance tests for the Windows service provider. The closest thing I could find is https://github.com/puppetlabs/puppet/blob/master/spec/integration/provider/service/windows_spec.rb and those tests don't really test any functionality. They mostly just test to make sure the results returned by the service provider make sense. I think we should add some as part of this ticket, especially once we work on some of the other service improvements in the epic + any future service improvements. It looks like Windows Powershell lets you create services directly from what I'm seeing in https://msdn.microsoft.com/en-us/magazine/mt703436.aspx so we can use that to create stub services we can delete after the tests. I'm interested in looking into this since it's work that can be done independently of what you're doing here.

How are you testing the FFI code? Are you testing it interactively as you develop it in e.g. an interpreter?

@ekinanp per the Win32::service code: we are trying to remove the Win32 gems entirely from puppet. They are old gems for which some might still be active (I think chef might own the service one now) but for the most part we have removed most of them due to their inactivity.
IMO we should keep consistent with the other windows providers and just leverage FFI and the windows API directly, that way we aren't splitting between some providers using FFI and others using gems we aren't sure about the state of.

jenkins please test this on windows2008r2-64a

jenkins please test this on windows2008r2-64a

@ekinanp so, I think wait_for_pending_transition begs a little explination:
The way I designed the usage of that function is the following, (using start as an example, stop is just the opposite anyway):

wait for the service to stop if and only if it's in STOP_PENDING. If the service is in STOP_PENDING, it won't accept any control codes sent to it (including the call to StartService). If the service is not in stop pending or stopped (i.e. if it's paused), just continue and let StartService error with windows messages.
execute StartService (or ControlService for the stop command), if the command fails use the Windows::Error util to report GetLastError()
wait for the service to move from START_PENDING to RUNNING, if the service isn't in those states then actually fail this time that the service wasn't really started

The reason we need to wait the first time on the STOP_PENDING (or START_PENDING for stop()) is MS themselves suggest that services do not accept any control codes or controls in pending states. You can see in the 'best practices' list under https://docs.microsoft.com/en-us/windows/desktop/api/Winsvc/nf-winsvc-setservicestatus#remarks that:
"Do not register to accept controls while the status is SERVICE_START_PENDING or the service can crash. After initialization is completed, accept the SERVICE_CONTROL_STOP code."
and I assume the same goes for SERVICE_STOP_PENDING.
Even in the provided examples from MS like: https://docs.microsoft.com/en-us/windows/desktop/Services/starting-a-service https://docs.microsoft.com/en-us/windows/desktop/Services/stopping-a-service they suggest waiting on the pending transitions of the opposite call (so wait on STOP_PENDING for start(), etc.).
Beyond that reasoning, I figured it would be easier to let StartService and ControlService error when the service was in anything other than stop/start and stop-pending/start-pending rather than fail on the first calls to wait_for_pending_transition, which is why that function returns a boolean false when the state doesn't match pending_state or final_state: https://github.com/puppetlabs/puppet/pull/7017/files#diff-cbf3560d761defff8b026335352495abR553
That might make sense to change, I can always just have it error there instead of returning false if you think that makes more sense.

jenkins please test this on windows2008r2-64a

@McdonaldSeanp Ohhh I missed the boolean returns -- brain fart on my part. Ok, that makes sense. Thank you for clarifying this. I can understand why too -- START_PENDING is when the thread's being created while STOP_PENDING is when it is being killed. PAUSE_PENDING + CONTINUE_PENDING is when the thread's already created so those can safely accept the STOP signal.
I do think that in stop, we should probably not raise an error if the service is in any other state but START_PENDING (I'm ignoring STOP_PENDING and STOPPED b/c Puppet will not call stop if the service is already in those states). I think this is the correct approach based on the transition diagram, and the fact that the docs you linked don't mention there being any issues with sending control signals in those other states. Also, there is a Windows ticket about managing paused services -- https://tickets.puppetlabs.com/browse/PUP-6702. My gut feeling tells me the work for that might require a new ensure value (paused) so it might be good to handle it here while the refactor's still ongoing.

@ekinanp Per the paused states and ticket: I would like to leave work for paused states out of this PR, and leave that ticket as work for someone once this work is merged. That way I can keep this PR as focused on one thing (the migration to FFI) as possible
I think we will need to have a discussion about if it's appropriate to land the paused states work in 5.5.x anyway, so I was going to leave that until after this was done.

jenkins please test this on windows10ent-64a,windows10ent-32a,windows2008r2-64a,windows2012r2-64a,windows2012r2_ja-64a%7Blocale=ja%7D,windows2016-64a

jenkins please test this on windows10ent-64a,windows10ent-32a,windows2008r2-64a,windows2012r2-64a,windows2016-64a

The development part of this PR is ready to go, the only thing left is to finalize what to do with Japanese windows testing.

I am making my way through acceptance/unit testing for services with localized names now, so this remains blocked.
However the functional code should be more or less complete.

testing is finished, this is ready for merge.

jenkins please test this on windows10ent-64a,windows10ent-32a,windows2008r2-64a,windows2012r2-64a,windows2012r2_ja-64a%7Blocale=ja%7D,windows2016-64a

@Iristyle @glennsarti I cross-checked the constants against the Windows 10 SDK, and updated them. I only included the constants used with the functions provided in this PR, so there might be more constants available to other functions but I figured I would leave them out for now to keep this PR focused.
Glenn, I updated the unit tests to be more focused on method behavior rather than testing for correct API invocations, hopefully the tests look a little cleaner.
The private functions are all included inside class << self so I believe the private functions should work., just tell me if it doesn't
Ethan, I'm not sure what we should do about :lpwstr causing failures with FFI::Struct. Should we open an issue with FFI?

jenkins please test this on windows10ent-64a,windows10ent-32a,windows2008r2-64a,windows2012r2-64a,windows2012r2_ja-64a%7Blocale=ja%7D,windows2016-64a

Manual testing of puppet resource with mutation is working, this should be GTG


[BUG] - Switch [selected=true] doesnt work

NextUI Version
2.2.5
Describe the bug
when using custom class names, [selected=true] doesn't work for thumb slots, while if I remove the =true and leave it as [selected] it works
eg group-data-[selected=true]:ml-8 doesn't works
group-data-[selected]:ml-[33px] works
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
custom Switch
{
        base: twMerge(
          'ml-1 max-w-md bg-content1 hover:bg-content2 items-center',
          'rounded-lg gap-2 border-2 border-transparent',
          'data-[selected=true]:border-primary',
          base,
        ),
        wrapper: twMerge(
          'p-1 bg-white group-data-[selected=true]:bg-cxc-orange w-16 flex shadow-lg',
          wrapper,
          wrapperColor,
        ),
        thumb: twMerge(
          '!size-6 border-2 shadow-lg',
          'group-data-[hover=true]:border-primary',
          //selected
          'bg-cxc-gray group-data-[selected=true]:bg-white group-data-[selected=true]:ml-8',
          // pressed
          'group-data-[pressed=true]:w-7',
          'group-data-[selected=true]:group-data-[pressed=true]:ml-8',
          thumb,
        ),
      }

the code provided is based on switch#custom-styles
this happened after changing from "@nextui-org/react": "^2.2.9" to "@nextui-org/switch": "^2.2.5"
Expected behavior
The thumb goes only to about ml-6 while I have a bigger wrapper
Screenshots or Videos
No response
Operating System Version
any
Browser
Other (add additional context)

Is it just being overwritten by thumb? Does it work if you move the args around? If not please provide a minimal, reproducible sandbox @nickaris1

Had the same error, use group-data-[selected=true]:ms-8 instead. This probably happens because the tailwind class that comes with the component on the thumb slot 'group-data-[selected=true]:ms-5' has more specificity than yours. group-data-[selected]:ml-[33px] works because it has more specifity than the default rule used, so it overrides it. I would suggest structuring your tailwind rule the same way the default one is structured, since that way, because of tailwind variants, the default rule will not exist because you have provided one with the same structure.

I should make that more specific. On the reproducible step is all you need to make it so it doesn't work.
when I was in dev  which is refreshing the state on save, the margin did work on the thumb.

thanks group-data-[selected=true]:ms-8 works
the thing that worked for me is to change in the thumb slot the [selected=true]:ml-8 to [selected]:ml-8

Is it just being overwritten by thumb? Does it work if you move the args around? If not please provide a minimal, reproducible sandbox @nickaris1

codesandbox

The custom styles example for Switch in the docs is using the wrong class. As @stefnto suggested, ms is the correct one to use. ms/margin-inline-start is used by default and should not be mixed with ml/margin-left.


OSX: How to open a folder in brackets by right clicking and "open in brackets"

OSX Brackets misses a convenience of opening any folder directly in Brackets or is there any way to achieve it?

Cmd + O
(international keyboards)

@vikasbnsl You can drag a folder to Brackets.

What if I want to open  2  or  more folder(project) at different locations e.g. as we can open as many projects as we want in Xcode.


render dependencies fail if SQL file name contains hyphens

Description
When using render, if a SQL file has a hyphen - in the name, any downstream dependencies that reference it will fail.
For example, a task with
SELECT *
FROM {{combine-homes}}
WHERE ROOMS > 0
AND ROOMS < 50

that references combine-homes.sql fails with jinja2.exceptions.UndefinedError: 'combine' is undefined. If the hyphens are replaced with underscores (combine_homes.sql), it works with no issues.
Desired Behavior
Either hyphens should be allowed in SQL file names, or if this is not feasible, we should be explicit in our docs that they are not allowed and underscores should be used instead.

@ashb what are your thoughts here? Does jinja normally allow hyphens? Seems like this might be some trouble

No, jinja cant allow hyphens.
That is combine - homes -- i.e. a subtract operator. Variables have to follow python identifier rules.
This is un-fixable with jinja.

Technically this would be fixable with some hack solution of operator overloading, buuut I'm not sure we want to do that.

@ashb yes I agree i think we should just set a documentation rule taht your sql file name should be a valid identifier

Is this still relevant @tatiana ?


Unwanted webhook logs coming on chatbot CLI

Rasa version:0.13.7
Python version: 3.6.0
Operating system (windows, osx, ...):Windows
Issue:NLU and core parts are trained. I'm hosting my action server separately. Now, when I turn on my chatbot CLI, basic functionality is working fine but with every response I'm getting webhooks related logs like "<IP_ADDRESS> - - [2019-04-20 18:43:30] "POST /webhooks/rest/webhook?stream=true&token= HTTP/1.1" 200 272 0.021942". How to disable these?
Content of configuration file (config.yml):
language: "en"

pipeline: spacy_sklearn

Content of domain file (domain.yml) (if used & relevant):

you can't, this is just registering that it received a request


[Product Landing Page] Resource Center Cards Block

Develop Resource Center Block , style it for all break points

already completed via Cards block (resource-center theme). See block-library for card variations
https://main--zemax--hlxsites.hlx.page/block-library/blocks/cards


Windows Builds dont work on latest patchlevels Windows 10

panic: dial tcp [::1]:1883: connectex: No connection could be made because the target machine actively refused it.
goroutine 1 [running]:
panic(0x83fd00, 0xc08200c9b0)
/code/go/go/src/runtime/panic.go:481 +0x3f4
main.main()
/code/happy-bubbles/presence-gh/main.go:616 +0x6cc

Nevermind -h gives options :)


make space-T1w to MNI transform more faithful

The current plan, which we think works,
The problem: fmriprep and ciftify space-T1w nifti images are beautifully aligned (when you open them in more viewers) but they have different underlying sforms.  This means that you can't just reslice one to the other using FLIRT (ident.mat) OR concatenate the ident.mat with the ciftify MNI transform to atlas transform the functionals from FMRIPREP.
Solution, (I think) is to use nilearn's nilearn.image.resample_to_image to do the reslicing (this seems to work/not be affected by the underlying different sforms). To get a FLIRT .mat file that also does this, I register the FMRIPREP func file to the nilearn resliced func file...
Will build this as a default behaviour of ciftitify_subject_fmri...also will switch the reg target to be the first image rather than the mean image.

officially done


Istio 1.0.4 Helm chart Grafana persistence errors

Deploying Istio 1.0.4 via Helm chart with
grafana:
  persist: true

I get
Release "istio" does not exist. Installing it now.
Error: release istio failed: Deployment.apps "grafana" is invalid: spec.template.spec.containers[0].volumeMounts[1].name: Not found: "dashboards-istio"

If I massage the chart's deployment.yaml so that the Grafana pod starts then the Grafana pod crashes:
GF_PATHS_DATA='/data/grafana' is not writable.
You may have issues with file permissions, more information here: http://docs.grafana.org/installation/docker/#migration-from-a-previous-version-of-the-docker-container-to-5-1-or-later
t=2018-11-28T20:46:36+0000 lvl=info msg="Starting Grafana" logger=server version=5.2.3 commit=cf4c090 compiled=2018-08-22T08:20:39+0000
t=2018-11-28T20:46:36+0000 lvl=info msg="Config loaded from" logger=settings file=/usr/share/grafana/conf/defaults.ini
t=2018-11-28T20:46:36+0000 lvl=info msg="Config loaded from" logger=settings file=/etc/grafana/grafana.ini
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from command line" logger=settings arg="default.paths.data=/data/grafana"
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from command line" logger=settings arg="default.paths.logs=/var/log/grafana"
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from command line" logger=settings arg="default.paths.plugins=/var/lib/grafana/plugins"
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from command line" logger=settings arg="default.paths.provisioning=/etc/grafana/provisioning"
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from command line" logger=settings arg="default.log.mode=console"
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from Environment variable" logger=settings var="GF_PATHS_DATA=/data/grafana"
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from Environment variable" logger=settings var="GF_PATHS_LOGS=/var/log/grafana"
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from Environment variable" logger=settings var="GF_PATHS_PLUGINS=/var/lib/grafana/plugins"
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from Environment variable" logger=settings var="GF_PATHS_PROVISIONING=/etc/grafana/provisioning"
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from Environment variable" logger=settings var="GF_AUTH_ANONYMOUS_ENABLED=true"
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from Environment variable" logger=settings var="GF_AUTH_ANONYMOUS_ORG_ROLE=Admin"
t=2018-11-28T20:46:36+0000 lvl=info msg="Config overridden from Environment variable" logger=settings var="GF_AUTH_BASIC_ENABLED=false"
t=2018-11-28T20:46:36+0000 lvl=info msg="Path Home" logger=settings path=/usr/share/grafana
t=2018-11-28T20:46:36+0000 lvl=info msg="Path Data" logger=settings path=/data/grafana
t=2018-11-28T20:46:36+0000 lvl=info msg="Path Logs" logger=settings path=/var/log/grafana
t=2018-11-28T20:46:36+0000 lvl=info msg="Path Plugins" logger=settings path=/var/lib/grafana/plugins
t=2018-11-28T20:46:36+0000 lvl=info msg="Path Provisioning" logger=settings path=/etc/grafana/provisioning
t=2018-11-28T20:46:36+0000 lvl=info msg="App mode production" logger=settings
t=2018-11-28T20:46:36+0000 lvl=info msg="Initializing SqlStore" logger=server
t=2018-11-28T20:46:36+0000 lvl=info msg="Connecting to DB" logger=sqlstore dbtype=sqlite3
t=2018-11-28T20:46:36+0000 lvl=info msg="Starting DB migration" logger=migrator
t=2018-11-28T20:46:36+0000 lvl=eror msg="Server shutdown" logger=server reason="Service init failed: Migration failed err: unable to open database file"

I'm not sure about the fix that was merged here.
Making it configurable is fine, but changing from ReadWriteOnce to ReadWriteMany is not a storage mode supported by any of the cloud provider default storageClasses (eg, gcp persistent disks etc) and is only supported if you're running a specific distributed shared storage.

@Stono I copied from https://github.com/istio/istio/pull/8416 which is in Istio 1.1; I found that I had to override with ReadWriteOnce for Amazon EBS FWIW. I guess the thing to do would be to open an issue against the 1.1 code.

Fix merged to release-1.0 with https://github.com/istio/istio/pull/10180.


Docs

Changes Unknown when pulling 80710091e17dcd9c666849b6e313124927301d30 on LinuxBozo:docs into ** on cfpb:milestone8**.

For cfpb/hmda-pilot#303


Allow custom display text

Right now there is now way to specify custom display text for a textarea.
Changing http://github.com/nathanvda/on_the_spot/blob/master/lib/on_the_spot/on_the_spot_helpers.rb#L28 to:
 options.reverse_merge!(:ok_text => t('on_the_spot.ok'),
                         :cancel_text => t('on_the_spot.cancel'),
                         :tooltip => t('on_the_spot.tooltip'),
                         :rows => 5,
                         :columns => 40,
                         :display_text => object.send(field.to_sym).to_s
                        )

  update_url = url_for(:action => 'update_attribute_on_the_spot')

  field_value = options[:display_text]

This would allow custom display text

I'm trying to work with markdown, I'd rather use RedCarpet than a javascript callback, so far I have this:
budgets/show.html.haml
= on_the_spot_edit @budget, :description, type: 'textarea', display_text: markdown(@budget.description), loadurl: description_budget_path(@budget)

routes.rb
resources :budgets do
  collection do
    put :update_attribute_on_the_spot
  end
  member do
    get 'description'
...

budgets_controller.rb
  def description
    @budget = Budget.find(params[:id])
    render :text => @budget.description
  end

application_helper.rb
  def markdown(text)
    Redcarpet::Markdown.new(Redcarpet::Render::HTML.new(hard_wrap: true),
      autolink: true, lax_html_blocks: true, autolink: true, ).render(text).html_safe if text
  end

Everything works as expected, the page loads with the markdown parsed. When edited, the user sees the raw markdown code but the problem is when it is then saved. In JEditable the save action would then return the rendered markdown
"However save.php should return rendered html" - http://www.appelsiini.net/projects/jeditable
but as far as I can tell update_attribute_on_the_spot returns the raw markdown (forgive me if I'm wrong). Is there a simple way to change this behaviour so that it can return anything, in this case the edited text via the markdown helper method?

Hi, definitely this is possible. I will provide some example code later today.
I have done that for a small side-project of mine (though I switched to using pagedown later).

In my view I had the following:
on_the_spot_edit note, :content, :type => :autogrow, :onblur => 'submit',
:url => {:controller => :notes, :action => :update },
:loadurl => note_content_url,
:display_text => display_note_content(note))
Let me explain this a bit: I specify a :loadurl as you had and the :displaytext, but also overwrite the :url that will be used to save the data (and will return the new data in return).
In my helper I had:
def display_note_content(note)
  note.content.blank? ? '' : BlueCloth.new(note.content).to_html.html_safe
end

In my controller my update looked like:
def update
  klass, field, id = params[:id].split('__')
  @note = current_user.notes.find(id)

  if @note.update_attributes(field => params[:value])
    render :text => BlueCloth.new(@note.send(field)).to_html.html_safe, :status => 200
  else
    render :text => @note.errors.full_messages.join("\n"), :status => 422
  end
end

Hope this helps.

Thanks Nathan! I've managed to implement something very similar now.
I might try and work on a feature so that models have a method like ots_FIELDNAME which will return the value of FIELDNAME formatted however you like. e.g.
def ots_description
	BlueCloth.new(description).to_html.html_safe
end

Then the default update_attribute_on_the_spot method can first check if the ots_FIELDNAME property exists and return that, otherwise it'd return the raw FIELDNAME property as it does currently.
This would remove the requirement for the extra controller methods. What do you think?

Something like this perhaps?
lib/on_the_spot/controller_extension.rb
parsed_data = JSON.parse(select_data.gsub("'", '"'))
return_value = object.respond_to?("ots_#{field}") ? "ots_#{field}" : field
render :text => parsed_data[object.send(return_value).to_s]

Yeah, this sounds very reasonable. I would rather have the look-up method or attribute to be configurable. But a very good suggestion. I will try to implement something like that later today.
Indeed, it think it would be very good/easy to not have to implement the method yourself, because it is almost identical.
So I am thinking more along the lines of the following:
 on_the_spot_edit note, :content, :type => :autogrow, :onblur => 'submit',
                    :loadurl => note_content_url,
                    :display_method => :display_content

And this will make sure it will always call the method display_content for that instance.
So you do not have to specify the url and the display_method is also called when doing the first rendering.
What do you think?
Thank you!

That looks great. Actually, I just looked over best_in_place's documentation and it has a :display_as property that behaves similarly to what you have just suggested. If something like the above could be implemented that would be brilliant. Thanks!

This gem is now perfect for me.
Thanks so much.

:D :D :D


Support for xliff state="new"

I'm using poedit to translate xliff 1.2 file generated from an angular application. I have a workflow were new items are created with a <target state="new">. This items aren't handled by poedit but treated as translated. Either if state="new" poedit should treat this as untranslated or show that it needs translations. Currently I have to search through all translations to find the new owns which are not translated.

new items are created with a .

I don't know what you mean by that, something seems missing?
If you could elaborate and describe the workflow in more detail, including providing some example files (or even the angular project, if you can, privately to help@poedit.net), that would be greatly appreciated.

Oh yes, sorry. Was a bit in a hurry. I can sand you a sample file. I use this https://github.com/martinroob/ngx-i18nsupport tool to generate and merge existing translations with new once.

I am bitten by the same problem, an explanation how this happens with Angular can be found in this issue. An minimal example file could read as follows:
<?xml version="1.0" encoding="UTF-8"?>
<xliff xmlns="urn:oasis:names:tc:xliff:document:1.2" version="1.2">
  <file source-language="de" datatype="plaintext" original="ng2.template" target-language="en">
    <body>
      <trans-unit id="extractTest" datatype="html">
        <source>
          Dies ist ein deutscher Text
        </source>
        <target state="new">
          Dies ist ein deutscher Text
        </target>
        <context-group purpose="location">
          <context context-type="sourcefile">app/front/templates/about.html</context>
          <context context-type="linenumber">20</context>
        </context-group>
      </trans-unit>
    </body>
  </file>
</xliff>

Note that the Angular translation tool has (imho unhelpfully) copied over the source text but it does set the state to new which marks it as untranslated. I guess this follows a reasoning along the lines of "better present the original text to the user than nothing", but it breaks the Poedit workflow. When opening the file above I get the following state of Poedit:

The linked issue also contains the following screenshot, where the untranslated unit is really hard to find.

In both cases Poedit happily tells me that the translation status is 100% and does not pull the missing translations to the top.

See the status of the issue and the above commit link: it's been already fixed and released.


[Engine] Update the Wiki to explain the test: all label and intent.

I.e. move the label description to https://github.com/flutter/flutter/issues/13843 to a Wiki.

Done in https://github.com/flutter/flutter/wiki/Engine-pre‐submits-and-post‐submits#post-submit.


Windowing does not work and suggestion to use min-max instead of level-width

version: eef16bcb5e84cf73b570a2b98c99288b4f7fd23f
Here's how my image looks like:

it ranges approximately from about 0 to 2000
According to my confit, window level is between 1 and 2000, so it should be all right:

Changing the level/width doesn't change anything on the appearance of the image.
BTW: I just realized that the windowing specifies 'level' and 'width'. I personally don't find it intuitive (I'd go with 'min-max' range). What does everyone think?

addressed in PR 109.


lwip submodule should point to a tag or branch tip in the remote repository

Currently the lwip submodule points to an arbitrary commit c385f31076b27efb8ee37f00cb5568783a58f299. Most git servers, including GNU Savannah, only allow shallow clones of branch tips and tags. This means if you do a shallow recursive clone of pico-extras (for example, in a CI pipeline), you will get the following error:
Error: error: Server does not allow request for unadvertised object c385f31076b27efb8ee37f00cb5568783a58f299
Error: fatal: Fetched in submodule path 'lib/lwip', but it did not contain c385f31076b27efb8ee37f00cb5568783a58f299. Direct fetching of that commit failed.

User must instead do a full clone of the submodule, which might mean doing a full clone of pico-extras too. This is slow, wastes bandwidth and electricity, uses up extra Actions minutes, and hammers the GNU servers.
This can be fixed by either changing the submodule to point to an existing tag or branch (if branch, you have to keep on updating the submodule whenever the branch tip changes), or by forking the upstream repository and adding a tag to the commit you want to use, and then adjusting the submodule to point to your fork.

I am not sure why it works now. It could be because the remote now points to the github mirror instead of savannah, and github might configure their servers differently. They never used to allow this either though.

um; i'm a bit confused, as this issue seems to refer repeatedly to pico-extras not pico-sdk... in any case it doesn't seem like #1369 is a suitable fix anyway, so won't fixing this for now - please re-open if there is still an issue

This issue was opened when lwip was a submodule of pico-extras. It is now a submodule of pico-sdk, and you moved the issue here when lwip was moved.

ha

anyways, i'm not sure there is much to be done, unless lwip makes a newer tag, right?

It does seem that this is no longer a problem. I am not sure why because github definitely is not advertising https://github.com/lwip-tcpip/lwip/tree/239918ccc173cb2c2a62f41a40fd893f57faf1d6 (you can confirm this with git ls-remote.) It seems to be a change in the behavior of the git command itself, to download the full remote even when you ask for a shallow clone - and then discard anything it doesn't need.


Rename the latlng option to focus

This is to bring this option's terminology closer in line with the Mapzen Search API, where focus is a concept that did not translate particularly well to the geocoder when the option was called latlng. The latlng option will still work for now, but it will display a deprecation warning for users if they continue to use it.
Also, focus is set to true by default, which means autocomplete results should always be a little more relevant to the current view (unless explicitly disabled). This may supercede #65.

:+1: Thanks for making it backwards compatible and adding the deprecation warning.


Ambari 2.0 - Alert system

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/periscope-build-pull-request/10/


Annual Reports

Summary reports for a user (as a submitter or author): a yearly report in HTML and/or doc (important to be editable) of how many submissions, their stats, etc. Basically a summary of Piwik statistics.
Setting in user profile to set whether user wants these reports.

Adding a bit of comment here. Now we are able to see "usage statistics" per item in the repository but it would be great to see total usage statistics (i.e total views and downloads and where they are coming from) for all items in the repository. This feature is important for annual reporting.

Hi Hemed, have a look at this: https://lindat.mff.cuni.cz/en/statistics
It might be pretty close to what you want. I just don't know where that code is, ask guys on Slack.
For annual reports of individual people maybe we need just to add a filter per submitter (or author name?) just like you can now filter per year for instance.  Then all that would remain would be to save these as PDFs (more or less :)

Hi Pavel. Thank you for your feedback.  This looks great!. As you pointed out, it may need some additional infomation and may be the ability to export statistics to PDF or something like that.
Can one also see other countries in "Country Wise Visits"? I will ask the guy on how we could also get this feature.

Implemented in https://github.com/ufal/lindat_piwik_reports.
Integrated to clarin-dev in https://github.com/ufal/clarin-dspace/commit/23f65cd2cf7346ee0173e7b83494eca55155a2a5


Web Chat

How to leave the chat open when entering the site?

Use botmanChatWidget.open()
Section API in https://botman.io/2.0/web-widget

Hi, where do I need to put the botmanChatWidget.open() on my code?
Thanks!


VsTest Console - TestCaseFilter via filter file

Description
The filtering of testcases can only be performed directly in the command line (https://github.com/Microsoft/vstest-docs/blob/master/docs/filter.md).
In my use case scenario, I need to filter individual test cases by FullyQualifiedName causing the arguments to get too long for the command line (>  32,767 characters).
It would be beneficial if it would be possible to pass the test filter via a file.
Steps to reproduce
Build a Filter that is too long for the commandline
"C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe"
/TestCaseFilter:"FullyQualifiedName=MainProject.Test.StringHelperTest.Concat_BothNotNull|FullyQualifiedName=MainProject.Test.StringHelperTest.Add_OnlyPositives|....
Expected behavior
Passing the TestCaseFilter via a filter file
Actual behavior
vstest.console not executable with such a long argument

@bastianbrinkmann VSTest.console accepts response file. Can you try it?
Example command: vstest.console.exe @filewithlenghty.rsp

I am having the same issue with my project.
Is there some way out for this?

There is an option in runsettings: https://github.com/microsoft/vstest/issues/2273#issuecomment-595292726
Another option is using response files. They currently don't work with dotnet test and dotnet vstest, but work just fine for vstest.console. Tracking that bug here: https://github.com/microsoft/vstest/issues/3513


Allow background to be transparent

Icons for notifications should be white with transparent background.
Had to edit the icons manually

Hi @romannurik.
@think-free is right. "Color" property does allow transparency, but "Background Color" property doesn't allow it.
This first image shows color pallet for Background Color property.

Overall it is good tool.
Thanks.

Hi @romannurik.
@think-free is right. "Color" property does allow transparency, but "Background Color" property doesn't allow it.
This first image shows color pallet for Background Color property.

Overall it is good tool.
Thanks.

The "None" shape was intended to allow transparent icons, but on Android all icons have background shapes now, so need to revisit the None shape per #219


Fixed Background of main page

Related Issue

Info about the related issue
Fixed Background of main page
Closes: #7

Describe the changes you've made
Give a clear description what modifications you have made
Modified index.html and index.css files
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Mention any unusual behaviour of your code (Write NA if not)
Any unusual behaviour of your code NA
Additional Info (optional)
Any additional information you want to give
Program : GSSOC'21
Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly whereever it was hard to understand.
[x] My changes generate no new warnings.
[x] Any dependent changes have been merged and published in downstream modules.

Kindly review it asap @rish-singhal and @avinal

Seems okay to me.

Yeah, seems okay!

Worked on this as a part of GSSoC '21.


No answer error handling

Created error handling for no response given by the bot as discordjs will just error out.

Thx


EUZ adjusted price enhancement

Hi,
Are the big mac prices really the same in the entire Euro zone, and if so, shouldn't be there a difference between a Big Mac in Germany and Slovenia or Greece for example?
I would assume that based on the GDP per capita there should be a difference that could be calculated for EUR countries.
Is this something you could consider for a future update?
Thanks,
Florian

Big Mac prices differ across the euro area. The price we use is a GDP-weighted average of member countries.


C_x only support u8

We currently only support C_x as u8. Given that the peer may choose a longer one, this is an interoperability hazard.
As far as I have found, there is no limit to their length in EDHOC, so the implementation will need to pick a limit, which can be reasonably 1, but right now this is not configurable.

As a first step, I'll start wrapping the u8 in a newtype that'll allow changing this later.


scvi - implement reset_parameters()

And also figure out what it's doing now, when we just pass.

Potentially see here for reference
https://github.com/cellarium-ai/cellarium-ml/blob/fc8a2d079222daee6b7222f952e43c44e66ca931/cellarium/ml/models/cellarium_gpt.py#L557-L576

Also see another example here for a different model
https://github.com/cellarium-ai/cellarium-ml/blob/5495aa899e418aac9be12479a7d0b073ee65c791/cellarium/ml/models/probabilistic_pca.py#L97-L103

        def reset_parameters(self) -> None:             # pass             self.z_encoder.encoder.fc_layers.apply(weights_init)             self.z_encoder.mean_encoder.apply(weights_init)             self.z_encoder.var_encoder.apply(weights_init)             self.decoder.px_decoder.fc_layers.apply(weights_init)


VersionByNamespaceConvention and Swagger

I'm trying to integrate both.
But i receive a
System.NotSupportedException:
HTTP method "POST" & path "v1/People" overloaded by actions - Test.Api.V2.Controllers.PeopleController.Post (Test.Api),
Test.Api.V1.Controllers.PeopleController.Post (Test.Api).
Actions require unique method/path combination for Swagger 2.0. Use ConflictingActionsResolver as a workaround

My Startup.cs is equal to the sample with
 options.Conventions.Add(new VersionByNamespaceConvention());
Any advice?

I'm having to speculate here, but based on the error message, it looks like you're using inheritance. That's going to cause you grief. I know folks try and want to do it, but APIs cannot be inherited; it's purely an implementation convenience. Clearly, the POST action is being discovered from both v1 and v2.
If you can share your controller definitions (at least the relevant signatures), then I should be able to provide some suggestions.

 [ApiController]
    [Route("[controller]")]
    [Route("v{version:apiVersion}/[controller]")]
    public class PeopleController : ControllerBase
    {
        /// <summary>
        /// Gets a single order.
        /// </summary>
        /// <param name="id">The requested order identifier.</param>
        /// <returns>The requested order.</returns>
        /// <response code="200">The order was successfully retrieved.</response>
        /// <response code="404">The order does not exist.</response>
        [HttpGet("{id:int}")]
        [Produces("application/json")]
        [ProducesResponseType(typeof(Person), 200)]
        [ProducesResponseType(404)]
        public IActionResult Get(int id) => Ok(new Person() { Id = id, FirstName = "Foo", LastName = "Bar" });

        /// <summary>
        /// Places a new order.
        /// </summary>
        /// <param name="order">The order to place.</param>
        /// <returns>The created order.</returns>
        /// <response code="201">The order was successfully placed.</response>
        /// <response code="400">The order is invalid.</response>
        [HttpPost]
        [MapToApiVersion("1.0")]
        [Produces("application/json")]
        [ProducesResponseType(typeof(Person), 201)]
        [ProducesResponseType(400)]
        public IActionResult Post([FromBody] Person person)
        {
            person.Id = 42;
            return CreatedAtAction(nameof(Get), new { id = person.Id }, person);
        }
    }

Controller v2 is the same but different namespace

Is it using inheritance?  It would seem may not.
Since you said they are the same, the issue is likely that you're using [MapToApiVersion("1.0")] in the POST action, which is unnecessary. This is a copy & paste error causing the issue in v2.  If you called the v2 API, I won't even expect it to work/resolve.

 ;) No worries.


I get message No adapters available

After command dbt init I get message .
No adapters available. Go to https://docs.getdbt.com/docs/available-adapters
python 3.8.9
dbt-exasol 1.4.0

Hi @prochanev-thw ,
unfortunately we are missing the two files for dbt to "see" that dbt-exasol is available and the profile template that is needed for dbt init - as requested here. We will be adding those two files now and backport to 1.2, 1.3 & 1.4.
In the meantime, you will have to manually create the project as described here.
Thx for letting us know and BR,
Torsten

Should be fixed in 1.3.1 and 1.4.1


Why doesn't adding overlapSize not increasing the total number of embeddings/chunks?

So I experiment with the chunk sizes and decided to use overlapSize. But what I encountered is that no matter whether I add it or not, it doesn't change anything in my search results nor in the total number of embeddings.
I use this code inside my ContentView (the one that was adviced in one of the previous issues on how to change chunksSize):
let splitter = RecursiveTokenSplitter(withTokenizer: BertTokenizer())
let (splitText, _) = splitter.split(text: documentText, chunkSize: 200, overlapSize: 100)
chunks = splitText

For recursive token splitter, it doesn't actually use overlap size in the algorithm, it just has it there to conform to the protocol.  It would be possible to do this ad-hoc since the chunk tokens are also returned as the second part of the tuple.

What do you mean ad-hoc? Sorry, im just starting out and don't understand what you mean.

Ad-hoc as in after the fact. Since splitText is an array, you could loop through the array and add some portion of the text from the end of the previous chunk to the start of the next chunk.
The reason it's not changing the total number of embeddings is because RecursiveTokenSplitter is setup to specifically get as close to the chunkSize without going over, so either way - overlapSize won't impact your total, only changes in chunkSize will affect this.

I added an overlapping functionality and it seems to be working:
        let overlapSize = 50
        let splitter = RecursiveTokenSplitter(withTokenizer: BertTokenizer())
        let (splitText, _) = splitter.split(text: documentText, chunkSize: 300, overlapSize: overlapSize)

        chunks = splitText
        embeddings = []

        for i in 0..<splitText.count {
            let previousChunk = i > 0 ? splitText[i-1].suffix(overlapSize) : ""
            let currentChunk = splitText[i]

            let chunkWithOverlap = previousChunk + currentChunk
            chunks.append(String(chunkWithOverlap))
        }

Resolved with #14

Noob question but it doesn't seem to work. I tried with multiple chunk sizes and overlap size but it never produce the correct answer like when I use with RecursiveTokenSplitter
Here is the warning I got when using RecursiveCharacterSplitter, chunksize 1000, overlapSize 200.
Input sentence is too long 9430 > 512, truncating.

A similarity search always gives 0 search results.

As far as I'm concerned, the embeddings model is 512 tokens long, so each chunk cannot exceed this limit anyways.

Yea, something seems suspicious about that large 9430 number, but the chunk size should definitely be a max of 510 since the model requires 2 tokens for the input and can only accept 512 total.


Wrong scroll end event on scroll example page

How to reproduce:

Open http://mobileangularui.com/demo/#/ in mobile browser Dolphin
Open left menu
Select scroll example
Scroll to the end of list and read alert() text
Now try to scroll up...
Fall...

Mobile browser: Dolphin
User agent: Mozilla/5.0 (Linux; U; Android 4.4.4; ru-ru; GT-I9300 Build/KTU84Q) AppleWebKit/537.16 (KHTML, like Gecko) Version/4.0 Mobile Safari/537.16

Also, working just fine in Chrome-based mobile browsers


Add more unit tests

Codecov Report

Merging #5 into master will increase coverage by 21.28%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master       #5       +/-   ##
===========================================
+ Coverage    59.1%   80.38%   +21.28%
===========================================
  Files           8        8
  Lines         313      413      +100
  Branches       10       15        +5
===========================================
+ Hits          185      332      +147
+ Misses        125       81       -44
+ Partials        3        0        -3

Impacted Files
Coverage Δ

sdc/rabbit/publisher.py
100% <100%> (+23.91%)
:arrow_up:

sdc/rabbit/test/test_consumer.py
100% <100%> (+2.77%)
:arrow_up:

sdc/rabbit/test/test_publisher.py
100% <100%> (+2.56%)
:arrow_up:

sdc/rabbit/consumer.py
100% <100%> (+65.38%)
:arrow_up:

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 766f3ad...7c9e60a. Read the comment docs.


playwright latest version installation with browser

Install playwright
Install [chromium, firefox, webkit] browser

Installation complete


Disable xdp on ubuntu 22.04

Ubuntu 22.04 stop using xdp as MsQuic drop support.

Note that this PR has a merge conflict with: https://github.com/microsoft/netperf/pull/266/files, but we can first merge this PR so we stop testing on un-supported builds.

The conflicts is expected.
Are these CI failures expected?


Line host all all all md5 not appended when using POSTGRES_PASSWORD_FILE

When setting the password with the direct POSTGRES_PASSWORD environment variable, everything works fine: I am able to connect to the database and create tables and stuff.
However, with the exact same docker-compose configuration, but using POSTGRES_PASSWORD_FILE instead (pointing to bind-mounted read-only file inside the container), then trying to connect to the database ends up with the no pg_hba.conf entry for host error.
I created both versions and looked at the configuration in $PGDATA/pg_hba.conf, and the one using POSTGRES_PASSWORD_FILE is missing the host all all all md5 line.
Here is a repository to reproduce, and here is the related DBA StackExchange post.
If I understand correctly the docker-entrypoint.sh script, the configuration should not change depending on the use of POSTGRES_PASSWORD or POSTGRES_PASSWORD_FILE. The file_env function simply fills the first one thanks to the second:
https://github.com/docker-library/postgres/blob/1805adb0693d9602bfb19b6bf2583b311c43b749/10/docker-entrypoint.sh#L8-L24
Then the host all all all md5 should be appended in this block:
https://github.com/docker-library/postgres/blob/1805adb0693d9602bfb19b6bf2583b311c43b749/10/docker-entrypoint.sh#L65-L92
Am I missing something or is it really unwanted behavior?

I can't seem to reproduce a failure as described here:
$ docker pull postgres:10
10: Pulling from library/postgres
Digest: sha256:d5787305ec0a3b9a24d0108cb5fdbb4befbd809f85639bf04aa1941138df9701
Status: Image is up to date for postgres:10

$ echo db-password > test

$ docker run -dit --name pg -v "$PWD/test":/passfile:ro -e POSTGRES_PASSWORD_FILE=/passfile postgres:10
bf1bfe8dd0fc92a4e36a9f1b75033aa012f610cb7c4f8897324f56fbf23c4aee

$ docker logs --tail=1 pg
2018-04-25 17:53:48.244 UTC [1] LOG:  database system is ready to accept connections

$ docker run -it --rm --link pg postgres:10 psql -h pg -U postgres
Password for user postgres:
psql (10.3 (Debian 10.3-1.pgdg90+1))
Type "help" for help.

postgres=# SELECT 1;
 ?column?
----------
        1
(1 row)

postgres=#

Well, I was not able to reproduce either on another machine. Also now on the problematic machine, the configuration without _FILE results in the same error... I don't understand what's wrong on this machine... I'm gonna try to investigate further but I think it's not an issue with this repository.
Thanks for trying it out @tianon

So if I remove the set -e line for the docker-entrypoint.sh script, everything works fine, which leads me to think that the entry point exits too early when I build start the container, missing to append the host all all all md5 line.

Got it. It was a permission issue.
database_1   | /usr/local/bin/docker-entrypoint.sh: line 19: /run/secrets/db_pass: Permission denied

My password file had -rw------- pawamoy pawamoy permissions. I added r for group and others, as well as r-x for group and others on its parent directory. That fixed the issue.

Additional details:
Since I'm binding sensitive files inside the container, setting read/execution permission for others is not very secure.
Instead, I now change the group ownership of the sensitive files to the ID of the user inside the container. In that case, the user is postgres, and that user is created with ID 999 in the Dockerfile.
In fact, it makes sense because Docker on my system is installed with that same ID 999 (is it like this for everybody?). So docker-postgres is actually doing a good job setting the ID of their postgres user to 999 because that way you can simply give ownership to docker on your system host, and that will be converted to postgres ownership in the container.
Is this the standard / logical way of doing things?

It is not something that we tried to orchestrate.  Matching the uid of your host docker user is just a coincidence.  Initially we just created the user and it happened to be 999, but it was improved to keep it 999 in the future so we don't break users:
https://github.com/docker-library/postgres/blob/01cf83804758d9067a7c75ba5349ecba71137f4d/Dockerfile-debian.template#L15


Allow creation of a bag if the directory already exists.

Resolve #7.
This is a minimal version of a possible implementation.  For the moment, it only adds a "create" flag to the constructor, as discussed in the issue, and ignores OSErrors from mkdir if the directory already exists in _create_bag().  It does not check whether any of the meta files (bagit.txt, ...) in the bag directory already exist and boldly overwrites them.  Thanks to the constructor flag, it does not change the behavior of pybagit, unless explicitly requested.
I think, there are three possible ways to proceed from here:

Take the PR as it is now.  It does the job and fits at least for my use case.  I care only for using already existing data, but not for meta files.  This works now.
Check if any meta file exists in the bag directory and throw an error if this is the case.  This would prevent overwriting existing files.
If any meta file exists, read it and adapt the newly created bag accordingly rather then writing this file.

Nr. 3 could become cumbersome and I'm not sure whether there is any real use case for it.
Any opinions?

Ping!
Could you have a look at the PR and comment on which variant you would prefer?


issue with running CMake (maybe ADD_REQUIRED_DEPENDENCY macro)

Hi,
I've been trying to compile gepetto-viewer on a Ubuntu distribution.
When I run cmake, however, I get the following error message.
-- openscenegraph >= 3.2 is required.
-- Checking for module 'openscenegraph >= 3.2'
--
CMake Error at /usr/share/cmake-3.5/Modules/FindPkgConfig.cmake:367 (message):
  A required package was not found
Call Stack (most recent call first):
  /usr/share/cmake-3.5/Modules/FindPkgConfig.cmake:532 (_pkg_check_modules_internal)
  cmake/pkg-config.cmake:288 (PKG_CHECK_MODULES)
  cmake/pkg-config.cmake:459 (ADD_DEPENDENCY)
  CMakeLists.txt:75 (ADD_REQUIRED_DEPENDENCY)

The issue seems to be related to the ADD_REQUIRED_DEPENDENCY macro in cmake/pkg-config.cmake failing to find openscenegraph.
openscenegraph is installed in /usr/local/lib64/; I tried both compiling and installing with apt-get install and it does not seem to make a difference.
I'm not used to the macro system being used for adding libraries to this project.
Any suggestion on how to fix/troubleshoot the issue?

/usr/local/lib64/pkgconfig  must be in the PKG_CONFIG_PATH environment variable.

That solved my issue; thank you.
Just for the record, when making the project I was getting error: ‘to_string’ is not a member of ‘std’ errors, which I managed to solve by switching from gcc5 to gcc6.


[ADD]: GAME NAMED "I WANT TO GOOGLE THE GAME" 👾

PR Description 📜
This PR is about adding a game named "I Want to Google the Game"
Fixes #892

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots 📸

Working Video 📹
https://github.com/kunjgit/GameZone/assets/92252895/da951802-6d2d-40bd-a866-b11aae1dffeb

[ The video can lag due to my low end pc, sorry for the inconvenience ]

Thank you soo much for contributing to our repository 💗

Thank you @0xabdulkhalid ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@0xabdulkhalid LGTM

Thank you  @0xabdulkhalid , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


hack/bootstrap.sh blocks on WSL2

Oddly, calling just run-wasi on WSL2 hangs indefinitely. After doing a bit of digging, I found that it was because hack/bootstrap.sh blocks on WSL2.
Changing this line:
token_secret=$(< /dev/random tr -dc a-z0-9 | head -c${1:-16};echo;)

To use /dev/urandom instead appears to work:
token_secret=$(< /dev/urandom tr -dc a-z0-9 | head -c${1:-16};echo;)

It looks as though reading from /dev/random blocks indefinitely.

Just got bitten by this and confirm changing to /dev/urandom fixed it for me.

Also now having a problem doing just run-wasi and just run-wascc concurrently:
Error: ApiError: certificatesigningrequests.certificates.k8s.io "hestia-tls" already exists: AlreadyExists (ErrorResponse { status: "Failure", message: "certificatesigningrequests.certificates.k8s.io \"hestia-tls\" already exists", reason: "AlreadyExists", code: 409 })

Caused by:
    certificatesigningrequests.certificates.k8s.io "hestia-tls" already exists: AlreadyExists

(Here hestia is the name of the local machine.)

And I don't know if this is related but I'm now seeing integration tests failing to create the hello-wasi pod with an image pull error.  Don't think this is related to anything I've done because I've backed out all my changes... @bacongobbler are you able to run the integration tests on WSL?

I saw the same as well. I'll have a look first thing tomorrow!

Pod log message:
{"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"container \"hello-wasi\" in pod \"hello-wasi\" is waiting to start: trying and failing to pull image","reason":"BadRequest","code":400}

If I understand this right, it looks like it's trying to schedule it on the docker-desktop node, which obviously won't work because it's not a containery container...
16:56 $ kubectl get pod -o wide
NAME         READY   STATUS         RESTARTS   AGE   IP          NODE             NOMINATED NODE   READINESS GATES
hello-wasi   0/1     ErrImagePull   0          19s   <IP_ADDRESS>   docker-desktop   <none>           <none>

If I roll back to before the first bootstrapping commit, the pods are scheduled to the appropriate krustlet-* nodes.  Possible red herring, but has something changed in the taints and tolerations stuff?

Deleting the node and letting just run-wasi recreate it seemed to fix the scheduling problem - the integration test pods are now scheduled to the Krustlet nodes.  I noticed a change in #275 that the start method would // Create the node. If it already exists, this will exit where previously it would "adopt" the node definition (kubelet.rs, line 64).  So maybe having nodes around that were created by older builds of krustlet-wasi and krustlet-wascc was the problem, and the switch from adoption to exiting meant that the new kubelet didn't get run?
However, even with the pod scheduled and running, I now get a new integration test failure: the WASI pod logs, instead of commencing with Hello, world! are now reporting "{\"kind\":\"Status\",\"apiVersion\":\"v1\",\"metadata\":{},\"status\":\"Failure\",\"message\":\"Get https://<IP_ADDRESS>:3001/containerLogs/default/hello-wasi/hello-wasi: tls: received empty certificates message\",\"code\":500}\n"...
(And just run-wascc still fails too.)

That looks like a cert issue @itowlson. Which means that something isn't working correctly with the serving cert bootstrap process

Also now having a problem doing just run-wasi and just run-wascc concurrently (which I need to do to run the e2e tests):
Error: ApiError: certificatesigningrequests.certificates.k8s.io "hestia-tls" already exists: AlreadyExists
(ErrorResponse { status: "Failure", message: "certificatesigningrequests.certificates.k8s.io \"hestia-tls\"
already exists", reason: "AlreadyExists", code: 409 })

Caused by:
    certificatesigningrequests.certificates.k8s.io "hestia-tls" already exists: AlreadyExists

(Here hestia is the name of the local machine.)

☝ Also, with this, it is due to the hostname being the same. In the e2e tests I delete the CSR before starting the other. I figured this would only be a problem locally (as you would probably not be running a double krustlet on a normal node in a "real" environment), so maybe the run-wasi and run-wascc should just have something that does kubectl delete csr $(hostname)-tls || true

Oh and @bacongobbler I should have documented this:

It looks as though reading from /dev/random blocks indefinitely.

That can happen on specific systems where it doesn't have the entropy to read from it. Changing it would make it insecure though. I think that we should probably set a variable like RANDOM_SOURCE with the default being /dev/random and an override in our justfile to use /dev/urandom

bash exposes $RANDOM, where each read is unique. Not sure if it's cryptographically secure either, but that could be used as a source, and because it's bash it should work cross-platform.
Curious to see if there's prior art here for other projects