Reconciliation should handle updates from unknown tasks

There's a chance that task reconciliation could return a status update for a task that the scheduler (and its persistentState) either does not know about, or no longer considers to be running. We should ensure that these untracked tasks are not silently ignored by the scheduler, since we may want to shut down an unknown task, or at least note it as currently running.

This should be fixed with the latest PR for reconciliation that was merged. Feel free to reopen if there are remaining issues (cc @gabrielhartmann )

@gabrielhartmann Did you write a test that proves that the above condition (implicit reconciliation returns status for an unknown/terminated task) is handled correctly? We wouldn't want unknown tasks to continue running without the scheduler ever realizing it.
Reopening until I see proof of a test that validates this use case. Ideally a commit link, but I would also accept confirmation that it was tested/considered somehow.

@adam-mesos:  Here's the line where the case of a notification for an unknown Task is noted during reconciliation.  Since we now do both Implicit and Explicit reconciliation we receive duplicate status updates for all Tasks.  This is handled in the sense that duplicate or unexpected Task updates are ignored as part of reconciliation.  They are not ignored from the state machine's perspective.  The LiveState still tracks all status updates and will make appropriate state transition decisions if the timing is reasonable.
However, there is definitely a race condition possibility which could leave an orphaned Task.  For example, if a Journal Node status update arrives for an unknown (not duplicate) Journal Task after the Journal Node creation phase state of the state machine has been exited, then a warning will be logged, but the extra Task won't be killed.
I'm not sure where this leaves this issue.  You write, "We should ensure that these untracked tasks are not silently ignored by the scheduler, since we may want to shut down an unknown task, or at least note it as currently running."  We certainly note all cases where an unknown task is encountered, however I think the better course of action would be to correct the state of the system such that it agrees with the desired state.  That is, we should handle unexpected status updates either by ignoring duplicates when appropriate or killing tasks when they exceed the desired task/node count.

Okay, I'm glad we're at least logging it, but what do we really want to do if we see an update for an unknown running task?


[Errno 13] Permission denied: '.' when call"az iot du update init v5"

Describe the bug
when send "az iot du update init v5" command, it will show permission denied
Related command
az iot du update init v5
Errors
[Errno 13] Permission denied: '.'
Issue script & Debug output
Command group 'iot du update init' is in preview and under development. Reference and support levels: https://aka.ms/CLI_refstatus
Skipping processing of 'company.', input format is key=value | key='value value'.
No value assigned to key 'manufacturer', input format is key=value | key='value value'.
Skipping processing of '-230223.signed', input format is key=value | key='value value'.
No value assigned to key 'path', input format is key=value | key='value value'.
Skipping processing of '-230223.signed', input format is key=value | key='value value'.
No value assigned to key 'path', input format is key=value | key='value value'.
The command failed with an unexpected error. Here is the traceback:
[Errno 13] Permission denied: '.'
Traceback (most recent call last):
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 233, in invoke
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 664, in execute
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 731, in _run_jobs_serially
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 701, in _run_job
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 334, in call
File "D:\a_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
File "C:\Users.azure\cliextensions\azure-iot\azext_iot\deviceupdate\commands_update.py", line 387, in manifest_init_v5
assembled_file_metadata = DeviceUpdateDataManager.calculate_file_metadata(assembled_file["path"])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users.azure\cliextensions\azure-iot\azext_iot\deviceupdate\providers\base.py", line 259, in calculate_file_metadata
with io.open(file_pure_path.as_posix(), "rb") as file_io:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
PermissionError: [Errno 13] Permission denied: '.'
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues
Expected behavior
should be create manifest file
Environment Summary
azure-cli                         2.60.0
core                              2.60.0
telemetry                          1.1.0
Extensions:
azure-iot                         0.23.1
Dependencies:
msal                              1.28.0
azure-mgmt-resource             23.1.0b2
Additional context
No response

Thank you for opening this issue, we will look into it.

Hi @SophiaZhang-eaton , how are you using the command? Only seeing the error here, not the form of input that caused it.
I am able to repro the issue using a directory for --file path= for example "--file path=." -- The command path is expecting a file path value, not a directory path.


Launchbadge main

It was a mistake on my part. Please disregard it.
On Wed, Dec 7, 2022, 6:50 PM CosmicHorror @.***> wrote:

What's this PR for?
dotenv is unmaintained with dotenvy being the desired replacement, and
sqlx-core is on the 2021 edition which automatically brings TryFrom into
scope
—
Reply to this email directly, view it on GitHub
https://github.com/launchbadge/sqlx/pull/2239#issuecomment-1341764411,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQTIR7WZXLGR5MUAQHLGLILWMEPDXANCNFSM6AAAAAASXJQIJQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Find and replace styling

Why
We want to improve style of find and replace widget. See mockup from #862

Looking good basically. Working on some improvements. E.g. the F+R panel should use flex not absolute positioning, otherwise content slips underneath it. Need to change the HTML structure for doing that. Will push a rebased version soon.

Yeah I thought, flex would be a better approach here.

Think we are good to merge, regarding styling.

Looks great! thx


Failing Tests in Gigasecond

I am not sure if I am missing something, but I think I have the problem solved correctly and the test is failing. Looking at what it is expecting back, the test seems it is expecting a date that it is much further in the future than it should be.
This is the test that is failing, it is passing in a date in 2011, and expecting a date in 2043.
12  def test_2011_04_25
13    gs = Gigasecond.from(Time.utc(2011, 4, 25, 0, 0, 0))
14    assert_equal Time.utc(2043, 1, 1, 1, 46, 40), gs
15  end

Here is the code that I have written:
def self.from(time_start)
    time_start += (10**9)/(60*60*24)
end

While I know that this can be written better, in the spirit of the red green refactor, I was trying to just get it to pass at this point, which it should.
This is the error that I am getting from minitest:
Finished in 0.002493s, 2807.3091 runs/s, 802.0883 assertions/s.

  1) Failure:
GigasecondTest#test_2011_04_25 [gigasecond_test.rb:14]:
Expected: 2043-01-01 01:46:40 UTC
  Actual: 2011-04-25 03:12:54 UTC

7 runs, 2 assertions, 1 failures, 0 errors, 5 skips

A gigasecond is 31.7 or so years, so 2043 seems correct.
When you have a date object in Ruby and add 1, you add a day, but when you have a Time object and add one you add a second. Could that be what's confusing things?

thanks @kytrinyx I will look into this further then.


position: relative on body tag breaks DND and Context Menu

I've noticed that the calculated position of the context menu and the indicators for drag and drop aren't correct when the body tag has position: relative.
I'm going to fix this in my implementations, but I wondered if there's a way to cope with this situation in jstree?

Nevermind, it was more related to collapsing margins from the first element in the body. Seems to mess up offset().
http://stackoverflow.com/questions/13384660/jquery-offset-broken-by-body-positionrelative-combined-with-element-margin


[Proposal] Design proposal to extend import-account command to import by seed-phrase or private-key

Currently, there are basically four ways to "add accounts":

