Add search result index to the SELECT_SEARCH_RESULT event

Problem

We don't currently track the page of API results that a selected search result is chosen from.
Description

Indirectly track the page of a search result by including its index in the analytics event payload. Perhaps we should call this something like "resultPageIndex" or simply "index" depending on what @WordPress/openverse-frontend thinks.
Alternatives

Additional context

@zackkrida To track the page of a selected search result include its index in the analytics event payload as resultPageIndex or index.


Crash when Sending Keys to Bot

26/06/2016 19.36.07 [!] EXCEPTION: UnhandledExceptionHandler()  Object reference not set to an instance of an object.
StackTrace:
at ArchiSteamFarm.Bot.b__64_0(Bot bot)
at System.Linq.Enumerable.WhereEnumerableIterator`1.MoveNext()
at ArchiSteamFarm.Bot.d__64.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at ArchiSteamFarm.Bot.d__43.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at ArchiSteamFarm.Bot.d__82.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at ArchiSteamFarm.Bot.d__96.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.AsyncMethodBuilderCore.<>c.b__6_1(Object state)
at System.Threading.QueueUserWorkItemCallback.WaitCallback_Context(Object state)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.QueueUserWorkItemCallback.System.Threading.IThreadPoolWorkItem.ExecuteWorkItem()
at System.Threading.ThreadPoolWorkQueue.Dispatch()
at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()

Version? Used command? Bot options?

@JustArchi Local version: <IP_ADDRESS> | Remote version: <IP_ADDRESS>

And what was the command, and sent to which one of those 3 bots?

the result same whether i send to 2 and 3.

Weird, what is the result of !statusall ? It looks like Dictionary contains Bots that are null, which should never be the case, unless issue. Can you show me the full log with the crash? It's important.

26/06/2016 19.33.40 [] INFO: CheckForUpdate()  Checking new version...
26/06/2016 19.33.42 [] INFO: CheckForUpdate()  Local version: <IP_ADDRESS> | Remote version: <IP_ADDRESS>
26/06/2016 19.33.42 [] INFO: CheckForUpdate()  ASF will automatically check for new versions every 24 hours
26/06/2016 19.33.42 [] INFO: RefreshCMs()  Refreshing list of CMs...
26/06/2016 19.33.43 [] INFO: RefreshCMs()  Success!
26/06/2016 19.33.43 [!!] ERROR: Load()  botDatabase is null!
26/06/2016 19.33.43 [!!] ERROR: .ctor()  Bot database could not be loaded, refusing to start this bot instance!
26/06/2016 19.33.44 [] INFO: Start()  Starting...
26/06/2016 19.33.44 [] INFO: OnConnected()  Connected to Steam!
26/06/2016 19.33.44 [] INFO: OnConnected()  Logging in...
26/06/2016 19.33.45 [] INFO: OnLoggedOn()  Successfully logged on!
26/06/2016 19.33.46 [] INFO: Init()  Logging in to ISteamUserAuth...
26/06/2016 19.33.48 [] INFO: Init()  Success!
26/06/2016 19.33.49 [] INFO: IsAnythingToFarm()  Checking badges...
26/06/2016 19.33.49 [] INFO: IsAnythingToFarm()  Checking first page...
26/06/2016 19.33.51 [] INFO: Start()  Starting...
26/06/2016 19.33.51 [] INFO: OnConnected()  Connected to Steam!
26/06/2016 19.33.51 [] INFO: OnConnected()  Logging in...
26/06/2016 19.33.52 [] INFO: OnLoggedOn()  Successfully logged on!
26/06/2016 19.33.52 [] INFO: StartFarming()  We don't have anything to farm on this account!
26/06/2016 19.33.52 [] INFO: Init()  Logging in to ISteamUserAuth...
26/06/2016 19.33.52 [] INFO: Init()  Success!
26/06/2016 19.33.53 [] INFO: IsAnythingToFarm()  Checking badges...
26/06/2016 19.33.53 [] INFO: IsAnythingToFarm()  Checking first page...
26/06/2016 19.33.56 [] INFO: StartFarming()  We have a total of 5 games to farm on this account...
26/06/2016 19.33.56 [] INFO: StartFarming()  Chosen farming algorithm: Simple
26/06/2016 19.33.56 [] INFO: FarmSolo()  Now farming: 400250
26/06/2016 19.33.59 [] INFO: ShouldFarm()  Status for 400250: 4 cards remaining
26/06/2016 19.33.59 [*] INFO: Farm()  Still farming: 400250
26/06/2016 19.36.07 [!] EXCEPTION: UnhandledExceptionHandler()  Object reference not set to an instance of an object.
StackTrace:
at ArchiSteamFarm.Bot.b__64_0(Bot bot)
at System.Linq.Enumerable.WhereEnumerableIterator`1.MoveNext()
at ArchiSteamFarm.Bot.d__64.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at ArchiSteamFarm.Bot.d__43.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at ArchiSteamFarm.Bot.d__82.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at ArchiSteamFarm.Bot.d__96.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.AsyncMethodBuilderCore.<>c.b__6_1(Object state)
at System.Threading.QueueUserWorkItemCallback.WaitCallback_Context(Object state)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.QueueUserWorkItemCallback.System.Threading.IThreadPoolWorkItem.ExecuteWorkItem()
at System.Threading.ThreadPoolWorkQueue.Dispatch()
at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()

26/06/2016 19.33.43 [!!] ERROR: .ctor() Bot database could not be loaded, refusing to start this bot instance!

As I initially thought, thanks for reporting this!
You should fix your bot database, while I'll fix the issue that can happen when ASF adds Bot with broken database to Dictionary.

The bug should be fully fixed since V<IP_ADDRESS> - https://github.com/JustArchi/ArchiSteamFarm/releases/tag/<IP_ADDRESS>
It won't magically fix your broken database, but ASF will no longer crash when bot with such database is used.
Thank you for report! :+1:

Any idea about how database can become broken? Any precautions about how to avoid it?

I'm not even sure how it's possible under normal conditions to catch broken database - I'm using ASF since version V0.1 24/7 on my server and I'm yet to face this issue. My theory is mainly about too aggressive AV scanning files being written on the go, and OS denying ASF permission to write. It's not possible under normal operation to catch it, it takes less than 100ms to write the file.
I also can't believe in JsonConvert generating invalid JSON when ASF saves the file, this is not possible, because we had to use different structure for loading and saving to cause incompatibility, which is not the case with current code.
So that leads to the question, what is the exact reason for ASF to catch such problem? No clue, I'm not even entirely sure how to harden it a bit, I think I'll just add some extra retry logic to file saving, because that is the only thing which crosses my mind.

This is the only thing which is possible for me to do. If despite our 5 tries and verifying that json is not-null, we will fail, there is really nothing more we can do.


PERMANENTLY CONNECT

how can I tie the connected printer so that no need to connect every time the application is loaded.

Try this
final myDevice = BluetoothDevice.fromMap({       'name': data['name'],       'address': data['address'],       'type': data['type']     });

where do I put this code?
On Fri, May 14, 2021 at 10:23 AM calimba @.***> wrote:

Try this
final myDevice = BluetoothDevice.fromMap({ 'name': data['name'],
'address': data['address'], 'type': data['type'] });
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kakzaki/blue_thermal_printer/issues/77#issuecomment-841066533,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFSGZW6ZK7GOV37JBARXWSDTNTFY3ANCNFSM44JGZBGA
.

Hi Calimba,
Kindly guide on where I should insert this piece of code for the bluetooth
printer to be permanently remembered whenever the printing is done.
On Tue, May 25, 2021 at 4:29 PM ALEXANDER KITHOME @.***>
wrote:

where do I put this code?
On Fri, May 14, 2021 at 10:23 AM calimba @.***> wrote:

Try this
final myDevice = BluetoothDevice.fromMap({ 'name': data['name'],
'address': data['address'], 'type': data['type'] });
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kakzaki/blue_thermal_printer/issues/77#issuecomment-841066533,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFSGZW6ZK7GOV37JBARXWSDTNTFY3ANCNFSM44JGZBGA
.

I do this
void _checkIfLoggedIn() async {
      SharedPreferences localStorage = await SharedPreferences.getInstance();
      var token = localStorage.getString('token');
      var userJson = localStorage.getString('user');
      var user = json.decode(userJson);
      var stringValue = user['dispositivo'].toString();
      var stringValue2 = user['address'].toString();
      var stringValue3 = user['tipo'];
      if(token != null){
        setState(() => {
            _isLoggedIn = true,
            userData = user,
            _dispositivo = stringValue,
            _direccion = stringValue2,
            _tipo = stringValue3
        });
      }

      if(_direccion != null){

          final myDevice = BluetoothDevice.fromMap({
            'name': _dispositivo,
            'address': _direccion,
            'type': _tipo
        });

        bluetooth.isConnected.then((isConnected) {

            if (!isConnected) {

              bluetooth.connect(myDevice).catchError((error) {
                setState(() {
                  _connected = false;
                  });
              });
            }
          });
      }
  }


docs: removal of unwanted extra closure of achor tag

Why: found unwanted extra closure of anchor tag in docs

What's being changed: removed unwanted extra closure of achor tag

Check off the following:

[] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

H #
be save

@codewithsnowbit Thanks so much for opening a PR with this fix! I'll get this merged once tests are passing 🍏

You're welcome @ramyaparimi


Headers not submitted?

