Upgrade Beam to Cython 3.0.0

Resolves a cython issue with a closure and brings the Beam codebase up to Cython 3.0.0.
Fixes #27526

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

(this is still a WIP with some strange type hinting issues popping up in the integration test, feel free to ignore)

waiting on author


[Feature] Mouse On Wheel Loader

Description
I want to add a " Mouse On Wheel " Loader in the Loaders Section
Screenshots
https://github.com/Rakesh9100/Beautiify/assets/140608790/18a61a66-8e44-4c71-a097-692b12df8460
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Go ahead @JiyaGupta-cs

@Rakesh9100 Please review the PR

@Rakesh9100 Please reopen it

PR is already open, fix that @JiyaGupta-cs


Task - Admin - Job Completion

Description
During conversations regarding the HR portal, it was mentioned that admin users will need to set the status of a job action from active to complete (whether it is successful or not) which will, in turn, affect the way that job displays to HR advisers down the road.

This should stop the clock on the "review applicants" page.
This should also move the job to the "My archived posters" on https://talent.canada.ca/en/manager/jobs

We'll need to store the time of archiving.

This can be accessible only via Admin Portal right now.

Rolled into #2189.


Fix rails deprecation warnings (use after_commit -> { })

Hello
That's part of the rails 5 deprecation warnings cleanup.
It fixes DEPRECATION WARNING: Passing string to define callback.
Change is almost mechanical and I hope new lambda syntax is quite old already (ruby 1.9).
I've also noticed specs failure with rspec 3, changed gemspec to use rspec 2 instead.

@lostboy any problem with this pull request?

@sergeykish This repo seems really inactive. Any thoughts on mantaining your own version with Rails 5 deprecation cleanup, and 5.1 support?

Hey guys, I've left this for so long, apologies for going awol. If people are still keen I can put some time to bringing the repo and gem up to speed.

@lostboy glad you're back. I've checked this in depth couple of months ago, and I've found one fork that has been mantained and updated to work with Rails 5: https://github.com/davidakachaos/workless_revived & It's working ok!
I think you should talk to @davidakachaos to join efforts! Thanks for coming back.


Thanks for your autoformer code,but when I try to use instruction "make init ." ,it just stuck here for quite a long time,is this normal?

Downloading and Extracting Packages
dbus-1.13.6          | 604 KB    | ########## | 100%
I type this by myself :omitting the unnecessary information for installing dependecy packages
typing_extensions-4. | 27 KB     | ########## | 100%
Preparing transaction: ...working... done
Verifying transaction: ...working... done
Executing transaction: ...working... done
Installing pip dependencies: ...working...

sorry,I supposed to have more patience,it's complete```


Fix current gas costs for CALLDATACOPY and CODECOPY

you're right, good catch, I used some source with an error. However, this requires regeneration of the plots in the paper, as well as an update of conclusions. I'll look into this soon.

Already fixed in #141


Support wrapped token and native token in one implementation

All functions of mint / borrow / redeem / redeemUnderlying / repay / repayBehalf / liquidateBorrow / _addReserves have a mirror functions in native token. Don't change the existing public function signatures since crWBNB (wrapped native delegator) on BSC has already been deployed.
Keep native token as cash (liquidity).
Can only flashloan and reducing reserves in the native token.

Note: The only cream wrapped native contract aiming to integrate the native token is crWBNB (on BSC). It will need to consider backward compatibility. For example, the original contract has internalCash but this storage is no longer needed. We will handle this by submitting another PR only to bsc branch after this one.

Still need to include internalCash in storage but clear its value when _becomeImplementation.
Unwrap the existing cash (liquidity) to the native token.
Check if gulp function being undefined is ok or not.

Note 2: The function signature of flashLoan is not consistent between CWrappedNative and the others (one receiver is payable and others are not). However, we will refactor the flash loan implementation to follow EIP-3156 after this PR. Will also align the function signature at that moment.


Ideas or Improvements (Design or Programming) in our Wordpress Theme

You can find our Theme in this file and you can see how it looks on our website. Now we have been only working on the prototype of the English version of our project.

To give your ideas and propose improvements you can post a comment here.
If you want to help us to do the improvement please make an Issue  here and apply the following labels

You can find our Theme in this file

I think maybe there is a link missing? I can't find the theme...


Home - Individual Listing Component

Implement individual listing component using Vivid! In the browser, command-click on any component to edit its code and hit Command-K to quickly preview styles. Command-Shift-K will let you add new empty components!

I'll work on this. Can you assign it to me?

Yes! Assigning it to you now


Don't block on shutdown

Addresses issue mentioned in  #373

We're having issues with hangs on Travis and the CI after running functional tests on Mono. I've identified the reason for this, which is that KestrelThread.ThreadStart() hangs on the second call it makes to _loop.Run()

Doesn't like dispose if stuff still outstanding.


IndexError: index 4 is out of bounds for axis 0 with size 4

