Attribution: who took down that node?

When a user takes down another node, how do we give credit for that?

Simplest is probably to assign victim nodes to each team or player

Are we imagining a Red team and a Blue team? Because that would make the accounting easy: just assign points to the opposite team. But in a multi-team environment (royal rumble or Red, Blue, and Green) we will need to track who sent which messages through the network or follow log files to assign points.

we can have as many teams as we want, just so long as we only assign each victim to one attacker

@pinheadmz You had mentioned that there was some tracking that core does which we could leverage to determine who the attacker and who is the victim. What was that called again?

https://github.com/bitcoin-dev-project/warnet/blob/main/docs/warcli.md#warcli-bitcoin-messages

Could we log the quantitative number of messages

We could (getpeerinfo)
    "bytesrecv_per_msg": {
      "pong": 32,
      "verack": 24,
      "version": 147
    },

... but that might not be a good metric depending on the attack.

It's not a silver bullet but in the chaos of node attacks, this is better than nothing for attacker attribution though, no?


No warning when PIPENV_DOTENV_LOCATION does not exist

Is your feature request related to a problem? Please describe.
Like the reporter of #1015, I often switch between multiple environment files (.env for local development, prod.env for accessing a production database, etc.), so I end up prefixing many commands with env PIPENV_DOTENV_LOCATION=prod.env. However, several times I've typed this out and been baffled when the command I run doesn't actually access my prod database, only to realize that I made a typo in the name of the file.
Describe the solution you'd like
It would be nice to show a simple warning when PIPENV_DOTENV_LOCATION is set to a file that does not exist, and then just continue execution as usual. I think this would be a simple modificiation of pipenv.core.load_dot_env. I'm thinking of a message like: File PIPENV_DOTENV_LOCATION=prod.env does not exist, not loading environment variables or something similar. I'd be happy to submit a PR for this if it's something anybody else would appreciate.
Describe alternatives you've considered
Another option is to exit 1 if the specified dotenv file doesn't exist, but that seems like a bit of an overreaction.
Additional context
Platform: MacOS
Python: 3.6/3.7

Seems to make sense to me, I don't see any reason not to add this :) thanks for the suggestion!

Great! I'm happy to work on this myself if that would be helpful.

+1 for sure, that looks pretty good


Add integration test to Airbrake hook

Add a test for the Airbrake hook to:
a) document how the hook is intended to work
b) test that an XML payload is received with the expected message

lgtm


[build system] Ship with catalyst lock for the python interpreter

Context: The library libcatalyst_python_interpreter.py's only purpose is to have a mutex for threads to share the python interpreter. This library is not explicitly shipped by Catalyst but it is implicitly included in Catalyst by auditwheel repair and delocate wheel in Linux and macOS respectively.
Description of the Change: This change fixes the rpath and includes the library explicitly to avoid relying on these processes.
Benefits: The build script is more explicit.
Note that this does not yet fixes the makefile to have something like make wheel && python -m pip install dist/PennyLane_Catalyst-0.10.0.dev0-cp311-cp311-linux_x86_64.whl
[sc-77254]

We may close this one in favour of #624 .

Closed in favour of #624


Questions about 10B-chinese

Hi,
I've tested GLM-10B-chinese pretraining in our cluster 256 x A100 (80G) with RDMA using config/ds_block_10B_chinese.sh and config/config_block_10B.json which choose stage 2 in ZeRO. The train batch size is 256*21=5376. However, the speed is too slow: each step cost 38s, and if we train 150k steps as script config/ds_block_10B_chinese.sh, it will cost 65 days to train 150k steps(stage 1 with 512 seq_len).  I've the following questions:
1. Is MP_SIZE=1 when pretraining 10B-chinese ?
2. Is the 10B-chinese model released pretrained using batch_size=256*21=5376, train_steps=150k and zero stage 2 as config/config_block_10B.json?
3.  Why use two stages to pretrain 10B-chinese (first train 150k steps with seq_len=512 and batch_size=256*21, second train 150k steps with seq_len=1024 and batch_size=256*8) ?
4.  What is a step elapsed in your previous pretraining 10B-chinese, and how many hours the pretraining each stage cost ?

Yes. Practically we found that using ZeRO optimizer is faster than model parallelism when possible.
2 & 3: the 10B-chinese model is pretrained with two stages. The first stage uses config/ds_block_10B_chinese.sh with 72k steps and the second stages uses config/ds_block_10B_chinese_longer.sh with 8k steps. The reason is that the time complexity increases with the square of the sequence length. Training with shorter sequence length can
Each step costs about 15s in the first stage and about 53s in the second stage.

@duzx16
Thanks for your reply.

Is the global_batch_size in Stage 1 256*21=5376 and 256*8=2048 in Stage 2 ?
The ds_config used in pretrain is config/config_block_10B.json ?


Support for virtual graphs

Presently we can specify custom dataset for a sparql query with FROM and FROM NAMED declarations, whose default-graph is merge of graphs declared with FROM, and whose named graphs are individual named graphs declared with FROM NAMED.
This pattern works fine until we have to search among merge of small number of named graphs, by enumerating all of them with FROM. But if we have to search across merge of thousands of small named graphs (like those, trusted-by-an-authority, extracted-from-specific-resources, have-certain-provanence), then it won’t scale to mention all of those constituent graphs of custom default graph in every sparql query.
BlazeGraph allows for such virtual views through their support for Virtual Graphs.
And Virtuoso supports it through Graph Groups
Is there any possible way we can do this in neptune?
Thanks for your work.. @beebs-systap, @iansrobinson and all.

@beebs-systap , @iansrobinson et al., Can support for virtual graphs which is there in blazegraph versions be available in neptune? It solves many practical problems. And seems easy to implement.
Thanks


Form label alignment bug

Reproduction code:
<c-form-item>
  <c-input slot="label"></c-input>
  <c-input></c-input>
</c-form-item>

The two inputs are NOT aligned correctly.

:tada: This issue has been resolved in version 0.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Raise NoCredentialsError when no credentials are found

Certain credential providers, such as the InstanceMetadataProvider  have the chance to fail to load credentials on start, even when the credentials exist and valid. Typically, such failures would get handled by the RefreshableCredentials class; however, that refresh only occurs if the provider was able to find credentials (thereby being the chosen credential provider).
The above situation results in a scenario where it's possible to create a session that does not have credentials. This will result in the client raising a NoCredentialsError upon the first usage of the credentials. That error will not have the ability to auto-resolve.
This PR moves the failure farther up the pipe, such that if credentials cannot be obtained, it will immediately bubble up to the user.
Having systematic credential failures like this are critical to be found upon session creation, rather than client use, as it enables services utilizing the session to know with certainty whether the session is usable or not.

Looking at the Java SDK, it does appear that when no credentials are able to be loaded an exception occurs. This is therefore a mismatch between AWS SDKs.

Might want to look at changing the tests?

Thanks - I was too quick to look over the output from Travis. I just pushed up the test fixes. At the very least everything under tests/unit passes locally

I determined my local tests passed because I had valid AWS credentials, so it was finding those. I updated those tests (ones that created a client) to simply have a mock credentials object. Hopefully this one will pass 🤞

Codecov Report

Merging #1752 into develop will decrease coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1752      +/-   ##
===========================================
- Coverage    92.49%   92.46%   -0.03%
===========================================
  Files           53       53
  Lines         9873     9874       +1
===========================================
- Hits          9132     9130       -2
- Misses         741      744       +3

Impacted Files
Coverage Δ

botocore/credentials.py
98.49% <100%> (-0.25%)
:arrow_down:

botocore/awsrequest.py
98.03% <0%> (-0.33%)
:arrow_down:

Continue to review full report at Codecov.

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

Tests fixed. All issues were around not mocking credentials, except the change to test_misconfigured_source_profile, which from what I could tell appeared to be a result of a missing parameter.

@tehtechguy You might wanna have a look at merge conflicts here.

Hi @tehtechguy,
Our apologies for the lack of attention to this.  We won't be able to merge this, as it would be a backwards incompatible change for some interfaces (not all client functionality requires credentials) to raise an error instead of returning None.  If you're still experiencing problems related to resolving credentials, we'd be more likely to accept something that makes credential existing credential providers more resilient.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.46%. Comparing base (e2ecebc) to head (dc3c7b3).
Report is 5477 commits behind head on develop.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1752      +/-   ##
===========================================
- Coverage    92.49%   92.46%   -0.03%
===========================================
  Files           53       53
  Lines         9873     9874       +1
===========================================
- Hits          9132     9130       -2
- Misses         741      744       +3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Sizing of Tagline looks odd, should be inline

The sizing of Tagline looks odd on mobile device, it would have been better inline.

Now the tagline should fit inline on most mobile devices.


4 registerPlugin events triggered