It appears that since the removal of core-ajax, custom headers aren't being submitted any longer. Looking through ajax-form.js, I don't see the headers being handled. (Is it possible I'm just missing something?)

Nope, you are 100% correct.  Guess I should have unit tests for this, to ensure it doesn't happen again.  I'll shouldn't be too hard for me to push out a hotfix release and some tests that address this in the near future.  Pull requests welcome though.

Fixed in 1.5.1 (just released), with a unit test.


[SMALLFIX] Allow Alluxio master to shutdown cleanly upon SIGTERM

@aaudiber Could you take a look? Thanks!
When master process receives SIGTERM, the hook will execute to cleanly
close the resources used by the process, e.g. open files.
Without this feature, master will not close the journal files, nor will
it release the lease (lock) held on the journals. Consequently, if HDFS
restarts afterwards, HDFS will not be able to access the journals.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18023/
Test PASSed.

@riversand963 This is a very elegant solution! Are we sure it successfully closes the journal files? Is there a limit on how long the JVM hook is allowed to run for, and are we at risk of exceeding that time with a normal shutdown?

Thanks @aaudiber !
I conducted a simple test on my local machine, and saw that all journal files were closed. They were renamed to startseq-endseq, instead of startSeq-0x7fffffffffffffff.
For your second and third question, I need to do more investigation and will keep you updated.

@aaudiber @calvinjia Thanks for your comments.
If the Java process is terminated with SIGTERM (kill -15 <pid>), the shutdown hook is guaranteed to be executed. If the process is killed with SIGKILL (kill -9 <pid>), then there is no guarantee on its execution.
Generally it is a good practice to let shutdown hooks finish their jobs quickly, but the Javadoc of Runtime#addShutdownHook does not say whether there is a limit on the time.
Currently bin/alluxio sends SIGTERM to Alluxio master first. If the latter does not finish the shutdown within 30 seconds, a SIGKILL is sent. In this case, the journal files may not be closed, and we will have the same problem.

@riversand963 In your experience, is 30 seconds sufficient for the hook to execute? What factors influence the execution time?

@calvinjia
Usually, alluxio-stop.sh finishes within 30 seconds. The execution time of the shutdown hook depends on the execution time of AlluxioMasterProcess#stop. If the system is in good state, I think that should be sufficient amount of time.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18035/
Test PASSed.

Our current AlluxioMasterProcess#stop method does
      stopServing();
      stopMasters();
      mJournalSystem.stop();

If stopping the rpc server or stopping the masters takes over 30 seconds, we won't get around to closing the journal files. We should either find a way to guarantee that those tasks complete within 30 seconds, or have a separate shutdown hook for closing journal files.

@aaudiber
Should we call stopServing and stopMasters before shutting down the UFS journals? If we shutdown Ufs journal concurrently before making sure Alluxio master stops receiving RPCs, is it possible to create a race condition? A user issues a file metadata operation, Alluxio master accepts that request, but finds that Ufs journal is not available, then starts blocking and retrying? This will not return incorrect result to user, nor will it cause the system to be in an inconsistent state, but it may cause confusion.

I agree, it's confusing to close the journal files without stopping the RPC server first.
The RPC server can take up to 60 seconds to shut down by default (controlled by alluxio.master.thrift.shutdown.timeout). If the alluxio-stop.sh script waits 2 minutes before sending SIGKILL, that ought to provide enough time for an orderly shutdown.

@aaudiber Could you PTAL? Thanks!

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18052/
Test PASSed.

Merged build finished. Test PASSed.


Training on Cycle GAN

Hello,
I am using your tensorflow implementation of the Cycle GAN for my own dataset.
However, there are few questions I want to ask about, and I hope you can be able to help me:
1- Before training, I create the csv file, which is pairing all images in both domains together randomly, but my question is, between every epoch and the other, do the batches get shuffled ? or for each epoch they are provided by the same sequence ?
2- I don't understand what is the "CycA" and "CycB" output images
3- If I don't want to use your network bidirectionally, is there something which I have/need to do ? I mean, I only want to translate from domain A -> B, and not vice versa
4- My dataset are images which are much higher in resolution than the provided dataset. They are 1242x375, will this be applicable for the deploying of the network ? or what shall I edit exactly in any part of the code to adapt my dataset
Thanks a lot in advance
Looking forward to hearing from you soon
Best Regards,
Mostafa

@nathansilberman @leehomyc

@mhusseinsh Hi, I have a problem here.  May I ask you how you implement this code? And is your csv file same to me ? There exists both ”/“ and ”\“ , and there is a blank row between two pairs of images.

Thanks in advance.


Need additional error information in "Register devices" window

Lately, I've been unable to automatically provision user tester devices through App Center. When I access the "Register devices" UI, and select my Apple Developer account from the list, it simply shows the following:

The message "Something went wrong. Please try again." isn't really descriptive enough for me to diagnose the issue. I've tried removing/re-adding my developer account, using a different account, using a different browser, making sure profiles/certs were still valid in my Apple Developer team, etc.

I'm haveing same issue at the moment.

I am also having the issue, it's also doesn't show any problems in the appcenter status. But I suspect there is a server related problem in here

We have the same problem but if you switch to appstore connect api key it might work, I say it might because it works at first now I get the all green tick boxes when distributing but then it leaves it unprovisioned


Create Windows Installation

Need to get an executable going for Windows.

Windows executable seems to be working 🎉 .
https://drive.google.com/file/d/1FUxEDef9uxd3t_98K6rNwvpYIzQrg41h/view?usp=sharing
It should be noted, however, that there is an issue with the nano/pump elements toggle corrupting save file data.


Filtering by Environment vars

Thanks for making this docker container.  I'm in a situation where I don't want to log every container and was thinking of extending this to have some sort of LOG_FILTERS=selecta,selectb to sort through a LOG_FILES environment variable on containers.
Would you like this as a feature, or should I just fork completely and keep the original dockerfile simple.
I'd appreciate any input you may have.
Cheers.

The further I'm getting in, the more I realize I might have to base off phusion/baseimage first.  :(  Out of curiosity, what's your LICENSE for your work?  If I end up forking, I'd need to borrow the logentries type connector - if that would be okay.

I'd really like to have that functionality :)
Changing base image is all right, but why would you need that?
I've added license and credits - connector is actually based on https://github.com/Woorank/fluent-plugin-logentries
I'd gladly accept your PR, that's really nice feature to have :)

Hi.  Since I needed to use the alt baseimage, I created a clean slate today at https://github.com/dataferret/docker-logentries
Primarily, I needed runit to allow docker-gen to reload td-agent whenever the config changed.  In the end, I also ended up using some other fluent plugins to massage the logs into json form.  Eek, it's kinda diverged way more than I expected so I'm leaning towards keeping these two separate.  There's a simplicity with yours that I think is nicer in some circumstances.
If you still want though, I can build a PR without the json output changes.

That's ok, your solution is more complete anyway. I think I'm gonna use it instead of mine :)
If you could also make automatic build of your repo on hub.docker.com that would be great!


Dataloaders

Motivation and Context
Integration with other popular tools which could be used alongside hangar is really useful for users. Dataloaders is probably one of them.
#13
Description
Right now, it's just a skeleton to start the discussion about the structure. I'll change it as it moves forward.
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Is this PR ready for review, or a work in progress?

[ ] Ready for review
[x] Work in progress

How Has This Been Tested?
Put an x in the boxes that apply:

[ ] Current tests cover modifications made
[ ] New tests have been added to the test suite
[ ] Modifications were made to existing tests to support these changes
[ ] Tests may be needed, but they are not included when the PR was proposed
[ ] I don't know. Help!

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have signed (or will sign when prompted) the tensorwork CLA.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@rlizzo I made the basic skeleton of the dataloader base class. For now, we could keep the other dataloaders also in the same file. Taking your discussion from #13, adding features such as batching, shuffling to the base dataloader is not required as of now since both PyTorch and Tensorflow does that quite well. They just need an object from where they could get one sample at a time. In my view, the base class can be inherited and make advanced dataloaders inside hangar itself when it's required in the future.

Hey sherin I think we should put this on the back burner right now. The data loaders are important, but the bulk of the work to define multithreaded (multiprocess?) and batch reads actually should be implemented in the hangar core and enabled by default for some operations. These utilities could just implement the framework specific interfaces and make calls straight to the public hangar API.
I'm also not thrilled about including them in the hangar-py/src directory. I would like to keep only User facing APIs and the backend processing workflows in there. Since these sorts of utility functions are just bolted on for convienence (similar to what we want to do with the CLI import/export) I'm thinking they should almost be stored as a separate module?
What I mean is that we may want to split the src dir into core (containing what is in the repo right now) and a convenience dir (or some other similar name) to hold these. To keep a unified experience for importing the Hangar Repo or Dataloaders classes in a dependent codebases we could just raise the "apparent" level of each module to a unified interface. When used as intended (and not like most home brewed projects do) we can do some really useful things with a modules __init__.py file. I can handle all of that when it comes time...
With the changes introduced in #71, multithreaded reading got a whole lot simpler. Let's coordinate on those changes and how the backend now thinks about files before I leave for holiday on Tuesday. In brief though the backend routing is now about as simple as it gets (essentially dynamic backend selection by returning accessor methods as key lookups in a dict, and then blindly passing in a hash record value as input to the returned function which then "magically" returns the data.) The implementation details aren't really your concern when operating at the level this would need to work on,  but I'll just say that its rather efficient, and we should have no issues with multiple threads (or even processes) reading at the same time (even if they actually need to read from the same file on disk).

@rlizzo Awesome, make sense. However, I would like to make a very simple (at least pytorch) data loader for the demo I have on 15th at PyCon Thailand. Then we can probably be iterating over it when we get more time for this. Although a utility in the core would be the best, I can write my own snippet example and tell the audience that something like this is coming into the core soon.

