index support for LIKE Operator wanted

We are facing performance issue when tried with like queries.
The field is indexed with full text.
Query used:
FOR node IN GROUP
    FOR acc IN account
        FILTER node.acc_key = acc.acc_key AND acc.name like '%abc%'
        LIMIT 10
    RETURN acc

When I remove the %, it is taking only < 5 ms, Otherwise it is almost 70 seconds.

LIKE cannot utilize any index, a full collection scan is required. Fulltext indexes support prefix matches, but you need to call FULLTEXT() to use it. Suffix matches or having wildcards on both sides are not possible, it would require a different type of index, which would consume quite some memory.
How much data are you scanning there?

I am scanning 18 million records.Is there any alternative as this is a critical use case for us.
Thanks,
Sajeev

do I get that correctly that you're missing a FILTER in front of node.acc_key = acc.acc_key and can you revalidate with explain that this part uses an index?
How much does it reduce the dataset? Means - on how many remaining documents do you have to do the like ?

If you really have to partial-match a lot of records, a workaround could be to store n-grams of the name in a separate attribute (managed by your application) and put an array index on it:
{
    acc: {
        name: "FooBar",
        ngrams: ["foo", "oob", "oba", "bar"]
    }
}

Note that a minimum search phrase length of 3 is assumed. When searching in the ngrams, you can use a hash array index to search for 3 consecutive letters, even if the search phrase is longer. This should already reduce the result set, and you can then check if the rest matches in a second step.
But be sure to check out dothebart's remark first.

@dothebart  ,Actually the FILTER is there
@Simran-B - the solution seems to be very difficult to manage. Are you planning to support like queries with index as this is one of the basic requirements for any project.

@dothebart  - this part - node.acc_key = acc.acc_key uses an index.

We're currently not yet set on the post 3.0 Roadmap, so I can't give you an ETA on whether & wen support for this is going to be implemented. So for the time being you will have to find a different solution to reduce the number of hits first so that the number of documents to actually be inspected by the LIKE filter gets into an acceptable range.
If you're searching for complete words, the fulltext index could be such a way; If you're able to reduce it to a right side match, you could use a skiplist index with range queries (>, >= ...).

ok. Hope to see this feature in the upcoming releases. I think incorporating this feature can replace the search engines like SOLR and elastic search which are lacking the join feature.

I agree that there is a need for efficient partial string matching. Have a look at another discussion regarding this topic: #1796
There are some interesting papers like this one: http://pages.di.unipi.it/grossi/PAPERS/sicomp05.pdf

@Simran-B , Is this feature included as part of the road map and any ETA if it is included in your list.

@Simran-B , Please let me know if this feature would be available in the upcoming releases.
Thanks

came across "What about substring search on an index (fast LIKE)? It can be made fast with a trigram index. [8][9]"
[8] https://about.gitlab.com/2016/03/18/fast-search-using-postgresql-trigram-indexes/
[9] http://blog.scoutapp.com/articles/2016/07/12/how-to-make-text-searches-in-postgresql-faster-with-trigram-similarity

It's almost 4 years, can we use trigram built-in on ArangoDb? As I don't want to install PostgreSQL just to use faster LIKE operator text search.

Hi @YukiSakuraVN
LIKE operator for text search will be part of the upgrades to ArangoSearch in the upcoming ArangoDB 3.7 release. The operator will make use of indexes for fast query execution.
Just FYI: The upgrades to ArangoSearch also include fuzzy search (ngram & Levenshtein), improved phrase & proximity search, wildcard search and a few improvements more You can find a few more details here https://www.arangodb.com/arangodb-training-center/search/arangosearch-tutorial-3-7/ and in our docs https://www.arangodb.com/docs/devel/aql/functions-arangosearch.html
Hope this was helpful

Hi @YukiSakuraVN
LIKE operator for text search will be part of the upgrades to ArangoSearch in the upcoming ArangoDB 3.7 release. The operator will make use of indexes for fast query execution.
Just FYI: The upgrades to ArangoSearch also include fuzzy search (ngram & Levenshtein), improved phrase & proximity search, wildcard search and a few improvements more You can find a few more details here https://www.arangodb.com/arangodb-training-center/search/arangosearch-tutorial-3-7/ and in our docs https://www.arangodb.com/docs/devel/aql/functions-arangosearch.html
Hope this was helpful

That sounds great. Looking for the release. Thank you very much.

Hi,
ArangoDB 3.7.2 has been released and is available for download.
For more details please check out: https://www.arangodb.com/2020/08/arangodb-3-7-a-big-step-forward-for-multi-model/


Added a helper function to extract joint groups.

This commit creates a helper function that copies OpenRAVE's
hardcoded table of joint derivative names and adds an accessor
function for retrieving them from a configuration specification.
This allows users to directly query the n-th derivative group of
a trajectory without having to iterate through other derivatives.
I could not think of a cleaner way to do this, but at least this
avoid having the values hard-coded in multiple places.
Note that this function only handles joint value derivatives.  A
corresponding function for affine derivatives could be added in the
future.

It's sad that this is necessary, but it is a very useful function to have.


add images folder to prevent no No such file or directory Error

#47

fix time logger fail to log on ret_result

Thanks for pull request.
Can I ask why the code bellow is required?
if os.name == 'nt':
    WK_VER_ROOT = r'C:\Program Files (x86)\Windows Kits\10\Include\10.0.14393.0'
    WK_INCLUDE = WK_VER_ROOT + r'\ucrt'
    WK_LIB_UM = WK_VER_ROOT + r'\um'
    WK_LIB_UCRT64 = WK_VER_ROOT + r'\ucrt\x64'

if os.path.isdir(WK_INCLUDE):
    os.environ['INCLUDE'] = WK_INCLUDE
 else:
     print('Include Path for Windows Kit not exists: ' + WK_INCLUDE)

 if os.path.isdir(WK_LIB_UM) and os.path.isdir(WK_LIB_UCRT64):
     os.environ['LIB'] = WK_LIB_UM + ';' + WK_LIB_UCRT64
 else:
     print('Lib Path for Windows Kit not exists: ' + WK_LIB_UM + ';' + WK_LIB_UCRT64)

It is for running NVCC on windows.
https://github.com/pfnet/PaintsChainer/issues/14
https://devtalk.nvidia.com/default/topic/969047/cuda-8-vs2015-corecrt-h-error/?offset=7

I merged it thanks


Rename our tools package

For sanity, we should make the following changes to our tools package naming:

[ ] directory flutter/packages/flutter_tools/* => flutter/packages/tools/*
[x] package name sky_tools => flutter_tools

I get the feeling I'm the only one who cares about this, so maybe we should just close this bug. Having the directories be named the same as the packages makes sense, even if it means that tab-completion doesn't work well.

Ok, I'm closing this one and #291. I seem to be the only one who wants this, and there are valid arguments against, in particular the clarity that comes from the directory name matching the package name. The only arguments in favour are tab completion and aesthetics.


Typescript support?

Hi Guys,
I would like to use your control but I am using typescript. Do you have support for this?

Just updating here... while I have no interest in TypeScript personally, Vue 3.0 will apparently remove much of the pain of adding it to a Vue component, so I will re-evaluate this when that is rolled out.


Fix Release Pipeline

Summary

Update goreleaser.yaml to new version.
Update github actions.
Fix docker publish.
Update Dagger workflow for release.

@Vad1mo ready for review.
I have done a test release in my github.
Published Images: https://registry.bupd.xyz/harbor/projects/1/repositories/harbor-cli/artifacts-tab
Github Release: https://github.com/bupd/harbor-cli/releases


chore(release): v0.4.2 🎉

:robot: I have created a release beep boop
0.4.2 (2024-06-07)
Bug Fixes

refs: unblock extract implicit refs in multi candidate suffix situations (4aa9891)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/sql-dao-generator/releases/tag/v0.4.2 :sunflower:


Problems encountered in playing iheartradio

Hi AVS team,
We find "error Server does not support seeking" in playing iheartradio,please help me check it.For more log,please see log file.We are using SDK 1.2.
Gst-error .txt
thanks.
E MediaPlayer:handleBusMessageError:source=source,error=Server does not support seeking.,debug=gstsouphttpsrc.c(1484): gst_soup_http_src_do_request (): /GstPipeline:audio-pipeline/GstURIDecodeBin:decoder/GstSoupHTTPSrc:source:
Server does not accept Range HTTP header, URL: http://c8icyelb.prod.playlists.ihrhls.com/2525_icy?at=1&fb_broadcast=false&deviceid=amzn1.account.AEYUG43ZOTWXICM2Y2Y3FFODEZOQ&callLetters=WWDC-FM&streamid=2525&clienttype=amazon&iheartradioversion=1.0&osversion=1.0&devicename=Indigo&terminalid=304&init_id=8169&profileid=400706100, Redirect to: (NULL)

Hi cuizhenke,
can you clarify what you mean by seeking?
iHeartRadio is a radio station that does not support seeking (no skip, no forward, no next/previous). They also have custom stations that support next and previous songs, but not seeking within songs.

Hi @jjamazon ,
I didn't do the seek operation.The problem appear when I play the iHeartRadio station normally.When it appears,the MediaPlayer doesn't work.Have you ever met with this problem?How to solve?
thanks.

Hi @cuizhenke,
Is this issue reproducible for you, or was it an isolated/rare intermittent failure?  Looking at the logs, it appears that you had a stutter (buffer underrun), and this somehow resulted in a seek, but I can't see precisely why a seek would have been attempted (the SDK code doesn't appear to be doing it directly).
Ken

Hi @cuizhenke,
Please try upgrading your SDK version and see if you're still having issues. I'm going to close this for now due to inactivity but please reopen if you're still seeing issues.
Thanks,
Sanjay


Memory/FileJobManagement

Renamed StringJobManagement to MemoryJobManagement and moved the class to the main source set, added tests for MemoryJobManagement and FileJobManagement.

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests


Add an option for HasX fields

Just as another way to handle optionality.  Although mostly it's turning out to be an issue in inputs, not results, where this is less useful!

Generics are probably strictly better.


[incubator/elasticsearch] Remove helm.sh/created annotations

This is a split-off of PR #425 to just contain the chart 'incubator/elasticsearch'.

@k8s-bot ok to test


As a CIF ops team member, I want to enter milestone information, so that I can track their status and completion

Description:
A milestone has input fields: description, type, max milestone amount, report due date, substantial completion date, professional designation, received date.
It would be good to have 1 milestone report empty by default. Option to add another milestone is card #595

Full wireframe: https://www.figma.com/file/PzOY8RyQnVXnjZJdSv2AqQ/Wireframing_Figma?node-id=1209%3A2663
Acceptance Criteria:
Given that I am creating or editing a project
When clicking on the "Milestones" link in the task list
Then I am redirected to the /cif/project-revision//form/milestones page
Given that I am viewing the "Milestones" form
When clicking the "Add a Milestone" button
Then a Milestones is added to the page, below existing milestones, if any.
From the development point of view, this means creating a form_change record for the reporting_requirement table
Given that I am viewing the "Milestones" form
When clicking the "Delete" button
Then a Milestone is removed from the page.
From the development point of view, this means archiving or deleting the corresponding form_change, using the useDiscardFormChange mutation
Given that I am viewing the "Milestones" form
I can define the following information:

Milestone Description: textarea, required
Milestone Type: dropdown, required, options: General; Advanced; Reporting; Performance; Project Summary Report;
Maximum Milestone Amount: radio, required, values = "Enter amount:"; "Not applicable"

Maximum Milestone Amount amount: number [preferably formatted as Canadian/US currency: $0.00], required if "Enter amount;" selected

Report Due Date: datepicker, required
Substantial Completion Deadline: datepicker, see #593
Professional Designation: dropdown, required, options: Professional Engineer; Chartered Professional Accountant,

Professional Engineer is pre-populated since it's the most common

Received Date: datepicker, optional

Development Checklist:

[ ] Same as quarterly and annual reports
[ ] A computed column for the milestone reports
[ ] A component, with a query with a connection to fetch the milestones
[ ] A mutation to create a new milestone report form_change
[ ] Editing and Deleting milestones form changes using exising mutations
[ ] Reports shown in order in which they were created
[ ] form validation works
[ ] tests: jest and e2e + a happo screenshot
[ ] Meets the DOD

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

[ ] User story is included
[ ] User role and type are identified
[ ] Acceptance criteria are included
[ ] Wireframes are included (if required)
[ ] Design / Solution is accepted by Product Owner
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

·Definition of Done (Note: If any of these points are not applicable, mark N/A)

[ ] Acceptance criteria are tested by the CI pipeline
[ ] UI meets accessibility requirements
[ ] Configuration changes are documented, documentation and designs are updated
[ ] Passes code peer-review
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Ticket is ready to be merged to main branch
[ ] Can be demoed in Sprint Review
[ ] Bugs or future work cards are identified and created
[ ] Reviewed and approved by Product Owner

Notes:

Good idea to keep the first iteration aa simple form, the interface doesn't differ much amongst milestone types. We can add plug the dependencies in subsequent iterations. Listing them here for future reference –
Different Milestone types would trigger a few changes on the Milestone report form. For instance:

For a 'Reporting' milestone type, there is no funding associated with it, so Maximum Funding Amount can be set to 'Not Applicable' by default.
The time period for substantial completion date differs depending on the milestone type. It is 30 days for general milestones, 60 days for project summary report (IA - final milestone) and 150 days for TEIMP report (EP Final Milestone). We can set the substantial completion date automatically depending on the milestone type chosen.
IA Funding stream typically has only Advanced Milestone + Project Summary Report Milestone only. There may be Reporting moving forward. Perhaps we can automate this funding stream type to have two kinds of milestones created by default once the funding stream is selected.

Hey team! Please add your planning poker estimate with ZenHub @BCerki @dleard @gurjmatharu @matthieu-foucault @mikevespi @Sepehr-Sobhani

@pbastia I see the development checklist includes "Editing and Deleting milestones form changes using existing mutations". I was going to create a separate ticket for deleting milestones, is that necessary?

@elliottSlee Not really. We're reusing components and patterns built for all reporting requirements, no new feature is needed for that

Thank you @dleard ! I tested in dev by adding, editing and deleting milestones.
A couple of observations during testing:

When I went to edit an existing project, there was no 'edit button' for the milestone report like there are for the other forms. Not sure if this was intentional? So, as a user, if I clicked the milestone in the task list, it was automatically editable.

time out issues: I was logged out every 2 minutes and the 'remain active' button did not work (i.e., it wouldn't keep me logged in). I believe this will be covered in ticket #429)

Good observations, there is a card in the sprint backlog that will create a 'read-only' version of milestones to be in line with the other pages (not editable right away & show an edit button) #587
Yes, I think the timeout issue will be covered in #429


Plugin doesn't work anymore

Describe the bug
Simply doesn't work anymore, all emotes behind nitro are grey-ed out, no autocompletion or anything
To Reproduce
Steps to reproduce the behavior:

Install Plugin
Tada

Expected behavior
It should be working
Screenshots
If applicable, add screenshots to help explain your problem.

OS : Windows 10
Version : 2.1.1

Duplicate of https://github.com/Yentis/betterdiscord-emotereplacer/issues/77?

well it probably was a discord or BD thing, because i completly forgot about it and it's working now.
sorry


stage1: ensure systemd-tmpfiles is available in pod

TestAppSandboxSmoke is currently failing on host and src flavors because:

stage1-host is missing a /bin directory
stage1-host doesn't install systemd-tmpfiles from host
stage1-src is built without systemd-tmpfiles

This should fix testsuite failures on those flavors.

This also addresses https://github.com/coreos/rkt/pull/3424#discussion_r90588327.
/cc @evverx

rekicked CI due to flakiness

@lucab, thanks!
The TestFetchFromFile hangs:
25074 ?        Ss     0:08 tmux
25075 pts/1    Ss     0:00  \_ -bash
21617 pts/1    S+     0:02  |   \_ make functional-check
22942 pts/1    S+     0:00  |   |   \_ /usr/bin/bash -c set -e; \ printf '  %-12s %s\n' 'GO TEST' "github.com/coreos/rkt/tests"; \ sudo RKT_STAGE1_DEFAULT_FLAVOR="src" \ ?RKT="/home/vagrant/rkt-gopath/src
22943 pts/1    S+     0:00  |   |       \_ sudo RKT_STAGE1_DEFAULT_FLAVOR=src RKT=/home/vagrant/rkt-gopath/src/github.com/coreos/rkt/build-rkt-1.20.0+git/tmp/functional/rkt ACTOOL=/home/vagrant/rkt-gopath
22944 pts/1    Sl+    0:00  |   |           \_ /usr/bin/go test -tags src -timeout 60m -v github.com/coreos/rkt/tests
23338 pts/1    Sl+    0:07  |   |               \_ /tmp/go-build197934847/github.com/coreos/rkt/tests/_test/tests.test -test.timeout=60m -test.v=true
20211 pts/15   Ss+    0:04  |   |                   \_ stage1/rootfs/lib64/ld-linux-x86-64.so.2 stage1/rootfs/usr/bin/systemd-nspawn --boot --notify-ready=yes -Zsystem_u:system_r:svirt_lxc_net_t:s0:c8,c35

$ sudo strace -p 20211 -yy -s 500 -v
strace: Process 20211 attached
writev(2</dev/pts/15>, [{iov_base="Container rkt-7c124d22-c6ce-415e-a61b-3a83e648a94a exited successfully.", iov_len=71}, {iov_base="\n", iov_len=1}], 2^Cstrace: Process 20211 detached
 <detached ...>

But seems like that's unrelated to this PR

@evverx yes, it is an unrelated problem (exacerbated by some recent increase in debug output). See https://github.com/coreos/rkt/pull/3443 for a fix and explanation. Which environment are you using for running the testsuite? Can you check if you spot any other hanging tests?

Which environment are you using for running the testsuite?

My last attempt was successful (all tests passed).

See #3443 for a fix and explanation

I'll take a look. Thanks!


Dev 3.1.2 - Added catspeak_get_self and catspeak_get_index

This PR introduces catspeak_get_self and catspeak_get_index, which play as counterparts to method_get_self and method_get_index. The reason for this is that method_get_index will return __catspeak_function__ or __catspeak_method__ from Catspeak programs. And even then, not as a method but as a GML function index. That's likely to cause all sorts of issues.
method_get_self also has the unfortunate issue where it could return the Catspeak program struct itself. Which may also be unintended depending on who you ask.
catspeak_get_index resolves this by always returning a method. Never a function ID. This means that any Catspeak functions or Catspeak methods, will always return their callee program "as is". And any GML function will be returned as a method with a bound to undefined.
catspeak_get_self resolves this by returning self from only GML functions and Catspeak methods. Anything marked as a Catspeak function (via is_catspeak) will retrieve self_ via accessor (if it exists), or returns undefined.

Can you add a couple of tests to validate the behaviour works, please?

Included a couple unit tests as per request!


Add a filter functionality to the article cards

The article cards needs to be rendered on the page according the time it was added to the project inside the article.js
You'd write a function that filters the arrays of object based on the date that it was added.
{
    date: moment("Oct 19, 2021").fromNow(),
}

Can you assign me?

Hi @ramonvheerde,
Thank you for fixing the issue. Kindly link your PR to this issue.
So I can merge it.


Error in c

Good day,
I have company communicator on azure and it only runs fine when I use the v3.x branch for deployment.
I attempted to change it from v3.x to v4.x because when I use master, it gives an error saying "The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.". But when I go into the app service deployment centre to change the branch it points to, the changes are not reflected.
after changing the branch and clicking on sync, is there something else I'm supposed to do?
I tried redeploying but the redeployment failed with this error
"code": "BadRequest",
"message": "There was a conflict. Conflict with existing ScmType: ExternalGit",
"details": [
{
"message": "There was a conflict. Conflict with existing ScmType: ExternalGit"
},
{

Hello @Ochuwa-sophie,
We are not able repro this issue at our end. I want to confirm that have you followed v3 Migration Guide?
Please let us know more details.
Thanks


Script that automatically pull contributors' names from Github and add to About page

We have contributor names in about page but we need to keep update this party manually.
A script to automatically pull contributors' names from Github would be very good.

Interested.
I suggest a build-time pulling from GitHub REST API v3. Since the GraphQL v4 needs authentication token to query anything.

I drafted a pull request, which is based on what I have mentioned above. Feel free to comment on the concept and its implementation. 🙂


High memory usage on upload

Storage Explorer Version: 1.16.0
Build Number: 20201109.5
Platform/OS: Windows 10
Architecture: x64
Bug Description
High memory usage when uploading a file to blob storage.

Steps to Reproduce
I was uploading 6GB file. Storage Explorer memory usage was also about 6GB. After upload completed memory usage decreased to 200MB

@suhrab when doing transfers the things being transferred will be read into memory. If you expand the process group for Storage Explorer, you should see that most of the memory usage is coming from AzCopy (the thing doing the transfers).
Ok?

@suhrab when doing transfers the things being transferred will be read into memory. If you expand the process group for Storage Explorer, you should see that most of the memory usage is coming from AzCopy (the thing doing the transfers).
Ok?

If the memory usage is causing problems, please consider tweaking the max buffer size that AzCopy can use. You can configure it at "Settings -> Transfers -> Buffer Size".

If the memory usage is causing problems, please consider tweaking the max buffer size that AzCopy can use. You can configure it at "Settings -> Transfers -> Buffer Size".

@JasonYeMSFT Thanks!!

@JasonYeMSFT Thanks!!


Page Scroll not working in Chrome 71.0.3

I've checked the #29 and #58 Already. But nothing did solved this issue.
Reproduce:

Open the demo page
go to full screen by clicking on the toggle button (or any other button that takes you there)
Try to scroll the page with your finger - not working

I am using windows 10 and Chrome version 71.0.3
Also checked on Firefox 64. Still nothing

This package package is just a compatibility wrapper for the native fullscreen API. If scrolling is not working, it's either the native fullscreen API that is at fault or you using it incorrectly.

Thank you for your response. I'm not even using it. I just checked the demo and I opened this as this is happening in the demo.


add a few more effects like responsive profile card , form etc.

Issue Type:

[ ] Bug Report
[x] Feature Request
[ ] Documentaion

Describe the bug

A clear and concise description of what the bug is.

Possible solution

Describe the solution you thought of.

Screenshots

If applicable, add screenshots to help explain your problem.

Hi sir , I am a Gssoc contributor and please assign this task to me.

@srinjoy-26 add ss.....

Sir I would like to work on this issue. Kindly assign this issue to me.


Add martinezedgar

What changes does this PR do?
Add martinezedgar to the Hactoberfest Community list.
Screenshots (If any)
Questions

Thank you very much!


TA Review (MIS/MG):

@samm82
Summary

[x] It could be presented more clearly (e.g., in a table); you're essentially repeating the information of which modules are being completed within each "subcategory" of dates.
[x] AC3 should be more specific; which libraries would be updated? The Game Room Module is an Abstract Data Type (since there will be more than one game room, and it exists within the context of your program). I think the descriptions of the puzzle modules' secrets could be made more explicit (e.g., what does a "Math Puzzle Module" entail?). Libraries are modules without state information (e.g., just a list of functions).
[x] Template content (like Dr. Smith's comments) should be removed. I think this also applies to "Software Eng means the module will be implemented by the Software Eng software." since this doesn't really make sense based on the rest of your document. Non-leaf modules (e.g., the Behaviour-Hiding Module), should have a dash indicating that they aren't leaf modules and therefore won't be implemented (as described in the second paragraph of the Module Decomposition). Minor grammar issues.
[x] How does an array differ from the already-specified sequence? Some function names are capitalized while others don't; this should be made consistent (and should ideally match the convention of whatever language this will be implemented in). The names of OnUserLoggedIn and related functions are unintuitive; are they names you picked or from Vivox/Unity? Abstract data types should have constructors defined (and have the type of the created object listed as an "Exported Type").
[x] Minor grammar issues/typos; "RefreshPlayerList" is called "RefreshList" when defining its semantics. In math mode, LaTeX treats letters as variables so their spacing gets messed up; make sure to format words so that they display correctly (try \text{}).
[x] It looks like these puzzles are inspired by Keep Talking and Nobody Explodes; if this is the case, you should probably explicitly say so (both to avoid plagiarism accusations and to help potential future developers understand them).
[x] No semantics for Initialize of Game Room Module. Functions should be in the same order when defining their syntax and semantics. There is a lot of information not defined; for example: Game Room Module: isJoining, GameScene, behaviour of unsuccessful JoinAsync, Lobby Voice Communication Module: Permission
[x] The "Local Variables" of the Text Communication and Simon Says Modules should be state variables, as should _vvm in the Voice Communication Module and success and code in the Code Puzzle Module. buttonsArray and colourDictionary aren't used. You shouldn't need to compare something to "1", "0", "True", or "False" to see if it's True or False. It doesn't really make sense for isMicPermissionGranted to have a transition if it is just outputted and doesn't actually change a state variable. What do the puzzle state Booleans represent? It doesn't look like the game modules define a logic for defining if a puzzle is completed or not. SymbolsArray should be defined more generally as a sequence of symbols (Strings?); you could specify the length of this array if that is relevant. An example for the Math Puzzle Module would make it more clear. The definition of CheckEntry seems pretty repetitive. You may be able to make some puzzle functions (e.g., ColourBlink) more generic (i.e., move them up to the Multiplayer Puzzle Level).
[x] Are skip limits associated with players or groups? Can one player skip a puzzle for everyone?
[x] Is an Error Manager Module really necessary if, as far as I can tell, there is only one exception thrown by your modules? (This also may be incorrect; there is usually many reasons for an exception being thrown.)
[x] I think that many of these modules have enough information to get the general sense of how to implement them, but there are many ambiguities/omissions. Some designs of what puzzle modules could look like would make them more clear.

Deadline
March 20th, 2024
Estimated Work Hours
6 hours

Updated MG to fix several TA issues in MG. Only thing remaining is fixing secret's for puzzles.

MIS:

Updated constants, variables, and functions to have consistent syntax
Added note where there are unintuitive names that we are not able to change
Added constructors to modules with multiple instances
Fixed all typos and syntax errors that we came across
Puzzles are significantly far enough away from any existing games that no credit is warranted. Will try to make this clear in the commit message
Removed Initialize from Game Room Module (couldn't find equivalent in code)
Reordered functions to have same order in Exported Access Programs and Access Routine Semantics
Defined previously undefined terms/ variables
Text communication local variables, Simon Says local variables, and _vvm moved to state variables
Removed state transition in IsMicPermissionGranted
Attempted to make puzzle completion aspect more clear with the active variable and puzzle/ batch completed functions, actual isComplete boolean shouldn't be necessary
SymbolsArray removed
Updated CheckEntry semantics to be more clear
Left functions such as ColourBlink as is since having them be more generic wouldn't benefit any of the puzzles, they are specific to the puzzle they were created for
Removed skip limits
Added note that even though the number of errors handled by the error module is few, it is still critical for a real time system such as this
Most ambiguity has been cleared up now that we have implementation to compare against

Partially implemented in #310

We good on MG


dts: st: f2/l4: Add missing maximum-speed property

Add missing maximum-speed property to usb nodes
for F2 and L4 series.
Fixes: #10472
Signed-off-by: Yannis Damigos<EMAIL_ADDRESS>

Codecov Report

Merging #10486 into master will increase coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10486      +/-   ##
==========================================
+ Coverage   53.18%   53.33%   +0.15%
==========================================
  Files         210      210
  Lines       25816    25816
  Branches     5682     5682
==========================================
+ Hits        13729    13770      +41
+ Misses       9780     9733      -47
- Partials     2307     2313       +6

Impacted Files
Coverage Δ

lib/posix/pthread_sched.c
41.66% <0%> (-20.84%)
:arrow_down:

lib/posix/pthread_mutex.c
77.77% <0%> (+1.38%)
:arrow_up:

lib/posix/pthread.c
80.36% <0%> (+12.32%)
:arrow_up:

lib/posix/pthread_cond.c
83.33% <0%> (+20.83%)
:arrow_up:

include/posix/posix_sched.h
100% <0%> (+100%)
:arrow_up:

lib/posix/pthread_barrier.c
100% <0%> (+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 ecc050e...0b38917. Read the comment docs.


KeycloackRestTemplate with service discovery capabilities

Is it possible to add service discovery capabalities to KeycloackRestTemplate class? I tried to annotate the KeycloackRestTemplate bean definition with @LoadBalanced annotation, but it seems that the KeycloackRestTemplate doesn’t resolve service name using Eureka Discovery Server.

Yes, same issue here too. java.net.UnknownHostException: No such host is known (gateway-server). Using Keycloak versin 15.0.2

Removing @Scope(ConfigurableBeanFactory.SCOPE_PROTOTYPE) made it work. Thanks.


TypeScript 2.1 regressed #4952 (UMD modules requirejs optimizer compatibility)

TypeScript Version:  2.1.1
Format of the generated UMD modules changed with TS 2.1 and isn't compatible with requirejs optimizer anymore (dependencies moved from define() to self invocation argument), please refer to the linked ticket #4952 for more detailed description.

This looks serious


RPM package

i was able to compile a RPM package, to see if we can add it to the current github actions. If is posible to contribute

This sounds nice! I am happy to accept patches for creating RPM packages.
I personally don't use the GitHub Actions so adding a plain script that spits out the package, or perhaps adding a Makefile that does it, would be of greater value.
These can then be called from whatever CI service we use (currently GitLab Pipelines and Sourcehut Builds).
Then again, maybe we should put an effort into bringing gcli to Fedora (or whatever the common place for RPM packages is)?
I am open for discussions here.

I personally would do the Makefile option but.  That's just me.

I have 0 skills written MAKE files. so i create a bash script to build it, but currently i am running it on a docker container.
You tell me how to give you the code. i am new with this kind of contributes.
😅

I have 0 skills written MAKE files. so i create a bash script to build it, but currently i am running it on a docker container.

A plain POSIX shell script is fine - we can maybe massage it a bit such that it can be turned into a Makefile.

You tell me how to give you the code. i am new with this kind of contributes.

You can either just send it inline in an E-Mail to our mailing list<EMAIL_ADDRESS>(probably the easiest)
... or clone the gcli repo, add the script into the tools/ directory, commit and open a pull request.

You can either just send it inline in an E-Mail to our mailing list<EMAIL_ADDRESS>(probably the easiest)

This reminds me @herrhotzenplotz, I made a bash script the user can run to see if they have the necessary packages/libraries installed to build/compile.. made because I don't always trust that the user knows everything including what's installed on their machine, including myself (there are so many machines).
Should I commit/PR that?
Thanks,
@gbowne1


Improve DX

add pyproject.toml
add ruff
add all
src: https://www.reddit.com/r/Python/comments/1f7jfgd/comment/lla43z0/?utm_source=share&utm_medium=web3x&utm_name=web3xcss&utm_term=1&utm_content=share_button

Can assign it to myself (@epg323 ) I am almost done, just need to test the new configuration. Should be ready tomorrow.


Great Code Awesome!!

The code is great but the sharing not really!


[MM-32691] - Send email when the users are not able to join a workspace

Summary
After a user is shown the error page that the cloud workspace is out of capacity. Emails are sent to atleast one admin requesting an upgrade of the workspace
Ticket Link
MM-32691
Related Pull Requests

Has redux changes

/update-branch

/cherry-pick cloud

Cherry pick is scheduled.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
From github.com:mattermost/mattermost-webapp
   4653122d1..1aef731af  release-5.33 -> upstream/release-5.33
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-mattermost-webapp-#7553-upstream-cloud-1614614005
Switched to a new branch 'automated-cherry-pick-of-mattermost-webapp-#7553-upstream-cloud-1614614005'
Branch 'automated-cherry-pick-of-mattermost-webapp-#7553-upstream-cloud-1614614005' set up to track remote branch 'cloud' from 'upstream'.

+++ About to attempt cherry pick of PR #7553 with merge commit e2b33e44dddf9a94279a61d8a3a604ff918f8f2b.

error: could not apply e2b33e44d... [MM-32691] - Send email when the users are not able to join a workspace (#7553)
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

+++ Conflicts detected:

UU package-lock.json
UU package.json
Aborting.

+++ Aborting in-progress git cherry-pick.

+++ Returning you to the master branch and cleaning up.


make build error, package not find and internal package package not allowed

After successfully run "make prepare". then I run "make build".
it still complains about missing package like BurntSushi/toml and such. from both $GOROOT and $GOPATH
including go-zoo/bone, hashicorp/consul/api, go.uber.org/zap, gopkg.in/mgo.v2, gopkg.in/mgo.v2/bson
I try to fix it by running "go get ...", then come up a new exceptions
github.com/edgexfoundry/edgex-go/internal not allowed
github.com/edgexfoundry/edgex-go/internal/pkg/config not allowed
github.com/edgexfoundry/edgex-go/internal/pkg/usage not allowed
what does this means? no permission? or missing files?

I think I run "go get github.com/edgexfoundry/edgex-go", but it does not show any message. No success or fail. Then I run git clone https://github.com/edgexfoundry/edgex-go
I install Glide using apt-get install glide. But then I updated it by "go get -u github.com/Masterminds/glide" to fix the package "." bug.

clean up and retry.
apt get install for glide and git.
go get edgex-go
curl | sh for zeroMQ
This time, no error.

https://medium.com/mainflux-iot-platform/edgex-in-anger-part-1-installation-5468c52be4d1


Added a test suite for the Plane object.

Some parts are not implemented as I don't know how to test for the Variant bindings. This is also my first go at contributing as well, and i am not quite sure if the values i chose as test inputs and outputs are useful for the context of the class. It definitely needs tweaking and finishing up by someone with knowledge in vector math to give more helpful values.

Please remove the empty lines shown here: https://github.com/godotengine/godot/pull/44492/checks?check_run_id=1576158367
When you're done, squash your commits following these instructions: https://docs.godotengine.org/en/stable/community/contributing/pr_workflow.html#the-interactive-rebase
And thank you for your first contribution! :tada:

Please use tabs instead of spaces for indentation: https://docs.godotengine.org/en/stable/community/contributing/code_style_guidelines.html#c-and-objective-c

Sorry for the delay further reviewing this. This seems like a good base for a Plane test suite, so let's get it merged. I rebased and amended the commit to address past review comments, and improve language/punctuation in some messages.


Update dependencies in requirements.txt

PR Type
Dependencies

Description

Updated lancedb from version 0.13.0 to 0.14.0.
Added tokenizers version 0.20.0 to the dependencies.
Updated numpy from version 1.25.2 to 1.26.4.

Changes walkthrough
Relevant filesDependencies

requirements.txtUpdate and add dependencies in requirements.txt

requirements.txt
Updated lancedb from version 0.13.0 to 0.14.0.  Added tokenizers version 0.20.0.  Updated numpy from version 1.25.2 to 1.26.4.

+4/-2

💐 🏠 The PR description has been modified to accurately represent the latest commit (https://github.com/SofarDatasAI/graphrag/commit/6c993fb7ab1e90d336c34d37cead1340064e30e8).


[Epic]: 🚀 Initial Aztec Sandbox Release

EPIC ISSUE. DO NOT CLOSE.
This issue only exists to improve visualisation of the Roadmap view of the project board.
(Unfortunately, the start and end date of github's 'Milestones' can't be shown on a roadmap view).
If the 'Target end date' looks unrealistic, let's chat! 😃

Complete! 🥳


Initial implementation of TextBlock.Inlines

This PR adds TextBlock.Inlines and a couple of TextElements:

Run
LineBreak

This allows one to format the text within a TextBlock, e.g.:
<TextBlock>
          A <Run Classes="code">TextBlock</Run> with <Run FontWeight="Bold">differing</Run>
          <Run FontStyle="Italic">styles</Run> <Run FontSize="24">and fonts</Run>.
        </TextBlock>

Currently only works fully on Direct2D. The Skia backend currently only supports Foreground in FormattedText - it will need to be updated with the new properties introduced in this PR. If anyone would like to implement these in Skia, please do!
There are also more TextElements to add, such as Span and InlineUIContainer. These will come in a separate PR I think.

As I understand, you will not allow use TextElement into TextBlock?

Sorry @Karnah - I don't understand?

I didn't know that in TextBlock I can use <Run /> and always format by another TextBlocks. Now I've tried and found out that you also can use TextBoxes in TextBlock in WPF!
<TextBlock>
    I see trees of <TextBlock Foreground="ForestGreen" Text="green" />, red roses too
    <LineBreak />
    I see them <TextBox Text="{Binding Bloom}" /> for me and you
    <LineBreak />
    And I think to myself what a <Run FontWeight="Bold" Text="wonderful world" />
</TextBlock>

It's hard for me to decide, is it necessary to support TextBlock and TextBox or not. But as I can understand, if you decide supporty this types, you should inherit them from Inline. Or am I wrong? Or this you'll implement in future? And I decided to clarify this :)

@Karnah ok, I think I understand: yes what you're asking for is InlineUIContainer which is an Inline which wraps controls. I mentioned this in the description above, and yes - we should probably have this if possible, though I'm not sure how difficult it would be!

Closing as this is superseded by #2423


Shrink size of hotpot binary files

Signed-off-by: leonwanghui<EMAIL_ADDRESS>
What this PR does / why we need it:
Currently it's too large for the size of hotpot built binary files, see below:
root@***:~/workspace/gopath/src/github.com/opensds/opensds# ls -lht build/out/bin/
total 78M
-rwxr-xr-x 1 root root 15M May  9 16:18 osdsctl
-rwxr-xr-x 1 root root 23M May  9 16:18 osdsapiserver
-rwxr-xr-x 1 root root 20M May  9 16:18 osdslet
-rwxr-xr-x 1 root root 22M May  9 16:18 osdsdock

But with -ldflags '-w -s' flag, we can shrink them a lot (see below):
root@***:~/workspace/gopath/src/github.com/opensds/opensds# ls -lht build/out/bin/
total 57M
-rwxr-xr-x 1 root root 11M May 10 03:59 osdsctl
-rwxr-xr-x 1 root root 17M May 10 03:59 osdsapiserver
-rwxr-xr-x 1 root root 14M May 10 03:59 osdslet
-rwxr-xr-x 1 root root 16M May 10 03:59 osdsdock

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Coverage remained the same at 30.262% when pulling c9a0d3dc39c6021cf9e868ca5c67671e02c26371 on leonwanghui:bshrink into e8037398d7af5c9fed568d5bf7e10c66336e5fad on opensds:development.


OutOfMemory

I was running the script and I hit memory limit very very soon.  Do you know if there is a way to configure the memory limits?
java.lang.OutOfMemoryError: GC overhead limit exceeded
Exception in thread "Thread-34" java.lang.OutOfMemoryError: GC overhead limit exceeded
Exception in thread "Thread-20" java.lang.OutOfMemoryError: GC overhead limit exceeded
Exception in thread "Thread-10" java.lang.OutOfMemoryError: GC overhead limit exceeded
Exception in thread "Thread-36" java.lang.OutOfMemoryError: GC overhead limit exceeded
Exception in thread "Thread-39" java.lang.OutOfMemoryError: GC overhead limit exceeded

Hello,
one option to decrease memory usage is to remove concurrent threads from script by simple removing thread from script.
I don't have deep experience with *kts scripts but you definitely can open it in intellij idea and tweak memory from project settings.
There is also similar problem solved already https://stackoverflow.com/a/56835181


[GHSA-f2gr-7299-487h] DOS and excessive memory usage when passing untrusted user input to to dag import

Updates

Affected products

Hi there @BigLep and @aschmahmann! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory


Icon missing if user is in different Tenant with Teams-Client

If you switch the tenant with your teams desktop client, the add-in for teams is missing. Steps to reproduce:

Close Outlook
Switch tenant in teams to a tenant that is not your "home" / where you are a guest-user
Start Outlook again -> Teams-Meeting-Button is missing

If you switch back to your home tenant and restart Outlook -> Teams Meeting Button is available.
My environment: Win 10 1809, Office 365 Pro Plus Monthly Channel.
If this is a common problem it should be in the troubleshooting section of this page.

Dokumentdetails
⚠ Bearbeiten Sie diesen Abschnitt nicht. Er ist für die Verknüpfung von docs.microsoft.com zum GitHub-Artikel erforderlich.

ID: 37e10f52-d726-d29a-e0a8-93729a32dfae
Version Independent ID: 83555f80-a8ca-ee85-c1fc-e39891029758
Content: Verwenden des Microsoft Teams-Besprechungs-Add-Ins in Outlook
Content Source: Teams/Teams-add-in-for-Outlook.md
Service: msteams
GitHub Login: @chuckedmonson
Microsoft Alias: chucked

@ihloff Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@ihloff this one looks like a product issue. Did you file the issue to Microsoft?

@ihloff I assume you've filed the issue so this issues will be closed shortly

Hi, I thought it would be a good addition to the section "Troubleshooting". I'm not sure if it is a bug or a missing feature of the product itself - I didn't file this to Microsoft but I'll do so next week.

@ihloff We appreciate any feedback that improves the content in the Microsoft docs  and will consider adding this in the future.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

I just got the information from Microsoft Support that it is not a bug. In this case I think it should be added in the documentation.

@ihloff do you have any official confirmation?

@dplotnikov
I opened a ticket at the support ( Ticket #:15956324) and got this information. Don't know if this official enough but I don't have any other ways to get an answer from Microsoft.


Link Bionomics and Occurrence entities in db

When a Bionomics or Occurrence data sheet is uploaded, the resulting entities should be linked to any existing relevant Bionomics/Occurrence data as appropriate. A Bionomics record is linked to an Occurrence record if the have the same:

Site
Reference
Species
Time frame (month start, month end, year start, year end)

Acceptance Criteria:

[x] When a bionomics csv file is uploaded, any existing occurrence entities that link to that bionomics entity through the links specified above have their bionomicsId set to that of the new entity
[x] When an occurrence csv file is uploaded, it is linked to any existing bionomics entity through the links specified above - the bionomicsId is set to that of the existing bionomics entity

Logic for this is complete, waiting on data team to answer a question about occurrence citation creation

Tested by following instructions here: docs\demos\phase1\sprint2\bionomics-occurrence-link.md


Add TrieMap collection type

We should adapt the code of the current collection’s TrieMap to the strawman design.

Note that the existing (2.12) TrieMap fails to respect custom hashing on hash collisions.  We should make sure the adaptation here fares better.  See https://github.com/scala/bug/issues/10481

I rebased to master and enabled the existing junit tests for TrieMap

Closed by #204


Coral fingerprint Barchart shows more than 5 categories

The vega spec should rank and filter only the 5 highest ranked categories. The same spec works in the online vega editor, but in path explorer there are often visualizations with more plots
possible cause could be bars with "equal" rank

with commit 8a5164e the top-k implementation in the vega spec was changed from the window op "rank" to "row_number"
see
https://vega.github.io/vega-lite/docs/window.html
this results in a maximum of 5 categories / bars being shown without additional duplicates


tabs malformed under 'new' war deploy

see issue2-gulp branch.  Select a data object and go to info page.  See screen shot attached

Fixed with latest commit.


[SPARK-27425][SQL] Add count_if function

What changes were proposed in this pull request?
Add count_if function which returns the number of records satisfying a given condition.
There is no aggregation function like this in Spark, so we need to write like

COUNT(CASE WHEN some_condition THEN 1 END) or
SUM(CASE WHEN some_condition THEN 1 END),
which looks painful.

This kind of function is already supported in Presto, BigQuery and even Excel.

Presto: count_if
BigQuery: countif
Excel: COUNTIF (It is a little different from above twos)

How was this patch tested?
This patch is tested by unit test.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Thanks for the PR @cryeo (I have executed the tests and they are passing, run scalastyle and there was no violation).
Ok to test.

ok to test

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9813/
Test PASSed.

Test build #104483 has started for PR 24335 at commit 9330d02.

As with lots of these -- we wouldn't add a new function unless it were standard SQL. With Spark SQL, it's pretty trivial to express count-if with a filter and count.

we wouldn't add a new function unless it were standard SQL.

Would you mind if I ask you the reason?
Presto and BigQuery provide this nevertheless it isn't ISO/ANSI standards.

With Spark SQL, it's pretty trivial to express count-if with a filter and count.

As you said, we can archive this with existing functions like followings, which are a little bit inconvenient.

COUNT(IF(very_complex_condition, 1, NULL))
COUNT(CASE WHEN very_complex_condition THEN 1 END)
SUM(IF(very_complex_condition, 1, NULL))
SUM(CASE WHEN very_complex_condition THEN 1 END)

However, I think that these are a little bit inconvenient and painful.

we wouldn't add a new function unless it were standard SQL.

Would you mind if I ask you the reason?
Presto and BigQuery provide this nevertheless it isn't ISO/ANSI standards.

With Spark SQL, it's pretty trivial to express count-if with a filter and count.

As you said, we can archive this with existing functions like followings, which are a little bit inconvenient.

COUNT(IF(very_complex_condition, 1, NULL))
COUNT(CASE WHEN very_complex_condition THEN 1 END)
SUM(IF(very_complex_condition, 1, NULL))
SUM(CASE WHEN very_complex_condition THEN 1 END)

However, I think that these are a little bit inconvenient and painful.

This is my opinion but I think it would be shared by others.
Presto and BigQuery are SQL-only and UDFs are relatively hard. Hence it makes sense to bake in a lot of SQL helper functions. In Spark it's easy to mix code in, so the value of SQL-only helpers isn't nearly as big.
df.filter("...complex condition").count() does this easily for example. It's not a great example, because I'm not even sure the current SQL equivalent is much more complex.
The SQL helper functions you see today are mostly to match Hive. If Hive supports something that's a more compelling argument to add it for interop.

Test build #104483 has finished for PR 24335 at commit 9330d02.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104483/
Test PASSed.

To me, I am not sure how useful it is too. But at least it's matched with some other DBs. Maybe it's better to be asked to mailing list and see if people like and need it. If there are not so much input about this, I wouldn't go for it for now.

OK. I'll ask to mailing list :)

To chime in here -- I feel this one is probably OK, given its ubiquity (also in excel?)
Question is ... should it be count_if or countif?

Another question is tho, are there like sum_if, avg_if too?

I personally agree with that @srowen said . I don't believe we need to clutter the API when we have a simple solution like that.

Hi, @cryeo . Did you ask the questions to the community as @HyukjinKwon recommended? I'm just wondering if the decision was made. If we are not going to proceed with this, we had better close this PR and JIRA issue.

Another question is tho, are there like sum_if, avg_if too?

Sorry for the late reply.
I think that the use cases of count_if are quite different from that of sum_if or avg_if.
That's why Presto and BigQuery provide only count_if.

Then I guess it's fine to add count_if alone. Is the name count_if prevailing or countif per https://github.com/apache/spark/pull/24335#issuecomment-483493191?

I have just found four products which provide this function: Facebook Presto, Google BigQuery, IBM Informix, Microsoft Excel. Only Presto supports as count_if, the others support ascountif. so, It seems that countif is more prevailing.

okie. can you rebase?

OK. I just did it :)

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10848/
Test PASSed.

Test build #105595 has started for PR 24335 at commit e4f0465.

Test build #105595 has finished for PR 24335 at commit e4f0465.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105595/
Test FAILed.

retest this please

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10880/
Test PASSed.

Test build #105627 has started for PR 24335 at commit e4f0465.

Test build #105627 has finished for PR 24335 at commit e4f0465.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105627/
Test PASSed.

cc @cloud-fan too

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11044/
Test PASSed.

Test build #105789 has started for PR 24335 at commit e4f0465.

Test build #105789 has finished for PR 24335 at commit e4f0465.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105789/
Test PASSed.

Merged build finished. Test PASSed.

@HyukjinKwon I just modified based on your review. Could you review it again?

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11347/
Test PASSed.

Test build #106095 has started for PR 24335 at commit 8764039.

Test build #106095 has finished for PR 24335 at commit 8764039.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106095/
Test FAILed.

retest this please

Test build #106105 has started for PR 24335 at commit 8764039.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11356/
Test PASSed.

retest this please

Test build #106108 has started for PR 24335 at commit 7448117.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11359/
Test PASSed.

Test build #106105 has finished for PR 24335 at commit 8764039.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106105/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106105/
Test PASSed.

WDYT guys? I think current approach makes sense and consistent with some expressions we have.

Test build #106108 has finished for PR 24335 at commit 7448117.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106108/
Test PASSed.

ok to test

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11486/
Test PASSed.

Test build #106238 has started for PR 24335 at commit 38eb38b.

Test build #106238 has finished for PR 24335 at commit 38eb38b.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class CountIf(predicate: Expression) extends UnevaluableAggregate with ImplicitCastInputTypes

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106238/
Test PASSed.

@cryeo . Please update the PR description with more SQL references. You already told us  Presto/BigQuery/Excel references. That will make this PR stronger.

cc @gatorsmile

Test build #106269 has started for PR 24335 at commit 81ab7e6.

@dongjoon-hyun Thanks for your review. I just modified code and PR description. Could you confirm it?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11515/
Test PASSed.

Test build #106269 has finished for PR 24335 at commit 81ab7e6.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106269/
Test FAILed.

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11521/
Test PASSed.

Test build #106275 has started for PR 24335 at commit 81ab7e6.

Thank you for updating, @cryeo . The PR description looks enough.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11526/
Test PASSed.

Test build #106281 has started for PR 24335 at commit 2f4d64e.

Test build #106275 has finished for PR 24335 at commit 81ab7e6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106275/
Test PASSed.

Test build #106281 has finished for PR 24335 at commit 2f4d64e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106281/
Test FAILed.

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11529/
Test PASSed.

Test build #106284 has started for PR 24335 at commit 2f4d64e.

Test build #106284 has finished for PR 24335 at commit 2f4d64e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106284/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11584/
Test PASSed.

Test build #106338 has started for PR 24335 at commit c0a3289.

Test build #106338 has finished for PR 24335 at commit c0a3289.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106338/
Test FAILed.

Retest this please.

Test build #106344 has started for PR 24335 at commit c0a3289.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11590/
Test PASSed.

Test build #106344 has finished for PR 24335 at commit c0a3289.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106344/
Test PASSed.

Merged to master.

Thank you all and @cryeo. Welcome to contributors :).


[fix bug 1308282] Add 'kab' locale.

Coverage remained the same at 92.88% when pulling 206bf38d2fa6b1867046331579603b72060a89ed on akatsoulas:1308282-kab-locale into 0d557ad02be52fd76fdcd682181ae8b4813abe41 on mozilla:master.


Save best model in machine translation

I find the best model saved by the machine translation example can't be load. I think we need to change sampling.py\BleuValidator()\_save_model()
            numpy.savez(
                model.path, **self.main_loop.model.get_parameter_dict())

to
            numpy.savez(
                model.path, **self.main_loop.model.get_parameter_values())

According to the blocks document on model: get_parameter_dict() returns a dictionary as dictionary{hierarchical name: shared variable} while get_parameter_values() returns a dictionary as dictionary{hierarchical name: numpy.ndarray}, where later one should be better option.

In fact, the custom serialization method used in this demo should be
dropped. Since about two months Blocks has a new implementation of
serialization, that should meet all the requirements for MT.
On 25 April 2016 at 17:27, Qiang Zhang<EMAIL_ADDRESS>wrote:

I find the best model saved by the machine translation example can't be
load. I think we need to change
        numpy.savez(
            model.path, **self.main_loop.model.get_parameter_dict())

to
        numpy.savez(
            model.path, **self.main_loop.model.get_parameter_values())

According to the blocks document on model: get_parameter_dict() returns a
dictionary as dictionary{hierarchical name: shared variable} while
get_parameter_values() returns a dictionary as dictionary{hierarchical
name: numpy.ndarray}, where later one should be better option.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/mila-udem/blocks-examples/issues/97

A temporary fix for this could be:
`def load_model_from_bleu_file(self, model, bleu_file_path):
assert os.path.exists(bleu_file_path)
    with closing(numpy.load(bleu_file_path)) as source:
        param_values = {}
        print source.items()
        for name, parameter in source.items():
            if name != 'pkl':
                name_ = name.replace(BRICK_DELIMITER, '/')
                if not name_.startswith('/'):
                    name_ = '/' + name_

                param_values[name_] = parameter.flatten()[0].get_value()

        model.set_parameter_values(param_values)`

I believe @orhanf also made a patch. Basically just like your code.


Netty does not call a packet encoder for specific packets

Currently known packet is PacketPlayOutEntityStatus (ClientboundEntityEventPacket).
https://gist.github.com/DrupalDoesNotExists/58e81b674f35652148f0feb5f23f4cf4
Maybe related to the fixes implemented in #8

Fixed in a merged PR. The problem was in an incorrect order of handlers in pipeline after injection.
Custom injector used the "prepender" as a predecessor which was wrong. This also caused an issue on clientside related to compression. The right value should be "compressor".


k8s: DNS resolution problems on windows nodes

I'm using a fairly recent commit (#520 d852abafd0291edf814e8a844af366be406eec1b) of acs-engine with kubernetes 1.6.2 including the winnat commit - however unfortunately I'm seeing some issues with DNS resolution that is pretty intermittent (to be fair I think they have been happening since I have been working on it, but there have been some other blocking issues)
Everything is working fairly well (I'm really enjoying the fast spin up time) and then all of a sudden calls will start to timeout in our backend. When I ssh into the box and run some powershell scripts I get something like the following:
PS C:\install> nslookup idnttest.table.core.windows.net
Server:  kube-dns.kube-system.svc.cluster.local
Address:  <IP_ADDRESS>

Non-authoritative answer:
Name:    table.by3prdstr02a.store.core.windows.net
Address:  <IP_ADDRESS>
Aliases:  idnttest.table.core.windows.net

PS C:\install> curl idnttest.table.core.windows.net
curl : The remote name could not be resolved: 'idnttest.table.core.windows.net'
At line:1 char:1
+ curl idnttest.table.core.windows.net
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-WebRequest], WebException
    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeWebRequestCommand

These commands were executed right after each other, so I don't understand why the name is not being resolved?
If I do a ipconfig /flushdns call the curl command (and my app) works again. After a little while (hour or two?) things will start to fail again.
I can confirm that not every external service is failing, I am able to curl other urls without an issue while this is happening. And I haven't seem the problem happen with internal cluster dns names, although this happens less frequently on our test app.
Maybe I don't really understand how the DNS lookup is working here. Anything I can do to alleviate this issue or debug it further?
Here's some more info that might be relevant:

I have a hybrid cloud with 2 windows nodes and 1 linux node
I am using an ingress controller (nginx) to handle traffic from a single load balanced service
All deployments also have a non-load balanced service attached to them (for the ingress to point to)

@ajorkowski What is the output of ipconfig /all? Does ping idnttest.table.core.windows.net fail too due to not able to resolve name? #539 might be related if you have service running on the Linux node.

@JiangtianLi When I mention 'service' I am talking about the kubernetes services, so we have some that are pointing to windows containers, and some that are pointing to linux containers, but I haven't seem any problems with the connectivity between the windows <-> linux nodes (yet). This seems to be a problem with accessing the outside internet from within the containers on windows nodes.
Later today I'll do some debugging and get the answers to your questions.

I'm having this exact same issue. After a (random) while the DNS lookup fails and never comes back fully until a  ipconfig /flushdns  is done inside the container. I noticed that when I manually enter the container and perform a nslookup  it succeeds most of the time but a ping command to the same host will result in 'no such name' /host unknown error.

Ok, here is another example (this time is it cdn.raygun.io and not the table services).
PS C:\install> nslookup cdn.raygun.io
Server:  kube-dns.kube-system.svc.cluster.local
Address:  <IP_ADDRESS>

Non-authoritative answer:
Name:    d1bs4b7zdgd8l3.cloudfront.net
Addresses:  2600:9000:202f:c000:17:62f0:2dc0:93a1
  2600:9000:202f:3a00:17:62f0:2dc0:93a1
  2600:9000:202f:8c00:17:62f0:2dc0:93a1
  2600:9000:202f:8800:17:62f0:2dc0:93a1
  2600:9000:202f:fa00:17:62f0:2dc0:93a1
  2600:9000:202f:1600:17:62f0:2dc0:93a1
  2600:9000:202f:2600:17:62f0:2dc0:93a1
  2600:9000:202f:7000:17:62f0:2dc0:93a1
  <IP_ADDRESS>
  <IP_ADDRESS>
  <IP_ADDRESS>
  <IP_ADDRESS>
  <IP_ADDRESS>
  <IP_ADDRESS>
  <IP_ADDRESS>
  <IP_ADDRESS>
Aliases:  cdn.raygun.io

PS C:\install> ping cdn.raygun.io
Ping request could not find host cdn.raygun.io. Please check the name and try again.
    : No
   DNS Suffix Search List. . . . . . : wxt5ky5hgd4uragneppffe5bod.dx.internal.cloudapp.net

Ethernet adapter vEthernet (Container NIC eabddebe):

   Connection-specific DNS Suffix  .   : No
   Autoconfiguration Enabled . . . .     : No
   Autoconfiguration Enabled . . . .  : Disabled

And just for completeness, I can query other urls:
PS C:\install> nslookup idnttest.table.core.windows.net
Server:  kube-dns.kube-system.svc.cluster.local
Address:  <IP_ADDRESS>

Non-authoritative answer:
Name:    table.by3prdstr02a.store.core.windows.net
Address:  <IP_ADDRESS>
Aliases:  idnttest.table.core.windows.net

PS C:\install> ping idnttest.table.core.windows.net

Pinging table.by3prdstr02a.store.core.windows.net [<IP_ADDRESS>] with 32 bytes of data:
Request timed out.

Ping statistics for <IP_ADDRESS>:
    Packets: Sent = 1, Received = 0, Lost = 1 (100% loss),
Control-C
PS C:\install> curl idnttest.table.core.windows.net
curl : <?xml version="1.0" encoding="utf-8" standalone="yes"?>
<error xmlns="http://schemas.microsoft.com/ado/2007/08/dataservices/metadata">
  InvalidUri
  <message xml:lang="en-US">The requested URI does not represent any resource on the server.
RequestId:7da2e81b-0002-0016-62f3-c2490c000000
Time:2017-05-02T03:26:06.7325700Z</message>
</error>
At line:1 char:1
+ curl idnttest.table.core.windows.net
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-WebRequest], WebException
    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeWebRequestCommand

This is interesting, if I do a ipconfig /displaydns I get the following:
PS C:\install> ipconfig /displaydns

Windows IP Configuration

    rt.services.visualstudio.com
    ----------------------------------------
    Name does not exist.

    wpad
    ----------------------------------------
    Record data for type ALL could not be displayed.

    cdn.raygun.io
    ----------------------------------------
    Name does not exist.

It looks like there may be intermittent failures and they are being cached?

@ajorkowski The failure is intermittent. I could repro once and as @skinny said, after ipconfig /flushdns, the same name could be resolved. I'll find a way to repro more consistently and investigate.

I have been using @skinny fix for the last day now (ie using the scheduler on the node to flush the dns in each container) and haven't had any problems so far. It is probably not that ideal though....

Yeah I needed to shorten the interval to 10 minutes to avoid any issues yesterday but it does the job until we get a proper fix
Im also going to experiment with the Windows DNS cache a bit to see if that helps

@skinny Awesome, I think for right now the 10 min ping is sufficient as we are really just running a test server at the moment, but its good to know that there is an alternative that is even more reliable. Thanks for sharing.

Just a bit of additional information for this thread - I created a v1.5.3 kubernetes cluster and I did not see this DNS issue. So it seems to be just affecting v1.6.2 from my own testing.

Just an update on this - I recently upgraded our cluster using 0.5.0 acs engine (1.7.2 kubernetes) and tried turning the DNS cache back and still getting this issue.

I found setting the pod dnspolicy to default stopped my timeout issues. This is fine for me as I only need to resolve public DNS records.
If you need cluster name resolution, try disabling the negative dns cache in your container. If you do get a DNS timeout, the bad record won't get stuck in your cache and usually resolves correctly next time.
https://support.microsoft.com/en-gb/help/318803/how-to-disable-client-side-dns-caching-in-windows-xp-and-windows-serve

@dm3r yes disabling DNS cache is what I have been using for a couple of months now (see my previous comments). However I am still experiencing lookup failures which causes the container to fail starting completely. After a couple of restarts (random up until 20 restarts) the lookup I need at application startup succeeds and the container starts fine.
So this issue is still present for me too unfortunately

I didn't explain properly but you can disable caching for failed DNS queries but still cache successful queries. Might be useful in certain situations.
Are you using internal cluster lookups or do you just need to get out to the internet?

I need both internal and external lookups. As the lookups are pretty quick I am just disabling all caching because that works at least for my use case.
Still the failing lookups are the real issue ;-)

I too am experiencing this with clusters I build today in ACS via the Portal or CLI. I can repro this easily. The commands skinny added to his dockerfile to disable dnscache have worked for me as a temporary fix. I'd love to see this get some attention from Microsoft, I am willing to assist with data if needed.

Is there any update on this issue?  We're hitting exactly this problem where DNS seems to just go after a period and then that status sticks until the pod is restarted.

I am still experiencing this issue and it's pretty frustrating that nothing has changed since April. I have k8s 1.7.9 clusters experiencing this. I have clusters with server 2016 ltsc and clusters with Server 2016 v1709 both experiencing this.
I have found that turning off the dnscache service works well but this is strongly not advised for production workloads. I have found this technique does not work with v1709 servers, I think there is a permission issue as I am unable to disable the service at all on v1709 nanoservers.

@brobichaud As you already know, acs-engine v0.9.2 and after is using Windows 1709 instead of Windows Server 2016. Windows 1709 has a different DNS issue but it has been fixed and should be out in the next Windows update.

/cc @madhanrm

@JiangtianLi is there any chance you could circle back around and update this thread once the v1709 update you mention is being used by new cluster creations? I'd be happy to try my scenario once that is available.

@brobichaud Sure, will do. Currently the DNS issue is on 1709 is random (due to race condition) and happens only in the first 15 minutes after container starts up. If you still have DNS issue after that, then it should be due to a different root cause and please let us know.

@JiangtianLi
acs-engine: 0.11.0
nodes: 1709
kubernetes: 1.7.9
Disabling cache in windows container not allowed:
PS C:\> Set-Service dnscache -StartupType disabled                                                                                Set-Service : Service 'DNS Client (dnscache)' cannot be configured due to the following error: Access is denied                                                                                At line:1 char:1                                                                                + Set-Service dnscache -StartupType disabled                                                                                + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                                                                + CategoryInfo          : PermissionDenied: (System.ServiceProcess.ServiceController:ServiceController) [Set-Service], ServiceCommandException                                                                      + FullyQualifiedErrorId : CouldNotSetService,Microsoft.PowerShell.Commands.SetServiceCommand

Deployed "simpleweb.yaml" service following these instructions:
apiVersion: v1
kind: Service
metadata:
  name: win-webserver
  labels:
    app: win-webserver
spec:
  ports:
    # the port that this service should serve on
  - port: 80
    targetPort: 80
  selector:
    app: win-webserver
  type: LoadBalancer
---
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  labels:
    app: win-webserver
  name: win-webserver
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: win-webserver
      name: win-webserver
    spec:
      containers:
      - name: windowswebserver
        image: microsoft/windowsservercore:1709
        command:
        - powershell.exe
        - -command
        - "<#code used from https://gist.github.com/wagnerandrade/5424431#> ; $$listener = New-Object System.Net.HttpListener ; $$listener.Prefixes.Add('http://*:80/') ; $$listener.Start() ; $$callerCounts = @{} ; Write-Host('Listening at http://*:80/') ; while ($$listener.IsListening) { ;$$context = $$listener.GetContext() ;$$requestUrl = $$context.Request.Url ;$$clientIP = $$context.Request.RemoteEndPoint.Address ;$$response = $$context.Response ;Write-Host '' ;Write-Host('> {0}' -f $$requestUrl) ;  ;$$count = 1 ;$$k=$$callerCounts.Get_Item($$clientIP) ;if ($$k -ne $$null) { $$count += $$k } ;$$callerCounts.Set_Item($$clientIP, $$count) ;$$header='
Windows Container Web Server
' ;$$callerCountsString='' ;$$callerCounts.Keys | % { $$callerCountsString+='
IP {0} callerCount {1} ' -f $$_,$$callerCounts.Item($$_) } ;$$footer='' ;$$content='{0}{1}{2}' -f $$header,$$callerCountsString,$$footer ;Write-Output $$content ;$$buffer = [System.Text.Encoding]::UTF8.GetBytes($$content) ;$$response.ContentLength64 = $$buffer.Length ;$$response.OutputStream.Write($$buffer, 0, $$buffer.Length) ;$$response.Close() ;$$responseStatus = $$response.StatusCode ;Write-Host('< {0}' -f $$responseStatus)  } ; "
      nodeSelector:
        beta.kubernetes.io/os: windows

The service & pod has been up for 21 minutes (Screenshot).
Not able to ping a server in the same vnet, that I am able to ping from linux containers in the cluster:
PS C:\> ping <IP_ADDRESS>

Pinging <IP_ADDRESS> with 32 bytes of data:
Request timed out.
Request timed out.
Request timed out.
Request timed out.

Ping statistics for <IP_ADDRESS>:
    Packets: Sent = 4, Received = 0, Lost = 4 (100% loss),
PS C:\> nslookup <IP_ADDRESS>
DNS request timed out.
    timeout was 2 seconds.
Server:  UnKnown
Address:  <IP_ADDRESS>

DNS request timed out.
    timeout was 2 seconds.
*** Request to UnKnown timed-out

Not able to curl internal dns of other service:
PS C:\> curl -UseBasicParsing http://MY_SERVICE_DOMAIN/MYROUTE
curl : The remote name could not be resolved: 'MY_SERVICE_DOMAIN'
At line:1 char:1
+ curl -UseBasicParsing http://MY_SERVICE_DOMAIN/MYROUTE
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-WebRequest], WebException
    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeWebRequestCommand

This issue has become incredibly frustrating. I am using ACS 1.7.7 and can only occasionally resolve a remote URL even with dns caching turned off. This is preventing us from moving forward with ACS.

@SteveCurran Sorry for the inconvenience. There is some update in another thread: https://github.com/Azure/acs-engine/issues/2027. With the next Windows/docker image update, it will mitigate DNS issue in most scenarios.

@SteveCurran I was able to overcome this issue in a new cluster using the following:
acs-engine: 0.13.0
kubernetes: 1.8.4
A couple things I learned when trying to update my cluster using these instructions: When i tried updating the 1.7/0.11.0 cluster to 1.8.4 using 0.13.0, it failed almost immidiately, so the update process has changed between the two acs-engine versions and has introduced new fields somewhere in the engine's output folder. When I tried updating 1.7/0.11.0 to 1.8.4 using the 0.11.0 engine, it failed during the ARM update steps. I have yet to debug the output (it didn't give me a helpful error).
So at this point I'm looking at:

try to figure out why the update is breaking and fix it
if i can't fix the update, i'm going to have to create a new cluster with 1.8.4.

Side note/rant: My cluster needs to communicate with other external servers in the same vnet. These servers take a lot of time and effort to configure once they're provisioned (due to the legacy applications that need to run on the servers).  Hybrid clusters using the acs-engine do not currently support deploying to an existing vnet. So I'm now on my third time deploying a new cluster, new vnet, and new legacy servers.  I'm keeping my fingers crossed that this gets fixed before we need to update a production cluster because at this point it's going to require a complete rebuild of the vnet and all the resources within it.

@patrick-motard did you manage to mitigate the issue with a 1.8.4 cluster? I'm running a 1.8.8 cluster now, and one of my pods seems to have internet access (I'm using Azure AD amongst others), but another pod still cannot resolve the host name, so the problem isn't going away.
I have the same issues you have, I have to rely on an Azure admin in my company to the cluster according to my acs-engine templates (rights issue), and I've had to bother them too many times already :(

@roycornelissen after working on this issue today some more, I am still having this bug. On 1.7 I was not able to communicat with the internal k8s DNS from inside a windows container in the cluster. On 1.8.4 I can now resolve service DNS from within a windows container. I still cannot access the internet nor any ips within the vnet (outside of the cluster).

Also @JiangtianLi, this issue does not seem to be the same as the 15 min race condition bug. My containers have been deployed for hours and are exhibiting this behavior.

@skinny 's dns flushing quick fix does not fix this issue for me either.

I'm having the same issue as @patrick-motard. Are there any updates to this?

@jdinard @patrick-motard There have been a lot of networking fixes as of acs-engine 0.19.2 and 0.19.3 -- do you still see this issue there?

No new responses here - closing. If you hit this problem on a newer deployment, please open a new issue.

Hitting this problem on a new deployment, will file a new issue.


Website: jdt8.net

Another domain following the format of #85, when a custom handler is made this can be reused for this domain too.
Possibility: Match two different domains?
https://jdt8.net/c/?si=17650&li=1761591&wi=382966&ws=checklist&dl=

If information is found on this domain please comment on issue #85, not here.


Conditionally render EmptyState or Search Results

Final Part of #716

@borisyankov Ready for review!

Would using react-flow-types make annotating react components as param types in functions more convenient?

Not sure how much help it is. Maybe complicates stuff and makes us dependent on another project that might not be supported actively in future.

@neerajwahi Not sure how to deal with this linting errors.
Before running prettier:

After running prettier:

@nashvail prettier with ternary conditional statements conflict sometimes with eslint
So either you can extract the bigger part into its own file or you     /* eslint-disable */ commands

