As a sequencer log viewer, I want to see which router a bid came from

Problem
Sequencer logs don't show which router has sent a bid. Will make it tough to debug version mismatches and other bid issues once more routers are approved.
Proposed Solution
Thoughts on solution design. We should do  to solve this problem.

[ ] Sequencer logs router address per bid

Acceptance Criteria
From an end tester and/or user perspective, the below criteria must be met to consider this done:

[ ] Can see which router a bid came from in sequencer logs

We've removed the version check anyways, but we should add the log anyways.


Add TimeCard to New-CsTeamsShiftsConnectionInstance and Set-CsTeamsSh…

Add TimeCard to New-CsTeamsShiftsConnectionInstance and Set-CsTeamsShiftsConnectionInstance
Signed-off-by: Adva Klinger<EMAIL_ADDRESS>

Docs Build status updates of commit e6727a9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Set-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit e82f436:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Set-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Someone with merge permission please merge this PR. Thanks

Docs Build status updates of commit 94b9ab6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Set-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@tiburd would you please help us merging this PR. Thank you :)

@get-itips @dariomws @tseward
Hi!
Will the last of you to review this PR please @ mention me so that I can merge it?
Thanks!

@get-itips, @dariomws, @tseward, please ping @tiburd so she can merge the PR. Thanks

Ping -t @tiburd. Could you merge it? Thanks!

Docs Build status updates of commit 5a68d60:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Set-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit f49fc12:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Set-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@advaklinger-ms
The PR cannot be merged until after you sign the CLA:

Please @ mention me after you sign it.
Thanks!

Hey @tiburd, I signed the CLA. please merge the PR

Docs Build status updates of commit 1687384:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Set-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Update logrotate conf

This seeks to change a couple of issues with logs growing to a size where we are alerted to a disk space issue on the server. More detail in commits.

Looks fine, and taught me about su, so merge away.

:+1: Nice work :eggplant:

I updated this because I realised that I was adding su for logs that are in /var/log which is incorrect.


Presence of cpg in shapefile causes TypeError

Expected behavior and actual behavior.
With a shapefile that comes with a .cpg file that contains the value "System", I encountered the following error when trying to read the file through geopandas. I am not 100% sure that the problem is not just caused by my setup, or is further upstream (i.e. in GDAL), but maybe someone here can at least shine some light on a possible cause for this apparent regression. (I read this dataset successfully on older setups, but with a different environment.)
If I remove the .cpg file, everything works fine.
Steps to reproduce the problem and work-around
$ git clone https://github.com/tum-ens/rivus
$ cd rivus/data/haag15
$ fio info building.shp
[TypeError, see below]
$ rm building.cpg
$ fio info building.shp
{"bounds": ...}

Error message
ERROR:fio:Exception caught during processing
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/fiona/fio/info.py", line 50, in info
    with fiona.open(input, layer=layer) as src:
  File "/usr/local/lib/python3.5/dist-packages/fiona/__init__.py", line 165, in open
    enabled_drivers=enabled_drivers)
  File "/usr/local/lib/python3.5/dist-packages/fiona/collection.py", line 153, in __init__
    self.session.start(self)
  File "fiona/ogrext.pyx", line 394, in fiona.ogrext.Session.start (fiona/ogrext2.c:8430)
  File "fiona/_err.pyx", line 183, in fiona._err.GDALErrCtxManager.__exit__ (fiona/_err.c:1809)
  File "fiona/_err.pyx", line 58, in fiona._err.CPLE_BaseError.__init__ (fiona/_err.c:1242)
TypeError: __init__() takes exactly 4 positional arguments (2 given)
Aborted!

Operating system
Ubuntu 16.04
Fiona version and provenance
Fiona 1.7.6 on Python 3, installed via pip3. fio --gdal-version reports GDAL, version 2.1.3, while a system gdalinfo --version tells me that I have GDAL 1.11.3 installed.
Is it possible that I just have a buggy setup? I would be glad if someone could confirm (or not), whether it's just my setup that is causing the problem.

With a shapefile that comes with a .cpg file that contains the value "System"

The .cpg files specify the codepage of the text fields in DBF file.  Just off the top of my head, I don't think "System" is a valid value.
dBase IV is one of the warts for supporting the shapefiles, and the format that predates UTF-8.  The .cpg file is a workaround.

I did your above $ fio info building.shp test with multiple versions of Fiona.
Fiona version 1.7.5 and before don't have the exception, it just works.  But versions 1.7.6 and 1.7.7 result in an exception.
ogrinfo doesn't seem to have any issues with building.shp.

@micahcochran: thank you for taking the time to reproduce my issue. Yes, I agree that System is not a particularly helpful (or valid) value in a .cpg file. Based on discussion to issue gdal#4920, I found a work-around for the command line:
SHAPE_ENCODING="UTF-8" fio info building.shp

In short: your suggestion for a bad codepage warning/exception might be a good approach for handling this problem here.

@ojdo
I refereshed myself on how GDAL handle shapefiles.  This may not be that helpful.
shapelib's dbfopen.c really doesn't seem to have any validation of the CodePage.  Shapelib just passes the CodePage information along.
GDAL has ogrshaplayer.cpp, which is GDAL's wrapper for shapelib.  There is CodePageConvert function that tries to make sense of what shapelib told it was the codepage.  If it can't figure it out, it just returns what it was given... perhaps  System in this case.

I think I've found the cause PR #444 updated CPLE_BaseError and as of merging that PR requires more arguments.
_err.pyx Line 185 raise exception_map[err_no](msg)
This does not appear to be providing enough arguments.
When I replace that line with this one
raise exception_map[err_no](err_type, err_no, msg)
It gives a better exception, but it probably should be a warning.
fiona._err.CPLE_AppDefinedError: b'Recode from System to UTF-8 failed with the error: "Invalid argument".'

@micahcochran good eye! Could you send me a PR to review?

@ojdo I'd be much obliged if you could try the new 1.7.8.

@sgillies: With 1.7.8, fio info now yields a much more informative error message:
ERROR:fio:Exception caught during processing
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/fiona/fio/info.py", line 50, in info
    with fiona.open(input, layer=layer) as src:
  File "/usr/local/lib/python3.5/dist-packages/fiona/__init__.py", line 165, in open
    enabled_drivers=enabled_drivers)
  File "/usr/local/lib/python3.5/dist-packages/fiona/collection.py", line 153, in __init__
    self.session.start(self)
  File "fiona/ogrext.pyx", line 395, in fiona.ogrext.Session.start (fiona/ogrext2.c:8432)
  File "fiona/_err.pyx", line 185, in fiona._err.GDALErrCtxManager.__exit__ (fiona/_err.c:1861)
fiona._err.CPLE_AppDefinedError: b'Recode from System to UTF-8 failed with the error: "Invalid argument".'
Aborted!

The message also propagates to geopandas (where I first encountered the error when trying geopandas.read_file('buildings.shp'). So :+1: from me and thanks!

now yields a much more informative error message:

As somebody not too known to shapefiles / gdal inner workings, the error message is not that clear to me :-) (is it a user error? is it a wrongly-written file? or a problem with one of the libraries?)
If you get something like that error message, what is the action you need to take to solve this?
In https://github.com/geopandas/geopandas/issues/629 we get "CPLE_AppDefinedError: b'Recode from CP437 to UTF-8 failed with the error: "Invalid argument".'"

@jorisvandenbossche sorry about the confusion. What's happening is that Fiona/GDAL detect that your shapefile is encoded using CP437 (from a value in a .cpg file, or maybe from your system's default encoding), that value is set on the dataset object, and then in reading features from the shapefile GDAL calls methods of libiconv to recode CP437 to UTF-8. The libiconv function fails, probably because the shapefile's encoding is not actually CP437. For some reason, "invalid argument" is all that CPLRecode() gets from libiconv. CPLRecode() then puts a GDAL error with the message "Recode from CP437 to UTF-8 failed with the error: x" and type "AppDefinedError" on the GDAL error stack. Fiona pops this error off the stack and turns it into a Python exception. A rather opaque one. Sometimes native Python programs emit UnicodeDecodeError from out of nowhere, and it can be confusing to be sure.
It's my intent that users never see CPLE_* exceptions, but we don't always succeed.
Because the GDAL error comes from deep in GDAL when you iterate over a collection's features and has a very generic type, we need to catch and handle CPLE_AppDefinedError in the iterator's next() method. In the handler we'll need to scrape details from the GDAL error message in order to raise a more helpful exception, which is gross but I don't see any way around that.
Exceptions raised during iteration are awkward, but scanning all features for coding errors before iteration will penalize users with proper data sources.

Thanks for the clear explanation!
What is not yet clear to me is for this specific case (https://github.com/geopandas/geopandas/issues/629, trying to read the geopandas included dataset on Ney York boroughs), is whether there is something wrong with the file or whether it is an installation issue. As depending on the environment, we see this error or not.
It's certainly possible that the shapefile we included in geopandas (or downloaded from NY's open data) has an error in the cpg file), but eg on travis (and locally also in a clean env) it is not failing.

importing the shapefile into QGIS and exporting as a new shp file fixed my issue.


Cant install Mediapipe <IP_ADDRESS>

OS Platform and Distribution
Windows 11 22h2
Compiler version
No response
Programming Language and version
Python 3.9.7
Installed using virtualenv? pip? Conda?(if python)
No response
MediaPipe version
No response
Bazel version
No response
XCode and Tulsi versions (if iOS)
No response
Android SDK and NDK versions (if android)
No response
Android AAR (if android)
None
OpenCV version (if running on desktop)
No response
Describe the problem
Ive looked everywhere and PyPI.org isn't letting me get the wheel to install Mediapipe version <IP_ADDRESS>. Is there someone who can help?
Complete Logs
I don't have any logs, as I haven't found a way to install it.

Hi @Prana-S,
We currently do not have MediaPipe version <IP_ADDRESS>. The latest available version on Nearby is 0.8.4, which can be verified on the GitHub release page. In the PyPI repository, the earliest version accessible is v8.7.2, as it was initially published through PyPI. Prior to this release, MediaPipe Python wheel packages were not provided.
Unfortunately, installing versions earlier than v8.7.2 using Pip is not possible. However, if your project relies on a specific MediaPipe version without available wheel files, you can still build the MediaPipe Python package from source using the provided guide here.
Thank you!!


