Feat: Build Hooks

This adds support for sending build hooks once per thread if action monitor posts have been created.

something is happening with my PRs missing files. . . I don't understand


EOSIO : How to get receiving information of my EOS account???

In the eos developers potal, I can't find API such as get_receive_actions, there is only get_actions.
It seems like show only send transactions. Am I right?
I want to see receive transactions of my account.............
Please help me........

In fact, get_actions already include the actions you're looking for. The reason you're not seeing them, I guess, is that the node you're requesting from filters them out due to performance and disk space consideration.
To prove this, run a local testnet with --filter-on "*" and send some actions. Then use cleos get actions on the receiving account.
In your case, you'd probably need to run your own full node with --filter-on set to your account. Remember to use --replay-blockchain if this is not a fresh start.
Also note that the history_plugin is being deprecated, the MongoDB plugin should be favored at this point as the deprecated plugin will be removed in the 1.3.0 release.

The history plugin is not scheduled for removal at this time, but is deprecated. The mongodb plugin is recommended.

@tbfleming You’re right. The one being removed is eosiocpp :)

@xJonathanLEI why are you removing eosiocpp? how will i compile code if its scheduled to remove ? developers.eos.io still shows to compile it use eosiocpp

I'm not from EOS... So it's not me removing it. Anyway, you're supposed to use eosio.wasmsdk instead of eosiocpp.


[1.16.5] how to add custom recipes? recipes.json resets on startup ...

Hi, I am trying to add custom recipes, but if I add something in the recipes.json, it resets on the next game start.
I tried to add the recipe via Datapack too, but nothing happened... any help?

recipes.json.txt
bronze_ingot.json.txt

this looks not correct too

Mutiple Questions

What version of AF are you running?
What recipe format have you tried? I know there is no documentation for the new version yet, I can write it up if you need it.
When did that thing in the GUI happen?

Alloy Forgery 1.2.0
here are the files:

recipes.json
bronze_ingot.json

any idea how to solve the recipe issues? I would realy like to use this mod in my Fabric Skyblock Modpack.

The legacy Alloy Forgery recipe format is completely different from the current iteration. We are still considering whether or not to backport to this version.
I will write a tutorial on the wiki later tonight for legacy versions. I want to note that with the old versions nothing is done through data. Everything is done through the config folder. You can either add stuff using the API in the mod, or just manually add stuff to the config, and ship it with your modpack.

Thanks. I will try to follow the tutorial when you have published it.


chore: [FX-4186] update Modal to align with Design

FX-4186
Description

full-screen size property has to take 100% of width and height of the screen (not 100% - Nrem as it is now)

How to test

use temploy to check all states in all screen sizes

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[x] Ensure that deployed demo has expected results and good examples
[x] Self reviewed

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers

full-screen changes work as expected!
https://github.com/toptal/picasso/assets/1390758/bc4d0806-1388-4c9e-be4c-c11376dd1875


refactor(app): async function declaration => arrow function

Closes #3

Thanks @mokuteki225 ! Next time add tag Closes: https://github.com/metatech-university/NodeJS-Fastify/issues/3 to commit description, not to PR description and follow this repo PR/Issue naming style you can catch from repo history.

See your landed commit edited title and description: https://github.com/metatech-university/NodeJS-Fastify/commit/ca5b93e454119774120abcae5cbeaf6a3d3f0147

Sure, thanks!


[scss] wrong color decorators on colors with variables

Feature request and possibly a bug with the colorDecorators
The integrated color previewer in VS code doesn't understand the variable value and show black. obviously there is a syntax difference from sass rgba and css rgba, would it be possible to change the behavior according to the syntax?
It was already requested to parse the variables and show their colors and I find that colorize does it pretty well.
Like in the other issue https://github.com/Microsoft/vscode/issues/21705, Atom has plugin for that.

VSCode Version: 1.16.1
OS Version: Windows 8

Steps to Reproduce:

Create a scss file
type inside

$input-border: #cccccc; //handled by colorDecorators and colorize
$input-border-focus: darken($input-border, 10%); //not handled by anyone
$input-border-test: rgba($input-border, 0.7); //not handled correctly by the colorDecorators for sass
$input-border-test1: rgba(0, 0, 0, 0.7); // handled by colorDecorators

$test: $input-border; //handled by colorize
$test2: $input-border-focus;  //not handled by anyone

Reproduces without extensions: Yes

@aeschli  Trying to verify this one... What's the solution here?
I dont see much difference except that for one of the cases we stopped showing the color
Below on the left is latest from master, in the right is stable 1.16.1

The fix is to only show color decorators on color literals. Values with variables, expressions or functions (sass/less) are skipped.
Getting the resolving 100% correct means to rebuild a lot of functionality from SASS and LESS, which is not planned. Extensions can fill in here.

Ah, got it. Verified!

Yep, hopefully the gap will be filled from extensions. In any case it is better to not show incorrect data. Thanks for looking into this.


data lookup for existing hosted zone

data lookup for existing hosted zone - so this doesn't create another hosted zone that doesnt link back to the same 'delegate servers'/namespaces that are registered in your existing domain IF using Route53 registrar.

Ignore, user error, I am supposed to import the TF state of my existing route53 hosted zone before applying so that TF doesn't create a duplicate. /close


Predicovid

Note that the visit table is basically empty - there are a lot of treatment fields present in the DD which were not provided in the data file.
Outcome is not explicitly provided, but I assume the patient has recovered if there is a monthly/month 12 follow-up form filled in. This still doesn't cover many subjects though unfortunately.

table
valid
total
percentage_valid

subject
734
948
77.426160%

visit
255
948
26.898734%

observation
57539
74346
77.393538%

subject

211: data cannot be validated by any definition
2: data must be valid exactly by one definition (0 matches found)
1: data must contain ['age', 'pathogen', 'subject_id'] properties

visit

693: data must contain ['country_iso3', 'outcome', 'start_date', 'subject_id', 'visit_id'] properties

observation

16807: data must contain ['phase', 'date', 'name'] properties

@ekamau could you check these changes for me before I merge the parser? Thanks!


REFERENCE and README changes

add Puppet Contact info to README
minor README changes
fix summary line for types in REFERENCE

Readme looks good.
Should the reference lines be cut off partway through sentences?

@nigelkersten The issue is that the summary line cannot be longer than 140 characters. I had a choice to either remove that altogether or keep the cut off version for now leaving "prettifying" the cutoff for a future PR. We could cut off after the first sentence and add something like an ellipsis (...). When you click on a type name, you go to the full documentation where the same information is present without a cutoff.
Shall we keep it like this or remove the summary from the overview of types? Note that the same happens with tasks a bit down REFERENCE.md.

Ah. Yes lets keep it like this and we can prettify on sentence boundaries later.

Quick question: these changes would require a new release? If so, I was wondering if there could be a small change updated with this. We wanted to have the 'Application-Type' HTTP header sent in each REST call that the module makes. If it make sense to do it in other PR, please go ahead with that instead. This was the change that I was thinking about in the lib/puppet/transport/powerstore.rb file:
def call_op(path_params, query_params, header_params, body_params, operation_path, operation_verb, parent_consumes)
      uri_string = "#{connection_info[:schema]}://#{connection_info[:host]}:#{connection_info[:port]}#{connection_info[:base_path]}#{operation_path}" % path_params
      if !query_params['query_string'].nil?
        uri_string = uri_string + '?' + query_params['query_string']
      else
        if operation_verb == 'Get' # && operation_path.include?('query')
          query_params['select'] = '*'
        end
        uri_string = uri_string + '?' + to_query(query_params) unless query_params.empty?
      end
++      # FIXME: Obtain the version value for 'Application-Type' dynamically (at least, upon first initialization)
++      header_params['Application-Type'] = "Puppet Module for PowerStore/0.8"
      header_params['Content-Type'] = parent_consumes

@ameer-jabbar This needs to be changed in the generation template. I will do that, run a test suite and submit a new PR specifically for that change.

@ameer-jabbar This needs to be changed in the generation template. I will do that, run a test suite and submit a new PR specifically for that change.

Sounds great, thank you!


Integrate llvm-project at 14504cae9a203b56f863031e4a6a6593cdee3bd8

I am taking the lock on this one. The DestinationStyleOpInterface is causing a massive headache, and  it will take me a while to resolve.


Added environment file command line flag

In order to help with Windows users, I've added a command line flag.

LGTM, @yayajacky ?


Changing heatmap value offsets

ScottPlot Version: 4.1.16
Application Type: WinForms
Question: Is it possible to set the value offsets for the heatmap plots? If not in the current version then maybe in next versions?
Following the default cookbook example - I tried setting the axis offsets like so:
        double[,] data2D = { { 1, 2, 3 }, { 4, 5, 6 } };
        var testHeatmapPlot = plt3.AddHeatmap(data2D);
        testHeatmapPlot.AxisOffsets = new double[] { -0.5, -2}; // this line

but the setting has no effect on the heatmap location. Even setting some unrealistic numbers such as -100 do not affect the behavior of heatmap plot.
There are 2 use-cases for this functionality

Being able to draw 2 or more heatmaps on the same plot area
bar/candlestick plots by default start drawing bars at location [-0.5; +0.5]. Heatmap is drawing at locations [0; 1]. So if you have 2 plots that correlate to each other - heatmap will always be shifted by 0.5. (p.s. now that I think about it, maybe it's easier to shift bar plots instead, as a temporary workaround :) )

Here is what I want to achieve (2 cases):

Hi @pavlexander,
It sounds like you're trying to display a heatmap that doesn't start at (0, 0)
I think this can be done with the "coordinated heatmap" plot type:
https://swharden.com/scottplot/cookbooks/4.1.16/category/plottable-heatmap/#coordinated-heatmap
The API is a little funny at the moment - it's surprising to have multiple heatmap types, and simplifying this is on the triage list #1028 - but I think this will meet your needs for now. Let me know if this isn't the case! 👍

@swharden thank you for help yet again!
The result:


Add ubuntu 24.04 docker support

Open issues

[ ] Ubuntu 24.04 not supported for nvidia cuda based image
[ ] rosdep not in sources list available....
[ ] Needs to be tested if works

This will be replaced with #169


refactor(semantic): transform checker do not output spans in errors

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5260  👈
#5259
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite


flyTo not working with google maps plugin

I made some minimal updates to the google maps leaflet  plugin to be compatible with leaflet 1 beta, following this PR:
https://github.com/shramov/leaflet-plugins/pull/175
If you select OSM as the baselayer, flyTo works. If a Google baselayer is selected, the flyTo animation does not work. No errors are thrown. I wonder if it's even possible to use flyTo in conjunction with a google layer?
My plunk > http://plnkr.co/edit/ouynceGB0XvjWk7TkWoh?p=preview

This is most likely a problem with the Google Maps Leaflet plugin, not an issue with Leaflet itself. You should create an issue with Leaflet Plugins

This is most likely a problem with the Google Maps Leaflet plugin, not an issue with Leaflet itself. You should create an issue with Leaflet Plugins

This is because leaflet support  non-integer zoom level, but google map don't.
When you try to set a non-integer zoom level, the google layer reset it to the nearest integer.

For ZoomAnimation, google layer listen to 'zoomanim' event and use the google's internal zoom animation.
I think an event is needed for flyTo to implement this.

googlemap zoom -1 for compability with mapbox or leaflet


feat: allow additional browserstack capabitilies to be configured ver…

…batim
resolves https://github.com/karma-runner/karma-browserstack-launcher/issues/66

@dignifiedquire please take a look when you get the chance :)

Any updates on that PR? It would be great to have the possibility to specify the resolution capability on Browserstack.
If there are not enough resources to complete that PR, I'd be happy to take over it.

I will commit the requested changes this weekend.
This project does seem to require a bit of TLC in general (see latest issue on http-proxy-agent) so perhaps we can start actively maintaining it again? Let me know if you're Keen to help :)