Import it from the web wallet
Create a named account by sending a transaction signed by some existing account
Create a new implicit account (and later fund it from somewhere else)
Manually add a new file to ~/.near-credentials/*/ keychain

While it is not required to add an account before signing a transaction since we allow signing transactions with a plain-text private key (and I will create a new proposal to enable signing by providing a seed phrase), it is far more convenient and secure if one uses macOS keychain to access the account than pasting seed-phrase or private key over and over again (not to mention that it is not convenient to do).
All these import-account commands will need to have the account id, but if we want to streamline the process, we want to get it from the user at the very end or (better) derive it, e.g. web-wallet can redirect to localhost:xxxx (as it is implemented in near-cli-js) and when we know the seed-phrase/private-key we can derive public-key and search the blockchain for the corresponding account ids (as web wallet does). The downside of this is that it requires to have access to the Internet and specifically, to some additional Wallet APIs.

After an offline discussion with @FroVolod, we decided that it makes total sense to add these sub-commands, so I merged this proposal


Regex: add Replace methods which take a callback

The System.Text.RegularExpressions.Regex has Replace methods which take a MatchEvaluator object. This allows the replacement text to be computed from the matched text.
This PR adds a similar feature to the OpenXmlRegex object. The callback takes an XElement and a Match, and returns the replacement text or null to indicate no replacement. As before, returning an empty string will delete the matched text.
The existing Replace methods in OpenXmlRegex  remain the same. This was straightforward to combine the Func<XElement, Match, bool> callback and fixed replacement text into a new callback that gets the boolean value and returns the text, or null.
As far as I can see there are no unit tests defined for this part of the library. I ran the example code and checked it functioned as expected.
I would appreciate your comments on whether this is a useful addition to the library.

@EricWhiteDev The user has deleted his repository. Please close this PR.

Source repo has been deleted.  Closing.


Cannot propose transaction in browser environment

Description
I am running into an issue where I cannot propose a transaction through the safe-core-sdk and safe-service-client when running this code in our react application in the browser.
I am constructing a multisig transaction with the relevant libraries following this guide
When it comes to calling proposeTransaction my application will throw with error createKeccakHash is not a function.
Environment

Safe Core SDK version: ^3.2.0
Safe contract version: ?
Environment:

browser

Steps to reproduce
Following the stack trace, this comes from the following:

SafeServiceClient.proposeTransaction calls a method getEip3770Address which calls validateEip3770Address from the safe-core-sdk-utils which then calls validateEthereumAddress which uses the package web3-utils isAddress method which has a dependency on node cryptography libraries which do not exist in the browser.

To get around this problem temporarily, I have patched the library to bypass the validation here and the proposal is able to go through.
I have not found a way to polyfil this and get this to work in the browser without patching. I am using webpack as my bundler.
Expected result
To be able to propose and execute a transaction using safe-core-sdk libraries in react/browser code.

@germartinez any update on this? The isAddress function is not working on browser, which is throwing errors on my site when I run it on production (sice I can not patch it).

The function was migrated to use ethers instead of web3 in the latest @safe-global/protocol-kit v4.0.0.


what is new in 0.8.0?

0.8.0 released on NPM but don't see any changes or update to changelog.

CHANGELOG has been updated. The only change is that this package no longer supports @angular/http since it's deprecated and is being removed in the next major release of Angular. Please use @angular/common/http if you aren't already.


Allow '-' In Classic Prompt

Description
The regex didn't allow hyphens in the classic prompt.  See discussion in https://github.com/scrapli/nornir_scrapli/discussions/73
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
I used this on my physical router.
Checklist:

[ ] My code follows the style guidelines of this project (no GitHub actions compalints! run make lint before
committing!)
[ ] I have commented my code, pydocstyle and darglint are happy, docstrings are in google docstring format, and all
docstrings include a summary, args, returns and raises fields (even if N/A)
[ ] I have added tests that prove my fix is effective or that my feature works, if adding "functional" tests please
note if there are any considerations for the vrnetlab setup
[ ] New and existing unit tests pass locally with my changes

LGTM - thanks for following up with this @MrPaulAR !

Oh man, I didn't realize PR was pointed to master -- looks like @juliencorsini already took care of this prompt update in #75 so I think we should be good to go.
Does develop branch look right/good to you @MrPaulAR ?
Sorry for not realizing that earlier, didn't notice till I updated the pr to point to develop.
Carl

Develop looks good.  Closing this.


Main add view count and full sub count oalca

This PR addresses 2 things separated by commit:
1.- New settings to add full subscriber count and new setting to show subscriber count

2.- Fixes old CSS styles in order to render in new versions of MM:

Before:

After

Hey @oscarolar, nice changes, I just merged another PR and changed the spacing a bit. Can you doublecheck if the changes still work, but it looks good for me. I will merge it afterwards

@choffmann looks good on my end


Resources .Content not available in shortcodes

If I have a single .md fie as a resource to a page and I place this in a template:
{{ with .Resources }}
	{{ range . }}

{{ .Title }}

{{ .Content }}

	{{ end }}
{{ end }}

Both the title  and the content of the resource renders
However, if I place this in a shortcode:
{{ with $.Page.Resources }}
	{{ range . }}

{{ .Title }}

{{ .Content }}

	{{ end }}
{{ end }}

Then only the title will render, the content will not.

This is a known limitation and there are other similar issue(s). You can ask further questions at https://discourse.gohugo.io/

It would have been nice if this was in the docs, not sure if I can contribute to that in some way.
I found a workaround so will leave it here in case it is useful to somebody else
shortcode.html
{{ partial "partial.html" $.Page }}

partial.html
{{ with $.Page.Resources }}
	{{ range . }}

{{ .Title }}

{{ .Content }}

	{{ end }}
{{ end }}


Update ERC-7528: ERC 7528 Clarifications

When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md
We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

The PR edits only existing draft PRs.
The build passes.
Your GitHub username or email address is listed in the 'author' header of all affected PRs, inside .
If matching on email address, the email address is the one publicly listed on your GitHub profile.

@eip-review-bot rerun

@eth-bot rerun


Ability to generate a random name with some prefix for the created migration

By default user is prompted to enter a migration name if it's not specified by the parameter. If it is difficult to come up with unique names for new migrations, or we have plenty of them, it's more convenient to generate a random name and offer it by default. If the option "default_migration_prefix" is specified in the config, a name will be offered starting with this line and ending with a random hash. Otherwise, current behavior will not change.

Please resolve this conflict before I can merge it. Thanks!

Thank you very much!

Thank you, @odan!


chore: enable full flake8 default rules list

This PR enables the full set of flake8 default rules by using extend-select and extend-ignore instead of select and ignore.
In addition, this PR makes the following notable changes:

Simplifies the Github action to just run flake8 . so it will use the same settings as the flake8 on local
Ignores the geom_median dir, since that's its own project, and its unclear if that should be linted by this
ignores E402 in dashboard_runner.py and generate_dashboards.py due to those files having sys.path.append() calls above imports. It's not clear to me why those calls are there, so I just left a TODO and ignored the error
removes # noqa statements that didn't seem necessary
adds r to regex strings, which I think was probably the intention anyway

closes #14

@chanind Thanks so much!


After the Bloxtrap Installation my Bloxtrap menu launcher just dissapeared and I don't know what to do, I tried different methods like searching it up on my Windows searchbar neither my file folders there was non of my Bloxtrap menu launcher to be found.

Pls can someone or the Creator of Bloxtrap fix this issue?

Also I only have the Game launcher after the download but never got the Menu launcher back,

I tried other options in order to delete the launcher fully with control panel after downloading It again the Menu launcher came back once but never came back after I Install it.

If you look through your application list, is it not there under the folder?

Issue has been awaiting clarification for over a month, closing as stale. If you have anything further to add, just respond and I'll reopen.


adds api support for enabling/disabling controls and rules

As discussed long ago in the Issues tracker. I've added old-style api stubs as well for this functionality.

@LRossman thanks for the comments. I agree with using the existing int/bool conventions - makes everything a bit clearer. I'll clean this up.

Thanks @samhatchett !
We need to add the four new functions to the different header files (pas\vb\def), right? I can probably do that next week if nobody will do that before :)

@eladsal sure, sounds good. I'm not as familiar with the pas/vb/def particulars so am happy to leave that in your capable hands. ok to merge this PR then, or would you prefer to wait on those additions?

@samhatchett yes I'll take care of it. Maybe just update ReleaseNotes2_3.md before you merge.


Temporarily disable bitcoin buy button on iOS

Need to disable buy bitcoin button in wallet and vault details screen.
https://www.figma.com/design/m5eMFTKD391AiJAXaEzgxB/Bitcoin-Keeper?node-id=5009-5093&t=8GxaT1R7pl9O2xoa-1

Verified this issue on ios app v 1.2.18(412)


add tag deletion event

Whenever a tag is deleted an event is sent out via the regular notification channels. The event will have repo and tag but digest will be empty.

LGTM
This approach looks fine to me, maybe @stevvooe or @aaronlehmann can give it a quick look to see if the event looks consistent to them.

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "tag-deleted-event" git@github.com:manishtomar/distribution.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354429072
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

LGTM.
@manishtomar why is Circle failing?

@dmp42 I fixed the circle failure but I don't know why Jenkins is failing with integration tests. The logs are not very clear. If you don't mind can you have a quick glance and let me know whats wrong? If not, I can try digging into it.

LGTM


Disable resource quota limitation

IMO, we can delete them entirely. And create them as needed in JenkinsConfig controller.

I prefer keeping it. And improve it later.
By the way, it's not easy to calculate the quota in the Jenkins Config Controller. At least, I didn't have a good solution for it.


remove duplicate error classes

Closes #701
Very minor changes removing the custom NotImplementError.
@s3alfisc I've noticed that pyfixest.errors.__init__ includes an undefined MultiEstNotSupportedError in __all__ which I have also removed. Please let me know if instead it should be defined as an additional error.

Test errors only due to #704 . Looks good to me and can be merged. Thanks!


change emoji names to actual emoji for better readability

Description
Most of the time, I prefer to use FontBakery by generating markdown reports via the --ghmarkdown option.
However, these reports have a limitation: the emoji names aren't displayed as emoji outside of GitHub (or at least not in my code editor, VSCode). This doesn't matter that much (either way) in the "code" view, but in a Markdown preview mode, this is a significant disadvantage – I have to somehow get the markdown onto GitHub (either by push or by copy/paste) in order to comfortably use the output. Sometimes, this nudges me to make commits that might be smaller than is truly useful.
So, this is a small change. Instead of placing emoji names into the gfmarkdown report, it inserts actual emoji. So, instead of :fail:, :pass:, etc, it shows 🔥, 🍞, etc.
VS Code Markdown Preview, before and after this change:

Code view, in various editors
The emojis also display in the code view. This isn't a view I use as much, but it's useful to know that it still works.
VS Code, before and after:

This also provides improvements outside of VS Code.
vi command line editor, before and after:

Sublime, before and after:

Atom, before and after:

On Windows
I wanted to check that the emoji also work in VS Code in Windows. This is definitely not a comprehensive test, but it helps show that this has no super-obvious drawbacks:

If anyone knows of a "gotcha" to this approach, I can accept the names as-is. If support for emoji in text editors is as broad as I think, however, I think this is a small but definite improvement.
To Do
I'm happy to add this to the Changelog if there aren't issues with this that I've missed. Please let me know if this seems doable, and I'll write a short description of the update.

[ ] update CHANGELOG.md
[ ] wait for checks to pass (except github/pages, which is stuck)
[ ] request a review

yeah, I like it. It seems to also work well in both text editors I use (pluma and vim).
But I will be picky in here... I like source-code to look like source :-) And ironically GitHub fails to render this properly:

So I suggest you change the implementation to this, which still does not look perfect in this rendered comment, but looks good on text-editors and seems more bullet-proof for people reading the source in the future (please copy and paste this exact block of code - the emojis are there :-D):
  def emoticon(self, name):
    return {
      'ERROR': "\U0001F494", # 💔	:broken_heart:
      'FAIL':  "\U0001F525", # 🔥	:fire:
      'WARN':  "\U000026A0", # ⚠	:warning:
      'INFO':  "\U00002139", # ℹ 	:information_source:
      'SKIP':  "\U0001F4A4", # 💤	:zzz:
      'PASS':  "\U0001F35E", # 🍞	:bread:
    }[name]

Also, please add the changelog entry so that we can merge. Thanks, @thundernixon

Thanks for such quick & helpful feedback, @felipesanches!
Glad you caught that issue. Out of curiosity, what OS/browser are you on?
I've updated the as requested, and it seems to work just as expected on my end. I've also drafted a line for the changelog, but feel free to adjust it as you see fit.

The branch as a conflict on the CHANGELOG.md file. Should be simple to merge both entries.
@thundernixon are you familiar with git fetch and rebase as well as fixing code conflicts?
If you prefer I can fix this conflict myself.

Glad you caught that issue. Out of curiosity, what OS/browser are you on?

That's an unbranded fork of Firefox.

Thanks for letting me take a crack at it, @felipesanches!
I've merged in the latest updates to master, and resolved the conflict in the changelog. I think I did things correctly, but please let me know if it looks like I made any unintentional changes or if there's anything I might improve further.

oh no! You'll have to do it once more :-P
Sorry...

Let me fix this for you. Sorry.

Thanks for merging that on your end, this time. :) It'll be good to have these emojis!

:-)


Switch to C++20 compiler

Description:

Currently the code requires a C++17 compiler. As of this writing, Linux packages default to gcc version 9 and clang version 10, both of which support C++17. However, that means clang is 4 versions behind, and gcc is 3 versions behind, so neither of them support C++20. I've avoided upgrading the code base to C++20 because the standard linux distributions are so far behind.
The code base makes extensive use of ttLib::cstr which is a class derived from std::string. The cstr class provides methods such as contains() and is_sameprefix() both of which have equivalents in C++20. Using the ttLib derivative classes makes perfect sense in our private repositories -- but in a public repository like this one, it lessens the usability of the code. By that, I mean that a reader can't just copy and paste a snippet of the code to another project because they would also have to pick up the ttLib code as well.
If we upgrade to C++20, we should be able to reduce the use of the ttLib derivative classes, using the std versions instead. There are also several other additions to C++20 that would increase the readability of the code base, and in some cases we might be able to reduce memory footprint due to the new constains() working on all associative containers.
The main downside is that building on Linux is going to require the user to get a newer tools package rather than the standard package. Ironically, I can't build on Linux right now anyway because gcc 9 doesn't like the ctor in ttLib::cstr. If I upgraded the code to C++20, we might not need ttLib::cstr, but then I can't build anyway because both gcc and clang are too old in the standard tools packages.
The C++20 Linux problems will go away 3-5 years from now. Switching now means fewer people can clone the project and build it in the short term, but it makes the code more readable and more maintainable by someone not familiar with the ttLib classes.

Actually, contains() for std::string and std::string_view didn't make it into C++20, but is instead in C++23. Also, ttLib::cstr is the best way to get a class that has many of the same methods as wxString but uses utf8 on Windows instead of utf16, making it considerably more efficient. In addition, our version of contains allows for a case-insensitive comparison, which is not the case for the C++23 version.
Bottom line: using std::string_view instead of ttLib::cview is preferable. ttLib::sview is the next best choice for passing around string pointers. ttLib::cstr is arguably still better than std::string since it acts a lot like wxString (such as appending use << operator).

This is essentially done, at least as far as a full-blown refactor pass.


The rename ${notename} variable for attachments name doesn't work

Yes, ${notename} does not exist among the options below. Check your screenshot. You can only choose ${original} which stands for the original name of the attachment. I realize that the explanation was not very clear. I will change the info description in the next version of the plugin (though the functionality won't change).
${original} for the original name of the imported attachment

${notename} is used to specify the name of the attachment folder. I will change the description there to:
${notename} for the name of the MarkDown note to which the attachment files are imported

This is how the description will appear in the next release:

I mean I wish I had this variable，I apologize for not being more explicit.

Ah I see. Ok. Can you be patient until the weekend? I will add it and release the new version.

That's certainly not a problem. Thank you.

It is recommended to rename other imported attachments as well, and to support the use of this variable, which will enable easy identification of the attachment directory and all attached files by  notename，
I say this because I found that after dragging in other file attachments, the attachments are saved with their original names，

I am not sure I understand your last comment. Possibly what you have in mind is using a single folder for importing attachments.
What I instead prefer is to use a subfolder carrying the same name of the note + (attachments):

and then I opt to hide those subfolders from the file navigator to keep a clean look:

I apologize for the confusion, as importing other attachments when there was only ${original} resulted in the original name, and I mistakenly thought that there was no rename step when importing other attachments, only to find out that there already was!

You can close this feedback now after you receive my apology

Ah I see. Ok. Can you be patient until the weekend? I will add it and release the new version.

this variable  ${notename}   .... gaga

What do you mean?

I added the requested feature. Please have a look at the new release: https://github.com/alberti42/obsidian-import-attachments-plus/releases/tag/1.5.11

I did not receive a feedback. Because the feature is implemented, I will close the issue. Feel free to reopen it if I missed something.


Added support for calendarClassName option

This is to be able to style some calendars differently. If we pass calendarClassName="my-class" the calendar container div will have class="react-datepicker my-class" instead of usual class="react-datepicker" - which is useful because that div is rendered as a direct body child.
Example is included:

Current coverage is 91.74% (diff: 100%)

Merging #715 into master will not change coverage

@@             master       #715   diff @@
==========================================
  Files            13         13
  Lines           618        618
  Methods          17         17
  Messages          0          0
  Branches        118        118
==========================================
  Hits            567        567
  Misses           13         13
  Partials         38         38

Powered by Codecov. Last update 418b8d3...4c5ee39

The build fails on some lint errors:
/home/travis/build/Hacker0x01/react-datepicker/docs-site/src/examples/custom_calendar_class_name.jsx
   7:3   error  Bad line breaking before and after ','                    comma-style
  33:11  error  Expected indentation of 12 space characters but found 10  react/jsx-indent-props
  34:11  error  Expected indentation of 12 space characters but found 10  react/jsx-indent-props
  35:11  error  Expected indentation of 12 space characters but found 10  react/jsx-indent-props

Can you please fix that, so I don't have to merge in a failing build?

This PR introduced the bug described in https://github.com/Hacker0x01/react-datepicker/issues/745

I was incorrect about that bug being caused by this PR


items effects not including skill multiplier in damage calculations

i've only noticed a couple instances of this so far: Mad Monarch's Scepter and Defender of Westmarch

What multiplier do you mean? I see 1400% on Mad Monarch's, plus whatever global multipliers you have.


Question about the prepared data

Should i transform the origin images' bx,by,bw,bh ?
e.g. if my image is 213213  bx =1 by = 1, i should transform the image to 416416 and replace bx = 2 . and put the transformed image's path and bx,by,bh,bw in data.txt ?

2019-03-17 13:48:00.646657: W tensorflow/core/framework/op_kernel.cc:1261] Invalid argument: TypeError: a bytes-like object is required, not 'str'
Traceback (most recent call last):
File "H:\py\lib\site-packages\tensorflow\python\ops\script_ops.py", line 206, in call
ret = func(*args)
File "D:\futurelab\yolo\utils\data_utils.py", line 130, in parse_data
pic_path, boxes, labels = parse_line(line)
File "D:\futurelab\yolo\utils\data_utils.py", line 19, in parse_line
s = line.strip().split(' ')
TypeError: a bytes-like object is required, not 'str'

@dunaizhuan 你可以参考一下#17，这是因为python2和python3的区别所导致的

@dunaizhuan 你可以参考一下#17，这是因为python2和python3的区别所导致的

解决了 可是这是怎么回事 loss很低了 但是准确率和召回为0？

@dunaizhuan 你可以参考一下#17，这是因为python2和python3的区别所导致的

谢谢了

解决了 可是这是怎么回事 loss很低了 但是准确率和召回为0？

检查一下数据的格式是否与readme写的一致，anchor之类的调整没有？

你好，我碰到和你相同的问题，准确率和召回率都为0，不知道你解决了吗？可否分享一下解决办法，谢谢。

Same problem. Please share the solution.

Same problem. Please share the solution.

Question about recall and acc ,i guess may be the iou must >=0.5 and the model will predict the outcome ,otherwise it will not predict(in paper).you can see it in the program.SO that's may because iou <0.5. By the way the total loss<2,model will save chekpoint in './chekpint'(may be the path?)

thank you.
one more question, i am getting an error like this. do you know the reason why
i am using own dataset of 10 classes.
File "/home/senth/madhu/YOLOv3_TensorFlow-master/utils/data_utils.py", line 140, in process_box
y_true[feature_map_group][y, x, k, :2] = box_centers[i]
IndexError: index 88 is out of bounds for axis 0 with size 88
Thank you.

你好，我碰到和你相同的问题，准确率和召回率都为0，不知道你解决了吗？可否分享一下解决办法，谢谢。

因为iou<0.5 模型当iou大于0.5才会做出预测？你可以在源文件还有论文中看到这一个。 我曾经改了iou<0.3, loss降低了但是他还是没有做出预测（可能iou还是太低了）。我做的一个比赛，是小物体的。因为conf loss 和class loss总是莫名其妙出nan。不做了 继续学习去了。没有解决这一个问题 。顺便提一下 当step%500==0应该是这个并且total loss<2 。他会保存 checkpoint  因该是在‘./checkpoint/’ 这个路径 我不知道他是否会自己创建这个文件夹，你翻下源码吧。记不大清了

thank you.
one more question, i am getting an error like this. do you know the reason why
i am using own dataset of 10 classes.
File "/home/senth/madhu/YOLOv3_TensorFlow-master/utils/data_utils.py", line 140, in process_box
y_true[feature_map_group][y, x, k, :2] = box_centers[i]
IndexError: index 88 is out of bounds for axis 0 with size 88
Thank you.

sorry, i don't know. perhaps

you can print(the dimension) in it Like this
. 88 out of size 88. may be 0 to  87 ,begin with 0?

thank you.
one more question, i am getting an error like this. do you know the reason why
i am using own dataset of 10 classes.
File "/home/senth/madhu/YOLOv3_TensorFlow-master/utils/data_utils.py", line 140, in process_box
y_true[feature_map_group][y, x, k, :2] = box_centers[i]
IndexError: index 88 is out of bounds for axis 0 with size 88
Thank you.
solution to ensorflow/core/framework/op_kernel.cc:1261] Invalid argument: TypeError: a bytes-like object is required, not 'str'
Traceback (most recent call last):

File "H:\py\lib\site-packages\tensorflow\python\ops\script_ops.py", line 206, in call
ret = func(*args)
File "D:\futurelab\yolo\utils\data_utils.py", line 130, in parse_data
pic_path, boxes, labels = parse_line(line)
File "D:\futurelab\yolo\utils\data_utils.py", line 19, in parse_line
s = line.strip().split(' ')
TypeError: a bytes-like object is required, not 'str'

你好，我碰到和你相同的问题，准确率和召回率都为0，不知道你解决了吗？可否分享一下解决办法，谢谢。

因为iou<0.5 模型当iou大于0.5才会做出预测？你可以在源文件还有论文中看到这一个。 我曾经改了iou>0.3, loss降低了但是他还是没有做出预测（可能iou还是太低了）。我做的一个比赛，是小物体的。因为conf loss 和class loss总是莫名其妙出nan。不做了 继续学习去了。没有解决这一个问题 。顺便提一下 当step%500==0应该是这个并且total loss<2 。他会保存 checkpoint 因该是在‘./checkpoint/’ 这个路径 我不知道他是否会自己创建这个文件夹，你翻下源码吧。记不大清了

谢谢回复，我用保存好的模型去评估，召回率准确率还是0，iou参数也调过，在检查代码中发现在计算准确率时符合条件的框数目（filter_boxes）一直是0个，所以导致了后面一系列0，不知道是哪儿出了问题，文件等等都已经排查了好多遍了。

thank you.
one more question, i am getting an error like this. do you know the reason why
i am using own dataset of 10 classes.
File "/home/senth/madhu/YOLOv3_TensorFlow-master/utils/data_utils.py", line 140, in process_box
y_true[feature_map_group][y, x, k, :2] = box_centers[i]
IndexError: index 88 is out of bounds for axis 0 with size 88
Thank you.

hi, have you solved this problem?

Not yet.

Not yet.

hi, I found my problem was the box inputs. Maybe the box is outside the image.  You can check it.

Not yet.

hi, I found my problem was the box inputs. Maybe the box is outside the image. You can check it.

Yes that was the problem for me too.
Thank you so much.

hi let me tell which annotation have you been used? labelimg or bbox label tool? and what was the format of annotation .xml or .txt?

Seems that problems get solved.
The previous loss function is buggy and now get fixed.
@kazim05 labelimg. The format is updated in the readme file.

add a script to generate the needed annotations from VOC xml files. see https://github.com/wizyoung/YOLOv3_TensorFlow/blob/master/misc/parse_voc_xml.py


[test] allow tests to run when using the firehose output and not us-east-1

to: @airbnb/streamalert-maintainers
size: small
resolves #460
Background
If you are deploying StreamAlert in a region other than us-east-1, and you configure a firehose output, currently ./tests/scripts/rule_test.sh will not work.  This fixes that.
Changes

Uses the configured region for the mocked lambda location.

Testing
In conf/global.json configure a firehose output and set the region to us-west-2.  Then run the tests and note they fail. Now apply this change, and note it passes.

Coverage remained the same at 95.636% when pulling a638a252f155a09394bb6b2751fa290a2178057e on 0xdabbad00:spiper-fix_firehose_test_for_those_running_in_other_regions into a329cf1c7c2dc1674b1df37e4577fc0688901b9e on airbnb:master.

Coverage remained the same at 95.636% when pulling f5d5b55855f856fc582ce50b63c9367b5833247d on 0xdabbad00:spiper-fix_firehose_test_for_those_running_in_other_regions into 6ae211b98a98940c264c961f533ca693f2a6e004 on airbnb:master.

hey @0xdabbad00 thanks for this change, it has now been merged :)


Workaround Powershell 6.2/7.0 breaking compatibility with Get-ChildItem

Powershell 6.2/7.0 in infinite wisdom decided to change the value of
what Get-ChildItem (and anything dealing with FileInfo/DirectoryInfo)
returns from relative to absolute path.
The only way to work around this for both existing versions and future
versions is to explicitly request the Name. Hopefully this doesn't
get changed again.
NOTE: This PS change breaks a lot of scripts, including many
of Microsoft's own first-party scripts. No workarounds were pushed
for any that I could see in the year+ since the issue was first
reported.

This looks good to me, and is more correct. Thanks @TheGoddessInari


About Abstract of Daily Papers - Hugging Face

Hi Yuxuan,
Thanks for putting together this awesome repository! It's super interesting and I really appreciate your efforts.
I have a small (and perhaps slightly irrelevant) question: I'm curious about how you discovered that the "Daily Papers - Hugging Face" is curated by Andrej Karpathy. From what I've found, aside from the Daily Papers Twitter account @ _akhaliq sharing the same initials as Andrej Karpathy, there doesn't seem to be a clear connection :)
Best,
Bocheng

Thanks a lot for your reminding! 😇 I have been always taking _akhaliq as Andrej Karpathy before today. I will fix it immediately!


fix the check (#14529)

This PR ports #14529 into release branch that addresses the #14481 issue.

Pull Request Test Coverage Report for Build 628101069

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.008%) to 43.464%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/common/ports.ts
4
74.14%

Totals

Change from base Build 594133608:
-0.008%

Covered Lines:
24937

Relevant Lines:
52550

💛  - Coveralls

@MaddyDev Do we need this anymore now that 1.27 is out?

@MaddyDev Do we need this anymore now that 1.27 is out?

Nope! Closing it out, Thanks!


Why aren't the download packages updated

Who can have any faith there is any maintenance being done on this project
The download package is exactly the same edition that has not changed in 10 months. The package repository for Ubuntu has not been updated in more than a year.
It seems highly likely to get any recent versions of this software you may have to build the packages yourself.
The whole point of a beta (as this software is described) is testing, then you get new packages that are built regularly. One open source software product I test regularly is building new packages daily throughout their test program. Obviously that doesn't happen here.

Please try again with the latest version and let us know if this issue needs to be reopened.


Remove Legacy ABI Compatibility

We are carrying a lot of dead-weight just by supporting the old GCC ABI which probably no one uses in combination with libobjc2.
There are still lot more ugly hacks which can be removed.

Isn’t there more to be deleted from the upgrade logic?

Yeah but I was not sure how much to delete. Do we want to leave the gsv1 upgrade logic in place or remove it as well?

And this would fall under OLDABI_COMPAT right?

Isn’t there more to be deleted from the upgrade logic?

Yeah but I was not sure how much to delete. Do we want to leave the gsv1 upgrade logic in place or remove it as well?

I'm not sure.  We currently don't support the new ABI on Mach-O in clang and it's quite useful to be able to run tests on macOS so that we can compare their behaviour against the Apple runtime.  I've not tried WebAssembly, but it probably also needs some custom work.
The main reason I'd drop the GCC ABI is that anyone using GCC should probably use their runtime.  Ideally we'd get every target supported by the v1 GNUstep ABI onto v2 and then drop v1 as well.

We currently don't support the new ABI on Mach-O in clang

I did not know that we support v1 on Macho-O. It would be very interesting to support v2 as well just for benchmarking purposes.

I'll remove the GCC upgrade logic then.

Google recently updated the clang prebuilds for the Android SDK. The fast path test now executes and fails instead of being skipped.

@davidchisnall can you take another look at it. I've rebased it onto the latest changes in master.


Live reload do not works with Content Security Policy

Looks like Hugo needs to alter Content Security Policy too when injecting script.

The Hugo dev server doesn't support HTTPS, and so doesn't set these headers at all (correctly). Sounds like the issue lies with whatever youre using to SSL-terminate the connections to the dev server, rather than the dev server itself?

I have this in my index.html:
<meta http-equiv='Content-Security-Policy' content="default-src 'self';
        script-src
            'self'
            https://www.googletagmanager.com
            https://www.google-analytics.com;
        connect-src
            'self'
            https://api.ipdata.co;
        img-src
            'self'
            https://www.google-analytics.com
            https://stats.g.doubleclick.net;
        font-src
            'self'
            https://fonts.gstatic.com;">

... I tried to reproduce this issue but now there is no live reload script in generated html... I run hugo server without flags.

Is the live reload script just blocked from your CSP rules? Take a look at your browser console incase there's a message in there.
Might be better to discuss this on the forum, as this doesn't immediately look like an issue with Hugo, rather with the tag itself.

As a workaround you could assign ENV variables and call the CSP meta tag only when the Hugo site is deployed. Have a look over here and for a longer discussion here
But I'm closing this issue. It is unlikely that Hugo server will provide CSP support since it's not meant for production.
If you need further assistance please post a topic at https://discourse.gohugo.io


docs: add article about soldering clever

@Alamoris, please, review the edits.

I read the article again, in my opinion everything is correct
Thanks for corrections


Compile error with gcc using hana::is_valid() to check for a non-deduced template call operator

I'm cross-posting this from Stack Overflow, as no one there has pointed out a way in which I'm using Hana wrong, so I suspect it could be a compiler/Hana bug.
In my application, I want to determine at compile time whether an arbitrary functor type Func has a nullary call operator that can be invoked with a given explicit template argument T. Based on a previous SO answer that I found, I came up with the following:
#include <boost/hana.hpp>
#include <iostream>
#include <type_traits>

namespace hana = boost::hana;

namespace detail
{
    template <typename T>
    auto can_call = hana::is_valid([](auto &&f) ->
        decltype(f.template operator()<T>()) { });
}

template <typename Func, typename T>
constexpr auto can_call() ->
    decltype(detail::can_call<typename std::remove_reference<T>::type>(
        std::declval<Func>())) { return {}; }

struct foo
{
    template <typename T, typename =
        std::enable_if_t<!std::is_same<T, char>::value>>
    void operator()() const { }
};

int main()
{
    std::cout << "char: " << can_call<foo, char>() << std::endl;
    std::cout << "int: " << can_call<foo, int>() << std::endl;
}

I would expect this example to print out:
char: 0
int: 1

Since the char template argument type is explicitly enable_if-ed out in foo. I've tried the following compilers:

Apple clang v8.0.0: The example compiles and runs as expected.

mainline clang v3.9.1+ (via Wandbox): The example compiles and runs as expected.

mainline clang v3.6.0 - v3.8.1 (via Wandbox): The compiler dies with an internal error.

g++ 7.0 trunk, 20170410 (via Wandbox): The compilation fails with the following errors:
  dd.cc: In instantiation of ‘auto detail::can_call<char>’:
  dd.cc:15:14:   required by substitution of ‘template<class Func, class T> constexpr decltype (can_call<typename std::remove_reference<_To>::type>(declval<Func>())) can_call() [with Func = foo; T = char]’
  dd.cc:25:50:   required from here
  dd.cc:10:10: error: ‘auto detail::can_call<char>’ has incomplete type
      auto can_call = hana::is_valid([](auto &&f) -> decltype(f.template operator()<T>()) { });
          ^~~~~~~~
  dd.cc: In function ‘int main()’:
  dd.cc:25:50: error: no matching function for call to ‘can_call<foo, char>()’
      std::cout << "char: " << can_call<foo, char>() << std::endl;
                                                  ^
  dd.cc:14:16: note: candidate: template<class Func, class T> constexpr decltype (can_call<typename std::remove_reference<_To>::type>(declval<Func>())) can_call()
  constexpr auto can_call() ->
              ^~~~~~~~
  dd.cc:14:16: note:   substitution of deduced template arguments resulted in errors seen above
  dd.cc: In instantiation of ‘auto detail::can_call<int>’:
  dd.cc:15:14:   required by substitution of ‘template<class Func, class T> constexpr decltype (can_call<typename std::remove_reference<_To>::type>(declval<Func>())) can_call() [with Func = foo; T = int]’
  dd.cc:26:48:   required from here
  dd.cc:10:10: error: ‘auto detail::can_call<int>’ has incomplete type
      auto can_call = hana::is_valid([](auto &&f) -> decltype(f.template operator()<T>()) { });
          ^~~~~~~~
  dd.cc:26:48: error: no matching function for call to ‘can_call<foo, int>()’
      std::cout << "int: " << can_call<foo, int>() << std::endl;
                                              ^
  dd.cc:14:16: note: candidate: template<class Func, class T> constexpr decltype (can_call<typename std::remove_reference<_To>::type>(declval<Func>())) can_call()
  constexpr auto can_call() ->
              ^~~~~~~~
  dd.cc:14:16: note:   substitution of deduced template arguments resulted in errors seen above

It seems to not like my use of hana::is_valid() to determine whether the specified operator exists. However, I think the way I'm using it is consistent with its intended use. It seems like there are a few potential explanations:

A gcc bug, which would be present in all versions that I've tested (I've also tried 6.3 as well as the 7.0 trunk). By definition, it seems that clang < 3.9.0 has a bug here since it ICEs.
This construct should not be allowed for some reason, and clang is being more accommodating than gcc in its interpretation of the standard.
I used Hana incorrectly in some way, or it's not capable of doing this specific type of check. If there's a better way to do this, I'm all for it!

It crashes on Apple Clang 7.3 too. It can be worked around by not using a lambda in your is_valid. I can't say if that is the same problem with gcc 7.0 though.
namespace detail
{
    template <typename T>
    struct check_can_call {
      template <typename F>
      constexpr auto operator()(F&& f) ->
        decltype(f.template operator()<T>()) { }
    };

    template <typename T>
    constexpr auto can_call = hana::is_valid(check_can_call<T>{});
}

Thanks for that suggestion. It looks like with the workaround you gave, it now compiles on clang 3.5.0+, like the rest of Hana. gcc 6.x/7.x still seems to reject it though. :disappointed:

Here's one that works on gcc 6-7 as well.
namespace detail
{
    template <typename T>
    struct check_can_call {
      template <typename F>
      constexpr auto operator()(F&& f) ->
        decltype(f.template operator()<T>()) { }
    };

    template <typename T>
    using is_call_valid = decltype(hana::is_valid(check_can_call<T>{}));

    template <typename T>
    constexpr is_call_valid<T> can_call{};
}

It just adds a another layer of indirection 😎

Every problem can be solved with another layer of indirection. :) Thanks! Should I close this issue? I'm not sure if there's any action for Hana itself here.

I'm not sure either. @ldionne?

Closing. I don't think this is Hana's fault.


Possible multi fork

Code might fork multi application if called many times a check should be done before the fork.

Fixed :+1:


How can I get the Depth and Color image registered with the Unity wrapper? Also, is there any Guide I can follow?

I've made a function in a .dll that registers 3 images together (color, depth, thermo), for that I need 3 IntPtr.
I need to make a UI useing Unity to show the registered images, so I'm using the RealSense Unity wrapper to pick up the color and depth images from the stream.
I'm following the "AligmentSample" from the "realsense.unitypackage" but I can't find where I can get both images registered (Color and Depth) at the same time.
I've tryed to get them from the "OnNewSample(Frame frame)" function in "RsStreamTextureRender.cs" but the problem is that on the first call of the function I get the color ant in the second de depth, but I need to get both at the same time.
Does anyone know how I can get Depth and Color frames at the same time?
If not, is there any Guide apart from the github readme?
I'll really appreciate any help.
Thanks!

You might be looking for something more along the lines of this c# example.
Unity wrapper provides a few abstractions over the .NET wrapper.
Alignment in the Unity sample is done in the RsAlign class here.
The RsStreamTextureRender extracts a single video frame, and creates a Unity texture from it.

Finally found out that the problem was that I wasn't usint the using statments.
Now I have in the UI the depth and the color image streams working perfectly, but now I need to show the RGB thermal image (an IntPtr) stream.
The problem is that it's not a RsDevice object so I can't use the RsStreamTextureRender or at least that's what I think.
So any idea how I can show the thermal image stream?
Thanks for your help!


Multiple instances in one scene don't work

Hi! Nice Plugin. I wanted to use it for my space game.
https://twitter.com/RecallSingular1
Steps to reproduce:

Open test scene "3D Text.tscn" which says "Hello! this is my 3D text plugin."
Duplicate the Label3D node
3a. Edit the text in the original node
3b. Edit the text in the duplicate node

Expected:
The two text fields should change by themselves, without affecting each other.
Result:
It's clear they are both rendering into the same backing buffer and the text is only changed when you edit the original node. However, the dimensions of the second are independant of the first's dimensions.

Thank you for reporting the bug! :)
It should be fixed now in the latest commit.

Wow, that was quick! Thanks.
I will test this new version soon and let you know.

Confirmed fixed! Thanks very much :)


[zh] Localize 'devsecops'

[ ] Localize the content/en/devsecops.md to content/zh-cn/ in dev-zh.

Repeated


Feat : Add new game : darkness_dungeon

https://github.com/RafaelBarbosatec/darkness_dungeon

@ashitaprasad  I want to add this game , please assign it to me . Thank you!


Implement Local Discontinuous Galerkin method for Poisson

The Local Discontinuous Galerkin method for Poisson requires "lifting operators" that involve mixed DG face integrators. Currently, MixedBilinearForm only supports trace integrators where the trial space is assumed to be continuous. I have added interior and boundary face integrators to MixedBilinearForm that allow both the test and trial spaces to be discontinuous. This required adding an AssembleFaceMatrix to BilinearFormIntegrator that accepts four elements: 2 trial elements from each side of the face and 2 test elements from each side of the face. I also added functionality to build the resulting rectangular parallel matrix.
The LDGTraceIntegrator implements the consistent term from integrating by parts as well as an optional upwinding term (the "switches"). DGPenaltyIntegrator is just the penalty term from DGDiffusionIntegrator. With the switches, penalization on interior faces is not required for stability.
I added ex30.cpp and ex30p.cpp. These solve the Poisson equation with LDG in first-order form by forming the Schur complement.
@pazner what do you think? @bensworth this branch should be good to go for your purposes (see ex30.cpp).

@smsolivier, sorry for the delay. This looks great! I will try to take a closer look at it soon, and review it more thoroughly.

@pazner no worries! This is still a draft. I can help fix things.
It's failing tests because I use KLUSolver in ex30.cpp without checking if that is available. I just wanted to show how LDG would work. It might make more sense to add this as an option in example 14.

Please mark as ready-for-review when this is ready.


node-sass does not seem like it is installing with version 4.5

Before opening an issue:

Read the common workarounds in the TROUBLESHOOTING.md
Search for duplicate or closed issues
Validate that it runs with both Ruby Sass and LibSass
Prepare a reduced test case for any bugs
Read the contributing guidelines

When reporting an bug, you must provide this information:

NPM version (npm -v): 4.1.2
Node version (node -v): 7.5
Node Process (node -p process.versions):
Node Platform (node -p process.platform):
Node architecture (node -p process.arch):
node-sass version (node -p "require('node-sass').info"): 4.5.0
npm node-sass versions (npm ls node-sass): 4.5.0

When encountering a syntax, or compilation issue:
node-sass

Open an issue on LibSass. You
may link it back here, but any change will be required there, not here

If you delete this text without following it, your issue will be closed.
i have installed node-sass and can see it is there but when i do node-sass -v or -help it doesn't do anything.

I'm also having issues with node-sass 4.5.0. My node_modules/ directory is empty. Here are the details:
$ npm ls node-sass
/home/developer/tmp/temp1
└── (empty)
$ npm -v
4.1.2
$ node -v
v7.6.0
$ node -p process.versions
{ http_parser: '2.7.0',
  node: '7.6.0',
  v8: '5.5.372.40',
  uv: '1.11.0',
  zlib: '1.2.11',
  ares: '1.10.1-DEV',
  modules: '51',
  openssl: '1.0.2k',
  icu: '58.2',
  unicode: '9.0',
  cldr: '30.0.3',
  tz: '2016j' }
$ node -p process.platform
linux
$ node -p process.arch
x64

Here are gists of my npm.err and npm.log: https://gist.github.com/oolongtea/724b7fe70c0a47a867862ee785118f94

@xtianus79 without any logs or OS information there is nothing we can do. I can confirm 4.5.0 is has been installed over a million times since release so there must be something about your environment.
@oolongtea unfortunately your logs are truncated before getting to the actual installation failure. You can turn the log level down.


Updates fsevents dependency

fsevents is not publishing binaries for its older versions on new node versions.
This updates fsevents so that package managers running on the latest node version will have binaries available

Misread the caret declaration -- dependency should be fine

Coverage remained the same at 98.448% when pulling 5f9e0cb9478d8ee239d2975d2390c6824006a5a3 on chrissrogers:patch-2 into 3d917818c957088b62c160e426c1a99787ae6319 on paulmillr:master.


Error "expected string, got nil"

Neovim version:
NVIM v0.9.4
Build type: Release
LuaJIT 2.1.1692716794

I'm using lazyvim and trying the plugin without any config:
return {
  {
    "nvim-telescope/telescope.nvim",
    dependencies = {
      "cljoly/telescope-repo.nvim",
      "nvim-lua/plenary.nvim",
      config = function()
          require("telescope").load_extension("repo")
      end,
    },
}

The extension get loaded since I had the autocompletion. But if I do :Telescope repo list or :Telescope repo list search_dirs=~/Documents I get:
[telescope] [WARN  20:27:28] /home/fgionghi/.local/share/nvim/lazy/telescope.nvim/lua/telescope/pickers.lua:669: Finder failed with msg:  vim/shared.lua:0: prefix: expected string, got nil

I get the telescope picket and If I keep writing in the input box (saying Git repositories (built on the fly) I got this error:
E5108: Error executing lua: ...re/nvim/lazy/telescope.nvim/lua/telescope/from_entry.lua:16: attempt to index local 'entry' (a nil value)
stack traceback:
	...re/nvim/lazy/telescope.nvim/lua/telescope/from_entry.lua:16: in function 'path'
	...escope-repo.nvim/lua/telescope/_extensions/repo/main.lua:177: in function 'run_replace_or_original'
	...re/nvim/lazy/telescope.nvim/lua/telescope/actions/mt.lua:65: in function 'run_replace_or_original'
	...re/nvim/lazy/telescope.nvim/lua/telescope/actions/mt.lua:65: in function 'key_func'
	...hare/nvim/lazy/telescope.nvim/lua/telescope/mappings.lua:290: in function <...hare/nvim/lazy/telescope.nvim/lua/telescope/mappings.lua:289>

Using instead :Telescope repo cached_list works.
checkhealt telescope:
Telescope Extension: `repo` ~
- OK Will use `batcat` to preview non-markdown READMEs
- WARNING Install `bat` for a better preview of other files
- OK Will use `batcat` to preview markdown READMEs
- WARNING Install `glow` for a better preview of markdown files
- OK locate: found `plocate`
  plocate 1.1.15
  Copyright 2020 Steinar H. Gunderson
  License GPLv2+: GNU GPL version 2 or later <https://gnu.org/licenses/gpl.html>.
  This is free software: you are free to change and redistribute it.
  There is NO WARRANTY, to the extent permitted by law.
- Repos found for `:Telescope repo cached_list`:
  /home/fgionghi/.cache/omf/https___github.com_oh-my-fish_packages-main_master/.git, /home/fgionghi/.cache/tldr-python/.git...
- OK fd: found `fd`
  fd 9.0.0
- Repos found for `:Telescope repo list`:
  /home/fgionghi/i3-easyfocus, /home/fgionghi/.nvm...

It has also found some repositories...
I know little either of neovim and lazyvim... chances are I'm doing something wrong.
Thank you.

I have the same issue.

Thanks for flagging, let me look into the proposed fix.

I’ve merged @rish987 changes, this should now be fixed could one of you @thuck @life00 @fgionghi please confirm that the bug is fixed for you as well on the latest master?

I’ve merged @rish987 changes, this should now be fixed could one of you @thuck @life00 @fgionghi please confirm that the bug is fixed for you as well on the latest master?

I confirm that I am unable to reproduce this bug on latest master.

Since this seems fixed (see also #72), I’ll close this issue.


Add https://github.com/fehbari/sunshine-scripts to Scripts section

Description
Adds my display automation scripts repository to the scripts section.
Type of Change

[x] Documentation update (changes to documentation)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated the in code docstring/documentation-blocks for new or existing methods/components

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Can user be removed from online meeting after transfer or can the online meeting be closed by IncidentBot?

Hi,
I had a question regarding the IncidentBot sample under:
https://github.com/microsoftgraph/microsoft-graph-comms-samples/tree/master/Samples/BetaSamples/RemoteMediaSamples
This sample works great and it is creating incidents followed by adding user to online meeting. I wanted to check if it is possible to remove the user from the online meeting they were added to earlier via IncidentBot?
I tried ending the call using ICall.DeleteAsync created at:
https://github.com/microsoftgraph/microsoft-graph-comms-samples/blob/9db883a1e8649be9889f8ed500969634695b3bcd/Samples/BetaSamples/RemoteMediaSamples/IncidentBot/Bot/Bot.cs#L214
However looks like that call got deleted when the user was transferred to incident meeting, appreciate any thoughts or comments on this. Additional question about this, can the IncidentBot close/end the online meeting it has joined/started?

Can you remind me of the flow please?  Sample calls a couple agents, then when they pick up they get transferred to the meeting?

Sure the flow I am using is to raise an incident, here are the steps in this flow:

user sends a HTTP POST request to raise incident
to process that request IncidentBot joins online meeting
IncidentBot makes a call to user
when user answers call it plays prompt to press 1 or 0
if user presses 1 IncidentBot adds user to online meeting

If you can send me some logs, that would help as well.  Best thing would be a single http headers snapshot of an incoming notification.

For 5) I don't see any usage of TransferAsync, only InviteAsync.  Invite will send a new invite to the user, but will not terminate the original call.  If the original call to the user is a P2P call, the call will end if any of the participants leave.

Apologies, the sample is doing things correctly... InviteAsync specifies replacesCallId, which will move the user from the original call into the meeting.

At which point the original meeting is terminated for all parties, you need not delete it.  You should have a notification raised that the original call is terminated and then removed from ICallCollection

Yes, it looks like original call is deleted when user is added to meeting at:
https://github.com/microsoftgraph/microsoft-graph-comms-samples/blob/9db883a1e8649be9889f8ed500969634695b3bcd/Samples/BetaSamples/RemoteMediaSamples/IncidentBot/Bot/MeetingCallHandler.cs#L63
I am trying out a case where once the user is done with the online meeting if IncidentBot can end the online meeting it started/joined or if it can remove the user from online meeting?

A couple terminology clarifications:
when you say online meeting, you mean the multiparty call?  We have 2 different concepts: online meeting, and call.  The online meeting is usually just the coordinates on how people can connect to each other, from there a call is created with the online meeting coordinates.  Same online meeting can have many multiparty calls.  Each call is ended once all participants leave, and started again once a participant joins.
We do not have any APIs to eject participants from multiparty calls.  Today we only support ending P2P calls.  What we do have a flag AllowConversationWithoutHost where if the bot is the first one to join the meeting, and it leaves, everyone is removed from the meeting.

Thanks for the clarification and explanation, if I understood it then as of now there is no API to end multiparty call. I would like to request that as an enhancement where a bot can end multiparty call via API that it started, please let me know if I need to create some ticket/request for that or if this issue can be used for that purpose.

If the bot always joins first, you can use AllowConversationWithoutHost as mentioned below.  But if you want to join a meeting then end it, you cannot.
@VinodRavichandran Request for an API to terminate a call for every participant.


add shiplift docker client to remote api client list

I'd like to add a Rust client I've been working on to the list of remote api clients.

Thanks @softprops, can you fix the missing quote, amend your commit, and do a force-push? That should automatically update the PR

pushed.

@softprops can you squash your commits, so that there's a single commit in the PR?

whoops. sorry about that. squashed into one commit

Oh, boy, apologies, I overlooked this one; can you swap it with docker-rust, so that they are in alphabetical order? Really sorry I missed that before :blush:

flipped and resquashed

Awesome!
LGTM

LGTM 🐲

thanks @thaJeztah !


Ice/Snow Maker Upgrade doesn't work

The fridge upgrade doesn't appear to provide snowballs for Pam's recipe (ice cream specifically). The multiblock used was fridge(both upgrades)/table/oven/sink and appropriate utensils. With snowballs in inventory the recipe appears in the table, and crafts without issue. Without the snowballs, but all other ingredients no recipe appears. Thanks.
Versions:

MC 1.12.2
Forge 2768
CfB 6.4.65
Pams 1.12.2za

This still seems to be the case. Is this being worked on?


feat: Promote the metaStore and stateStore

What's changed and what's your intention?

As title

Checklist

[x] I have written the necessary docs and comments
[x] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

Update the branch to trigger the new CI.

Besides, I think we need test coverage on those reference fields.


Routes Generator - Undefined method in Rails 4.2.1

When trying to generate a route using Rails 4.2.1, an error is being thrown due to an undefined method.
Example:
$ rails g rest_kit:route api_v1_emotion_categories --ios-path=../../iOS/VentiOS --model="EmotionCategory"

/Users/ezescaruli/.rvm/gems/ruby-2.1.5/bundler/gems/restkit_generators-e6937957ee34/lib/generators/rest_kit/route/route_generator.rb:86:in `ios_route_path': undefined method `optimized_path' for #<ActionDispatch::Journey::Route:0x007ff039589d60> (NoMethodError)

Fixed in 5c774e6ceb486ec0bf27ef55b2c38a815f492480


Zombies double infinitely

Unfortunately, as of version 1.5.1, zombies double infinitely, which has lead to unfortunate situations for some.
I plan to fix this very soon.

Supposedly fixed. README.md has details:
When the number of zombies reaches 120 or more, 75 zombies will be killed.

Fix is now tested and approved working.


xplat: Fix absolute addressing issue [clang] [x86]

Code signed PIE but used in Js::ScriptContext::ProfileModeDeferredParsingThunk::RecyclableObject::CallInfo

Is that "script profile mode" which we turned off? If so we may try fix some ifdef to not use it.

Is that "script profile mode" which we turned off? If so we may try fix some ifdef to not use it.

@jianchun no actually. This was an xcode compile time issue on macos side.
Since then we made some code cleanup and general updates.
This issue no longer exist.


Lock down solution guides

Coverage remained the same at 61.898% when pulling c3ca5ac7afe546564378dbe5e51b38e3851c8424 on lock-paid-solutions into 41263c37a901fa5ab48b30980c4ce1d4d190d6f2 on master.

Looks good!


[Insight] Text files should end with a newline character - in resources/…/ManyToMany/model.blade.php, line 31

in resources/views/template/ManyToMany/model.blade.php, line 31

This file ends with no newline character. It won't render properly on a terminal, and it's considered a bad practice. Add a simple line feed as the last character to fix it.

     */
    public function remove{{ucfirst(str_singular($model))}}(${{str_singular($model)}})
    {
        return $this->{{str_plural($model)}}()->detach(${{str_singular($model)}});
    }