I am trying to implement the declustering function on my dataframe which consists of longitude, latitude, and sediment accumulation variables, but I am getting an out of index error. I am having trouble fixing this error myself, could someone help me figure this out? Below is the traceback:
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\etachen\AppData\Local\JetBrains\PyCharm 2021.2.2\plugins\python\helpers\pydev_pydev_bundle\pydev_umd.py", line 198, in runfile
pydev_imports.execfile(filename, global_vars, local_vars)  # execute the script
File "C:\Users\etachen\AppData\Local\JetBrains\PyCharm 2021.2.2\plugins\python\helpers\pydev_pydev_imps_pydev_execfile.py", line 18, in execfile
exec(compile(contents+"\n", file, 'exec'), glob, loc)
File "C:/Users/etachen/PycharmProjects/ai_algo_tests/declustering_medium_test.py", line 21, in
W, Csize, Dmean = geostat.declus(df,'Longitude', 'Latitude', 'Average Accretion (mm)',
File "C:\Users\etachen\Anaconda3\envs\ai_algo_tests\lib\site-packages\geostatspy\geostats.py", line 1607, in declus
cellwt[icell] = cellwt[icell] + 1.0
IndexError: index 4 is out of bounds for axis 0 with size 4

I am also getting this error, but changing noff doesn't fix the issue for me. I'm using a geodataframe with dimension (515,20) and the following DECLUS inputs:
weights, cell_sizes, dmeans = geostats.declus(ddf,'X','Y','dZ',
iminmax = 1,
noff= 5, ncell=1000,
cmin=1, cmax=50)
The problematic dataframe can be downloaded here (link to download gpkg from box).
I've tried resetting the dataframe index, using a different subset of rows/columns, converting to a standard Dataframe, varying the values for iminmax/noff/ncell/cmin/cmax but I haven't been able to isolate the cause.
Here is the specific error message I am getting:
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
Cell In [30], line 9
      6 # number of cell sizes to iterate through, if taking too long, decrease this
      7 ncell = 1000
----> 9 weights, cell_sizes, dmeans = geostats.declus(ddf.loc[:, ['X', 'Y', 'dZ']],'X','Y','dZ',
     10                                           iminmax = 1,
     11                                           noff= 10, ncell=ncell,
     12                                           cmin=min_cell_size, cmax=max_cell_size)
     14 ddf['W'] = weights
     15 ddf.head()

File /srv/conda/envs/notebook/lib/python3.10/site-packages/geostatspy/geostats.py:1607, in declus(df, xcol, ycol, vcol, iminmax, noff, ncell, cmin, cmax)
   1605     icell = icellx + (icelly - 1) * ncellx
   1606     index[i] = icell
-> 1607     cellwt[icell] = cellwt[icell] + 1.0
   1609 # The weight assigned to each datum is inversely proportional to the
   1610 # number of data in the cell. We first need to get the sum of
   1611 # weights so that we can normalize the weights to sum to one
   1612 sumw = 0.0

IndexError: index 3109994 is out of bounds for axis 0 with size 3109994```


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/491 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target.git

Docs Build status updates of commit 61b229d:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-BackendEngineExceptionError] An exception occurred during the build process. Please check the raw log and retry building your content again. If the issue persists, please open a ticket in https://SiteHelp and include the URL of publishing report as well as below information: One or more errors occurred..

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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Directs logging to outputter

To allow easier formatting and a unified channel for all human facing language.
Based on our conversations on #546 and #537 I revisited logging and output a bit. This version directs console-logging through our outputter. That lets us use the same formatters as the method outputs (saving duplication) and gives us a single point for translation.
I added a Tee logger which sends log output to multiple loggers. That lets us do file logging as well as sending info to the console. Each logger can have separate log level thresholds so --silent operation can still log to a file.
With outputters handling formatting, there didn't seem to be as much use for the sub-class of KLogger so I'm using it directly. We could still subclass it to meet our needs but our overrides should be much simpler.
The next step would be to add a formatHelp/outputHelp pair to the outputter and to pass a translation object to the PrettyFormatter so it can translate text before sending to the user.
This code gives us more functionality and more options with less code. It also loosens the coupling quite a bit (no more looking at the $config array inside the logger). It also gives us a really nice single point for all human-language. That will be great for i18n.
I think it may be a good step forward. Thoughts?

I tried out the changes and I like them a lot better than what we had before. The only issue I'm seeing is that the interpolator isn't switching out the {var} elements.

Unless it's not supposed to. I'm not quite sure.

True.
I'm on the fence as to whether that's a bug or a feature. What is the appropriate machine representation of a log message? It may be that the uninterpolated (and possibly untranslated) is the appropriate output for json.
I can imagine sending this to another system for persistence and display later, in which case you would not want the translation and interpolation done for you. That's how the drupal watchdog table works, for example. Logs are stored untranslated so that they can a be translated when viewed.

If the translation is not already done, will we need to create some sort of tool which only displays these logs, using i18n to change them, or is the point to have something like "auth.login.failure" as the message in the log?

is the point to have something like "auth.login.failure" as the message in the log?

It seems weird, but... maybe? I really don't know what sort of 3rd party tool will be consuming Terminus logs in machine format, but it's certainly possible that some sort of consistent error or message key would be more useful than a human-readable string. In we'd probably want to send both. If that's the case, then all of our formatters will need a translator which makes this a slightly less clean implementation.

This PR is 4 months behind master.


Toggle feature

Hi @levouh, thanks for the plugin!
Is there a way to toggle the feature?

Not at the moment, but I'd be happy to add one here in a bit.

Merged the above PR, feel free to re-open this issue or create another one if it does not work as expected.

Hi @levouh thank you! That will help a lot!!
if I'm not asking for too much, it will be nice to have a property exposed to know if it is enabled ou disabled, or a toggle function that encapsulate that state.

No worries. I'll track this with #18, follow it there.


Podboat to autodownload and autoclose when finished

Newsboat version (copy the output of newsboat -v or the first line of git show):
newsboat 2.18.0
System: Linux 4.15.0-88-generic (x86_64)
Compiler: g++ 7.4.0
ncurses: ncurses 6.1.20180127 (compiled with 6.1)
libcurl: libcurl/7.58.0 GnuTLS/3.5.18 zlib/1.2.11 libidn2/2.2.0 libpsl/0.19.1 (+libidn2/2.0.4) nghttp2/1.30.0 librtmp/2.3 (compiled with 7.58.0)
SQLite: 3.22.0 (compiled with 3.22.0)
libxml2: compiled with 2.9.9

This is a question, not an issue.
I am using podboat after having enqueued some files from within newsboat.
After selecting the files (lets say 10 interesting podcasts), I want to download them. My workflow is usually :
podboat
Press <a> to autodownload
Wait for the download
Press <q> to quit

Eg, I don't really use the UI of podboat itself, except to say to autodownload and quit.
I would like to have some script that does it for me (so that I can put it in a cron for example).
Maybe something like this ?
podboat --autodownload --autoclose

In that case, it would not even be useful to have a UI popup, just having the program wait (or maybe print to stdout the currently downloaded song ?)
What do you think of that ?

Hi! I think it's a fine idea. It might make more sense to mimic Newsboat, and add something like this:
podboat --execute download

Pros:

similarity with Newsboat makes it easier to explain, understand, and remember
easier to evolve, e.g. by letting people specify multiple operations delimited with e.g. semicolons (podboat -x 'download; do-something-fun'). That's a long shot though :)

Cons:

--autodownload and -x download look confusingly similar

Similar option in Newsboat already exists and automatically quits the program once operation finished. People run newsboat -x reload from their cron jobs all the time.

I like the option of --execute indeed, it makes sense to be able to do multiple operations.
Maybe you could deprecate or remove --autodownload since it can be done with execute.


Add setting to specify external modules by package name pattern

I'm proposing a new setting to specify external modules by a package name pattern to satisfy the following use case:
We have a monorepo consisting of multiple packages that depend on each other. We'd like to enforce that every package that is imported by another package in the same repository is listed as a dependency in package.json. We want to use the import/no-extraneous-dependencies rule to enforce this, however, we can't specify that each package in the monorepo is an external dependency because:

We resolve package dependencies using a custom resolver that resolves packages to their source location
Eslint detects these dependencies as internal modules because their resolved path does not fall under node_modules

In the past, our packages were scoped which resulted in them being treated as external modules because of #1293. However, since that has been fixed we now run into this problem.
The existing import/external-module-folders setting isn't sufficient for us because:

Our packages don't reside as a flat list under a single directory such as node_modules
The package dir name does not match the package name in all cases, especially not with scoped prefixes

Instead, a new import/external-module-patterns setting that allowed one to specify an array of string/regex patterns to match against package names, could be used to match all of our packages as external since they all share a common scope.

A second option could be to add some further metadata in the returned value from custom resolvers to specify whether a module is external or not.

Alternatively, it would be worth revisiting the decision made in #943 to only report errors for dependencies that can be resolved and provide a config option to opt-in to reporting errors for unresolved dependencies. As then, we would no longer need to mark our modules as external and would just remove our custom resolver altogether so that they are always unresolved and therefore marked as external.

Appreciate this. Assuming, I understand the intention, import/external-module-patterns: [/^@reach/] (or for my non-@-prefixing org [/^@/]) would allow me to group @reach/router with other external imports?
I'd suggest adding a field for internal module patterns as well. I'm using tilde-led aliases for paths, and it'd be nice not to treat them as unknowns.

I'm also want this feature.
Some of my dependencies resolved as symlinks with different paths between every launch.
So, I just can't wtite correct import/external-module-folders, because it looks at path, not at package name.

Made a PR ^. Tests included. Please look at it.

This seems to be related to #1597 and should be al least partially fixed by #1605 since now you can use import/external-module-folders and it will work regardless of whether dir name matches package name or not.


Config: optional flag that enables Strict Mode

Strict mode assumes exit(1) as soon as at least one error or unexpected condition is hit.
Strict mode can be useful: 1) for debugging 2) for integration tests to make sure that there is nothing unexpected.
@AlexDenisov suggests that the following behavior can be hidden inside diagnostics to reduce the boilerplate:
if (error) {
  llvm::errs()...
  if (strictMode) {
    exit(1)
  }
}

I have been waiting for this feature for 3 years, and I feel like now it is time 🗡


Create a new Page | License

The following changes are needed to be done on the footer section of the website
Todo

Create a new HTML page with a matching color scheme with the content from here

Important Points

All changes should be responsive
Before creating a PR, send a screenshot in this issue thread

Working on it

Screenshots for reference:

Looks great 👍
For further contributions, please wait until you are assigned before you start working on any issue.


Kafka documentation update

Description of the change

Refined the Kafka doc and added information on the serializing data using Avro.

Type of documentation

[ ] New documentation
[x] Documentation update
[ ] Hotfix

Notion ticket link

Kafka Avro support

thanks @ItsSudip I will update the connection settings image and add this information.

@ItsSudip I've incorporated all the suggestions - let me know if the changes look good.


CDK-944: Added a broken test to demonstrate this bug

I posted a fix to the bug exposed by the test case.

+1
Thanks, Joey!


Added jmx exporter and init-fs to deployment type

fixes #195

Closing this pr as it does not seem to be fitting into the new pr policy that was introduced with #199


Is it possible to make Role::class extensible?

It would be useful to make Role class extensible. By adding it to config file in the same way as it was done for User model.
'userModel' => env('USER_MODEL', 'App\User'),
'roleModel' => env('ROLE_MODEL', 'Silvanite\Brandenburg\Role'),

At the moment it's impossible to extend your class functionality.

Hi @wolkodlack. Thanks for raising this. Sure that could be implemented. Happy for someone to PR this but will also add it to my list.


json to pogo misses some json element in POJO

Hi,
I was using json2pojo to convert json to pojo; but saw a anomaly while using it at one case.
so if i have json like :
(below files extn changes from json to TXT to upload)
DQValidationCheck.txt
I am able to create three pojo files which are absolutely correct as per the json.
(below files extn changes from java to TXT to upload)
Column.txt
DQValidationCheck.txt
Validation.txt
however if i use the below json:
(below files extn changes from json to TXT to upload)
DQValidationCheck.txt
i get
(below files extn changes from java to TXT to upload)
DQValidationCheck.txt
Validation.txt
Column.txt
Check the Validation java class where the Default member is missing the second time.
Basically i did some trials and found that only the last json list item is looked upon to make POJO.
I am not sure if it supposed to be, kindly suggest.
Many Thanks,
Varun

I am only using the below config in pom.

gson
json
${basedir}/src/main/resources
${basedir}/src/main/java

Indeed, this is a limitation of the JSON handling. Since we don't support either union types or tuples, only the first element in an array is read.

ahh thanks.


Added support of non-ASCII characters in IRIs

Solution for issue # 14 (https://github.com/sdl/dxa-web-application-dotnet/issues/14)

Incorporated in DXA 1.3


Release v5.0.4

Questions
Answers

Description?
Release v5.0.4

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?

How to test?

@matks yep, right there: https://github.com/PrestaShop/ps_linklist/pull/132 :)


Treat warnings as errors and silence them in third party code

We've been trying to figure out how to set this a while and keep running
into warnings in third party code. I think this should take care of it.
We can add additional exclusions as necessary.

Requires https://github.com/google/iree/pull/2469


Inserting attempt when no connection info has been inserted

I had an bug that I can not reproduce. When running the script it skipped over a connection message and tried to process an attempt associated with the connection. When it queried the database for a connection ID to bond it with, it got nothing and crashed. I ran the script again and it was processed fine. But that was will the messages be all pulled in the first 100, not pulled individually  cause it was processing in real time. A fix to the issue would be, catch the error. Have it pull from elastic search again, and try to processes the message again, hopefully with the additional message.

Elastic search does not support micro/nanoseconds when sorting. When I sort on @timestamp its already rounded, so connection messages or disconnect messages and attempt messages might be out of order. I tried to sort on Local_Timestamp which does have microseconds, but even on search elastic search truncates it to milliseconds, thus leading to the same issue.
What I ended up doing is making a sort function in Nectar. It slows down the program slightly but it helps that the list is semi sorted. That fixed a majority of the issues but there is an issue when the out of session is on the board of two pulls. In order to fix this I added a second function for finding connection ids. Hopefully once elastic search support micro/nanoseconds, these two functions will not be needed.
Additionally, this fix is only for if the attempt is after a disconnect message within a given session. Supposedly, an error should happen if the attempt is before the connection message too. But I have not ran into this case yet.
Closing for now, up will reopen later.


How to construct ExoPlayer as TextureView rather than SurfaceView in C# .NET?

Goal
I am trying to create an ExoPlayer using Android code within .NET (MAUI) that renders to a TextureView (rather than the default SurfaceView).
Background
This is discussed in terms of Android in this thread.
In .NET/MAUI we have the following constructor which would apply for the ExoPlayer:
public unsafe StyledPlayerView (global::Android.Content.Context? context, global::Android.Util.IAttributeSet? attrs)
So you could run this as:
var PlayerView = new StyledPlayerView(androidContext, attributes);
//presuming you have a working 'attributes' file...

The second 'attributes' argument global::Android.Util.IAttributeSet? attrs is where we are supposed to specify we want a TextureView.
From the link above, we must pass in an XML file which is first converted into the IAttributeSet object to specify the TextureView as a target surface.
An example XML given is:
<com.google.android.exoplayer2.ui.PlayerView
     android:id="@+id/player_view"
     app:surface_type="texture_view"
     android:layout_width="match_parent"
     android:layout_height="match_parent"/>

The XML file of this nature must be converted into the attribute needed for the constructor by:
val xmlAttributes = context.resources.getXml(R.xml.player_view).let {
    try {
        it.next()
        it.nextTag()
    } catch (e: Exception) {
        // do something, log, whatever
    }
    Xml.asAttributeSet(it)
}

Question
If I want to work completely in a C# script, and I want to have no XML files to load from anywhere, but rather want to just declare this minimal short XML snippet as a string or written in my C# code directly, can I do this?
Can I then pass that into the code for creating the attribute? Or alternatively, can we make the IAttributeSet for the constructor directly in C# without even bothering with creating the XML first?
My Best Attempt
My best attempt shown provides an error which I don't understand.
In Visual Studio 2022, if one goes:

File > New > Android Application
Add the ExoPlayer NuGet Xam.Plugins.Android.ExoPlayer
Then in MainActivity.cs, one can try the following code:

protected override void OnCreate(Bundle? savedInstanceState) {
    base.OnCreate(savedInstanceState);

    string xmlString =
        "<?xml version=\"1.0\" encoding=\"utf-8\"?>\n" +
        "<Com.Google.Android.Exoplayer2.UI.PlayerView " +
        "android:id=\"@+id/player_view\" " +
        "app:surface_type=\"texture_view\" " + //KEY NEEDED LINE FOR GOAL
        "android:layout_width=\"match_parent\" " +
        "android:layout_height=\"match_parent\"/>";

    XmlReader xmlReader = XmlReader.Create(new StringReader(xmlString));
    xmlReader.Read();
    System.Diagnostics.Debug.WriteLine("XML READER " + xmlReader.AttributeCount);

    Android.Util.IAttributeSet attributes = Android.Util.Xml.AsAttributeSet(xmlReader);
    System.Diagnostics.Debug.WriteLine("ATTRIBUTES " + attributes.AttributeCount);

    Com.Google.Android.Exoplayer2.UI.StyledPlayerView styledPlayerView = new(this, attributes);

Issues
No errors are given in Visual Studio on coding. However two problems are evident on running it:

If you attempt to debug out the xmlReader with the following code, it says **System.Xml.XmlException:** ''android' is an undeclared prefix. Line 2, position 52.':

while (xmlReader.Read()) {
    switch (xmlReader.NodeType) {
        case XmlNodeType.Element:
            System.Diagnostics.Debug.WriteLine("<{0}>", xmlReader.Name);
            break;
        case XmlNodeType.Text:
            System.Diagnostics.Debug.WriteLine(xmlReader.Value);
            break;
        case XmlNodeType.CDATA:
            System.Diagnostics.Debug.WriteLine("<![CDATA[{0}]]>", xmlReader.Value);
            break;
        case XmlNodeType.ProcessingInstruction:
            System.Diagnostics.Debug.WriteLine("<?{0} {1}?>", xmlReader.Name, xmlReader.Value);
            break;
        case XmlNodeType.Comment:
            System.Diagnostics.Debug.WriteLine("<!--{0}-->", xmlReader.Value);
            break;
        case XmlNodeType.XmlDeclaration:
            System.Diagnostics.Debug.WriteLine("<?xml version='1.0'?>");
            break;
        case XmlNodeType.Document:
            break;
        case XmlNodeType.DocumentType:
            System.Diagnostics.Debug.WriteLine("<!DOCTYPE {0} [{1}]", xmlReader.Name, xmlReader.Value);
            break;
        case XmlNodeType.EntityReference:
            System.Diagnostics.Debug.WriteLine(xmlReader.Name);
            break;
        case XmlNodeType.EndElement:
            System.Diagnostics.Debug.WriteLine("</{0}>", xmlReader.Name);
            break;
    }
}

Debug taken from: https://learn.microsoft.com/en-us/dotnet/api/system.xml.xmlreader.read?view=net-8.0

The ExoPlayer constructor line returns the error **Java.Lang.ClassCastException:** 'android.util.XmlPullAttributes cannot be cast to android.content.res.XmlBlock$Parser'

Why is this code is failing on these two points? The only other place I see the error from (2) is here and I am not sure of how this works.
In Short ...
Can anyone provide a working sample of C# code that can make the attribute needed for the StyledPlayerView constructor to allow a TextureView Exoplayer to be created in .NET?
Thanks for any help.

Posted a reply on #1375 -- this may be a duplicate.

Duplicate of #1375


boot: zephyr: migrate to new log subystem

The old log subsystem has been deprecated in Zephyr.
Migrate to the new subsystem to avoid compilation warnings.
@nvlsianpu I might have missed something, please have a look.

I am not aware of those Zephyr changes, but does it really need to be registered in every single file? Wouldn't registering in main.c be enough?

@lemrey Did you test this PR? - I can't get any logg using it.

@nvlsianpu I tested that it compiled without warnings, I did not try to run it.

@nvlsianpu what issue did you see? I might have some time to look at this now.

^^ @lemrey updates are welcome

If the module consists of multiple files, then LOG_MODULE_REGISTER() should
appear in exactly one of them.

I'm pretty sure we should be using LOG_MODULE_REGISTER() in exactly one place, and LOG_MODULE_DECLARE() in the other files.

In-place log processing is selected as it is required as MCUBoot is
one thread application. Now it works properly.

The new version still should be calling LOG_MODULE_REGISTER in only one place, and LOG_MODULE_DECLARE in the other files.

@d3zd3z -  Your remarks were taken into account. This implies registration one log-module for entire mcuboot (versus multiple log-modules previously).


CI did not run

On PR #511 github actions did not run. This might be worth investigating.

Haven't seen this since. No idea why it didn't run that time. I'm inclined to ignore it until it raises it's ugly head again.


Transcaucasian

I would love to see this as a country
https://en.wikipedia.org/wiki/Transcaucasian_Socialist_Federative_Soviet_Republic

I would consider this quite unrealistic.Any movement for a united Transcaucasia had died out by 1936 when the Soviet Union decided to split up the Transcaucasian SFSR into the Georgia,Armenia and Azerbaijan SSRs. The first attempt at establishing a united Transcaucasian state was largely a failure with it only lasting from 22 April to 28 May 1918,it only existed for about a month.It collapsed due to infighting within the ethnic groups.The three successor states of the Transcaucasian Federation, Georgia, Armenia and Azerbaijan all waged war with each other bickering over territorial borders while completely ignoring the biggest threat to these three countries the Russian SFSR, Russia managed to swiftly conquer it's former territories in the Caucasus from 1918 to 1921 all while in a civil war.From there it set up the Transcaucasian SFSR as a puppet state. Giving how quickly this union dissolved it's highly unlikely that those three highly nationalistic ethnic groups ever want to reunify into that union. If they did it would be a lot like Yugoslavia but maybe a much more quicker demise considering how just after independence Armenia went into straight out war with Azerbaijan over the Armenian inhabited Nagorno-Karabakh enclave. Even after the Nagorno-Karabakh war there has still been conflict between those two countries the most recent being in 2016. Not to mention that the idea Transcaucasia as a single country was created by the Russian Empire without any regard for ethnic differences, they just suddenly clumped all these ethnic groups together a bit like what the Colonial Powers did in Africa and the Middle East. I also do believe that Transcaucasia also shares a similar status with Khiva and Bukhara as they were territories that were dissolved permanently by the Soviet Union decades before the startdate of this mod therefore it is a highly antiquated historical region that was dismantled decades ago.
https://en.wikipedia.org/wiki/Armenian–Azerbaijani_War
https://en.wikipedia.org/wiki/Georgian–Armenian_War#Bibliography
https://en.wikipedia.org/wiki/Nagorno-Karabakh_War
https://en.wikipedia.org/wiki/Georgian_Civil_War
https://en.wikipedia.org/wiki/Red_Army_invasion_of_Georgia
https://en.wikipedia.org/wiki/Red_Army_invasion_of_Azerbaijan

Agree, on it being unrealistic and a violation of NWO's "Prime Directive".


Sidebar active category not collapsing onClick (toggle)

Hi, I don't know if this was made on purpose, but collapsing a category in the sidebar (new beta docs) by clicking on the active category doesn't do work. I have to click on an other category for the active category to collapse.

Yeah, that's intentional. We know it's a bit confusing and might revisit in the future, but this is more like a navigation link than a toggle.


Can you add README file for this project?

Hi @rherwig,
I really liked your implementation of react-16's  server-side rendering. I like to understand this example better, so would need some assistance in understanding the code. As I am also planning to create react app with will have ssr feature, which will have backend express app which will call some third party api and render the response to react components.
Thanks!!

Sure, I'm working on that.
Any particular information you wish for?

Are you going to make a build environment too?

I will. On it atm.

I added build scripts and a basic readme. For further documentation please refer to the documentation inside the code or create an issue with your specific question. :-)


Consider a less aggressive scraping strategy on Fridays

Recently, our aggressive event scrapes failed with 104 status codes for a period of several hours. Metro uploaded agendas during this team, but they did not show up on the site due to failures in the scraper (indeed, the problem the aggressive scrapes were meant to prevent!) Consider a less aggressive schedule. @reginafcompton suggests two scrapes, one complete and one with a small window, per hour.

For reference: the crons themselves.
@shrayshray, to decode the crons, we scrape all events at 0, 15, 30, and 45 minutes after the hour. We implemented this solution to handle issues arising with Legistar timestamps that did not change as expected, e.g., https://github.com/datamade/la-metro-councilmatic/issues/310 and https://github.com/datamade/la-metro-councilmatic/issues/267
We also aggressively scrape bills on Friday (again, to address timestamp issues, e.g., https://github.com/datamade/la-metro-councilmatic/issues/328)
I think we should consider a multi-faceted revision of our crons:

Narrow the window for aggressive scraping. Currently, we scrape all bills and events, from 4:00 pm CST until 11:50 pm CST.
Minimize the number of times we hit Legistar by scraping all bills and events, once an hour, and include small windowed scrapes one or two times per hour.

@shrayshray - can you let us know how this sounds? and how we should prioritize making changes?

@reginafcompton this sounds good. This should be high priority. Do you have any concerns about implementing it right away, as there will also be an agenda posted this Friday?

@shrayshray - I can implement this solution tomorrow. I'd rather test this with our upcoming agenda on Friday, rather than wait until we have multiple agendas posted in April (which seems higher risk to me).

@reginafcompton sounds like a plan, thank you!

@shrayshray - we can close this issue via https://github.com/datamade/scrapers-us-municipal/pull/32!
Summary of changes
On Fridays, from 2-10:00 pm CT ––

we scrape all events, on the hour,
we scrape recently updated events, at 30 and 45 after the hour
we scrape all bills, at 5 after the hour
we scrape recently updated bills, at 35 and 50 after the hour

This minimizes the load we place on Legistar, which should prevent an abundance of 104s.


Game does not open

When i click to launch the game after load the pakbypass, dbd anti cheat loads and then it closes automatically, pls help

When i click to launch the game after load the pakbypass, dbd anti cheat loads and then it closes automatically, pls help

Pak Bypass is not updated, wait for next update. Join the discord for more info.


Added kops hacktoberfest tweet

/hold
@rajula96reddy thanks for the PR! We've had a lot of spam since hacktoberfest started, so we might want to revisit tweeting about it.
/assign @mrbobbytables

At this point I don't think we should. We're getting spammed with trivial PRs and the last thing we want to do is encourage it :(

Going to close this since the kops office hours was scheduled for 9am EST today. Thanks anyway, @rajula96reddy :)


PM-2937: Local messaging protocol

Added a checkpointing.interpreter.messages.InterpreterMessage to capture the requests and responses that can go between the Checkpointing Service and the Checkpointing Interpreter.

Yes, a full specification for message passing is one of the first items @fishermanymc and the team are working on.


Generate AppStream data when publishing [enhancement]

It would be amazing if aptly would generate AppStream data when it publishes a repository. This would allow packages in the published repo to be seen by new-age app centers, including gnome software.
Here is a generator written in D for reference.

As long as Aptly reads the dep11/ directory and puts the correct checksums for Components-* into the InRelease file, asgen and Aptly should work well together.
I would advise against writing an own metadata generator, since that is a huge pain, and keeping it up-to-date is an issue. But making asgen easier to use from other tools would be possible (at time, it is mainly built for large distro repositories and to be a standalone tool).

I'm changing this issue's scope to generate checksums for all Components files. This will be the easiest solution to this issues, and some other issues related to metadata generation.
Expected behavior:

Have a script that generates Components files from the aptly repository.
Link the folder so you end up with a route similar to /aptly/public/stable/dists/xenial/main/dep11/Components-arm64.yml.gz
When aptly switches published repo or snapshot, it includes checksums for all Components files in /aptly/public/stable/dists/xenial/main

I also put a $15 bounty on this for anyone who want's to add it:
https://www.bountysource.com/issues/33086807-generate-checksums-for-all-components-files

Just something to add before it gets forgotten: The same thing needs to be done with the icons-* tarballs in the same dep11 subdirectory.
Also, the checksums for the uncompressed files need to be in InRelease, but the actual uncompressed files need not (and probably should not) be present.

Bumped the bounty to a total of $50 :)

I have been looking this, alas, it is a bit tricky. I'd absolutely love some input on this @smira
We need a bunch of information which could be different per source/publishedrepo/usecase.
Namely the following appstream-generator attributes:

UseIconTheme (overrides icon to use)
MediaBaseURL (fully qualified URI to a media directory)
HTMLBaseURL (URL to a html directory, if I am not mistaken this can actually be relative)
DataPriority (allows overriding of other appstream data)
The other attributes we can pretty much fill with information we already have in aptly anyway.

With that in mind I am currently thinking about the following changes to make this happen:
api/publish.go for both initial publishing and updating gets a new attribute AppStream: {} with the aforementioned attributes for appstream-generator.
Final payload should be something like
{
  "Prefix": "floof",
  "SourceKind": "local",
  "Sources": [{"Name": "local-repo"}],
  "AppStream": {
    "MediaBaseURL": "http://floof.com/media",
    "HTMLBaseURL": "http://floof.com/html",
    "IconTheme": "breeze",
    "DataPriority": 1
  }
}

We probably ought to mangle this information internally a bit more. MediaBaseURL for example should probably be expanded to be "prefix-unique" by appending the actual publishing prefix, so what we give to asgen would be $MediaBaseURL/$Prefix -> http://floof.com/media/floof.
deb/index_files.go gets a new Dep11Asset type, this type is used for all files we want to dump in dist/$component/dep11/ (namely icons-* and Components-*). Ideally this should be only an addition. Which is a bit meh because index_files does assume the bufio.Writer gets decompressed data, while appstream gives us compressed data.
To that end utils/compress.go should probably get a new helper to decompress a file and load it into memory. This should work for starters but will ramp up memory consumption with appstream's icons.tar, so long-term refactoring of index_files to not assume uncompressed data would probably be good.
deb/publish.go probably gets a new argument for the AppStream options we got via the API, it gets a new loopedy loop which does:

construct a Config for asgen using available information
adds options obtained via API
writes that into a path under $AptlyRoot/asgen/$Prefix
runs asgen
foreach export/data/$Dist/$Component/*

decompresses file into memory
constructs Dep11Asset
writes decompressed data into bufio of the asset
(the assets get finalized alongside the rest of the index_files stuff) at which point they get compressed again

This should give us a largely working lineup for local repositories. How to deal with remote publishing endpoints I am not really sure to be honest.
All of this also assumes that some other piece of software grabs $AptlyRoot/asgen/$Prefix/export/media (i.e. the screenshots) to publish them under the URL we were provided.
We will probably also need some global .aptly.conf option to configure an appstream-generator run string so that one can "intercept" and redirect the appstream-generator run (e.g. to run in a container, or on a different host). Running appstream-generator on the same system as aptly is a bit meh from a security POV.


Wrap Top Level Element

The current behavior is to replace the container given to makeDOMDriver(container).
Given this as input
h('div', {}, ['Hello, world'])

will make

Loading...

equal

Hello, world

However Cycle-DOM wraps the top element, in this example that would be  #app. Given the same input it would output

Hello, world

I'd like to gather a consensus on whether this behavior should be implemented or be left the way it is.

Benefits I see of wrapping the virtual trees with the root element:

Ease of migration from Cycle -> Motorcycle
Support for any styles that may be applied to the root Element

Benefits I see for the current behavior:

Allows modification of the root element

I think the reason given by @staltz in this commit https://github.com/cyclejs/cycle-dom/commit/866a5b37e8422e599804cfa56892347151b8a350

Diff and patch root element is not anymore a child of the
given container (to the DOM Driver). It is now the given
container itself. Diff and patch will not override and destroy
the identity of the given container. Instead, it will preserve it.
In fact, the given container element is parsed to a vtree, and is
used as the first vtree in the diff & patch sequence. This enables
avoiding to rerender the same tree client-side if it was already
rendered server-side, which saves processing power especially on
mobile devices. The semantics of the root element Observable,
get(':root'), for the DOM Driver has also changed: the first
emission is what was given as the container. This is important for
attaching event listeners.

... is such a strong argument that the implementation ought to be the same in motorcycle-dom.

It's kind of hard to find any reason to disagree with all of that.

Shall we stick with the same implementation?

I'm starting to think it may be best. I've tried starting for scratch multiple times, but no luck. Probably should close this PR and keep the issue open for the future. Any objections to that?

No objections. Just remove the milestone from the issue.


Support for pure markdown generation

fixes #5

@rrmcguinness I thought that creating a config type rather than adding more parameters would avoid explosion of function signature, and also make it easier to add in the future additional flags.

@rrmcguinness fyi, I am out of office next week, so take your time with the review :)


Fix bug viewed schedule disappearing when person viewed edited

Codecov Report

Merging #182 (402e624) into master (4b78852) will decrease coverage by 0.09%.
The diff coverage is 20.00%.

@@             Coverage Diff              @@
##             master     #182      +/-   ##
============================================
- Coverage     66.73%   66.63%   -0.10%
- Complexity      665      666       +1
============================================
  Files           107      107
  Lines          2363     2368       +5
  Branches        312      313       +1
============================================
+ Hits           1577     1578       +1
- Misses          665      667       +2
- Partials        121      123       +2

Impacted Files
Coverage Δ

...edu/address/logic/commands/person/EditCommand.java
89.47% <0.00%> (-4.98%)
:arrow_down:

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
95.23% <33.33%> (-3.10%)
:arrow_down:

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 4b78852...402e624. Read the comment docs.


Specifying a range for train_size question

Quick question to the developers, because I couldn't find an answer in the documentation: what is the reasoning for specifying a range of train_size and not a fixed value? I know it is possible to specify a fixed value, just wanted to know why specifying a range is also an option...

Hi @bart-vanvlerken
One of our main goals with Robyn is minimizing analysts' bias in "smart ways". When splitting the data we could fix a train size (say 80%) and continue the process as is, or let the user pick any value (based on X). But why .8 and not .7 or .75? Given we are iterating and building models by minimizing multiple errors (MOO) we don't want to set a single value that works for all time-series nor businesses. That is why we pass the train size as a hyperparameter so that nevergrad multi-objective optimization finds which train size minimizes the errors, thus one less (biased) decision on the hands of the analyst. And, as you can see in the time-series validation plot, even though when we provide a .5-.8 range, it doesn't always try to "overfit" by taking as much data as possible (.8) but finds an in-between value that works best for the optimizer.

That makes sense! Thanks for the quick answer!

This makes sense regarding removing potential bias in defining the split but with data points usually quite low for MMMs, even with daily data, have you ever considered that reducing sample size is problematic? Also, is there a case for not using ts_validation and just fitting the model to the data while checking the account residuals and R2? Thanks.


Update "early" meeting time

AFAICS there was no objections?
cc @kingdonb @scottrigby

No objections raised at the last meeting, please update the time to +2h. Thanks @dholbach

This is done on the flux-dev list calendar. Requested through service desk to be mirrored on the shared cncf calendar. Let's keep this open until that ticket is closed. I'll update here when it is 🙂

AFAICT it was updated. It's what Amye says on the ticket too.


Bridger weapon perk interaction

Neither the ballistics skill nor the heavy weapon skill provide boosts to the damage output of the Bridger grenade launcher. Both include the Bridger in their perk art so I believe we can assume its an oversight.

Details
The Bridger has two sources of damage, first the projectile itself does some amount of bulk damage and then the explosive does splash damage of a lesser amount. Currently the only perk effecting it is explosives and it effects both, ideally I think the explosive splash would be effected by explosive and the main damage would be effected by heavy weapons and ballistics
Game Version
<IP_ADDRESS>
New Game
Yes
Reproduction Steps
Spawn the bridger weapon or any variant, test damage or verify using in game menu, apply ballistics or heavy weapons perks and test again

This is my own fault for not testing more closely before posting this but please note that I had it backwards, the bulk of the damage is explosive splash with a small amount that is not. the rest remains accurate

It also does not benefit from the ballistic weapon skill magazines

I don't necessarily agree with the ballistics keyword being missing, as this is a rocket launcher weapon. However, it should definitely have the WeaponTypeHeavyGun keyword.
The Negotiator also appears to be lacking this keyword.
Both will be fixed in 0.1.8.

Been reconsidering the ballistic keyword, and I will add it to these weapons as well. These are my reasons.

It is in the Ballistics perk icon, as you mention.
It currently does not benefit from rapid reloading without this keyword.
The Rev-8's turret benefits from Lasers, Demolitions, and Heavy Weapons. This would bring these rocket launchers in line with that weapon.
The damage of these weapons is still fairly low without the Ballistics keyword.
At the end of the day, the 40MM XPL rounds are just simply ballistic projectiles.


Latest Rockstar Games Launcher (needed for GTAV) unable to launch with dxvk

The latest launcher update spits out an "Unable to launch game, please verify game data" error when attempting to launch with dxvk, it works fine with wined3d.
Using a custom winecfg for GTA5.exe and manual/symlink dxvk means you can get into game but it's not very practical for proton.
I've included a wine log output running wined3d below.
Software information
Rockstar Games Launcher
Due to other install bugs the .exe needs to be manually unpacked as described here:
https://github.com/ValveSoftware/Proton/issues/37#issuecomment-532857480
It also requires vc runtime 2019 (and corefonts if running in wine).
System information

GPU: GTX 1070
Driver: 435.21
Wine version: Proton/Lutris 4.15
DXVK version: 1.3.4

Log files
wined3d output - output.txt

This is a bug in Wine's Direct2D implementation. I do not plan to work around this in DXVK.

I tried that commit and it made no difference, unfortunately an update today has broke the launcher even more & now the game doesn't launch with the wined3d workaround in the first post.

This comment may be relevant for testing today's launcher update.

This comment may be relevant for testing today's launcher update.

I've just done those patches and combined with latest dxvk master the launcher now works graphically at least, however today's update has caused some other issues,
I'm confident that the issue with dxvk not working with the launcher now has a potential fix so I can probably close this, just the administrator issue for now to resolve.
Thanks again @doitsujin for the dxvk related fixes.


_WD_UpdateDriver fix for $oNameSpace.items

Pull request
Proposed changes
fix for:  $oNameSpace.items The requested action with this object has failed
Checklist
Put an x in the boxes that apply. If you're unsure about any of them, don't hesitate to ask. We are here to help!
This is simply a reminder of what we are going to look for before merging your code.

[X] I have read and noticed the CODE OF CONDUCT document
[X] I have read and noticed the CONTRIBUTING document
[ ] I have added necessary documentation or screenshots (if appropriate)

Types of changes
Please check x the type of change your PR introduces:

[X] Bugfix (change which fixes an issue)
[ ] Feature (change which adds functionality)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (functional, structural)
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?
https://github.com/Danp2/au3WebDriver/issues/203
What is the new behavior?
works without errors
Additional context
https://github.com/Danp2/au3WebDriver/issues/203
System under test
not related

@seadoggie01  could you confirm that this works for you ?

Also posted request for @neypro here:
https://www.autoitscript.com/forum/topic/205553-webdriver-udf-help-support-iii/?do=findComment&comment=1496880


[1.10.2-5.06] Item duping with Storage Drawers mod

If you craft something in the storage scanner that has an routable drawer from the Storage Drawers mod v3.2.1 by jaquadro, items that are used from the drawer get duplicated in the drawer itself. So after using one single piece from the drawer, the drawer gets filled up to its potential limit (defined by upgrades).
I don't know if thats RFTools' fault or the Storage Drawers' but there's obviously something wrong with the remaining items calculation.
EDIT: watched it closely again.. You don't even have to craft something. Hovering over the buttons (craft 1, 4, 8, stack) is enough to duplicate the items in the drawer. So it's more likely to be RFTools' issue..

I can confirm that I've seen this behavior as well, but I've only actually noticed it with compacting drawers.

Tested it with all: compacting, normal, 1x2, 2x2... Same thing... The interesting part is, that it doesn't work with all items.. Diamonds, emeralds and lapis don't dupe, iron, gold and redstone do (and many more)..

Weird. For me, only compacting drawers have duped anything, including diamonds, emeralds, and lapis. It does't always go instantly to max capacity though. I saw my emeralds go from 100-ish to 1000-ish then like 6k, then suddenly 600k.

Yeah somehow my emeralds, diamonds and so on now decided to dupe, too.. I realized that it counts exponentially upwards.. About every 4 ticks your amount doubles..

Confirming that I've had this issue too, specifically involving the Storage Scanner and Compacting Drawers.  I've had the inventory numbers suddenly shoot to the drawer's max capacity, and I've also had inventories disappear to zero when extracting from the drawer.

I've noticed the same issue, I suspected the storage scanner as well. Not all items seemed to do it. But hard to pin it down..

I don't see how this can be RFTools fault (although it is always possible) because I just access the storage drawer like I access any inventory (chest or whatever). My guess is storage drawer is doing something weird in its inventory handling. Was this reported to them too?

Not yet, but as I mentioned above: hovering over your craft 1,4,8,stack buttons is enough to dupe the items so there should be virtually no item handling until then unless your code stores the items somewhere while hovering over these buttons...

There is already item handling then as the crafting system will actually extract the items to see if it has enough and then places them back in exactly the same slots. That's how the hover operation works so even that does work on the item handler

That's weird.. I never had any duping issues with the drawers and other methods of extracting and inserting items like pipes or applied energistics storage busses respectively refined storages external storage (raoulvdberge writes in his wiki for refined storage, that "There is special integration for Storage Drawers.")... Maybe they need special handling but otherwise everything works just fine

Please link me to the section of code that does this item handling.
Without any prior knowledge of what's going on, my guess is that you may be making assumptions that are valid for normal stack-limited inventories, but are not enforced by the contract of IItemHandler.  I can try to review that interaction.
Unlike 1.7.10, I don't believe it's necessary to use my API to take full advantage of deep storage interactions; there are ways to get the necessary information with just IItemHandler.

The code starts here where the storage scanner gets a request to test if a crafting operation is possible: https://github.com/McJty/RFTools/blob/1.10/src/main/java/mcjty/rftools/blocks/storagemonitor/StorageScannerTileEntity.java#L105-L120
This creates an item source which is basically a collection of all routable inventories (and will include the storage drawer)
This will go to:
https://github.com/McJty/RFTools/blob/1.10/src/main/java/mcjty/rftools/craftinggrid/StorageCraftingTools.java#L193-L235
The actual inventory handling is done in the item source:
https://github.com/McJty/RFTools/blob/1.10/src/main/java/mcjty/rftools/craftinggrid/TileEntityItemSource.java
That code implements an RFTools interface to abstract away ItemHandlers and IInventories. It will always try to use the item handler first. This is probably the code you want to look at. I might be doing something wrong there but I wouldn't know what

Alright, who can describe how to create a repro case (preferably with pictures?).  Assume I have no working knowledge of RFTools.
Tangentially related: in my five minutes of fumbling around I put down a storage scanner, made a drawer routable, double-clicked an item shown, and crashed out to the launch screen.  Large wall of stacktrace here: https://gist.github.com/jaquadro/b6060e3aaea37d198501d01270351d27

@jaquadro I can add pictures in a minute, but basically:

Place down some drawers (any type, doesn't matter)
Put a drawer controller on them
Place a storage scanner
Set the radius on the scanner to include the drawer controller, but not the actual drawers
Put over 1 stack of items in the drawer
Go to the storage scanner, make sure the drawer controller is starred, and set a recipe for something that uses whatever you put in the drawers (e.g. cobble stairs if you put cobblestone in the drawer)
Hover over any of the craft buttons
You should now be duplicating items

Note that the storage scanner requires power. A creative powercell from rftools is the easiest way to give that

@McJty I think the issue is https://github.com/McJty/RFTools/blob/1.10/src/main/java/mcjty/rftools/craftinggrid/StorageCraftingTools.java#L135
I changed that line to
undo.put(key, new ItemStack(input.getItem(), input.stackSize)); and the bug appears to be fixed

a) I don't get why that would fix the bug
b) That's not a good fix as it will destroy NBT on items that are in a chest
c) input.copy() should make a proper copy of the itemstack WITH NBT

a) Looking at this again, I realize I shouldn't code while I'm this tired. >_<
b) Ah, yes. I'm really new to modding MC. Didn't think about NBT issues. Good to know.
Could the issue be related to itemStacks that have stackSize > 64? I noticed that if I log the undo map, diamond shows up with a stack size > 64 when using a storage drawer. However, if I have a modular storage with an equal amount of diamonds, it shows up as < 64
I don't know the specifics about how MC handles item stacks, but seems like storage drawers might be reporting items as aggregate stacks, instead of multiple stacks <= 64 in size, which then causes duping when you call .putStack on line 160.
Anyways, I don't have any more time to test this tonight. Hopefully this is helpful. Like I said, I'm new to MC modding, but I'm glad to help out with testing this further :) I've learned a lot about how item stacks/storage work from investigating this bug

Does storage drawers perhaps have a custom ItemStack implementation that doesn't properly support copy() or so? Just guessing here as I don't really know what is going wrong

Put a drawer controller on them
Set the radius on the scanner to include the drawer controller, but not the actual drawers

I wanted to note that I saw the duping behavior in a setup without any Drawer Controller, and with the drawers (Compacting Drawers, to be specific) included in the Storage Scanner listing.

I guess we can safely say that either rftools is making an assumption on how to use IItemHandler that isn't valid or else that storage drawers is implementing its ItemHandler wrong

I need to wait until evening to look at this again.  I could craft theories if it were just the compacting drawers, or just the controller at fault.  I don't have anything for the general case right now.

Alright, I think I've got to the bottom of it, and it's the kind of squishy ambiguous semantics business I thought it might be.
I think the problem is in TileEntityItemSource.putStackInSlot.  A call is made to handler.getStackInSlot(), which returns an ItemStack for what's in that slot.  If we check the API, its documentation states:
 * The result's stack size may be greater than the itemstacks max size.

So SD happily returns you an arbitrarily large stack.  Then you attempt to call handler.extractItem() with the same count.  Now here's where the problem creeps in.  Let's review the documentation for that API entry:
 * Extracts an ItemStack from the given slot. The returned value must be null
 * if nothing is extracted, otherwise it's stack size must not be greater than amount or the
 * itemstacks getMaxStackSize().
 *
 * @param slot     Slot to extract from.
 * @param amount   Amount to extract (may be greater than the current stacks max limit)
 * @param simulate If true, the extraction is only simulated
 * @return ItemStack extracted from the slot, must be null, if nothing can be extracted

Based on the wording and some of the conversation in the PR that introduced IItemHandler, I interpret it as extractItem only allowing the extraction of stacks up to getMaxStackSize(), even if you ask for more.  And thus repeated calls would be needed to drain the inventory.  On the other hand it's a stupid-looking constraints and not consistent with with the rest of the API.  So it's conceivable I didn't understand this right, and I'd be happy to make a change if that's the case.  Ultimately I'm trying to stick to spec on this implementation.
So jumping back to your code, the extract didn't pull out as many items as you assumed it would, and then you try to insert a much larger stack that I assume was derived from another call to getStackInSlot.  Pretty easy to see how the item count goes into runaway after repeated calls.
Happy to have your input on this, and any other authoritative info on IItemHandler that you may by privy to.

On a side note, I'd suggest calculating a diff and adding or subtracting that amount rather than removing the entire inventory and then re-applying the original amount.  Some inventories like SD do react immediately to an inventory going to 0, although I don't think it will do anything harmful currently.

Thanks for the investigation. I will see if I can make my code safer to avoid the problem

I'm changing this code to only extract amounts like you suggested. That should make it a lot more stable and robust

I can no longer reproduce the dupe with storage drawers. Will be fixed next release


can't resolve 'querystring'

Compiled with problems:X
ERROR in ./node_modules/kiteconnect/lib/connect.js 9:18-40
Module not found: Error: Can't resolve 'querystring' in 'D:\Project\node_modules\kiteconnect\lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "querystring": require.resolve("querystring-es3") }'
- install 'querystring-es3'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "querystring": false }
I see 'querystring' is used in kiteconnect but is not included in dependency in the project

What node version you are using?

I guess you are trying to compile this to JS for frontend but this lib supports only Node JS.

Install this package - npm i query-string. (because old querystring version does't support with node 5 or above version).
Goto this path - node_module/kiteconnect/lib/connect.js.
replace var querystring = require("querystring"); to var querystring = require("query-string");
Run server again.


Support Lustre 2.10.3

Fixes #442
Signed-off-by: Joe Grund<EMAIL_ADDRESS>

This change is

Did you double/triple check that the tests actually used 2.10.3?

Looks like it from the sos reports:
MESSAGE=Lustre: Lustre: Build Version: 2.10.3


Fix issue with python executable

Currently seeing the issue below with non-standard python paths in custom base images. Since the dockerfile sets this env var (both in the case of custom image, and in the case of normal image) we should rely on it for the python to run the inference server, instead of only looking for standard python path's, which we are not using to install the necessary requirements.
Context:
Config excerpt:
base_image:
  image: "custom/image:tag"
  python_executable_path: /app/env/bin/python

Since /app/env/bin/python is not in the current PYTHON_EXECUTABLE_LOOKTUP_PATHS, we're always going to be running the wrong python.
This results in the following error
root@baseten-model-qz48jow-00001-deployment-5ffdc55685-gkbzv:/app# /usr/bin/python3 /app/inference_server.py
Traceback (most recent call last):
  File "/app/inference_server.py", line 4, in <module>
    import yaml
ModuleNotFoundError: No module named 'yaml'

because the server requirements are installed using  /app/env/bin/python -m pip install ..

Passing integration tests: https://github.com/basetenlabs/truss/actions/runs/8258993722
Should be good to go.


Avoid compiling requirement or give info in Readme

Currently psycopg2 has to be compiled when pip install karmabot (in some cases?)
Error: pg_config executable not found.

pg_config is required to build psycopg2 from source.  Please add the directory
containing pg_config to the $PATH or specify the full executable path with the
option:

    python setup.py build_ext --pg-config /path/to/pg_config build ...

or with the pg_config option in 'setup.cfg'.

If you prefer to avoid building psycopg2 from source, please install the PyPI
'psycopg2-binary' package instead.

For further information please check the 'doc/src/install.rst' file (also at
<https://www.psycopg.org/docs/install.html>).

See #50 for information.
We should figure out a way to remove this step or add some info to the Readme.

Thanks @asottile https://pypi.org/project/psycopg2-binary/ :)


⚠ ClamAV has degraded performance

In a613f92, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in 4896bfe.


Angular 11 upgrade triggers deprecated packages warnings (resolve-url, urix, tslint, uuid, querystring, chokidar, request, fsevents) on npm install

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc
[x] install

Is this a regression?

Description

🔬 Minimal Reproduction

🔥 Exception or Error
🌍 Your Environment

Anything else relevant?

The content in the issue was lost.. Will raise a new issue


Windows 10 Build 17134 not returning update

Expected Behavior
Note of latest CU as seen here
PS C:\Users\sgates-dadmin> Get-LatestUpdate -WindowsVersion Windows10 -Build 16299
KB        Note                                                                                URL

KB4467681 2018-11 Cumulative Update for Windows 10 Version 1709 for x64-based Systems (KB4467681)     http://download.windowsupdate.com/c/msdownload/update/software/updt/2018/11/windows10.0-kb4467681-x64_092c977b72ae27aab88c65ab31b81d3d861dda3a.msu
KB4467681 2018-11 Cumulative Update for Windows Server 2016 (1709) for x64-based Systems (KB4467681)  http://download.windowsupdate.com/c/msdownload/update/software/updt/2018/11/windows10.0-kb4467681-x64_092c977b72ae27aab88c65ab31b81d3d861dda3a.msu
Current Behavior
No response
PS C:\Users\sgates-dadmin> Get-LatestUpdate -WindowsVersion Windows10 -Build 17134
PS C:\Users\sgates-dadmin> Get-LatestUpdate -WindowsVersion Windows10 -Build 17134
Failure Information (for bugs)
Please help provide information about the failure if this is a bug by issuing the command using the -Verbose command.
PS C:\Users\sgates-dadmin> Get-LatestUpdate -WindowsVersion Windows10 -Build 17134 -Verbose
VERBOSE: Adding variable for dynamic parameter 'Build' with value '17134'
VERBOSE: Check updates for 17134 Cumulative.*x64
VERBOSE: Downloading https://support.microsoft.com/app/content/api/content/feeds/sap/en-us/6ae59d69-36fc-8e4d-23dd-631d98bf74a9/atom to retrieve the list of updates.
VERBOSE: GET https://support.microsoft.com/app/content/api/content/feeds/sap/en-us/6ae59d69-36fc-8e4d-23dd-631d98bf74a9/atom with 0-byte payload
VERBOSE: received -1-byte response of content type application/atom+xml; charset=utf-8
VERBOSE: Found ID: KB4467682
VERBOSE: GET http://www.catalog.update.microsoft.com/Search.aspx?q=KB4467682 with 0-byte payload
VERBOSE: received 30768-byte response of content type text/html; charset=utf-8
VERBOSE: Ids found:
VERBOSE: innerText found. Parsing KB notes```

**_If it is not a bug, please remove the rest of this template._**

## Steps to Reproduce

Please provide detailed steps for reproducing the issue.

Type the command Get-LatestUpdate - WindowsVersion Windows10 -Build 17134

## Context

Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Tested on two machines. Server 2016 and Windows 10 1809. Other builds work.

This is due to MS pulling the update. Sorry for bothering you

I have an update coming that will use an RSS feed that should fix this.


Fence Rep keeps resetting

Versions

SPT-AKI Server version 3.8.0
SPT-AKI Profile Editor version Newest Version
List of installed mods
aMoxoPixel-Painter
BarlogM-Andern
EpicRangeTime-Weapons
ExpandedTaskText
MoxoPixel-BlackCore
MoxoPixel-MagTape
MoxoPixel-TacticalGearComponent
Skwizzy-NoDiscardLimit
SWAG
zSolarint-SAIN-SeverMod

Describe the bug
A clear and concise description of what the bug is.
The Fence rep/Scav rep keeps resetting after i change it in profile editor. from max rep to fresh wipe rep
Additional context
SPT-AKI Profile file
Just create zip archive with your profile file and drag'n'drop here. Bug reports without attached profile file will be closed without any help if they related to user profile

@Tombstonex  attach profile before and after editing, describe all your actions in PE, and when rep resetting happening

before - 660b319a00048c97e43853dc.json
after - 660b319a00048c97e43853dc.json
I got this reported twice in the support channel of SPT and just figured out that PE is why it's happening
Profile editor is only writing the PMC data for Fence Rep - when you run as a scav on extract/death it'll overwrite the PMC fence rep as it no longer checks PMC fence rep for calculations in 3.8.0
If you run a PMC raid after setting it, the value will stay (and I believe writes to scav fence rep value, but a SPT dev can confirm)

Fixed in e4033a01afbe2fe5b8b6d299d6fae365551bc25d, but this is not a critical issue, so there are no plans for the next release yet


Cleanup Leaderboard

[Store] Created a reset method that handles everything needed to reset the game
[Store] reset is no longer a property on control
[utils] Moved everything from utils into a single data file.
[data] Added a default config that can be overridden by environment variables
[data] Don't create a new client on every request
[data] Changed method names to reflect the name of the database: scores
[Finished] Tweaked wording: "You belong on our leaderboard!"
[Leaderboard] Fixed bug where clicking on the button would delete the game state
[Leaderboard] Fetch scores when opening the leaderboard and no more than once every 3 seconds
[Vehicle] renamed ctrl to controls
[Vehicle] imported only the used methods from three
[Vehicle] use maxSpeed instead of vehicleConfig.maxSpeed
[Store] Moved angularVelocity, position, & rotation back to the store
[Store] Don't export wheelInfos

Some issues with the Finished component:

Setting your name should also submit your score (you have to click add me twice to get your score submitted now)
There's no error handling for the data requests
There's no progress indicator for the data requests (if they take too long, people will resubmit and we'll get duplicates)
The time seems to continue after you've finished