Fixed the linting errors. Travis is currently failing on files that hasn't been changed in the PR

@borisyankov updated!

@borisyankov done!

Cool. Merging.


Questions on the GAT conv layer

❓ Questions & Help
I have a few questions from a newbie in PyTorch Geometric regarding the GAT model:
1/ In the forward method, we call propagate as follows:
 out = self.propagate(edge_index, x=(x_l, x_r), alpha=(alpha_l, alpha_r), size=size)
Should I understand that x_l will map to the source features (x_j in message) whereas x_r will map to the target features (x_i in message) ? same thing for alpha ?
2/ In message, we treat the messages on all edges of the graph (well, the graphs in the mini-batch); I have hard times understanding how we can perform a softmax here; it is as if the message method was dealing only with the messages to a single target (can I stick to this view though it might not be right?). How is it done under the hood?
3/ What is the difference between a Tensor and a OptTensor ?
Thanks !

Hi,

That's correct. We refer to x_l as the source nodes, and x_r as the target nodes (in a bipartite graph). So for example, GATConv((128, 256), 256) will aggregate 128-dimensional feature vectors and combine them to 256-dimensional feature vectors.
Usually, message can be seen as a method that operates independently for each edge. However, in GATConv messages are inter-dependent (via the softmax). PyTorch Geometric provides a softmax function (torch_geometric.utils.softmax) that normalizes inputs across the same target nodes. This function is used here.
OptTensor describes an optional tensor, i.e. a Tensor or None. This might be useful in same cases, e.g., when you have a bipartite graph for which features only exist for source nodes (and you want to obtain target node features based on aggregating incoming edges). You can then do something like: conv((x_l, None), edge_index).