Posted from SensioLabsInsight

Thanks!!


fix(searchParameters): avoid mutating provided objects

Fixes #1130
Thanks a lot @morhaus for your help when debugging

The hardest bug have always the smallest solution :) LGTM 👍


[Feature Request] Setup some language-specified paths

Is your feature request related to a problem? Please describe.
I've setup Mango for some of my friends who reads manga in french, or english if not available in french.
With the MangaDex downloader, all the manga are stored in the same folder, without distinction for language.
Describe the solution you'd like
It could be great to have the choice to set a french folder for the manga downloaded in MangaDex with the french language, and set another folder for the english language.
Describe a small use-case for this feature request
This settings could apply to another languages (japanese/english, ...).
Additional context

Exemple of a way to do it, for the end user, in the config.yml file:

+1, i'd like to have this feature too. although i would much rather every title in library could have multi-language supprot.\
-.
├── Title1
│   ├── Volume 1.cbz
│   ├── Volume 2.cbz
│   ├── Volume 3.cbz
│   └── Volume 4.zip
└── Title 2
    └── en
        └── Ch.1 - Ch.3
            ├── 1.zip
            ├── 2.zip
            └── 3.zip
    └── fr
        └── Ch.1 - Ch.3
            ├── 1.zip
            ├── 2.zip
            └── 3.zip