Yeah if u can just write it as a user defined script on ur local system that would be best for now. Feel free to post it in a gist so we can use that as a jumping off point in the future.
I don't want to limit you too much or sound too dictatorial, but it's important to me that we keep the code logically ordered (while maintaining ease of use). This is complicated enough as is, I won't accept bloat (I already want to kill some redundancies in the records/queries.py and records/heads.py and I think some of the functions in records/parsing.py are either unreachable or redundant right now...) Hope you can understand. We'll work together to make sure that these Bolton utilities can do what they need while respecting the structure of the core

Make sense. Closing this for now!


Django 2.0 support

I want to use this app to my django 2.0 project.

@ctxis any time frame for a new release supporting Django 2.0?
Thanks mate

It is planned for this weekend. Probably sooner.
Thanks,
Lefteris

Awesome! Thanks mate

@lefterisnik : Do you have any problem where I could help you ?
All the best and merry christmas
Jonathan

Merry Christmas all,
Thanks @DEKHTIARJonathan, some tests are failing because something has changed on Django 2.0 on the request object, Arghh. I am looking on that.
Thanks,
Lefteris

Hi all,
I've just pushed the new release which supports Django 2.0.
Sorry for the delay.
@DEKHTIARJonathan the issue wasn't the request object but the MIDDLEWARE attribute on settings.py.
Thanks,
Lefteris


Structuring Large Applications with Blueprint and Application Factory…

Article checklist
ATTENTION: In your PR - add the ISSUE (topic form) # that this PR corresponds to. You can do this by also adding a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.
Closes #2365
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
If you have not already please go over our Resources Page for more tips and tricks on how to improve our overall technical writing.
Preliminary Checks - Formatting and Structure