Server-generated _id of inserted document

How do I get the server-generated _id of an inserted document that did not have a _id property? It is not set on the document object and does not appear to be available in the result object. Here is what I am doing:
var mongodb = require("mongodb-core");
var server = new mongodb.Server();
var doc = { foo: 1 };
server.insert("test.test", [doc], function(err) {
  // _id not available in result
  // doc._id not set
});

You cannot as the insert does not return the _id, This is the reason the driver generates it's own _id's. What are you planning to use mongodb-core for ? If you plan to use it for your application that's advised.

I see now that the mongodb wire protocol does not send back the _id at all. I will generate my own _id client-side. Thanks.


Generating sentences using the RNN model

How do we generate sentences with the trained rnn model in:
https://github.com/tensorflow/models/tree/master/tutorials/rnn/ptb
I found some people doing it here and there but nothing that is well documented with a working code, at least not ones that are compatible with TF 1.0
I found this answer on stackoverflow but I keep getting errors with it
http://stackoverflow.com/a/38729349/7568128
Here is a copy of the file I'm working with
ptb_word_lm2 (copy).txt

That's a great question for stackoverflow, where we monitor all questions with the tag tensorflow. We try to keep the github issues to bugs and feature requests only. Thanks.


Fixed issues with missing or uncertain bases in TajD calculation

Just an update to my previous pull request. I've realised there are some issues when missing or uncertain bases are present due to my previous changes. Just a quick update to fix this, with an appropriate test case as well.
Cheers,
Andrew

Merged in development-master, 0f86d44.

Thanks for keeping on top of this. Appreciated!

No worries!


Add descheduler annotations support

What this PR does
Add the descheduler annotations, allowing the compatibility with the Descheduler
Before this PR:
There is no compatibility with the descheduler since the way KubeVirt handles eviction requests causes the descheduler to make wrong decisions and take wrong actions that could destabilize the cluster.
After this PR:
KubeVirt adds the proper annotations to the virt-launcher pods so that the descheduler is correctly instructed on handling the eviction response and the virt-launcher pods.

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Design proposal:

https://github.com/kubevirt/community/pull/258
https://github.com/kubevirt/community/pull/295

Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[x] Design: A design document was considered and is present (https://github.com/kubevirt/community/pull/258,https://github.com/kubevirt/community/pull/295 ) or not required
[x] PR: The PR description is expressive enough and will help future contributors
[x] Code: Write code that humans can understand and Keep it simple
[x] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[x] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[x] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[x] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[x] Community: Announcement to kubevirt-dev was considered

Release note

Add descheduler compatibility

/test all

/test all

/cc

Thank you @jean-edouard for your review!
I addressed your comments, but one.
I also added some refactoring commits reducing the cycle dependency I found. Please let me know, thanks

/approve
My original comments were addressed and the refactoring looks great!
Just one thing I think could be improved is the release note. I needs to more user-focused, i.e. we don't break descheduler anymore.
Thank you!

/approve My original comments were addressed and the refactoring looks great! Just one thing I think could be improved is the release note. I needs to more user-focused, i.e. we don't break descheduler anymore. Thank you!

Thank you for the suggestion. Edited with: Add descheduler compatibility

/lgtm
Thanks,  Great job!

/hold
Feel free to unhold once all comments are addressed

As requested:

Re-added test operation during annotation removal
Changed descheduler function names
Added kubevirt as value for the eviction in progress annotation

@acardace @Barakmor1 PTAL
Thank you!

/retest

@fossedihelm @acardace I'm not sure this will merge with the coveralls check failing.

@fossedihelm @acardace I'm not sure this will merge with the coveralls check failing.

Actually I don't know why this causes this coveralls coverage decrease...

/lgtm

Added controller unit test, trying to help coverall

/lgtm

The huge decrease in coverage is due to constants and utility functions being moved out of vmi.go, there is no actual change in code coverage.
I don't think this failure should block this PR, it really has no value.
@xpivarc, @brianmcarey if you agree I'd override the the coveralls lane for this one time.

The huge decrease in coverage is due to constants and utility functions being moved out of vmi.go, there is no actual change in code coverage.
I don't think this failure should block this PR, it really has no value.
@xpivarc, @brianmcarey if you agree I'd override the the coveralls lane for this one time.

Reduction of coverage hints about the potential degradation of code-quality. 1.3% is a lot.
If code moved out and lost coverage, it is a real problem. This usually happens when coverage is implicit and not explicit. But implicit is better than none.
Perhaps you should reconsider the refactoring part and split it to a follow up PR.

Looks like https://github.com/kubevirt/kubevirt/pull/12100 coverage is not complaining, actually is increasing...
Basically it drops the https://github.com/kubevirt/kubevirt/pull/12025/commits/a7556cdc7cf1676af8ce2d834e8ff504e1a79480 commit

Go figure, looks like the tool just got in the way this time around.

/lgtm

/test pull-kubevirt-e2e-k8s-1.28-sig-storage

/retest


Remove CR Carriage Return and LF Line feed

Hi,
I am working with EDI.Net for the first time and have a question:
in the example:
var grammar = EdiGrammar.NewTradacoms(); var interchange = new Interchange(); // fill properies  interchange.TransmissionDate = DateTime.Now; ... // serialize to file. using (var textWriter = new StreamWriter(File.Open(@"c:\temp\out.edi", FileMode.Create))) {     using (var ediWriter = new EdiTextWriter(textWriter, grammar)) {          new EdiSerializer().Serialize(ediWriter, interchange);     } }
there is a Carriage Return and Line feed after every ' in the out.edi file... is it possible to write it in a file without CR,LF?

Hi @tomtom4848 you can set the Formatting property on the EdiTextWriter class to Formatting.None instead of the default which is line per segment

That worked great! Thank's @cleftheris


Combobox options not displaying when dot used in search input

Describe the bug
In v5, when using the  Combobox with a list containing users, searching using the users name works as expected. When attempting to search with an email address, the dot character does not seem to be accepted by the Combobox showing no results.
How to reproduce
Create a list with a property that contains a dot. Set the Combobox field to that property. Search for the value in the combobox and no results will be shown in the results dropdown.
Search user name

Search users email

Expected behaviour
Options display all list items it is linked to.
Versions
Clarity project:

[x] Clarity Core
[x] Clarity Angular/UI

Clarity version:

[x] v5.0.1

Framework

[x] Angular

Framework Version
Angular 11

I don't see a reproduction and I'm unable to reproduce it locally.
Though, I can make some assumptions:
You probably have different data and display value. The filter works on the data, not the display value.
You probably also have field specifier on the *clrOptionItems iterator. If you have this specifier (which is optional), it will only be searching in the specified field. If you skip it, the combobox will search/filter in all item property values.

HI Jinnie, thanks for the response, I've tried removing the field specifier as you've said but still get the same issue, am I missing something? It looks good to me. I've checked the Clarity source code and don't think I'm missing anything.
  <clr-combobox-container>     <clr-combobox       clrMulti="true"       [(ngModel)]="searchedUsers"       (clrInputChange)="lookupUser($event)"       [clrLoading]="isLookingUpUsers$ | async"     >       <ng-container *clrOptionSelected="let selected">         {{ selected?.name }}       </ng-container>       <clr-options          <clr-option *clrOptionItems="let searchedUser of searchedUsers$ | async;" [clrValue]="searchedUser">           {{ searchedUser.name }} ({{ searchedUser.emails}})         </clr-option>       </clr-options>     </clr-combobox>   </clr-combobox-container>
SearchedUser:
export interface SearchedUser {   id: string;   active: boolean;   email: string; }

I created a stackblitz, based on your code and it seems to work ok.
https://stackblitz.com/edit/clarity-light-theme-v5-uhzzjs?file=src%2Fapp%2Fapp.component.ts
Please try to reproduce the problem using the above link.

Thanks, issue is resolved now. The Stackblitz is much appreciated.

Great to hear that, thanks for the quick feedback


Failed to initialize the Crashlytics SDK

Please fill in the following fields:
Unity editor version: 2020.1.1f1
Firebase Unity SDK version: 6.15.2
Source you installed the SDK (.unitypackage or Unity Package Manager): from firebase website
Firebase plugins in use (Auth, Database, etc.): crashlytics only
Additional SDKs you are using (Facebook, AdMob, etc.): nothing
Platform you are using the Unity editor on (Mac, Windows, or Linux): Windows
Platform you are targeting (iOS, Android, and/or desktop): Android (UCB)
Scripting Runtime (Mono, and/or IL2CPP): Mono
Please describe the issue here:
Compact Logs:- https://docs.google.com/document/d/1L1o0DMJquwM856T_rhfaAg2eIfIz64X08mmJ1ZZId54/edit?usp=sharing
Full logs:- https://docs.google.com/document/d/1Hi1jECTAvRuBmI0ZGhsw6mSyJnjEST250qNEUITpLtY/edit?usp=sharing
Hi, I built Crashlytics mainscene over Unity Cloud Builds. (Android)
I manually imported the crashlytics unitypackage downloaded from firebase website and had done all the basic configurations done. After installing it into my Android phone I was able to view my user count go 0 to 1 in the firebase console but the crashlytics console wasn't being initialised. I tried crashing the app manually first, didn't saw any changes in the console. And then I tried customised non-fatal crashes by using scripts on firebase docs. Even then, the console didn't initialise.
Below are the photos I saw on the console.

Please help me out as this issue had cracked me up for almost a month. :/
Please answer the following, if applicable:
Have you been able to reproduce this issue with just the Firebase Unity quickstarts (this GitHub project)?
I tried building the scene giving in this GitHub repository, even that didn't help.
What's the issue repro rate? (eg 100%, 1/5 etc)
What is repro rate? I'll fill this later when I get to know its meaning.

This issue does not seem to follow the issue template. Make sure you provide all the required information.

Where did I missed? :(

@patm1987 help needed. 🙏

Thanks - could you try replacing the code you're using to test crash the app with the following:
Utils.ForceCrash(ForcedCrashCategory.Abort);
And see if you're able to get past the onboarding screen with that?

@kevinkokomani Correct me if I'm wrong, I've to replace this with line number 38. Right?
Also CrashlyticsTester.cs is not my code, that's taken from Firebase docs https://firebase.google.com/docs/crashlytics/test-implementation?authuser=0&platform=unity

I believe line 38 is correct. You followed the instructions correctly, I just wanted to check if that would help you get past the onboarding screen - I've seen this help one other developer in the past.

I did a cloud build and got these following logs.
ERRORS:
[Unity] -----CompilerOutput:-stdout--exitcode: 1--compilationhadfailure: True--outfile: Temp/Assembly-CSharp.dll
[Unity] Assets/CrashlyticsTester.cs(39,12): error CS0103: The name 'Utils' does not exist in the current context
[Unity] Assets/CrashlyticsTester.cs(39,29): error CS0103: The name 'ForcedCrashCategory' does not exist in the current context
[Unity] Assets/CrashlyticsTester.cs(39,12): error CS0103: The name 'Utils' does not exist in the current context
[Unity] Assets/CrashlyticsTester.cs(39,29): error CS0103: The name 'ForcedCrashCategory' does not exist in the current context
Failed to start server: Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:10010
[Unity] -----CompilerOutput:-stdout--exitcode: 1--compilationhadfailure: True--outfile: Temp/Assembly-CSharp.dll
[Unity] Assets/CrashlyticsTester.cs(39,12): error CS0103: The name 'Utils' does not exist in the current context
[Unity] Assets/CrashlyticsTester.cs(39,29): error CS0103: The name 'ForcedCrashCategory' does not exist in the current context
[Unity] Assets/CrashlyticsTester.cs(39,12): error CS0103: The name 'Utils' does not exist in the current context
Truncated to 10 lines.
[Unity] Initialize engine version: 2020.1.4f1 (fa717bb873ec)
[Unity] -----CompilerOutput:-stdout--exitcode: 1--compilationhadfailure: True--outfile: Temp/Assembly-CSharp.dll
[Unity] Assets/CrashlyticsTester.cs(39,12): error CS0103: The name 'Utils' does not exist in the current context
[Unity] Assets/CrashlyticsTester.cs(39,29): error CS0103: The name 'ForcedCrashCategory' does not exist in the current context
[Unity] Assets/CrashlyticsTester.cs(39,12): error CS0103: The name 'Utils' does not exist in the current context
[Unity] Assets/CrashlyticsTester.cs(39,29): error CS0103: The name 'ForcedCrashCategory' does not exist in the current context
UPM Server already running, skipping...
[Unity] Initialize engine version: 2020.1.4f1 (fa717bb873ec)
[Unity] -----CompilerOutput:-stdout--exitcode: 1--compilationhadfailure: True--outfile: Temp/Assembly-CSharp.dll
[Unity] Assets/CrashlyticsTester.cs(39,12): error CS0103: The name 'Utils' does not exist in the current context
[Unity] Assets/CrashlyticsTester.cs(39,29): error CS0103: The name 'ForcedCrashCategory' does not exist in the current context
[Unity] Assets/CrashlyticsTester.cs(39,12): error CS0103: The name 'Utils' does not exist in the current context
[Unity] Assets/CrashlyticsTester.cs(39,29): error CS0103: The name 'ForcedCrashCategory' does not exist in the current context
publishing finished successfully.
Do you think this is because of something like missing namespace?

@midopooler
First of all, if you are building Android App using Unity 2020, take a look of this blog post.
https://firebase.googleblog.com/2020/08/firebase-compatibility-with-unity-20201.html
It is unfortunately painful but all the fixes should be coming in the next release.
One note is that in Crashlytics Support section, instead of modifying launchTemplate.gradle, you can just rename Assets/Plugins/Android/Firebase to Assets/Plugins/Android/Firebase.androidlib. Make sure you see crashlytics_bundle_id and google-services.xml under Assets/Plugins/Android/Firebase.androidlib/res/values/
Second, AFAIK, you cannot hard-crash Unity Android app because Unity will handle all the exceptions.  If you really want to hard-crash Android App, like for testing, throw a Java exception from a non-main-thread.


Update URLs to use https://

Codecov Report

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

@@          Coverage Diff          @@
##           master    #49   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines         136    136
  Branches       28     28
=====================================
  Hits          136    136

Continue to review full report at Codecov.

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


remove ActiveModel::Errors#get deprecation for Rails 5.1

https://github.com/joecorcoran/judge/issues/80

Tests are passing in Travis, it looks like the failures there are on some post-test dependency.


truncate begin #18

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Tests pass?
✔

PR related to issue #18

@samdark I updated the PR :)

Thanks!


stateSave for reorder and resize not working

Hello @jeffreydwalter ,
Thank for this plugin. It works fine except the stateSave part for me. Please see the fiddle below.
https://jsfiddle.net/t5L29opg/3/
It doesn't save the colorder and colSize. Please let me know if you need more information.
Thanks again !
-Mag

@ruby4mag, I don't really understand what you were trying to show with the fiddle. Please take a look at the datatables documentation. That's where the state saving is done: https://datatables.net/examples/basic_init/state_save.html

Hi @ruby4mag and @jeffreydwalter ,
I am also trying to use stateSave with ColReorderWithResize, column reorder and resize works perfectly but its state is not saved when refreshing. Did you find the solution?
Thank you,
Rosa

Hi,
that would be a great addon, allthough the author mentioned that he is not going to maintain/improve this plugin any further.
@ruby4mag, @rosaricla
Any updates on this topic?

I don't use this plugin personally anymore, but I'm happy to accept pull requests.

I don't use this plugin personally anymore, but I'm happy to accept pull requests.

I will try.

Hello @jeffreydwalter  @tefod-zz
Could you tell me if you have any update for stateSaving for resize and reorder column?
Thank you very much for the excellent plugin

Hello @jeffreydwalter  @tefod-zz
Could you tell me if you have any update for stateSaving for resize and reorder column?
Thank you very much for the excellent plugin

Not yet.
Saving the state (size and order) of remapped columns and loading them on table initialisation is principally not that hard.
But I stopped using column reordering, hence this affects the whole style on responsive table with hidden (child rows) columns. Together with searchPanes (if you use it) its getting a mass, if you reorder columns.
There a major changes necessary for letting this work without breaking the design. So, i have to do without it :)