Finish original figma desgin 🖌

[x] Empty questions at room and admin room
[x] Enter the room button
[x] AdminRoom hover icons
[x] Outline in textarea for questions
[x] Filter on avatar if question is answered
[x] Can't like, if not logged in, copy style from button disabled to like button
[x] Delete question modal
[x] Close room modal

Issue completed 🎉
with commits:

Empty questions design feature completed ✅ #6
Adding the "enter the room" icon 🔆
AdminRoom hover icons 🔆 #6
Adding an outline for question input 🖌 #6
Adding b&w filter if question is answered ⚫⚪ #6
Using "disabled" on like button 🚫 #6
Confirmation modals completed ✅ #6
Desgin adjustments in NewRoom 🖌 #6


cmd/go: TestScript/build_acl_windows fails on windows-arm builder

windows-arm builders seems to be broken
https://build.golang.org/log/2579a41ab2b6bafce55876919188aa9933e746d2
with
--- FAIL: TestScript (0.08s)
    --- FAIL: TestScript/build_acl_windows (12.53s)
        script_test.go:189:
            # Create $WORK\guest and give the Guests group full access.
            # Files created within that directory will have different security attributes by default. (3.085s)
            # Build a binary using the guest directory as an intermediate (6.094s)
            # Build the same binary, but write it to the guest directory. (3.162s)
            # Read ACLs for the files. (0.072s)
            > exec powershell -Command 'Get-Acl main.exe | Select -expand AccessToString'
            [stderr]
            S�t�a�r�t�i�n�g� �t�h�e� �C�L�R� �f�a�i�l�e�d� �w�i�t�h� �H�R�E�S�U�L�T� �7�e�.�
�
            �[exit status -65536]
            FAIL: testdata\script\build_acl_windows.txt:20: unexpected command failure

FAIL
FAIL	cmd/go	918.021s

Probably since https://go-review.googlesource.com/c/go/+/165752 was submitted.
The error message is not very helpful. And I do not have windows-arm computer to debug. @jordanrh1 can you, please, help to debug this? Thank you.
Alternatively, maybe we could revert CL 165752 ? Leaving for @bcmills to decide.
Alex

The difference is probably due to the more restricted environment for script tests. The setup for that environment is here:
https://github.com/golang/go/blob/fd080ea3bcc2b170b787b38ab7920d170ca65682/src/cmd/go/script_test.go#L101-L132
Given the generally shaky state of the windows-arm builder, I'm inclined to add a skip until the test is fixed rather than roll back the change.

Looks like exec.LookPath uses the PATHEXT variable on Windows. We should probably preserve that.
https://github.com/golang/go/blob/fd080ea3bcc2b170b787b38ab7920d170ca65682/src/os/exec/lp_windows.go#L61

The difference is probably due to the more restricted environment for script tests.

I agree, this could well be the problem here. It just we don't know which environment variable must not be skipped.

Looks like exec.LookPath uses the PATHEXT variable on Windows. We should probably preserve that.