[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Please place your article through a 3rd party plagiarism checker? We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution guidelines
For first-time contributors and for more details, see Contributing Guidelines

@louisefindlay23 and @srishilesh  here is the PR in a new branch and all changes

@corpsgeek, thanks for making a new branch. However, your forked repo seems to still being using the old repo structure (because the articles directory exists instead of /content/articles).
In order for me to final review your article, you should:

First, download the most recent files off this PR
Next, delete your articles directory
Then, pull the upstream branch (main EngEd repo) into your forked repo
Finally, upload your article files to the new articles folder inside content ie. /content/articles.

Let me know if you have any issues.
cc - @srishilesh

I guess its fixed now @louisefindlay23 ?

I guess it is fixed now @louisefindlay23 ?

@corpsgeek
Doesn't look like it's fixed. Your last commit shows changes in more than 6000 files.

Yeah, I pulled from the master at the same time it uploaded other folders structure on my end.

updated*

The alternative route would be for me to fork the master branch afresh and upload the article then create a new PR

The alternative route would be for me to fork the master branch afresh and upload the article then create a new PR

@corpsgeek
Yep. I believe that would be the easiest way to go forward.
Make sure to add the latest reviewed files.
Ping me and @louisefindlay23 after you create a new PR.
Thanks

@corpsgeek, thanks for making a new branch. However, your forked repo seems to still being using the old repo structure (because the articles directory exists instead of /content/articles).
In order for me to final review your article, you should:

First, download the most recent files off this PR
Next, delete your articles directory
Then, pull the upstream branch (main EngEd repo) into your forked repo
Finally, upload your article files to the new articles folder inside content ie. /content/articles.

Let me know if you have any issues.
cc - @srishilesh

@louisefindlay23
Just a quick question. I failed to notice this earlier.
I believe these files follow the expected directory structure /content/articles/. Not sure if you had sent this message by mistake.
Thanks.


Consider using variadic functional arguments

Currently some methods support optional configuration parameters wrapped in a struct.
Although the function signature is variadic in nature, only the first argument will be used if one is supplied. E.g.,
opts := InboxOptions{}
if len(options) >= 1 {
	opts = options[0]
	if err := opts.validate(); err != nil {
		return nil, err
	}
}

It is the caller's responsibility to only ever pass a single optional argument.
An alternative may be to create functional options for each option. The benefit would be:

configurability
self documenting options
reusability between similar options. E.g., inbox, outbox and usage all use Period, StartDate and EndDate

Satisfied with using structs as options for now.


SafariURLHandler: missing "public" access for externally configurable variables

Hi,
Thanks for the great work of making this happen - it helped us not to reinvent the wheel big time!
I came across the issue where the following variables on SafariURLHandler weren't accessible from outside of the framework (missing public annotiation):
    var animated: Bool = true
    var delegate: SFSafariViewControllerDelegate?
    var presentCompletion: (() -> Void)?
    var dismissCompletion: (() -> Void)?

Since I needed to customise the behaviour of this class I needed to set these - and at the same time I thought these were probably intended to be externally configurable originally. (otherwise these are not used inside the framework, so there's no point of their existence:)
Cheers,
Mark

Yes public is missing, on "factory" too
thanks for the PR


More-native Glint integration?

I don't have a deep enough understanding of how XState's types work to provide super precise input here, but based on a shallow reading it seems like it should be possible to avoid needing the asComponent function here, instead doing something like this in src/glint.ts:
declare module 'xstate' {
  export interface StateMachine<...> /* or maybe `StateNode`? */
    extends ComponentLike<{
      // derived from the zillion type params on `StateMachine`
    }> {}
}

// Maintain as just the identity function for backwards compatibility
export function asComponent<T extends AnyStateMachine>(machine: T): T {
  return machine;
}

Then downstream consumers would only need to import 'ember-statechart-component/glint' to "activate" the Glint integration and their other code could stay as-is.

yeah! -- it would be ideal if there wasn't a an asComponent function to use. I had a go at re-defining StateMachine -- but it was getting hairy -- I then looked at the implementation: https://github.com/statelyai/xstate/blob/780458c921d4525c7a00119c7eb43d4833978861/packages/core/src/types.ts#L1027 -- and I'm not sure if it makes sense to re-define all that from a maintenance perspective.
And I know XState v5 is around the corner -- not sure what changes are coming there -- but maybe this'll be a good thing to explore in v5?

You shouldn't need to redefine anything—the interface you define there will be merged with the one from xstate itself.

ah ok, well, I think this does it: https://github.com/NullVoxPopuli/ember-statechart-component/pull/417
:partying_face:
Thank you!!


Intelligently rerun nbdev_export in nbdev_merge for more automated conflict resolution

I've had a few instances now where notebook conflicts were resolved but where the modules and/or _modidx were still in a conflicted state. We might be able to run nbdev_export in the merge driver to fix that. I haven't yet thought about how complicated the logic would need to be.

@seeM is the logic as simple as this?: if there are no conflicted notebooks, then run nbdev_export
Or alternatively - would we always want to try to run it? Is there a reason we ever wouldn't?

Good questions. One possible issue with always running is that the conflict markers would be excluded from exported modules since they’re markdown cells, so it would hide conflicts.
Exporting only notebooks without remaining conflicts wouldn’t have that issue. If we’re only exporting certain notebooks though, we might have issues with notebooks that export to multiple modules or multiple notebooks exporting to one module.
Alternatively, we could try to make export retain conflicts then always run it?

One possible issue with always running is that the conflict markers would be excluded from exported modules since they’re markdown cells, so it would hide conflicts.

That might be OK though - because you'll know that the source NB has a conflict. And once you've fixed that, you just export.

Hmm yea that makes sense. Let's try it out. I'll make a tiny PR. We can always roll back if it doesn't work well

Actually it's a little trickier because merge runs per-file. We'd need some way of skipping merging .py files and instead use the exported ones. Or some sort of post-conflict hook that exports a single time after everything else

For .py files can we just always use an "ours" strategy if they are exported from a notebook (i.e they have a cell marker somewhere in them)?

Alternatively, could we try to make export retain conflicts?
On Mon, 05 Sep 2022 at 09:43, Wasim Lorgat @.***> wrote:

Good questions. One possible issue with always running is that the
conflict markers would be excluded from exported modules since they’re
markdown cells, so it would hide conflicts.
Exporting only notebooks without remaining conflicts wouldn’t have that
issue. If we’re only exporting certain notebooks though, we might have
issues with notebooks that export to multiple modules. For example if nb_a
and nb_b both export to mod_a, it gets tricky if one nb has a conflict and
the other doesn’t. Does that make sense?
On Mon, 05 Sep 2022 at 08:01, Jeremy Howard @.***>
wrote:

@seeM https://github.com/seeM is the logic as simple as this?: if
there are no conflicted notebooks, then run nbdev_export
Or alternatively - would we always want to try to run it? Is there a
reason we ever wouldn't?
—
Reply to this email directly, view it on GitHub
https://github.com/fastai/nbdev/issues/880#issuecomment-1236405759, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEISACMVECECNU3OVG4NPTV4T52NANCNFSM56XTN7BQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ignore the last message. I’m pretty sure I typed that one month ago. No idea why it just sent now!

Good questions. One possible issue with always running is that the conflict
markers would be excluded from exported modules since they’re markdown
cells, so it would hide conflicts.
Exporting only notebooks without remaining conflicts wouldn’t have that
issue. If we’re only exporting certain notebooks though, we might have
issues with notebooks that export to multiple modules. For example if nb_a
and nb_b both export to mod_a, it gets tricky if one nb has a conflict and
the other doesn’t. Does that make sense?
On Mon, 05 Sep 2022 at 08:01, Jeremy Howard @.***>
wrote:

@seeM https://github.com/seeM is the logic as simple as this?: if there
are no conflicted notebooks, then run nbdev_export
Or alternatively - would we always want to try to run it? Is there a
reason we ever wouldn't?
—
Reply to this email directly, view it on GitHub
https://github.com/fastai/nbdev/issues/880#issuecomment-1236405759, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEISACMVECECNU3OVG4NPTV4T52NANCNFSM56XTN7BQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Decode the database name in SQL/HTTP connections

A url::Url does not hand you back a URL decoded value for path values,
so we must decode them ourselves.
Link: https://docs.rs/url/2.5.2/url/struct.Url.html#method.path
Link: https://docs.rs/url/2.5.2/url/struct.Url.html#method.path_segments
Signed-off-by: Tristan Partin<EMAIL_ADDRESS>

Huge footgun, imo. This seems to have been an issue since this feature was introduced.


gocql.Unmarshaler with GetRelease

Hi, I have a problem with UnmarshalCQL. I have a struct that implements the gocql.Unmarshaler interface. What I do is this:
**** m implements gocql.Unmarshaler *****
var m = MyClass{}
stmt, names := qb.Select("my_table").Where(qb.EqLit("id", id), qb.Eq("another_key")).ToCql()
err := database.S.Query(stmt, names).Bind(anotherVar).GetRelease(&m)
I expect
the library to call my Unmarshaler.
And what I get is this
err:expected 1 column in result while scanning scannable type struct but got 24
version 2.3.0, interx.go:96
I am also sure that with where clause containing key for id and another_key being anotherVar there has to be at least 1 result
Am I understanding something wrong or what?

// Unmarshaler is the interface implemented by objects that can unmarshal
// a Cassandra specific description of themselves.
type Unmarshaler interface {
	UnmarshalCQL(info TypeInfo, data []byte) error
}

How do you expect that to work with 24 columns?

When I print out the stmt, and put in the value of anotherVar ( its 0) by myself instead of placeholder "?", I get a query, if I execute that query on the database I get only one column

https://pkg.go.dev/github.com/scylladb/gocqlx?utm_source=godoc#readme-example
var p Person
stmt, names := qb.Select("gocqlx_test.person").Where(qb.Eq("first_name")).ToCql()
q := gocqlx.Query(session.Query(stmt), names).BindMap(qb.M{
"first_name": "Patricia",
})
if err := q.GetRelease(&p); err != nil {
t.Fatal(err)
}
Let's compare it with this example, the whole Person struct has the interface implemented, why wouldn't that work?

My motivation for this is that I have a lot of code using that struct, and that struct has a few fields of type string that store datetime, I do not want to change those string types into time.Time; Do you have a better solution for me?

Then you should use a custom type for the string encoded time that would implement the unmarshaller.
Consider this example https://github.com/scylladb/gocqlx/blob/master/iterx_test.go#L96.

I considered that, thanks. Seems like I will have to go with that approach. But any ideas why wouldn't my approach work?

Unmarshaller will work if you manually dump everything to a single column say encode it as json.


jetson nano repvgg make failed

Env
GPU, Nano
OS,
Cuda version 10.2
TensorRT version
About this repo
repvgg
Your problem
sudo ./repvgg -s RepVGG-B2 // serialize model to plan file i.e. 'RepVGG-B2.engine'
finished load weights
erorr @ 281line

APIToModel(netName, MAX_BATCH_SIZE, &modelStream);// failed


Duplicate entry for FELIX_WIREGUARDMTU

Description

In the v3.15 manifest for Calico, there is a duplicate entry for FELIX_WIREGUARDMTU. This prevents it from being applied.
Docs suggest that WireGuard can only be enabled when CALICO_NETWORKING_BACKEND=none:
https://github.com/projectcalico/calico/blob/8ce59324b6cba4d6e82999e7429a8d6d7e930d4a/security/encrypt-cluster-pod-traffic.md#L25
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 9120bc8ded49e68a23dfc0165b360985ff5d93fe
https://deploy-preview-3725--calico.netlify.app

This is the error from kube-apiserver.log:
E0630 05:47:35.072855       1 fieldmanager.go:159] [SHOULD NOT HAPPEN] failed to update managedFields for /,
Kind=: failed to convert new object (/v1, Kind=Pod) to smd typed: .spec.containers[name="calico-node"].env:
duplicate entries for key [name="FELIX_WIREGUARDMTU"]


Backers design

Hi!
Currently splash page shows backers list, like this:

I checked Splash design https://github.com/skipjack/webpack-design/blob/master/Splash.psd, but not found information about how should it look.
I want to help fixing it!

@montogeek yea it definitely needs fixing. The design is pretty outdated at this point so I wouldn't refer to it anymore and the sponsor's section was added recently.
Did you have an idea about how to fix it? I was thinking about trying to wrap them but since it's an svg it's not easy fix. Let me know your thoughts, and check out #260 for the latest splash page design.

@skipjack I think this can be closed.


Update recipe for py312_patch

Preparation for py312 release.

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: clone/recipe/meta.yaml:33: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
Errors were found

LGTM, though I share the same curiosity regarding having both {{ pin_compatible('scipy') }} and scipy. Is this based on whether py<312 or py==312? If so, should we add that distinction?

That was a mistake, I have removed the extra version.

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: clone/recipe/meta.yaml:33: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: clone/recipe/meta.yaml:33: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
Errors were found


User enumeration via timing attack

Rancher versions:
Verified in rancher/rancher: 2.0.7
Steps to Reproduce:
Uses this proof of concept: here
Results:
The bcrypt function is only evaluated when the user exists. This causes a difference in the response time between an existing user and a non-existing user.
Can you confirm this?

I'm going to fix this. Will do a PR soon. :smile:

For testing, beyond attempting to observe a difference in response time for a user that exist vs one that doesnt, ensure that logging in as a local user still works in order to test for regresssions.

thanks @codexlynx

@cjellick The fix cannot be validated on master 11/20
there are still apparent difference  in response time for a user that exists vs one that doesn't.
The following are testing result on rancher v2.0.7 and master
-------- v2.0.7 -------
existing user 1
0.3913440704345703
existing user 2
0.37647271156311035
non-existing user
0.5524568557739258
-------- master -------
existing user 1
0.38080739974975586
existing user 2
0.37538599967956543
non-existing user
0.5406830310821533

Here are the response times as per v2.7-head:

Valid User
Invalid User

0.439390
0.455134

0.478864
0.456086

0.482091
0.471082

0.484299
0.476599

0.493461
0.484188

0.499115
0.484880

0.501356
0.495628

0.519505
0.508600

0.525394
0.511593

0.542414
0.561285

They are fairly close together, having averages of 0.496589 and 0.490508 respectively.
Considering that the fix was merged a few years back, we're closing this issue for now.
Feel free to reopen it if you have any further questions or if there's new information to add.


Replace IBM FHIR Server in /metadata CapabilityStatement

we have at least 3 IBM references left in the CapabilityStatement resources that we server in LinuxForHealth FHIR Server 5.0.0:  https://github.com/LinuxForHealth/FHIR/blob/main/fhir-server/src/main/java/org/linuxforhealth/fhir/server/resources/Capabilities.java#L116-L118
lets get those fixed up for 5.1.0

the string "IBM" no longer appears in our capabilitystatement (even when mode=terminology)


#253 - Updating logo for the yii-logo-light in README.md

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
#253

Updating logo in README.md for a transparent one.

👍


CORE-7954: Allow us to discover all bundles containing system flows.

Identify all bundles containing "system flows" so that we can reduce the number of components being declared as "system services".
While I've currently only implemented a new corda.system=flow capability, we could also create capabilities corda.system=persistence and corda.system=verification should the need arise.
The idea is that bundles containing classes which are recognised as being "system flows" are included along with a sandbox's "main" jars in its "bundle wiring" calculations. There's no realistic prospect of determining whether a particular CPK will use those system flows, though.

Based on my understanding, looks plausible. Does the consensual ledger demo pass with this change?

If the consensual ledger demo is currently enabled, then "yes".


AMBARI-24388. Install HSI using Add HiveServer2 Interactive button fa…

…iling
What changes were proposed in this pull request?
Install HSI using Add HiveServer2 Interactive button failing
How was this patch tested?
manually
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3379/
Test FAILed.
Test FAILured.


Bitbucket Provider

https://stackoverflow.com/questions/29887925/how-do-i-make-a-releases-builds-and-or-snapshots-with-bitbucket


Fix build docs

Install Erlang from source on Ubuntu 16.04.
Erlang solution repository would be better, but we don't support latest erlang
(21) yet.
https://www.pivotaltracker.com/story/show/159649299

I see two approvals. Merging.


Compiler error for counter example (nRF52_pca10040)

I'm trying to compile counter example for nrf52_pac10040 board
Examples:  samples/drivers/counter/alarm_
but getting an error
-- Configuring done
CMake Error at ../../../../cmake/extensions.cmake:418 (add_library):
No SOURCES given to target: drivers__counter
Call Stack (most recent call first):
../../../../cmake/extensions.cmake:395 (zephyr_library_named)
../../../../drivers/counter/CMakeLists.txt:3 (zephyr_library)

Added CONFIG_RTC0=y
It's compiling now.


Extension causes high cpu load

Issue Type: Performance
Extension Name: vsliveshare-audio
Extension Version: 0.1.48
OS Version: Windows_NT x64 10.0.17134
VSCode version: 1.33.0

:warning: Make sure to attach this file from your home-directory:
:warning:C:\Users\jhdra\ms-vsliveshare.vsliveshare-audio-unresponsive.cpuprofile.txt
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

Perhaps the same root cause as 1824. We should make those same improvements to the audio extension.

Perhaps the same root cause as 1824. We should make those same improvements to the audio extension.

Perhaps the same root cause as 1824. We should make those same improvements to the audio extension.

Perhaps the same root cause as 1824. We should make those same improvements to the audio extension.

I am also experiencing live share using 100% cpu constantly while joined in a session.

I don't know if it is related. But while joined in a session VS code seems to constantly work on "Indexing Ruby source files".

Also, the 100% cpu problem only occurs when I join someone elses sessions. Not when I host one myself.

This issue should now be resolved. Please re-open it if you're still experiencing this problem. Thanks!


Rename rg3d to Fyrox in README

Objective
rg3d has been renamed to Fyrox for a little while now and we should reflect that. It's not particularly urgent since the old link redirects to Fyrox.
Solution
Rename rg3d to Fyrox in the README

bors r+


chore: split Data.Matrix.ConjTranspose from Basic

Possible next steps:

Split RowCol/Invertible/Block to delay the star stuff
Invert the import order, moving star stuff from RowCol/Invertible/Block into ConjTranspose

From https://leanprover.zulipchat.com/#narrow/channel/287929-mathlib4/topic/late.20imports/near/480550246

Thanks :tada:
If CI passes, please remove the label awaiting-CI and merge this yourself, by adding a comment bors r+.
bors d+

bors merge


create shortcut for files

What feature or improvement do you think would benefit Files?
a feature that can make a shortcut to a folder in that opens in files
Requirements
alow to create a shortcut that opens a folder in files
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22622.590
Windows Version
22622.590
Comments
No response

You can right click a folder and make a shortcut. Then move it to where you want.


Add map for resolving of type by enum of oneof

Added: A new static const map in class that is built on 'oneof'
Why: This map allows to resolve this object to the specific type it holds: using whichSubtype() as the map index, you get the specific class type.
Important: This PR does not change or remove any existing features, only adds a new field.
The field created will be (for example):
  static const $core.Map<ApiObj_Subtype, $core.Type> TypeByApiObj_Subtype = {
    ApiObj_Subtype.batchApiObj : BatchApiObj,
    ApiObj_Subtype.authenticationRequest : AuthenticationRequest,
    ApiObj_Subtype.authenticationResponse : AuthenticationResponse,
  };

Example: Very useful for implementation of an Api Bus which can dispatch messages to callbacks according to the message type:
class ApiBus {
  Map _callbackByMessageType = new Map<Type, List<Function>>();

  void publish(ApiObj msg) {
    var type = ApiObj.TypeByApiObj_Subtype[msg.whichSubtype()];

    var callbacks = _callbackByMessageType[type];
    if (null == callbacks) {
      return;
    }
    for (int c = 0; c < callbacks.length; c++) {
      callbacks[c](msg);
    }
  }

  void subscribe<T>(void callback(ApiObj message)) {
    if (null == _callbackByMessageType[T]) {
      _callbackByMessageType[T] = List<Function>();
    }
    _callbackByMessageType[T].add(callback);
  }
}

@sigurdm Can you please take a look at this PR?


401's in aurelia-api are not redirecting to login via aurelia-authentication

I'm using aurelia-api v3.1.1 and aurelia-authentication v3.1.0 and calls to  aurelia-api endpoints that result in 401's are not redirecting to login via aurelia-authentication.
I'm using aurelia-authentication with simple username/password against my own backend -- I have a login page and a logout page.
my aurelia-api configs are as in the docs
my authConfig is
	storageKey: (/localhost|local.host|<IP_ADDRESS>|<IP_ADDRESS>/.test(window.location.hostname)) ? 'localKey' : 'serverKey',
	configureEndpoints: ['auth', 'api', 'notes', 'history', 'ui'], // these want the bearer token
	endpoint: 'auth',
	loginUrl: 'login',
	loginRoute: 'login',
	storageChangedReload: true,    // ensure secondary tab reloading after auth status changes
	expiredRedirect: 1,            // redirect to logoutRedirect after token expiration

whoops, my mistake -- the redirects are working as long as the calls are happening in code that is on a route
I have some code that runs in a static header, I'll have to catch errors in those and re-route myself

👍  for finding it yourself. That always makes me happy :D

Hmm, for me its not working. Might just be my mistake too but I don't know if I'm missing something.
I had an API call in a non-route but with it commented out it isn't working either. (don't know if this matters)
authconfig
          baseConfig.configure({
              endpoint: 'auth',
              loginUrl: '/login',
	      loginRoute: 'login',
	      storageChangedReload: true,
            //   loginRedirect: '/',
              authTokenType: 'JWT',
              configureEndpoints: ['auth', 'api'],
          });