Not yet.
Saving the state (size and order) of remapped columns and loading them on table initialisation is principally not that hard.
But I stopped using column reordering, hence this affects the whole style on responsive table with hidden (child rows) columns. Together with searchPanes (if you use it) its getting a mass, if you reorder columns.
There a major changes necessary for letting this work without breaking the design. So, i have to do without it :)


JointIndex values

Hi,
Sorry for this stupid question but I tried to check all the JointIndex values from
models/simple_humanoid.urdf
For this I modified a test in value.hpp, the code is here:
https://github.com/olivier-stasse/pinocchio/blob/topic/JointIndexPb/unittest/value.cpp#L193
The result is the following:
model.parents.size():32
model.njoints:32
l_wrist:24
r_wrist:38
model.parents.size():31
model.njoints:31
CHEST idTest:31
WAIST_P idTest:27
WAIST_R idTest:29
LARM_WRIST_R idTest:45
LARM_WRIST_P idTest:43
LARM_WRIST_Y idTest:41
LARM_ELBOW idTest:39
LARM_SHOULDER_Y idTest:37
LARM_SHOULDER_R idTest:35
LARM_SHOULDER_P idTest:33
LLEG_ANKLE_R idTest:13
LLEG_ANKLE_P idTest:11
LLEG_KNEE idTest:9
LLEG_HIP_Y idTest:7
LLEG_HIP_P idTest:5
LLEG_HIP_R idTest:3
RARM_WRIST_R idTest:59
RARM_WRIST_P idTest:57
RARM_WRIST_Y idTest:55
RARM_ELBOW idTest:53
RARM_SHOULDER_Y idTest:51
RARM_SHOULDER_R idTest:49
RARM_SHOULDER_P idTest:47
RLEG_ANKLE_R idTest:25
RLEG_ANKLE_P idTest:23
RLEG_KNEE idTest:21
RLEG_HIP_Y idTest:19
RLEG_HIP_P idTest:17
RLEG_HIP_R idTest:15
You can see that r_wrist and several other joints are above the number of joints.
Is it normal ?
If yes, then how can we use the field model.parent ?
Thanks for your time and your help.
Olivier.

In fact, you asked the id of the frame attached to the joint and having the same name. You should retrieve the Frame from model.frames[id] and get the parent with model.frames[i].parent (which corresponds to the parent joint).

@olivier-stasse told me that the problem has been solved.


Is Oracle JRE free to use (for commercial purposes) with Self-Hosted Integration Runtime?

Is Oracle JRE free to use (for commercial purposes) with Self-Hosted Integration Runtime?
This is required to work with Parquet file using Self-Hosted Integration Runtime
Thank you

@eperfilyev  Please help with the link to the document on which the above document feedback request is created.
In case your query is not related to document content, open a new forum thread in Microsoft Q&A or Stack Overflow as it could be better place to get help on your scenario. These forum community members could provide their expert guidance on your scenario based on their experience. Thanks.

@eperfilyev Since we have not heard back from you 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.


gulp autocompletion broken

function $$gulp_completion {
    compls=$(gulp --tasks-simple 2>/dev/null)

    completions=(${=compls})
    compadd -- $completions
}

compdef $$gulp_completion gulp

If I I manually type in compdef $$gulp_completion gulp then it works.
I added the following lines at the end:
echo g: $_comps[gulp]
which compdef

This gives me:
g:
compdef () {

}

Which was a little bit surprising.
I am using antigen.
I am wondering if the gulp plugin is using a legacy mechanism for enabling completions?

Sadly, I have the same problem with golang plugin.

The function name used $$ which expands to the process ID of the shell. I've pushed a fix that should work for you. The golang plugin is a separate thing, open a new issue for it.


airflow dlt helper (dlt operator)