I don't see what you you mean by that. script_test.go uses standard os/exec package to implement everything - it should just work. No?
Alex


the tutorial code for Azure Web PubSub does not compile

The c# code does not compile out the the box :(
negotiate.cs needs more reference
Microsoft.Azure.WebJobs.Extensions.WebPubSub
notification.cs needs more reference
Azure.Messaging.WebPubSub
System.Threading.Tasks
Microsoft.Azure.WebJobs.Extensions.WebPubSub
Then it compiles,
it may be a better way to fix it so feel free to change to code as long as it works
last on step 7.
Its not https
should be http://localhost:7071/api/index

Dokumentinformasjon
⚠ Ikke rediger denne delen. Den kreves for koblingen docs.microsoft.com ➟ GitHub-problem.

ID: 4b610110-f417-3565-bbd6-35e90a567a2d
Version Independent ID: 53b5d6ae-54ef-c45d-19f1-cd2ebd7e1bd6
Content: Tutorial - Create a serverless notification app using Azure Web PubSub service and Azure Functions
Content Source: articles/azure-web-pubsub/tutorial-serverless-notification.md
Service: azure-web-pubsub
GitHub Login: @JialinXin
Microsoft Alias: jixin

@nimrof -
Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Asset based XDMs are too content management centric.

What are the schemas that are affected by the issue
https://github.com/adobe/xdm/blob/master/schemas/assets/asset.schema.json
https://github.com/adobe/xdm/blob/master/schemas/assets/video.schema.json
https://github.com/adobe/xdm/blob/master/schemas/assets/image.schema.json
What are examples of products that are impacted by the issue:
AdCloud, Video Analytics
Video
Image
These all have inherited Asset required fields which do not easily translate to how AdCloud categorizes videos:
AssetID – this has a very specific format requirement not currently implemented in AdCloud or the ad industry.  Can we be more flexible on this field?
eTag – information not currently available at event stream capture time
format, lastModifiedDate, name, path, size, version – these are currently stored in our facts tables and not relevant at event processing time.
For consumption based processes the required fields are not as relevant and the information is not available in many cases.
We need to review if it makes sense for some of these required and strongly structured fields, or if we should use a different schema doe the consumption aspects of assets.