And my API call is in route /#/tasks (constructor)
    this.apiEndpoint = config.getEndpoint('api');

    this.apiEndpoint.find('tasks')
      .then(tasks => {
        this.tasks = tasks;
        this.setupValidation();
      })
      .catch(console.error);

GET http://localhost:3000/api/tasks 401 (Unauthorized)
Response {type: "cors", url: "http://localhost:3000/api/tasks", redirected: false, status: 401, ok: false…}
Any ideas what I could be missing?

@bennibeef Did you configure the login redirect? https://aurelia-authentication.spoonx.org/baseConfig.html


[Bug] Operator should DECOMMISSION BE when Scale-in the cluster

Currently, operator just simply shutdown BE node when updating the replica number of BE.
This will cause data loss.
It should be changed to use DECOMMISSION  to scale-in the cluse.
As DECOMMISSION is an async operation, operator must monitor its status until it is completed before proceeding with the lower-level Kubernetes statefulset scale-in.
cc @kevincai @shileifu @imay

We second that! Currently we always have to do manual decommission before we reduce the replica in the statefulset


Correction for raw affiliation CIRAD, UMR PHIM, Montpellier, France; PHIM, Plant Health Institute, Univ Montpellier, INRAe, CIRAD, Institut Agro, Montpellier, France

Correction needed for raw affiliation CIRAD, UMR PHIM, Montpellier, France; PHIM, Plant Health Institute, Univ Montpellier, INRAe, CIRAD, Institut Agro, Montpellier, France
raw_affiliation_name: CIRAD, UMR PHIM, Montpellier, France; PHIM, Plant Health Institute, Univ Montpellier, INRAe, CIRAD, Institut Agro, Montpellier, France
new_rors: 051escj72;03fkrcm40;03rnk6m14;05kpkpg04;05y5s3h28
previous_rors: 051escj72;03fkrcm40;03rnk6m14;05kpkpg04
works_examples: W3129193919
contact: b86ff6f0893d93f50f41110eaece2368:58287c009d05c2137b8356098fb084d4ebc8 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


JavaScript (JS), Theme (LESS/CSS) and CSHTML/HTML files are not indexed and unable to search in these files

Let me know how do we bring the JS, CSS/LESS and CSHTML/HTML files to the source browser with search indexing feature.

Unfortunately SourceBrowser only supports C# via Roslyn.
We'd need a language service to understand .js, .css, .html, .cshtml etc. files to enable semantic search there.

I'll close this as a duplicated of #65.


Update sbt-scala3-migrate to 0.6.4

About this PR
📦 Updates ch.epfl.scala:sbt-scala3-migrate from 0.5.1 to 0.6.4
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala", artifactId = "sbt-scala3-migrate" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "ch.epfl.scala", artifactId = "sbt-scala3-migrate" }
}]

labels: sbt-plugin-update, early-semver-major, semver-spec-minor, commit-count:1

Superseded by #1046.

Superseded by #1056.


Allow callers to set custom HTTP headers

The Android implementation of HttpURLConnection has a bug that causes it to submit requests in an infinite loop when an HTTP 401 response is returned from an authenticated request (i.e. the caller provided a username/password, but the server still returned 401).
One workaround is to pass the credentials with the initial request so that the authenticator never needs to be called. However, in order to do this, the caller would need to have access to the request headers, which the proxy classes do not currently provide.
To address this and other potential use cases requiring direct header access, the web service proxy classes should be updated to include an API for specifying custom headers.

Since there is currently no way to access headers in an HTTP-RPC service implementation, this may be overkill. It might be better to simply allow the caller to set a username/password on the proxy instance and automatically pass the Authorization header when present. Basic authentication could be supported initially, and extended to other authentication forms in the future if needed.

An abstract authentication class with a concrete implementation for Base64 authentication might be the best approach. An "authentication" property could be added to the proxy classes, which would be called prior to executing the actual request. For example, the Java version might be defined as follows:
public interface Authentication {
    public void authenticate(NSURLConnection connection);
}

Custom headers is not the correct solution. Will create a new issue for adding customizable authentication support.


Image preview zooming issue

VSCode Version: 1.21.0-ins
OS Version: win7x64

When you try to zoom an image, using "Ctrl + Mouse Wheel scrolling"....
scrolling it up (outwards) - will make an image zoomed out...
scrolling it down (to you) - will make an image zoomed in..
How come?
Why do we need this inversion?

Duplicate of #43138


Quilt: Internal Error (Something Went Wrong While Loading Your Game)

Trying out quilt for the first time using modrinth, however it hasn't been working. I know/heard that you can use most fabric mods on quilt thus I made a massive quilt and fabric modpack which I have been trying to use but it keeps bringing the same error that it was an internal problem. I have made my modpack smaller to see if it was a mod or not. However, issues pursist. Please Help!

Mods:
Crash Report [Mods].txt
Quilt Loader Version: 0.23.1

-- Crash --

org.quiltmc.loader.impl.FormattedException: java.lang.BootstrapMethodError: java.lang.RuntimeException: Mixin transformation of net.minecraft.class_3988 failed
	at org.quiltmc.loader.impl.game.minecraft.MinecraftGameProvider.launch(MinecraftGameProvider.java:553)
	at org.quiltmc.loader.impl.launch.knot.Knot.launch(Knot.java:84)
	at org.quiltmc.loader.impl.launch.knot.KnotClient.main(KnotClient.java:28)
Caused by: java.lang.BootstrapMethodError: java.lang.RuntimeException: Mixin transformation of net.minecraft.class_3988 failed
	at net.minecraft.class_1299.<clinit>(class_1299.java:270)
	at net.minecraft.class_3103.<clinit>(class_3103.java:28)
	at net.minecraft.class_3031.<clinit>(class_3031.java:84)
	at net.minecraft.class_6800.<clinit>(class_6800.java:20)
	at net.minecraft.class_6803.method_39702(class_6803.java:24)
	at net.minecraft.class_5458.method_44104(class_5458.java:98)
	at net.minecraft.class_5458.method_30566(class_5458.java:105)
	at java.base/java.util.LinkedHashMap.forEach(Unknown Source)
	at net.minecraft.class_5458.<clinit>(class_5458.java:104)
	at net.minecraft.class_2378.<clinit>(class_2378.java:326)
	at net.minecraft.class_2966.method_12851(class_2966.java:50)
	at net.minecraft.client.main.Main.method_44604(Main.java:161)
	at net.minecraft.client.main.Main.main(Main.java:51)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.base/java.lang.reflect.Method.invoke(Unknown Source)
	at org.quiltmc.loader.impl.game.minecraft.MinecraftGameProvider.launch(MinecraftGameProvider.java:551)
	... 2 more
Caused by: java.lang.RuntimeException: Mixin transformation of net.minecraft.class_3988 failed
	at org.quiltmc.loader.impl.launch.knot.KnotClassDelegate.getPostMixinClassByteArray(KnotClassDelegate.java:459)
	at org.quiltmc.loader.impl.launch.knot.KnotClassDelegate.tryLoadClass(KnotClassDelegate.java:241)
	at org.quiltmc.loader.impl.launch.knot.KnotClassDelegate.loadClassOnly(KnotClassDelegate.java:161)
	at org.quiltmc.loader.impl.launch.knot.KnotClassDelegate.loadClass(KnotClassDelegate.java:147)
	at org.quiltmc.loader.impl.launch.knot.KnotClassLoader.loadClass(KnotClassLoader.java:239)
	at java.base/java.lang.ClassLoader.loadClass(Unknown Source)
	at java.base/java.lang.ClassLoader.defineClass1(Native Method)
	at java.base/java.lang.ClassLoader.defineClass(Unknown Source)
	at java.base/java.security.SecureClassLoader.defineClass(Unknown Source)
	at org.quiltmc.loader.impl.launch.knot.KnotClassLoader.defineClassFwd(KnotClassLoader.java:321)
	at org.quiltmc.loader.impl.launch.knot.KnotClassDelegate.tryLoadClass(KnotClassDelegate.java:296)
	at org.quiltmc.loader.impl.launch.knot.KnotClassDelegate.loadClassOnly(KnotClassDelegate.java:161)
	at org.quiltmc.loader.impl.launch.knot.KnotClassDelegate.loadClass(KnotClassDelegate.java:147)
	at org.quiltmc.loader.impl.launch.knot.KnotClassLoader.loadClass(KnotClassLoader.java:239)
	at java.base/java.lang.ClassLoader.loadClass(Unknown Source)
	... 20 more
Caused by: org.spongepowered.asm.mixin.transformer.throwables.MixinTransformerError: An unexpected critical error was encountered
	at org.spongepowered.asm.mixin.transformer.MixinProcessor.applyMixins(MixinProcessor.java:392)
	at org.spongepowered.asm.mixin.transformer.MixinTransformer.transformClass(MixinTransformer.java:234)
	at org.spongepowered.asm.mixin.transformer.MixinTransformer.transformClassBytes(MixinTransformer.java:202)
	at org.quiltmc.loader.impl.launch.knot.KnotClassDelegate.getPostMixinClassByteArray(KnotClassDelegate.java:454)
	... 34 more
Caused by: org.spongepowered.asm.mixin.transformer.throwables.MixinPreProcessorException: Attach error for #origins-classes:origins-classes.mixins.json:MerchantEntityMixin from mod origins-classes during activity: [Transform -> Method buildAdditionalOffers()Lnet/minecraft/class_1916; -> GETSTATIC -> net/minecraft/class_7923::field_41178:Lnet/minecraft/class_7922;]
	at org.spongepowered.asm.mixin.transformer.MixinPreProcessorStandard.attach(MixinPreProcessorStandard.java:313)
	at org.spongepowered.asm.mixin.transformer.MixinPreProcessorStandard.createContextFor(MixinPreProcessorStandard.java:277)
	at org.spongepowered.asm.mixin.transformer.MixinInfo.createContextFor(MixinInfo.java:1289)
	at org.spongepowered.asm.mixin.transformer.MixinApplicatorStandard.apply(MixinApplicatorStandard.java:294)
	at org.spongepowered.asm.mixin.transformer.TargetClassContext.apply(TargetClassContext.java:421)
	at org.spongepowered.asm.mixin.transformer.TargetClassContext.applyMixins(TargetClassContext.java:403)
	at org.spongepowered.asm.mixin.transformer.MixinProcessor.applyMixins(MixinProcessor.java:363)
	... 37 more
Caused by: java.lang.RuntimeException: java.lang.ClassNotFoundException: net.minecraft.class_7923
	at
 org.spongepowered.asm.mixin.transformer.MixinPreProcessorStandard.transformMemberReference(MixinPreProcessorStandard.java:786)
	at org.spongepowered.asm.mixin.transformer.MixinPreProcessorStandard.transformField(MixinPreProcessorStandard.java:777)
	at org.spongepowered.asm.mixin.transformer.MixinPreProcessorStandard.transform(MixinPreProcessorStandard.java:740)
	at org.spongepowered.asm.mixin.transformer.MixinPreProcessorStandard.attach(MixinPreProcessorStandard.java:307)
	... 43 more
Caused by: java.lang.ClassNotFoundException: net.minecraft.class_7923
	... 47 more

Without the full crashlog, I can't completely diagnose the problem, but it looks like the version of origin-classes you have might be for a different minecraft version.

Without the full crashlog, I can't completely diagnose the problem, but it looks like the version of origin-classes you have might be for a different minecraft version.

Hey thanks for pointing it out! It didnt fix the crash, however I was able to find the full crash log and it had something to do with my model gap fix mod and continuity.


cargo update demands an ecdsa key for github.com

Problem
Since recently, cargo update started to complain about an unknown SSH host key for github.com, suggesting that github's ecdsa key be added to known_hosts:
$ cargo update
    Updating crates.io index
error: failed to get `anyhow` as a dependency of package `electrs v0.9.13 (/home/intelfx/devel/ext/electrs)`

Caused by:
  failed to load source for dependency `anyhow`

Caused by:
  Unable to update registry `crates-io`

Caused by:
  failed to fetch `https://github.com/rust-lang/crates.io-index`

Caused by:
  error: unknown SSH host key
  The SSH host key for `github.com` is not known and cannot be validated.

  To resolve this issue, add the host key to the `net.ssh.known-hosts` array in your Cargo configuration (such as /home/intelfx/.cargo/config) or in your OpenSSH known_hosts file at /home/intelfx/.ssh/known_hosts

  The key to add is:

  github.com ecdsa-sha2-nistp256 AAAAE2VjZHNhLXNoYTItbmlzdHAyNTYAAAAIbmlzdHAyNTYAAABBBEmKSENjQEezOmxkZMy7opKgwFB9nkt5YRrYMjNuG5N87uRgg6CLrbo5wAdT/y6v0mKV0U2w0WZ2YB/++Tpockg=

  The ECDSA key fingerprint is: SHA256:p2QAMXNIC1TJYWeIOttrVc98/R1BUFWu3/LiyKgUfQM
  This fingerprint should be validated with the server administrator that it is correct.

  See https://doc.rust-lang.org/stable/cargo/appendix/git-authentication.html#ssh-known-hosts for more information.

(I traced this down to my local configuration that disables ECDSA host key algorithms in HostKeysAlgorithm, thus causing regular OpenSSH client to automatically remove any ECDSA keys from known_hosts, together with a configuration that rewrites HTTPS github.com URLs to ssh ones.)
However, I have other perfectly valid keys for github.com in my known_hosts. cargo should use them instead of demanding a host key in a specific cryptographic system and failing if one is not available.
Steps

Create an empty ~/.ssh/known_hosts and add GitHub keys to it, skipping the ECDSA key:$ ssh-keyscan github.com | grep -v ecdsa > .ssh/known_hosts

Make any cargo request that contacts GitHub over ssh (e. g. add an insteadOf statement in your .gitconfig, drop the registry index cache and do a search):$ cat >>~/.gitconfig <<EOF
[url "git@github.com:"]
       insteadOf = git://github.com/
       insteadOf = https://github.com/
EOF
$ rm -rf ~/.cargo/registry
$ cargo search syn

Possible Solution(s)
Depending on the actual implementation details: either do not force cargo's built-in SSH client to use a specific cryptosystem for verifying host keys, or do not make it a hard error if a specific host key is not available (but others are).
Notes
No response
Version
cargo 1.68.2
release: 1.68.2
host: x86_64-unknown-linux-gnu
libgit2: 1.5.0 (sys:0.16.0 vendored)
libcurl: 8.0.1 (sys:0.4.59+curl-7.86.0 system ssl:OpenSSL/3.0.8)
os: Arch Linux Rolling Release [64-bit]

The error you are getting is likely since #11556 , but before that PR it probably also just used ECDSA keys, just not verifying them at all. I think the problem is that cargo does not read HostKeysAlgorithm.

The fix is available on nightly since nightly-2023-04-12. Close as fixed. Thanks!