Personally, I feel:
 customLaunchers: {
      bs_firefox_mac: {
        base: 'BrowserStack',
        browser: 'firefox',
        browser_version: '21.0',
        os: 'OS X',
        os_version: 'Mountain Lion',
        resolution: '1024x768'
      }

is more intuitive than:
 browserStack: {
     extraCapabilities: {
       resolution: '1024x768'
     }
 }
 customLaunchers: {
      bs_firefox_mac: {
        base: 'BrowserStack',
        browser: 'firefox',
        browser_version: '21.0',
        os: 'OS X',
        os_version: 'Mountain Lion',
      }

The former format provides greater flexibility (according to me), but the latter will result in a DRYed config.
If we go with the latter one, I would prefer using:
 browserStack: {
   resolution: '1024x768'
 }

over
 browserStack: {
   extraCapabilities: {
      resolution: '1024x768'
   }
 }

thoughts ?

Totally agree with that. I don't actually see the point of manually copying the capabilities based on the arguments.
See this comment.
If we just take the arguments and assign the global options then we would have the maximum flexibility and we wouldn't need to maintain each option individually.
e.g. browserSettings = {..globalSettings, ...browserSpecificArgs}

check CLA

Other than those two comments, the API and code looks good to me. But I'm not in the position to give this PR a go 😄

@dignifiedquire when you have a minute please read through the comment thread and review my latest changes

@pulkitsharma07 thanks for adding your review. I've committed my changes :)

@dignifiedquire would you mind glancing over my latest commits please?

Thanks :octocat:


youtube embed added

Please review this PR.

Please merge to pagesource:development

I think we can merge after renaming the component.
@areai51, @sbmadhav @nawazsk :Does it have merit to use a generic Video Embed Component with YT,Vimeo,... options to render ?

The Component has renamed. Please merge this.


Net connection show body diff

Thanks, I'll take a look now

Hmm, I've tried both 2.1.0 and 2.2.1 and was unable to reproduce failed specs locally. I did remove an unnecessary require that I had overlooked in #480 .
How can I help you fix the builds? Does travis take any special steps I might have missed?

There shouldn't be any special setup, but I am very new to contributing to this project, and have just started getting more familiar with the CI setup.
I am going to start digging into the log https://s3.amazonaws.com/archive.travis-ci.org/jobs/62266252/log.txt
And maybe if I recreate using the exact same dependencies, I can get it failing local.
But your guess is as good as mine, as to what the problem is. If I mind anything I'll post it here.

I've created a virtual machine with Ubuntu 12.04 (which is what Travis seems to use) and followed all the steps in the log.
I had to install libcurl4-openssl-dev to get curb to compile its curl extension, but other than that, no issues and no rspec errors either. Same gem versions and the same ruby version as in the log.

Thanks for trying that out, but bummer it didn't help!
I know there are warnings about the addressable dependency, and I see 2 of the builds are failing, because of something involving addressable. So I am going to start exploring that.
There is also a chance there might be intermittent failures on Travis, which I haven't looked into yet.

I was able to reproduce failing specs with Ruby 2.0 - I'll have a closer look this weekend

@yurivm The specs that were failing, did not have anything to do with this feature and are now fixed. So I have rebased this branch of these changes on master, and everything is green on CI now.
I'm now just reviewing everything to make sure I understand all the changes (more to help my own knowledge as I am still becoming familiar with all the areas of WebMock) and we can get this merged in!

@yurivm I took a look at everything and it all looks good.
I wrote some unit tests for RequestBodyDiff and RequestSignatureSnippet, to just have better test coverage and make myself more familiar with the code.
And as a result of this test writing, I ended up refactoring RequestBodyDiff's internals to read a little simpler, and I also restricted RequestSignatureSnippet's API to just #stubbing_instructions and #request_stubs.
I also ma

@yurivm I'm going to ahead and merge this in. Thanks for the feature!


beets: no longer builds

Describe the bug
The beets package no longer builds. I suspect this was caused by some Python dependency bump (cc. @jonringer @FRidh) but I'm not sure which
To Reproduce
Steps to reproduce the behavior:

nix build -f. -L beets

Expected behavior
For beets to build, like it had before
Screenshots
beets> ======================================================================
beets> ERROR: test_config_include_paths_false (test.test_web.WebPluginTest)
beets> ----------------------------------------------------------------------
beets> Traceback (most recent call last):
beets> File "/build/source/test/test_web.py", line 45, in test_config_include_paths_false
beets> response = self.client.get('/item/1')
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 1006, in get
beets> return self.open(*args, **kw)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/testing.py", line 222, in open
beets> return Client.open(
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 970, in open
beets> response = self.run_wsgi_app(environ.copy(), buffered=buffered)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 861, in run_wsgi_app
beets> rv = run_wsgi_app(self.application, environ, buffered=buffered)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 1096, in run_wsgi_app
beets> app_rv = app(environ, start_response)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 2464, in __call__
beets> return self.wsgi_app(environ, start_response)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 2450, in wsgi_app
beets> response = self.handle_exception(e)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 1867, in handle_exception
beets> reraise(exc_type, exc_value, tb)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
beets> raise value
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 2446, in wsgi_app
beets> ctx.push()
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/ctx.py", line 396, in push
beets> self.match_request()
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/ctx.py", line 350, in match_request
beets> result = self.url_adapter.match(return_rule=True)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/routing.py", line 1873, in match
beets> rv = rule.match(path, method)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/routing.py", line 902, in match
beets> new_path = "|".join(self.build(result, False))
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/routing.py", line 1040, in build
beets> return self._build(**values)
beets> File "<werkzeug routing>", line 1, in <builder:'/item/<idlist:ids>'>
beets> File "/build/source/beetsplug/web/__init__.py", line 172, in to_url
beets> return ','.join(value)
beets> TypeError: sequence item 0: expected str instance, int found
beets> -------------------- >> begin captured logging << --------------------                                                                                [297/4479]
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: item_removed
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> --------------------- >> end captured logging << ---------------------
beets> ======================================================================
beets> ERROR: test_config_include_paths_true (test.test_web.WebPluginTest)
beets> ----------------------------------------------------------------------
beets> Traceback (most recent call last):
beets> File "/build/source/test/test_web.py", line 37, in test_config_include_paths_true
beets> response = self.client.get('/item/1')
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 1006, in get
beets> return self.open(*args, **kw)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/testing.py", line 222, in open
beets> return Client.open(
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 970, in open
beets> response = self.run_wsgi_app(environ.copy(), buffered=buffered)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 861, in run_wsgi_app
beets> rv = run_wsgi_app(self.application, environ, buffered=buffered)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 1096, in run_wsgi_app
beets> app_rv = app(environ, start_response)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 2464, in __call__
beets> return self.wsgi_app(environ, start_response)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 2450, in wsgi_app
beets> response = self.handle_exception(e)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 1867, in handle_exception
beets> reraise(exc_type, exc_value, tb)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
beets> raise value
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 2446, in wsgi_app
beets> ctx.push()
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/ctx.py", line 396, in push
beets> self.match_request()
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/ctx.py", line 350, in match_request
beets> result = self.url_adapter.match(return_rule=True)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/routing.py", line 1873, in match
beets> rv = rule.match(path, method)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/routing.py", line 902, in match
beets> new_path = "|".join(self.build(result, False))
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/routing.py", line 1040, in build
beets> return self._build(**values)
beets> File "<werkzeug routing>", line 1, in <builder:'/item/<idlist:ids>'>
beets> File "/build/source/beetsplug/web/__init__.py", line 172, in to_url
beets> return ','.join(value)
beets> TypeError: sequence item 0: expected str instance, int found
beets> -------------------- >> begin captured logging << --------------------
beets> beets: DEBUG: Sending event: database_change                                                                                [242/4479]
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: item_removed
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> --------------------- >> end captured logging << ---------------------
beets> ======================================================================
beets> ERROR: test_get_multiple_albums_by_id (test.test_web.WebPluginTest)
beets> ----------------------------------------------------------------------
beets> Traceback (most recent call last):
beets> File "/build/source/test/test_web.py", line 129, in test_get_multiple_albums_by_id
beets> response = self.client.get('/album/1,2')
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 1006, in get
beets> return self.open(*args, **kw)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/testing.py", line 222, in open
beets> return Client.open(
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 970, in open
beets> response = self.run_wsgi_app(environ.copy(), buffered=buffered)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 861, in run_wsgi_app
beets> rv = run_wsgi_app(self.application, environ, buffered=buffered)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/test.py", line 1096, in run_wsgi_app
beets> app_rv = app(environ, start_response)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 2464, in __call__
beets> return self.wsgi_app(environ, start_response)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 2450, in wsgi_app
beets> response = self.handle_exception(e)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 1867, in handle_exception
beets> reraise(exc_type, exc_value, tb)
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
beets> raise value
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/app.py", line 2446, in wsgi_app
beets> ctx.push()
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/ctx.py", line 396, in push
beets> self.match_request()
beets> File "/nix/store/rcync2hhpzaw3zjg9lzx3pymhgxsrfbk-python3.8-Flask-1.1.2/lib/python3.8/site-packages/flask/ctx.py", line 350, in match_request
beets> result = self.url_adapter.match(return_rule=True)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/routing.py", line 1873, in match
beets> rv = rule.match(path, method)
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/routing.py", line 902, in match
beets> new_path = "|".join(self.build(result, False))
beets> File "/nix/store/2yp5874nib71k55984nsac0hdpbbfh7d-python3.8-Werkzeug-1.0.1/lib/python3.8/site-packages/werkzeug/routing.py", line 1040, in build
beets> return self._build(**values)
beets> File "<werkzeug routing>", line 1, in <builder:'/album/<idlist:ids>'>
beets> File "/build/source/beetsplug/web/__init__.py", line 172, in to_url
beets> return ','.join(value)
beets> TypeError: sequence item 0: expected str instance, int found
beets> -------------------- >> begin captured logging << --------------------
beets> beets: DEBUG: Sending event: database_change                                                                                [188/4479]
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: item_removed
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> beets: DEBUG: Sending event: database_change
beets> --------------------- >> end captured logging << ---------------------

...

Additional context
Add any other context about the problem here.
Notify maintainers

@aszlig @domenkozar @pjones
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.6.15, NixOS, 20.09pre-git (Nightingale)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.6`
 - channels(root): `"nixos-20.09pre215936.93fd4b7f007"`
 - channels(bemeurer): `"home-manager"`
 - channels(cloud): `"home-manager, nixpkgs-unstable-20.09pre218717.05f0934825c"`
 - nixpkgs: `/run/current-system/nixpkgs`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: beets
# a list of nixos modules affected by the problem
module:

https://github.com/NixOS/nixpkgs/issues/91131


build: add basic tsconfig.json

The frontend config now requires a tsconfig for some reason. Probably update that got in too early.
This is just really basic tscript.json from template, to appease the plugin.
No idea why it's now needed, but shouldn't hurt :shrug:

/retest

Well, it breaks builds :thinking: xD


added nitro check to prefix provider and pod controller

Issue #, if available:
Description of changes:

added nitro check to prefix provider

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I'm checking why the unit test failed here, and it looks like it's a connectivity issue with Github. The unit tests are passing locally. I can force update to trigger the check again.


Extra step required for "Install the creator kit solution" Step 1

[Enter feedback here]
When downloading the CreatorKitCore from https://aka.ms/creatorkitdownload, the download happens as a folder with contents and not a zip or compressed file.
New users will need to know that the contents of the folder need to be compressed into a zip file so that step for on page https://docs.microsoft.com/en-us/power-platform/guidance/creator-kit/setup under "Step 1 can be carried out.
Thanks,
Rui

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

ID: 9a869972-ce34-c5c9-c591-fd25347181c1
Version Independent ID: 9a869972-ce34-c5c9-c591-fd25347181c1
Content: Set up the Creator Kit - Power Platform
Content Source: power-platform/guidance/creator-kit/setup.md
Service: power-platform
Sub-service: guidance
GitHub Login: @denisem-msft
Microsoft Alias: demora

Thanks for your comment, @PrimeDime1. This issue has been assigned to the doc author for review.

Hi,
It is throwing me error "power apps Solution "Creator Kit" failed to import: Import Solution Failed: Object reference not set to an instance of an object."
can u please guide

This should be fixed (both issues reported) and download zip is in a state that allows direct import. Please reopen if you're still having issues.


Policy file not found

Describe the bug
I've got a sample license and policy file in a repo (see this current PR), but the action run is saying the repo can not be cloned.  I'm trying to test this in a single repo in our org to validate that we can scan for licensing issues in dependencies, assuming I can get this to work in a single repo then I would likely move the policy file to our org's .github repo and reference that policy file in other repos but for now have the policy file in the same repo where the licensing action is running.  Problem is I'm not sure if I'm referencing the policy file correctly or not.  Also not sure after reading the auth docs if we've got a Personal Access Token set up correctly for the ACCESS_TOKEN in the action.  So I'm hoping you can take a look at our PR and failed action run to recommend any changes to the licensing action, the policy file and its location, or our access token set up.  Once we get it triaged, I'd be happy to help with a PR here to add to the docs for places where I got tripped up.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Talked in https://github.com/10up/insert-special-characters/pull/122, closing issues now


Ribamod: support uncoupled MODFLOW boundaries and Ribasim basins

The spatial extents of the MODFLOW and Ribasim model do not necessarily overlap. This means a part MODFLOW boundaries and a part of the Ribasim basins should not be set.
This requires the creation of mask_ variables, similar to the MetaMod driver, since the value_ptr arrays exposed via bmi/xmi always contain the full set of bounds and basins.
A secondary question is how to "mask" values for Ribasim basins, see #998

Closed by @Huite with https://github.com/Deltares/imod_coupler/pull/243


404 Not Found<EMAIL_ADDRESS>

npm install errors on MacOS 10.14.5 with node 10.15.1.
> node ./installNativePlatform.js

(node:11417) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 SIGINT listeners added. Use emitter.setMaxListeners() to increase limit
npm install --no-save<EMAIL_ADDRESS>/Documents/Hackathons/Infrahack/simple-viewer-app/node_modules/@bentley/imodeljs-native/installNativePlatform.js:40
      throw error;
      ^

Error: Command failed: npm install --no-save<EMAIL_ADDRESS>npm ERR! code E404
npm ERR! 404 Not Found<EMAIL_ADDRESS>
npm ERR! A complete log of this run can be found in:
npm ERR!     /.npm/_logs/2019-05-17T08_55_48_793Z-debug.log

    at ChildProcess.exithandler (child_process.js:294:12)
    at ChildProcess.emit (events.js:189:13)
    at maybeClose (internal/child_process.js:970:16)
    at Socket.stream.socket.on (internal/child_process.js:389:11)
    at Socket.emit (events.js:189:13)
    at Pipe._handle.close (net.js:600:12)

Hi @alexander7161,
We don't currently publish a MacOS version of the native code, it's currently only Windows and Linux.  We've been focusing on those two for our initial 1.0 release but Mac support will coming in future updates so stay tuned for more updates...
However, the error message you received could be better and we will work on it clearer that Mac is not yet supported.


Highlight just played tiles?

It'd be nice if the just played tiles had a border around them.
Maybe even each players tiles with their own colour?

Hi,
I use this to play with my mom, and she had the same suggestion.
The game would flow much better if the just played tiles were somehow differentiated from the rest. Maybe just a background colour change would suffice.
Ideally, this could also be combined with an audio cue when someone has finished their play, possibly with a different one when it is your turn #2.
Thanks for the excellent work and for giving me and my family some joy these days!

Merged in fd69a0159cc0501f6d3a9f5cbd25dbdcf56854ba, with more commentary in the #6 thread.


DNS lists unique casing only

created using sort -fu
In case the lists were in a specific order on purpose, don't merge.
fix #553
Either this or #555 (or none).
git diff --stat
 Discovery/DNS/bitquark-subdomains-top100000.txt                | 199912 ++++++++++++------------
 Discovery/DNS/deepmagic.com-prefixes-top500.txt                |    918 +-
 Discovery/DNS/deepmagic.com-prefixes-top50000.txt              |  99892 ++++++------
 Discovery/DNS/dns-Jhaddix.txt                                  | 809760 ++++++++++++++++++++++++++++++++++++++++++++++++-------------------------------------------------
 Discovery/DNS/fierce-hostlist.txt                              |     12 +-
 Discovery/DNS/namelist.txt                                     |      2 +-
 Discovery/DNS/shubs-stackoverflow.txt                          | 129133 ++++++++--------
 Discovery/DNS/shubs-subdomains.txt                             | 968427 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------------------------
 Discovery/DNS/sortedcombined-knock-dnsrecon-fierce-reconng.txt |   7805 +-
 Discovery/DNS/subdomains-top1million-110000.txt                | 228391 ++++++++++++++--------------
 Discovery/DNS/subdomains-top1million-20000.txt                 |  35953 +++--
 Discovery/DNS/subdomains-top1million-5000.txt                  |   9810 +-
 12 files changed, 1241327 insertions(+), 1248688 deletions(-)

Rather not have the orders altered (as it can be by frequency).
Same dups removed as #555 ?


Add support to private module in graph extension

All elements in private module will be draw onto
the result DOT file. Note: the interface that used
in higher level and defined in lower level will be
shown twice in the generated graph.

Hi, I've signed the CLA. Thanks.


Different colour icon for menu bar?

Great little app. Would it be possible to offer an option to change the colour of the icon? As you can see here:  it is a bit dark

.

Totally agreed for this, not friendly for dark mode

Great little app. Would it be possible to offer an option to change the colour of the icon? As you can see here: it is a bit dark

.

Can you write some information:

Macbook model and year
MacOS version

Hi,
I think it is working correctly..
My apologies
Roland

On Oct 10, 2022, at 5:29 AM, Alexey Tarutin @.***> wrote:
Great little app. Would it be possible to offer an option to change the colour of the icon? As you can see here: it is a bit dark
https://user-images.githubusercontent.com/49558872/184690232-e0693a6e-1cf5-44a2-94f8-c0fa49df7e57.jpg
.
Can you write some information:
Macbook model and year
MacOS version
—
Reply to this email directly, view it on GitHub https://github.com/tarutin/hovrly/issues/2#issuecomment-1273241238, or unsubscribe https://github.com/notifications/unsubscribe-auth/AL2DKWD7IUMFFLBCD7ULJ3LWCQD2XANCNFSM56TADBZA.
You are receiving this because you were mentioned.

@rvdh70 Hm. Interesting. Why is that problem resolved itself? Maybe you installed new version of MacOS?
@jona-hsueh Do you still have this problem?

Thank you all! I`ll testing Auto Theme

I think this issue has to do with macos automatically changing the color of the icons depending on the color of the wallpaper in the background. I am having this issue as well.

@kunwarsahni01 Correct, moreover this happens when you have the 'Dynamic' option activated this changes the colours of the wallpaper (darker or lighter) irregardless of Dark Mode/Light Mode

I am actually using custom wallpapers so no dynamic wallpaper in this scenario.

Fixed at 2.4.4


M1 MacBooks Login Screen Not Centered

The login screen does not fill the entire screen and is displaying in the upper-left corner. Once logged in, if you log out, it displays centered and fills the screen. Every time you reboot, it goes back to the upper-left corner and not filling the screen. I have attached screenshot.

.

Any ETA on when this might get fixed? My Swift skills aren't that good, but this does not sound like something that should be hard to fix?

I have this issue too when an external display is connected.

@macshome - do issues opened here get looked at or is there some other way they're tracked?

I do too have the same issue on MacBook Pro M1, haven't tested with others but most of my devices we manage are M1 and above. I may have to dig around in swift coding, I am not good at this either.

Happens regardless of external display status for us.

I can fix it "permanently" on a device if I login as an admin user while connected to the external display and set the external display to something like 1920x1080 and/or turn off display mirroring while logged in as the admin user.

Having the same problem here with M1 Macbooks and NoLoAD 1.4 on Monterey.
Slightly off-topic:

 .

@sabrown13 how did you manage to display the serialnumber below? We had to abuse the DisplayUsernameFieldPlaceholder because we saw no other solution, but your solution looks neater. Care to elaborate?

Not a problem...I used iMazing Profile Editor app and changed the Default
System Information drop down selection from IP to Serial under the Design
category.
Thanks,
Sarah
Sarah Brown
Mobility Specialist | Apple Certified Macintosh Technician
Pulaski Community School District
Phone:<PHONE_NUMBER>  |  Fax:<PHONE_NUMBER>  |  Email:
@.***
Website http://www.pulaskischools.org/  |  Facebook
http://www.facebook.com/pulaskischools  |  Twitter
http://www.twitter.com/pulaskischools
On Tue, Oct 4, 2022 at 10:06 AM Simon Müller @.***>
wrote:

Having the same problem here with M1 Macbooks and NoLoAD 1.4 on Monterey.
Slightly off-topic:
[image: IMG_6575]
https://user-images.githubusercontent.com/100800838/156439006-670baf6e-7a7d-4b0e-9eb5-cbf5ee24231a.jpg
.
@sabrown13 https://github.com/sabrown13 how did you manage to display
the serialnumber below? We had to abuse the DisplayUsernameFieldPlaceholder
because we saw no other solution, but your solution looks neater. Care to
elaborate?
—
Reply to this email directly, view it on GitHub
https://github.com/jamf/NoMADLogin-AD/issues/1#issuecomment-1267151511,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYBBSRUMOAYFKSL7BCTWWFTWBRBYBANCNFSM5PYODFRA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
The district does not discriminate in admissions to any school, class,
program or activity based on the traits of sex (including transgender
status, change of sex, or gender identity), race, color, national origin,
religion, creed, ancestry, marital or parental status, sexual orientation
or physical, mental, emotional or learning disability, or any other
characteristic protected by Federal or state civil rights laws.

Any updates on this? Same issue for us.


Feature Request: use function name as sub-system nane

it would be cool, if the extension would auto-configure the CORALOGIX_SUB_SYSTEM form function name.
the current behaviour uses logs as default sub system, which is a bit poor.

It was an initial idea to use lambda name as default subsystem name. But lambda extension API doesn't provide mechanism to retreive lambda function name. So, it's impossible to implement this.


Bug: "wrong" PHP 8.3 version installed when setup-php used twice in workflow

Describe the bug
When installing PHP, setup-php will always install the latest released version of specific PHP minor.
The bug I'm currently seeing is one where this is not happening when setup-php is used twice in a workflow and the second time PHP 8.3 is being installed.
I've done some investigating and it looks like this issue only occurs with PHP 8.3.
This is a table with my findings (can be verified via the GH Actions logs link posted below):

PHP
Single install
Double install

7.2
7.2.34
7.2.34

7.3
7.3.33
7.3.33

7.4
7.4.33
7.4.33

8.0
8.0.30
8.0.30

8.1
8.1.31
8.1.31

8.2
8.2.26
8.2.26

8.3
8.3.14
8.3.6 :point_left

8.4
8.4.1
8.4.1

Version

[X] v2
[ ] v1

Runners
GitHub Hosted
Operating systems
ubuntu-latest
PHP versions
8.3
To Reproduce
I've testing the issue in isolation - a demo of the issue can be found here:

Workflow: https://github.com/jrfnl/bug-report-reproduction-scenarios/blob/setup-php/php-version-issue/.github/workflows/test.yml
Test run: https://github.com/jrfnl/bug-report-reproduction-scenarios/actions/runs/12393747114/job/34595664865

Expected behavior
I expected PHP 8.3.14 to be installed, but I got PHP 8.3.6 instead.
Screenshots/Logs

Additional context
Would love to submit a PR, but I searched the code base and couldn't find any direct clue as to why this is happening.
Are you willing to submit a PR?
No

@jrfnl
Please test if this happens even if you set update env to true in the setup-php step.

I have identified why this happens.
We have a logic to force update if the ondrej/php PPA is not present on the runner (ubuntu-22.04 and ubuntu-24.04). So when it runs for the first time with PHP 7.4, it adds that PPA and on the next run, now that it has the PPA update is not set to true.
I will try to fix this in the next release that we maintain the update logic on subsequent runs in the same job, but the workaround to specify update: true env should help in avoiding this all together.

@shivammathur Thank you - as always - for your super-fast response.
I've added a second commit to the reproduction branch and can confirm that adding the update env variable fixes the issue.
The thing I find weird about it, is that the issue only occured with PHP 8.3, not with any of the other PHP versions ?
I very much appreciate that you are looking into the root cause of this, but yes, for my specific usecase, the update env will fix this.

ubuntu-24.04 has php 8.3.6 preinstalled.
https://packages.ubuntu.com/noble/php8.3

Fixed in https://github.com/shivammathur/setup-php/commit/cdfde5ebec2b1d6902daaeee939e77a6560409ff. Will be in the next release.

Released in 2.32.0.


Use of Deprecated componentWillMount Method

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
I get this warning any time I use <Helmet>:
Warning: componentWillMount has been renamed, and is not recommended for use. See https://fb.me/react-unsafe-component-lifecycles for details.

* Move code with side effects to componentDidMount, and set initial state in the constructor.
* Rename componentWillMount to UNSAFE_componentWillMount to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run `npx react-codemod rename-unsafe-lifecycles` in your project source folder.

Please update the following components: SideEffect(NullComponent)

I assure this has nothing to do with my own code. Without react-helmet, the console has nothing to complain about. So I believe that react-helmet needs to be updated to not use componentWillMount.
If the current behavior is a bug,
please provide the steps to reproduce and if
possible a minimal demo of the problem.
Your bug will get fixed much faster if we can run your
code and it doesn't have dependencies other than React and react-helmet.
Paste the link to your JSFiddle (https://jsfiddle.net/Luktwrdm/) or
CodeSandbox (https://codesandbox.io/s/new) example below:
Run this demo and check the console for a warning:
https://codesandbox.io/s/epic-sun-28y0e
What is the expected behavior?
See the warning above that pops up in the console.
Which versions of React and react-helmet, and which browser / OS are affected by this issue?

React 16.10.2
react-helmet 5.2.1
Safari 12.1
macOS 10.14.4

Did this work in previous versions of React and/or react-helmet?
Not sure; this is the first time I’m working with react-helmet.

Duplicate of #426


Optimize image_templates after goreleaser#1969

After https://github.com/goreleaser/goreleaser/issues/1969 is now fixed, we can optimize the image_template setting which doesn't list duplicate image tags in the Prereleases anymore.
BUT we have to wait until Goreleaser and Goreleaser Action are bumped with the latest release
Checklist

[x] Keep pull requests small so they can be easily reviewed.

Coincidentally, Goreleaser has just been released :tada:

I dry-ran the release locally with latest Goreleaser. This should work as expected.

Makes sense to handle it this way. As I don't really know goreleaser I can't really comment on the implementation thoug.


readme updation

Let me know if this needs to be done

What updation are you suggesting?

What updation are you suggesting?


Shitf+Cmd+Left is broken

Shitf+Cmd+Left is supposed to make a selection from the cursor to the beginning of the line on OSX.

Could not reproduce...


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 97 ms

Resolved: ClamAV performance has improved in 57cfbf5.


[#5156] Restore the previous SpanEvent

Codecov Report

Merging #5214 into master will increase coverage by 0.01%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #5214      +/-   ##
==========================================
+ Coverage   40.48%   40.49%   +0.01%
==========================================
  Files        2667     2667
  Lines       80143    80147       +4
  Branches    10692    10692
==========================================
+ Hits        32442    32456      +14
+ Misses      44803    44793      -10
  Partials     2898     2898

Impacted Files
Coverage Δ

...orp/pinpoint/profiler/context/AsyncChildTrace.java
0% <0%> (ø)
:arrow_up:

...ercorp/pinpoint/profiler/context/DefaultTrace.java
54.47% <100%> (+0.75%)
:arrow_up:

...p/pinpoint/rpc/client/WriteFailFutureListener.java
56.25% <0%> (-18.75%)
:arrow_down:

.../profiler/sender/StandbySpanStreamDataStorage.java
72.88% <0%> (-3.39%)
:arrow_down:

...pinpoint/profiler/monitor/DeadlockMonitorTask.java
18.6% <0%> (-1.17%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
79.86% <0%> (+0.34%)
:arrow_up:

...m/navercorp/pinpoint/profiler/AgentInfoSender.java
76.57% <0%> (+0.9%)
:arrow_up:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
85.36% <0%> (+2.43%)
:arrow_up:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
83.58% <0%> (+2.98%)
:arrow_up:

...ler/context/recorder/WrappedSpanEventRecorder.java
30.68% <0%> (+4.54%)
:arrow_up:

... and 2 more

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 3fce467...f2da124. Read the comment docs.


Update to API version 10

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[x] Internal code
[x] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
This implements the required changes for version 10.
See Changelog
TODO: Check behavior of attachments in all message edit endpoints. This wasn't mentioned in the changelog but was supposed to change in v10.

I've added a warning for the new content intent to raise awareness. Whenever you use message.getContentRaw() (and other related methods) without this intent, you will now receive this warning:
Attempting to access message content without GatewayIntent.MESSAGE_CONTENT.
Discord now requires to explicitly enable access to this using the MESSAGE_CONTENT intent.
Useful resources to learn more:
	- https://support-dev.discord.com/hc/en-us/articles/4404772028055-Message-Content-Privileged-Intent-FAQ
	- https://jda.wiki/using-jda/gateway-intents-and-member-cache-policy/
	- https://jda.wiki/using-jda/troubleshooting/#im-getting-closecode4014-disallowed-intents
Or suppress this warning if this is intentional with Message.suppressContentIntentWarning()

Converting back to draft for now because of the breaking changes to the attachments field in v10, which we still have to handle in the message edit rewrite.


ID2D1Bitmap1.Map() only returns 1 row of pixel data

Hi @smourier,
As my post from https://github.com/smourier/DirectN/issues/34, I modified the function a bit to load all pixel data into a DataGridView.
public void LoadPixelDataIntoDataGrid()
{
    var bmpProps = new D2D1_BITMAP_PROPERTIES1()
    {
        bitmapOptions = D2D1_BITMAP_OPTIONS.D2D1_BITMAP_OPTIONS_CANNOT_DRAW | D2D1_BITMAP_OPTIONS.D2D1_BITMAP_OPTIONS_CPU_READ,
        pixelFormat = new D2D1_PIXEL_FORMAT()
        {
            alphaMode = D2D1_ALPHA_MODE.D2D1_ALPHA_MODE_PREMULTIPLIED,
            format = DXGI_FORMAT.DXGI_FORMAT_B8G8R8A8_UNORM,
        },
        dpiX = 96.0f,
        dpiY = 96.0f,
    };

    var size = _imageD2D.GetSize().ToD2D_SIZE_U();
    using var bitmap1 = _device.CreateBitmap<ID2D1Bitmap1>(size, bmpProps);
    bitmap1.CopyFromBitmap(_imageD2D);

    var map = bitmap1.Map(D2D1_MAP_OPTIONS.D2D1_MAP_OPTIONS_READ);
    var dataSize = (int)(size.width * size.height * 4);

    var bytes = new byte[dataSize];
    Marshal.Copy(map.bits, bytes, 0, dataSize);
    bitmap1.Unmap();

    // load all pixel data into data grid view
    dt.DataSource = bytes.Select((i, index) => new
    {
        Index = index,
        Value = i.ToString(),
    }).ToList();
}

The problem
It only returns the first row of the pixel data; the rest values are just 0!
In the demo, I used 4x4 PNG image with alpha pixels, and draw 100X onto the window.

Full source code:
WinFormsApp1.zip

I managed to fix it.
- var dataSize = (int)(size.width * size.height * 4);
+ var dataSize = (int)(size.height * map.pitch);

According to https://learn.microsoft.com/en-us/windows/win32/api/d2d1_1/ns-d2d1_1-d2d1_mapped_rect, pitch is the size in bytes of an individual scanline in the bitmap. So the dataSize should be calculated by image height times pitch.

Yes, this is also called the stride, it depends on the bitmap layout in memory, you never want to assume things like "4" https://learn.microsoft.com/en-us/windows/win32/medfound/image-stride


Upgrades and split into three OS versions

@DieterHolz you're fast ;-)

Shouldn't the crowpi example project be included in https://github.com/Pi4J/pi4j-os/tree/upgrade-jdk18-jfx18/crowpi/resources ?

Shouldn't the crowpi example project be included in https://github.com/Pi4J/pi4j-os/tree/upgrade-jdk18-jfx18/crowpi/resources ?

Yes, of course.
Will fix it.

Now, after double-checking CrowPiOS, I'm not really sure what you mean with "Shouldn't the crowpi example project be included".
First, I assumed that "HelloFX" and "MinimalPi4J" is missing. But they are included.

This one: https://github.com/Pi4J/pi4j-example-crowpi

My previous opinion was that we should push our students towards using their laptops as developer machine and the Pi for running the apps only.
Therefore I would not include the crowpi-example project in CrowPiOS.
If you really want to use it directly on Pi, it's only a single git clone away.
The READMEs of cropwpi-examples and Pi4J-OS needs to be adjusted, accordingly.
But maybe it's time to do things differently ....

Therefore I would not include the crowpi-example project in CrowPiOS.

valid point!


Removing the ability to listen to a secure port ('vts' port).

This is generally unused, and we'll use encrypted gRPC if needed.
@sougou @enisoc @nurblieh

LGTM

Very nice. I think we can also get rid of the port map. It should just be "port".

You mean with the gRPC port being on a different port and all? ;)

Are you sure? I think gRPC can use the http port. The protocol is supposed to be established in the handshake, or may be URL.

gRPC uses HTTP 2 streaming protocol, needs an HTTP 2 server, and the go server is not there yet. So it has to be some custom code for now until it's fine.


DISCUSSION: Enforce user-friendly data in datastore

This is two-fold:

Keep users from doing things which are allowed but may come back to bit them. For example, spaces in field names, negative dataset IDs

Document code paths which are allowed by our library but will fail on the backend. (Discussions in #443 and #529 need to be incorporated).

The second point is still debatable:
Is it better to add lots of conditionals and (IMO brittle) validity checks? Or, is the pragmatic approach (less code, let backend indicate failure) better?
/cc @jgeewax

We should probably document the two kinds of cases (stuff the backend would allow, but which can't be spelled with our APIs, and stuff which our APIs would allow, but which the backend will reject) in a single place.

I personally like the idea of letting the back-end tell us when stuff is broken rather than lots of client-side validity checks. However I would like to make our stance clear on how we deal with bugs like ".filter() doesnt let me filter on properties that start with spaces" (in that case, I think our reply should be: "we're not going to change our API to support that disastrous edge case").
Is this ticket a good place to collect the list of things?

@jgeewax This is a good place to collect.
RE: .filter(), as of #430, this is not something we need to actively worry about, though we could discourage the use of spaces by checking the property name. I am (moderately) against unnecessary branching (but aware Python is slow anyway).

I think we may have done everything we can do here... Thoughts?

Well we haven't document anything and there are at least 6 bugs which ended up referring to this issue due to user confusion with the backend (i.e. rejecting an Entity that was indexed, they can never be).


VI mode <M-...> bindings

I would like to have VI insert mode bindings for <M-...> where by <M-> I don't mean traditional 8bit meta but escape prefixed sequence. E.g., <M-h> would be ^]h that is Escape + h.
So I did:
registry.add_binding(Keys.Escape, 'h', filter=insert_mode)(...)

Now that works ok, but it seems to me prompt_toolkit doesn't have a notion of timeout for when there are multiple bindings for so far ambiguous input, i.e., with above in place I can't go to command mode just by pressing Escape. And even if there is no conflicting binding, there is delay (actually as I've figured out it's the eventloop's INPUT_TIMEOUT of 1s by default).
I think we need at least one timeout (see keyseq-timeout in readline) that can be set to something relatively low, in my case I set it to 10ms. You might also have a look how vim does it, it has ttimeoutlen for keycodes, and then separate timeoutlen for mappings.

Hi @mkoskar,
Thanks for opening this issue! This is interesting.
I didn't know about ttimeoutlen. On my system it's 100ms by default. Prompt_toolkit 2.0 (see 2.0 branch) uses 500ms. Maybe we can decrease this. Maybe 200ms or 300ms, but not too much, because Python can be a little slower than C, and we don't want to call it too early.
Further, prompt_toolkit indeed doesn't support timeoutlen yet. I had no idea that Vim has this option. In my setup it's 1000ms for this one. I think we can do the same thing here.
This feature request makes sense. I can probably implement this in the 2.0 branch.
Jonathan

Hi @mkoskar,
Here you go: https://github.com/jonathanslenders/python-prompt-toolkit/commit/faa202ffc57ba0e43aa32e6541427be57cbaa923
It's on the 2.0 branch. Hope that works for you.
FYI: we would like to release 2.0 in the coming months.

Closing this, given that it's implemented. Both timeoutlen and ttimeoutlen are now attributes of the Application object.


How to do to install

When I try to install from : Extensions; Install Add On from File; I have the error: the add on is incompatible with this version.
Thanks a lot
Marco

There is not yet a version that is compatible with Zotero 5.

+1 for Zotero 5 support.

+1 Great add on I used also some years ago, now there seems to be no good alternative to it. Will appreciate an update, thanks!

:robot: this is your friendly neighborhood build bot announcing test build 0.6.61.emile.Emilianos-MacBook-Air.local (undefined).

:robot: this is your friendly neighborhood build bot announcing test build 5.0.0.emile.Emilianos-MacBook-Air.local (undefined).

:robot: this is your friendly neighborhood build bot announcing test build 5.0.0.emile.Emilianos-MacBook-Air.local (undefined).

:robot: this is your friendly neighborhood build bot announcing test build <IP_ADDRESS> ("what now circle?!").

<IP_ADDRESS> should work, but is untested; I don't have my dev system at hand for another week.

Can confirm that it worked on my Zotero 5.0.54 with no noticeable problems. Thanks a lot for quick reaction!!

Thanks for the confirmation, I've scheduled 5.0.1 for release.


Documentation doesn't show how to check the progress of function creation in Azure Portal

Hello,
The documentation says "It may take several minutes before the image has loaded. You can monitor progress using the Azure Portal", but it does not show you how to monitor progress in the portal. Where can I check this information?
I followed the tutorial until the end and when I put the file in the input directory, nothing happens. Also, I can't see the created function.

Is it possible to update the tutorial with more details?
Best regards,
Cristina

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

ID: a5111706-3d4f-f436-5f72-49b879c8d3fa
Version Independent ID: a34c079f-a4e6-219d-1fb5-56ee900c2a58
Content: Deploy ml models to Azure Functions Apps (preview) - Azure Machine Learning
Content Source: articles/machine-learning/how-to-deploy-functions.md
Service: machine-learning
Sub-service: core
GitHub Login: @vaidya-s
Microsoft Alias: vaidyas

@crisansou we'll review your feedback and get back to you shortly. Thanks.

@vaidya-s I tried above steps and observed same issue as above feedback. Can you please review and updates steps accordingly? Thanks.

Hi,
https://docs.microsoft.com/en-us/python/api/azureml-contrib-functions/azureml.contrib.functions?view=azure-ml-py#package-workspace--models--inference-config--generate-dockerfile-false--functions-enabled-false--trigger-none----kwargs-
package(workspace, models, inference_config, generate_dockerfile=False, functions_enabled=False, trigger=None, **kwargs)

I just want to make sure you have set functions_enabled=True
By default this value is false

Hi @shivanissambare ,
Yes, I set the parameter functions_enabled to True as instructed in the tutorial.
https://docs.microsoft.com/en-us/azure/machine-learning/how-to-deploy-functions#create-the-image

Hi,
https://docs.microsoft.com/en-us/python/api/azureml-contrib-functions/azureml.contrib.functions?view=azure-ml-py#package-workspace--models--inference-config--generate-dockerfile-false--functions-enabled-false--trigger-none----kwargs-
package(workspace, models, inference_config, generate_dockerfile=False, functions_enabled=False, trigger=None, **kwargs)

I just want to make sure you have set functions_enabled=True
By default this value is false

This might be a bug on our side then, this is a preview functionality.
May I know your use case for using functions?
I'd recommend to use Managed Endpoints
Azure machine learning launched managed endpoints, please take a look :

https://docs.microsoft.com/en-us/azure/machine-learning/concept-endpoints
https://docs.microsoft.com/en-us/azure/machine-learning/how-to-deploy-managed-online-endpoints

Hi @shivanissambare,
Thanks for the updates. For now I'm just testing the existing options for deploying a regression model.

This might be a bug on our side then, this is a preview functionality.
May I know your use case for using functions?
I'd recommend to use Managed Endpoints
Azure machine learning launched managed endpoints, please take a look :

https://docs.microsoft.com/en-us/azure/machine-learning/concept-endpoints
https://docs.microsoft.com/en-us/azure/machine-learning/how-to-deploy-managed-online-endpoints

got it, thanks!
Yup, our recommendation would be to try Managed Endpoints.

#please-close


feat: add zinter command

Hi there,
This PR adds the zinter command which was missing. I tried to follow the same patterns I saw used for the other commands. Fun fact I learned today - sinter also works with sorted sets, but there's not WITHSCORES options which is what I needed.
Cheers

Hi @uki00a
Thanks for the review! I ran make_mod and added the signature you requested. Looks like my editor also removed some trailing commas, I'll revert those now.
Is there any way to ensure consistency here - do you use any kind of setup here that I should be aware of?

@ClaudiuCeia Thanks for the making changes! We run deno fmt --check in CI to keep the code style consistent. Could you run deno fmt locally on the changed files?

@ClaudiuCeia Thank you for the contribution! I'll release this later


updata1.20

please updata1.20 thx

Purpur now uses 1.20 as their main version (1.19.4 is discontinued)


Plot circuit layout test

The PR #2928 showed that plot_circuit_layout was not tested. This PR adds that test.
This PR goes on top of #2928

Looks like your having fun with #2855 type issues as well.

After some debugging (tough task to do in the CI), it seems that, indeed, the CI is generating a different graph:

Reference image:

I'll have a look.

Moving the issue to #2945.


php7 - Empty Trash

Trying to empty the trash with php 7 throws this error:
{"error":{"type":"Whoops\\Exception\\ErrorException","message":"Cannot use 'Null' as class name as it is reserved","file":"\/concrete5\/5.7\/concrete\/vendor\/zendframework\/zend-queue\/library\/ZendQueue\/Adapter\/Null.php","line":24,"trace":[{"file":"\/concrete5\/5.7\/concrete\/vendor\/zendframework\/zend-queue\/library\/ZendQueue\/Adapter\/Null.php","line":24,"function":null,"class":null,"args":[]}]},"errors":["Cannot use 'Null' as class name as it is reserved"]}

This is probably related to https://github.com/concrete5/concrete5/issues/2812

hhmm.. then maybe you guys should consider altering this page accordingly:
http://documentation.concrete5.org/developers/installation/system-requirements

@Remo I think this is a dead end: https://github.com/zendframework/ZendQueue/issues/25

@fmaruna maybe you should have a look into this?

v8b2: this is still a thing.

@krebbi I think you should remove your vendor directory and do a composer install again.

installed the beta2 from the website..
nevertheless i get the same result with composer install.

@krebbi ah, gotcha. Yes that PR isn't merged I see (#4213). I think you could manually change the composer.json and composer.lock files from that PR and then run the composer install again.

changed my composer.json accordingly, but still the the same error.

It will install based on your composer.lock file, so you have to change that file as well.

that works. thx!
but have you noticed that deleted pages don't appear und "Trash" in the sitemap? is that intensionally?


Bigtable Support

Requirement - what kind of business use case are you trying to solve?
Implementing a new storage backend for GCP's Bigtable
Problem - what in Jaeger blocks you from solving the requirement?
Probably issue #422
Proposal - what do you suggest to solve the problem or improve the existing situation?
Implementing and maintaining the new backend plugin
Any open questions to address
Does it have any chance to be merged? Is #422 preventing any backend from being added?
This issue could be added to #638

Hi @olivierboucher

Does it have any chance to be merged?

The maintainers have limited resources to maintain and enhance storage implementations, so our strategy is to stick with the two supported open source solutions (Elasticsearch and Casandra), and enable other implementations to be done as community-supported plugins. We would be fine to host those plugins under the jaegertracing org, e.g. jaegertracing/storage-bigtable repository, where the original implementors will have maintainer rights.

Is #422 preventing any backend from being added?

Unfortunately, yes, we have been short-handed to move on that issue, even though the amount of work is not large, if we use the harshicorp plugin & the new protobuf model. If you would like to contribute the basic plugin framework for storage, it will enable all those listed in #638 to proceed. I am happy to provide guidance & reviews about that.

Hi @yurishkuro
This sounds like an interesting project. Should PRs be based from master?

Yes, we base everything off master. If you're interested, I would recommend building a prototype with harshicorp plugin and the existing in-memory storage, just to demonstrate the plumbing (and maybe some comparative perf tests of IPC-based plugin vs. in-process plugin).

@douglas-reid an interesting project for istio perhaps?

There has been no activity here, I assume there is not enough interest, so closing as wontfix.


fix(docs-playground): dynamic live font styles

📝 Description
The theme of sandpack makes DynamicLive applied $sm font-size and line-height, which is the font-size and line-height of LiveCode.
As the live preview is not the $base font-size and $md line-height, it causes some content shift.
https://github.com/nextui-org/nextui/blob/66153ecb82e4f4b838d41763e4f83c6f8ac95c2e/apps/docs/src/components/sandpack/themes.ts#L29-L34
⛳ Current behavior (updates)
Reapply the right font-size and line-height to InView.

💣 Is this a breaking change (Yes/No):
No
📝 Additional Information
Before:
before.webm
After:
after.webm

Codecov Report

Merging #962 (4486799) into main (66153ec) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #962   +/-   ##
=======================================
  Coverage   75.85%   75.85%
=======================================
  Files         223      223
  Lines        3528     3528
  Branches     1116     1116
=======================================
  Hits         2676     2676
  Misses        841      841
  Partials       11       11

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


'Player Fade In' field for SEGSAdmin should be given default value of 380.0 to avoid error message

Steps to reproduce behavior/symptoms

Run SEGSAdmin
Start Authserver
Should produce: Critical: Badly formed float for 'player_fade_in':  ""

SEGS version
Latest master
Operating system
Windows 10
As I understand it, leaving the field blank will just make the server default to 380.0, but it would be nice to avoid any panic the user may have from receiving an error message and not knowing what value to put in.

Thanks James! Good spot.
@broxen @nemerle Thoughts? If I restrict input to this field to a float value does this resolve the issue?
Thanks,
Miles

Let's give it a default value (380.0) and restrict to float.

In fact we should ensure that all fields have a default value (max_char_slots = 8, etc)

I will provide a default value and restrict the field to a float value.
@broxen Is there any scenario where any of the settings.cfg fields can be null? If not, the easier solution would be just to ensure no field is blank when saving settings within SEGSAdmin.

Not at this time, no. If for some reason those values are NULL or blank, the server will populate them with a default value and spit out an error. IMO SEGSAdmin should just fill in the defaults if nothing else has been provided.
Those defaults can be found in the code server side, probably inside ReadConfig functions. Sorry I'm on mobile and don't have the full method names at this time.

This is fixed locally, reset to defaults function now exists and will default every field as per the originally created settings.cfg that SEGSAdmin produces. I've also ensured that when writing changes to settings.cfg player_fade_in is correctly formed "n"+".0"


compiler bug (regression in pxt)

The following program gives an error message in makecode.microbit.org:
error: main.ts(8,9): error TS9210: Cannot compute enum value
for the last line (Bar.Delete)
enum Foo {
    First,
    Last,
}

enum Bar{ Last=Foo.Last, Delete };

let x = Bar.Delete;

However, in beta, we get a compiler failure on the same line
error: main.ts(8,5): error TS9200: Cannot read property 'pxt' of undefined

Here is the code that created the error message (master). Looks like failure is probably happening in constantFoldDecl
        function emitPropertyAccess(node: PropertyAccessExpression): ir.Expr {
            let decl = getDecl(node);

            const fold = constantFoldDecl(decl)
            if (fold)
                return emitLit(fold.val)

            if (decl.kind == SK.SetAccessor)
                decl = checkGetter(decl)

            if (decl.kind == SK.GetAccessor)
                return emitCallCore(node, node, [], null, decl as GetAccessorDeclaration)

            if (decl.kind == SK.EnumMember) {
                throw userError(9210, lf("Cannot compute enum value"))

More debugging. en.initializer is undefined because of Last=Foo.Last:
      function constantFoldDecl(decl: Declaration) {
            if (!decl)
                return null

            const info = pxtInfo(decl)
            if (info.constantFolded)
                return info.constantFolded

            if (isVar(decl) && (decl.parent.flags & NodeFlags.Const)) {
                const vardecl = decl as VariableDeclaration
                if (vardecl.initializer)
                    info.constantFolded = constantFold(vardecl.initializer)
            } else if (decl.kind == SK.EnumMember) {
                const en = decl as EnumMember
                const ev = enumValue(en);
                if (ev == null) {
                    console.log(en.initializer);
                    info.constantFolded = constantFold(en.initializer)

As, a result, we call pxtNode with n==undefined, leading to the dereferencing error:
    export function pxtInfo(n: Node) {
        if (!n.pxt) {

Fixed


Upgraded ameba to 0.9.2. & fixed remaining Performance/FirstLastAfterFilter

Fixed remaining Performance/FirstLastAfter (#269)
Related Issues

https://github.com/SushiChain/SushiChain/issues/269

awesome work :)


Object types such as the Knex.Config are not exported for importing types into application

I'd like to create some wrapper functions for my applications around the Knex initializer in TypeScript. The Knex.Config interface is not reference-able. Is it possible to make it reference-able?
I'd like to be able to do something like
import Knex from 'knex';

function wrapper(config: Knex.Config) {
    return Knex(config)
}

How is this related to objection?


Brighten colour for survey mortgages

Currently the colour is a bit too dark.

Survey colours are moved to survey code


Put version to the generated executabe jar name

Description:
Currently generated executable jar name is moduleName-executable.jar it will be useful to append the version number as well. So the new  executable jar name would be moduleName-executable-1.0.0.jar

what about "moduleName-1.0.0-executable.jar" ?


RUMM-1738: Target Android 12

What does this PR do?
This change brings support of Android 12 as it was done in the main SDK repo before.
DO NOT MERGE for now, because this change requires Android 12 compatible main SDK, which is not released yet.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #45 (dd1dfe3) into main (1ac6e98) will decrease coverage by 1.48%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main      #45      +/-   ##
============================================
- Coverage     84.87%   83.39%   -1.48%
  Complexity      117      117
============================================
  Files             9        9
  Lines           271      271
  Branches         33       33
============================================
- Hits            230      226       -4
- Misses           30       34       +4
  Partials         11       11

Impacted Files
Coverage Δ

...n/com/datadog/android/bridge/internal/BridgeRum.kt
75.31% <0.00%> (-3.70%)
:arrow_down:

...n/com/datadog/android/bridge/internal/BridgeSdk.kt
92.55% <0.00%> (-1.06%)
:arrow_down:

...adog/android/bridge/internal/ProxyAuthenticator.kt
96.15% <0.00%> (ø)


Fix: sa.key missing position bug.

Fix bug of missing key "sa.key" in "pool-coordinator-static-certs" secret , which is put to "pool-coordinator-static-certs" key in "sa" secret.

/approve


Add new stage for configuring DNF Automatic

Add a new stage org.osbuild.dnf-automatic.config for configuring DNF
Automatic.
The stage changes persistent DNF Automatic configuration. Currently, only
a subset of optoins can be set:

'commands' section

apply_updates
upgrade_type

Fix #908
Signed-off-by: Tomas Hozza<EMAIL_ADDRESS>

I just realized that some stages (e.g. https://github.com/osbuild/osbuild/blob/main/stages/org.osbuild.sshd.config) tend to have an extra config top-level property, compared to this stage. I don't think that this is a critical thing to have, but wanted to point this out in case @gicmo would have a preference in this...

I just realized that some stages (e.g. https://github.com/osbuild/osbuild/blob/main/stages/org.osbuild.sshd.config) tend to have an extra config top-level property, compared to this stage. I don't think that this is a critical thing to have, but wanted to point this out in case @gicmo would have a preference in this...

I would indeed prefer to move it to a config subsection, mostly to keep us flexible for the future if we wanted to add options that concern the stage itself and not the config. Otherwise looks good to me. I was for a moment thinking if it should be org.osbuild.dnf.automatic.config, but I think the current name is fine as well.

@gicmo I changed the schema. Also force-push didn't invalidate @achilleas-k review, which is probably not ideal?

@thozza good point, left-over from my experiments with Mergify.io


Bug: imported json from beepbox (and some other mods) has all volumes set to 25

When importing a json song exported from unmodded BeepBox all of the instrument volumes get set to the maximum 25 volume. It would make more sense to default to 0 or try to convert volumes to the [-25, 25] range.

(This is a duplicate of #68 but I won't close the issue)
This is inherited behavior from Jummbox


Dashboard does not fully display system info

UPDATE 1: I am able to make it work by ... runing nodejs
but still it's not working with  php, python

PREVIOUS post:
I understand that there are two dashboards:

demo: located in /linux-dash/ (this one uses static data, I just skipped it and moved on to the real one below)
real: located in /linux-dash/app/. I access my server URL at : http://myIP/linux-dash/app/

Even though there are requests from module "disk_partitions" like these

But still I got no results displayed on the dashboard

I also manually ran the command produced by the shell script successfully

Do you have any ideas what need to be corrected here? Thank you!

Appears you're having the same issue w/php that I ran into.  If you have a look at my issue report I include a work around.

Nice!
I was able to make index.py (python script) work by doing the same way, so I suspect it's not the problem  of  index.* files in side /app/server/ but the process of handling the return value afterward
Only index.js works without any modification, as it's native to javascript, angularjs scripts
Specifically, it's the problem of "linuxDash.min.js" inside /app directory
Example:
Line 34, I add the backslash stripper:

/app/server/linux_json_api.sh
Line 5:
#echo -n "$data" | sed -r "s/\"/\\\\\"/g" | tr -d "\n";
====>
echo $data

This should now be addressed (php)

I seem to have the same issue but with nodejs. Tried the fix above, didn't work, can anyone help?


Feature Request: command line agrument to tail a socket host/port

Hello,
I have a custom python hander to emit jsonl to a socket.  I used toolong to monitor the socket and it works great.  I have to pipe it using this command: "nc -lk 9999 | tl".  It would be awesome if it had a command line agrument that would accept a  socket host/port.  This would make this use case workable in Windows easy.   I attached the custom hander.
Another feature would be a short cut key to clear the log/session.
Thanks!
jsonl_socket_logging.zip

Tired netcat for windows:
ncat -l -p 9999 | tl
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "D:\pypoetry\virtualenvs\aebusspy-huV59KkV-py3.12\Scripts\tl.exe_main.py", line 7, in
File "D:\pypoetry\virtualenvs\aebusspy-huV59KkV-py3.12\Lib\site-packages\click\core.py", line 1157, in call
return self.main(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\pypoetry\virtualenvs\aebusspy-huV59KkV-py3.12\Lib\site-packages\click\core.py", line 1078, in main
rv = self.invoke(ctx)
^^^^^^^^^^^^^^^^
File "D:\pypoetry\virtualenvs\aebusspy-huV59KkV-py3.12\Lib\site-packages\click\core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\pypoetry\virtualenvs\aebusspy-huV59KkV-py3.12\Lib\site-packages\click\core.py", line 783, in invoke
return __callback(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\pypoetry\virtualenvs\aebusspy-huV59KkV-py3.12\Lib\site-packages\toolong\cli.py", line 55, in run
with open("/dev/tty", "rb", buffering=0) as tty_stdin:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: '/dev/tty'


[Marker.io] Close Button should be on top

Steps to reproduce: Open APi page and open Language popup
Actual result: Close button underneath navbar
Expected result: close button should be on top

Reported by: Himanshu<EMAIL_ADDRESS>- Contact via Marker.io
Source URL: https://wf-b.stg.prezzee.com/en-us/corporate-gifting-solutions/api?marker=true
Issue details: Open in Marker.io
Device typedesktopBrowserChrome <IP_ADDRESS>Screen Size1440 x 900OSOS X 15.0.1Viewport Size1440 x 812Pixel Ratio@2x

🤖 Marker.io bot - This issue was marked as ✅ Resolved in Marker.io


📃: Random quote generator

🔴 Title :Random quote generator
🔴 Tech stack : html css javascript
🔴 Objective : create a new project
🔴 Summary :
📸 Screenshots
na

✅ Details to Include When Taking the Issue:
Name :
Participant Role (Specify the Open Source Program name, e.g., GSSOC, Hacktoberfest, etc.):

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

the project is already present


Hitting spacebar only pauses the video, doesn't exit the screensaver

I'm on the latest El Capitan. My screensaver is password locked. When I hit the spacebar, assuming that it'll bring up the password box like it does on other screensavers, it instead pauses the video.
Maybe that's a desirable thing for some reason I'm not seeing, but in that case, it would be better as a non-default option users who need it can turn on.
The space bar, by default, should bring up the password prompt (or exit the screensaver) just as it does for almost all other screensavers.

Also worth mentioning since the issue is similar: the J, K and L keys manipulate the video too. L speeds it up, J rewinds it, and K pauses it, but doesn't unpause it. Maybe default Quicktime controls?
In any case, anyone whose password starts with any of these characters is going to get really confused.

:+1: this is really annoying

This is fixed in the latest beta https://github.com/JohnCoates/Aerial/releases/

Thanks :D

Confirmed! Thank you so much, John. This is the best screensaver ever. Everyone who sees it is immediately like, "Whoa, where did you get that?!" I tell them, of course.

That's awesome to hear @joshlewis!

AWESOME! +1


Whitespace and method order for wp-admin files

No functionality changes!
Corrected whitespace with:
phpcbf ./lib/admin --standard=WordPress

Then manually reordered methods to match display order and a handful of empty lines added/removed.

🤝


DMA (alternative to #209)

Rebased #209 onto current trunk.
Updated to embedded-dma 0.2
Current plan is to drop everything except for SingleChannel, since I think that should end up being simpler and more flexible than the current approach.
To get bidirectional functionality from SingleChannel, will need to update peripherals to provide TX/RX endpoints so that the user can have 2 channels targeting the same peripheral for reading/writing.
For double-buffering I was thinking to pass the second DMA channel as an Option to the constructor, so the same interface is used regardless of single-shot/double buffering.
Will probably rename SingleChannel to Channel or Transfer or something to match the above changes.
EDIT:
After splitting it up, it looks like it already resembles the above somewhat so this might only require minor changes
Other goals:

support transfer widths other than 32bit for peripherals
address concerns with current interface raised in #13 and #209

embedded-dma constrains DMA traits to WORD sizes, where WORD is u8/u16/u32/u64/i8/i16/i32/i64.
64bit sizes compile fine but break at runtime. There are now tests that show this is broken, not sure about the best way to proceed to fix that.

Removed target tests from this branch since there's no practical way to keep CI happy with them in there. Will keep them in a separate repo for now.

Turns out the solution to the CI problem was to exclude the on-target test project from the workspace.
There's still the rest of the problems that come from having a top-level .cargo/config, but those are at least potentially resolvable.

I merged in https://github.com/9names/rp-hal/pull/4 and rebased on latest main, apologies for the noise

Any issues preventing it from being merged?

Well there's at least one issue: it's not finished yet.
The examples are not finished, tests only cover memory-to-memory operations, not all peripherals are implemented yet, and parts of the DMA driver are not complete.
Are you offering to help?

I could help a little. So, what is left to do?
In the code, I find the following:

[ ] SPI example TODOS?
[ ] Ring transfer
[ ] Drop for SingleBuffering
[ ] Documentation improvements
[ ] Fix/enhance pio_dma.rs example
[ ] Fix/enhance spi_dma.rs example
[ ] Callbacks?

The examples are not finished, tests only cover memory-to-memory operations, not all peripherals are implemented yet,

In any case, we should not wait until everything is polished and final before merging, as a partially working DMA support is still better than no DMA support.

MVP as far as I'm concerned:

Remove support for everything that isn't finished.
Fix up documentation for everything that is left.
Add an RTIC example to ensure that it is compatible with resources.
Add an example that uses a peripheral as a source (ADC would be a good candidate).
Add an example that does TX and RX on the same peripheral (SPI/UART would be good candidates for this)
Add an example that does peripheral to peripheral transfers (ADC -> UART would be my choice)
Add an example that does mem-mem transfers (this can be copy-paste from the on-target tests)
Verify after we've done all this that everything still works for anyone currently using code from this branch (slint is the only project I'm aware of).

I'm planning on doing a merge from trunk today, then remove the incomplete stuff and start on the docs, if you want to help with the examples that would be much appreciated.

To implement Bidirectional support correctly we really need something to start multiple channels simultaneously.
single_channel only allows setting one bit at a time.
https://github.com/9names/rp-hal/blob/7c832ea7f21784c062bd2171ef9a5bd62d6d4776/rp2040-hal/src/dma/single_channel.rs#L206
The examples + tests I've added "work" but are unsafe and inherently racy.

Another suggestion: Could you rename SingleBuffering to SingleBufferedTransfer (and Bidirectional accordingly)?
Found the other names a little clumsy when trying it out.

Bidir is now working for SPI, at least with my trivial tests/examples.
One blocker down.

I suggest resolving https://github.com/rp-rs/rp-hal/issues/528 before merging, as this would be a major API change afterwards.

Alright, it's now updated to the API discussed in #528.
Looks okay to me :) but would appreciate another pair of eyes to review how the API looks @jounathaen

I think it is very nice now! Couldn't find anything more to complain about :wink:
Thank you so much for your efforts!

I did a test rebase and it looked sensible, so I'm going to trust that Github's "Squash and merge" is going to do the right thing :crossed_fingers:


[WIP] Task annotations

[x] Tests added / passed
[x] Passes flake8 dask

Related distributed PR https://github.com/dask/distributed/pull/2180


Add flows endpoint

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

30 of 34 (88.24%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.3%) to 96.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

v2/flows/flows.go
27
31
87.1%

Totals

Change from base Build<PHONE_NUMBER>:
-2.3%

Covered Lines:
144

Relevant Lines:
150

💛  - Coveralls


docs: add november contributions post

TODO

Vite 6.0 mention (@patak-dev or @bluwy , maybe one of you can help me out with what to say here? if anything. if its not worth a section, I'm happy to remove it)
More collaboration entries?

I don't have a huge amount in this post yet, so let me know if there's something notable I should mention
Reviewers / mentions
@marvinhagemeister / @jviide - I mentioned the blue sky lockfile change, if you want to check that section makes sense
@filipsobol - I mentioned sonda. if you could review that section, that would be helpful
@cometkim - I mentioned unicode-segmenter FYI
@fabiospampinato - I mentioned kasi here FYI

If that makes sense you could also mention my tool from https://github.com/fuzzyma/e18e-tools and devminer whos hosting the db.
Would be more of a "we are refining our strategies to find more places to enhance" thing.
Its not really something that's interesting to people outside of e18e.

pushed a social image, maybe we could ask if we have a designer in our community that would like to start doing these (instead of me tweaking the web page to take a screenshot 😄)

The section about Vite looks good to me @43081j, I don't think we need to add the exact size reduction. Linking for reference in case you think it is better to add it from filip's post:

The size of the Vite 6 npm package is 2.8 MB, down from 3.27 MB in the last 5.x release.

I'll add a mention about us moving to blue sky, then should be ready to publish I think 👍


Bad files after pull

I've initialized and pulled my drive with ignore-name-clashes.
There were a couple of operation timed out errors but most things got downloaded.
Files reported in these errors seem to exist locally but the contents look corrupted.
In case of errors, local files should either be removed or re-downloaded until they are correct.
Another problem is that when I've tried pull second time, those files were not reported as modified, which I assume means that drive will not try to repair them for some reason. On the other hand a small list of files that were not modified and seem correct are reported as conflicts (These 7 file(s) would be overwritten).

When I added -ignore-conflict all bad files are reported with X status so I guess the second problem doesn't exist.
However the list is of added files (+ status) is quite long even though I didn't modified/added any of them recently.

Hello again @mwisnicki, by "corrupted" what do you mean?

I mean they are not valid jpg/pdf files (cannot open them).

Oh, and the second pull failed with go stacktrace - should I report that as another issue or add it here ?

And the files with the similar names are not jpg nor pdf files either? Am quite surprised by the results of your pulls, "Operation timed out"?
If you ignored name conflicts and the contents were mismatched, please check to see that files themselves upstream are what you think they are.
Also, drive version after go get -u github.com/odeke-em/drive/drive-gen && drive-gen?
Sure you can post the stacktrace here. Did you by any chance compile this with cgo?

Timeout errors looked like this:
pull: /some/file.pdf err: read tcp <IP_ADDRESS>:443: operation timed out

Upstream files are correct, checked both online and as downloaded by official windows client.

The stacktraces on drive pull look like this:
Addition count 170 src: 942.47MB

goroutine 1340 [chan receive]:
github.com/odeke-em/drive/src.func·030()
        /home/marcin/.local/go/src/github.com/odeke-em/drive/src/pull.go:702 +0x67
created by github.com/odeke-em/drive/src.(*Commands).singleDownload
        /home/marcin/.local/go/src/github.com/odeke-em/drive/src/pull.go:710 +0x5d8

There are 8 of them.

And third invocation of drive pull --ignore-name-clashes --ignore-conflict had different summary:
Addition count 163 src: 1011.46MB
Modification count 1 src: 1.21MB dest: 1.21MB
Clashing modification count 1 src: 120.62MB dest: 97.12MB

and finished without issues. Previously broken files look correct now (at the ones I checked).

However executing this for fourth time, it still wants to download something:
Addition count 44 src: 89.49MB
Clashing modification count 34 src: 46.16MB dest: 67.20MB
Modification count 4 src: 4.22MB dest: 4.22MB

Yeah ignoring name clashes will perform a greedy grab of the first file that can be found. It doesn't fix the fact that the unpulled files shouldn't remain unpulled. Unfortunately, It is a nasty work around for users that have clashing names. You can always drive rename paths.... :)

What about addition count ? It keeps reporting around 40 new files every time (each time different amount) ?

Yeah sure, about 40 missing files that haven't yet been pulled and cannot be pulled because they'll clash -- Google Drive handles clashing paths, your file system won't :( So would renaming be alright?

Hmm... indeed they seem to be duplicates. Mostly photos, I think with auto-enhancements done by google+.

Aha I hear ye, you could write write dirty script to auto rename them for you. You could use output piped from drive stat, then to a clash detector to grab Name and fileId, then figure out what to auto rename. I wish I could help, but I've been up for too long, now off to bed. On top of that, I will be at an event from morning till later in the day so away from my computer for a few hours.

Hello again @mwisnicki aside from renaming clashing files, I can't offer much help until we can have a solution that allows the local file system to handle clashing file paths.

Closing this issue as the user needs to take out duplicates themselves.


Adding WHO Stahe

Adding WHO Stage to the Intermediate_Latest_obs table and the Fact_ART table

LGTM @Marymary-dev