Regarding "assetID", it is definitely inconsistent (within the context of XDM) that it is a URN versus a URI as used elsewhere.  We use URIs because that is the "JSON-LD way".
I know that "assetID" and its format are driven by how we have implemented in the past, and so I'm not suggesting we try to change it, but we might want to either allow it to be more flexible in the context of XDM (just a string) or have an adjacent field that would allow for a URI-based ID (an @id).
Of course, even if we do this, if someone wants to point to assets in the repo they will need to understand "assetID".

Here's my suggestion:

For XDM, just define repo:assetID as a string. The value is required to be treated as opaque by clients, so suggesting an internal structure is both unnecessary and leads to bad behaviors. For XDM and API, it's only the semantics of this identifier that are important--when it is created, that it isn't re-used, and so on.

Implementers should feel free to use some internal structure when assigning this value, as this is useful when troubleshooting. In our ACP implementations at Adobe, we happen to have agreed on a urn-based scheme that provides some hints as to which system generated the identifier, and in which region--in addition to a GUID. But again, these are purely implementation details, and need not be mentioned in XDM.

Changing assetID to be a plain string seems to be an easy enough change.
@kstreeter: we already have an optional @id property for every asset.
Should we make assetID optional, given that the loose string prescription might lead to many empty strings anyway?

@trieloff @kstreeter - I would like to make a move on the Asset-Reference concept. We are preparing new schema proposals that refer to media (movies, audio clips etc) and they refer to an asset with most of the same properties, but will not always have all the required properties as in the Asset and related models.
How should I proceed? It does not seem right to just create a mirror of the current schemas withthe addition or *Reference.


Send users an email confirmation on logins on a new device

We should send users an email notice when they login on a new device; this is a standard security nicety.   It's worth looking at whether Django has a standard thing for this before spending too much time working on it.

This sounds like an interesting task that I will like to contribute to. Can you please assign this to me.

Cool @hollywoodno!  Consider it assigned; GitHub requires overhead of "adding someone to a project" in order to assign issues to them, so we just use the issue thread to see who's working on what.

So far I do not see that Django has any specific or straightforward way to send notifications on logins from new devices. There's access to user agents but no specifics on obtaining device specific information to be able to associate a particular device with a previous log in. So far it seems it can be accomplished using a combination of user agents, Django session ID's and cookies.

For logins on web, we can use: https://docs.djangoproject.com/en/1.10/ref/contrib/auth/#module-django.contrib.auth.signals to find out about new logins.  Mobile goes through a different codepath (api_fetch_api_key in zerver/views/auth.py), but we can just hook it there.


Affected rows reports wrong number for create/drop statements

Describe this problem

Failed to execute query, err: Server(ServerError { code: 500, msg: "Failed to convert output, query: DROP TABLE demo;. Caused by: Rpc error, code:500, message:Failed to convert output - affected rows: 1" })

A drop table tells one row is affected
Steps to reproduce
drop table demo;

Expected behavior
mysql> create table xx like meta_2 ;
Query OK, 0 rows affected (0.11 sec)

Additional Information
These SQLs don't actually count rows, 1 is the default value. Change the default value should fix this.

can this be assigned to me? this looks like I didn't set it up before

Thanks for your interest! This is an existing behavior. There are some places that are designed to act like this. E.g: https://github.com/CeresDB/ceresdb/blob/ddaa3e8346d14bcc4432a4b3ac24a28e316d42ca/table_engine/src/table.rs#L445-L454

It looks like I need to modify the implementation of the Table trait so that it defaults to 0?👀

ok i'll check it.


Changed to smbus2 and added requirements.txt

I couldn't find the original smbus package, but I found this one, which seemed to work fine: https://pypi.org/project/smbus2/
They have a link to the original one, but it seems to be down.
Also added is a requirements.txt to more easily get the required packages. (Can be installed by pip install -r requirements.txt)
I also tested the script on a raspberry pi 3B+ which worked great, so if you want, I can also change the Compatability here to "Work well" for Pi 3.

Thank you very much. I will test and verify this in the future, and may add some modifications compatible with the old version of smbus.

Hello, I just flashed a new Raspberry Pi image and didn't manually install the 'smbus' package. I imported it directly and it works fine. Maybe you can provide us with more information to understand the issue you encountered earlier.

It's worth mentioning that SMBus seems to be directly referenced in SMBus2, and it is indeed a pre-installed library. For most people, they need an easier way to use it, so my library will remain the same. However, when you use a more personalized approach, you will learn more, which is also quite good.


Add invalid helloworld to Cosmos test v4

This package aids in testing out the Circe Parsing errors.

Merging this.


Reset property button displayed for all non-default values

When opening an existing scene, properties of nodes that are not instanced (and are not children of instanced nodes) that have a non-default value display the reset property button. Clicking it has no effect, but it won't disappear either.

To reproduce:

Create a new scene
Add Node2D
alter transform/pos
Save & close
Reopen
Check transform/pos property

Seems to concern all properties that are non-default as per PROPERTY_USAGE_STORE_IF_NONONE and PROPERTY_USAGE_STORE_IF_NONZERO

This issue persists for the following types of properties:

properties of instanced nodes and their children
properties of non-base nodes in inherited scenes

Not sure if it's expected for the first, but the latter seems incorrect.

Is that not intended?
On Oct 17, 2015 7:34 PM, "eska014"<EMAIL_ADDRESS>wrote:

This issue persists for the following types of properties:

properties of instanced nodes and their children
properties of non-base nodes in inherited scenes

Not sure if it's expected for the first, but the latter seems incorrect.
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/2614#issuecomment-148956693.

I think this is expected behavior, if something is changed from original
scene, revert appears
On Sat, Oct 17, 2015 at 8:13 PM, Juan Linietsky<EMAIL_ADDRESS>wrote:

Is that not intended?
On Oct 17, 2015 7:34 PM, "eska014"<EMAIL_ADDRESS>wrote:

This issue persists for the following types of properties:

properties of instanced nodes and their children
properties of non-base nodes in inherited scenes

Not sure if it's expected for the first, but the latter seems incorrect.
—
Reply to this email directly or view it on GitHub
<https://github.com/okamstudio/godot/issues/2614#issuecomment-148956693
.