Thanks for your prompt reply.


Use Celery to synchronise single objects to Elasticsearch

Description of change
This:

Stops instantiating the search apps. Although on the one hand instantiating them is nice, they don't have any state and it simplifies things elsewhere as apps can simply be imported in order to use them.
Refactors search signal receivers by only forwarding the model instance to receiver functions (as other arguments were unused).
Changes sync_object_async() to take a search app instead of a search model and Django model. This is to simplify logic elsewhere.
Use Celery instead of the thread pool for copying single objects to Elasticsearch. This is to stop the synchronisation happening in the web process. Retries are also enabled.

This does mean that Redis and Celery are now basically required for running the app. We could accept that, or expose CELERY_TASK_ALWAYS_EAGER as an environment variable to allow running the app without them. Any views?
Checklist

[x] Has a new newsfragment been created? Check changelog/README.rst for instructions
[ ] Have any relevant search models been updated?
[ ] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[ ] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[ ] Has the admin site been updated (for new models, fields etc.)?
[ ] Has the README been updated (if needed)?

@reupen I would expose CELERY_TASK_ALWAYS_EAGER mainly for the testing platforms. Users will tend to expect updates to be reflected in search instantly. Now that there could be a delay although small its worth having this option available in configuration.


Iron Note Block improvements

Add volume
Add pling and bass instruments from Computronics
Allow registering of instruments via MoarP API