testutil: Add a semaphore build script

Consolidate all build commands into a single script which can also be
run outside of semaphore for greater reproducibility.
Signed-off-by: Rob Bradford<EMAIL_ADDRESS>

Coverage increased (+0.01%) to 65.395% when pulling 7443ddd68ab28dfcfcd716ee343e1660d72e82ef on rbradford:semaphore-build-script into fabdde43d25d4365ddc84a5727829bf7b13438c4 on ciao-project:master.

Coverage increased (+0.009%) to 65.39% when pulling 37c329d71817fed9614a87fa61f5c78800e31d12 on rbradford:semaphore-build-script into fabdde43d25d4365ddc84a5727829bf7b13438c4 on ciao-project:master.


Feature: adding Kafka transactions.

I would like FastStream to support transactions from aiokafka. I assume that its implementation should not be complicated and it is realized by adding one method to broker or publisher.
Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Clearly and concisely describe the desired outcome or solution.
Feature code example
At the moment there are no ideas about proper transaction design, but I will leave a link to the aiokafka documentation that shows how to work with transactions.
Describe alternatives you've considered
None.
Additional context
https://aiokafka.readthedocs.io/en/stable/examples/transaction_example.html

I assume that it is necessary to add a method to AioKafkaFastProducer, which returns the called method in the producer aiokafka method transtaction, and already in the object KafkaBroker to make access to this method.
class AioKafkaFastProducer:
...
def transaction(self) -> TransactionContext:
   return self._producer.transaction()

I assume that it is necessary to add a method to AioKafkaFastProducer, which returns the called method in the producer aiokafka method transtaction, and already in the object KafkaBroker to make access to this method.
class AioKafkaFastProducer:
...
def transaction(self) -> TransactionContext:
return self._producer.transaction()

Related to #1157

I think, we can close it as a copy


Audio continues when a new window is closed

Describe the bug
A second window opened at startup will continue to play audio
Steps To Reproduce
Steps to reproduce the behavior, just this:
#![cfg_attr(
    all(not(debug_assertions), target_os = "windows"),
    windows_subsystem = "windows"
)]

fn main() {
    tauri::Builder::default()
        .setup(|app| {
            tauri::WindowBuilder::new(
                app,
                "oops-this-will-close-but-audio-will-continue",
                tauri::WindowUrl::External("https://www.youtube.com/watch?v=JhSZw8fiTLc".parse().unwrap()),
            )
                .always_on_top(true)
                .build()?;
            Ok(())
        })
        .run(tauri::generate_context!())
        .expect("error while running tauri application");
}

Expected behavior
When I close the second window, I expect the youtube video to stop playing, but instead the audio continues in the background
Screenshots
If applicable, add screenshots to help explain your problem.
Platform and Versions (please complete the following information):
OS: macos
Rustc: 1.65
Would you assign yourself to resolve this bug?

[ ] Yes
[x] No

Additional context
Add any other context about the problem here.

This is a big blocker for me, is there anything I can do to help track down the bug ? 🙏🏻

Ok re-adding that fixes it, but a bigger issue seems to be the lack of automated test for this to prevent such a bad bug from popping up again in the future. I will have a think of a way to add one :)

16d1924 was actually the fix. the about:blank navigation merely resets the process, but doesn't close it which is what we actually want and what #556 implemented back then. But some commit between #556 and today broke it again (it indeed worked correctly, as in no zombie processes, for this timeframe)

ah sorry i messed up the PR numbers, #587 was the fix. #556 was only the reset workaround, to stop audio from playing.

at least I can help with a git bisect to find exact commit :)

this is the first bad commit https://github.com/tauri-apps/wry/commit/2eba8c9c26ff5f9512b0039ac04bc7fd27a5256f


changing the frame rate in one section of code

Hey, it's me again! I was wondering if there was a way to make a specific part of my code's frame rate slower than the rest? I tried using the frameRate() function inside of a push() pop() sandbox that contained the code I want to slow down, but it still changed the frame rate of my entire code.

Just to check, do you have a specific frame rate set for the code outside of the push() pop()?

@ReadReedRed No, I don't.

@kya-hover  Maybe give that a try; specifying in one place and not in the bulk might be what's overwriting the default. Worth a shot, anyway!

@ReadReedRed Well, I tried setting the frame rate outside of my sandbox and it still had the same effect. Thanks for trying to help though! I still feel like it should be possible to have two different frame rates running, but I must be going about it wrong...


User-agent in request Object is truncated for nodejs app

User-agent in request Object is truncated for nodejs app
Code:
context.log(req.headers);
Output :
2016-11-11T19:20:51.513 { connection: 'Keep-Alive',
  accept: 'text/html',
  'accept-encoding': 'gzip',
  'accept-language': 'en-US',
  host: 'testfuncappm.azurewebsites.net',
  'max-forwards': '10',
  'user-agent': 'Mozilla/5.0',
  'upgrade-insecure-requests': '1',
  'x-liveupgrade': '1',
  'x-original-url': '/api/HttpTriggerNodeJS1…
………………………………………
}

Expected behavior
Here is Request bin output from same browser :  http://requestb.in/1ho30ok1?inspect
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.71 Safari/537.36

@mathewc Please close when it is released.

@lindydonna @mathewc  How can i get these new changes in my existing function app ?
I'm using latest function app Runtime (~0.9)

@prashanthmadi This should be ready by tomorrow.

Thanks @lindydonna

Guys, this isn't actually in the 1.0 release package. It didn't make it in time - it won't be out until our next bugfix release.

I'm reopening so that we close when this is released.

@prashanthmadi I spoke too soon earlier--you'll get the fix soon, in the next bugfix release.


M test

some change

created a branch, created a file and made a commit, and opened a pull request


ARTEMIS-3670 Fixing Divert Update tests

@jbertram this is a fix for that divert PR, if you could review this one please? (and merge it if you're ok with it).


UNZ_BADZIPFILE after resumed a file download, despite file being ok.

So, I'm using AFDownloadRequestOperation to download a zip file. This works fine. But, if network connection drops, then come back on again, it refused to unzip the file giving a UNZ_BADZIPFILE. Howeever, if I manually go in and check the device tmp dire the downloaded zip file is there and I can copy it to desktop and unzip it on my Mac, so file does seem to be correct.
Any ideas?

Have you figured out what was wrong, @helloniklas / @oscarvgg?

From what I've see (in my own testing), this bug has been fixed. Not sure if this was fixed by someone in the SSZipArchive community, or if this was a problem with AFDownloadRequestOperation.


Dart Analyzer Error

Analyzer Feedback from IntelliJ
Version information

IDEA AI-183.6<IP_ADDRESS>92245
2.4.0
AI-183.6<IP_ADDRESS>92245, JRE 1.8.0_152-release-1343-b01x64 JetBrains s.r.o, OS Windows 10(amd64) v10.0 , screens 1920x1080, 1080x1920

Exception
Dart analysis server, SDK version 2.4.0, server version , FATAL error: NoSuchMethodError: The getter 'path' was called on null.
Receiver: null
Tried calling: path
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:50:5)
#1      Driver._getSdkPath (package:analysis_server/src/server/driver.dart:760:9)
#2      Driver.start (package:analysis_server/src/server/driver.dart:394:28)
#3      main (file:///C:/b/s/w/ir/k/src/third_party/dart/pkg/analysis_server/bin/server.dart:12:11)
<asynchronous suspension>
#4      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:299:32)
#5      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:12)

For additional log information, please append the contents of
file://C:\Users\skha436\AppData\Local\Temp\report1.txt.

Thanks for reporting this issue! It was previously reported at #35226; please follow along there.

Duplicate of #35226


[core] Adding host tags via the -e flag does not work

**Output of the info page **
====================
Collector (v 5.17.2)
====================

  Status date: 2017-11-30 19:46:20 (12s ago)
  Pid: 27
  Platform: Linux-4.4.0-1030-aws-x86_64-with-debian-8.9
  Python Version: 2.7.13, 64bit
  Logs: <stderr>, /var/log/datadog/collector.log

  Clocks
  ======

    NTP offset: 0.0032 s
    System UTC time: 2017-11-30 19:46:32.949955

  Paths
  =====

    conf.d: /etc/dd-agent/conf.d
    checks.d: /opt/datadog-agent/agent/checks.d

  Hostnames
  =========

    ec2-hostname: ip-10-0-114-178.ec2.internal
    local-ipv4: <IP_ADDRESS>
    local-hostname: ip-10-0-114-178.ec2.internal
    hostname: web-ops-art-hut-rap
    socket-fqdn: 1acd0644f1e4
    agent-hostname: web-ops-art-hut-rap
    instance-id: i-0aa108455fb0da4b4

  Checks
  ======

    check_number_docker_containers (custom)
    ---------------------------------------
      - Collected 0 metrics, 0 events & 0 service checks

    check_deploy_state (custom)
    ---------------------------
      - Collected 0 metrics, 0 events & 0 service checks

    process (5.17.2)
    ----------------
      - instance #0 [OK]
      - instance #1 [OK]
      - Collected 34 metrics, 0 events & 2 service checks

    ntp (5.17.2)
    ------------
      - Collected 0 metrics, 0 events & 0 service checks

    disk (5.17.2)
    -------------
      - instance #0 [OK]
      - Collected 32 metrics, 0 events & 0 service checks

    docker_daemon (5.17.2)
    ----------------------
      - instance #0 [OK]
      - Collected 60 metrics, 0 events & 1 service check

  Emitters
  ========

    - http_emitter [OK]

2017-11-30 19:46:33,316 | WARNING | dd.dogstatsd | utils.service_discovery.config(config.py:31) | No configuration backend provided for service discovery. Only auto config templates will be used.
====================
Dogstatsd (v 5.17.2)
====================

  Status date: 2017-11-30 19:46:31 (2s ago)
  Pid: 21
  Platform: Linux-4.4.0-1030-aws-x86_64-with-debian-8.9
  Python Version: 2.7.13, 64bit
  Logs: <stderr>, /var/log/datadog/dogstatsd.log

  Flush count: 354
  Packet Count: 4605
  Packets per second: 2.0
  Metric count: 12
  Event count: 0
  Service check count: 0

====================
Forwarder (v 5.17.2)
====================

  Status date: 2017-11-30 19:46:31 (2s ago)
  Pid: 20
  Platform: Linux-4.4.0-1030-aws-x86_64-with-debian-8.9
  Python Version: 2.7.13, 64bit
  Logs: <stderr>, /var/log/datadog/forwarder.log

  Queue Size: 417 bytes
  Queue Length: 1
  Flush Count: 1194
  Transactions received: 896
  Transactions flushed: 895
  Transactions rejected: 0
  API Key Status: API Key is valid

======================
Trace Agent (v 5.17.2)
======================

  Pid: 19
  Uptime: 3548 seconds
  Mem alloc: 753584 bytes

  Hostname: web-ops-art-hut-rap
  Receiver: <IP_ADDRESS>:8126
  API Endpoint: https://trace.agent.datadoghq.com

  Bytes sent (1 min): 0
  Traces sent (1 min): 0
  Stats sent (1 min): 0

Additional environment details (Operating System, Cloud provider, etc):
$ cat /etc/*-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=16.04
DISTRIB_CODENAME=xenial
DISTRIB_DESCRIPTION="Ubuntu 16.04.3 LTS"
NAME="Ubuntu"
VERSION="16.04.3 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.3 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

Steps to reproduce the issue:

run the docker container with -e TAGS=env:staging.

According to the docs:

TAGS set host tags. Add -e TAGS=simple-tag-0,tag-key-1:tag-value-1 to use [simple-tag-0, tag-key-1:tag-value-1] as host tags.

https://github.com/DataDog/docker-dd-agent#environment-variables

Check UI, and see that no additional tags have been set (definitely not env:staging).

It does show up in the container env vars though.
root@1acd0644f1e4:/# env | grep TAGS
TAGS=env:ops

Add the tag in the datadog.conf instead. Everything works as expected.

Describe the results you received:
Do not see env:staging in the host tags.
Describe the results you expected:
Should see env:staging in the host tags.
Additional information you deem important (e.g. issue happens only occasionally):
https://github.com/DataDog/docker-dd-agent/blob/12.3.5172/config_builder.py#L61-L63

Hi @agamdua, thanks for reaching out.
Could you try with a more recent version of the agent? I can't reproduce with 5.20.1.
I'd also be interested in seeing the command you used to run the agent. Do you have a custom entrypoint? If you pass the env var, the default entrypoint should update datadog.conf.

Howdy,  I have a very similar issue with agent v 5.26.0.
The interesting thing is that some metrics do include my tags, and some don't.
Any tips?


ds: Clean up joystick-related stuff

Not sure what the best approach for that type cast linter warning is.

Is there any official guidance on whether to use Result or Option for these simple erroring cases? The best I could find was a reddit post quoting docs (that seem to no longer exist) arguing Result is preferrable if the scenario is an "error." I do see the case for less to type with Option, and also one could argue there isn't an "error" but a "search" for a button that finds nothing. I guess I'm OK with using Option
With regards to the typecast, just using i32::from seems fine to me. I doubt types will change though, so it doesn't matter really. Also, JoystickError is now dead code and should be removed.

The previous contract said that these could return any JoystickError, which is objectively worse for ergonomics when it is simply either valid or invalid. You can use ? on an Option anyway.

With regards to the typecast, just using i32::from seems fine to me.

i32::from isn't a const fn.

That said, maybe it might be better to have the stick getters return a Result with a singleton error.

I've come up with a better idea for the stick getters. We'll have new methods that return newtypes around the HAL joystick structs, and a get method on those that return an Option (with the semantics being "the thing you asked for is missing").

Throwing in the common axes as associated constants so teams don't have to construct them themselves.
Also, people like gamepads, so throwing in XInput constants as well.

Ok I'm done here. Here's a way you could wrap around this functionality for convenience:
struct Joystick<'a> {
  ds: &'a DriverStation,
  port: JoystickPort,
  prev_buttons: JoystickButtons,
  curr_buttons: JoystickButtons,
  buttons_pressed: u32,
}
impl Joystick<'_> {
  fn read_buttons(&mut self) {
    let buttons = self.ds.stick_buttons(self.port);
    self.prev_buttons = self.curr_buttons;
    self.curr_buttons = buttons;
    self.buttons_pressed |= buttons.pressed_since(self.prev_buttons).bits();
  }
  fn buttons(&self) -> JoystickButtons {
    self.curr_buttons
  }
  fn axes(&self) -> JoystickAxes {
    self.ds.stick_axes(self.port)
  }
  fn pov(&self) -> Option<i16> {
    self.ds.stick_povs(self.port).first()
  }
}

To correctly track button presses, you'd have to call read_buttons whenever a DS packet arrives (so if a team is using start_timed they'd have to spawn another thread to wait for DS data).

I like these changes. I'd also like to remove the lifetime from DriverStation at some point, but that's another PR.


Convert line endings from CRLF to LF.

This commit introduces no code change!
Please do not commit files with Windows line endings (CRLF) since this
causes a permanent full-file-diff on Unix machines. The Git config
can be setup to automatically convert line endings back and forth on a
Windows machine.

Formatting and whitespace issues are some of the more frustrating and subtle problems that many developers encounter when collaborating, especially cross-platform. It’s very easy for patches or other collaborated work to introduce subtle whitespace changes because editors silently introduce them, and if your files ever touch a Windows system, their line endings might be replaced. Git has a few configuration options to help with these issues.

core.autocrlf

If you’re programming on Windows and working with people who are not (or vice-versa), you’ll probably run into line-ending issues at some point. This is because Windows uses both a carriage-return character and a linefeed character for newlines in its files, whereas Mac and Linux systems use only the linefeed character. This is a subtle but incredibly annoying fact of cross-platform work; many editors on Windows silently replace existing LF-style line endings with CRLF, or insert both line-ending characters when the user hits the enter key.

Git can handle this by auto-converting CRLF line endings into LF when you add a file to the index, and vice versa when it checks out code onto your filesystem ...

Source: Git documentation - Configuration - Formatting-and-Whitespace

@johnjohndoe the files that have Windows line endings are .bat files, they're the Windows version of some scripts and are required to be understood by Windows tooling when we release them.
I think the experience will be broken for some users if we change the line endings here. But do you think differently?

As stated above the repository for multi-platform project should not mix line-endings. Git can handle this for you:

Git can handle this by auto-converting CRLF line endings into LF when you add a file to the index, and vice versa when it checks out code onto your filesystem ...

@johnjohndoe This is a local configuration option though, right?
We need to release files with Windows line endings. This isn't about mixing line endings of different styles in a single code base, or having people edit in their own preferred line-ending style, it's about creating and releasing correctly formatted bat files.

Yes, the configuration is on behalf of the client.
The release process could take care of the conversion as Git does with clone, checkout, ...

I'm not going to merge this as I prefer to have these scripts runnable on permanently runnable Windows and not modified by the release process.

I like to add that you should be aware that having files showing up as changed all the time makes it harder for less experienced developers to contribute. Each time one clones the repository, checks out a branch or pull changes they will have to deal with the full file diff of these files getting in the way.


Use Runge-Kutta 4

Using Runge-Kutta 4 instead of Euler integration might help increasing the accuracy of the simulation while improving performance.
See Gaffer on Games for a good presentation of the principle.

Looking into it.

Have fun! It might actually require to reorganize quite a lot of code to do it properly, so take your time to grok it ;-) .


Can't run "npm start"

When I run "npm start", I see a lot of build messages, and then it keeps hanging on "webpack: bundle is now VALID.". The build directory is not created and the server not started. What could be the issue here?

Same here with error : ERROR in multi main
Module not found: Error: Cannot resolve module 'server' in /Users/IOSoftwareSAS/WebstormProjects/angular2-authentication-sample @ multi main

Just updated to rc3. Can you try it again?

I still have same issue. Please check out the log output:
14 verbose stack Error<EMAIL_ADDRESS>start: `npm run front`
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter.<anonymous> (C:\Users\kbw780\AppData\Roaming\npm\node_modules\npm\lib\utils\lifecycle.js:239:16)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at EventEmitter.emit (events.js:191:7)
14 verbose stack     at ChildProcess.<anonymous> (C:\Users\kbw780\AppData\Roaming\npm\node_modules\npm\lib\utils\spawn.js:24:14)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at ChildProcess.emit (events.js:191:7)
14 verbose stack     at maybeClose (internal/child_process.js:852:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:5)
15 verbose pkgid<EMAIL_ADDRESS>16 verbose cwd C:\MyStuff\01Projects\github\public\angular2-authentication-sample
17 error Windows_NT 6.1.7601
18 error argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\kbw780\\AppData\\Roaming\\npm\\node_modules\\npm\\bin\\npm-cli.js" "start"
19 error node v6.2.2
20 error npm  v3.8.5
21 error code ELIFECYCLE
22 error<EMAIL_ADDRESS>start: `npm run front`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>start script 'npm run front'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the angular2-authentication-sample package,
23 error not with npm itself.

same error ,webpack: bundle is now VALID

I get the exact same thing happen when I run npm start.  Is there any fix for this?