—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/2614#issuecomment-148959684.

--
OkamStudio

This example should reproduce what I mean:

Create a new scene (not inherited)
Add 1 Node
Save & close
Scene->New Inherited Scene
Choose just created scene as base
Add a Control node as child to the root in this new Inherited scene
Observe that margin/right, margin/bottom, rect/size, size_flags/horizontal, size_flags/vertical of the new Control node all have the button without any functionality.

I'm not sure if this intended. I don't think properties of nodes added in an inherited scene should have a revert button while editing that same inherited scene (since there is nothing to revert to).


CPU/heap profiling at runtime

Having access to CPU and heap profiles would be very useful while debugging production incidents. We should add support for taking such profiles at runtime.
It's probably too expensive to always enable profiling, but taking e.g. a 10-second profile on demand (via some API) would typically be sufficient.
We can either add profiling to the binaries themselves or explore integrating with some third-party observability service that supports taking such profiles.

Some options:

Grafana Cloud Profiles: continuous profiling. Wraps pprof-rs.
Polar Signals: continuous profiling. Uses pprof profiles.
pprof-rs: in-process profiler. No heap profiles.
cpuprofiler: in-process profiler. Supports pprof.

See also: neondatabase/cloud#14888

CPU and heap profiling is in place for Pageserver and Safekeeper, so I'll close this out.

https://github.com/neondatabase/neon/pull/9764
https://github.com/neondatabase/neon/pull/9778

The heap profiles aren't automatically symbolized, this will be handled in #9964. Continuous profiling is tracked in https://github.com/neondatabase/cloud/issues/14888.


alumacc: merge independent of sign

I think I'd want an equivalence test written for this to make sure that this transformation is correct.


Typo in report makes generation fail

mapped_reads_intrv = c(mean(qc_table$Mapped_reads_pct) - 2 * d(qc_table$Mapped_reads_pct),

I guess d should be sd.

sorry about that. Fixed and updated.


API: endpoint "settings" returns "Access denied or invalid endpoint."

When I choose the endpoint settings to request all the Bludit settings, I get the following response:
{"message":"Access denied or invalid endpoint."}

This error can be fixed by deleting && $writePermissions in line 159 in the file plugin.php.
158 // (GET) /api/settings
159 elseif ( ($method==='GET') && ($parameters[0]==='settings') && empty($parameters[1]) && $writePermissions ) {
160     $data = $this->getSettings();
161 }

Steps to reproduce the problem
curl -X GET  -G "https://example.net/api/settings"  -d "token=YOUR_VALID_API_TOKEN"
Bludit version
3.14.1 (as well 4.x)

You have to send the authentication token to get access to the settings.
{
    "token": "24a8857ed78a8c89a91c99afd503afa7",
    "authentication": "193569a9d341624e967486efb3d36d75"
}

curl  -X POST \
    -H "Content-Type: application/json" \
    -d @data.json \
    "https://www.example.com/api/settings"


How to expose the kubernetes-dashboard?

$ kubectl get svc --namespace kube-system
NAME                   TYPE        CLUSTER-IP       EXTERNAL-IP   PORT(S)                       AGE
heapster               ClusterIP   <IP_ADDRESS>   <none>        80/TCP                        9h
kube-dns               ClusterIP   <IP_ADDRESS>    <none>        53/UDP,53/TCP                 9h
kubernetes-dashboard   ClusterIP   <IP_ADDRESS>   <none>        80/TCP                        9h
monitoring-influxdb    NodePort    <IP_ADDRESS>    <none>        80:31002/TCP,8086:31003/TCP   9h

Currently the docs do not discuss how to access the kubernetes-dashboard. Could you please add a few lines to suggest how an operator can view this? Sorry that this might be a super new person/generic question.

@drnic there are several ways to do this:

Run kubectl proxy and access dashboard on <IP_ADDRESS>:/ui

Expose dashboard service using NodePort and access it via :

@alex-slynko I tried it as well with kubectl proxy and <IP_ADDRESS>:8001 then browsing to http://<IP_ADDRESS>:8000/ui but I instantly received a connection refused.

@drnic Is Alex's answer good for you? We don't plan on adding instructions on how to expose the dashboard in our documentation because it's a Kubernetes-specific question.
@jhiemer You mentioned two different ports, can you post the exact commands and output you're getting?

For reference: https://github.com/kubernetes/dashboard/wiki/Accessing-Dashboard---1.7.X-and-above

Thanks. I'll try to revisit this issue + all the helpful ideas as soon as I stop tearing down + rebuilding my kubo cluster over and over during https://github.com/cloudfoundry-incubator/kubo-deployment/pull/237

Looks like the issue is resolved for now. Feel free to re-open if the solution doesn't work.


Support Preview Java Language Levels

Checklist

[x] I have filed an issue about this change and discussed potential changes with the maintainers.
[x] I have received the approval from the maintainers to make this change.
[x] This is not a stylistic, refactoring, or cleanup change.

Please note that the maintainers will not be reviewing this change until all checkboxes are ticked. See
the Contributions section in the README for more
details.
Discussion thread for this change
Issue number: https://github.com/bazelbuild/intellij/issues/3080
Description of this change
Intellij project language levels may include variants of major Java
versions such as "17 (preview)". Today, the Bazel plugin
java_language_level option specified in in the Project View File
accepts an integer value which defines the major Java version to use.
As an integer, it is not possible to specify preview versions.
This changes adds support for specifying any Java language level by
specifying a string for java_language_level that is attempted to be
matched against the LanguageLevel enum value names (found in
com.intellij.pom.java.LanguageLevel). If no match is found, the
previous parsing logic is used to parse the given language version.
To enable Java 17 preview features, you can set:
java_language_level = jdk_17_preview

in your project view file.

Thank you @aarone!
I tried building the plugin with this change but when I try it with this project and the .bazelproject updated with
java_language_level: jdk_17_preview I get this error on sync even though I've jdk17
Error:Unable to find a JDK 17 (Preview) - Pattern matching for switch installed.
After configuring a suitable JDK in the "Project Structure" dialog, sync the project again.

hi @mai93  - thank you so much for reviewing this change!
I think this error occurs if the plugin can't find an installed JDK that provides the requested language level. I'm pretty sure that behavior of how SDKs are discovered by selected language level is unchanged by this change. For example, today, if you have no JDKs installed and set the java_language_level to '11' you'd get a similar error.

Thanks @aarone.
Yes, I get that but although I've JDK 17 installed I still get this error when I use java_language_level: jdk_17_preview. I think the problem is in our use of LanguageLevel$isAtLeast() as based on the order of the versions in LanguageLevel class, JDK17 is less than JDK17 preview. Does this make sense? Or there is another reason for why I'm getting this error even though JDK 17 is installed.

thanks for the info, i wasn't seeing the same error with newer jdks installed so didn't realize the nature of the issue; i'll look into it

Alright, i've made changes that should address the issue; thanks for your help and patience.

Thank you @aarone!


feat: /stats endpoint

This is supplementary to the /metrics endpoint, and arguably sort of pointless considering that exists, but I started building it for debugging and then realised it was kinda cool, so I built it out a little to make it more "production-ready".

Let's do this mergy


Azure SQL Database Managed Instance

It mentions in the limitations that elastic queries work for Azure SQL Database only. Could I just confirm whether or not this includes Azure SQL Database Managed Instance?

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

ID: 85bf6940-1cbe-b07c-aede-131130ceb616
Version Independent ID: 447635d9-5aea-f447-6cb4-8fd44ea5e900
Content: Elastic query overview - Azure SQL Database
Content Source: articles/sql-database/sql-database-elastic-query-overview.md
Service: sql-database
Sub-service: scale-out
GitHub Login: @MladjoA
Microsoft Alias: mlandzic

@aihaslett, yes, it does.
Not though that, if you are interested in the so-called vertical partitioning scenario of elastic query,  with Azure SQL Managed Instance you can use 3-part and 4-part names instead, as SQL Server's native solution for cross-database and cross-instance queries.

Thanks for the quick response. Sorry my initial question was a bit vague - specifically what I'm wondering is whether you can connect from a standalone Azure SQL Database, to a database running on Azure SQL DB Managed Instance, using elastic query?
As this article makes clear, you can't do this if you're connecting to an on-premises or VM SQL Server, but I was hoping that moving the database I'm trying to remotely connect to onto Managed Instance would solve the problem?
Thanks for the help.

@MladjoA Please check the follow up question from the customer here.

@aihaslett, elastic query between SQL DB and SQL MI does work if there is a connectivity between the two. Given that SQL MI is always injected into a Virtual Network and gets an IP address from customer's private address space, it means that public endpoint needs to be enabled on SQL MI for SQL DB to be able to connect to it.
So, once you enable public endpoint on your MI, this is how your external source would look like:
CREATE EXTERNAL DATA SOURCE MyElasticDBQueryDataSrc WITH
(TYPE = RDBMS,
LOCATION = 'yourmi.public.4fe59e585a5d.database.windows.net**,3342**',
DATABASE_NAME = 'Blah',
CREDENTIAL = YourEQCred,
)

Great, thanks for the explanation, much appreciated.
From: Mladen Andzic (Msft<EMAIL_ADDRESS>Sent: Tuesday, 19 May 2020 10:51 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: aihaslett<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Azure SQL Database Managed Instance (#54805)
@aihasletthttps://github.com/aihaslett, elastic query between SQL DB and SQL MI does work if there is a connectivity between the two. Given that SQL MI is always injected into a Virtual Network and gets an IP address from customer's private address space, it means that public endpoint needs to be enabled on SQL MI for SQL DB to be able to connect to it.
So, once you enable public endpoint on your MI, this is how your external source would look like:
CREATE EXTERNAL DATA SOURCE MyElasticDBQueryDataSrc WITH
(TYPE = RDBMS,
LOCATION = 'yourmi.public.4fe59e585a5d.database.windows.net**,3342**',
DATABASE_NAME = 'Blah',
CREDENTIAL = YourEQCred,
)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/54805#issuecomment-630475720, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOQU63WQPH2OWXH2V6IS3PDRSG3WPANCNFSM4NAEIGUA.

@MladjoA Thanks for the explanation.
@aihaslett Glad that this helped you. We are going ahead and closing this thread.


fix: deprecated bgp_base_cidr in direct link service

PR summary
deprecated bgp_base_cidr in direct link service
Fixes: <! -- link to issue -->
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

:tada: This PR is included in version 0.12.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket: