Summary modal

Adds a modal window when hitting the save button to add a summary and optional legacy version for the newly added revision (when saving)
If the document version is > 1 / > 1.0.0 the summary and legacy version fields will be prefilled with the data from the last revision history item.
Also changes the existing fields for legacy version to inputs instead of textareas. These are typically short strings.

The markdownlint action failed as two translated documentation files contained ill-formatted links. This must have slipped through when merging https://github.com/secvisogram/secvisogram/pull/255 and I quickly fixed that here to pass all tests.
The underlying issue is referenced in https://github.com/secvisogram/secvisogram/issues/300

closed in favor of https://github.com/secvisogram/secvisogram/pull/305


Set scopes properly in discovery doc

We now provide the api's defined oauth scopes in the discovery doc. Additionally, scopes can be passed as an object which contains both the scope name and a description.
For example:
@api(name='example', scopes=['https://www.googleapis.com/auth/santa'])

This lacks a description, so it will be presented to the user as that URI. Instead, we can do this:
scope = OAuth2Scope(
    scope='https://www.googleapis.com/auth/santa',
    description='Access your letter to Santa')
@api(name='example', scopes=[scope])

And the description will be included in the discovery doc.

Codecov Report

Merging #91 into master will increase coverage by 0.12%.
The diff coverage is 94.93%.

@@            Coverage Diff             @@
##           master      #91      +/-   ##
==========================================
+ Coverage   84.62%   84.74%   +0.12%
==========================================
  Files          40       40
  Lines        5659     5730      +71
==========================================
+ Hits         4789     4856      +67
- Misses        870      874       +4

Impacted Files
Coverage Δ

endpoints/users_id_token.py
87.68% <100%> (+0.46%)
:arrow_up:

endpoints/test/users_id_token_test.py
98.62% <100%> (+0.03%)
:arrow_up:

endpoints/discovery_generator.py
92.79% <100%> (+0.08%)
:arrow_up:

endpoints/api_config.py
96.89% <88.88%> (-0.24%)
:arrow_down:

endpoints/test/discovery_generator_test.py
83.73% <93.93%> (+1.91%)
: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 81abc02...92b9eb0. Read the comment docs.


PlayerId not same as seen from both players

Hi,
Some of the features in our games rely on having the list of PlayerIds from a turn based match. Since the last update of Google Play Games / Google Play Services those features are broken.
When a Gamer ID is created for a player, the PlayerId seen by a user differs from the PlayerId seen by the opponents.
Is there any way to enable the PlayerId as it was working before Gamer ID was created? Does it give the correct id if asking for Plus scope or anything?
We need some way to uniquely identify all the players in a match, independently of the player who requests it, and it should be the same between matches too.
Thanks,
Jorge

We are talking to google about this issue now too. Currently the answer is 'you shouldn't be using Ids like that'. Far from useful.

I think lack of documentation leads us developers to make assumptions, that may turn out to be wrong. To avoid some of these pitfalls, it would be great if google could answer the following questions:
TurnBasedMatch ID:

can we trust this is globally unique?
can we trust this never changes for a specific match over time?
can we trust all participants in a match will always see the same match ID?
can we trust a match ID is never reused for another match?

Participant ID:

can we trust this never changes for a specific participant in a specific match?
can we trust all participants in a match will see the same participant ID for a specific participant?
can we trust this is not reused (eg if an automatched player leaves a match and another joins)

Player ID:

can we trust this is globally unique?
can we trust this never changes for a specific player over time?
can we trust this is never reused?
can we trust all players see the same player ID for a specific player? (answer to this is obviously "no")

TurnBasedMatch ID:

I know it is unique per device-player, I would need to check that is globally unique.  I would not recommend sending the matchid from one player to another for example.
The ID is stable for the match, including rematches.
I would need to test this, but again, I would not recommend sharing match ids between players without using the proper TBMP APIs.
Yes, a player will never see the same match id for a different match.

Participant ID:

Yes, participant ID is stable for a match.
Yes, I believe the participant IDs are the same for all participants in the match.
Yes, the participant Ids are unique within a match.

Player ID:

Yes Player ID uniquely identifies a player.
Mostly yes :/  There is a migration underway right now that disassociates the player's Plus account with the gamer account.  This is causing players to be assigned a new ID.  Once the migration is complete, then it will be stable.
Yes, player IDs are never reused.
Yes player Ids are stable and unique - everyone sees the same ID for the same player.

During the migration things are confusing since depending on the local player's migration status and the other player's migration status the ids may be different.  This is a temporary problem and will eventually subside.
Please share any cases where your experience is different from these answers and we can see what is going on.

Thanks a lot Clayton for the very detailed answers! Thas was very helpful.
(We currently rely on match ID being the same for all players in order to do match score calculations on our server, but since it is not recommended we will have to change that)

On the PlayerID side of things, there isn't much we can rely on.
Player ID:
can we trust this is globally unique?

The ID should only identify one player, but there can be multiple IDs per player
can we trust this never changes for a specific player over time?
No, this is changing right now, so it could happen again. Using this for anything outside google APis is not supported
can we trust this is never reused?
Probably.
can we trust all players see the same player ID for a specific player? (answer to this is obviously "no")
No, currently, if you created an account before the switchover you will get you old ID when querying the API, however your friends will see the new ID.


feat: add more helpers for pendle v2

PR-Codex overview
The focus of this PR is to enhance the PendleV2 integration in the SDK by adding more helper functions and improving existing ones for Pendle protocol interactions.
Detailed summary

Added simulateContract helper function
Updated BuyPrincipleTokenArgs to include minPtOut
Updated SellPrincipleTokenArgs to remove principalTokenAddress
Added new functions for interacting with Pendle protocol
Improved readability and efficiency in the PendleV2.ts file

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

/snapshot


Q: Is there a default "ai_user" (guid id)? Could there be?

I'm using AI to send telemtry for my VS Code extension and have been setting the ai_user like this:
appInsightsClient.context.tags[appInsightsClient.context.keys.userId] = <my own generated guid>;
I set up the appInsightsClient like this (I collect nothing automatically by default):
appInsights.setup(<my-production-insightsKey>)
.setAutoCollectConsole(false)
.setAutoCollectPerformance(false)
.setAutoCollectRequests(false)
.setAutoCollectExceptions(false)
.start();
It appears that I always need to set the context userId tag myself. So if I want to make sure that the user id doesn't change for a particular user of my extension, I need to manage that guid myself (persist it somehow so the next time my VS Code extension runs, I have to ensure I use the same guid).
I just attempted to send AI telemetry without sending the userId and the data appeared in the AI portal as user -.  I'm using v0.15.16 of AI and am wondering if this ability exists in later versions (or even if this is a reasonable request)?
To be clear, this is a client-side (non-web) application (Visual Studio Code) from which I am sending AI telemetry.

On second thought, I don't think this is a reasonable request.  😄  I was (mistakenly) under the impression that the userId had to be a particular format and that's not the case.  I've been able to secure another method to create the id without having to 'manage' it.


Tracing from Browserstack

I am executing my test script at Browser Stack
Test script is storing the tracing locally
Browser.New Context    tracing=browser/traces/traces.zip    recordVideo={'dir':'videos', 'size':{'width':1080, 'height':720}}    viewport={'width': 1512, 'height': 865}
On executing the test cases at  BrowserStack it is throwing an error
Error: Path is not available when connecting remotely. Use saveAs() to save a local copy.
Is there any provision for this.

I tough this issue is fixed, which version you are using and have you tried with the latest release?

Ah now I recall, we made fix in file download, but not for traces: https://github.com/MarketSquare/robotframework-browser/issues/2615 So this is valid issue, most likely same logic can be applied also for traces.

@allcontributors please add @gbhandarkavathe for bugs

Actually when I was able to reproduce the problem, the error is not caused by tracing, it is caused by creating the video.

Could not easily find way to save video on remote browser. Just decided to suppress the error and not support video on remote browser.


Update.bat not running

Please tick all applicable boxes.

[x] I am using Python 3.5 or higher (run python --version on the command line)
[x] I have followed the official guides to install the bot for my system
[ ] I have updated my dependencies to the latest version using the appropriate update script
Can't do this one...

Which version are you using?

[x] The latest master version (1.9.8)
[ ] The latest review version

What type of issue are you creating?

[x] Bug
[ ] Feature request
[ ] Question

Description of issue
Error in running update.bat
Steps to reproduce

Run update.bat

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.
I don't have one... I tried to run update.bat and got an error, I did this a few days ago and it worked fine. I uninstalled it and installed it again, downloaded the correct version of python and everything, but now i can't even use the bot anymore.. so here is what happened..
Here Is the error
Error running update.bat.TXT

cffi need a c++ compiler. You need to install the visual c++ build tools. You can download them here: https://visualstudio.microsoft.com/visual-cpp-build-tools/
Note that while the error is calling for vc++ 14.0 - everything will work with newer versions of visual c++.

Thanks!


remove gifs

gifs are heavy on the repo
reference:
https://www.reddit.com/r/neovim/comments/wsg5yx/attention_to_all_plugin_authors_newcomers_careful/

Nice catch, I didn't see that they were that big 😅


[Help] Open file using source path, do not using input file

I want to open file xlsx using source path, do not using input file. Please help me!

use XMLHttpRequest !
var filePath = 'C:\your\local\path\filename.xlsx';
var xhr = new XMLHttpRequest();
xhr.open("GET", filePath, true);
xhr.overrideMimeType('text\/plain; charset=x-user-defined');
xhr.onload = function(e) {
    var data = xhr.responseText;
    // dummyObject
    var f = new File([], 'sample.xlsx', {type:'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet'});

    var reader = new FileReader();
    reader.onload = function(e) {
        var projectTime = {};
        var wb;
        wb = XLSX.read(data, {type: 'binary'});
        // do something
    };
    reader.readAsBinaryString(f);
};
xhr.send(null);

This is old but maybe it'll help someone.
I do this with my nw app:
this.workbook = xlsx.readFile( path.join( url, "" ) );

i did this in making chrome addon.

@mrdance92 if you can follow up with some more details we can figure out what's going on
@Inateno if you have a small NW sample we can link to it in the wiki!

Sorry unfortunately it was a project in a previous company and I can't share anything from it.
I'll try to do a sample if I have some time this week.


Updated CODEOWNERS and MAINTAINERS files

Addressing https://github.com/notaryproject/.github/issues/55 https://github.com/notaryproject/.github/issues/56 and https://github.com/notaryproject/.github/issues/57

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

Comparison is base (ef1051f) 88.27% compared to head (4ff04e3) 88.27%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   88.27%   88.27%
=======================================
  Files           7        7
  Lines         273      273
=======================================
  Hits          241      241
  Misses         27       27
  Partials        5        5

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


Adapt new sbi API

adapt to new sbi API for abc
move KDE from sbibm scripts to sbi.

fixes #20

closed in favour of #26


Remove Environment step in app creation wizard

Description

Remove the Environment creation step
Type of change

[x] Refactoring (no functional changes, no api changes)

Screen shots / Gifs for design review

Add Application:

Add Component:

Browser conformance:

[x] Chrome
[x] Firefox
[x] Safari
[ ] Edge


Use alongside Apollo Server

From what I can gather from the README, this seems like it's a replacement to running a GraphQL Server, like Apollo, and executing Queries/Mutations over WebSocket as well.
Would it be possible to run it alongside an Apollo instance, purely to handle Subscription? My immediate thought was to just pass the same schema to createInstance that I do to Apollo, and use createInstance for the publish function via my existing Context.

I also make heavy use of Apollo Client and it's client-side caching, since this uses graphql-ws, would integrating the connection over the graphql-ws client work out-of-the-box?

Hey @oyed thanks for getting involved!

Would it be possible to run it alongside an Apollo instance, purely to handle Subscription?

Totally - you can configure your client to use HTTP/S for queries and mutations (this is usually a default) 👍


Detect failed rake generate

A couple of times now, the rake generate has failed, but the rake deploy just keeps on trucking, which results in no site being published, obviously, this is bad!
Need to update the appveyor.yml file to detect if the rake generate failed, and only deploy if it was successful.

No longer an issue with Wyam version of site.


[Fix #4823] Make unused argument cops aware of reassignments

As it stands now, unused argument cops (Lint/UnusedBlockArgument and
Lint/UnusedMethodArgumentSpec) treat reassignments as argument
usages.
This change makes unused argument cops aware of reassignments.
The following cases are handled:

Argument was unused before the reassignment (cop complains):def foobar(baz)
  baz = 42
  puts baz
end

Argument was used before the reassignment (cop does not complain):def foobar(baz)
  puts baz
  baz = 42
  puts baz
end

Argument was used at the rhs of reassignment (cop does not
complain):def foobar(baz)
  baz = 42 + baz
  puts baz
end

Argument was reassigned in the conditional (cop does not complain
since it is unknown whether the branch will be executed)def foobar(baz)
  if condition?
    baz = 42
  end

  puts baz
end

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests(rake spec) are passing.
[x] The new code doesn't generate RuboCop offenses that are checked by rake internal_investigation.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[ ] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

This is some awesome work! We should change the specs to use the new matcher format (in another PR), but for the sake of ease of reviewing, I'm glad you didn't do it as part of this change. 🙂

Great work indeed! 👍


Find solution for starting / stopping gifs in our blog

Having gifs next to some text section can be quite distracting. There's probably some simple solution that we just have to search for.

One possible solution: http://rubentd.com/gifplayer/


Not work install ubuntu 19 Dragonfire 1.0.3

Installation is interrupted:
Collecting wikipedia==1.4.0
Using cached https://files.pythonhosted.org/packages/67/35/25e68fbc99e672127cc6fbb14b8ec1ba3dfef035bf1e4c90f78f24a80b7d/wikipedia-1.4.0.tar.gz
Collecting PyUserInput==0.1.11
Using cached https://files.pythonhosted.org/packages/d0/09/17fe0b16c7eeb52d6c14e904596ddde82503aeee268330120b595bf22d7b/PyUserInput-0.1.11.tar.gz
Collecting tinydb==3.9.0.post1
Using cached https://files.pythonhosted.org/packages/bc/0e/a177660a4c334d2c67237f98f41585ebb0b6dcd69c0197f3f3a0d3f91573/tinydb-3.9.0.post1-py2.py3-none-any.whl
Collecting youtube_dl
Using cached https://files.pythonhosted.org/packages/67/99/977114c1c11cb3afdcc685f166ce844d052baae3ba7aa5d9f26da140c409/youtube_dl-2019.11.22-py2.py3-none-any.whl
Collecting spacy==2.0.13
Using cached https://files.pythonhosted.org/packages/05/39/03dfd0164c884f8a0250605cf64edf55d9ac8315c3c345453780a7dae7d8/spacy-2.0.13-cp37-cp37m-manylinux1_x86_64.whl
Collecting pyowm==2.9.0
Using cached https://files.pythonhosted.org/packages/28/ce/a2d6db7f6c6073e677a53e13c155d9faf1d9a6ef12f3b2c2db188552ab91/pyowm-2.9.0-py3-none-any.whl
Collecting tensorflow==1.0.0
Could not find a version that satisfies the requirement tensorflow==1.0.0 (from versions: 1.13.0rc1, 1.13.0rc2, 1.13.1, 1.13.2, 1.14.0rc0, 1.14.0rc1, 1.14.0, 2.0.0a0, 2.0.0b0, 2.0.0b1)
No matching distribution found for tensorflow==1.0.0

@genikineg I'm currently upgrading TensorFlow version from 1.0.0 to 1.14.0 https://github.com/DragonComputer/Dragonfire/commit/b5144bcabb325c452bfd46c48c4b4b21de5c1a50
Please try sudo ./install.sh

@genikineg I'm currently upgrading TensorFlow version from 1.0.0 to 1.14.0 b5144bc
Please try sudo ./install.sh

Yes, I run sudo ./install.sh and I get an error
Collecting wikipedia==1.4.0   Using cached https://files.pythonhosted.org/packages/67/35/25e68fbc99e672127cc6fbb14b8ec1ba3dfef035bf1e4c90f78f24a80b7d/wikipedia-1.4.0.tar.gz Collecting PyUserInput==0.1.11   Using cached https://files.pythonhosted.org/packages/d0/09/17fe0b16c7eeb52d6c14e904596ddde82503aeee268330120b595bf22d7b/PyUserInput-0.1.11.tar.gz Collecting tinydb==3.9.0.post1   Using cached https://files.pythonhosted.org/packages/bc/0e/a177660a4c334d2c67237f98f41585ebb0b6dcd69c0197f3f3a0d3f91573/tinydb-3.9.0.post1-py2.py3-none-any.whl Collecting youtube_dl   Using cached https://files.pythonhosted.org/packages/67/99/977114c1c11cb3afdcc685f166ce844d052baae3ba7aa5d9f26da140c409/youtube_dl-2019.11.22-py2.py3-none-any.whl Collecting spacy==2.0.13   Using cached https://files.pythonhosted.org/packages/05/39/03dfd0164c884f8a0250605cf64edf55d9ac8315c3c345453780a7dae7d8/spacy-2.0.13-cp37-cp37m-manylinux1_x86_64.whl Collecting pyowm==2.9.0   Using cached https://files.pythonhosted.org/packages/28/ce/a2d6db7f6c6073e677a53e13c155d9faf1d9a6ef12f3b2c2db188552ab91/pyowm-2.9.0-py3-none-any.whl Collecting tensorflow==1.0.0   Could not find a version that satisfies the requirement tensorflow==1.0.0 (from versions: 1.13.0rc1, 1.13.0rc2, 1.13.1, 1.13.2, 1.14.0rc0, 1.14.0rc1, 1.14.0, 2.0.0a0, 2.0.0b0, 2.0.0b1) No matching distribution found for tensorflow==1.0.0

Work! need clone git!

Building wheels for collected packages: en-coref-sm
Running setup.py bdist_wheel for en-coref-sm ... done
Stored in directory: /root/.cache/pip/wheels/f2/f2/e0/2958eceb8ab3381bdc219bf8ba6213265df8bab8f145963a14
Successfully built en-coref-sm
Installing collected packages: en-coref-sm
Successfully installed en-coref-sm-3.0.0
[nltk_data] Downloading package names to /root/nltk_data...
[nltk_data]   Unzipping corpora/names.zip.
[nltk_data] Downloading package brown to /root/nltk_data...
[nltk_data]   Unzipping corpora/brown.zip.
[nltk_data] Downloading package wordnet to /root/nltk_data...
[nltk_data]   Unzipping corpora/wordnet.zip.
[nltk_data] Downloading package punkt to /root/nltk_data...
[nltk_data]   Unzipping tokenizers/punkt.zip.
Dragonfire is successfully installed to your computer.

$ dragonfire -h
Invalid instruction (memory stack flushed to disk)

@genikineg I have published a new release now.
Please try to install the new .deb package: dragonfire_1.0.4_amd64.deb
or
git fetch --all
git checkout v1.0.4
make install


Changes

These events can be ignored. We need to point it out as it raised some concerns from OEM.
See the following bugs for details.
https://dev.azure.com/ContentIdea/ContentIdea/_workitems/edit/109957
https://microsoft.visualstudio.com/OS/_workitems/edit/19691288
https://microsoft.visualstudio.com/OS/_workitems/edit/15854923

Rich Text Diff view:
https://github.com/MicrosoftDocs/windows-itpro-docs/pull/5440/files?short_path=e928d43#diff-e928d432d2aa8c20cd0f68575585d5eb
(this view might be unstable in this PR, so try again if you get a viewing error from Github)

@Dansimp should these changes be live? I do not see them reflected in the public doc yet.

The live page (https://docs.microsoft.com/en-us/windows/security/identity-protection/smart-cards/smart-card-events) has not been updated since 04/19/2017 (according to the info on the top of the page), so it appears that commit https://github.com/MicrosoftDocs/windows-itpro-docs/commit/6aaf8f631f93fbdbe802a1b917f872118a767d53 has not migrated yet. Not sure why it has taken so long for it to migrate, because I don't know what determines when these commits go live.


Shortcuts

Shortcuts
For example pressing f2 with open the rename dialog and so on.

Can you please create a list of all shortcuts you wish?

All the ones from IDA :trollface:
n should be rename, not f2

make it configurable!

maybe add like presets
for example the shortcuts from ida as @xcfrg said and others

Please create a list of all shortcuts you like to have implemented. So it's possible to plan out.


[SPARK-18994] clean up the local directories for application in future by annother thread

What changes were proposed in this pull request?
clean up the directories of the app by asynchronous method in future block

Can one of the admins verify this patch?

Please read http://spark.apache.org/contributing.html  You'll need to close this and open vs. master instead.


Hard Crash followed by "database disk image is malformed"

Mac shutdown completely overnight with crash report identifying Chia 1.0.1. Tried to restart the app this morning and it just hangs trying to connect. Figured I would start fresh, so deleted the app, the preference file, the folder within Application Support, as well as all of the unfinished plots. Also deleted keychain from Keychain access. Now when I launch and try to recover via Seed Phrase, it gets me to the "Restore Metadata for Coloured Coins" step, but as soon as I hit skip, I get a "database disk image is malformed" error. If I click "skip" again, it just keeps throwing "fingerprint" as the error.

 As it stands, I can't do anything. No plotting, no farming, no connecting to the network for status even.

@mikeindustries Nope, I'm dead in the water. Bit frustrating @hoffmang9

You should completely remove the ~/.chia folder and try opening the app again.
Make sure all Chia processes are killed (and restart pc) before you open it.


Activating T2 can let prereq cards for T1 be selected all over again

I unlocked Champion (T2) and then I could suddenly choose "Pulse" again to replace "Earthquake" (the T1 upgraded version).
Pulse progress has been reset to 0/3.
I don't think Champion (T2) even worked tbh... No one was pushed back from me dashing into them, and I dashed straight through entire groups of enemies for several minutes.

I don't think Champion (T2) even worked tbh... No one died from me dashing into them, and I dashed straight through entire groups of enemies for several minutes.

bro
you duplicated your bug report

Duplicated how? I can only see one...

If you choose the card it will just reroll it, it won't let you change the power when it's activated.


New configuration seems to change positions of the board's svg

here is the standartconfiguration of calliope before storing

which is ok
and here how it looks after storing and loading again

I found more problems with this way of the configuration svg block:
e.g.

which is not that big of a problem, since it isn't saved anyway.
Or

which is a big problem, since it causes a server error.

The problem should be solved. For testing purposes I have created some examples (exported and stored in the db).
issue-933-testing.zip
In the DB is one user, username Ayk and password AykAyk who has programs for Calliope, Microbit and Arduino.


Implementation of Projects Page

Please assign this to me.

@rishicds sir, also assign this to me.

assigned @Shariq2003


Change requester of a request

Description
Many staff are still used to a workflow where they send in forms / details of a count to be conducted (e.g. by email), and Data Collection enters those details into the relevant system (previously ATS, now MOVE).  This means Data Collection is still entering requests into MOVE on users' behalf - but for accuracy of request history, it's useful to link those requests to the original requester.
As an interim solution (until workflows are changed), this task adds the ability for Data Collection to change the requester of a request (i.e. from the Data Collection user to the original requester).
Acceptance Criteria

[ ] UserController adds an autocomplete suggestion endpoint that:

[ ] matches users on either their username or their email address;
[ ] is available to all authenticated users;

[ ] this autocomplete suggestion endpoint is wrapped into a new FcInputUserSearch component in the UI;
[ ] in View Request, users with STUDY_REQUESTS_ADMIN scope can change the requester of a request;
[ ] users with STUDY_REQUESTS scope but without STUDY_REQUESTS_ADMIN scope cannot see the relevant controls.

This feature request may be revisited in future, but is not a part of near-term priorities.


initDepPkgs fix: allow pubName not found

Codecov Report

Merging #130 (8c6a367) into main (3d3e43d) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #130   +/-   ##
=======================================
  Coverage   87.41%   87.41%
=======================================
  Files          10       10
  Lines        3298     3298
=======================================
  Hits         2883     2883
  Misses        330      330
  Partials       85       85

Impacted Files
Coverage Δ

cl/multifiles.go
81.76% <100.00%> (ø)

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 3d3e43d...8c6a367. Read the comment docs.


How to update TLS certificates

Reminder to self for when this needs to be done again:
1) get the new certs:
- change the CNAME for these domains from gateway-int.ipfs.io to earth.i.ipfs.io (which is still a functioning gateway)
- wait 2 minutes
- ssh earth
- edit Caddyfile.ssl
- `docker stop nginx ; ./caddy -conf Caddyfile.ssl ; docker start nginx` (it's finished when it prints the http and https urls, ctrl+c then and nginx takes over again)
- change CNAME back to gateway-int.ipfs.io
2) install the certs
- secrets.sh -d to make sure your local secrets are up-to-date
- scp the .crt and .key from earth to secrets/
- Example command: `scp 'root@earth.i.ipfs.io:/root/.caddy/acme/acme-v01.api.letsencrypt.org/sites/wikipedia-on-ipfs.org/*.{crt,key}' secrets/`
- deploy to scrappy and chappy (it should print that the certs changed)
- Example command: `./provsn build chappy && ./provsn upload chappy && ./provsn install chappy`
- secrets.sh -e and commit the new .crt and .key files in secrets_secure/
- make and merge pull request to update `secrets_secure/` git submodule to the latest ref

@VictorBjelkholm Wouldn't it be possible to use acme.sh and a dns challenge instead? https://github.com/Neilpang/acme.sh/tree/master/dnsapi#25-use-dnsimple-api

Yeah. We currently have two different infrastructures running, the one available here and one that is about to get open sourced, but we're not quite there. For the future, we definitely want something that's automatic to renew the certificates, probably nginx + acme-nginx or something like that. But for now, I think time is spent more wisely just going through the manual steps...

@VictorBjelkholm can this be closed now with your recent update to the README?

Closing this one as it's not applicable anymore.


bumpversion uses trailing spaces and tabs for it's configuration

bumpversion uses trailing spaces and tabs if it updates it's own configuration. As long as the configuration is kept in a separate .bumpversion file this is just a cosmetic issue.
But since release 0.5.1 bumpversion allows users to configure it by using setup.cfg. Unfortunately setup.cfg will be messed up by bumpversion by replacing spaces with tabs for indentation and by adding trailing spaces to equal signs.
The optimal solution would be if bumpversion could respect the indentation style that is used and add no trailing spaces at all.

Bumpversion is using ConfigParser underneath to read and write the configuration (both .bumpversion.cfg and setup.cfg). While I certainly have some influence on how the files are read and written (as in #75), your request might be too difficult for that.
I'm still researching whether I can tell ConfigParser to write the file in the same manner it was read.
While I don't like software re-indenting and/or replacing tabs with spaces myself, what is the problem this causes with other software then reading setup.cfg ?

I'm not sure if I understood your question the right way. If the configuration of bumpversion consists of several sections in setup.cfg and you bump the version, the sections are no longer at the same place as before. Furthermore trailing spaces added and leading spaces are converted to tabs. This creates a commit that looks awful.
Therefore we use a separate .bumpversion.cfg file so that setup.cfg is not messed up.

Yes, ConfigParser rewrites the whole file and I don't know if there's a way around that.
After that initial "awful looking" rewriting commit, is there a "real" (you know I hate tabs and trailing whitespace more than any other person, but if we let's set that aside for a moment) problem with other software also reading/writing setup.cfg (as was in #75) ?

Yes, ConfigParser rewrites the whole file and I don't know if there's a way around that.

As mentioned in a similar issue in setuptools, maybe do it like this:

Generate new setup.cfg with ConfigParser
Generate a diff ignoring whitespace changes with diff -w or an equivalent Python library.
Apply the minimal non-whitespace-changing diff to the original setup.cfg.


Order independent hash

Given a fixed-size container, say std::array<T, 3>, I am wondering if it is possible to create a hash with HashState::combine(...) that is independent of the ordering of the data without sorting?
For example, with std::hash, I can do the following (despite it being a horrible hash function):
size_t hash(const std::array<int, 3>& data) {
    std::hash<int> h;
    return h(data[0]) ^ h(data[1]) ^ h(data[2]);
}

assert(hash({1, 2, 3}) == hash({3, 2, 1}));

It is unclear to me if something similar is possible with the opaque HashState class.

We don't provide such an API yet, but this is a gap we're aware of.  I can report, at least, that we've done a fair bit of implementation work in this area internally.  My progress on it got stalled out by user code in our local depot that makes use of internal APIs that we will need to change.  This bug is a good reminder for me to pick this issue back up.
The proposed API in development looks like:
HashState::combine_unordered(state, it_begin, it_end)
and behaves much as you suggest.
(Note that xor is not an ideal state mixing operation, since it works badly with multisets.  We have been looking at addition, with the carry bit being re-inserted back into the state.)

Thanks @vslashg !  It is great to hear that this issue is being addressed!  A built-in HashState::combine_unordered is exactly what we need.

I'm happy to report that this has landed.  stdlib hash tables and SwissTable types are now hashable by absl::Hash out of the box, and you can use combine_unordered to write AbslHashValue implementations for unordered collections of your own.


[Bug]: Exporting load order deletes gustav entry

BG3 Mod Manager Version
<IP_ADDRESS>
BG3 Game Version
Latest - full release
Bug Summary
When you export mod load order, it does that, but it also deletes the gustav entry in modsettings.lsx which makes the game crash on start up
Links
No response

Already reported in #14 whit temp solution

Add mentioned in #14, make sure the Gam Data Path is set correctly in the Settings -> Preferences window, then refresh so the base mod data is loaded. If you don't see "Main" in the campaign dropdown, that's why GustavDev isn't exported to the modsettings.lsx. GustavDev is loaded from the game data folder.


Formatter takes two runs to fully format comment after match arm

Input
fn main() {
    match 1 {
        _ => {},
        // hello!
    }
}

Output
fn main() {
    match 1 {
        _ => {}
        // hello!
    }
}

Expected output
fn main() {
    match 1 {
        _ => {} // hello!
     This means that the formatter needs multiple runs to fully finish formatting the code. That does not seem right. I want to be able to trust the formatter in that it will do it all in one run so that I never get into the habit of running it twice each time just to be 100% sure. Therefore I report this as a bug.
Meta

rustfmt version: rustfmt 1.4.30-stable (8c6769d 2021-01-18)
From where did you install rustfmt?: rustup

Thanks for the report, idempotency issues are indeed a bug. I feel like this is likely a duplicate of an existing issue, but will leave it open for now as I can't seem to track down the existing issue I had in mind.

I am working on this issue.

This is likely a duplicate of #4654 with the only difference being that comments are removed instead of added. The underlying issue seems to be the same, where the comment moves depending on whether or not the source code has a comment or not.


Can I still use range query or boolean comparison if all of my data types is in String?

Given how flexible JSON is, someone can accidentally change types inside one of the JSON field. This will raise an error in Elassandra.
So, I am wondering if I can just configure all fields as string. If the Lucene query inside Elassandra still works, then I won't have any type errors anymore.
Is this a bad idea?

Once a field has a type in the elasticsearch mapping, you cannot change it anymore, any data not matching the initial defined type will be rejected.

On 8 Jul 2020, at 01:27, Didip Kerabat<EMAIL_ADDRESS>wrote:
Given how flexible JSON is, someone can accidentally change types inside one of the JSON field. This will raise an error in Elassandra.
So, I am wondering if I can just configure all fields as string. If the Lucene query inside Elassandra still works, then I won't have any type errors anymore.
Is this a bad idea?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/strapdata/elassandra/issues/359, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACOMPGMYIPUUPGFQM2FXYGTR2OVOLANCNFSM4OT3LHJA.

Sorry for not being clear.
I am planning to create a brand new index with all the fields discovered typed as string.
Then I'll re-push the data from to this new index.

Ordering on numbers is not the same as ordering on strings, so you should use the right type.

On 8 Jul 2020, at 01:37, Didip Kerabat<EMAIL_ADDRESS>wrote:
Sorry for not being clear.
I am planning to create a brand new index with all the fields discovered typed as string.
Then I'll re-push the data from to this new index.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/strapdata/elassandra/issues/359#issuecomment-655194461, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACOMPGMYU5CDBF5WX2K5DFDR2OWTRANCNFSM4OT3LHJA.

Got it. Thanks again for answering quickly.


SHACL on Graphs

closes #24

It will be good to pick a default way to ignore linter's errors.
Examples of two approaches below, assuming there are link duplication (links differs only in #fragments treated as the same) and item formatting problems:

Ignore several lines

<!--lint disable awesome-list-item double-link-->
A whole document
or a section,
or several list items here
<!--lint enable awesome-list-item double-link-->

Ignore non-standard list items one-by-one (one per line)

  <!--lint ignore awesome-list-item double-link-->
  - [SomeSoftware](https://acme.com/some-duplicated-link) some text within an item's name but not a part of the link - description starts with lower-case letter and without an ending comma
  <!--lint ignore awesome-list-item double-link-->
  - [SomeSoftware](https://acme.com/some-duplicated-link) some text within an item's name but not a part of the link - description starts with lower-case letter and without an ending comma

I added code suggestions to the relevant lines according to the approach from n.2 (one-by-one).


Fix uneccessary calls to volume.Unmount()

Fixes #22564
When an error occurs on mount, there should not be any call later to
unmount. This can throw off refcounting in the underlying driver
unexpectedly.
Consider these two cases:
$ docker run -v foo:/bar busybox true

$ docker run -v foo:/bar -w /foo busybox true

In the first case, if mounting foo fails, the volume driver will
get a call to unmount (this is the incorrect behavior).
In the second case, the volume driver will not get a call to unmount
(correct behavior).
This occurs because in the first case, /bar does not exist in the
container, and as such there is no call to volume.Mount() during the
create phase. It will error out during the start phase.
In the second case /bar is created before dealing with the volume
because of the -w. Because of this, when the volume is being setup
docker will try to copy the image path contents in the volume, in which
case it will attempt to mount the volume and fail. This happens during
the create phase. This makes it so the container will not be created
(or at least fully created) and the user gets the error on create
instead of start. The error handling is different in these two phases.
Changed to only send unmount if the volume is mounted.
While investigating the cause of the reported issue I found some odd
behavior in unmount calls so I've cleaned those up a bit here as well.
Signed-off-by: Brian Goff<EMAIL_ADDRESS>

ping @tonistiigi @anusha-ragunathan PTAL

I'm not sure I understand this. Do I understand it correctly that this calls umount(MNT_DETACH) ignoring the volume activity count. If this is the case then how is this safe for parallel cp.

@tonistiigi This is the existing behavior.

Also yes, I'm not sure why it's doing the syscall, I just cleaned up the call path for it.

LGTM
Umount doesn't seem to specify MNT_DETACH (see here)

@mlaventure detachMounted does https://github.com/docker/docker/pull/27116/files#diff-5dfdabc4cd5100a2aee949c12d2afe6cR341

are we good on this one @mlaventure @tonistiigi ?

@cpuguy83 When you do the rebase please add a comment to volumeMount.ID != "" that this is the best way to check had been actually mounted(if I understand this fix correctly). LGTM after that, although I think detach part needs to be fixed in follow up and volumes either shouldn't be in MountPoints if they are not mounted or they should do their own internal reference counting.

Updated

LGTM

LGTM


Dachoka3000/#24/ Add Project Readme

#24 Create README for the project

I've made the requested changes:

[x] Add docker setup section
[x] Add versions in the technologies used section
[x] Add section for setting up environment
[x] Add a more detailed contribution guide
[x] Add shields to indicate stats about the project. (The license shield was there already)
@Albert-Byrone  @otsembo


Upgrade to Angular 15

This should upgrade all the Angular libraries and dependencies to v15, following the upgrade guide.
Odds and ends that were done along the way:

Upgraded the GitHub Actions configs to use Node 18
Early on none of the builds were working because of a version mismatch with jasmine-core; upgrading it from v3 to v4 got the builds to work again.

This was obviated by #1069.


FIX: only acting user should track channel on creation

Other users of the channel should only start tracking once a new message is sent.

Looks like a major spec failure 😅

Looks like a major spec failure 😅

Yep I often let the full suite run on CI.. and apparently something totally broke here :D Will look later

I tried this out locally BTW and it worked great :+1: I was confused at first since in the DM draft you have to send a message for the channel to get created, but then I reread your message and you mentioned creating channels from the user card. I have added this to the PR description for additional context.


buttons to move cards between zones

a "move to" choice
and contextually:
play/discard for cards in hand
tap/untap/destroy/exile for battlefield
exile/return to hand for graveyard

buttons should appear on mouseover when zoomed in enough or in the enlarged popup when a card is clicked


NPM On Windows: NPM Init uses  line endings

npm init creates a package.json that is not properly formatted and uses linux line endings - should at least use crlf on windows

Hm... I'm not so sure about this.  The package.json files are used in a lot of places, and \n line endings are the standard on every system except Windows.  I think it's probably better to just take v8's JSON.stringify output as-is, and not mess with it.
If there's a really compelling case where it causes problems, I'd love to hear it, but otherwise, I'm inclined to go with the simplest workable approach here, and leave them as \n.


[Input] Doesnt support some especific key-combinations

Godot version:
3.3 rc6
OS/device including version:
Windows 10 x64
Issue description:
using some key combinations doesnt work with:
Input.is_action_just_pressed() & Input.is_action_pressed()
So, lets supose you can move your player using WASD , run with SHIFT, and dash with SPACE
If the gamer is using a keyboard he cant simply use some key combinations, i mean pressing 4-5 keys at same time, neither 3 in some cases, for example:
ps: Ive made this app to show keyboard pressed keys
Godot allows:
SD + SPACE = down-right dash

But Godot doesnt allows:
WA + SPACE = up-left dash

**Test Results: **
Not allowed combinations:
W+D+SHIFT+SPACE
W+S+SHIFT+SPACE
Allowed Combinations:
S+D+SHIFT+SPACE
A+S+SHIFT+SPACE
A+S+D+SHIFT+SPACE
OBS: if you look, all W related key combinations doesnt work.
Minimal reproduction project:
KeyboardTest.zip

This is likely keyboard ghosting, i.e. a mechanical problem with many keyboards.


fix bugs

HI @ChrisTitusTech you moerged this fork but it currently doesnt work see https://github.com/ChrisTitusTech/ArchTitus/pull/189#issuecomment-1020888649 So I am creating a PR in hopes you will see this

This also fixes https://github.com/ChrisTitusTech/ArchTitus/issues/219


Hmm

I'm just curious.. if there's option To disable controls how to enable them?

They're enabled by default.
If it does not work by default on your device (Xiaomi?), read https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#mouse-and-keyboard-do-not-work or https://github.com/Genymobile/scrcpy#prerequisites.

Weird.. I have both, USB debugging and security one.. I'll try again
On Fri, Mar 17, 2023, 13:23 Romain Vimont @.***> wrote:

Closed #3831 https://github.com/Genymobile/scrcpy/issues/3831 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/Genymobile/scrcpy/issues/3831#event-8777731763, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AGMGD3TYSHQO22YLBKE62QLW4RJTHANCNFSM6AAAAAAV6NNJBQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Feature/manual emf model

Moves to a manually managed EMF model. This allows to add properties and enum values not in the grammar (see StorageClassSpecifier).
@AtomCrafty Please check if this works with gradle and your nevironment

+1 for going this route, but manually fiddling with these XML files looks terrible 😂 Are there any tools in Eclipse to modify them?

No one sait it would be easy B-)
There is no need to deal with ECore files directly. You may the Ecore editor (the fastest for quick changes) coming with the DSL flavor of Eclipse or you may use the EcoreTools available via the MarketPlace

How does this change affect the future workflow when modifying the grammar? Will all manual changes be discarded when the mwe2 workflow is executed again? Does "manual" mean the automatic generation of classes from the grammar is not available anymore?

Basically yes, 'manual' means the class definitions have to be maintained manually. Therefore I did not put it on develop yet rather to put it on a PR for discussion. If there are larger grammar changes (new or different rules) we should stick with the automatic approach. Unfortunately the the ability to change the generated classes has been removed from the XText framework,
Using this PR a grammar change will regenerate the infrastructure but not the ecore model. So if attributes or rules change you need to put this explicitly into the ecore model so that thge classes are generated accordingly.

In that case I believe this isn't the best move right now. The language is still fairly immature and I expect it to change quite a bit as development progresses.
The issue we're trying to solve here can easily be circumvented by introducing a new StorageClass enum for internal use instead of extending the existing StorageClassSpecifier enum.

But how do you intend to tie it to a declaration as this is a generated class?
But ok, we just drop the PR.

The same way we tie any other analysis results to the AST nodes - with a map. The CoreDslAnalyzer.analyze method returns an AnalysisContext instance that contains all analysis results.


Waypoint API extended

Description
Extended the waypoint API with new functions:

carla.Waypoint.lane_change: Returns a carla.LaneChange.
carla.LaneChange: A Python enum that tells whether traffic rules allow a lane change.

0: None
1: Right
2: Left
3: Both
Can be used as a flag e.g.:

LaneChange.Right & LaneChange.None => False
LaneChange.Right & LaneChange.Left => False
LaneChange.Right & LaneChange.Both => True
LaneChange.Right & LaneChange.Right => True

carla.Waypoint.lane_type: Returns an string that codifies the type of the current lane, e.g.: "driving", "bidirectional", "shoulder", "sidewalk", "median", "parking", "none", ...
carla.Waypoint.get_right_lane(): Returns another carla.Waypoint (although the rules allow the change or not) at the closest right neighbor lane. If there is no right lane, it returns None.
carla.Waypoint.get_left_lane(): Returns another carla.Waypoint (although the rules allow the change or not) at the closest left neighbor lane. If there is no left lane, it returns None. Notice that if the new waypoint is on a different signed lane_id, it does mean that it's in the opposite direction.

Where has this been tested?

Platform(s): Linux
Python version(s): 2.7 & 3.5
Unreal Engine version(s): 4.21

This change is

Hello,
This was a good update from Carla side, I just want to post a simple question, regarding this update.
With this update, we can predict the entire road information for a given position. Now , I just wanted to know, is there any particular way, where we can get the LaneSection of the warpoint in that road ? just curious to know about this.
Thanks in advance

Hi @aravindSwamy94.
We want to be transparent about opendrive in our API.
But in the next release, we will provide the waypoint OpenDrive's current s parameter so, knowing the current road and lane id of the waypoint, you will be able to parse yourself the .xodr file and know everything without our API.
In short, currently not.
Cheers!

@marcgpuig Thank you so much for your reply, getting the s parameter for the waypoint will really solve this issue. Like we can read the s parameters of lane Sections from .xodr file and correspondingly place our points in that section.
Is this change available in 0.9.4 release from Carla.
I once again thank you so much for the update.


Add interface to edit roles

Ok, so I tried this, but I ran into some problems:

We have to check for cycles if we allow editing of the role inheritance (creating a cyclic inheritance blocks all users in those roles from everything)
Doctrine doesn't remove the associations when only persisting the form. In other controllers (e.g. Entity/Controller/TaxonomyController this is handled seperately. I didn't manage to implement the same logic for roles.


Allow organisation filtering to respect date range

Why?
Trello card
Currently after filtering if the user clicks on a content item title the page will
not load due to an empty date range param. This bug was discovered after we added
the filter by organisation functionality.
What?
This PR allows for empty date range params and persists any existing date range params
i.e if in the future we filter the index page to show data from the last 2 years then the page data
page will reflect this.

 (1) Index page - Filter by organisation


Added the fourth Fortran code

I was scared of this part, I'm actually a beginner in Fortran (1 Month experience) and when i saw the class in the video I was terrified that i fail, but now I think that Fortran has a very natural syntax (more verbose though)
I tested the program by first creating the objects then overwriting the random weights with the weights in the tutorial (before deleting/adding class) and it matched :D, so we are on the same page

In this group, you are currently the Fortran master :D


Fixes #1948

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

@yecril71pl : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @corob-msft & @TylerMSFT - Could you review the proposed additions to the content? I don't know whether, if approved, the commit would need to be moved to the private repo for staging before being merged. (It seems like it might not be needed for a new .yml file...) Thanks.

@yecril71pl, could you remove the change to .github/workflows/blank.yml.
It's a small nit, but our style guidelines for voice/tone are to use contractions. Could you modify 'is not' to be: isn't.
Then I'll be happy to approve this change.

@yecril71pl : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@corob-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@corob-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.


rhods_test_jupyterlab: Add test Pod startup delay

Sleep for JOB_COMPLETION_INDEX*SLEEP_FACTOR seconds to avoid DDoSing Oauth.

Un-hardcode SLEEP_FACTOR (time to wait in between users).

Define rhods_test_jupyterlab_ods_sleep_factor from SLEEP_FACTOR.

& sleepfactor: test 0.2

/assign @kpouget


Farooq Mulla - #N/A - Converting Project to Support ARC.

I did changes to the project to support ARC.  Please review and provide you feedback.


Audio Driven - Video Input instead of image

Thanks for the great work; do you know how we can run audio-driven reenactment but have the input be a video instead of an image? (i.e. sync the lips in that video)

Hi, were you able to solve this issue?


(Docs) Moved supported companion computers to bigger link

Thanks :+1:

Agree that the companion computer option needed to be highlighted, and given that we don't really have any information in the docs, pointing off to wiki probably a good idea at this point.
My only problem is it feels like in the docset we've given up on helping people working with companion computers. I think I'd have been happier having the existing material that you moved turned into own doc "Getting started with Companion Computers" with more and better links into the wiki that we'd eventually build out. Fine for now.
This doesn't feel quite right to me as a solution - we talk about companion computers but we don't in the docset really provide an
Short term this is a good solution - we don't have detail docs in this docset for companion computers so linking elsewhere makes sense. I would have slightly more happy having


Finish implementing "Edit fridge item" page

When editing an existing item, the information (text and images) from that item should be rendered/auto filled

implemented


HBAO and DOF

Same as #5673 but with the review comments addressed.

@bagnell, thanks for the pull request! Maintainers, we have a signed CLA from @bagnell, so you can review this at any time.
I noticed that CHANGES.md has not been updated. If this change updates the public API in any way, fixes a bug, or makes any non-trivial update, please add a bullet point to CHANGES.md and comment on this pull request so we know it was updated. For more info, see the Pull Request Guidelines.
I am a bot who helps you make Cesium awesome! Thanks again.


Azure Connection Type?

The descriptions of the Resource Group and Enable Copy Prerequisites arguments refer to an argument called Azure Connection Type, but no argument Azure Connection Type is documented.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 622daa35-c6a1-dac3-17ee-3f9308fc60cd
Version Independent ID: 2c28a59b-8ef5-d7ab-46a6-58748ec552f5
Content: Azure File Copy task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/azure-file-copy.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

Please re-open this issue.
An "Azure Connection Type" is referenced in several contexts on this page.
Please define "Azure Connection Type" on first reference.


remove Secrets permission from user access list

Contributor and Maintainer does not have any permissions for Secrets, ref docs here.
Bug found by @thegreyd

Is this still a valid place for a PR or should it instead be created against https://github.com/konflux-ci/konflux-ui?

Yes, this is a small change and can be merged in hac-dev.

/retest


Add link buttons for headers in docs

BootStrap has this neato feature where hovering over a header in the docs shows a handy link button.  **[Post a bounty on it!](https://www.bountysource.com/issues/11890992-add-link-buttons-for-headers-in-docs?utm_campaign=plugin&utm_content=tracker%2F10548672&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F10548672&utm_medium=issues&utm_source=github).

I think those are called 'anchors.'


why my project on visual studio won't  run ?

Description

Configuration

Regression?

Other information

my project on visual studio won't run, it is written that the issue was the ASP.NET framework, I did almost everthing possible following the instruction given, yet struggling to understand what should I do next.. as of currently I'm a student so I don't have much acknowledge and practical experience about that.. please help me with this

Are you following any tutorials? What happens when you try to run the project?

Yes I followed the first tutorial where I had to input codes into command
prompt, then I followed another tutorial specifically for API when using
visual studio/JavaScript.
I also check for any updates in visual studio installer, and updated my pc,
and did a Microsoft update  as well.
What should I do next?
On Tue, 11 Jul 2023, 09:07 Maira Wenzel, @.***> wrote:

Are you following any tutorials? What happens when you try to run the
project?
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/core/issues/8607#issuecomment-1630196758, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BBF3CDP5O425HL2UAWS4MF3XPTURNANCNFSM6AAAAAA2FFFP3E
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks for the information so far. Can you share which tutorial are you following and what error do you get? Do you get any error messages when you try to build?


Fedora Workstation live pre-release images do not show pre-release status (because buildstamp 'isfinal' property is hardcoded 'True')

Fedora pre-release images are meant to show some warnings of their pre-release status. They should show a red "PRE-RELEASE / TESTING" text at top right, under the release version. Like this, from the livemedia-creator-created image:

But the osbuild-built image does not show that warning:

There's some subtlety to how exactly anaconda decides whether to show the warning that I'm too tired to remember right now. I'll try and remember it tomorrow and add it as a comment. CCing @nirik for the releng angle here (I know for at least one case we have to call a tool with a special argument to mark images as pre-release or not...)

Thanks, I'll also cc @jkonecny12 on this as they might know how to set the pre-release.
I'd have expected that to live in fedora-release maybe?

well, yes, one of the paths is that it gets shown if fedora-release has a release (in RPM terms) of less than 1. But I don't remember if that's the one that applies to live images, or exactly how that works in detail. There must be something more tricky to cause it to show on the livemedia-created image but not on the osbuild-created one. I'll try and figure it tomorrow if I get time (I'm at a conference ATM).

OK, so I looked it up again. The logic happens here, where we look for a buildstamp file in a couple of places and fall back on environment variables if we can't find it.
The path that runs through the fedora-release version is here - we find the thing that provides system-release (this is an abstraction mechanism for rebuilds of Fedora, so they can have a (somethingelse)-release package that provides system-release), and set ANACONDA_ISFINAL - one of the environment vars that the logic code reads - based on it.
This is making me remember that, the other day, I noticed that Cloud images built with Kiwi didn't have fedora-release on them at all (IIRC). I think nothing actually requires system-release very hard any more, and it's entirely possible to build an image which doesn't have it. That might be what's going on here - we have no fedora-release on the osbuild-built live, so this breaks down. I'll see if I can find a manifest to check.

Yes, it's in buildstamp file created by Lorax.

Live images don't have that file, though, which is why we have that other path in liveinst.

So, yeah, the cause here is what I suspected:
liveuser@localhost-live:~$ rpm -q fedora-release
package fedora-release is not installed
liveuser@localhost-live:~$ rpm -q fedora-release-common
fedora-release-common-40-0.36.noarch
liveuser@localhost-live:~$

I'm a bit confused, though, because if I go to the task that built that image and click on x86_64-live-installer.manifest.json, the list there does include the fedora-release package. Similarly the list under metadata->anaconda-tree->org.osbuild.rpm->packages in x86_64-live-installer.log.json does include fedora-release.
Do neither of those actually indicate what packages are included in the live image? If so, is there anywhere that does tell me what packages are included in the built image? livemedia-creator does provide this information.

Oh, okay, so the problem is quite "simple": it looks like osbuild does write a buildstamp file into live images, unlike livemedia-creator, so we never fall back on the environment variable.
liveuser@localhost-live:~$ cat /.buildstamp
[Main]
product = Fedora
version = 40
isfinal = True
uuid =<PHONE_NUMBER>15.x86_64
variant = Workstation

[Compose]
osbuild = devel

we just need to make that say isfinal = False for builds before the Final RCs. However, this seems to be hardcoded into osbuild as true, here and here. I'll see if I can do anything about that...

If you want me to I can pick this up as well @AdamWill just let me know :)

well, you'd probably do it faster but I'd learn more...:P

well, I may know more about the other end of this, if it helps with the design. Pungi has the concept of whether the compose is "supported", which can be manually set but which in fact we rely (I think) on being set automatically, by this bit of magic. Basically it gets set for composes with a label like RC-x.y, which is Final candidate composes.
It then uses that property in various paths. e.g. in the buildinstall and ostree_installer phases, it passes a --isfinal arg to lorax if it's True, which causes lorax to set this value as True in the buildstamp file. On the createiso and extra_isos phases, it winds up passing a --supported-iso arg to /usr/bin/implantisomd5 (I'm not sure what that does, but it doesn't really matter).
We probably want to follow that same logic here, if possible; the osbuild phase should pass that compose.supported property to osbuild somehow - via the opts passed to koji.koji_proxy.osbuildImage I guess? - and osbuild should use that to decide what to write into the buildstamp file.

So, there's a lot of 'figuring out what sends what to what' to do if you don't already know it, but so far I think I want to add this to koji-osbuild:
diff --git a/plugins/builder/osbuild.py b/plugins/builder/osbuild.py
index db5be9f..c9c752a 100644
--- a/plugins/builder/osbuild.py
+++ b/plugins/builder/osbuild.py
@@ -112,10 +112,11 @@ class Repository:

 class ImageRequest:
-    def __init__(self, arch: str, image_type: str, repos: List):
+    def __init__(self, arch: str, image_type: str, repos: List, final: bool):
         self.architecture = arch
         self.image_type = image_type
         self.repositories = repos
+        self.final = final
         self.ostree: Optional[OSTreeOptions] = None
         self.upload_options: Optional[Dict] = None

@@ -126,7 +127,8 @@ class ImageRequest:
             "image_type": self.image_type,
             "repositories": [
                 repo.as_dict(arch) for repo in self.repositories
-            ]
+            ],
+            "final": self.final
         }
         if self.ostree:
             res["ostree"] = self.ostree.as_dict(self.architecture)

then figure out what bit of osbuild picks up those requests and ultimately get that passed through as a property of the types defined in osbuild-images pkg/image/*.go (e.g. AnacondaLiveInstaller)...

Good grief, I don't mean to be rude but the chain of image types creating other image types creating other image types is a bit of a stretch to wrap your head around here, trying to implement this...

So I got some way through the maze here but I think I'm gonna wave a white flag and ask @supakeen to do this, it's probably better for me to learn by looking at what he does. Sorry, Simon :)

It's OK; as we discussed on Matrix I'll likely initially do a similar thing to what I did for ISOLabel's and then we can figure out after how we're going to get this from koji into images through osbuild-composer's APIs.

So, to recap from discussion in https://github.com/osbuild/images/pull/522 and elsewhere, this is a hard problem to solve at the osbuild level. Let me recap just the different cases I can remember in Fedora:

We do nightly composes of the 'Fedora' compose which are true nightlies in productmd terms (they have '.n.X' in their compose ID, and no label). These should always have isfinal = False.
We have nightly composes of the 'Fedora-IoT' compose which are not true nightlies; they have no '.n.' in their compose IDs, and they have a label starting 'RC-', i.e. they look exactly like release candidates. These should sometimes be 'final' and sometimes not. This has never worked correctly (before osbuild, they always got isfinal = True).
We do candidate composes of the 'Fedora' compose for Beta and Final. For Beta, these have labels starting 'Beta-'. For Final, they have labels starting 'RC-'. The Beta candidates should be isfinal = False, the Final candidates should be isfinal = True.
You can never get a fully accurate result by logic based on the release number, as when we're building the Final RCs, the Final RCs and nightlies for the same release have the same release number but should have different 'isfinal' values.

The way this works for lorax-built installer images is that pungi has a heuristic which decides that, if the compose label starts 'RC-', 'Update-', or 'SecurityFix-', then it's a "supported" compose, and it passes --isfinal to lorax. This produces the correct behaviour for mainline Fedora composes (and I guess RHEL composes, I don't know), but not for IoT, because IoT composes always have 'RC-' labels, so they never get pre-release warnings even if they clearly are pre-releases.
I think possibly the best way to do this would be to let us override the pungi heuristic with an explicit setting in the pungi config. We already have separate pungi configs for Fedora-nightly, Fedora-beta, Fedora-final, and Fedora-IoT, with branches in git. So we could potentially make sure to always set this 'correctly' in the pungi configs (IoT could flip it whenever they feel the next candidate really "is" an RC for final). Then, provide a way for pungi to pass the info to osbuild, the way it does to lorax with --isfinal. If nothing is passed in, osbuild should probably assume it should be isfinal = False.


56 pull chicago api ridership data

The script has the functionalities to extract daily data from the Chicago Portal for subway and buses.
I additionally created functions for ingesting data into the database. Since the models are not yet pushed from #27 those are not supposed to run, but I pushed it as an example code for those ingesting data.
If I left it uncommented I failed the tests, but here is a screenshot of the tests (which I'm expecting to add a few more) passing.

Asking for a re-review here @mbjackson-capp @jimenasalinas @meganhmoore .
The code has been refactored and it's ready to push station data (Some station ridership data has been ingested). I'm facing a problem with Route data but I'll open a new issue to not delay merging this branch.


Bump Marathon to 1.6.348-e45e6e08d.

High-level description
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

MARATHON-8063
MARATHON-8086
MARATHON_EE-1791
MARATHON-8045
MARATHON-8097
MARATHON-7751

Checklist for all PRs

[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[x] Test Results: Master Build 730
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label [Ready For Review]

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot ship_it_label_status_check

@mesosphere-mergebot pr-status-checks

@mesosphere-mergebot ship_it_label_status_check


SAK-43357 allow rubrics component to save evaluations

https://jira.sakaiproject.org/browse/SAK-43357

This needs testing via pulling the change and trying it out. It's rubrics ...

Tested locally, after grading using a rubric, adding comments, and attaching files, the rubric values are persisted.

However, I found some Javascript errors while editing comments:

I think the values are saved despite the errors.

That ckeditor error exists in master already. It needs fixing, but is not a critical and should be in another jira.

Well, I tested the changes and they worked for me, thanks @adrianfish


About 4.5. Applying VL-PET Designs to Existing Methods

您好！感谢您的杰出工作！我想问一下您这个4.5节的实验细节，是保持现有方法在解码器的设计不变，在编码器添加VL-PET吗？

Hi, @123456789asdfjkl
As mentioned in section 4.5, we would like to transfer some of our VL-PET designs to state-of-the-art PET techniques.

For encoders, we introduce Granularity-controlled Mechanism to the encoder PET modules (e.g., G_large). By the way, the encoder LNs are trainable.
For decoder, to simplify the validation of lightweight PET module designs, we only retain the decoder PET modules in the cross-attention modules from their conventional designs. That is, we remove their decoder PET modules in the self-attention modules and feed-forward modules.

好的，感谢您的解答

Thanks for you issue. I think we can close this issue now:)


added specificity at 99% sensitivity

This PR isn't set up correctly. It needs to merge your new branch into master.
@rustylogic33 I can't fix this in-place. I'm closing, and I'll reopen with the correct branches.


Date clarity in the documentation

On the readme, Date is used in the examples. Date really is Timex.Date. There is a use Timex when explaining "Erlang datetime tuples", which explains how to get it in the proper scope. This can be confusing for newer elixir users (me).
It might be better to use the full names in the documentation (Timex.Date) to leave off any confusion for newer Elixir users.
Thoughts?
Thanks again bitwalker :smiley:

I feel now, after looking at the larger blocks of text, that I missed the message saying you can do the "use Timex" part.
No worries now. :smiley:


Mostly fix !visualprospectingnearspawn

Adds methods to reload spawn the first time a world is loaded, fixing the issue without recaching all chunks. For old worlds it Just Works:tm:, but for new ones it only gets fixed on the second time the world is loaded.
This is NOT ready for merging, it needs a PR in Enklume to work.

if its ready you should undraft it. :)

After careful headdesking I have managed to make a working solution. However, I'm not enough of a Git wizard to bash this branch into a working state, so I'm nuking this PR and opening a new one with a clean source.


Refactor: Make shader paths dynamic based on build type

Move to #184

Refactor: Make shader paths dynamic based on build type

[x] Added conditional compilation in CMakeLists.txt to define PTSD_DIR macro
[x] Modified Image.cpp and Text.cpp to use PTSD_DIR macro for finding paths
[ ] Improves flexibility and adaptability of the code across different build environments

Originally posted by @Dragonfly911117 in https://github.com/ntut-open-source-club/practical-tools-for-simple-design/commit/a2eb16474f3083448b40db7233b78f50ea2e8eea

Move to #184


Update demo app

Instead of just printing values, the demo app should feature:

a play/pause button (this would respond to changes in rate)
a progress indicator (responding to changes in the playhead progress)

Another consideration: Apple doesn't recommend subclassing AVPlayerViewController which I did in the demo app. I think that also should be changed to follow best practices.

Good point on the sub-classing. I see the recommendation from Apple but find no specific alternative for a custom UI playing in full-screen mode.
After re-watching the best-practices session (https://developer.apple.com/videos/play/wwdc2019/503/) I am leaning to think the kind of presentation we want to use in this case is "inline" even though our "inline" presentation takes the whole screen. This makes use of the containment API.
I will have to adjust the code on PR #15


Incompatibility with HA 0.113.0

After updating to v0.113.0, the background of the card is showing for some reason.

content: |
  # Bedroom
style:
  .: |
    ha-card {
      --paper-card-background-color: none !important;
      box-shadow: none !important;
    }
  ha-markdown:
    $: |
      h1 {
        font-size: 20px;
        font-weight: bold;
        font-family: Helvetica;
        letter-spacing: '-0.01em';
      }
type: markdown

No offense but did you even take time to read the forum or the repository before posting?
@N-l1 fixed it already yesterday and you can also find your fix on the forum.

You're right. I had no idea the code changed.


Update document and examples for v0.3.5

What this PR does / why we need it?
Update document and examples for v0.3.5.
Any background context you want to provide?
What are the relevant tickets?

Fixes #
Checklist

[x] Added relevant tests or not required
[x] Didn't break anything

From now on, we will perform both yorkie version bump and example/documentation update in single PR.
PR closed.


[20.09] matrix-appservice-discord: patch to use new API endpoint

Motivation for this change
Discord's API is migrating from discordapp.com to discord.com
and will only be accessible through the latter domain after 2020-11-07.
This patch allows the current stable release of the bridge program to
continue working beyond that date.
See https://github.com/Half-Shot/matrix-appservice-discord/issues/611
(cherry picked from commit afc0337c22afc726452f01b7ac86d140006c652e)
Backport of https://github.com/NixOS/nixpkgs/pull/99298.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Obsoleted by https://github.com/NixOS/nixpkgs/pull/102973.


Reorganize

A lot of reorganization because the project structure was getting confusing

I'll just merge this because I have tested it and it is working better than the current master. Just figured you guys should know how things are being restructured


Could not build Objective-C module 'Toaster'

I don't want just for this library change something in my configuration

How did you install Toaster?

Please show your Podfile, Podfile.lock and eventually a sample project demonstrating the problem. Also make sure you have use_frameworks! or explain why you don't.

Well, please see my previous message and we'll reopen the issue if needed.

Hi
I can't install Toaster

help my please

I edit
open class ToastView: UIView {
add
case .Mac:
in 3 places
@objc open dynamic var bottomOffsetPortrait: CGFloat = {
@objc open dynamic var bottomOffsetLandscape: CGFloat = {
switch UIDevice.current.userInterfaceIdiom {


invalid json response at https://us-api.coolkit.cc:8080...

hello, i keep getting the following error when my node app tries to login.
FetchError: invalid json response body at https://us-api.coolkit.cc:8080/api/user/login reason: Unexpected token < in JSON at position 0
This seems to have started either yesterday or today. every now and then (randomly) it will be able to login and everything starts working fine then after 'X' amount of time my app no longer as the ability to refresh my plug data. So I then close my node app and re open it i get the above error message.
I was wondering if anyone knows what is going on here?
thanks for the help
-Nick

Hello,
I also get authentication issues also since yesterday.

Check the response before trying to parse

Same issue.
I had same issue. Debugging leaded me to conclusion that server in selected region ("us" by default) is on maintains. Changing target region (to "eu" in my case) resolved the issue.

I faced this issue yesterday.
I can't fix it even I change the region.
Does any way to fix it?
Or it's the server issue, I just need to wait?


Fixes 'm365 spo sitedesign list' command. Closes #2006

Closes #2006
Removed request digest from m365 spo sitedesign list implementation.

Thank you for the quick turnaround 👏

Merged manually. Thank you! 👏


MySql

I wonder if the MySql queries are okay. Will this cause any problems later on ?

Will be using Doctrine for database connection and queries.


Abstract away our dependency on Foundation for JSON encoding/decoding

This PR moves all our uses of JSONEncoder and JSONDecoder to a new namespace enum, JSON, and provides interfaces that don't use Foundation.
This change does not remove our dependency on Foundation's JSON functionality, but it should allow us to replace that functionality in the future with JSON encoding/decoding logic from another library (or home-grown, if necessary.)
Checklist:

[ ] Code and documentation should follow the style of the Style Guide.
[ ] If public symbols are renamed or modified, DocC references should be updated.

@swift-ci please test

@swift-ci please test Linux

Linux failure is because we've removed some Foundation imports (which reexport Glibc, I guess.) Explicitly adding the necessary headers to Includes.h.

@swift-ci please test

@swift-ci please test

@swift-ci please test


Support for eslint 0.24.0

Since eslint is updated rather frequently, it might be a good idea to allow the user to provide their own version of eslint when calling this plugin.  At the very least, it would enable users to update eslint without waiting for this plugin to update as well
I imagine this would involve replacing https://github.com/adametry/gulp-eslint/blob/master/index.js#L15 with something like
var Engine = options.CLIEngine || CLIEngine;
var linter = new Engine(options);

or any number of alternatives.  It also looks like util.resolveFormatter would also need to change.
I'd be happy to work on this if you think it's a good idea, @adametry.

@cmaher I think this is a great idea.

+1

btw, eslint 0.24.0 is the last release before 1.0.0.

A config-defined eslint creates more problems than it solves. Besides side-stepping the tested eslint version, it makes changes to how gulp-eslint uses eslint a potential breaking API change. It's also worth noting that substantial changes to internal eslint modules change (e.g., CLIEngine) are not uncommon, and I expect 1.0.0 will have several breaking changes.
On the bright side<EMAIL_ADDRESS>– using<EMAIL_ADDRESS>– is now available.

Thanks!


HTML escape

This plugin should escape HTML inside the JSON object. Otherwise that HTML is rendered by browser.

Probably the issue is that JSON string contains :. Honestly I don't understand why you exclude : in string regex /("[^:].*")/.

To reproduce try this JSON: "colon": "\: html goes <here> \"bar".

Hi,
Thanks for your feedback. I've updated the package to 1.0.4. I hope this solves your problem.


Return the Matching Selector of an Alert

This enables one to trace back, why an alert is part of the response.

Shall I provide more information or context to the PR?


add a example

The Logic Analyzer did not have an example.
The sketch has explanation about the appropriate decoders in PulseView.

Thank you!


Please add a license

https://help.github.com/articles/open-source-licensing/#what-happens-if-i-dont-choose-a-license

In short, without license no-one can use your code.

I also want to use this library. So it's great if this provides license

Thanks, added LICENSE of MIT.


fix(port): retry a fixed number of times to find the port

Summary of Changes
retry a fixed number of times to find the port exposed by the docker images as docker-py is flaky and raise an exception if the app is routable but no port is specified.
Issue(s) that this PR Closes
closes #677
Associated End To End Test PR(s)
Associated Documentation PR(s)
Associated Design Document(s)
Testing Instructions
Please provide a detailed list for how to test the changes in this PR.

Create a Deis Cluster
Register an app and it should properly
Deploy an app with no port exposed in the docker file  and no proc file, you should get an error.

Pull Request Hygiene TODOs
Please make sure the below checklist is complete.

[x] Your pull request is concise and to the point (make another PR for refactoring nearby code)
[x] Your commits are squashed into logical units of work
[x] Your commits follow the commit style guidelines

:cherry_blossom: Thank you! :cherry_blossom:

Are we absolutely sure it's due to docker-py flakiness? From the logs in the original issue I don't see any indication that was the error. Any way we can add tests to confirm this?

yes it is...we don't see anything in the logs because we are hiding the exception stack trace by handling it and ignoring it

So everyone understands, after this patch, if you have a non-buildpack app and we can't divine the port from your image, the whole deploy fails

Except user can still deploy it as a non-routable app by specifying the a different proc type than web or cmd in the procfile

[Current coverage][cc-pull] is 86.39%

Merging [#689][cc-pull] into [master][cc-base-branch] will increase coverage by +1.03%

@@             master       #689   diff @@
==========================================
  Files            29         29
  Lines          2547       2521    -26
  Methods           0          0
  Messages          0          0
  Branches        400        394     -6
==========================================
+ Hits           2174       2178     +4
+ Misses          261        234    -27
+ Partials        112        109     -3

Powered by Codecov. Last updated by [0a541e6...fed53db][cc-compare]
[cc-base-branch]: https://codecov.io/gh/deis/controller/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/deis/controller/compare/0a541e63ddf4efbea330466b3c0e80041a5b4fd8...fed53dbe6c07b45d56df83a266fc1281ad18a8a9
[cc-pull]: https://codecov.io/gh/deis/controller/pull/689?src=pr

Are all the things you mocked out actually docker image apps? I just want to make sure the mocking isn't being too aggressive

yes...they all fail if i don't mock and all of them do a request to the build hook with the 'image': 'autotest/example'  as the contents in the body

Is it possible to pass through a more useful error message? I removed PORT from the example-dockerfile-python Dockerfile and here's what git push had to say:
...
{"status":"Image already exists","progressDetail":{},"id":"1c0e9a1c2c6a"}
{"status":"Image already exists","progressDetail":{},"id":"5b6d72625828"}
remote: ":"git-34fb087e: digest: sha256:416f733b3e1ee451142e5871633ef8cb820e7bb7c4ee712e2dc0f4af9f5a04bb size: 7565"}

size of streamed logs 3425
Waiting for the deis/dockerbuild-velvet-umbrella-34fb087e-e5361ffd pod to end. Checking every 100ms for 15m0s
Done
Checking for builder pod exit code
Done
Build complete.
Controller request POST /v2/hooks/build
{"sha":"34fb087e","receive_user":"matt","receive_repo":"velvet-umbrella","image":"velvet-umbrella","procfile":{},"dockerfile":"true"}

remote: 2016/05/05 19:52:20 Error running git receive hook [publishing release (Deis controller endpoint http://<IP_ADDRESS>:80/v2/hooks/build: expected status code 200, got 400)]
Launching App...
To ssh://git@deis.<IP_ADDRESS>.nip.io:2222/velvet-umbrella.git
 ! [remote rejected] hide-port -> hide-port (pre-receive hook declined)
error: failed to push some refs to 'ssh://git@deis.<IP_ADDRESS>.nip.io:2222/velvet-umbrella.git'

@mboersma that is something we need to fix in the builder because builder doesn't use the error message sent from the controller, currently the error message i am raising is "Expose a port or make the app non routable by changing the process type".
I am creating an issue in the builder to fix this https://github.com/deis/builder/issues/323


Regenerate CRDs to match current definitions

Using operator-sdk version v0.8.2
Regenerating openapi and k8s via the cli to match what's currently in the definitions.

@rogbas @rafael-azevedo can you think of any reasons the new required fields could cause issues? I don't think so since OCM will pass these along. What about existing CR's?


Seed training site with placeholder data on daily wipe

Would like to seed training with a base set of placeholder data on the daily reset. It’s currently completely wiped (except for users for some reason), which I believe makes it hard to use the training site as intended. Some data that may be seeded:
3-4 facilities
30 patients (enough to trigger pagination)
20 results
5-10 users

Related to #3443

Seed data should also include device data, as referenced in #3443, and should resemble prod as closely as possible.

closed in favor of #3443


Don't spawn a thread for every config watcher notifications

Fixes an issue whereby changes to settings won't be notified introduced by this "fix"
https://github.com/lapce/lapce/pull/2023
This should still mean that no bad feedback loop happen because read-only operations wouldn't
trigger this notify.
Sorry about that!

[ ] Added an entry to CHANGELOG.md if this change could be valuable to users

strace of a crash. Try to find something weird here...
strace.txt

Codecov Report

Merging #2034 (823a55e) into master (c52a76d) will decrease coverage by 0.07%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master   #2034      +/-   ##
=========================================
- Coverage    9.00%   8.93%   -0.07%
=========================================
  Files         135     135
  Lines       58820   59251     +431
=========================================
- Hits         5296    5295       -1
- Misses      53524   53956     +432

Impacted Files
Coverage Δ

lapce-data/src/config.rs
0.00% <0.00%> (ø)

lapce-data/src/keypress/keypress.rs
73.02% <0.00%> (-0.21%)
:arrow_down:

lapce-ui/src/explorer.rs
0.00% <0.00%> (ø)

lapce-ui/src/settings.rs
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I think the crash isn't caused by a feedback loop. It is the OS that is interrupting the process since its spawning too many threads?  And these threads are long-lived too because this if delay_handler.lock().take().is_some() { will hold on to the lock until the scope ends and until it gets it.

I've reimplemented this yet again using blocking channels to manage the job. It should now at most have 2 reload config job when there are huge changes when installing/uninstalling plugins for example.