I prefer the way utils was setup before, but I like the rest of the proposed changes!
(I'm currently working on auth and having those functions separate is much cleaner

I prefer the way utils was setup before, but I like the rest of the proposed changes!
(I'm currently working on auth and having those functions separate is much cleaner

The problem with utils is that it becomes a dumping ground because technically everything is a utility function.

okay I understand your concern, how about we have a directory just for supabase?

okay I understand your concern, how about we have a directory just for supabase?

There's a data file that exports two functions, no need to futureproof it now. When the data API changes the file structure can change to match it.

im happy with the changes so far. as for @njm222 suggestion, maybe we rename the folder called store and put the store in it, constants, mutation and supabase? would you like that @bjornstar ?

im happy with the changes so far. as for @njm222 suggestion, maybe we rename the folder called store and put the store in it, constants, mutation and supabase? would you like that @bjornstar ?

The file structure here is not permanent, they can be changed after this PR lands. I tried not to change too much as several of my changes have been reverted recently.


CocoaPods v6.0 is not found

Library setup method: CocoaPods
Version of the library. 6.0.
Xcode version. 10.0
OS version. iOS 12.0

Thanks for reporting, @kuoliangkwong. Can you explain the problem and supply the error message please, if there is one? I'm not quite sure what you mean. :)

I got this error when pod install using BadgeSwift v6.0
Here's the error I got
[!] CocoaPods could not find compatible versions for pod "BadgeSwift":
In Podfile:
BadgeSwift (~> 6.0)
The latest version I can get is v5.0.1

Hmm, that's weird. Is it the entire error message. Does it say what it means by "compatible versions"? I've also found this discussion on stackoverflow that might be relevant.

Hmm i tried again, and it seems working now. What I did is pod update and pod install again. Did u change anything at ur end?

I'm glad that it's working now. I did not make any changes to the repository for the last two weeks.

Seems odd~ I didn't do anything from my end either haha. Glad it's working now too. I'll proceed close this issue


Just thought I'd say I love the deploy button :)

@SteveALee did you check out this as well? [http://functionlibrary.azurewebsites.net/](Azure Function Library)

@jefking I did thanks! And I hope to add something of value in time


fix broken link

@mairaw LGTM. Although for clarity it might be good to add ?displayProperty=fullName to the xref links to methods.

You're right @rpetrusha. I wasn't really sure how this would render, However, I did fix the wrong link 😆
The one that was broken was the second one.

LGTM.


Restrict build status to master branch

I don’t want people to see a broken build badge for PRs :octocat:

Thanks @mischah! :purple_heart:


Databases require character set utf8

https://www.cloudera.com/documentation/enterprise/latest/topics/install_cm_mariadb.html#install_cm_mariadb_newdbs

Closed as fixed by #17


RecursionError: maximum recursion depth exceeded

➜  py12306 git:(master) ✗ python main.py
######## py12306 购票助手，本程序为开源工具，请勿用于商业用途 ########

日志已输出到文件中: runtime/12306.log

WEB 管理页面已开启，请访问 主机地址 + 端口 8008 进行查看

 * Serving Flask app "py12306.web.web" (lazy loading)
 * Environment: production
   WARNING: Do not use the development server in a production environment.
   Use a production WSGI server instead.
 * Debug mode: off
Unhandled exception in thread started by <bound method Thread._bootstrap of <Thread(Thread-5, started daemon<PHONE_NUMBER>34720)>>
Traceback (most recent call last):
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/threading.py", line 926, in _bootstrap_inner
    self.run()
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/threading.py", line 870, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/robot/Documents/Projects/gitRepository/py12306/py12306/user/job.py", line 63, in run
    self.start()
  File "/Users/robot/Documents/Projects/gitRepository/py12306/py12306/user/job.py", line 76, in start
    self.check_heartbeat()
  File "/Users/robot/Documents/Projects/gitRepository/py12306/py12306/user/job.py", line 86, in check_heartbeat
    if not self.handle_login(): return
  File "/Users/robot/Documents/Projects/gitRepository/py12306/py12306/user/job.py", line 114, in handle_login
    return self.login()
  File "/Users/robot/Documents/Projects/gitRepository/py12306/py12306/user/job.py", line 126, in login
    answer = AuthCode.get_auth_code(self.session)
  File "/Users/robot/Documents/Projects/gitRepository/py12306/py12306/helpers/auth_code.py", line 30, in get_auth_code
    img = self.download_code()
  File "/Users/robot/Documents/Projects/gitRepository/py12306/py12306/helpers/auth_code.py", line 61, in download_code
    return self.download_code()
  File "/Users/robot/Documents/Projects/gitRepository/py12306/py12306/helpers/auth_code.py", line 61, in download_code
    return self.download_code()
  File "/Users/robot/Documents/Projects/gitRepository/py12306/py12306/helpers/auth_code.py", line 61, in download_code
    return self.download_code()
  [Previous line repeated 953 more times]
  File "/Users/robot/Documents/Projects/gitRepository/py12306/py12306/helpers/auth_code.py", line 52, in download_code
    response = self.session.get(url)
  File "/Users/robot/.pyenv/versions/venv37/lib/python3.7/site-packages/requests/sessions.py", line 546, in get
    return self.request('GET', url, **kwargs)
  File "/Users/robot/Documents/Projects/gitRepository/py12306/py12306/helpers/request.py", line 60, in request
    response = super().request(*args, **kwargs)
  File "/Users/robot/.pyenv/versions/venv37/lib/python3.7/site-packages/requests_html.py", line 672, in request
    r = super(HTMLSession, self).request(*args, **kwargs)
  File "/Users/robot/.pyenv/versions/venv37/lib/python3.7/site-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/Users/robot/.pyenv/versions/venv37/lib/python3.7/site-packages/requests/sessions.py", line 668, in send
    history = [resp for resp in gen] if allow_redirects else []
  File "/Users/robot/.pyenv/versions/venv37/lib/python3.7/site-packages/requests/sessions.py", line 668, in <listcomp>
    history = [resp for resp in gen] if allow_redirects else []
  File "/Users/robot/.pyenv/versions/venv37/lib/python3.7/site-packages/requests/sessions.py", line 247, in resolve_redirects
    **adapter_kwargs
  File "/Users/robot/.pyenv/versions/venv37/lib/python3.7/site-packages/requests/sessions.py", line 646, in send
    r = adapter.send(request, **kwargs)
  File "/Users/robot/.pyenv/versions/venv37/lib/python3.7/site-packages/requests/adapters.py", line 449, in send
    timeout=timeout
  File "/Users/robot/.pyenv/versions/venv37/lib/python3.7/site-packages/urllib3/connectionpool.py", line 600, in urlopen
    chunked=chunked)
  File "/Users/robot/.pyenv/versions/venv37/lib/python3.7/site-packages/urllib3/connectionpool.py", line 384, in _make_request
    six.raise_from(e, None)
  File "<string>", line 2, in raise_from
  File "/Users/robot/.pyenv/versions/venv37/lib/python3.7/site-packages/urllib3/connectionpool.py", line 380, in _make_request
    httplib_response = conn.getresponse()
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/http/client.py", line 1336, in getresponse
    response.begin()
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/http/client.py", line 330, in begin
    self.headers = self.msg = parse_headers(self.fp)
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/http/client.py", line 224, in parse_headers
    return email.parser.Parser(_class=_class).parsestr(hstring)
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/email/parser.py", line 68, in parsestr
    return self.parse(StringIO(text), headersonly=headersonly)
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/email/parser.py", line 57, in parse
    feedparser.feed(data)
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/email/feedparser.py", line 176, in feed
    self._call_parse()
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/email/feedparser.py", line 180, in _call_parse
    self._parse()
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/email/feedparser.py", line 295, in _parsegen
    if self._cur.get_content_maintype() == 'message':
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/email/message.py", line 594, in get_content_maintype
    ctype = self.get_content_type()
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/email/message.py", line 578, in get_content_type
    value = self.get('content-type', missing)
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/email/message.py", line 471, in get
    return self.policy.header_fetch_parse(k, v)
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/email/_policybase.py", line 316, in header_fetch_parse
    return self._sanitize_header(name, value)
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/email/_policybase.py", line 287, in _sanitize_header
    if _has_surrogates(value):
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/email/utils.py", line 57, in _has_surrogates
    s.encode()
RecursionError: maximum recursion depth exceeded while calling a Python object

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/threading.py", line 890, in _bootstrap
    self._bootstrap_inner()
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/threading.py", line 936, in _bootstrap_inner
    (self.name, _format_exc()), file=_sys.stderr)
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/traceback.py", line 167, in format_exc
    return "".join(format_exception(*sys.exc_info(), limit=limit, chain=chain))
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/traceback.py", line 121, in format_exception
    type(value), value, tb, limit=limit).format(chain=chain))
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/traceback.py", line 497, in __init__
    _seen=_seen)
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/traceback.py", line 497, in __init__
    _seen=_seen)
  File "/Users/robot/.pyenv/versions/3.7.4/lib/python3.7/traceback.py", line 497, in __init__
    _seen=_seen)
  [Previous line repeated 494 more times]