SoniEx2/NBSPlayer#2

:+1:

Btw you can make todo lists

[ ] a
[ ] b
[x] c

etc
- [ ] a
- [ ] b
- [x] c

I am aware, but thank you.

Volume and strings are waiting on theoriginalbit/Peripheral-Framework#3 which I should get more of a chance to work on once Uni has finished in two weeks.

Could I suggest that instead of adding support for extra sounds within MineCraft - sounds which may or may not be present within any given install - you instead give access to whatever MIDI synth is available? That'd provide a TON of sounds, pretty much all of which could be found on most any system! Probably even on those dodgy "Mac" thingies!

well the plingand bass are vanilla sounds. I have added sound effects like so.
I have been considering adding some kind of synth, though it's all about trying to get the one that is built in Java to work well with Minecraft in similar ways to the note (i.e. hearing distance and volume), which is no small feat.

@BombBloke doesn't work well with Minecraft.


Error installing library

I followed the instruction from documentation. have installed SDK and trying to link library to my project. And I get following error while building my project.
/Users/.../node_modules/react-native-mixpanel/RNMixpanel/RNMixpanel.m:12:9: Module 'React' not found

header search path is as follows
$(inherited) => non-recursive
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include => recursive
$(SRCROOT)/../../React => recursive
$(SRCROOT)/../../node_modules/react-native/React => recursive