In talk-control-slave, in :
this.eventBusSlave.on(MASTER_SLAVE_CHANNEL, 'registerPlugin', ({ pluginName }) => {

Add :
if (pluginName === 'touchPointerInput') {
    console.log('slave => registerPlugin', pluginName, this.shadowRoot);
}

For layout stage view, there are 4 of thoses events triggered, but it should be only 2 : reveal engine talk control slave and slide-view talk control slave.
Why ?

This bug causes the zooming behavior with dlbclick to not work on branch feat/61-handle-input-generators-as-plugins

Found !
We must not call super.init() in components slaves (like slide-view-slave.js)


Update logoutSessionsOnSensitiveChanges and injectOptionsFromRemoteContext configs

Loopback throws off the below warnings:
The user model "User" is attached to an application that does not specify
whether other sessions should be invalidated when a password or
an email has changed. Session invalidation is important for security
reasons as it allows users to recover from various account breach
situations.

We recommend turning this feature on by setting
"logoutSessionsOnSensitiveChanges" to true in
server/config.json (unless you have implemented your own solution
for token invalidation).

We also recommend enabling "injectOptionsFromRemoteContext" in
User's settings (typically via common/models/*.json file).
This setting is required for the invalidation algorithm to keep
the current session valid.

We should consider setting them as recommended.

/cc @BerkeleyTrue to confirm.

Sure, anything to get rid of these annoying messages

Sure, I'll check and add this for LB2.
And Just FYI, I got rid of them with the update to LB3 that I attempted with #14727.
Unfortunately, I got stuck midway with that PR, but yeah I'll make that once again possibly soon too.


egui-wgpu: Add a wgpu texture

Is your feature request related to a problem? Please describe.
egui-wgpu has a method to upload raw image pixels and create an egui texture. It is missing a method to allocate an egui texture from a wgpu texture. This was available in egui_wgpu_backend: https://docs.rs/egui_wgpu_backend/0.17.0/egui_wgpu_backend/struct.RenderPass.html#method.egui_texture_from_wgpu_texture
Describe the solution you'd like
Add a similar method to egui-wgpu so that any texture usable by wgpu can also be used directly in egui.
Describe alternatives you've considered

Stick with egui_wgpu_backend instead and update it to be compatible with egui 0.18. This was originally tracked in https://github.com/hasenbanck/egui_wgpu_backend/issues/60, but closed when egui-wgpu became a viable alternative.
Animated textures can be updated every frame with the existing update_texture method, but this is unnecessary and probably a performance hit.

Additional context
This is an important feature for pixels, which provides access to the wgpu texture view. And generally important for any application that renders to a texture.

closed in https://github.com/emilk/egui/pull/1660


Storage Accounts Per Subscription

This page says 200 Storage Accounts per subscription.  But the following article I link to just below says 200 Storage Accounts Per Region Per Subscription meaning you can have 200 Storage Accounts in US East and another 200 Storage Accounts in US Central.  Document needs updating.
https://docs.microsoft.com/en-us/azure/storage/common/storage-scalability-targets

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

ID: 1cd57b6d-3c6f-e546-b3ea-f83892f4c23d
Version Independent ID: 27aecc7b-f107-96d4-b044-ae026bca2727
Content: Azure subscription limits and quotas
Content Source: articles/azure-subscription-service-limits.md
Service: billing
GitHub Login: @rothja
Microsoft Alias: byvinyal

@Lagrik Thanks for the feedback. We are actively investigating and will get back to you soon.

@Lagrik Thanks for the feedback! I can confirm that the storage account limit is 200 storage accounts per region per subscription, but this area of the document about Azure Resource Manager Limits

Storage accounts per subscription | 200

I have assigned the issue to the content author to evaluate and update as appropriate.

@Lagrik Thanks for the feedback, and apologies for the long delay in responding. I'm going to get this fixed.


Update to design-system 0.32.0

TODO:

The filter tags on the blog page, after the style changes to the tag in #1777, makes the "X" closing button move to a second line underneath the "Data, research, and reports" filter tag at full desktop viewport width (at least with my setup). It looks a little weird, so was wondering if we wanted to change that?

Currently on cf.gov (DS 0.31.0)
On local (DS 0.32.0)

What should I do about the Snyk warnings? Recent commits look like they ignored these errors, but I wanted to make sure. I also don't have access to see what's failing, but have requested access via Snyk's UI.
Testing on IE, Edge, and Android (on actual hardware, not just dev tools)? This might be tricky unless someone grants me access to push these changes to one of the dev environments, which I've only heard about? I don't know if this browser testing is usually done for these DS updates, but I wanted to make sure I was extra thorough for this first update.

Changes

Update deps: design-system 0.32.0 (2023-10-24)

Breaking

cfpb-core

#1785 fix: [Lead paragraph] Adjust margins

Bug Fix

cfpb-buttons, cfpb-core, cfpb-layout, cfpb-notifications, cfpb-pagination, cfpb-tables, cfpb-typography

#1789 Remove redundant margin | add responsive style code comments

Enhancement

cfpb-layout

#1781 New component: o-text-introduction

cfpb-forms

#1777 [Tag] Update styles per approved redesign

How to test this PR

Does updating the design-system dependency cause any unintended consequences for the cf.gov styling or functionality? (see screenshot comparisons below)

Screenshots
#1785 fix: [Lead paragraph] Adjust margins (desktop)

Currently on cf.gov (DS 0.31.0)
On local (DS 0.32.0)

#1789 Remove redundant margin | add responsive style code comments (mobile)

Currently on cf.gov (DS 0.31.0)
On local (DS 0.32.0)

#1781 New component: o-text-introduction
currently only used in the design system
#1777 [Tag] Update styles per approved redesign
NOTE: Tag length on Blog now makes the "X" closing button move to a second line underneath "Data, research, and reports" filter tag at full desktop viewport width (at least with my setup). It looks a little weird, so was wondering if we wanted to change that?

Currently on cf.gov (DS 0.31.0)
On local (DS 0.32.0)

Checklist

[x] PR has an informative and human-readable title

PR titles are used to generate the change log in releases; good ones make that easier to scan.
Consider prefixing, e.g., "Mega Menu: fix layout bug", or "Docs: Update Docker installation instructions".

[x] Changes are limited to a single goal (no scope creep)
[x] Code follows the standards laid out in the CFPB development guidelines
[x] Future todos are captured in comments and/or tickets
[x] Project documentation has been updated, potentially one or more of:

This repo’s docs (edit the files in the /docs folder) – for basic, close-to-the-code docs on working with this repo
CFGOV/platform wiki on GHE – for internal CFPB developer guidance
CFPB/hubcap wiki on GHE – for internal CFPB design and content guidance

Front-end testing

Browser testing
Visually tested in the following supported browsers:

[x] Firefox
[x] Chrome
[x] Safari I noticed a bug with the category dropdown on the blog site on Safari, but it's not a new issue and I believe may be addressed by https://github.com/cfpb/design-system/pull/1792 in a future DS release
[ ] Edge 18 (the last Edge prior to it switching to Chromium)
[ ] Internet Explorer 11 and 8 (via emulation in 11's dev tools)
[ ] Safari on iOS
[ ] Chrome on Android

Accessibility

[x] Does not introduce new errors or warnings in WAVE

Other

[x] Does not introduce new lint warnings
[x] Flexible from small to large screens

Makes the "X" closing button move to a second line underneath the "Data, research, and reports" filter tag at full desktop viewport width (at least with my setup). It looks a little weird, so was wondering if we wanted to change that?

Yeah this jumped out to me as well. We'd want to fix this.

What should I do about the Snyk warnings? Recent commits look like they ignored these errors, but I wanted to make sure. I also don't have access to see what's failing, but have requested access via Snyk's UI.

Mentioned in chat, can be ignored.

Testing on IE, Edge, and Android (on actual hardware, not just dev tools)? This might be tricky unless someone grants me access to push these changes to one of the dev environments, which I've only heard about? I don't know if this browser testing is usually done for these DS updates, but I wanted to make sure I was extra thorough for this first update.

Mentioned in chat, tools are available.

Superseded by https://github.com/cfpb/consumerfinance.gov/pull/8009


Remove CanModify argument and Modify arguments from Factory...

From @reflection-emit on February 16, 2017 17:33
... for performance reasons.
Copied from original issue: reflection-emit/Cauldron#23

Removed in 1.1.0


Removing IDisposable from NetworkConnection

clientOwnedObjects is a hashset which doesn't need to be disposed
DestroyOwnedObjects already clears list which is called by networkmanager when connection disconnects

nice find @James-Frowen .
this actually fixes the bug I encountered yesterday where calling connection.Dispose caused a NRE


Symmetric Variable / VarKey ==

Closes #1138
@whoburg ready for review

Test models please

I think __eq__ should distinguish between things which have a .key attribute and things which are VarKeys; a string a Variable and a VarKey are not equal, but they may both refer to the same VarKey via either their .key attribute or the VarKey's .keys attribute


SRS Review: Goal Statements Section

TODO
Any point that requires more discussion should be pulled out to a different issue; I grouped these all here since they are related and small.

[x] The list of inputs seems ambiguous. There should be a "the" in front of each input; without it, it implies that the constant length and modulus of elasticity are also across cross-sections of the beam as well (which might be the case; if so, there should only be a "the" in front of "constant length").
[x] There shouldn't be a comma after "beam".
[x] What is the point of calculating the reaction of the supports under load? If it is outputted to the user, it should be in the System Context section.

I just put the "of moment ..." bit in parentheses. I think that solves it?
:heavy_check_mark:
I removed this goal altogether.

Closed in c8a5a64b1c7b5a21a7a417a4deef992eaf7e7b71

I think so - I realized that you could also remove the parentheses if you also remove the first "and".


Doesn't work with tailwind

By using "prettier-plugin-tailwindcss" and adding "plugins": ["prettier-plugin-tailwindcss"] to .prettierrc.json prettier stops working for any type of file. If I remove this line it works for everything except the tailwind classes.
I also tried "eslint-plugin-tailwindcss" without changing ".prettierrc.json" I get lines below the code where there is a tailwind error. This is coming from Eslint, but on save prettier doesn't format tailwind classes. It does the rest of the code.

I'm also having this same issue. prettier-eslint does not work with "prettier-plugin-tailwindcss" in the VSCode plugin

Running into the same issue

Works great with https://github.com/francoismassart/eslint-plugin-tailwindcss


I can't loading ES6 Dynamically in the Browser since v0.17

Hello
With the v0.16.6, I could loading ES6 Dynamically in the Browser
(https://github.com/ModuleLoader/es6-module-loader/tree/v0.16.6#getting-started)
like that:
<script src="js/vendors/browser.js"></script>
<script src="js/vendors/es6-module-loader.js"></script>

<script>
  System.transpiler = 'babel';
  System.import('js/index').then(function(module) {
    console.log("loaded.")
  }).catch(function (e) {
    console.error(e);
  });
</script>

Since the v0.17 I've got an error when loading script
I use es6-module-loader-dev.js instead of   es6-module-loader.js as explained here https://github.com/ModuleLoader/es6-module-loader#getting-started
and I've got this in the browser console:
GET http://localhost:8080/js/index 404 (Not Found)
t       @ es6-module-loader-dev.js:7
g.fetch @ es6-module-loader-dev.js:7
g.fetch @ es6-module-loader-dev.js:7
(anonymous function) @ es6-module-loader-dev.js:7

Not Found: http://localhost:8080/js/index
  Error loading http://localhost:8080/js/index(anonymous function) @ (index):18

Regards
Philippe

You need to include the file extension (js/index.js) as of the latest major version.

It works fine, thank you very much :+1:


While picking file from external memory or google drive getting exception.

While picking file from external memory or google drive getting exception. The plugin is only working for internal memory file picking. Has any solution for this problem?

Do you have the same issue as  #44 #27 #24?
If so my pull request #52 can help you.

@ArtjomP
I have used your pull request source code building and replacing all of dll files but getting such exception:
System.MissingMethodException: Method 'Plugin.FilePicker.Abstractions.FileData.ReadFully
Do you have any suggestion?


Added 404 page

Related Issue
#254
Description
I have added animation in 404 page
Type of PR

[x] Feature enhancement

Screenshots / videos (if applicable)

Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
Kindly add relevant labels to this PR

conflicts

conflicts

Resolved conflicts

@PatilHarshh Kindly review it now


TextInput doesn't work with placeholderTextColor and conflict to transparent

The placeholderTextColor props doesn't work.
When I set the Input 's style of backgroundColor to transparent, the Placeholder words is gone.

See #997 - try pulling down that diff to see if it fixes your problem

@brentvatne  Yes, it works well. Whether transparent or solid background. thanks

When using experiementing  with an earlier version of react-native I made an implementation of placeholder that was different than this, but find it useful. Basically placeholder propType is an object with shape of {text: string, {fontSize: number, fontStyle: string, fontWeight: number, fontFamily: string}. This way it's a unified prop. Then on the UIKit side, all font attributes default to properties in self.font or defaultView.font.
If anyone is interested in this, happy to make a PR.

cc @vjeux ^

A more React Native-like api would be
<TextInput placeholderText="abc..." placeholderStyle={{ color: 'red' }} />
Where placeholderStyle overrides style attributes

Thanks @vjeux, that looks good to me

I, too, is more like it. I could take up the implementation of styles for textInput.


Issue with Alcock vapor pressure data

Describe the bug
The chemical Metadata in chemicals/Vapor Pressure/Alcock_Itkin_Horrigan_metalic_elements_sublimation.tsv only contains parameters A, B, C, D The documentation states those parameters can be used for the chemicals.dippr.EQ101 equation. However the EQ101 takes a fifth parameter E which default value is 0. With this default value the equation does not produce the original data published by Alcock.
When manually setting E=1 one can obtain correct values from EQ101. The easiest fix would be to change the default value of the python function code of EQ101 from 0 to 1. (not sure if this can break something else) Alternatively one could add a column with E=1 to the chemical metadata. With E=1 the above metadata seems to reproduce exactly the original publication data...
There is a second related issue in the liqud vapor pressure metadata chemicals/Vapor Pressure/Alcock_Itkin_Horrigan_metalic_elements.tsv In this dataset there is a column E that has the (allegedly wrong) value -3
This makes the equation non-isomorphous to that used originally Equation by Alcock. However even when setting E=1 manually the parameters fail to reproduce (exactly) the original published data by Alcock. So maybe on this dataset something else went wrong during parameter transoformation. (the error is luckily relatively small with the pressure off within a factor of 0.5-2 or so)
Additional context
For most elements the difference is small so that it can be overlooked. However for Tungsten (W) the error gets HUGE when using EQ101 without manual setting E=1 on the function. (8,36e3 Pa @ 2500K) where in fact (8,05e-4 Pa would be correct)
according to Wikipedia the 10000 Pascal pressure should not be reached until about >>5000K
bonus error
the chemical metadata table lists a Tmin of 220 K for Tungsten. However the original Publication from Alcock has the validity of this data specified from 2200K (there is a 0 missing)
For the range from 298K to 2200K and Tungsten the original Alcock publication lists an additional parameter set which is as follows: A=-0.5570 B=-43830.0000 C=2.5145 D=-0.2074 Tmin=298 Tmax=2200 (Parameters of the original Alcock publication which uses base 10 logaritm), They would need to be transformed before being useable in EQ101.

Hi Martin,
You are correct, 1.0 as last coefficient was implied and without setting it the D coefficient does nothing. Thermo did use this data set, and used the 1.0 E coefficient correctly. I do hate to waste by adding a column with all the same values, but in this case I have changed the file to have the extra E column set to 1 anyway - it is a small file.
Regarding the temperature range for tunsten, I can confirm there was a zero missing. The 2200-2500 K range was used. The second parameter set won't be added, but someone could refit the data to cover both ranges. The higher temperature range was used because the data with a higher vapor pressure can be more accurately measured.
The -3 coefficient an error mistake, I see now the author intended for the D coefficient to be multiplied by 0.001 in the equation (and so in thermo would have a value in the .tsv 1000x less than the table). I have corrected that error. I have updated the documentation.
The improvements will be out in the next chemicals release.


Partial fix for nested child components updates bug

Partial fix proposal for #530.
As I explained in the issue, the bug seems to be linked to the fact that script tags where not updated for each child components displayed after the first page load. Resulting in children being ignored and no actions could work on it.
With this fix, actions worked for me.
But to test it I had to set children components data field manually on client side according to their state on server, because I don't know how to get the data field efficiently for each new child. We need it to validate the checksum and send a request to the server.

Let's try to get this in a mergeable state :bowing_man:
@adamghill can you please elaborate on "how to get the data field efficiently for each new child"?

Yep, so what @gschurck is saying is this fix would require the code to look at every DOM element node on the page every time a new AJAX call is received. I need to test the performance implications of this and I'd also like to figure out if maybe there is a simpler and/or more straight-forward approach. I'll let you know what I come up with.

@imankulov this is a fundamental feature, please check with @adamghill

I think this should be fixed by 0.58.0 of Unicorn which drastically improves child components so I'm going to close this PR. If it is still a problem feel free to re-open a new issue. Thank you!


exportArchive exportOptionsPlist error for key "method" expected one {} but found app-store

Steps to reproduce

flutter build ipa --release --build-number xx --export-options-plist path/to/exportOptions.plit

Actual results
Archiving com.company.x...
Automatically signing iOS for device deployment using specified development team in Xcode project: XXXXXXXX
Running Xcode build...
 └─Compiling, linking and signing...                        31.1s
 └─Compiling, linking and signing...                        13.6s
Xcode archive done.                                         308.1s
✓ Built build/ios/archive/Runner.xcarchive (871.7MB)

[✓] App Settings Validation
    • Version Number: 1.0.13
    • Build Number: 96
    • Display Name: AppName
    • Deployment Target: 12.0
    • Bundle Identifier: com.company.x

[!] App Icon and Launch Image Assets Validation
    ! App icon is set to the default placeholder icon. Replace with unique icons.
    ! Launch image is set to the default placeholder icon. Replace with unique launch image.

To update the settings, please refer to https://flutter.dev/to/ios-deploy

Building App Store IPA...                                          607ms
Encountered error while creating the IPA:
error: exportArchive exportOptionsPlist error for key "method" expected one {} but found app-store

Try distributing the app in Xcode: "open /build/ios/archive/Runner.xcarchive"

Logs

Logs

<!-- Paste your logs here -->

Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.5, on macOS 15.1.1 24B91 darwin-arm64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 16.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.95.3)
[✓] Connected device (4 available)
[✓] Network resources

• No issues found!

Hi @InfinitySoftware-House, can you share the content of your exportOptions.plist file?
Please also provide the output of flutter build ipa --release --build-number xx --export-options-plist path/to/exportOptions.plist -v
The output may be huge, so kindly put it into a .txt file and attach it here.
Thank you

Hi!
This is the exportOptions.plist file content:

compileBitcode

destination
export
method
app-store
signingStyle
automatic
stripSwiftSymbols

teamID
XXXXXXX
thinning
<none>

The output log is too big, 66MB, when the maximum to upload is 25, what can I do?

Hi @InfinitySoftware-House, you can upload it pastebin or a similar site.
Can you try making the method app-store-connect instead of app-store to see if it works? I might have seen somewhere that the key is no longer app-store but I can't find it at the moment.

Hi @InfinitySoftware-House, you can upload it pastebin or a similar site.
Can you try making the method app-store-connect instead of app-store to see if it works? I might have seen somewhere that the value is no longer app-store but I can't find it at the moment.

It's the same:
exportArchive exportOptionsPlist error for key "method" expected one {} but found app-store-connect
This is the url for the log.txt:
https://jumpshare.com/s/u1w2jPWEZ6qjxPh6s9q7

Hi @InfinitySoftware-House, curious, do you get this same issue with the counter app?

Hi! Thank you for your help! The "counter app" is building the IPA, but I see that the ios folder is slightly different, the counter app has no pods or podfile.
Is it changed during flutter's updates?
If yes, how can I change the project to not use the Pods?

Hi @InfinitySoftware-House, kindly share the ExportOptions.plist file and the flutter build -v logs for the counter app so that we can try comparing both of them to see if there's any differences.
Thank you

Hi @danagbemava-nc, for the build of the counter app I am not using the custom ExportOptions parameters, but the default one.
This is the log file for the counter app "build ipa" command:
https://jumpshare.com/s/jk9F605n5cBoSqhCm7B2
I don't really know what changed between my project and the counter app...
Please help me🥲

Ok, I don't know why, but the issue was caused by a library, "awesome_notifications", I've tried to remove it, now it works!

Hi @InfinitySoftware-House, do you get the same issue if you add the awesome_notifications plugin to the counter app?

Hi @InfinitySoftware-House, do you get the same issue if you add the awesome_notifications plugin to the counter app?

I was also facing the same issue after adding awesome_notifications plugin but finally found the solution on their discord .
add these 3 lines to your podfile
unless target.name == 'Runner'
config.build_settings['SKIP_INSTALL'] = "YES"
end
it will look like this
post_install do |installer|
installer.pods_project.targets.each do |target|
target.build_configurations.each do |config|
unless target.name == 'Runner'
config.build_settings['SKIP_INSTALL'] = "YES"
end
end
flutter_additional_ios_build_settings target
end

Thank you! I resolved the issue! @entrepreneur1001
Thank you too! You were very helpful! @danagbemava-nc

Glad to see it has been resolved.
Closing this as solved.


tensorflow_serving protobufs

Hi!
Could you please add tensorflow_serving protobufs to the Go package as well? They depend on the tensorflow/core/ package, which is part of the graft.

@yutkin yes, I'll look into adding these

I've added these protos into the nightly branch

I've added these protos into the nightly branch

Thanks a lot!

Hello @wamuir!
Btw, is it possible to add gRPC service of TF-serving to graft like this?
Without it, it's impossible to use TF-Serving with graft package via gRPC.


Feature: Add errorprone to the build

You should check out http://errorprone.info/

@hestad We already have PMD and Findbugs. What does this do additionally?

It breaks your build when you have an errorprone-error, but they also operate with warnings. It evaluates the source-code and not the bytecode like PMD/Findbugs. I have used it with maven and it's configured under maven-build-plugin. The gradle version is here: https://github.com/tbroyer/gradle-errorprone-plugin. I see that it only supports Java 7 and 8, while you guys still support 6. But, I also see that you try build with 7/8 and should add it then :)
You can read more about it here: https://www.reddit.com/r/java/comments/30hjui/error_prone_google_library_to_catch_common_java/

Closing this unless someone is willing to submit a PR for this


dm: fix panic of openapi operate-schema

What problem does this PR solve?

Issue Number: close #10839
What is changed and how it works?

for a task in invalid stage(create task but no start), report an error instead of panic

Check List
Tests

Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/retest

/retest

/cherry-pick release-8.1


Added a new public method stringHasXss

Added a new public method stringHasXss, useful when we just want to know if the given string has or not a malicious XSS code. Useful in a pre-dispatch method, to make sure we are going to call a safe URL when typed by the user.
Updated the unit tests for the new method.

This change is

Coverage decreased (-87.4%) to 10.484% when pulling bf8e3634c86369d1113017ca6e015fcabf9d3c7f on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Coverage decreased (-87.4%) to 10.484% when pulling bf8e3634c86369d1113017ca6e015fcabf9d3c7f on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Coverage decreased (-87.4%) to 10.484% when pulling bf8e3634c86369d1113017ca6e015fcabf9d3c7f on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Codecov Report

Merging #25 into master will decrease coverage by 86.62%.
The diff coverage is 8.33%.

@@             Coverage Diff             @@
##           master      #25       +/-   ##
===========================================
- Coverage   97.76%   11.13%   -86.63%
===========================================
  Files           1        1
  Lines         402      404        +2
===========================================
- Hits          393       45      -348
- Misses          9      359      +350

Impacted Files
Coverage Δ

src/voku/helper/AntiXSS.php
11.13% <8.33%> (-86.63%)
:arrow_down:

Continue to review full report at Codecov.

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

Coverage decreased (-87.7%) to 10.16% when pulling bf8e3634c86369d1113017ca6e015fcabf9d3c7f on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Coverage decreased (-87.7%) to 10.16% when pulling bf8e3634c86369d1113017ca6e015fcabf9d3c7f on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Coverage decreased (-87.7%) to 10.16% when pulling bf8e3634c86369d1113017ca6e015fcabf9d3c7f on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Coverage decreased (-87.7%) to 10.16% when pulling bf8e3634c86369d1113017ca6e015fcabf9d3c7f on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Coverage increased (+0.006%) to 97.861% when pulling 69fc98f4c67bc4a361a451a898ebbd3d445d6792 on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Coverage increased (+0.006%) to 97.861% when pulling 69fc98f4c67bc4a361a451a898ebbd3d445d6792 on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Coverage increased (+0.006%) to 97.861% when pulling 69fc98f4c67bc4a361a451a898ebbd3d445d6792 on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Coverage increased (+0.006%) to 97.861% when pulling 69fc98f4c67bc4a361a451a898ebbd3d445d6792 on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Coverage increased (+0.02%) to 97.878% when pulling 5c5e3cf1fc4f5e37cc50f25ae8d04b85535876e3 on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.

Coverage increased (+0.02%) to 97.878% when pulling 5c5e3cf1fc4f5e37cc50f25ae8d04b85535876e3 on brunoflmg:new-function-string-has-xss into 10ae436578ed19c7141f8fbae6cc14c200b6b659 on voku:master.


Log format support request

I'm embarrassed I can't figure out the correct log format using the specifiers :( Can you help?
website.com:80 <IP_ADDRESS> - - [04/Dec/2017:06:30:29 -0500] "GET /apple-touch-icon-180x180-precomposed.png HTTP/1.1" 304 153 "https://website.com/browse" "Mozilla/5.0 (Linux; Android 6.0.1; SAMSUNG SM-G900P Build/MMB29M) AppleWebKit/537.36 (KHTML, like Gecko) SamsungBrowser/6.2 Chrome/56.0.2924.87 Mobile Safari/537.36"

I've tried about 100 permutations of this:
%v:%^ %h %^ %^ [%d:%t %^] "%r" %s %b "%R" "%u"

I'm getting: Token '-' doesn't match specifier '%h'
Many, many thanks in advance.

Looks like a COMBINED with virtual host, please give this a shot:
goaccess access.log --log-format=VCOMBINED --enable-panel=REFERRERS --enable-panel=KEYPHRASES

Thanks for the quick response. I had tried that as well, but no luck. Now that I look at the full error, it looks like the issue is deeper. Here's my command string:
goaccess /var/log/apache2/other_vhosts_access.log -o /var/www/report.html --log-format=VCOMBINED --enable-panel=REFERRERS --enable-panel=KEYPHRASES
Here's the error:
/var/log/apache2/access.log
Parsed 10 lines producing the following errors:

Token '-' doesn't match specifier '%h'
...

Why is it referencing a different log file than I submitted in the command string? I've confirmed the log file reference in the command exists at the specified location and has read permissions for all.

interesting, I think this is issue was addressed upstream. However, try adding --no-global-config
goaccess --no-global-config /var/log/apache2/other_vhosts_access.log -o /var/www/report.html --log-format=VCOMBINED --enable-panel=REFERRERS --enable-panel=KEYPHRASES

Voila. That did the trick. Does that mean that goaccess.conf was overriding my execution command? Is the log file to parse specified in the .conf? Just curious.
Thank you!

Glad that did it. That's right. You can try to build from development, I believe that was address a while back and let me know if that solves the issue. The log file to parse can also be specified in the conf file.


Revert IPAM (subnets input)

Commenting the subnets feature as per the backend API will be released for CNV 4.18
Before:

After:

/cherrypick release-4.17


How to modify fonts？

Due to the requirements of stupid journals, I want to know how to modify fonts in Python scripts.

in the tex file you can update with following line
\tikzset{every node/.style={font=\large }} %font size of nodes


error: Future#WaitForCompletion: context has been cancelled: StatusCode=200

Describe the bug
error: Future#WaitForCompletion: context has been cancelled: StatusCode=200 -- Original Error: context deadline exceeded
I0121 07:05:24.811176       1 mutate_app_gateway.go:174] END AppGateway deployment
To Reproduce
We have a deployment with ~35 sites.. and setup http redirect.. so ends up with ~70 listeners.. This is initial deployment, plan on having more towards ~200 sites. Any new deployment that makes a change to the app gateway, times out and then it appears to go into a endless loop of trying to update it.
initially starting occurring on version 1.0.0.. tried the 1.0.1 rc-2 same results
Ingress Controller details

Output of kubectl describe pod <ingress controller> . The  pod name can be obtained by running helm list.

ame:           ingress-azure-7f7fc8cb-6bhlp
Namespace:      default
Priority:       0
Node:           aks-nodepool1-14687019-vmss000000/<IP_ADDRESS>
Start Time:     Tue, 21 Jan 2020 06:49:56 +0000
Labels:         app=ingress-azure
pod-template-hash=7f7fc8cb
release=ingress-azure
Annotations:    prometheus.io/port: 8123
prometheus.io/scrape: true
Status:         Running
IP:             <IP_ADDRESS>
IPs:
Controlled By:  ReplicaSet/ingress-azure-7f7fc8cb
Containers:
ingress-azure:
Container ID:  docker://2757311949c55f45d6d84a92402a69f36cf0a1edd7d82f8403429276c9c85c5a
Image:          mcr.microsoft.com/azure-application-gateway/kubernetes-ingress:1.0.1-rc2
Image ID:       docker-pullable://mcr.microsoft.com/azure-application-gateway/kubernetes-ingress@sha256:1543bf5f9b............965a
Port:
Host Port:
State:          Running
Started:      Tue, 21 Jan 2020 06:50:12 +0000
Ready:          True
Restart Count:  0
Liveness:       http-get http://:8123/health/alive delay=15s timeout=1s period=20s #success=1 #failure=3
Readiness:      http-get http://:8123/health/ready delay=5s timeout=1s period=10s #success=1 #failure=3
Environment Variables from:
ingress-azure  ConfigMap  Optional: false
Environment:
AZURE_CONTEXT_LOCATION:  /etc/appgw/azure.json
AGIC_POD_NAME:           ingress-azure-7f7fc8cb-6bhlp (v1:metadata.name)
AGIC_POD_NAMESPACE:      default (v1:metadata.namespace)
AZURE_AUTH_LOCATION:     /etc/Azure/Networking-AppGW/auth/armAuth.json
Mounts:
/etc/Azure/Networking-AppGW/auth from networking-appgw-k8s-azure-service-principal-mount (ro)
/etc/appgw/azure.json from azure (rw)
/var/run/secrets/kubernetes.io/serviceaccount from ingress-azure-token-mmwtp (ro)
Conditions:
Type              Status
Initialized       True
Ready             True
ContainersReady   True
PodScheduled      True
Volumes:
azure:
Type:          HostPath (bare host directory volume)
Path:          /etc/kubernetes/azure.json
HostPathType:  File
networking-appgw-k8s-azure-service-principal-mount:
Type:        Secret (a volume populated by a Secret)
SecretName:  networking-appgw-k8s-azure-service-principal
Optional:    false
ingress-azure-token-mmwtp:
Type:        Secret (a volume populated by a Secret)
SecretName:  ingress-azure-token-mmwtp
Optional:    false
QoS Class:       BestEffort
Node-Selectors:  kubernetes.io/os=linux
Tolerations:     node.kubernetes.io/not-ready:NoExecute for 300s
node.kubernetes.io/unreachable:NoExecute for 300s
Events:
Type     Reason                     Age    From                                                     Message

Normal   Scheduled                  22m    default-scheduler                                        Successfully assigned default/ingress-azure-7f7fc8cb-6bhlp to aks-nodepool1-14687019-vmss000000
Normal   Pulling                    22m    kubelet, aks-nodepool1-14687019-vmss000000               Pulling image "mcr.microsoft.com/azure-application-gateway/kubernetes-ingress:1.0.1-rc2"
Normal   Pulled                     22m    kubelet, aks-nodepool1-14687019-vmss000000               Successfully pulled image "mcr.microsoft.com/azure-application-gateway/kubernetes-ingress:1.0.1-rc2"
Normal   Created                    22m    kubelet, aks-nodepool1-14687019-vmss000000               Created container ingress-azure
Normal   Started                    22m    kubelet, aks-nodepool1-14687019-vmss000000               Started container ingress-azure
Warning  FailedApplyingAppGwConfig  6m57s  azure/application-gateway, ingress-azure-7f7fc8cb-6bhlp  Failed applying App Gwy configuration:
{
-- App Gwy config --    "id": "/subscriptions/b9.....3/resourceGroups/aks-scus-prod-01-rg/providers/Microsoft.Network/applicationGateways/aks-scus-prod-01-ag",
-- App Gwy config --    "location": "southcentralus",
-- App Gwy config --    "id": "/subscriptions/b9...3/resourceGroups/aks-scus-prod-01-rg/providers/Microsoft.Network/applicationGateways/aks-scus-prod-01-ag",
-- App Gwy config --    "location": "southcentralus",
-- App Gwy config --    "properties": {
-- App Gwy config --        "autoscaleConfiguration": {
-- App Gwy config --            "maxCapacity": 3,
-- App Gwy config --            "minCapacity": 2
-- App Gwy config --        },
-- App Gwy config --        "backendAddressPools": [
-- App Gwy config --            {
-- App Gwy config --                "id": "/subscriptions/b9......3/resourceGroups/aks-scus-prod-01-rg/providers/Microsoft.Network/applicationGateways/aks-scus-prod-01-ag/backendAddressPools/defaultaddresspoo
l",
-- App Gwy config --                "name": "defaultaddresspool",
-- App Gwy config --                "properties": {
-- App Gwy config --                    "backendAddresses": []
-- App Gwy config --
.........
....lots more likes like the ones above with the "App Gwy config"
.....
....
-- App Gwy config --        "webApplicationFirewallConfiguration": {
-- App Gwy config --            "disabledRuleGroups": [],
-- App Gwy config --            "enabled": true,
-- App Gwy config --            "exclusions": [],
-- App Gwy config --            "fileUploadLimitInMb": 100,
-- App Gwy config --            "firewallMode": "Detection",
-- App Gwy config --            "maxRequestBodySizeInKb": 128,
-- App Gwy config --            "requestBodyCheck": true,
-- App Gwy config --            "ruleSetType": "OWASP",
-- App Gwy config --            "ruleSetVersion": "3.0"
-- App Gwy config --        }
-- App Gwy config --    },
-- App Gwy config --    "tags": {
-- App Gwy config --        "ingress-for-aks-cluster-id": "/subscriptions/b9...3/resourcegroups/aks-scus-prod-01-rg/providers/Microsoft.ContainerService/managedClusters/aks-scus-prod-01-cl",
-- App Gwy config --        "last-updated-by-k8s-ingress": "2020-01-21 06:50:19.514982449 +0000 UTC m=+7.103336064",
-- App Gwy config --        "managed-by-k8s-ingress": "1.0.1-rc2/2105e9e7/2019-12-10-07:01T+0000"
-- App Gwy config --    }
-- App Gwy config --}
error: Future#WaitForCompletion: context has been cancelled: StatusCode=200 -- Original Error: context deadline exceeded

Output of `kubectl logs .
-- see above

Any Azure support tickets associated with this issue.
120012021001411

We're currently experiencing this in our production environment too. No traffic is coming through to newly created pods since the ingress cannot update the appgw.
We only use 1 certificate.

We're currently experiencing this in our production environment too. No traffic is coming through to newly created pods since the ingress cannot update the appgw.
We only use 1 certificate.

any news on this one, since we are experiencing the same behavior. The longest time range I have experienced the AppGw being in Updating state and rejecting any traffic on the respective routes was about 10 hours.

@iamNoah1 Did you get hold on the error? I have the same problem. AppGW is in updating state & the logs are not very useful

@prateekchawla to be honest we decided to not use AGIC anymore and now use Traefik Ingress Controller for that specific customer project


ALVR not detecting headset

For some reason the alvr software on both devices wont see eachother

hello, can you ping the headset from the pc ?
what version of ALVR are you using ?
have you opened the firewall port ?
are you on 2.4Gz or 5Gz wifi ?


Enhancements to forked-to createLink

LINKED ISSUES:
Resolves #3416
Resolves #3415

TEST URLS:
Have a fork of this repo
https://github.com/sindresorhus/refined-github/edit/master/.editorconfig

SCREENSHOT:
None

I am confused between #3415 and #3414 are we using default branch or are we using the current branch?

@fregante can you see my comment about and clear me what we want to do? I am super confused

I think you're also trying to support https://github.com/sindresorhus/refined-github/issues/3415, right? Not just #3416

To answer https://github.com/sindresorhus/refined-github/pull/3420#issuecomment-667110407
#3416 is unrelated to branches. It’s just about enabling it on isEditingFile
#3415 means that it should point to the current branch if it exists, otherwise point to the default branch.

@fregante I fully test my code (I have 2 forks of RH).
2 Things.

Am I on the right track here? (I think the code can be improved, for now I made it work)
how do you want to handle editing until the network request finishes. Sometimes github takes a second or 2 to respond. If the url route is edit and the branch is still HEAD it will land on a 404. Should I change it to blob and later change it back to edit?

Toughts?

Very Good!
I am not in from of my computer, I will test it later.

The only issue is if it does not exist on the default branch it will lead to a 404 :(

The only issue is if it does not exist on the default branch it will lead to a 404 :(

It's a risk we can run, instead of having to do 2 HTTP requests for every page visited on forks. Once on the 404 the user can still open the repo.

Before, this wasn't possible because isRepoTree includes isRepoRoot and setting the branch would break that page, but now we're not setting the branch, so we can't skip it.

But if your on isRepoRoot //undefined/undefined
Because of
https://github.com/sindresorhus/refined-github/blob/a918f9a3eb6827dd67efdd1bcf362df2e7152406/source/github-helpers/github-url.ts#L70
The last 2 return undefined.
Should we add a .replace('/undefined','') ?

[this.user, this.repository, this.route, this.branch, this.filePath].filter(Boolean).join('/')

This looks good, but I'm afraid that it would make it hard to debug a missing route, because it would just become user/repo/develop instead of user/repo/undefined/develop
This is inelegant but it works correctly .replace(/(undefined[/])+$/g, '')

This looks good, but I'm afraid that it would make it hard to debug a missing route, because it would just become user/repo/develop instead of user/repo/undefined/develop

Me too.
Or we can just go back to the "less readable version" code

This is inelegant but it works correctly .replace(/(undefined[/])+$/g, '')

Make the change here or in GithubURL?

In GithubURL

I did not check your code but That other one is/was for a trailing slash.

@fregante I need your feedback on how you want to handle isEditingFile

@fregante I need your feedback on how you want to handle isEditingFile

What about it?


statistics: export statistics

It whould be great to have some statistics and internal number available outside.
Implementing an ioctl to get statistics seems the best solution.

Implemented via #20


Fix WebGLPoints layer with non-point geometry

There are two problems with non-point geometries.
If the geometry is null when the source is initialized, the feature is not added to the featureCache_ because forEachFeature skips features with null-geometry. (See also #16433)
Initially this is not a problem, but when the feature has any update it will be added to the cache without  increasing the cache size.
https://github.com/openlayers/openlayers/blob/2b3e645f94967959e346553daf8525d11b2d29d1/src/ol/renderer/webgl/PointsLayer.js#L347-L351
https://github.com/openlayers/openlayers/blob/2b3e645f94967959e346553daf8525d11b2d29d1/src/ol/renderer/webgl/PointsLayer.js#L397-L404
If a non-point geometry is in the cache, the renderInstructions_ array will be initialized too large, and an icon will be drawn in the map center.
https://github.com/openlayers/openlayers/blob/2b3e645f94967959e346553daf8525d11b2d29d1/src/ol/renderer/webgl/PointsLayer.js#L517-L522
Feature is skipped, but render instructions array keeps its length with zero values or whatever value it had last.
https://github.com/openlayers/openlayers/blob/2b3e645f94967959e346553daf8525d11b2d29d1/src/ol/renderer/webgl/PointsLayer.js#L530-L537

I'm wondering if it is worth maintaining both the WebGLPoints and WebGLVector layers.
I haven't tested myself - is the points layer significantly better than the vector layer at rendering points?

I guess the plan would be to discontinue the WebGLPoints layer eventually. But I wouldn't know. Maybe @jahow can enlighten us.


Does a simple merge() on two sequences replace TimeDistributed(Merge(..))?

I want to merge two sequences together - I have an LSTM that encodes an input into a single vector that I replicate, and then I want to merge the resulting sequence with the input to another LSTM, zip style. I'm using the functional API.
text_input = Input((self.text_maxlen,), dtype="int32")
title_input = Input((self.title_maxlen,), dtype="int32")
embed = Embedding(  # TODO re-enable masking once merge supports it
    input_dim=self.vocab_size, output_dim=self.text_embedding_size, mask_zero=False, name="word_embeddings",
    weights=[get_word_vector_matrix()]
)
title_embedding = embed(title_input)
title_encoded = LSTM(self.title_lstm_size, return_sequences=False)(title_embedding)
title_encoded = Dropout(0.3)(title_encoded)

title_encoded_repeat = RepeatVector(self.text_maxlen)(title_encoded)

text_embedding = embed(text_input)

merged_aux_and_text = merge([title_encoded_repeat, text_embedding], mode="concat")

merged_text_lstm = LSTM(self.text_lstm_size, return_sequences=False)(merged_aux_and_text)

Is this the correct way to do this? I would use TimeDistributed here usually, but since merge is a function rather than a layer, I'm not sure it applies. Anyway, the model compiles, I just want to verify whether this does what I think it does.

Duplicated due to GH issues, closing.


Dram Bandwidth below 20bytes/clk/channel for InterleavedToSharded

InterleavedToSharded operation has lower bandwidth when data format is BFLOAT8_B or BFLOAT4_B. Here is an example for tensor dimension used in MLP of Falcon7B.
             OP CODE  IN0_Y  IN0_X INPUT_0_DATA TYPE  DEVICE FW DURATION [ns]  i0_tile_size       bytes/clk
InterleavedToSharded   2048   4608           FLOAT32                   137986        4096             22
InterleavedToSharded   2048   4608          BFLOAT16                    74224        2048             21
InterleavedToSharded   1024   4608           FLOAT32                    73215        4096             21
InterleavedToSharded   2048   4608         BFLOAT8_B                    51907        1088             16
InterleavedToSharded   2048   4608         BFLOAT4_B                    43149         576             10
InterleavedToSharded   1024   4608          BFLOAT16                    40772        2048             19
InterleavedToSharded    512   4608           FLOAT32                    34342        4096             22
InterleavedToSharded   1024   4608         BFLOAT8_B                    27082        1088             15
InterleavedToSharded   1024   4608         BFLOAT4_B                    22265         576              9
InterleavedToSharded    512   4608          BFLOAT16                    18775        2048             20
InterleavedToSharded    256   4608           FLOAT32                    17208        4096             22
InterleavedToSharded    512   4608         BFLOAT8_B                    13555        1088             15
InterleavedToSharded    512   4608         BFLOAT4_B                    11685         576              9
InterleavedToSharded    256   4608          BFLOAT16                    10146        2048             19
InterleavedToSharded    256   4608         BFLOAT8_B                     7321        1088             14
InterleavedToSharded    256   4608         BFLOAT4_B                     6418         576              8

Hey @miacim , what is the performance impact of the overall model? try to prioritize this.

i will call this bug for now.. although i am not sure.

@bbradelTT has started to look into optimizing these OPs

@miacim could you please provide the steps that you used to generate the table?

sorry for the confusion. we are now using a generic op_cat: TM in place of op_cat: perf TM, op_cat:Pytorch TM, etc. I will delete these deprecated TM labels to avoid confusion.

@miacim could you please provide the steps that you used to generate the table?

I used data generated from profile_this, and then just do the math. I tested on 1GHz clk frequency, so math is as follows, for first example.
InterleavedToSharded   2048   4608           FLOAT32                   137986        4096             22
Num_Channels = 12
Num_Tiles = 2048 * 4608 / 32 / 32
Num_Bytes = Num_Tiles * 4096
Bytes/Clk = (Num_Bytes / Fw_Duration) / Num_Channels = 22.8

I looked at the code. I think there are a couple of reasons for the different number of bytes transferred.
First, the smaller data type sizes have extra data overhead on the tile.
tt_metal/common/constants.hpp shows:
constexpr uint32_t BFLOAT8_B_TILE_HW = TILE_HW + 64;        // Value is in bytes - 1B per BFP8 datum + 4x16B exponent section
constexpr uint32_t BFLOAT4_B_TILE_HW = (TILE_HW / 2) + 64;  // Value is in bytes - 0.5B per BFP4 datum + 4x16B exponent section
Effectively, bfloat8_b has a 7% overhead and bfloat4_b has a 13% overhead in terms of bytes per tile element.
Second, for tile layout, the code eventually reaches tt_eager/tt_dnn/op_library/sharded/kernels/dataflow/reader_unary_sharded_blocks_interleaved_start_id.cpp, which performs operations on a per tile basis, including calling a read barrier:
    for (uint32_t h = 0; h < block_height_tiles; h++) {
        uint32_t tile_id = curr_tile_id;
        for (uint32_t w = 0; w < block_width_tiles; w++) {
            noc_async_read_tile(tile_id, s, l1_write_addr);
            tile_id++;
            l1_write_addr += tile_bytes;
            noc_async_read_barrier();
        }
        curr_tile_id += input_width_offset_tiles;
    }

Therefore the smaller data type sizes have more overhead due to executing more code and having more read barriers per a given number of bytes. The overhead for that is much more than 5% (and probably above 10%) for the float32 case, and would be 2x, 4x, and 8x when the size goes down. Using 15%, and looking at the first few entries, the adjusted bytes/clk (bytes/clk*(extra byte mult + code/barrier mult) are pretty well aligned:

in0_y
in0_x
data type
bytes/clk
extra byte mult
code/barrier mult
adj. bytes/clk

2048
4608
32
22
0
1.15
25.3

2048
4608
16
21
0
1.3
27.3

2048
4608
8
16
0.07
1.6
26.72

2048
4608
4
10
0.13
2.2
23.3

I think we'd need to write more data at one time for a single read barrier and loop iteration to be able to make the code run faster.
That would mean either writing more tiles or somehow making the tiles bigger.

@miacim could you please provide the steps that you used to generate the table?

I used data generated from profile_this, and then just do the math. I tested on 1GHz clk frequency, so math is as follows, for first example. InterleavedToSharded 2048 4608 FLOAT32<PHONE_NUMBER> 22 Num_Channels = 12 Num_Tiles = 2048 * 4608 / 32 / 32 Num_Bytes = Num_Tiles * 4096 Bytes/Clk = (Num_Bytes / Fw_Duration) / Num_Channels = 22.8

Hey @miacim
Is input in DRAM, if yes, then
Total memory BW achieved = total bytes / time
(2048*4608/32/32) * 4096 B / 137986 cycles = 273 B/c,  this is basically 273 GB/s , which would be close to theoretical DRAM BW of 288 GB/s (IP spec).
Can you confirm?

@yugaoTT it would be good to see if your u-bench with 4096 transfer size achieves the same BW

@miacim could you please provide the steps that you used to generate the table?

I used data generated from profile_this, and then just do the math. I tested on 1GHz clk frequency, so math is as follows, for first example. InterleavedToSharded 2048 4608 FLOAT32<PHONE_NUMBER> 22 Num_Channels = 12 Num_Tiles = 2048 * 4608 / 32 / 32 Num_Bytes = Num_Tiles * 4096 Bytes/Clk = (Num_Bytes / Fw_Duration) / Num_Channels = 22.8

Hey @miacim
Is input in DRAM, if yes, then
Total memory BW achieved = total bytes / time (2048*4608/32/32) * 4096 B / 137986 cycles = 273 B/c, this is basically 273 GB/s , which would be close to theoretical DRAM BW of 288 GB/s (IP spec).
Can you confirm?

Yes. Inputs are in DRAM. For Float32 and Float16 it is close to theoretical maximum.

@davorchap I'm only getting ~185GB/s for 4096 tile size for the baseline test, and ~215GB/s for the hybrid NoC/round-robin VC test.

@miacim I tried a interleaved to sharded test with input shape of 2048x4608 (FLOAT32), sharded on 8x8 cores, I get FW duration of 198611, which equals to 15.8 GB/s per channel. Not sure why we get diff perfs. Here is my test code
`
@pytest.mark.parametrize(
"input_shape, shard_scheme, shard_size",
[
([1, 1, 2048, 4608], ttl.tensor.TensorMemoryLayout.BLOCK_SHARDED, (256, 576)),
],
)
@pytest.mark.parametrize(
"shard_orientation", [ttl.tensor.ShardOrientation.ROW_MAJOR]
)
@pytest.mark.parametrize("input_dtype", [ttl.tensor.DataType.FLOAT32])
@pytest.mark.parametrize("output_dtype", [ttl.tensor.DataType.FLOAT32])
def test_sharded_tile(
device, input_shape, shard_size, shard_scheme, shard_orientation, input_dtype, output_dtype, function_level_defaults
):
grid_size = device.compute_with_storage_grid_size()
input_size = torch.Size(input_shape)
x = torch.arange(input_size.numel()).reshape(input_size).bfloat16().float()

xt = (
    ttl.tensor.Tensor(
        x.reshape(-1).tolist(),
        x.shape,
        input_dtype,
        ttl.tensor.Layout.ROW_MAJOR,
    )
    .to(ttl.tensor.Layout.TILE)
    .to(
        device,
        ttl.tensor.MemoryConfig(
            memory_layout=ttl.tensor.TensorMemoryLayout.INTERLEAVED,
            buffer_type=ttl.tensor.BufferType.DRAM,
        ),
    )
)

yt = ttl.tensor.interleaved_to_sharded(
    xt, grid_size, shard_size, shard_scheme, shard_orientation, output_dtype=output_dtype
)

zt = ttl.tensor.sharded_to_interleaved(
    yt,
    ttl.tensor.MemoryConfig(
        memory_layout=ttl.tensor.TensorMemoryLayout.INTERLEAVED,
        buffer_type=ttl.tensor.BufferType.DRAM,
    ),
    output_dtype=input_dtype,
)

tt_og = xt.cpu().to(ttl.tensor.Layout.ROW_MAJOR).to_torch()

tt_got_back = zt.cpu().to(ttl.tensor.Layout.ROW_MAJOR).to_torch()

if input_dtype == output_dtype:
    passing, output = comp_equal(tt_og, tt_got_back)
else:
    passing, output = comp_pcc(tt_og, tt_got_back, 0.999)
logger.info(output)

assert passing

`

@miacim could you please share your test for interleaved to sharded test with input shape of 2048x4608 (FLOAT32)
? I can try repro on my end.

fyi @acejkov

@miacim are you DRAM links on WH machine training to 12G ?
This is what my machine gets:

I tried calling noc_async_read_barrier() less frequently (e.g. moving it to the outer loop). That did not change the performance behaviour of my micro-benchmark. The overhead seems to be in the rest of the code.
Next, I'm looking at using both the reader and writer core as reader cores. However, the way that cb_reserve_back, get_write_ptr, and cb_push_back (in tt_metal/hw/inc/dataflow_api.h) work together, it looks like these cannot be called on multiple cores for the same circular buffer. Therefore I'm looking at using two circular buffers next.

It turned out that configuration of the test was L1 interleaved, not DRAM interleaved. Please check if there is something additionally unusual in code.
@pytest.mark.parametrize(
    "dtype", [ttl.tensor.DataType.BFLOAT16, ttl.tensor.DataType.BFLOAT8_B,ttl.tensor.DataType.FLOAT32, ttl.tensor.DataType.BFLOAT4_B]
)
@pytest.mark.parametrize(
    "y", [256, 512, 1024, 2048])
def test_my_sharding(device, dtype, y):
    input_dtype = dtype
    output_dtype = dtype
    shard_scheme = ttl.tensor.TensorMemoryLayout.BLOCK_SHARDED
    shard_grid = ttl.tensor.CoreRangeSet({ttl.tensor.CoreRange(ttl.tensor.CoreCoord(0, 0), ttl.tensor.CoreCoord(7, 7))})

    interleaved_mem_config = ttl.tensor.MemoryConfig(
        memory_layout=ttl.tensor.TensorMemoryLayout.INTERLEAVED,
        buffer_type=ttl.tensor.BufferType.L1,
    )

    sharded_mem_config = ttl.tensor.MemoryConfig(
        memory_layout=ttl.tensor.TensorMemoryLayout.BLOCK_SHARDED,
        buffer_type=ttl.tensor.BufferType.L1,
    )
    x = torch.randn((1, 1, y, 4608)).bfloat16().float()

    xt = (
        ttl.tensor.Tensor(
            x.reshape(-1).tolist(),
            x.shape,
            input_dtype,
            ttl.tensor.Layout.ROW_MAJOR,
        )
        .to(ttl.tensor.Layout.TILE)
        .to(
            device,
            interleaved_mem_config,
        )
    )

    yt = ttl.tensor.interleaved_to_sharded(xt, shard_grid, (y//8, 576), shard_scheme, ttl.tensor.ShardOrientation.ROW_MAJOR )

Here are the numbers for interleaved DRAM with CLK 800MHz. There is still difference in performance for different data types.
                  OP CODE  IN0_Y  IN0_X INPUT_0_DATA TYPE  CORE COUNT  DEVICE FW DURATION [ns]  i0_tile_size  bytes/ns per channel
15   InterleavedToSharded   2048   4608           FLOAT32          64                   244274        4096.0                  12.0
11   InterleavedToSharded   1024   4608           FLOAT32          64                   130888        4096.0                  12.0
12   InterleavedToSharded   2048   4608          BFLOAT16          64                   121322        2048.0                  12.0
14   InterleavedToSharded   2048   4608         BFLOAT8_B          64                    85726        1088.0                   9.0
13   InterleavedToSharded   2048   4608         BFLOAT4_B          64                    73420         576.0                   6.0
7    InterleavedToSharded    512   4608           FLOAT32          64                    67160        4096.0                  11.0
8    InterleavedToSharded   1024   4608          BFLOAT16          64                    63189        2048.0                  12.0
10   InterleavedToSharded   1024   4608         BFLOAT8_B          64                    44682        1088.0                   9.0
3    InterleavedToSharded    256   4608           FLOAT32          64                    39072        4096.0                  10.0
9    InterleavedToSharded   1024   4608         BFLOAT4_B          64                    37932         576.0                   5.0
4    InterleavedToSharded    512   4608          BFLOAT16          64                    37370        2048.0                  10.0
6    InterleavedToSharded    512   4608         BFLOAT8_B          64                    24140        1088.0                   8.0
0    InterleavedToSharded    256   4608          BFLOAT16          64                    21186        2048.0                   9.0
5    InterleavedToSharded    512   4608         BFLOAT4_B          64                    20084         576.0                   5.0
2    InterleavedToSharded    256   4608         BFLOAT8_B          64                    14034        1088.0                   7.0
1    InterleavedToSharded    256   4608         BFLOAT4_B          64                    11304         576.0                   4.0

I've implemented a heuristic to put a read barrier after 8k of data in read requests
https://github.com/tenstorrent-metal/tt-metal/tree/pjosipovic/6729
This has improved dram read 50% - Bfp4, 30% Bfp8, 10% Float16, and no change for Float32

Thanks @pavlejosipovic
I need to figure out why I didn't see any performance change when I tried that.
One suggestion. It might be better for get_barrier_read_threshold() to be something like
uint32_t threshold = 2;
try {
    threshold = 8 / datum_size(data_format);
} catch (const std::exception &e) {
}
return threshold;

There are other data types that can be 1, 2, or 4 bytes.

I've measured this change on GS/WH, DRAM/L1 interleaved tensors and I've placed results here.
https://docs.google.com/spreadsheets/d/1uB3_AYX5PmID6DWW6WZaRN8D2FqTuTkp/edit#gid=1816091962
In general it seems that this change helps in all case.

I talked to @pavlejosipovic
I was looking at the wrong column.
He wants to extend the logic to matmuls and merge in one change.


config.js - how is it generated?

What would you like to know?
Hi!
Could you please share if you are manually updating and managing the config.js or is it automatically created parsed from the actual markdown headers?
If you could share the process I would really benefit from it within our community to build a tip and tricks site.
Thank you!

Hello @baky0905. Thanks for your question. The config.js is provided by Vuepress - the static site generator which we use for our blog. In that config.js we enabled search and customized it, added navigation, customized template for each page/markdown file etc. You can read more about the config.js on Vuepress here I hope this helps.

Thanks, @vaheminasyan2 for your prompt reply. It helped slightly for sure. I was just wondering if you maintain each new tip and trick and add it to the sidebar/config manually, or did you do some sort of automation of parsing and extraction. Is it possible to get access to the source code/configuration files/branch or is this something that you would like to keep private?
Thanks once again, really appreciate it.

@baky0905 we use automation for that. I uploaded vuepress folder into this repo where you can find config and other related files which handle that process.

Many thanks @vaheminasyan2 , this is an excellent motivation!


Media player script not found (Spotify)

Hi, i'm trying to do this:
[module/spotify]
type = custom/script
exec = ~/.dotfiles/config/i3/scripts/mediaplayer
tail = true

the mediaplayer script is mediaplayer but it does not work, i run in terminal and found:
$ /usr/bin/env sh -c ~/.dotfiles/config/i3/scripts/mediaplayer

Do you get any errors when starting the bar?
Is the whole bar blank or is the module just missing?

@patrick96 bar in blank, i no open Spotify and test and show:
Connection to player failed: No players found

But open Spotify the bar is blank. 😢
Before i using i3blocks and this config the script work:
[mediaplayer]
command=perl ~/.dotfiles/config/i3/scripts/mediaplayer
instance=spotify
interval=5
signal=10

This doesn't seem to be an issue with lemonbuddy.
What happens if you call
/usr/bin/env sh -c ~/.dotfiles/config/i3/scripts/mediaplayer
from the command line?
Also try this:
exec = env BLOCK_INSTANCE=spotify ~/.dotfiles/config/i3/scripts/mediaplayer

Output:
$ /usr/bin/env sh -c ~/.dotfiles/config/i3/scripts/mediaplayer
The Strumbellas - Spirits - Sondr Remix

exec = env BLOCK_INSTANCE=spotify ~/.dotfiles/config/i3/scripts/mediaplayer not found :(

I give up :(

I'll have a look at the script, but I can see that the script output in your terminal doesn't end with a new line \n which would cause a problem.

This would work. Tailing won't work since there's no continous output from the script.
[module/spotify]
type = custom/script
exec = BLOCK_INSTANCE=spotify ~/sandbox/mediaplayer.pl | xargs echo
interval = 5

@jaagr Awesome, thank!!


perf(cli): enable Node.js compile cache

Summary
Enable Node.js 22 new compile cache in Rspack CLI, this makes Node.js startup faster.

before:

after:

Related Links

https://nodejs.org/id/blog/release/v22.8.0
https://github.com/nodejs/node/pull/54501
https://github.com/web-infra-dev/rsbuild/pull/3627

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

Are we going to apply v8-compile-cache as well to the lower Node.js versions? I suppose those are pretty the same thing, right?

As far as I know, they are basically the same.
From the Node.js blog:

This is a built-in alternative to the v8-compile-cache/v8-compile-cache-lib packages, but have better performance and supports ESM.

However, I do not want to introduce v8-compile-cache as it is a third-party dependency and has some limitations. Node 22 has reached LTS so more users will be able to use the native enableCompileCache soon.


sample/reference solution diagrams will help jump start

Team,
Thanks for publishing the BTP solution diagram guidelines and how-to exercise in draw.io.
What will greatly help is if you could also publish some sample/reference diagrams from where others can import into draw.io and make their own changes based on the use case - rather than having to start from scratch every single time.
Ashok.

Hi, This file seems to be not present. Can you please place it back? Thanks...
https://github.com/SAP-samples/teched2023-XP286v/blob/main/exercises/ex0/libraries/SAP_Service_Icons_TechEd2023.drawio


Updating docs for R2

Updating documentation for release 2

Installation guide for GCE
Exercises for OAI and Free5GC

Hi @arora-sagar, Looks good. My only concern is that there is a lot or duplication here. The main diff I see is the cluster provisioning and the nf deploy. I think the main purpose of the orig guide was to show the diff ways to deploy a pkg. ie kpt vs PV/PVS vs UI. I think we should keep that common to both use cases. Maybe in a separate section/file?

@efiacor I don't really get it how you want to do that?

Hi @arora-sagar, Looks good. My only concern is that there is a lot or duplication here. The main diff I see is the cluster provisioning and the nf deploy. I think the main purpose of the orig guide was to show the diff ways to deploy a pkg. ie kpt vs PV/PVS vs UI. I think we should keep that common to both use cases. Maybe in a separate section/file?

@efiacor I don't really get it how you want to do that?

We prob should break some of the exercise steps into their own sections/files and reuse them. Say for instance the "inter-cluster networking" is common to both.

Hi @arora-sagar, Looks good. My only concern is that there is a lot or duplication here. The main diff I see is the cluster provisioning and the nf deploy. I think the main purpose of the orig guide was to show the diff ways to deploy a pkg. ie kpt vs PV/PVS vs UI. I think we should keep that common to both use cases. Maybe in a separate section/file?

@efiacor I don't really get it how you want to do that?

We prob should break some of the exercise steps into their own sections/files and reuse them. Say for instance the "inter-cluster networking" is common to both.

@efiacor Actually that section is not common either. Making the workload cluster in both of them is different the topology is different and the networking is also different.
Both exercises don't share the network, network topology right now is in dedicated sub-folders. That is the reason I am using the export commands to use the right path.

Hi @arora-sagar, Looks good. My only concern is that there is a lot or duplication here. The main diff I see is the cluster provisioning and the nf deploy. I think the main purpose of the orig guide was to show the diff ways to deploy a pkg. ie kpt vs PV/PVS vs UI. I think we should keep that common to both use cases. Maybe in a separate section/file?

@efiacor I don't really get it how you want to do that?

We prob should break some of the exercise steps into their own sections/files and reuse them. Say for instance the "inter-cluster networking" is common to both.

@efiacor Actually that section is not common either. Making the workload cluster in both of them is different the topology is different and the networking is also different.
Both exercises don't share the network, network topology right now is in dedicated sub-folders. That is the reason I am using the export commands to use the right path.

Ye maybe not the best example as the networking is all a bit hacky. But say for instance, these files are identical - https://github.com/nephio-project/test-infra/blob/main/e2e/tests/free5gc/clab-topo.gotmpl
https://github.com/nephio-project/test-infra/blob/main/e2e/tests/oai/clab-topo.gotmpl
We may need to revisit this later. I'll ignore the duplication for now.. :smile:

@arora-sagar Cna you just check the links, it could be something with my setup that they are not working as I'm not using GCE so I'm just checking the links on my local machine.

The tag v2.0.0 tag mentioned in the document doesn't exist for the moment. The documentation is released before the tag is released.

/lgtm
/approve


Pushback deprecation for v0.7 release

Pushes back the deprecation warnings to v0.8 as we're doing an express release for a forgotten commit.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


AVRO-3670: Add NET 7.0 support

What is the purpose of the change
Add support for NET 7.0, AVRO-3670
Verifying this change
This change just adds a new target framework, updates package versions and disables EOL SDK related warnings.
Documentation

Does this pull request introduce a new feature? (no)

Hi @martin-g and @KalleOlaviNiemitalo ,
As I know application written with .NET Core 6.0 will not be able consume nuget package for .NET 7.0
So it can mean that after merge this PR and create new package users will not be able upgrage to new version without upgrade whole app to NET 7.0.
Are you going publish two versions of Apache.Avro? one for 6.0 and another for 7.0?

Are you going publish two versions of Apache.Avro? one for 6.0 and another for 7.0?

Apache.Avro and compression helpers are packaged as a class library and targeting netstandard2.0 and netstandard2.1 (Both targets are part of the published nupkg). All .NET SDKs are supported, because of the netstandard 2.x targets (including most of modern .NET Frameworks).
At this time only the Avro.Tools are packaged as an executable tool and the nuget package will have all .NET versions included, 3.1, 5.0, 6.0, and 7.0. When the tool is installed, nuget installer will select the the target matching the current installation.
Merging this PR will extend the supported NET SDK and not limit it. https://github.com/apache/avro/tree/master/lang/csharp#project-target-frameworks

@zcsizmadia Is there anything that should be done for this PR ?
You should be able to merge it yourself.

@martin-g The PR is ready for merging IMO. I thought 2 approvals was required.

@martin-g I thought I had merge privis, but it seems not.

The RollForward seems like an interesting idea to try
Adding support for NET 7 gives advantage for additional unit testing and code analytics. I had projects where moving from 6.0 to 7.0 exposed unit test faiures, caused by fixes in the .NET runtime libs. E.g. MS fixes issues in those librraies, and not backported to earlier major version, and the code behaves differently.
AFAIK  System.CodeDom versioning looks similar to the major NET SDk version, however they work with previous NET SDKs as well. Just like e.g. System.Text.Json. You can use the 7.0 of these liibs with netcoreapp3.1 or net5.0. System CodeDom v7+ is not tested against netcoreapp3.1 and net5.0, however they will still support it.

Security fixes will be part of all major NET SDk releases, however if there is any fix which cause issues, because 7.0 is left behind, the dependency can be reduced to 6.0. However I feel that moving froward with the versions make more sense than keepongthem, especially when it comes to to e.g. code generation. Without knowing all the differences between CodeDom 6 and 7, most likely there are many improvements introduced in 7.0.

I thought I had merge privis, but it seems not.

Please visit https://gitbox.apache.org/boxer/ and link your Apache account with your Github account.


add current profile annotations to CVO manifests

This is matches openshift/enhancements#414 and doesn't change existing behavior

Initial annotations were added a month ago via https://github.com/openshift/cluster-kube-apiserver-operator/pull/957. This file was added recently and probably the two PRs were interleaved.

/lgtm
/approve


Evaluation section needs to be finished -- waiting for tomorrow's meeting.

The discussion on the MVP is not yet done.

Fixed in c5d6b200878c1eb09a08546ff518ff2cda54a022.


Fix: on running server scripts

fixes #121
the command :
cp -rf demo dist && NODE_ENV=production postcss<EMAIL_ADDRESS>-o dist/demo/styles.css is the cause of the issue because it is not compatible on windows environnement, so it will can added "SET" before "NODE_ENV" so the script will be compatible with windows environnement.

Thanks @elkarouani. Take care!


Logging

Log everything

This guy looks like he knows what he's doing
https://github.com/SmBe19/RedditBots/blob/master/BotSkeleton/BotSkeleton.py

I think we're good on this

Still got room to improve


Remove the require-restart from the plugin.xml

Find a way to allow enabling or disabling of the plugin without the need to restart the IDE.

https://plugins.jetbrains.com/docs/intellij/dynamic-plugins.html#plugin-loadunload-events


Static type improvements

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 86.57%. Comparing base (e712294) to head (01cc502).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #50   +/-   ##
=========================================
  Coverage     86.57%   86.57%
  Complexity      299      299
=========================================
  Files            33       33
  Lines           968      968
=========================================
  Hits            838      838
  Misses          130      130

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


How to read compressed data in IoT hub

We are  looking to send compressed data using Protobuf compression to IoT hub. How can our test team or any other resource perform testing to verify whether data received in IoT hub is as per expectation or not. Is there a way that we can read the compressed data in IoT hub using some utility. Please suggest.

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

ID: 994a485f-92db-68b8-ece0-6fbc114ae26f
Version Independent ID: f8f3c3f8-84f5-c915-a3e8-cb68f2e98a18
Content: Manage Azure IoT Hub cloud device messaging with Azure IoT Toolkit extension for Visual Studio Code
Content Source: articles/iot-hub/iot-hub-vscode-iot-toolkit-cloud-device-messaging.md
Service: iot-hub
GitHub Login: @formulahendry
Microsoft Alias: junhan

Hello @ampac28 . Thank you for your feedback. You should be fine using Visual Studio Code Extension Azure IoT Toolkit.
Let us know if you see issues when using it?
Thanks!

Hello @ampac28 . Thank you for your feedback. You should be fine using Visual Studio Code Extension Azure IoT Toolkit.
Let us know if you see issues when using it?
Thanks!

.  Can you please share the steps on how we can view compressed data in IoT using Azure IoT toolkit?  We are looking for a sample that can help us

I did some more research and my understanding is that we will need to use Azure Stream Analytics on IoT Edge to deserialize protobuf compressed messages: https://docs.microsoft.com/mt-mt/azure/stream-analytics/stream-analytics-previews?view=azurermps-4.4.1
You can use Azure IoT Toolkit to view any type of message, but in this case you will see it compressed…
See also this feedback: https://feedback.azure.com/forums/321918-azure-iot/suggestions/18708898-support-data-compression-for-telemetry
Thank you!

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Cleanup operator cache on config deletion in revisioned control plane

Please provide a description of this PR:
In revisioned control plane, while handling deletion of IstioOperator CR, pruned resources are not cleaned from the controller cache. That means recreation of the same IstioOperator resource does not result in k8s resources because rendered resources are already in controller cache.

@vikaschoudhary16: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
377efba89da6bf6b219e86299228a558fea9db5b
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@vikaschoudhary16: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
785702d6f5581e313101cad807be58cfec1c0f76
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@vikaschoudhary16: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
785702d6f5581e313101cad807be58cfec1c0f76
link
true
/test release-notes_istio

lint_istio
785702d6f5581e313101cad807be58cfec1c0f76
link
true
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@vikaschoudhary16: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
785702d6f5581e313101cad807be58cfec1c0f76
link
true
/test lint_istio

unit-tests_istio
2e95df7a22f3109b0b402c67e47b0e788b1c49cd
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests_istio

cc @litong01 can you take a look at this?

In response to a cherrypick label: new pull request created: #38723


Use the mingw64_runcmd.bat in our scripts directory.

mingw64_runcmd.bat calls bash directly, without --login,
so that it sources ~/.bashrc
@derekstucki Where there any other changes to mingw64_runcmd.bat that you needed to make?
I assume that mingw64_runcmd.bat doesn't need to be in C:\msys64, so we might as well use the one in the civet repo. Does this change work for you?

That looks great. Thanks!


pandoc.exe: Error running filter .\examples\caps.py

Hello,Sir.
Thx for your project very much.
I'm interested in Markdown. So, I found pandoc is something I wanted. But It couldn't parse * [] task1 to
<input type="checkbox"/>task1
. Then, I decided to write a filter to resolve it.
Then

I installed Python 3.5.
Clone your project to test the filters
I opened Powershell on Windows 10 as administrator
Go to the director of the project
Run Python step.py install
Run pandoc --filter ./examples/caps.py

But there was an error after I had run pandoc --filter ./examples/caps.py.
The error is
PS C:\Users\MyName\Downloads\pandoc-filter\pandocfilters-master\examples> pandoc --filter .\caps.py
abcd
^Z
pandoc.exe: Error running filter .\caps.py
C:\Users\MyName\Downloads\pandoc-filter\pandocfilters-master\examples\caps.py: createProcess: invalid argument (Exec format error)
PS C:\Users\MyName\Downloads\pandoc-filter\pandocfilters-master\examples>

And then, I checked my way again and again, it was still the same.
So, I asked google, but nobody had this error.
In the end, I ask you for help. Was anything forgot to write in the readme?

What version of pandoc?
And, do you get the same error if you leave off the .
before caps.py?
+++ Feng [Dec 21 16 01:57 ]:

Hello,Sir.
Thx for your project very much.
I'm interested in Markdown. So, I found pandoc is something I wanted.
But It couldn't parse * [] task1 to task1. Then, I decided to write a filter to
resolve it.
Then
* I installed Python 3.5.
* Clone your project to test the filters
* I opened Powershell on Windows 10 as administrator
* Go to the director of the project
* Run Python step.py install
* Run pandoc --filter ./examples/caps.py
But there was an error after I had run pandoc --filter
./examples/caps.py.
The error is
PS C:\Users\MyName\Downloads\pandoc-filter\pandocfilters-master\examples> pandoc
--filter .\caps.py
abcd
^Z
pandoc.exe: Error running filter .\caps.py
C:\Users\MyName\Downloads\pandoc-filter\pandocfilters-master\examples\caps.py: c
reateProcess: invalid argument (Exec format error)
PS C:\Users\MyName\Downloads\pandoc-filter\pandocfilters-master\examples>
And then, I checked my way again and again, it was still the same.
So, I asked google, but nobody had this error.
In the end, I ask you for help. Was anything forgot to write in the
readme?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, [1]view it on GitHub, or [2]mute the
thread.
References

https://github.com/jgm/pandocfilters/issues/53
https://github.com/notifications/unsubscribe-auth/AAAL5HDJNsf3LXxW_cpb8Hku--HcYRytks5rKPgKgaJpZM4LSw9P

Pandoc version: 1.17.2
Yes, it's the same in that occasion.

Try the latest Pandoc (1.9). Had the same error and with 1.8 and 1.9 it's gone

Try the latest Pandoc (1.9). Had the same error and with 1.8 and 1.9 it's gone

@kexxxfeng, he means 1.18 and 1.19. Anyway, it is always a good idea to use the latest versions of everything (in this case, pandocfilter and pandoc). Often bugs are already fixed in the latest version.
And also you might check out @sergiocorreia's alternative of pandocfilters — panflute. @jgm recommended this, and @sergiocorreia has ported the examples, including the caps.py you were trying.
And regarding your need to use GitHub checkbox, you might also want to see jgm/pandoc#3051, especially this comment by @jgm. He already made an example on what syntax to use (basically a span enclosing the checked/unchecked item). But note that the discussion over there isn't finalized.


Please provide your stack

Hello ! as part of the pre-hackfest checklist, we asked you to describe the stack of software that you will be using in your project.
We want you to do this with StackShare. Please create a new stack and let us know the url of your stack. Only one per project.

Hi @trinitysege  Can you let us know the link to your stack on stackshare please ?
Thanks !
Bruce

hello,
This is the link:  https://stackshare.io/trinitysege
Regards.
On Fri, Nov 25, 2016 at 11:45 PM, Bruce Becker<EMAIL_ADDRESS>wrote:

Hi @trinitysege https://github.com/trinitysege Can you let us know the
link to your stack on stackshare please ?
Thanks !
Bruce
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sci-gaia/LagosHackfest/issues/24#issuecomment-263049806,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ART1WEV9uknNY-q_V0_89Q-zfqm7YsO7ks5rB-OWgaJpZM4K6d1Y
.

Hi @trinitysege - thanks ! but we  need you to create a stack and fill it in with the things that you're using. See  http://www.africa-grid.org/hackfest-site//before-checklist/  for instructions.

okay
On Sat, Nov 26, 2016 at 2:33 AM, Bruce Becker<EMAIL_ADDRESS>wrote:

Hi @trinitysege https://github.com/trinitysege - thanks ! but we need
you to create a stack and fill it in with the things that you're using. See
http://www.africa-grid.org/hackfest-site//before-checklist/ for
instructions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sci-gaia/LagosHackfest/issues/24#issuecomment-263056194,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ART1WMYJDV4RqEmeq9jKNll8g9VfYEDKks5rCArngaJpZM4K6d1Y
.

https://stackshare.io/obafemi-awolowo-university-ife/obafemi-awolowo-university-ife/
Hope i got it?
On Sat, Nov 26, 2016 at 3:08 AM, segun oyeyiola<EMAIL_ADDRESS>wrote:

okay
On Sat, Nov 26, 2016 at 2:33 AM, Bruce Becker<EMAIL_ADDRESS>wrote:

Hi @trinitysege https://github.com/trinitysege - thanks ! but we need
you to create a stack and fill it in with the things that you're using. See
http://www.africa-grid.org/hackfest-site//before-checklist/ for
instructions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sci-gaia/LagosHackfest/issues/24#issuecomment-263056194,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ART1WMYJDV4RqEmeq9jKNll8g9VfYEDKks5rCArngaJpZM4K6d1Y
.

looks good, thanks.


enlive

Fixes #37
Moving templates over from Selmer to enlive one piece at a time.
This PR moves the existing templates over from Selmer to use enlive instead.
The main benefit here is that we can now use namespaced keywords and Clojure functions to transform our data within our templates (I personally think that it is a lot more pleasant to develop with).

Looks great, thank you!


Re-factoring and cleanup

Have re-configured library paths and fixed the problem of why it wasn't building

No new dependencies
Have copied across 3rd-party-libs from kuksa.invehicle repo
Added a setup_build script for making authentication easier prior to build
Corrected misleading example text and value setting in test-client

@SebastianSchildt  has agreed to test this works on his machine also

Tested and works. Can be merged. This does not include any new dependencies. Unfortunately this is large this also contains some changes Bojan has ben working on before but did not yet merge. We will try to keep further PRs smaller, but we need to get this one out of the door, because we are working from here....
Hope @rohoet  or @daschubert  can merge this. If there questions please ask.

Ok, thanks for the update!
How about turtle? I noticed lots of references in the code and it seems to be a BSD licensed mock object library based on Boost that we do not have a CQ for, yet.

Ahhrg. It is for our unit tests. I did miss, that we did not upstreamed them here :(  Sorry. What kind of information and what kind of incentive it would take for you to open a CQ? I assume turtle version, link to source code, license information and promise of a sufficient amount of chocolate next time we meet?

No worries 😊 Test-only dependencies are workswith requests and can be combined in a single CQ.
We already have such CQs and I'll just enter another one for turtle.
This is usually only crucial for the next release so I think merging this PR should be fine for now.