Background
We want to make running dlt pipeliens with airflow as easy as possible in a manner similar to dbt helpers. Users should be able to add dlt pipelines to DAGs.
The idea is to create Airflow operator for dlt pipelines that will contain all required boilerplate.
The ideal operator would: (1) decompose the dlt pipeline into several DAG nodes (2) enable reading secrets from Airflow hooks (3) configure logging and notifications of dlt.
PoC of the operator is here: https://github.com/dlt-hub/airflow_operator/tree/draft
Tasks

[x] implement a simple airflow operator that executes the whole pipeline as a single task

[ ] the helper function should accept pipeline instance, source instance, some additional settings (ie. if to redirect dlt logger into airflow logger #167 ) + additional parameters that airflow may need

[x] it should return Airflow artifact (Operator/TaskGroup) that users can place in their DAGs

[ ] helper will become part of our Airflow deployment #140

[x] the airflow code should be placed in dlt/helpers/airflow

[x] dlt will not install airflow by default so please use MissingDependencyException to inform user on missing dependencies

Tests
We need tests in tests for this. The test should assume working local airflow instance. The test will be executed on github CI in separate workflow

[x] implement tests for a few DAG cases, preferably using duckdb to verify loaded data

[ ] add definiton for github actions workflow that spins up airflow and executes tests

https://github.com/marketplace/actions/airflow-dags-test could be useful to setup local airflow instance on github CI

we will not need an airflow instance


Add this step

Connect you phone to your PC via USB. I had to plug my phone into my PC's onboard USB port, not my USB hub as this did not work
You should be prompted to trust your PC (trust it)
Fire up Visual Studio
Open your solution and your device should now be available.

@CobyT7 is this in relation to a specific page in the docs?


[question]: about require this package

Hi there.
this is my code:
const Migrate = require('onetable-migrate')

Migrate just a object.I can not use it like this: "new Migrate({...})"
Right?Your readme.md show us some code like that: " const xx = new Migrate()".
But, Migrate is a object,I gotto use the default property.
const Migrate = require('onetable-migrate').default

My English is poor.Maybe my word is hard to understand.
Thanks!

Yes, you are right. Currently it is using an "export default class Migrate"
So you need to import with
const Migrate = require('onetable-migrate').default

or
import Migrate from 'onetable-migrate'

We are going to fix this in 0.7.1 so that you won't need the ".default".
It will then look like:
const {Table} = require('dynamodb-onetable')
const {Migrate} = require('onetable-migrate')

or
import {Table} from 'dynamodb-onetable'
import {Migrate} from 'onetable-migrate'

this

@mobsense thanks.I know.
😂
I just want to use:
const Migrate  = require('onetable-migrate')

I just thinks the ".default" is ugly.
So, can we export default class like before?
and write code like this:
// commonjs
const Migrate = require('onetable-migrate')
// or
// esm
import Migrate from 'onetable-migrate'

source code contains default export class
😁

We prefer
const {Migrate} = require('onetable-migrate')
for 2 reasons:

Consistency with `const {Model, Table} = require('dynamodb-onetable')
It provides an upwards migration path if we need to export other classes

Migrate is not a plain object. It is an ES6 class.
So this solution isn't as simple as your const Migrate = require..., but ESM is the future.
Let me know if you have strong push back or if I'm missing something.

thanks.


Why evt.from is undefined in AngularJS

Why evt.from is undefined in Angularjs.
In my Angular controller I have:
angular.module('myApp', ['ng-sortable'])
    .controller('demo', ['$scope', function ($scope) {
        $scope.items = ['learn Sortable', 'use gn-sortable', 'Enjoy'];
        $scope.secondList = ['one' , 'two', 'three'];

        $scope.from = null;

        $scope.conf = {
          'animation': 150,
          'group': 'shared',
          'onAdd': function(event) {
            alert(event.from);
            console.log(event.from);
            $scope.from = event.from;
          }
        };
    }]);

Live demo: http://jsbin.com/debevojito/1/edit?html,js,output

// Element is dropped into the list from another list
onAdd: function (/**Event*/evt) {
    var itemEl = evt.item;  // dragged HTMLElement
    evt.from;  // previous list
    // + indexes from onEnd
}

use onSort instead

Hi @KuroNoDev
I had mistake with Live demo.
Please see my implementation: http://jsbin.com/pehacujamo/1/edit?html,js,output
// Element is dropped into the list from another list
onAdd: function (/**Event*/evt) {
    var itemEl = evt.item;  // dragged HTMLElement
    evt.from;  // previous list
    // + indexes from onEnd
}

This implementation working only if you use jQuery. For Angular evt.from is undefined.

@ddanailov-nmdp ah sorry about that. Yeah, just also realized that ng-angular is returning a different set of data.
What are you trying to achieve? Should it be possible without needing to pass DOM elements.

Hi @KuroNoDev,
I have two groups with elements. Groups are Group A and Group B.
Group A follow elements:

learn Sortable
use gn-sortable
Enjoy

Group B follow elements:

one
two
three

If I move 'two' from Group B to Group A my Group A will be:

learn Sortable
two
use gn-sortable
Enjoy

To find which is original group I should iterate all elements and via matches to find this.

Group A has 3 matches (learn Sortable, use gn-sortable, Enjoy).
Group B has 1 match (two)

Ok in this case: Original group is Group B and current group is Group A.
Will be nice if this information is available in event object.


Iterated expectations in RiskyContrib

There is a speed-up trick that can be used in portfolio problems when income is independent of the risky return. This is already incorporated in ConsPortfolio, but I had not applied it to RiskyContrib.
The trick is to use iterated expectations when computing the expectation of value (and derivatives) next period. The idea is to create a function that interpolates E[f(income, R)|R] and then integrate it over R, instead of jointly integrating over income and R. Turns out that---in this model---this change makes the solution 3x-5x faster.
This PR introduces this trick into RiskyContrib's solver. I left the general solver for cases where income and returns are correlated as an option.

[x] Tests for new functionality/models or Tests to reproduce the bug-fix in code.
[x] Updated documentation of features that add new functionality.
[x] Update CHANGELOG.md with major/minor changes.

Since the REMARK has now been published, I took the opportunity to edit the file header and link to it.

Massive thanks to @alanlujan91, as I decided to implement this trick after reviewing his code for the housing project.

Codecov Report

Merging #1058 (22d4337) into master (10fe1ef) will increase coverage by 0.13%.
The diff coverage is 97.18%.

@@            Coverage Diff             @@
##           master    #1058      +/-   ##
==========================================
+ Coverage   71.71%   71.85%   +0.13%
==========================================
  Files          66       66
  Lines        9989    10042      +53
==========================================
+ Hits         7164     7216      +52
- Misses       2825     2826       +1

Impacted Files
Coverage Δ

HARK/ConsumptionSaving/ConsRiskyContribModel.py
79.42% <96.72%> (+1.64%)
:arrow_up:

...sumptionSaving/tests/test_ConsRiskyContribModel.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

@mnwhite are you able to review this?


Make use of new term deploy-on measurements

A list of additional measurements taken during capture of the animal at the start of the deployment. Recommended best practice is to define units and use a key:value encoding schema for a data interchange format such as JSON.


Add support for S-Series MK2 speed-sensitive knobs

What does this PR change?
Adds support to use the speed-sensitive knobs on S-Series keyboards.
Links to Issues PR addresses or fixes

Fixes #19

How was this PR tested?
Tested with a Komplete Kontrol Mk2 S49. I can't test with other keyboards to verify that adding the sensitivity to the adjustMixer() function will not cause issues. But, following the logic inside mixer.py, I don't think there should be any problems.

First of all, thanks for contributing!
The issue you have been experiencing with much bigger increment values than expected during your testing is that you haven't applied the modification in the right place. Notice that all the code for the knobs and the mixer is located at line 300 in controller_definition.py regardless of the controller, which in combination with your changes make it so that if a S-Series keyboard is used then the script will adjust the same value twice (one as a part of Core.OnMidiMsg() and another one as a part of S_SeriesMK2.OnMidiMsgAdd()). Solving that should make the adjustments act the same way as before, so no need for the behavior divergence on the code that normalizes the speed by dividing by 15 or 10 depending on the action.
With that said, there's no need whatsoever to fragment the code for the knobs so that it acts differently depending on the controller used. Instead of that, you could change the declaration of adjustMixer() so that sensitivity: int = None becomes speed: int (making it obligatory) in and feed the argument with:
# Increase
event.data2

# Decrease
nihia.mixer.KNOB_DECREASE_MIN_SPEED - event.data2 + 1

And then, to convert it to a 0-1 range inside the function you can just do speed / 63. But even with all of that it wouldn't "feel" right, as people tend to be more conscious of their movements when they try to do more meticulous movements than when doing faster or more general ones. This would make a linear value mapping to seem inconsistent and unresponsive to the way people turn their knob. In order to get an increment/decrement offset value that properly responds to the way people are turning the knob, you need a numeric expression that treats changes in the lower spectrum of possible values as more relevant that changes in the higher spectrum, being logarithms the perfect thing use here.
# Volume
config.KNOB_INCREMENTS_VOL * log(speed, 63)

# Panning
config.KNOB_INCREMENTS_PAN * log(speed, 63)

In the expressions above, log(speed, 63) will result in 1 if speed = 63 and make speed changes in the lower end much significant that in the higher end of the 0-63 range. config.KNOB_INCREMENTS_<param> will define the maximum value the offset can be.
I'll be reviewing the commits using the GitHub revision UI for pull requests so I can mark all the changes more clearly, but that should be everything! 😄

I will add the documentation annotations and remove that leftover parameter default. If you are interested in keeping the commits short, could we squash when merging rather than rewriting the git history?
I would love to dive into the "sensitivity" subject, as that problem took me a while to tackle, and I'm open to discussing my solution. While I think something log-based works on the surface, consider how clockwise and counterclockwise rotations get calculated. More specifically, with the counterclockwise spins, just going off of log by itself became tricky. After a bit of tinkering with different graphs, I came up with, y = abs(-1/x-64):

I tested those changes here https://github.com/hobyst/flmidi-kompletekontrol/pull/20/commits/7a91a031f4e519c4278140d8336ce50af7dcf43d#diff-b08e51789bb8c1a5a312a819c3fbf2bdf8413f40d7549cac4d60cc4493e3c5a5L439-R440, and I found that when calculating sensitivity this way, I get fine-tune controls when I want them, but I also don't feel held back by the knob, either. Side note, once I finally got this calculation working, I could tell when my "Shift" button was kicking in, as the precision dropped down to what I think are only sensitivity values 0-9 and felt VERY smooth.
You made an excellent point about other models and how they will use the sensitivity value. The calculation sensitivity = abs(-1 / (speed - 64)) where the keyboard sends a value speed = 63 or speed = 65, will result with sensitivity = 1. So think that this solution will work for these models.

If you are interested in keeping the commits short, could we squash when merging rather than rewriting the git history?

Being honest, I could talk for weeks about all the possible policies when it comes to commit history organization 😂. I personally prefer to keep commits as independent code attributions that can be reversed and/or dropped independently so in case regressions happen debugging becomes more easy. In this case
When drafting and implementing code every single person has different approaches to it, so while I don't really care about how others integrate the concept of commits on their workflow, it does matter when merging pull requests. In this case doing a squash and merging would do it since you already amended the issues from the first commits on the last ones and would make sense to make all of this a single commit since the code modification isn't that big, but in other cases I might consider it better to keep some things separated from each other in different commits.

I would love to dive into the "sensitivity" subject, as that problem took me a while to tackle, and I'm open to discussing my solution.

Yeah, that's exactly the main reason why I haven't had implemented it myself. The whole sensitivity calculation is rather complex to tackle since it does not only involve getting to pick the right calculation algorithm but also comparing it to what it actually feels like. I don't have an S-Series MK2 keyboard so for me this whole thing would have been me constantly sending mathematical functions to someone else that actually has the keyboard to get their feedback, everything but seamless 😅.
The function looks good, although to get more feedback I'll be sending the patch to someone else I know that has the keyboard to get their opinion on the overall feel of the knobs and reply you back with their thoughts.

After testing it for a while with with some people, I've now uploaded the updated controller_definition.py file to the thread for the script in the Image-Line forum to get a more general feedback before merging the pull request.

Sorry for taking so long to manage the PR, but I completely forgot about this one after waiting for people to share feedback on the FL Studio forums.
After uploading the patched controller_definition.py to the FL Studio forums, it wasn't until July 2023 that someone tried the patch and gave their approval. Because of the lack of feedback on the forums, I'll assume either too little people have tested it and/or those who tried it didn't have anything negative to say about it so I'll merge the PR as it is right now.
Thanks for the contribution!


complex regex

I can do
^[0-9][0-9][0-9][0-9][0-9]*\.md$
but with
^[0-9]{4,5}\.md$
I get a "Unclosed counted closure near index" error

To fix the error, you can try escaping the curly braces with backslashes (\{ and \}) like this: ^[0-9]\{4,5\}\.md$. This will ensure that the curly braces are treated as literal characters and not as part of the regex syntax.
Here's the updated regex pattern: ^[0-9]\{4,5\}\.md$

@hiteshbedre what kind of string do you think I'm trying to match?


Sword parry mod compatibility

https://beta.curseforge.com/minecraft/mc-mods/sword-parry
It looks like it uses normal model files to add block states to the model. I am not sure how I would hack that into the system, because I believe that would require some really hacky configurations.

This seems to work perfectly fine without any modifications.


Properly handle SETUP frames with RESUME_ENABLED

Properly parse SETUP frames with resumption but no support internally besides that.

I tried writing a integration test to verify that we can reject the SETUP frame but that whole functionality seems to be broken, see https://github.com/ReactiveSocket/reactivesocket-java/issues/234


Frontend: Edit personal information

User story

As a {HR-employee}
I want / need {to edit personal information of my connected DIs}
So that {changes in basic personal data (at the current status: name, surname, email address) can be adjusted without creating a new DI}

Acceptance criteria

On page Overview of DIs: "edit" button at each entry which triggeres an edit pop-up window
Edit pop-up window:

= form pre-filled with the basic personal data (of the corresponding DI) from the database
possibility to change entries and save them via clicking on a save button (-> request to backend)
cancel button which discards changes and closes pop-up window

Definition of done (DoD)

Added only after week 5
The same for all features
Here goes the project specific part

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

Finished in sprint 5: 8b9cdf974b0592bb76a4d87a3d35fe9834c38272


Update symbolic-link-programming-considerations.md

Corrected reparse point limitation from 31 to 63 and added Win2003/XP limit of 31.  ref: https://github.com/MicrosoftDocs/win32/blob/docs/desktop-src/FileIO/reparse-points.md

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

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

@msatranjr @v-kents @drewbatgit @mijacobs @DCtheGeek did I do it wrong?

Thanks for your contribution! I've merged your changes and they should be live soon.


feat: Add multiple intra route tables

Description
This adds multiple intra route tables per the number of intra subnets deployed.
Motivation and Context
This resolves an issue where multiple intra route tables are required such as when using them in a multiple availability zone inspection vpc.  Each intra subnet needs to forward to the particular network firewall endpoint which is in the local availability zone
Breaking Changes
There are no breaking changes as this only creates the same number of route tables as intra subnets.
How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects
I have tested by running the seperate-route-tables example in this branch and outputs return everything as expected to include the multiple route table ids.
[ ] I have executed pre-commit run -a on my pull request

Very intrested with this feature, we are facing the same issues to manage routing NFW endpoints per AZ!


feat: add title to BarChartRodStackItem

Closes #1371

It would be better if we keep a generic type of data (not only strings). For example, we can keep a dynamic type which could be anything

Thank you😀, I have made the changes.

It would be better if we keep a generic type of data (not only strings). For example, we can keep a dynamic type which could be anything

Thank you😀, I have made the changes.

It would be better if we keep a generic type of data (not only strings). For example, we can keep a dynamic type which could be anything

Thank you😀, I have made the changes.

title is not a good name anymore as it is a dynamic field.
I suggest renaming it to something like payload or data.
Also, you need to rebase your branch and write some unit tests for it (read our contributing guidelines carefully).

Closing, because it has no activity for a while


Cookie sharing between applications

I've attempted to implement cookie sharing between applications.
I followed the sample here - http://docs.asp.net/en/latest/security/data-protection/compatibility/cookie-sharing.html
It however doesn't appear to work.  I'm using beta5.
Essentially when I log in a cookie is created and stored in the specified location, however my other application (which lives within the same domain under a path (i.e. localhost:12345, and localhost:12345/api) does not read the cookie.
Is there any additional information I can read on how to make this work?

I've attempted to upgrade my project to beta6 so that I can step through your shared cookie example code.  All I get is continual dependency issues.  The build says no errors, but if I go to the output tab it has errors.  When I look at the errors it says it can't figure out what my objects are.  I look at the code and the objects have intellisense and are in my dependency configuration.
I'm exhausted.  I've spent days trying to figure out why there are so many dependency inconsistencies.
Is there a way to share cookies in beta5?  So far it's the only group of dependencies I've been able to get a project to build and run in.

e72a5639024eba48a021960f2bc141372c337afc


Use under docker1.9 version  Dockerfile to build docker mirror, RUN the gem install fpm ...  error, can not download from rubygems.org

Use under docker1.9 version  Dockerfile to build docker mirror, RUN the gem install  fpm  ... error, can not download from rubygems.org

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Could you provide some more information, and preferably the exact steps to reproduce?

@thaJeztah thanks, my problem has solved, thank you once again

@thaJeztah  My question is as follows:
in Docker1.9.0 Dockerfile in the source file, there is a command is RUN gem install FPM...
Cannot perform the instructions, the reason is unable to access rubygems.org, directly in the rubygems.org is not directly download the FPM,

I just tried to gem install fpm from inside a container and don't have this issue. Are you using a proxy? I see you're also running into problems when not running inside a container (https://github.com/rubygems/rubygems/issues/1426), so I highly suspect this is an issue in your network setup.
I'll close this issue because I don't think this is a bug in docker, but feel free to comment if you think I closed this by mistake


Fixes #2030 First repl command assumes multiple statements are supported

Fixes #2030 First repl command assumes multiple statements are supported
Switches to get the language version from a better API.

👍


Add configuration for Bash command line completion

Fixes #1
I've also added a new command frames to list all frame IDs to support completion for the edit and remove commands. I've put this into an extra commit, so you can choose to include it or not.
Also added some documentation for it in the readme.

Could you please pass shellcheck on the script file?

I ran shellcheck watson.completion, but the complains it has do not make sense to me:
In watson.completion line 1:
# Bash completion support for watson
^-- SC2148: Tips depend on target shell and yours is unknown. Add a shebang.

In watson.completion line 19:
          COMPREPLY=($(compgen -W "-e --edit" -- ${cur})) ;;
                                                 ^-- SC2086: Double quote to prevent globbing and word splitting.
[a lot more of those...]

This is not a script, but a something to be sourced by /etc/bashrc, therefor it has no shebang line.
${cur} must not be quoted, because otherwise COMPREPLY is an array.

Thank you! Looks good to me, so :+1:

@SpotlightKid Thank you for this!
Commands and projects completions work just fine, but tags completion does not :( I always have +0 as completion...
FYI my bash release is GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin15)

Hmm, maybe your version of sed works differently than mine.  Maybe your sed doesn't support this placeholder?
Can you replace sed 's/.*/+\0/' with awk '$0="+"$0' in both places and see if that works for you?

@SpotlightKid don't you have to use a group for that? (i.e. adding parenthesis)

In fact, I just noticed that adding plus signs is not needed when completing the argument of the -T option to log and report (line 33). Amended the commit.

@willdurand \0 normally is the entire match. It works on my system (sed (GNU sed) 4.2.2). However, the awk solution should work better, so I used this and squashed the commits again.

cool! BTW, what's your OS? Because we run OSX here, and there are differences between the unix tools..

Manjaro (Arch Linux derivate)

@SpotlightKid the awk solution works like a charm on OSX!
Thank you so much for this!


Ability to build Deleting Compaction Strategy with Maven

At the moment there is no way to compile the Deleting Compaction Strategy library with Maven. I have created a POM file on my local to build the DCS library. It seems to build much quicker than building the library with sbt.
Would you be interested in PR to enable this?

I'm a little on the fence.  I'm not sure I'll be able to do a satisfactory job of maintaining a secondary build file.  Also total clean build time right now is 7 seconds on my machine.  I'm sure the POM file is faster, but the current build isn't particularly prohibitive.
With the tests currently living in Scala, is your POM able to execute those?


zsh plugin

I create zsh plugin for buffalo and it is under development here.

Added here: https://gobuffalo.io/en/docs/integrations :)

Thank you very much.


Code styling needs clean up

Code as written so far is not properly styled. Before submission, it should be properly restyled

#5


libssh dependancy

On ubuntu, it seems that we need to use the use libssh.so file rather than the one distributed by XPRESS.

Yes, the libssh.so file shipped with XPRESS has an incompatible ABI with the system libssh. I've had issues using git + XPRESS's provided libssh.so (by accident).
Ask FICO not to include libssh.so by default ? libxprs.so isn't linked against libssh.so, so it shouldn't be needed anyway.


Add a "Sign Up" button to the Application/Intro page

Is your feature request related to a problem? Please describe.
If you don't have Sitecore ID you cannot log in.
Describe the solution you'd like
Add a "Sign Up" button to the Application/Intro page
Sign up page here: https://www.sitecore.com/account/signup

This is now closed as the repository is archived.
This has already been addressed in the new repository.


std::any

I just wonder, how difficult it is to add std::any support to cython?!

I would suggest you try adding it to the Cython/Includes/libcpp/ directory as a new .pxd file and see how far you get with it. Then we know what else needs solving. Treating it as an arbitrary C++ type in Cython for now might already help with some use cases.


doc: update the conv1 shape].

Update the conv1 shape from [5, 5, 32, 32] to [5, 5, 1, 32]. (1 means gray image.)
Although this is irrelevant with the key content in the chapter, one might be confused when looking at this detail.

Can one of the admins verify this patch?

@MarkDaoust WDYT?

It's reasonable for the example model to start with a 1 channel image.
But this variable_scope.md doc was just merged into variables.md, so accepting this now would just set us up for a merge conflict when the sync happens.
If you want to update variables.md after the sync, that's fine with me.

Got it. Thank you. I'll update this PR after sync.

rebased.

Jenkins, test this please.


[Harpoon 2] Duplicate list items cause all following list items to be deleted

Steps to reproduce

Mark a file
Go to a different file
Create a duplicate of the list item and then select it from the UI by hitting <CR>

Open UI again and select first item (now the file is opened)

I would expect this to take me to README.md on step 3, but it does not. I go to my back to the current file.
Logs
config_default#create_list_item README.md
HarpoonList:append { index = -1, item = { context = { col = 6, row = 1 }, value = "README.md" } }
ui#toggle_quick_menu#opening __harpoon_files
select by keymap '' <-- Does not work here
ui#select_menu_item selecting item 2 from { "README.md", "README.md" } options
ui#close_menu name: __harpoon_files win and bufnr { bufnr = 17, win = 1010 }
toggle by BufLeave
ui#toggle_quick_menu#closing
ui#save { "README.md", "README.md" }
ui#toggle_quick_menu#opening __harpoon_files <-- open up UI for second time
select by keymap '' <-- Works this time
ui#select_menu_item selecting item 1 from { "README.md" } options
ui#close_menu name: __harpoon_files win and bufnr { bufnr = 11, win = 1005 }
toggle by BufLeave
ui#toggle_quick_menu#closing
ui#save { "README.md" }
config_default#select { context = { col = 31, row = 7 }, value = "README.md" } __harpoon_files
Deletes items after the first duplicate
Steps to Reproduce
Same as above, but there is another list item already. So when hitting <CR>, the list looks like:
1 README.MD
2 README.MD
3 file1.txt

and my cursor is on the second README.md.
When I open the list back up, the list looks like:
1 README.MD

Logs 2
ui#toggle_quick_menu#opening __harpoon_files
select by keymap ''
ui#select_menu_item selecting item 2 from { "README.md", "README.md", "file1.txt" } options
ui#close_menu name: __harpoon_files win and bufnr { bufnr = 4, win = 1002 }
toggle by BufLeave
ui#toggle_quick_menu#closing
ui#save { "README.md", "README.md", "file1.txt" }
config_default#create_list_item file1.txt
ui#toggle_quick_menu#opening __harpoon_files
select by keymap ''
ui#select_menu_item selecting item 1 from { "README.md" } options
ui#close_menu name: __harpoon_files win and bufnr { bufnr = 6, win = 1003 }
toggle by BufLeave
ui#toggle_quick_menu#closing
ui#save { "README.md" }
config_default#select { context = { col = 32, row = 7 }, value = "README.md" } __harpoon_files

This issue seems to be related to PR #573. Duplicate item are turned into blank-lines (nil-items), which then trigger the issue, that's addressed in the PR.

Duplicates are not allowed

I'm going to have to think about how to solve this.
We most certainly cannot have duplicate items

@ThePrimeagen Does that go for empty lines also? In which case there would only be a single blank line left; or none, if they're not allowed?!

Blank should be fine which should make this fix easy
Next week I should get another 4 hours on harpoon, finish this off


screen displayed as video

Hello, I have a small/big problem. Because it can cause some problems depending on the implementation. Well, by sharing my video after clicking the button, I also share the screen. Unfortunately, it is displayed as a video, not a screen. Why?

This shows by calling console.log on a local window. It seems to me that somewhere in the code isScreen is being passed and it should be screen. Because others have audio and video in idInstance. Probably should also be screen instead of isScreen to parse well.


Expo Client on Android/iOS | Requires v37.0.0

I tried spinning up this project using expo, and on the client app, it gives the error:
"Something went wrong.
This project uses SDK v35.0.0, but this version of the Expo client requires at least v37.0.0."

Same issue here.

Same issue here.

Same issue here.

Can confirm still happening.

Can confirm still happening.

Can confirm still happening.

Still happening.

Can confirm still happening.

Still happening.

i followed this : https://stackoverflow.com/questions/52287710/the-experience-you-requested-uses-expo-sdk-vnull-but-this-copy-of-expo-client
In short :
in app.json change  "expo" => "sdkVersion" to "37.0.0",
in package.json change "dependencies" => "expo" to "^37.0.0",
in package.json change "dependencies" => "react-native" to"https://github.com/expo/react-native/archive/sdk-37.0.0.tar.gz",
delete "node_modules" folder.
run in terminal : npm install
run in terminal: expo-start
Done!
Maybe some other depemdemcies need to be upgraded, but at least app starts...

i followed this : https://stackoverflow.com/questions/52287710/the-experience-you-requested-uses-expo-sdk-vnull-but-this-copy-of-expo-client
In short :
in app.json change  "expo" => "sdkVersion" to "37.0.0",
in package.json change "dependencies" => "expo" to "^37.0.0",
in package.json change "dependencies" => "react-native" to"https://github.com/expo/react-native/archive/sdk-37.0.0.tar.gz",
delete "node_modules" folder.
run in terminal : npm install
run in terminal: expo-start
Done!
Maybe some other depemdemcies need to be upgraded, but at least app starts...

i followed this : https://stackoverflow.com/questions/52287710/the-experience-you-requested-uses-expo-sdk-vnull-but-this-copy-of-expo-client
In short :
in app.json change "expo" => "sdkVersion" to "37.0.0",
in package.json change "dependencies" => "expo" to "^37.0.0",
in package.json change "dependencies" => "react-native" to"https://github.com/expo/react-native/archive/sdk-37.0.0.tar.gz",
delete "node_modules" folder.
run in terminal : npm install
run in terminal: expo-start
Done!
Maybe some other depemdemcies need to be upgraded, but at least app starts...

It works! Thank you, tresorama!

i followed this : https://stackoverflow.com/questions/52287710/the-experience-you-requested-uses-expo-sdk-vnull-but-this-copy-of-expo-client
In short :
in app.json change "expo" => "sdkVersion" to "37.0.0",
in package.json change "dependencies" => "expo" to "^37.0.0",
in package.json change "dependencies" => "react-native" to"https://github.com/expo/react-native/archive/sdk-37.0.0.tar.gz",
delete "node_modules" folder.
run in terminal : npm install
run in terminal: expo-start
Done!
Maybe some other depemdemcies need to be upgraded, but at least app starts...

It works! Thank you, tresorama!

i followed this : https://stackoverflow.com/questions/52287710/the-experience-you-requested-uses-expo-sdk-vnull-but-this-copy-of-expo-client
In short :
in app.json change "expo" => "sdkVersion" to "37.0.0",
in package.json change "dependencies" => "expo" to "^37.0.0",
in package.json change "dependencies" => "react-native" to"https://github.com/expo/react-native/archive/sdk-37.0.0.tar.gz",
delete "node_modules" folder.
run in terminal : npm install
run in terminal: expo-start
Done!
Maybe some other depemdemcies need to be upgraded, but at least app starts...

Great solution. Thanks a lot !


Adding test for correct year records

Signed-off-by: Pihu1998<EMAIL_ADDRESS>

In this case we need to record emissions data in 2 different years and then verify that the calculations were correct based on the different results.  For example, record utility data in 2018 and 2019, then look for the emissions factors, and compare the recorded result vs what they should be.


Remove merge options covered by idiomorph

Use cases for inner, outer, and upsert_attributes can all be handled by morph and signals.

Can you please explain?

@potatotech  upsert_attributes isn't covered by idiomorph

Restored upsert_attributes

Needs to be removed from datastar.go and any examples as well but getting close!

I think I’ve changed all instances in all examples now.

LGTM will put off until v20 release

Use cases for inner, outer, and upsert_attributes can all be handled by morph and signals.

@potatotech can you please help me understand how inner can be handled by morph?

If it turns out that the only redundant merge strategy is outer, then I’d opt for leaving it in, as it still offers the benefit of allowing you to target an element by selector without requiring matching IDs.

I’ve come to the strong conclusion that there is no upside whatsoever in removing merge options. Even outer can be useful considering that the back-end should have full control over deciding what DOM elements to swap. Without it, an ID must already exist in the front-end. I vote not to merge.

The backend is the frontend with hypermedia. If there's a missing id, add it in the template. When swapping fragments, datastar needs to target a specific element, and the only unique selector is id. Morph is sufficient because all other selectors are less specific*.
* There are things like #id[attribute] or #id.class, but id is already unique, so these are not necessary. You definitely should not be using them to store state either.

I'm going to leave it @potatotech, while I :100: agree you should always use the idiomorph I can see value in the effiency of inner/outerhtml for sheer speed in certain situatins.  Thank you for your effort!


Cherrypicking from core-js module (via require) when bundling & transforming

I have a problem of which I am not able to figure out a proper approach. I am using browserify to bundle js files and minify it by using a cmd script. You can see the directory as
 .
 |-- browserify
           |-- bundler.js
           |-- node_modules**
           \-- package.json
 |-- project1
           |-- script.min.js
           |-- script1.js
           |-- script2.js
...
 \-- projectN
           |-- script.min.js
           |-- script1.js
           \-- script2.js

with the script. This is a simple example. Each project consists of multiple directories with their own script files and dependencies. But the end result should appear as what you see. I'm using bundler.js to bundle dedicated javascript files in a given project X folder (or just all when deploying). I then use babelify to transform each file before generating a bundle. Then I minify it. End result: script.min.js. It all works fine. There is always a script1.js which is used as "configuration" script.
However the problem is that I have to provide a polyfill for some ES6 features that is being used in some script files. For that, I'm using core-js which is installed in the same directory as browserify.(folder **)
Instead of picking the whole polyfill script, I want to cherry-pick it. For example, if a script only uses Array.from. It is not supported in IE. So, when deploying to a test environment once the initial script is developed, I would add a polyfill of that to a production script to be tested. So that our testers can use it on their browsers, even IE. To add the polyfill, I have in script1.js the following:
if (typeof PROD !== 'undefined') {
  require('core-js/library/fn/array/from');

  // other requirements + globals (some HTML pages have onclick="foo()" attributes ... )
}

where PROD is being defined when compressing the file. This of course does not work. One option is to use
// bundler.js (b = browserify)
b.require('core-js/library/fn/array/from', {expose: 'ArrayFrom'})

// script1.js
if (typeof PROD !== 'undefined') {
  require('ArrayFrom');
}

Sadly, it doesn't work well and would be a maintenance hell if I have to control lots of those polyfills. (not only Array functions, but also Promises, Set, ...). It also include everything that is being "required", even if there is no Array.from use in the script.
Currently, after checking the handbook; I have figured out an approach:
b.external('core-js/library/fn/array/from')
 .external('core-js/library/fn/array/every')

// script1.js
if (typeof PROD !== 'undefined') {
  require('core-js/library/fn/array/from');
}

Which works well. But even then, I have there a list of "externals" ... It is also sorta anti-pattern.
Is there a better approach for this? If so, can you give me some pointers?
I have also checked browserify-shim but that is not something I want.

There are a few options here:

If I am correct in guessing that your first method does not work because the coreJS file still gets required, you might look at the excellently-named uglifyify.
coreJS allows you to make custom builds. You could have a build step that dumps a polyfill-based build somewhere in your project directory, then do b.add(<path to custom build>).
You can also do the above method but put the built file in the HTML as a script tag.

If I am correct in guessing that your first method does not work because the coreJS file still gets required, you might look at the excellently-named uglifyify.

That transform delivers a different behavior than what I want. I see that they have an exclude thing which may help, but it's a hassle to retrieve information from the script files I'm going to bundle so that I can include/exclude things.

coreJS allows you to make custom builds. You could have a build step that dumps a polyfill-based build somewhere in your project directory, then do b.add().

That's a nicer approach. But even then, some project may have duplicate core-js requires. Especially Array.from is frequently used. One project picks that, another one too, but third one will pick the whole array polyfill because it uses many array operations.
I was thinking at an extra module, a stub. It consists of a list of items that got picked from core-js. When exporting it with browserify (by .external(pick-from-core-js.js), I then can sub-require some polyfill I need. It seems that my situation is kinda unique because I could only find few solutions on this matter. Unfortunately, they're more bad approach than what I did at my current working example atm. Hence I opened an issue to see if some of you could think about a workaround (I didn't have checked browserify under the hood yet, to comprehend how it works) or something more useful.
Heh. That's not a priority atm. Thanks for replying though.


[Feature] Type instead of Class has been added for mapping

Added .as() method to map any Type. It is more general solution than Class<T> so now we are able to use Gson, Jackson and Jackson2 by default which was not supported by Rest-assured.
Added necessary tests.

This is great stuff, thanks a lot! And it should be backward compatible as well right?

Yep, it backward compatible should be as well.
I tested this on my projects.

Without the PR I can't customize mapping of my own http client based on Rest-assured https://github.com/swagger-api/swagger-codegen/pull/7492/files#diff-4ed6d557101bb6b37c06fbc3f44572b1R117 . Rest-assured client in swagger:  https://github.com/swagger-api/swagger-codegen/pull/7492
Mapping in swagger is complex and it operates with Type a lot — it based on it.
Now I don't have any idea how to keep backward compatibility.

Thanks for your comment, I think it's an ok trade-off since it shouldn't be something that most people are using and it should be easy to update. Really cool that you're working on  swagger integration.
And thanks for the PR!


Feature: Launch command as positional argument

Feature discussed in #39
I took the liberty of bumping clap to 4.3.24 which is the last version before the MSRV changed and it doesn't seem to require any changes in envio

Thank you for your contribution @Rubensei, really appreciated

Thank you for your work!
PS: I was going to review the MSRV but I guess it wasn't much of an issue

I hope envio meets your expectations!
If you have any new ideas or encounter any other issues then please feel free to let me know by opening a issue.


Website redirects for certain languages

In NL "storing" means disruption.
For more languages there might be a better domain name.

we got cryptobewaarportemonnee.nl . Nice!


Objective-C header files are incorrectly parsed as C

(Original issue 826 created by al45tair on 2012-12-19T13:42:14.956892+00:00)
There is a long-standing issue with Objective-C headers being parsed as C simply because they use the same file extension (.h).  This is fixed by pull request #141.

(Original issue 826 last updated on 2013-01-09T12:25:49.913831+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by al45tair on 2013-01-09T12:25:49.940709+00:00)
Fixed as of c037265


Resolve syncing conflicts from repo_sync_working_branch to public

Pull request opened by Docs to resolve syncing conflicts from repo_sync_working_branch to public.

I've resolved the merge conflicts. However, I'm not going to review the 147 files in this PR.


Ability to configure log level

https://github.com/atlassian/gostatsd/blob/1e74a750fea0915b9ebc64dc7f433fbfc93b76d8/cmd/gostatsd/main.go#L210-L212
Would be great to get this exposed as log levels rather than just --verbose, so we can turn off INFO in certain use cases. For example, using this package in a test suite to write statsd metrics to stdout.

Thanks for opening this issue. I don't think we'll add that level of config to logging.
If i understand your example, you have some application code emitting metrics and you want to validate (or see) what is being sent. You may be able to do what you need by injecting a mock statsd client in your application code to capture or expose what you need without needing to rely on logs from gostatsd. This is how we do this kind of check for our other services.


Not refreshing code with mocha --watch

I've tried to use this module as mocha --compilers tsx:typescript-node/register and it works great, but when I add --watch, it will always run the same code as the first run of mocha. I suspect it's caching the compiled files and not recompiling until running again.
I'm using it with typescript@next (1.6.0-dev.20150908) and .tsx files (for JSX support).
When I was using babel compiler for my js tests, --compilers js:babel/register, was working fine.

Thanks for the report! Yes, you are correct - files are currently being cached in memory. I'll have a patch tonight, so I'll keep you posted in this issue.

@markogresak Could you try the latest release, v0.0.9 - it uses the version (mtime) of the file to check for updates.

It's working now, thanks!


Settle on long term name

I'm thinking "if this were merged into cargo, what name should it have?"
Quick thoughts on criteria

Brief
Informative, including the idea of non-semver checks

Ideas

cargo api (currently taken)
cargo crate-api (I have this one)
cargo diff (available)
cargo preflight (reserved by @obi1kenobi)

I have cargo query, which might be another more general option. If we end up namespacing checks (and allowing user-specified checks and/or namespaces), that could for example be cargo query <namespace(s)> as in cargo query semver. Or perhaps cargo query <cmd> <namespace(s)> if we want additional modes, like diff vs check vs suggest version etc.

I feel like query would be misleading.  I would assume from the name that it is for searching crates at the current moment while cargo semver-checks is for linting the evolution of a crate's API

I see your point. Naming is hard :) Maybe as we make progress, some of the folks that worked on the original API evolution RFC might be interested in this and might have some naming ideas as well.

I just reserved the cargo-diff crate. This is to prevent someone stealing the name and refusing to transfer it now that it's been mentioned here as a possibility. Happy to transfer it at any point if desired.

Added cargo-preflight (like a pre-flight checklist) to the idea list and reserved the name. Idea came up in #86.

am happy with semver-checks, otherwise semver works too


Allow installation via hooks into nativescript project

tns plugin install does some magic we can use to make setup easier (and remove need for template app)
We can also add a hook to the before prepare step that compiles the svelte files to their own bundle, thus bypassing the need to run with --bundle all the time

After investigation there doesn't seem to be a simple way to do this, especially after NS 6.0's migration to bundle only webpack builds 😞


Create Camara-OAS-Sections-Guidelines.md

What type of PR is this?
Add one of the following kinds:

cleanup
documentation

What this PR does / why we need it:
Creates a new document with guidelines for OAS sections. Will add all the content related to OAS sections which was originally part of API design guidelines doc,
Which issue(s) this PR fixes:
Fixes #306
Special notes for reviewers:
Changelog input
 release-note

Additional documentation
This section can be blank.
docs

18/OCT: Fine to me so far


ndk/media/image_reader: Don't assume ownership of NativeWindow

When implementing proper ownership acquire() and release() semantics for NativeWindow in https://github.com/rust-mobile/ndk/pull/207, I thought I had checked all existing calls to NativeWindow::from_ptr() to make sure that they return a pointer that we get ownership over, and have to clean up ourselves.  This turns out to not be the case for AImageReader_getWindow():
The ANativeWindow is managed by this image reader. Do NOT call ANativeWindow_release on it. Instead, use AImageReader_delete.

And can be trivially reproduced by creating an ImageReader and calling .get_window().  Dropping the NativeWindow is fine but subsequently dropping the ImageReader results in a NULL-pointer SEGFAULT.
For now calling clone_from_ptr() is enough to first acquire an extra reference on the pointer so that ownership remains balanced, but in the future we'd like to investigate a new non-owned NativeWindow type similar to HardwareBuffer.
As of writing the following calls to from_ptr() remain, that are all confirmed to transfer ownership and require cleanup via _release():

ASurfaceTexture_acquireANativeWindow();
AMediaCodec_createInputSurface();
AMediaCodec_createPersistentInputSurface();
ANativeWindow_fromSurface().

Yup, should be enough asserts there.


How to make any force field?like wind.

I am new in three.js and physijs.I know a bit of [applyImpulse] or [applyForce],but it can't make a wind scene.

I'm sorry about my English,it's not good.


Hiring Developer

Hello, we hire developer for:

Proper installation of Peatio to CentOS/Ubuntu
Can add BTC/ETH, BTC/BCC, BTC/XRP , BTC/LTC exchanges. (BCC is not a must.)
Can make some customisations based on design that we will provide
Can give support after.
Please send me your skype id's for live talk.

Hello, I can help you with installation of cryto-currencies and I can also integrate your provided design. Please contact on skype khubab or email at<EMAIL_ADDRESS>thanks.

Cool

pls contact with live:apollo447722

Hello. Could you please add me to skype?. Mine is gjnardoni.

I can help you, my skype : sushilraghav

I've already developed and deployed 3 exchanges based on Peatio.
I can add more coins and customize exchanges with peatio.
Also have experiences in wlox.
Feel free to message me any time.
skype : live:apollo447722
mail<EMAIL_ADDRESS>Regards.


Ruotong's Open Source Writeup

Great work!  Virtual box is a fun way to learn linux.
FYI it seems like the project's list of testing and documentation tasks may be a little outdated- that's unfortunately very common for smaller projects.  No worries though, you did well! In the future for a real contribution it's always OK to email the list to confirm that the task you're about to undertake is still needed.


Update the RDS root CA list

The current root is expiring in a few months, so we need to migrate to a new one. We'll be copying similar code to perf, but we can start with making sure it works with triagebot.
I've checked that the new CA file contains the old certificate, so this should keep working with our current database (i.e. doesn't need to be synchronized deployment wise with anything).

Merging - I have some bandwidth tonight to fix problems if we notice them, so will monitor for a bit.


Different config UIs

I noticed the following issues with the Config UI:

As seen in the first part of the video, when navigating to Configs and then selecting the channels or inputs of my only configured TV, only the first entry shows up. When clicking "Add Input/Channel", the next entry that is present in config.json is revealed.

When selecting my configured TV from the list and clicking the button to edit that config, the Channels list is loaded completely, but has a different layout as in the first part of the video.

https://user-images.githubusercontent.com/9931588/135852993-b8a893d6-edc9-48e3-bf3f-363626137193.mov

Hey @lukasmalkmus

its because you use the „advanced static config schema“ which can not be handled through config ui x very well due to the nested arrays.
The layout is different because u are using the „advanced static config schema“. This all is intended and not a issue by the plugin

Use this form of config if you can handle the „advanced“ configuration. Its only for peoply with the required knowledge what want to add „other“ inputs which can not be cached through the plugin
Why u dont use the „edit icon“ to edit your configuration? Because the above „Configs“ navi IS NOT THE SAME as the edit button

Alright, got it.


Changing Latvian currency from LVL to EUR

Latvia switched to Euro from 2014-01-01. See http://en.wikipedia.org/wiki/Latvian_euro_coins

Sorry that took a while - published now as 0.0.17 including the Latvian change.


Fixed a bug to include Verify deployment pre-upgrade section in upgra…

…de-packet-core-azure-portal topic and formatted command to snippets in enable-azure-active-directory topic

@djrmetaswitch
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#please-close


Artifacts from entrance filter mesh visible in the 335 channel after pdf deconvolution

In GitLab by @kreevescfa on Jan 25, 2023, 14:08
There are noticeable artifacts in the 335 image after deconvolution with the PSF.  Attached are images of deconvolution on the same file done with IDL and aiapy.  In the aiapy version, there are artifacts in a grid pattern, probably from incorrect removal of the entrance filter mesh pattern.  Also attached is the original fits file.  The aiapy deconvolution was done with the following code:
 m = sunpy.map.Map(file)
 psf = aiapy.psf.psf(m.wavelength)
 map_deconvolved = aiapy.psf.deconvolve(m, psf=psf)

The IDL deconvolution was done with the following code:
read_sdo, file, index, data
psf=aia_calc_psf('335')
data_deconvolved=aia_deconvolve_richardsonlucy(double(data),psf)

In both cases, the PSF deconvolution was done before image registration, pointing update, and normalization (aia_prep), as recommended.

aia_lev1_335a_2017_09_10t16_25_24_64z_image_lev1.fits

In GitLab by @wtbarnes on Jan 25, 2023, 14:25
Thanks very much for reporting this. Just for clarification, the PSF deconvolution, in both the IDL and Python cases is done on the full-disk image and not the cutout that is shown in these images, correct?

In GitLab by @wtbarnes on Sep 4, 2023, 12:23
Hi @kreevescfa. Sorry for the very long delay in following up on this.
Unfortunately, I am not able to reproduce the entrance filter pattern you're seeing in the Python deconvolution case. I took the image you attached here, calculated the PSF in IDL and in Python with aiapy and then ran the deconvolution in aiapy with both PSFs. The figure below shows the two cases:

As you can see, I don't find the same entrance filter pattern you've shown above in either case. Furthermore, when doing a percent difference between the two, I find the differences all lie along the PSF arms and that all are under 0.5%,

I've attached the notebook I used to generate these results here: aia-psf-testing.ipynb
A few more follow up questions:

What versions of sunpy and aiapy were you using to get the result you showed above? You can get the version info by typing sunpy.__version__ and aiapy.__version__
Were you using a GPU to calculate the PSF and/or perform the deconvolution in the Python case?

In GitLab by @kreevescfa on Jan 25, 2023, 14:29
Yes, correct.  I just plotted a submap of the portion of the FOV I was interested in, but the deconvolution was done on the full image (anyway, it won't work on a submap unless you fiddle around with the PSF).

In GitLab by @wtbarnes on Sep 6, 2023, 11:26
This is also related to #75 as the PSF images using both cupy and numpy show the same sorts of artifacts.


Missing some classes?

Idk which ones, but you're missing Cog Sci C100? (I know it exists cause I phase 1-ed it :O)

I am not sure why this is happening.
So, on searching for all courses from the official database, only COG SCI 100 shows up. However, the database says that 'COG SCI 100' is not being offered next semester, but 'COG SCI C100' is. Has this course been cross-listed only recently? What's it cross-listed with?

The course has traditionally been cross-listed with Psychology C120

I investigated this further, and couldn't find a systemic cause for this. I have manually added COG SCI C100 to my API crawling pipeline, so it will now be listed. But I can't be sure if there are other classes missing.
Closing this issue for now, but will reopen it if we find more classes missing.


Expose OS-chosen address and port from async_server

From https://github.com/cpp-netlib/cpp-netlib/pull/729

LGTM -- thanks @tribal-tec!


Able to login with incorrect password on second attempt

I am trying achieve headless using web API, I created a API which take username and password, it does not have the Authorize filter. I am able to get a valid access token, but I am able to authenticate user using invalid password from the second attempt. Below is my sample code
public class LoginController : ApiController
{
    private static string aadInstance = System.Configuration.ConfigurationManager.AppSettings["ida:AADInstance"];
    private static string tenant = ConfigurationManager.AppSettings["ida:Tenant"];
    private static string clientId = ConfigurationManager.AppSettings["ida:ClientId"];
    private static string authority = String.Format(CultureInfo.InvariantCulture, aadInstance, tenant);

    private static string todoListResourceId = ConfigurationManager.AppSettings["todo:TodoListResourceId"];
    private static string todoListBaseAddress = ConfigurationManager.AppSettings["todo:TodoListBaseAddress"];
    // GET api/todolist
    [HttpGet]
    [Route("api/Todo/Login/{password}")]
    public async System.Threading.Tasks.Task<string> LoginAsync(string password)
    {
         var authContext = new AuthenticationContext(authority);
        UserCredential uc = new UserPasswordCredential("<upn>", password);
        AuthenticationResult result = authContext.AcquireTokenAsync(todoListResourceId, clientId, uc).Result;

        return result.AccessToken;
    }
}

@LearnToCodeKM : did you read https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/wiki/AAD-B2C-specificities ?

The above link is using B2C policies which is not a headless authentication.

I did find this https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/wiki/Username-Password-Authentication , but i just wanted to know what should be the client id and the process to register the app.

I was able to authenticate the user using MSAL.net with Azure B2C, but I am not able to secure my API using that token. Is there any way to do it?
Is it possible to tweak this solution to meet the token received from MSAL
https://jonhilton.net/security/apis/secure-your-asp.net-core-2.0-api-part-2---jwt-bearer-authentication/

@LearnToCodeKM : you might want to have a look at this sample: https://github.com/azure-samples/active-directory-dotnet-native-aspnetcore-v2
Again it's not for B2C, but this should also work for B2C.


Use bundle URL version as the cache version

Merge / deployment checklist

Run test builds as necessary. Can be on this repository or elsewhere as needed in order to test the change - please include links to tests in other repos!

[ ] CodeQL using init/analyze actions
[ ] 3rd party tool using upload action

[x] Confirm this change is backwards compatible with existing workflows.
[ ] Confirm the readme has been updated if necessary.

The unit tests look really good to me and look like they validate the behaviour pretty well. I'm running a quick test locally too just to make sure, but I don't expect it to fail.