Am I missing something? I am using react-native 0.25.1

Hi @SushilShrestha , I removed React dependency.
Please try v.0.0.10, let me know if it works!

It's working now. Thanks @davodesign84.


Missing CONTRIBUTING.md File in Repository

The repository currently lacks a CONTRIBUTING.md file, which is an important resource for guiding potential contributors on how to engage with the project effectively. A CONTRIBUTING.md file outlines the contribution guidelines, coding standards, and procedures for submitting pull requests, thus streamlining the contribution process and ensuring consistency across contributions.
Could you please assign this issue to me under GSSOC 24.

Hey @suhanipaliwal , you can go ahead with this


Allow Multiple Terminals

I am trying to create a demo where I start a server and then curl the API. It would be nice if I could open a new terminal session (it would look pretty cool if it shows up as a tab at the top) and switch back and forth while adding responses to each one independently

@rafaelcamargo Hi again.
What's your idea to handle tab?

@rafaelcamargo  I think you must add a tab component that need to handled by application.
Because of backward compatibility you need to have default write() command() response() but you can consider that every things happened in a tab.So by calling write() or command() response() considered that client want to write to default tab and here you can add new function in application that call openTab(name) or openTab(id) that creates new tab and writing was happened in that tab.(you can handle switch tab by openTab(name) like openApplication()

I agree with @mostafaebrahimi. I think that API would be most consistent and convenient

@mostafaebrahimi The way is fine. Tab must certainly be a new component to be orchestrated by an application.
Regarding the API, I think tab could be a new option for the method openApp:
gdemo.openApp('editor', { tab: 'tabName' }).end();

If no tab is specified, the default one is used.
However, how would you handle window title in the application? After creating the tab component, I could have more than just one open file in the editor for example. 🤔
What's your thoughts?

@rafaelcamargo I think some changes we need here:

create new method openTab that creates new tab(editor-applicaiton) or find application and open it
create new Animation for handle tabs(like animtaion in openApplication that scale - here switch -)

@rafaelcamargo  What's your thought?

@mostafaebrahimi So, I am still figuring out how to solve that, but the approach discussed above seems to be really fair. Since this feature is a little complex, I'll focus on getting that done on this weekend and will resume my thoughts on this issue on the next week 👍

I am interested in this feature too!  It would be great to have tabs in the editor for different files.  Has progress been made on this?

Hi @quinton-ashley!
No, this feature is not in development yet. Since I'll be working the next weeks on another glorious project to release it, I'd be very happy If you consider you're be able to offer this contribution ✌


Responsive options dropped when doing updateOptions via exec

Codepen
https://codepen.io/anon/pen/OBpYXW
Explanation
Pen is a fork of the responsive example from ApexCharts docs.
Set your viewport size smaller than the breakpoint 1000.
Now select "2009" from the dropdown (it triggers the exec).
Notice the chart swings back to the wide screen options.

This is very similar to #74. Thanks for the detailed explanation.
Hopefully, will be fixed in the next release

Thanks for the quick fix! I tested the patch and confirm it is working.

I don't understand how it is possible, but it looks like the patch didn't work after all. If you open the Codepen example, go to Settings - JavaScript and change the url to https://cdn.jsdelivr.net/npm/apexcharts?v=214 (which should have the patch) or https://cdn.jsdelivr.net/npm/apexcharts?v=250 it makes no difference. The problem of the chart changing back to horizontal bars upon dropdown change remains.
I also confirm the problem with local .js files.
On 10 Oct, I manually applied the patch and tested and I have no idea why I now see different results.

v2.1.6 broke the responsive options as there were some major changes.
Will be fixed soon

Fixed in 2.5.1.
Please update using<EMAIL_ADDRESS>

Thanks for the work, but I am still seeing the reset to horizontal both in the Codepen and locally when using 2.5.1

I didn't update the codepen, but I see in your codepen you have appended ?v=251
That is not the correct way to get the latest version.
Please use the following URL to get a specific version
<EMAIL_ADDRESS>Here is my codepen - https://codepen.io/apexcharts/pen/xmPJEL

Thanks, that codepen certainly confirms the fix! Yet, the dist/*.js files in the 2.5.1 release seem to contain the old version.


test(extension): update tests related to LW-9270

Checklist

[ ] JIRA - <link>
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Explain how does this PR solves the problem stated in JIRA ticket.
You can also enumerate different alternatives considered while approaching this task.
Testing
Describe here, how the new implementation can be tested.
Provide link or briefly describe User Acceptance Criteria/Tests that need to be met
Screenshots
Attach screenshots here if implementation involves some UI changes

coverage: 52.865%. remained the same
when pulling 9526ca64b8abbd4388f179d62f8f70115084f5da on test/LW-9636_update_tests_related_to_LW-9270
into bca4586312e937019b2386a2a88d7ddc14ed282f on main.


Need sandbox.io example

Could you provide a working sandbox for this project on sandbox.io as an example for integration?

See also Glitch.com... this starter made it really easy to give a-frame a spin: https://glitch.com/~aframe

Hi, I made this template https://codesandbox.io/s/theatre-example-b49m9i
Let me know if it helps or not :+1:

Hi there! I'm having trouble launching the Studio in my r3f project, and I'm seeing the same issue with the example you provided @leonardomerlin - I can't see the studio interface at all. Are you able to?