RecursionError: maximum recursion depth exceeded

Getting the same error.
Were you able to find a solution?
Thanks!

Getting the same error.
Were you able to find a solution?
Thanks!


Add SocketTimeOut exception for CloudFunctions

I'm going through some old PRs to see what we can do with them. First, sorry for dropping this on the floor, we do appreciate your contributions though apparently we don't do a good job of showing it.
For this PR specifically, I'm going to close it because it is lacking tests and all our PRs need to have tests.


ci: fix environment variable references in system tests

@tswast Is there any reason not to stick the service account JSON creds into a repo secret?

I guess the decryption would be the reason at the moment :)

Ok, there's an actual bug here! I'll fix it

@tswast You're right about the head sha bit. I misread the pull_request_target docs, pull_request_target events run against the base of the pull request.

Looks like it's working now. Thanks! https://github.com/ibis-project/ibis-bigquery/runs/4717044031?check_suite_focus=true

Working in PRs as well: https://github.com/ibis-project/ibis-bigquery/runs/4717105215?check_suite_focus=true


FAST Platform: RGBW / LED channel support

In MPF 0.57, FAST LEDs do not work properly for RGBW colors. Also they might not work for channel, next, and previous configuration settings. Investigation is needed as the tests do not cover this.

This has been fixed.


Ssh terminal doesn't shown on some osx machines

As we can understand something changed in osascript. Need to research and fix that.

@Lezh1k I think we already fixed this right?

Fixed


Add explicit license information

Currently (2023-03-14) the json manifests simply say "license": "CUDA Toolkit". On the first read, I thought that it "obviously" refers to the CUDA Toolkit's EULA whose existence precedes NVIDIA's redist packages. This leaves the following questions still open:

How does a user or distribution maintainer resolve the "license": "CUDA Toolkit" stanza into an actual license text?

The redistributable packages include extra paths like e.g. pkg-config/cudart-11.7.pc that are not explicitly white-listed by the EULA text

The nvidia_driver redistributable package contains nvidia_driver-linux-x86_64-530.30.02-archive/lib32/libcuda.so.530.30.02 which, under the EULA text, may only be redistributed when derived from an NGC docker container image and distributed as a part of a docker image

The EULA's text only explicitly allows redistribution of...

variations of these files that have version number or architecture specific information embedded in the file name -

...but not, for instance, in the ELF header, as would be required for some Linux distributions.

More of a rhetorical question: license-wise, what is the difference between the cudatoolkit's .run file and the "redistributable" packages?

A resolution might involve all or a subset of the following:

[ ] Permissions and restrictions are distributed in machine-readable form together with the json manifests
[x] License text is distributed together with the packages: cf. e.g. cuda_cudart/LICENSE.txt
[ ] The license string is replaced with a dictionary { name :: String, url :: String } where url refer to a downloadable file with the license text and name be a stabilized name, from which permissions and restrictions could be inferred
[ ] Redistribution of the redistributable components of CUDA Toolkit is explicitly permitted by the linked license text
[ ] Usage of patchelf prior to redistribution is explicitly permitted by the linked license texts
[ ] Redistribution of the user-space cuda driver (libcuda.so) from nvidia_driver be explicitly permitted. For instance,

[ ] jq .nvidia_driver.license redistrib_12.1.0.json links to a license that allows to ship the contents of nvidia_driver-linux-x86_64-530.30.02-archive.tar.xz with an application.
[ ] The clarifications about docker images are dropped

[ ] Licenses explicitly allow to patch paths in pkg-config files

Thanks!

The license key-value is meant as a "short name" indicator for package maintainers.
a. There is a LICENSE.txt in top-level of each component directory, as well as the included LICENSE text file within each tarball or zip archive.
b. For manifest schema v3, I've added a new license_path key that explicitly points to $component/LICENSE.txt

The pkg-config .pc files are metadata, much in the same way the JSON manifests are. IMHO, much like the .deb/.rpm package templates we provide on Github, these are outside the scope of the payload's licensing.

The nvidia_driver bin-archive tarball contains the extracted contents of NVIDIA-Linux-*.run
a. Also contains extracted contents of matching nvidia-installer, nvidia-modprobe, nvidia-persistenced, nvidia-settings, nvidia-xconfig, and open-gpu-kernel-modules tarballs.
b. The NVIDIA driver license has some interesting changes in >= 535.43.02
c. Thinking about this some more, it's very likely the row mentioned in the table is referring to the cuda_compat component, which is only provided as .deb and .rpm packages. These are pre-installed in the official container images for Forward Compatibility; not supported on GeForce SKUs.

Need some clarification about what this question implies?

Consider the following scenarios for repackaging components for Conda, Debian, Nix, PKGBUILD, RPM, etc. formats
a. CUDA runfile

i. Download the entire .run
ii. Run the installer or extract the contents (--tar mxvf)
iii. Roll new tarballs
iv. Run package build commands

b. Debian or RPM packages

i. For each component, download multiple runtime and devel .deb/.rpm packages
ii. Extract the contents
iii. Roll new tarballs
iv. Run package build commands

c. "redist" bin-archive deliverables

i. For each component, download one* tarball (* some are provided in split archives)
ii. Run package build commands


Allow manual checkpointing

It'd be nice if there was a way to manually checkpoint the WAL file if you need everything in one file at a certain moment.

@northeastprince Thanks for asking about this. Can you share a bit more information about your use case? Adding a wrapper for sqlite3_wal_checkpoint_v2() wouldn't be hard, but I do want to know what it's being used before committing to adding it to the gem.

You may be able to use the pragma for this: https://www.sqlite.org/pragma.html#pragma_wal_checkpoint


perf(updating): avoid creating subproject copy

I've performed an internal change to the update algorithm to avoid creating a subproject copy, which turns out to be a performance regression introduced by #1587.
As of #1587, the update algorithm computes a diff between the dirty HEAD of the subproject's Git repository – dirty because of potentially staged files during a pre-migration task – and a fresh copy based on the old version of the template. Before this PR, a full copy of the subproject directory (excluding the .git/ directory) was created and the copy was initialized as a new Git repository with one commit comprising all files. For large subprojects, creating a subproject copy incurs non-negligible overhead as reported in #1709.
This PR takes a different approach that does not involve copying the subproject; in fact, it does not even involve fetching shallow copies of local Git repositories. First, a reference of the dirty HEAD of the subproject (after the pre-migration task has run) is obtained by calling git write-tree, which creates a tree object without a commit and returns a reference. Also, the HEAD SHA of the temporary Git repository containing a fresh copy from the new template is determined. Then, the diffs involving the dirty HEAD of the subproject or the fresh copy of the new template are computed by referring to these object references. A trick for comparing local Git repositories makes computing the diffs efficient by avoiding fetching shallow copies from local repositories.
Honestly, I'm not super happy with this solution because the update algorithm is getting more and more complex and it feels like we're increasingly stretching the limits of Git. But it keeps the test suite passing while avoiding the subproject copy. :shrug:
WDYT?
Fixes #1709.

Well, it's working except on Windows. I'll investigate tomorrow.

Now, Windows is happy, too. I missed the Windows-specific alternate object directory separator in the Git documentation (; for Windows, : for Unix): https://git-scm.com/docs/git#Documentation/git.txt-codeGITALTERNATEOBJECTDIRECTORIEScode
This is ready for a critical review now. :balloon:

Honestly, I'm not super happy with this solution because the update algorithm is getting more and more complex and it feels like we're increasingly stretching the limits of Git. But it keeps the test suite passing while avoiding the subproject copy. 🤷

Yep, not great for future maintainers, but what other option do we have :confused:? Git is pretty well documented, and you commented the code well too, so I guess it's OK.

I've rebased this PR onto the tip of master and factored out enhancing the git command with additional object directories into a separate function, as we need it more than once now.

Hi,
There is also a regression in 9.3.0, 9.3.1, copier update fails if there is a directory in the destination with no read/write access for the current user (on linux). (because of copytree)
This PR is solves it.

I've added a test that involves updating a project with a separate Git directory. To make this test pass, I've replaced the manual construction of the Git objects directory path by a subprocess call of git that returns the correct path also when a separate Git directory is used.
While working on this fix, I gained a better understanding of Git object sharing, mainly from these resources:

https://git-scm.com/docs/gitrepository-layout#Documentation/gitrepository-layout.txt-objectsinfoalternates
https://docs.gitlab.com/ee/development/git_object_deduplication.html

I had been a little unhappy about using the environment variable GIT_ALTERNATE_OBJECT_DIRECTORIES and introducing a helper function that constructs this environment variable and assigns it to a Plumbum git command. Then, I realized that Git object sharing can be configured also by adding paths to other repositories' objects directories to $GIT_DIR/objects/info/alternate. The result is the same, but the semantics of configuring a Git repository with alternate object stores feels conceptually more sound to me than calling git with a particular environment variable.
So, the new implementation involves configuring alternate object directories in the temporary repository created by the update algorithm. Then, a regular git diff-tree command works with object references borrowed from linked repositories.

Oh, Windows isn't working – again. I'll check what to do about it.

It's working on Windows now, too. Git is sensitive to newline characters in the .git/objects/info/alternates file; it must be the \n character on both Linux/macOS and Windows, not \r\n on Windows. I hadn't realized that Path(...).write_text("\n") writes \r\n on Windows, that's what was keeping the Windows CI jobs from passing.
I think this PR is ready for another review.

Thanks again for bringing up the --separate-git-dir case and for confirming it's working now, @hparfr. :pray:

In essence, instead of fetching copies of other Git repositories, their object stores are linked (via entries in the .git/objects/info/alternates file) which avoids duplicating objects. Then, a foreign object can be referenced by its object ID. This only works for local repositories, but Copier's update algorithm operates on local repositories only (after cloning the old and new template repositories), so all good.
Thanks for the positive feedback. :pray: I'll press merge now.


HTML

HTML
User Story
As a Bootcamp student
I want the prework notes to be structured on a webpage
So that I can easily find and read information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, GIT and JAVASCRIPT with associated study notes listed

Completed


Add operation to generate project make script

Goal is so the make script doesn't need to be tracked by .git, and can be reconstituted easily using the dev tool.

[ ] The driver.sh script need not be in the project directory


Follow-up improvements from #2060

Signed-off-by: Juraci Paixão Kröhling<EMAIL_ADDRESS>Which problem is this PR solving?

Addresses open comments from #2060

The Cassandra integration test is failing, but isn't apparently related to this PR. Master was showing the build as passing, but it failed with the same reason once I started the job again:
https://travis-ci.org/jaegertracing/jaeger/jobs/655134449


prompt overwrites text line if no newline is found

Platform Windows 11
Terminal software Windows Terminal
Describe the problem you are observing.
When the steps are followed below, you see the text initially appear on the screen and then the prompt overwrites the last line.
Steps to reproduce (from within nushell)

touch a
notepad a
type some text, hit enter, type some more text and DO NOT hit enter (this is key, the last line must not have a crlf)
cat a or type a shows both lines of text, and then when the prompt is drawn, it overwrites the last line

Could not reproduce on linux with either  cat testfile or nu's open --raw testfile (File had to be stripped of the last new line by perl -pi -e 'chomp if eof' testfile.txt). The prompt initialization assumes if the cursor is in the first column it can start there else it introduces a new line. Maybe the windows file dumping has different behavior there?

Maybe it's only a Windows thing?

@elferherrera, you use Windows. Can you reproduce this error?

I followed the instructions but it seems to work fine

I'm pretty sure it's only on the last/bottom line of the terminal.

I could reproduce the bug with open --raw testfile on windows terminal as well as wezterm and alacrtitty. I could not reproduce with 'type' command.
The other mentioned bug "reloading config hotkey deletes top line instead of adding it to scrollback" #5305 is reproducible on windows with windows terminal, wezterm and alacrtitty.
This was with 0.61.0

From my experience such issues might be dependent on terminal size or whether it was resized.