Bugfix/782

fix for #782
Test:
mytest.ts
/// <reference types="cypress" />
import { DataTable, Given } from '@badeball/cypress-cucumber-preprocessor'

Given('I access the {string} webpage', (url: string) => {
    cy.visit(url)
})

Given('I print a table', (url: DataTable) => {})

mytest.feature
Feature: Printing

    I want to print all tables!

    @focus
    Scenario: Print with Cucumber
        Given I print a table
            | tal  | tal  |
            | tal1 | tal2 |
            | tal3 | tal4 |
            | tal5 | tal6 |

        Given I print a table
            | tal  | tal  | new  |
            | tal1 | tal2 | new2 |
            | tal3 | tal4 | new3 |
            | tal5 | tal6 | new4 |

Output:

the change is breaking a few tests, wip.

Can you see if you can use eg. cli-table to pretty format the table as well? Making this look appealing as well is a prerequisite for me FYI.

@badeball / @nilgaar - just a question, will this be fixed for Javascript as well or just typescript?

@badeball The output is in a Markdown format, I'll check what's the best we can do with it.
@RankaPalak this will be fixed in the package, running in either JS or TS projects.

I think this is the best we can get @badeball , considering the limitations of the cy.console

Can you share what you have discovered regarding that limitation?

The main problem is that, for the message parameter, the 1st line is indentated differently from the others:

So, we have to assume this (because there's no way to make it fit perfectly with blank spaces) or either we skip the 1st line:

Hey, wanna jump on this because I've been watching this conversation. As a second opinion, I like skipping the first line, but had some questions.
Can you show what happens based on the following data table transformations:

dataTable.rawTable
dataTable.raw() (rows without a header; traditionally I use this as dataTable.raw().flat() to use in steps)
dataTable.rows()
dataTable.hashes()
dataTable.rowsHash()
dataTable.transpose()
docString for steps that use a doc string instead of a dataTable
I'm referring here for the different transformations.

https://github.com/cucumber/cucumber-js/blob/main/docs/support_files/data_table_interface.md

I totally must have missed that in my thought process. Will this work the same for a docString where it will print the text below the step?

It doesn't, but that's a cool idea for a follow-up PR 🕵🏻♂


Upgrade to socket.io v4

We just updated to v3, but v4 has an important improvement that should help karma-runner on large web app tests:
https://github.com/socketio/engine.io-client/commit/81d7171c6bb4053c802e3cc4b29a0e42dcf9c065
The update would be a breaking change.

We don't depend on engine.io-client NPM package and we already require the latest socket.io (which depends on the engine protocol v4 according to the migrations docs). I think we can close this issue as done.


Added 3D google map in the map section

Pull Request Template for GSoC 2024
Related Issue

closes:#177

Description of Changes
Added 3D google map in the map section
Type of Pull Request

[ ] Bug Fix
[X] New Feature
[X] Performance Improvement
[ ] Documentation Update
[ ] Code Refactoring
[ ] Security Enhancement
[ ] Other (Specify): _______________

Screenshots / Demo (if applicable)

Checklist for Contributor

[X] I have reviewed my code before submission.
[X] I have read and followed the project's Contribution Guidelines.
[X] I have tested the code thoroughly.
[X] I have attached related issue numbers, screenshots, or videos (if applicable).
[X] I have added comments to my code where necessary for clarity.
[X] I have checked the existing code base and my changes do not introduce any breaking changes.
[X] My PR addresses only one issue; I have created separate PRs for other issues.
[X] I have ensured that the PR does not include unrelated files.
[X] I am mindful of the project's coding standards and best practices.

Additional Information
None
Contribution Guidelines Reminder
Thank you for contributing to our project! Please ensure you follow the guidelines below:
Contribution Levels

Task Categories:

Basic Tasks: Primarily focused on small changes like minor bug fixes, documentation improvements, and unit testing. Labeled as Easy.
Intermediate Tasks: Moderate-level tasks that involve enhancing existing features, code refactoring, or performance improvements. Labeled as Medium.
Advanced Tasks: Complex tasks like building new features, significant refactoring, or critical bug fixes. Labeled as Hard.

Acknowledging Effort:

Exceptional contributions will be recognized with a higher contribution level if substantial effort and innovation are demonstrated.

Review Process

PRs will be reviewed by the maintainers, and feedback will be provided. Contributors should address feedback promptly and resubmit the PR if necessary.

Contributors are encouraged to participate in the review process of other PRs for community growth.

We appreciate your time and effort in improving this project and look forward to your valuable contributions!

Once check with the PR
let me know if to make any changes

@PraveenUppar the map you have added is not 3D and we want it for particular station wise maps

ok i will work on it


Adjust tooltip of status-only subscription button

The button group for but the current label ("Subscribe just to the close event") is a little misleading because that subscription level also includes other status change events (e.g. merging a PR or reopening an issue). I slightly reworded the tooltip to be a bit more accurate.
Test URLs
This PR, for instance.
Screenshot
Before:

After:

This was intentional, it subscribes to up to three events and I’m not sure I want to make it too verbose. I suppose that it could be both short and correct though:

Subscribe to closing, reopening, merging events

Does this make the tooltip too long? Does it cause issues on mobile? I think we could alternatively wrap it manually via %A0 character (if I remember correctly) to make sure it’s never longer than the column.

I think we could alternatively wrap it manually via %A0 character (if I remember correctly) to make sure it’s never longer than the column.

Ah, nice idea. U+00A0 is actually the non-breaking space; newline is U+000A. I believe that in this case, since it's part of an HTML attribute, we need to encode it as an HTML entity (
, or
 in decimal notation), not as URL-encoded (%0A). I'll make the change.

@fregante here's what it should look like now:


"File IO" Lecture

Not done yet.

We should add eventually add details on:

writing cross-platform code with files (e.g. joinpath, etc.)
downloading files from the web
Pandas/etc. loading of data
Storing as HDF files for matlab/python
RDatasets for exploration
Maybe DataDeps when it is ready.


Internally tagged enums are not working

I was trying this library out and I noticed that internal tagging is not deserializing properly. I get runtime errors after simply serializing to a JsValue and deserializing from it.
I looked through your code and found the internally tagged enum test, but it looks like you guys aren't expanding the #[serde(tag = "tag")] attribute in your test_enum! { ... } macro.
I started getting compile errors in the tests/serde.rs file after I added the expansion to the macro_rules block:
macro_rules! test_enum {
    ($(# $attr:tt)* $name:ident) => {{
        #[derive(Debug, PartialEq, Serialize, Deserialize)]
        $(# $attr)*    // RIGHT HERE
        enum $name<A, B> {
            Unit,
            Newtype(A),
            Tuple(A, B),
            Struct { a: A, b: B },
        }

        test_via_json($name::Unit::<(), ()>);
        test_via_json($name::Newtype::<_, ()>("newtype content".to_string()));
        test_via_json($name::Tuple("tuple content".to_string(), 42));
        test_via_json($name::Struct {
            a: "struct content".to_string(),
            b: 42,
        });
    }};
}

Console output:
serde-wasm-bindgen$ wasm-pack test --node
[INFO]: Checking for the Wasm target...
   Compiling serde-wasm-bindgen v0.1.3 (serde-wasm-bindgen)
error: #[serde(tag = "...")] cannot be used with tuple variants

error[E0277]: the trait bound `enums::InternallyTagged<(), ()>: enums::_IMPL_DESERIALIZE_FOR_ExternallyTagged::_serde::Serialize` is not satisfied
   --> tests/serde.rs:89:23
    |
39  |   fn test_via_json<T>(value: T)
    |      -------------
40  |   where
41  |       T: Serialize + DeserializeOwned + PartialEq + Debug,
    |          --------- required by this bound in `test_via_json`
...
89  |           test_via_json($name::Unit::<(), ()>);
    |                         ^^^^^^^^^^^^^^^^^^^^^ the trait `enums::_IMPL_DESERIALIZE_FOR_ExternallyTagged::_serde::Serialize` is not implemented for `enums::InternallyTagged<(), ()>`
...
218 | /     test_enum! {
219 | |         #[serde(tag = "tag")]
220 | |         InternallyTagged
221 | |     }
    | |_____- in this macro invocation

These errors do make sense because internally tagged enums don't work with tuple variants. It's just that they weren't being tested before.
I'm looking at this bug to see if I can fix it, but unfortunately I don't know much about serde. I'm not sure if it's a simple fix or not, but if I figure anything out I'll open a PR.
In the meantime, if you guys have any thoughts or guidance on this, I'd appreciate it!

I created pull request #4 with changes that appear to fix this issue.


[#4238] Read multiple records in one unwrap call

Motivation:
OpenSslEngine.unwrap(...) should try reading multiple ssl records in one call.
Modifications:
Keep on reading if not everything was read.
Result:
Same behavior as SSLEngineImpl

Thanks @normanmaurer, the code change looks good, but let me verify it with the real app to be sure. I should be able to do it today.

Thx!

Am 23.09.2015 um 11:39 schrieb Ismael Juma<EMAIL_ADDRESS>Thanks @normanmaurer, the code change looks good, but let me verify it with the real app to be sure. I should be able to do it today.
—
Reply to this email directly or view it on GitHub.

I checked this and it works fine as long as I don't limit the size of the applicationBuffer to sslEngine.getSession().getApplicationBufferSize() (I set it to an artificial large size to test it, but one could also expand it as needed without). Current users that call unwrap in a loop while bytesProduced > 0 already have to handle this case, but it seems more justifiable in that case. What are your thoughts, do you think the new behaviour is within the spec?
Also, I updated the test to pass with this change (and not pass without) in case you're interested:
https://github.com/ijuma/netty/commit/0f6eff24885bfbe965d0cabd61115067d14feedc

@ijuma what you mean by "limit the size" ? Can you show me some code that will fail with this but work with jdk impl ?

@normanmaurer This is the existing code (not written by me):
               sslEngine.unwrap(netReadBuffer, appReadBuffer);
               netReadBuffer.compact();
                // handle ssl renegotiation.
                ...
                if (unwrapResult.getStatus() == Status.OK) {
                    read += readFromAppBuffer(dst);
                } else if (unwrapResult.getStatus() == Status.BUFFER_OVERFLOW) {
                    int currentApplicationBufferSize = sslEngine.getSession().getApplicationBufferSize();
                    appReadBuffer = Utils.ensureCapacity(appReadBuffer, currentApplicationBufferSize);
                    if (appReadBuffer.position() >= currentApplicationBufferSize) {
                        throw new IllegalStateException("Buffer overflow when available data size (" + appReadBuffer.position() +
                                                        ") >= application buffer size (" + currentApplicationBufferSize + ")");
                    }

With the change in this PR, sslEngine.getSession().getApplicationBufferSize() returns a size that is not large enough to accomodate the Status.BUFFER_OVERFLOW.

@ijuma I guess this is because we try to handle multiple ssl records here and not all of them fit in the buffer. Does this make sense ? I wonder if in this case we should just add as many records as possible before return. Which means as long as one fits in it should be ok. Agree ?

Yes, that's right. It is true that if we add as many records that fit into the buffer, than we'll only overflow if one record doesn't fit (which should never happen given the current implementation of getApplicationBufferSize()).
Having said that, it does mean that we still don't produce everything that we consume (although this would happen more rarely now). Worth an implementation note in the javadoc probably.

@ijuma yep but I think that is ok. Let me update the code and ping you again

@ijuma please check again

I tried the latest version and from a user's code perspective, it works similarly as it did before. Less unwrap calls may be needed, but since one cannot rely on that (without knowing how many SSL records there are in the source buffer at least), the same scenarios have to be handled as before.
So, to me it looks like an internal optimisation and it works correctly as far as I can tell.

Thanks

Am 24.09.2015 um 16:25 schrieb Ismael Juma<EMAIL_ADDRESS>I tried the latest version and from a user's code perspective, it works similarly as it did before. Less unwrap calls may be needed, but since one cannot rely on that (without knowing how many SSL records there are in the source buffer at least), the same scenarios have to be handled as before.
So, to me it looks like an internal optimisation and it works correctly as far as I can tell.
—
Reply to this email directly or view it on GitHub.

@ijuma I need to do some more testing with this... So this will most likely not be in the next release. Just FYI ;)

No problem. Thanks for the update. :)

@normanmaurer out of interest what do you need to check before merging this?

@johnou perf regression testing... had not time yet though ...

@normanmaurer no worries, just curious.

@ijuma sorry for the delay but I will close this for now and we will do a new PR for some improvements.

No problem at all. Thanks @normanmaurer. :)


DropdownItem link invisible in inverse/dark Navbar.

components: DropdownItem, Navbar
reactstrap version #6.4.0
import method import in typescript
react version #16.3.2
bootstrap version #x.x.x
What? I use Reactstrap instead of bootstrap, right?

What is happening?
When using a dark/inverse Navbar, the Navbar looks great, but the background of the DropdownItem is still light, making the light text on it invisible.
What should be happening?
The DropdownItem should either have dark text, or the background should be dark, like the Navbar.
As it is, I had to manually add the className="bg-dark"attribute on all DropdownMenus and DropdownItems.
Steps to reproduce issue
Run the code.
Code
				<Navbar color="dark" dark expand="md">
					<Nav className="ml-auto" navbar>
						<UncontrolledDropdown nav inNavbar>
							<DropdownToggle nav caret>
								This looks fine
							</DropdownToggle>
							<DropdownMenu>
								<DropdownItem>
									Invisible text
								</DropdownItem>
							</DropdownMenu>
						</UncontrolledDropdown>
					</Nav>
				</Navbar>

bootstrap version #x.x.x
What? I use Reactstrap instead of bootstrap, right?

reactstrap replaces bootstrap.js, not bootstrap css. You still need bootstrap to for styles.
Here is what the code you provided in action, I can see the "Invisible text"
https://stackblitz.com/edit/reactstrap-v6-bu8xia?file=Example.js

reactstrap replaces bootstrap.js, not bootstrap css. You still need bootstrap to for styles.

Right. I had https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/css/bootstrap.min.css included. I had totally forgotten about that. I was searching through my npm imports.

I can see the "Invisible text"

Even with the older Bootstrap css I was using, The stackblitz example still looks correctly. Something else must be wrong in my setup. I'll dig deeper.

I found the difference. I was using a NavLink inside the DropdownItem. That made it invisible. It outputs a link inside a button, which is clearly wrong.
some example of how you shoul use links in menues would be helpful. I opted for <DropdownItem tag="a" href={href}>. But that was not very obvious.


Removing ValidateSet for Location

Get locations from get-AzureLocation

Hi @SiddharthChatrolaMs, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


Changing Accent color in VB.Net

Hi, I'm trying to make an option in my application to let the user choose the accent color but I don't know if it's possible...
How can I change the accent color directly from a button for exemple (In VB.Net) ?
Thanks,
Clément.
EDIT : I must learn to read :p It is a ChangeAppStyle which can be used easily... It's solved :)
EDIT2 : It only changes MahApps accent, not Material accent :/ It's not solved

UP :) Is there a way to change Material Design accent in VB.Net (for exemple with a button) ? Thanks :)

Hi, yes there should be a way: all themes are build using dynamic resources so they can change at runtime...but I don't have an example right now.  I'll have a look when I look at your other bug, hopefully this week, but am gonna prioritize your other bug first.

Okay thank you @ButchersBoy :) Your work is amazing :)

If you like it, just hit the :star: button :wink:

Done ;)

Legend :+1:

Nearly there...got a bit of tidying up to do before I make it available...

OK.  First cut of palette selection is in.

You will need to update MaterialDesignColors
But MaterialDesignThemes now has a (nuget) dep on MaterialDesignColors...so just update MaterialDesignThemes
There is no palette control: there are helper classes SwatchesProvider and PaletteHelper.  There is a window in one of the test projects which shows how to build a basic palette selector.  Maybe I will also build one as part of the lib in the future.

...let me know - once you've plugged this in - how your dark theme issue is.

Hi. It would be nice to add a function like this new PaletteHelper().ReplacePrimaryColor (ColorName); where ColorName = "blue" for an example. Is it possible?
P.S. Thank you for your work!

yeah can probably do that...

Thanks for your amazing work ! :) But how to change the theme now ? If I do something like
MahApps.Metro.ThemeManager.ChangeAppTheme(Application.Current, "BaseDark")
It's like before, with the bug :/

And you are changing the Material theme to dark at the same time using new PaletteHelper().SetLightDark(isDark);?
Now I've got the API in place, can you post a little demo project?  It will save me some time, then we can get the bug cleared up once and for all...

You mean something like this ?
MahApps.Metro.ThemeManager.ChangeAppTheme(Application.Current, new PaletteHelper().SetLightDark(isDark)) ?
I'm in VB.Net, I'm not really good to do the translation between C# and VB.Net, I'll create a sample project :)

Yeah, post a VB project, I'll figure it out, and get the TimePicker working.

This is a little sample I just made : http://www.mediafire.com/download/6mfzie0arh7o4s2/MaterialTemplateProject.zip

OK, I'll have a look over the weekend and add suggestion by @KlioHome

I precise that MahApps.Metro.ThemeManager.ChangeAppTheme(Application.Current, New MaterialDesignThemes.Wpf.PaletteHelper().SetLightDark(isDark:=True)) isn't working :/

Yeah, think you've misunderstood that one, you should do:
MahApps.Metro.ThemeManager.ChangeAppTheme(Application.Current, "BaseDark")
New MaterialDesignThemes.Wpf.PaletteHelper().SetLightDark(isDark:=True)

I got few errors : http://i.imgur.com/T8lFVgT.png (Syntax error here)
http://i.imgur.com/onndNoE.png (A class can't be used as an expression)

OK...I can't look today, but I'll have a look at your test project over the weekend.

I tried another code, I don't have any error at the building but I got an error when trying to change the theme

Here is the error : http://pastebin.com/fqJLMByX

@KlioHome you can no do new PaletteHelper().ReplacePrimaryColor("Blue");

@furiousbuilder We are finally in business with this one :smile:

The MD Toolkit PaletteHelper will now automatically detect MahApps styles and change them for you, so in line with that, and other recent changes, I've updated your code-behind and app.xaml, as per this gist (remember to update from nuget first).
If you're happy with this please close off this issue so I know we're done.

@furiousbuilder, thank you so much! It's amaizing!!!

Is this possible to have the sample you just modified please :) To understand which lines you modified :)
Thank you

Using you code, I have errors :/

The error : http://pastebin.com/8cxHqP1j

The problem occurs when I add this to my app.xaml :

Works on my PC :wink:
Uploaded code here: https://github.com/ButchersBoy/MaterialDesignInXamlToolkit_Issue36

Use a full pack URL:
<ResourceDictionary Source="pack://application:,,,/MaterialTemplateProject;component/Resources/Icons.xaml" />

Oh yes, thanks for your amazing work ! :D

I don't know if it's a bug but now, Flyouts with theme "Accent" are whites, like with this flyout :
Controls:Flyout x:Name="ParamFlyout" CloseButtonVisibility="Hidden" Header="Paramètres" Position="Right" Width="350" AnimateOpacity="True" Theme="Accent"
I need to set the background manually

I think there is a bug : with this version now, when you create a Flyout with "Accent" theme it appears White in Light mode and dark in Dark mode, so the accent isn't used :

Ok, can you raise a superstar issue for this? I have similar issue with MahApps dialogs, and I am starting to prioritise now what goes into version 1.0. MahApps integration issues may go into 1.1 so I can finalise the core Material stuff. Also, if you update the test project with a pull request showing the issues we can work on and test the solution a lot quicker when I get to it.

Hi, I am also building my VB.net application using this great work, and I also want to change accent color. But when I built the test application listed above(https://github.com/ButchersBoy/MaterialDesignInXamlToolkit_Issue36) using VS2013, I got this error
The project currently contains references to more than one version of MaterialDesignColors, a direct reference to version <IP_ADDRESS> and an indirect reference (through 'MaterialDesignThemes.Wpf.PaletteHelper.ReplaceAccentColor') to version <IP_ADDRESS>3. Change the direct reference to use version <IP_ADDRESS>3 (or higher) of MaterialDesignColors.
But in VS2017 the test application works fine. How can I fix this?
Thanks for your attention.


[WIP] Bz1327133 - Remove the "Required" condition for AMQP hostname and security protocol

https://bugzilla.redhat.com/show_bug.cgi?id=1327133

In WIP right now, since we are expecting more UI changes.
/cc @blomquisg @Ladas @juliancheal

Closing this in favor of https://github.com/ManageIQ/manageiq/pull/8231


Npm run build issue

Hi i have a React application which is running fine on "npm start".
But when i try to create a build for prod deployment, its taking too long to prepare the build and finally error is thrown.
Tried below options too:

re install the node modules
Cleared the cache
new install
and also tried
updating NPM to the latest version.

running the application form fresh folder with all new NPM installations.
nothing seems to be helpful to resolve the issue,
i am kind of stuck with this and any kind of Help or suggestions are really helpful and appreciated.
Error reported:

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory

Complete error details:
<EMAIL_ADDRESS>build E:\Santosh\Remitrix\Web-UI\Remitrix_Datta
npm run build-css && react-scripts build
<EMAIL_ADDRESS>build-css E:\Santosh\Remitrix\Web-UI\Remitrix_Datta
node-sass-chokidar --include-path ./src --include-path ./node_modules src/ -o src/
Wrote 2 CSS files to E:\Santosh\Remitrix\Web-UI\Remitrix_Datta\src
Creating an optimized production build...
<--- Last few GCs --->
[48072:0000024CAA06F980] 3178097 ms: Mark-sweep 1427.9 (1530.0) -> 1427.9 (1526.0) MB, 1909.8 / 0.0 ms last resort GC in old space requested
<--- JS stacktrace --->
==== JS stack trace =========================================
Security context: 000001E058EA5879
1: /* anonymous /(aka / anonymous */) [0000006130A022D1 :~8150] [pc=000000CE8067D72D](this=0000006130A022D1 ,self=000001964D603741 <AST_Call map = 000002D0511C6779>,compressor=00000147A5802979 )
2: optimize [0000006130A022D1 :~5511] [pc=000000CE8074ECA4](this=000001964D603741 <AST_Call map = 000002D0511C6779>,compres...
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
1: node_module_register
2: v8::internal::FatalProcessOutOfMemory
3: v8::internal::FatalProcessOutOfMemory
4: v8::internal::Factory::NewCodeRaw
5: v8::internal::Factory::NewCode
6: v8::internal::modulo
7: v8::internal::compiler::ControlFlowOptimizer::TryBuildSwitch
8: v8::internal::ZoneVector<v8::internal::compiler::MoveOperands * __ptr64>::ZoneVector<v8::internal::compiler::MoveOperands * __ptr64>
9: v8::internal::compiler::Pipeline::AllocateRegistersForTesting
10: v8::internal::compiler::ValueNumberingReducer::operator=
11: v8::internal::CompilationJob::FinalizeJob
12: v8::internal::compiler::InstructionSelector::isolate
13: v8::internal::Compiler::FinalizeCompilationJob
14: v8::internal::OptimizingCompileDispatcher::InstallOptimizedFunctions
15: v8::internal::StackGuard::HandleInterrupts
16: v8::internal::wasm::WasmOpcodes::TrapReasonMessage
17: 000000CE7F9043C1
npm ERR! code ELIFECYCLE
npm ERR! errno 3
npm ERR<EMAIL_ADDRESS>build: npm run build-css && react-scripts build
npm ERR! Exit status 3
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR! absolute path\2018-07-04T13_53_48_536Z-debug.log

Thank you for the reply.
But still the same issue. one thing i can observe is time to throw the error has increased.

meanwhile tried with latest web pack installations,
added the mentioned suggestion #4536 in config file too. still issue is there.

[["webpack-dev-server": "node --max_old_space_size=4096 node_modules/webpack-dev-server/bin/webpack-dev-server.js", "webpack": "node --max_old_space_size=4096 node_modules/webpack/bin/webpack.js"]]
tried on a MAC machine too. exact issue is reported there also. was trying my luck. :)
still the same issue is there

Maybe see if using the latest alpha would help? https://github.com/facebook/create-react-app/issues/3815
You'll no longer need to run the build-css step as a bonus.

yeah tried updating react-scripts too. no luck.
by the way same application is Built successfully on my colleges Linux machine.
Mine is windows 64 bit with 8 GB RAM.
we are removing that Linux machine, So wanted to start the build and deploy from windows. this is where the problem started.

Hey! It sounds like you guys have an incredibly large bundle.
If you introduce some code-splitting into your application, it will build no problem.
As a last ditch effort, you can disable source maps by setting GENERATE_SOURCEMAP=true in your .env file.


Reverse-mode autodiff didn't throw error with invalid input shape for API tf.reduce_max as expected

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
tf 2.9
Custom Code
Yes
OS Platform and Distribution
Linux Ubuntu 20.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Reverse-mode autodiff didn't throw any error with invalid input shape axis.shape=(1,1) on line 13 in the code below.
Forward-mode autodiff threw the ValueError as expected. So the behaviour for tf.GradientTape should be the same but not.
Standalone code to reproduce the issue
import tensorflow as tf

input = tf.constant([0.9165313, 0.9094733], shape=(2,), dtype=tf.float32)

# axis is with invalid shape
axis = tf.constant([[0]], shape=(1, 1), dtype=tf.int32)

res = tf.reduce_max(input,axis=axis,keepdims=False)
print(res)
with tf.GradientTape() as g:
    g.watch(input)
    # should threw ValueError here, but it didn't
    res_backward = tf.reduce_max(input,axis=axis,keepdims=False)
print(g.gradient(res_backward,input))

with tf.autodiff.ForwardAccumulator(input,tf.constant([1.,0.])) as acc:
    res_forward = tf.reduce_max(input,axis=axis,keepdims=False)
print(acc.jvp(res_forward))

Relevant log output
ValueError: Shape must be at most rank 1 but is rank 2 for '{{node gradient_tape/Sum}} = Sum[T=DT_FLOAT, Tidx=DT_INT32, keep_dims=false](gradient_tape/Cast, inputs_1)' with input shapes: ?, [1,1].

@sushreebarsa Hi, as you replicated, the ValueError is thrown by forward-mode auto differentiation at line 17. But the input shape [2] is actually invalid so it should throw the same error in reverse-mode at line 13, that is
res_backward = tf.reduce_max(input,axis=axis,keepdims=False)

I think it was caused by some missing input validation in reverse-mode autograd.

Forward and backward gradients are computed in different ways and that is the reason you are seeing different behavior.
Regarding invalid inputs, in gradient calculation, validating of inputs will not be performed in all the cases considering the computation cost which will effect the performance. Thanks!


[Documentation - Internationalisation] Add noRowsToShow

It's missing on the Internationalisation pag, and maybe some others ?

thanks for raising. will fix when i get back, should be no others, i didnt' intend leaving that one out.

thanks for raising. will fix when i get back, should be no others, i didnt' intend leaving that one out.

done


ecpdap: 0.1.5 -> 0.1.6

Also put the new udev rules in /etc/udev/rules.d

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thanks!
On Mon, May 3, 2021 at 5:49 PM Luke Granger-Brown @.***>
wrote:

Merged #121275 https://github.com/NixOS/nixpkgs/pull/121275 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/121275#event-4677228670, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGRJXGZDNHLOQ4X2MM6RHLTLZWO3ANCNFSM434JAKDA
.


Use released updates on virthost too

What does this PR change?
This PR adds the OS released updates to the KVM virtualization host too.
(BTW, the test suite module has no Xen virtualization host).

Waiting for @cbosdo 's blessing.


Add flake8-pep585 dev dependency

Add flake8-pep585 dev dependency so that our CI flake8 checks that no deprecated type from typing is used, as established by PEP 585.
Fix #1808.
CC: @severo

Codecov Report
Patch coverage has no change and project coverage change: -3.40% :warning:

Comparison is base (62c8ce4) 90.43% compared to head (6bc6de5) 87.04%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1809      +/-   ##
==========================================
- Coverage   90.43%   87.04%   -3.40%
==========================================
  Files         231      116     -115
  Lines       14267     4616    -9651
==========================================
- Hits        12903     4018    -8885
+ Misses       1364      598     -766

Flag
Coverage Δ

jobs_cache_maintenance
99.44% <ø> (ø)

jobs_mongodb_migration
85.95% <ø> (ø)

libs_libapi
88.14% <ø> (ø)

libs_libcommon
?

services_admin
87.51% <ø> (ø)

services_api
87.23% <ø> (ø)

services_rows
82.78% <ø> (ø)

services_search
79.16% <ø> (ø)

services_sse-api
94.34% <ø> (ø)

services_worker
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 115 files with indirect coverage changes

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

Yes, @severo:
src/rows/routes/rows.py:6:1: PEA001 typing.List is deprecated, use list instead. See PEP 585 for details


[stable/hl-composer] Update owners

What this PR does / why we need it:
Update owners for HL-Composer Chart
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

@nicolapaoli, pre-approving, but @dbandin and @inzamam-iqbal need to be member of the Helm organisation to be added as Owners

Hi @mattfarina please advise here.
OWNERS file is updated but it looks @dbandin and @inzamam-iqbal need to be part of the Helm organisation.

Just invited @dbandin and @inzamam-iqbal to be collaborators on this repo

/verify-owners

Aha, great, @inzamam-iqbal worked, but still waiting for @dbandin. @dbandin, please let me know as soon as you join and I'll re-verify all PRs.

@prydonius, thanks so much for your swift help.

Hi @alexvicegrab, thanks for looking at this. Got the invite accepted.

/verify-owners


Tapping on the main menu doesn't start the game.

Probably something to do with upgrading from Unity 3.5 to 5.

The main menu scene isn't used.
For now we just load straight into the main scene.


v0.5.3 throws when executable path is nil

Background
This project is using wkhtmltopdf, so we don't have headless Chrome installed locally or in CI.
The Issue
Starting the application returns the following error:
** (Mix) Could not start application pdf_generator: PdfGenerator.start(:normal, []) returned an error: shutdown: failed to start child: PdfGenerator.PathAgent
    ** (EXIT) an exception was raised:
        ** (FunctionClauseError) no function clause matching in IO.chardata_to_string/1
            (elixir) lib/io.ex:461: IO.chardata_to_string(nil)
            (elixir) lib/file.ex:214: File.exists?/2
            (pdf_generator) lib/pdf_generator_path_agent.ex:63: PdfGenerator.PathAgent.raise_or_continue/1
            (pdf_generator) lib/pdf_generator_path_agent.ex:37: PdfGenerator.PathAgent.init_opts/1
            (elixir) lib/agent/server.ex:8: Agent.Server.init/1
            (stdlib) gen_server.erl:374: :gen_server.init_it/2
            (stdlib) gen_server.erl:342: :gen_server.init_it/6
            (stdlib) proc_lib.erl:249: :proc_lib.init_p_do_apply/3

It appears that File.exists?(path) throws when path is nil, so lines 62 and 63 here will always raise if wkhtml_path or chrome_path is nil.
Possible fix?
As an experiment, I resolved the error locally by switching line 63 to the following to coerce any nils to an empty string:
chrome_exists = File.exists?(options[:chrome_path] || "")

That way File.exists? will return false for nil paths, and raise_or_continue can do its thing and raise or not depending on configuration.
If this seems like a reasonable way to get around the File.exists? nil problem, I can open a PR.
Thanks!

Hey @radditude - thanks a lot for reporting! This is due to me not having a test in place for non-existing Chrome on path. Mea culpa.
Strigifyig via || "" seems fair enough. I'd gladly take a PR for this. Please do the same for the wkhtmltopdf_path as well.
Again, thank you for your collaboration!

Fixed in https://github.com/gutschilla/elixir-pdf-generator/releases/tag/v0.5.4

@radditude Thanks a lot for your contribution!


[#871] Add RetirePool generator and correctness property

In the process of implementing this I encountered a bug in hedghog's Gen
internals. We decided to not bother debugging this and to just switch the
generators and properties over to QuickCheck.
In this commit I duplicate the generator logic for QuickCheck, introducing a
number of new generator modules which have the suffix .QuickCheck. This allows
us to make a gradual transition over to QuickCheck, rather than forcing all
generators to be rewritten at once.
I also define shrinkers, and I use those & the generators in the RetirePool
property.
One notable design decision I've made is to define a generator and shrinker at
the type of SourceSignalTarget for POOL. This gives more minimal
counterexamples, since we are only concerned with the bits of the LEDGER trace
which are projected out to the POOL level. It also works around an issue I
encountered with the STS transition failing when I used forAllTrace.
Closes #871.

This looks good! A belated PR approval from me...


RuntimeError: Cannot find any model weights with vicuna

Does vllm supports vicuna? I check the doc and find it supports, but when I use:
python3 -u -m vllm.entrypoints.openai.api_server --host <IP_ADDRESS> --model /home/huggingface_models/models--lmsys--vicuna-13b-v1.5/snapshots/3deb0106f72a3a433f0c6ea0cb978bdf14bcd3a6 --tensor-parallel-size 2 --load-format safe-tensors --port 8401
It returns:
RuntimeError: Cannot find any model weights with /home/huggingface_models/models--lmsys--vicuna-13b-v1.5/snapshots/3deb0106f72a3a433f0c6ea0cb978bdf14bcd3a6``
In /home/huggingface_models/models--lmsys--vicuna-13b-v1.5/snapshots/3deb0106f72a3a433f0c6ea0cb978bdf14bcd3a6, I have

Do you know what is wrong?

I think the problem is you are using --load-format safetensors. It requires some files with .safetensors.

I think the problem is you are using --load-format safetensors. It requires some files with .safetensors.

Thank you so much!


Fix CI/CD pipeline for docker

:(
something is wrong with 'volumes' on docker-compose and trying to test it with GitHub Actions. Must be because its on my local device, and not on the repo itself.

TBH, for a small scale project, CI/CD is not necessarily needed. Honestly, might've wasted 2 hours of my time doing this, but hey, I learnt something

Can be considered as a "future implement"


Document technical support roadmap

Who currently provides technical support for this product?
Until when will they be supporting this product?
Who if anyone will support it after that time?

@irtazabarlas-pif

@irtazabarlas-pif will provide the technical support at the code level till his term-limited assignment is valid as Presidential Innovation Fellow.  At present, it may be valid till Sept 30, 2021.
Given the scope of the application and its relationship to the current pandemic (Covid-19), it is not expected that this application will be needed after Sept 30, 2021.

closed by https://github.com/presidential-innovation-fellows/covid-screener-federalist/pull/3


How can past events be kept in the calendar without marking them Done?

Hello everyone,
Currently an event only gets added to the calendar if it's a todo, later, now type of tag. This is not a problem, the real problem is that if we don't mark these todo tags as done they'll be removed from the calendar view and be put on the top of the calendar. How can we keep them in their right time slot for past reference and organization?

Is there a way to solve this?
Thanks very much! This is an incredible plugin 🥇

In my opinion, there are three situations for overdue tasks.

If this task doesn't need to be completed, then I will set it as CANCELED.
If this task needs to be completed at an uncertain time in the future, I will remove its time information and put it into backlog.
If this task needs to be completed in the near future, then I will rescheduled it.

If a task has been overdue for a long time, It's likely not important or at least dose not require immediate attention.

Agenda3 has a brand new calendar, you can give it a try.

Agenda3 has a brand new calendar, you can give it a try.


tests_l2: Fix typo in readme

tests_l2: Fix typo in readme
For sgx l2 test yaml
Signed-off-by: vbedida79<EMAIL_ADDRESS>

Closed. Addressed in https://github.com/intel/intel-technology-enabling-for-openshift/pull/133


Avoid empty fallback list for cudnn v8.2

This PR bumps up the cudnn frontend repo commit to v0.2 to resolve the issue of returning empty fallback list when cudnn == 8.2. In addition, the patch file is reformatted to contain more info.
https://github.com/NVIDIA/cudnn-frontend/releases/tag/v0.2
cc. @nluehr

I think this PR can be replaced by https://github.com/tensorflow/tensorflow/pull/49244 which upgrades the cudnn frontend to v0.3 and adds some related code change. So closing this one.


is_location method

I've added a new method to the controller to check the current location of the webdriver instance, including a timeout and error-out capabilities.

Codecov Report

Merging #5 into master will decrease coverage by 0.6%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
- Coverage   78.22%   77.61%   -0.61%
==========================================
  Files           5        5
  Lines         326      335       +9
==========================================
+ Hits          255      260       +5
- Misses         71       75       +4

Impacted Files
Coverage Δ

pyscc/element.py
81.95% <ø> (-0.52%)
:arrow_down:

pyscc/controller.py
67.27% <60%> (-0.06%)
: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 d56c279...b3e37d2. Read the comment docs.


Reties With not empty URL Path

transport builds an incorrect request path if the address is configured with a path, for example:
	eCnf := elasticsearch.Config{
		Addresses: []string{
			"https://server.com/somepath"
		},
	}

it happens when retries is enabled and the API returns an error, transports adds to the request "/somepath" on each retry instead of building a new path.

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

CA is signed, but I don't see any triggers to rerun the check

Hi @f-dg
Thank you for noticing this, LGTM!
I'll push a minor change and adapt a test to reflect the fix.


error for generater target textmate

When I try to run the following command, i get for any of the examples in the textx reqro the following error.
C:\local\textX-LS\examples\strictdoc>textx generate tx_strictdoc/strictdoc.tx --target=textmate --name tx-strictdoc
Generating textmate target from models:
C:\local\textX-LS\examples\strictdoc\tx_strictdoc\strictdoc.tx
Traceback (most recent call last):
  File "C:\Python38\lib\runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Python38\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Python38\Scripts\textx.exe\__main__.py", line 7, in <module>
  File "C:\Python38\lib\site-packages\click\core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "C:\Python38\lib\site-packages\click\core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "C:\Python38\lib\site-packages\click\core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "C:\Python38\lib\site-packages\click\core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "C:\Python38\lib\site-packages\click\core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "C:\Python38\lib\site-packages\click\decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "C:\Python38\lib\site-packages\textx\cli\generate.py", line 111, in generate
    model_params = {k: v for k, v in custom_args.items()
  File "C:\Python38\lib\site-packages\textx\cli\generate.py", line 112, in <dictcomp>
    if k in metamodel.model_param_defs}
AttributeError: 'TextXMetaMetaModel' object has no attribute 'model_param_defs'

dont know how to deal with this, if more information is needed just let me know.

Thanks for reporting. At first I thought it is a problem in textx-gen-coloring generator project, but after some investigation it seems to be a regression in textX. I'm working on a fix.


remove fallback date for nextRunAt

This just creates wrong data. Old jobs that have been executed already and will not be executed again appear as jobs that will run at the current date (as their date is null). It should just be null instead like it is written in the persistence layer. No need for a fallback.
Is this behaviour important for some other logic?

@fermentfan  As of now, deleting this value may cause side effects. Please post the specific case where deleting this is necessary in the issue, and I will take a look.

We simply query jobs like this:
'''
await this.pulse.jobs({ type: 'normal' });
'''
This returns a set of jobs, in which all jobs that originally have a null 'nextRunAt' in mongodb, have the fallback applied.


Created my profile.  data: Lakatabu01

Fixes Issue
Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

I will close this PR for the reasons mentioned above.
@Lakatabu01 if you would like to edit your BioDrop Profile then I would suggest doing so through forms.  I am including the link to the documentation here: https://www.biodrop.io/docs/how-to-guides/editing-forms

Okay, Thank you for the information
On Mon, 18 Dec 2023 at 10:35 AM, Sara Jaoude @.***>
wrote:

I will close this PR for the reasons mentioned above.
@Lakatabu01 https://github.com/Lakatabu01 if you would like to edit
your BioDrop Profile then I would suggest doing so through forms. I am
including the link to the documentation here:
https://www.biodrop.io/docs/how-to-guides/editing-forms
—
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/BioDrop/pull/9930#issuecomment-1859935763,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVWGK6DP5KREWFDUAZ7WAV3YKAE4VAVCNFSM6AAAAABAZDCXCKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJZHEZTKNZWGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Self test issue: JPP_0253

In some test cases of section "(PARAMETER_SUBSTITUTION)-[BADCASE]", e.g. the JPP_0253, some expressions contain
a not existing parameter (${IAMNOTEXISTING}).
Previously in test case JPP_0253 the JsonPreprocessor throws the following exception (like expected):
The variable '${IAMNOTEXISTING}' is not available!'
The latest version of JsonPreprocessor throws this exception instead:
''Could not overwrite parameter VAR-${variant_${list_variant_numbers.str(${list_variant_index})}}_MS-${milestone_${list_milestone_numbers.str(${IAMNOTEXISTING})}}_param due to wrong format.
           Please check key 'VAR-${variant_${list_variant_numbers.str(${list_variant_index})}}_MS-${milestone_${list_milestone_numbers.str(${IAMNOTEXISTING})}}_param' in config file!!!''

This makes no sense. This does not help. The previous error message was completely OK and should be restored.

Hi @namsonx ,
what is the status of this ticket?
Thank you,
Thomas

Solved and retested.


Modify bin/amigo.js to allow more than 1000 parameters.  Set it to 10…

…0000 and 50mb.
This fixes an issue in planteome/amigo where the API fails if the request contains more than 1000 parameters.

@elserj This is to limit input from users?

Yes, we would get a failure if the input list had more than 1000 parameters.  Tracked it down to the default of the body-parser.  This change ups it from the default to 100000, which should be plenty as I can't think of any species with more genes or an ontology with more terms that would be searched with the API.  I probably should have made an issue for this, but I was unsure where the problem was, ie. Apache, JS... until I fixed it.

Ah, so this is upping it. No problem there.

It would be good to catch exceptions generated from problems like this, but we can worry about that later (like when we start using this API as the JS backend for AmiGO, replacing the perl parts).


right panel improvements

next steps

alternate tabs styling (the underline only version)
make icons throughout open directly to the correct tab in the details panel

bors merge


fix: prevent block scope for init-referenced bindings

Previously we would turn e.g. var x = x; into a let (or const) declaration, but that introduces a TDZ violation since the initializer for the binding is inside the TDZ. We have to check that the initializer for the binding does not reference the binding being initialized.
I'm not entirely sure why babel-traverse does not count the xxx in var-with-own-binding-in-initializer as a referencePath, but it does show up in constantViolations so we use that too.
Closes decaffeinate/decaffeinate#724

Also, TIL Scope#dump() in babel-traverse.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Create Outreach Communities

Successfully we were able to help start and perpetuate a community for China for webpack. It's been extremely successful and we continue to see their traffic as 2nd to USA in docs, and repo page visitors.
However it came to our attention on twitter that we could maybe be doing more. I'm all for equality as access to opportunity, and furthermore since all of us have little to no bandwidth, we'd want to find a group of developers/advocates who are willing to lead the initiative. First step would simply be to create the guidelines for how one would approach this outreach.
https://twitter.com/truth2_truth/status/848567713798791169 <== this twitter account nominated themselves to write this guideline, or seems to have experience in it.

It sounds cool. However what does an outreach community do/accomplish? You mean trying to help more people learn about webpack?

@TheLarkInn  I would like to take this if no one has taken it, but as @IAMtheIAM pointed out what should  an "outreach community do/accomplish" ? Thanks

Ahhh sorry for the delay on this. Okay so this is about working to help elevate the opportunities of those who might not feel they are worthy or equipped to contribute, and to help them on board.
What's great is that if you have less experience it's a really great way to learn through teaching. I'll try and follow up with this some more.

Is this still active? I would love to know more about this, if it is still an issue!

I think we can do the same for India as well, I can see many Indian developers around.


Get rid of APIs related to pending transactions

When working on something else I came across those APIs that were only used in a couple tests
Removed them!

A pending transaction was one that is in a block which has not been mined/finalized yes?

Partially: in some cases they might not be in any block at all. Two kinds of relevant pending transactions that follow slightly different rules:

peer-received: This populates the txpool. Can be dropped across restarts, or by failing arbitrary (trinity-chosen) tests about gas price, etc.
locally-issued: A user requested to broadcast this transaction (eg~ via `eth_sendRawTransaction'). We likely want to persist these across trinity restarts, and want to keep these as pending regardless of the txpool garbage collection rules related to gas price, etc.

I think if so then I support removing these specialized getters assuming we can do what we need without them.

Because they might not be in any block, I think we need to keep them.

Maybe they were for looking up transactions by hash when they had not yet been included in a canonical block?

Right, it's valid to make a json-rpc call requesting the body of a pending transaction. Even if we punt for a while on doing this from the transaction pool, doing this for locally issued transactions early seems important.

Right, I understand this might be needed at some point, but right now there was nothing using add_pending_transaction() to store those transactions, and I believe it's better to re-add that whenever we need it than to maintain unused code. Not a big deal, though, so feel free to either merge or close this

I believe we'll deal with all of the use cases outlined by @carver with trinity specific code.


Walking paths absent from vector export

fixed with https://github.com/datadesk/web-map-maker/commit/c7577c882ebdc2c4c9d6808f3ce0fc544391d69d


Breaking change due to _check_import_id_fields

Describe the bug
This is a breaking change in 4.0 vs 3.x.
In my resource, I've declared a before_import_row() to add another column 'Object' to the row, which is used as an identifier.
object = resources.Field(column_name="Object", attribute="object")

def before_import_row(self, row, **kwargs):
....
        row["Object"] = hashlib.sha256((row.get("Object 1 Name") or "").encode()).hexdigest()
...

    class Meta:
...
        import_id_fields = ["time", "object"]

However, after upgrading to 4.0.2, the import throws an error
File "***/python3.12/site-packages/import_export/resources.py", line 1186, in _check_import_id_fields
raise exceptions.FieldError(
import_export.exceptions.FieldError: The following fields are declared in 'import_id_fields' but are not present in the file headers: Object

This check on headers seems not taking into accounts that the rows can change.
Versions (please complete the following information):

Django Import Export: 4.0.2
Python 3.12
Django 5.0.6

Can you see if this solves your issue:
    def before_import(self, dataset, **kwargs):
        dataset.headers.append("Object")
        super().before_import(dataset, **kwargs)

Thanks I ended up using the before_import to modify the whole dataset instead, but seems this behavior should be documented.


Revert "Improve @objcImplementation member checking"

Reverts apple/swift#63668

@swift-ci please smoke test

@swift-ci please smoke test Linux


Test formats and minor tweaks

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0747e41). Click here to learn what that means.
The diff coverage is 88.88%.

@@            Coverage Diff            @@
##             master       #4   +/-   ##
=========================================
  Coverage          ?   88.23%
=========================================
  Files             ?        2
  Lines             ?      102
  Branches          ?        0
=========================================
  Hits              ?       90
  Misses            ?       12
  Partials          ?        0

Impacted Files
Coverage Δ

Sources/Debugging/Debuggable.swift
93.33% <100%> (ø)

Tests/DebuggingTests/FooErrorTests.swift
80.95% <80.95%> (ø)

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 0747e41...0703e21. Read the comment docs.


Flutter SDK 3.10.5 not working because I don't have access to open Powershell.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
I am using Flutter 3.7.12, and it works without any problems.
However, when I update Flutter to version 3.10.5, it stops working, and running any Flutter command closes the command prompt.
I believe the issue is that I don't have permission to access PowerShell, as the IT team in my company has restricted its usage on company devices.
Is there an alternative method to use Flutter SDK 3.10.5 without relying on PowerShell?
Expected results
Flutter 3.10.5 work without problem.
Actual results
Flutter 3.10.5 not working.
Flutter Doctor output
Doctor output for version 3.7.12

[√] Flutter (Channel stable, 3.7.12, on Microsoft Windows [Version 10.0.19045.3086], locale en-US)
    • Flutter version 3.7.12 on channel stable at D:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4d9e56e694 (3 months ago), 2023-04-17 21:47:46 -0400
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at C:\Users\ashaker\AppData\Local\Android\sdk
    • Platform android-33, build-tools 33.0.2
    • Java binary at: C:\Program Files\Java\jdk-17\bin\java
    • Java version Java(TM) SE Runtime Environment (build 17.0.7+8-LTS-224)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.6.4)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.6.33815.320
    • Windows 10 SDK version 10.0.22000.0

[√] Android Studio (version 2022.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)

[√] VS Code (version 1.79.2)
    • VS Code at C:\Users\ashaker\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.68.0

[√] Connected device (3 available)
    • sdk gphone x86 64 (mobile) • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
    • Windows (desktop)          • windows       • windows-x64    • Microsoft Windows [Version 10.0.19045.3086]
    • Chrome (web)               • chrome        • web-javascript • Google Chrome 114.0.5735.199

[√] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

I believe the issue is that I don't have permission to access PowerShell, as the IT team in my company has restricted its usage on company devices.

@ahmed951
I think you need to reach out to your IT team for solution as I don't think there's anything to do from Flutter side.

Can be there an option to choose between PowerShell and CMD?

@ahmed951
The Windows installation document does refer to have PowerShell:
https://docs.flutter.dev/get-started/install/windows#system-requirements


[Dependency Scanning] Do not specify placeholder modules by default as inputs to scanning actions

Instead allow the dependency scanning oracle to be configured such that the scanning actions use placeholders.
In contexts where the driver's client can guarantee that all module dependencies have been built prior to planning of the dependee, placeholders are not necessary in the scanner, becuase the binary modules will simply be discovered on the filesystem.
Resolves rdar://102339586

@swift-ci test

@swift-ci test Linux platform

@swift-ci test


Fixed cannot unmarshal number into Go value of type gogs.commitDetail…

… error
Gogs commit fetch without Sha reference returns error.
Commit Checklist
Thank you for creating a pull request! To help us review / merge this can you make sure that your PR adheres as much as possible to the following.
The Basics

Commit is a single logical unit of work, only use multiple commits if doing different tasks
Commit does not include commented out code or unneeded files
rebase of main branch

The Content

Must include testing for bug or feature
Must include appropriate documentation changes if it is introducing a new feature or changing existing functionality
Must pass existing test suites

The Commit Message

Short meaningful description (ex: remove deprecated steps)
Uses the imperative, present tense: "change", not "changed" or "changes"
Includes motivation for the change, and contrasts its implementation with the previous behavior

The Pull Request

What is the reason for this change
Example usage of the failure for a bug, or configuration and expected output for a feature
Steps to test the change

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @losintikfos
Thank you for your submission, can you include a unit test for this change, please?

@eoinmcafee00 test coverage already exist TestFindRef(t *testing.T) {.... Obviously the coverage is generic as oppose to client driver specific calls.

Any chance this could make it in soonish? Or, is there a workaround?

This PR has been automatically closed due to inactivity.


http transport should not attempt to parse error bodies

When we get back an HTTP error in response to ?service=upload-pack, it appears that we attempt to parse the body instead of propagating the error. This results in weird "invalid pkt-len" errors instead of a useful 500 or 400 or whatever.

@taralx Thank you for taking the time to report this.
I'm closing this issue as duplicate of https://github.com/src-d/go-git/issues/322#issuecomment-292154719 since this is one of the few cases that we do not handle correctly.


feat: update currency rate controller polling input

Explanation
Prepares the currency rate controller to be polled across chains.  We want a unique polling loop for each unique native currency (e.g. ETH, BNB).  Rather than for each network client id.  So this PR updates the polling input to {nativeCurrency: string}
References
Changelog

@metamask/assets-controllers

BREAKING: The CurrencyRateController polling input is now {nativeCurrency: string} instead of a network client id.

Checklist

[ ] I've updated the test suite for new or updated code as appropriate
[ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate
[ ] I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

@metamaskbot publish-preview


Feat/lesq 435/share card container [LESQ-435]

Description

Create component and story for share card group
Refactor out some shared logic between download and share cards
Rename prop on resource card to be more generic between download/share

How to test

Go to https://deploy-preview-2085--owa-storybook.netlify.thenational.academy/?path=/docs/download-components-share-card-group--docs
Click on download components / share card group
You should see stories for a few different resources present

Screenshots
How it should now look:

Checklist

[ ] Should match figma design

:tada: This PR is included in version 1.298.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update usr.sbin.dnsmasq profile for network api

Possible solution:
add /var/lib/lxd/networks/** rw, to usr.sbin.dnsmasq profile
Without no network will start.
Long version:
Currently dnsmasq profile includes extra rights for lxd.
  # lxd-bridge pid and lease files
  /{,var/}run/lxd-bridge/dnsmasq.pid   rw,
  /var/lib/lxd-bridge/dnsmasq.*.leases rw,

Better solution will be to make #include <abstractions/lxdsomething> in upstream for dnsmasq (could take longer then expected), however lxd api is still only in git, and profile works ok for repository version obviously. Heh!
Another more raw, but simple solution will be to pack own dnsmasq profile with lxd or make posintall profile editing and reloading. Obviously fixing ppa package is not a problem at all, but this slight issue should be taken in count when new lxd with network api will come out in official repository. Since, I bet no one want to lose network in containers with upgrade. :8ball:
Info:
Ubuntu xenial and lxd from lxd-git-master from localhost:
 certificate fingerprint: d1f6f03fa58795d62a7141005e95d3b76af87e2e0c4ef07f45e018b24a0105e7
  driver: lxc
  driverversion: 2.0.4
  kernel: Linux
  kernelarchitecture: x86_64
  kernelversion: 4.4.0-41-generic
  server: lxd
  serverpid: 3322
  serverversion: "2.3"
  storage: zfs
  storageversion: "5"
config:
  core.trust_password: true
  storage.zfs_pool_name: lxd
public: false

The dnsmasq apparmor profile isn't shipped by default on Ubuntu systems. It's only present if the user manually installs the apparmor-profiles package.
Can you please file a bug about this at https://launchpad.net/ubuntu/+source/apparmor/+filebug and I'm sure someone on the apparmor team (who maintains that unofficial profile) will be happy to update it.

It's against Ubuntu and Debian packaging policies for a package to alter a file shipped by another one so there isn't anything we can do about this in the LXD packaging.


PEAR-1765 - Focus doesn't move to the toast message

Description
+ Also addresses PEAR-1726 - Keyboard focus not returning to the active element (table button element) after Summary Modal closes.
Checklist

[ ] Added proper unit tests
[ ] Left proper TODO messages for any remaining tasks
[ ] Scanned for web accessibility with aXe, and mitigated or documented
flagged issues

Screenshots/Screen Recordings (if Appropriate)

The focus moves to the toast message as expected, and the screen reader reads it out. Additionally, the focus returns to the selected button.
However, the screen reader reads the toast message twice. Since the time interval is not long enough, we only hear part of the second read. I tried to figure out why this was happening and how we could fix it, but I couldn't find a solution. I suggest approving it as is for now. After we push this to production, I'd like to test it with NVDA using Assistiv Labs.

HI @cemile-ctds, I looked into this and the root cause is the fact that the mantine notifications have role=alert on them. That is a correct/appropriate role for a notification but it does seem to have a known problem of screenreaders reading content of a focused alert twice (see https://webaim.org/blog/aria-cause-solution under "role=alert"). Not sure what the best fix here is 🤔

The screen reader reads the toast message, and it works fine on my end.
I'll be opening a new ticket for the 508 violations related to the actionable elements in the toast message.


Boilerplate: Update to 946651c6eb6934f25ff1c80d8f99dc07eea3ff85

Conventions:

openshift/golang-osd-operator: Update

https://github.com/openshift/boilerplate/compare/3bf1715324ffdacc05ac888f039d0ae9334c9acc...946651c6eb6934f25ff1c80d8f99dc07eea3ff85
commit: 7e944022e0b144fed844626bd2ef163b50e8f268
author: Eric Fried
Minor fixups
A grab-bag of small nonfunctional changes.

Add some more formatting to the output of
openshift/golang-osd-operator's update script.
Consistent error formatting in prow-config (fup from
https://github.com/openshift/boilerplate/pull/75#discussion_r507851220)
Rename a handful of test case files from snake_case to kebab-case for
consistency.
Get rid of trailing whitespace, which shows up in annoying ugly red in
diffs when consumers onboard.

commit: f5b3cf91cd8c7637136d8cad0b5932788a2dc0d5
author: Eric Fried
README: Add a "Quick Start" section
commit: b5b9ec147f85ab6f3003864714066465b090bcea
author: Eric Fried
Run image build in a temp directory
The build script was valiantly trying to clean up after itself, but was
leaving a few things behind:
sh-4.2$ git status
On branch master
Untracked files:
  (use "git add <file>..." to include in what will be committed)
	config.log
	config.mak.autogen
	config.status
	git.tar.gz

...which were causing CI failures when they ended up in the image used
for pr-check.
This refactor moves all the processing to a temp directory which it
blows away when done.
commit: d82ff2ab073455b0c2a81a82fc2ac4ba6828ee57
author: Eric Fried
Set up in-repo prow build root
commit: 263bf0b41ad795368c087e8b8fec2db9f5c60e07
author: Eric Fried
openshift/golang-osd-operator: prow config script
Add a script to generate standard prow configuration for
openshift/golang-osd-operator subscribers in an openshift/release clone.
Invoke as
make prow-config
or
make RELEASE_CLONE=/path/to/existing/clone/of/openshift/release prow-config
Jira: OSD-5421
commit: 14b7dc1dfa8b4ce3078f1c4551c53b78b5bbae9c
author: Eric Fried
Fix bracing in push.sh
D'oh.
I copied the skopeo commands from
the make target
and I didn't notice that the variable interpolation was using
parentheses rather than curly braces.
Fixed.
And since this messes with the build config, I need to publish a new
image tag. (We'll never use image-v0.1.0.)
commit: 6dddc7040d9299961a6c538c9cb6b7a05577cdb4
author: Eric Fried
Consumers build/test with tagged images
The openshift/golang-osd-operator convention now syncs the image used by
prow and appsre by editing build/Dockerfile and .ci-operator.yaml. The
image tag used in both is the latest image-v* tag from the boilerplate
repository.
commit: 3fbe8859e36c7c377b8d9407886176bfb61d9331
author: Arjun Naik
Added variable for git hash
Signed-off-by: Arjun Naik<EMAIL_ADDRESS>commit: 6be78b08d4db67808b28b7435e0057c5a9dc97e1
author: Arjun Naik
Added a missing double quotes
Signed-off-by: Arjun Naik<EMAIL_ADDRESS>commit: cd79e932726e39ee96d03be44be9fe74e3d777a7
author: Arjun Naik
Fixed the build and push with skopeo
Signed-off-by: Arjun Naik<EMAIL_ADDRESS>commit: aafa039ecc2bb506c38b29fd3bbd7f31819bf52c
author: Rob Rati
Updates to standard.mk to support building on other platforms and broke down operator IMG into usable sub-components
commit: ee247ba8d7b1b92fa17776d027847aeb819008ea
author: Eric Fried
Use 7 characters for the commit SHA
...to align with what appsre is using.
commit: 93b5e68b8695162780ea77226e62f4e827a2c992
author: Eric Fried
Add in-repo prow build_root configuration
commit: f92583fc3a06e517f9aaf08e57f6b94aa4c67744
author: Eric Fried
Move generate-check under validate
...as was originally intended (see the referenced Jira).
Jira: OSD-4673
commit: 3b93083cf02aecf21db7420f220e5f3a6168a7dc
author: Eric Fried
Make the default target work
Some versions of make don't honor the default target. Others don't
pay attention to .DEFAULT_GOAL. Fix standard.mk in the
openshift/golang-osd-operator convention to accommodate all of them.
commit: 043bae4f0d6907fe3fb1c8457a4e62b069861f06
author: Eric Fried
Make make validate include freeze-check
...to ensure boilerplate-owned files are not changed.
This adds a make target in the boilerplate framework itself called
boilerplate-freeze-check and includes it in the chain of the
validate target in the openshift/golang-osd-operator convention.
Jira: OSD-5366

100% generated PR. Highlights:

Start using boilerplate-produced backing image for checks & builds.
make validate prow check will now forbid you from editing boilerplate-owned files.

Some questions to @2uasimojo  on the boilerplate image, but not blocking this PR.  Tracking those elsewhere.
/lgtm
/approve

Naveen approved above, not sure why the label didn't appear. So I added it.


Reducing the Secant Function X calculation

Looking at the code here, we want to reduce the duplication of code but we need to untangle the interaction of the left-and right-most secant calculations and Q in the various iterations of the MC calculation.
Three questions to work on --

why the limits on X (we suppose this is just a matter of the textbook explanation of what X is in the first place)?
why the different limits on X for right side vs. left side of secant calculation?
why the lagged use of Q from previous cycle calculation?

https://github.com/NOAA-OWP/t-route/blob/63e1578af5edf7086df7a4106314d5f31bfdbc71/src/fortran_routing/mc_pylink_v00/MC_singleSeg_singleTS/MCsingleSegStime_f2py_NOLOOP.f90#L276-L299
@dnyates @groutr

For reference, there could be additional context for question 3 here:
https://github.com/NOAA-OWP/t-route/blob/63e1578af5edf7086df7a4106314d5f31bfdbc71/src/fortran_routing/mc_pylink_v00/MC_singleSeg_singleTS/MUSKINGCUNGE.f90#L152-L160
and here:
https://github.com/NOAA-OWP/t-route/blob/63e1578af5edf7086df7a4106314d5f31bfdbc71/src/fortran_routing/mc_pylink_v00/MC_singleSeg_singleTS/MUSKINGCUNGE.f90#L222-L233

See additional documentation in #287


Update ClustResult struct

Updated ClustResult, ClustResultAll, AbstractClustResult
Updated constructor for ClustData to have delta_t as an optional input argument, similar to the constructor for ClustDataMerged
Updated run_clust() and run_clust_method() to have them more structured by functionality
Fixed bug in run_clust() -- output of ClustResultAll didn't work before.

If #83 is an easy fix ( move round.() from run_clust to CapacityExpansion into the load data process), could you add it there? If it somehow causes trouble to move, let's just keep it as is.
Have a look at this one, and after we've merged to dev here, I'll make a PR on CapacityExpansion.jl to adjust.

Adresses #85 , test cases will need to be written in more detail in future. Right now, it just tests if the functions don't break.


adding reference to trained checkpoint in GUI

🔧 Proposed code refactoring
Currently, it is possible to start a fine-tuning job based on the last experiment's checkpoint by toggling an option in the GUI. However, I noticed that if I do that more than once(i.e. I create more than 1 'new' experiment from a "base" fine tuned experiment) the new experiment only loads the weights from the backbone and not the checkpoint. This also happens if I update the original data source in the GUI to another fine-tuning dataset. In these cases, I have resorted to using the CLI and hard-coding the checkpoint I want to start my training from. I think it would be a good addition to the GUI to allow us to specify the specific checkpoint instead of just having a toggle button to prevent this error.
Motivation
see above.

Thanks @Franjcf - I will look into it.
In the meantime, you can use the following setting:

There you can specify a link to a previous checkpoint

Hi psinger. Thanks for looking into this so quickly. Makes total sense, I was not aware of this field. I guess I was just confused because the interface changes a bit if we initialize it from a finished experiment (as opposed to a completely new experiment.) I apologize for the oversight.

Pretrained Weights also solves this.


[ros2] Update documentation for installation instructions and bridge examples

🦟 Bug fix
Summary
Few minor changes to installation instructions and examples for ROS 2. Tested on Ubuntu Focal, with Edifice built from source (Docker).
Installation instructions:

rosdep install complains if Ignition libraries are included in requirements

Skipping keys for Citadel, Dome and Edifice versions of the required libraries (I wish there was a wildcard support)

Both README.md and ros_ign_bridge/README.md contained installation instructions

Removed instructions from ros_ign_bridge/README.md
Note: Please let me know if there was a reason behind this. If so, we can revert the change and update the instructions there so that they are correct

Examples:

Source command of ROS 2 workspace was using invalid ROS distribution (melodic instead of foxy)

Fixed

Bridge example for image transport contained a screenshot for the example using ROS 1

Replaced with ROS 2 alternative

Checklist

[x] Signed all commits for DCO
[x] Updated documentation (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)

Note to maintainers: Remember to use Squash-Merge

Note: Added one more commit into this PR to remove FluidPressure TODO from support matrix. Demo for it was enabled and tested in https://github.com/ignitionrobotics/ros_ign/pull/144, but I added the commit here to avoid merge conflicts.


Change the jsonlbundle output date format to YYYY/MM/DD

This PR changes the output format of the path from jsonlbundle to the same format used by the existing datatypes (YYYY/MM/DD).
I can move any files with the old format in pusher-mlab-standbox and pusher-mlab-staging to the new path.

This change is

Coverage: 85.447%. Remained the same when pulling c0ed3993603305256589f227de30b77ca3687cba on sandbox-cristinaleon-date into b8aac83ec2ad81239301eb1b61ba94ebc7e3216f on main.


Can I live stream my web cam or live video ( HLS, RTSP and RTMP) to gemini vision api?

Description of the feature request:
No response
What problem are you trying to solve with this feature?
No response
Any other information you'd like to share?
No response

@vishwakneelamegam correct. The API supports streaming output, but no streaming input. This is impossible right now.

I have made a workaround with Open AI https://github.com/vishwakneelamegam/openai-video-narrator.


power bi dashboard embedded not working on mobile browser

Hi,
I have tried these code samples, those are working fine on web browser. When it comes to mobile browser not working. May I know is this support for mobile browsers ?
Thanks,
Infaaz

what do you mean by "not working"?

closing as no response.


Newly added data not being found

Hi, First of all thank you for the great package. I am encountering a problem. Once I add a new record to the table on which I am searching in, it is not being found. Is there some sort of re-indexing I need to do or something?
Thanks
Jason

I solved it. It was an issue from my end. Thanks


feat: add unit tests for ModifyOrder and SessionInvalidate

Add unit test for InvalidateAccessToken and update test response for ModifyOrder.
@rhnvrm Can you update the go kc mock submodule with the latest version?  session_logout.json and order_modify.json files are missing in the current mock.

@ranjanrak Feel free to rebase, I have updated mock_responses on master

LGTM


Can't parse alt_std_dev from GST correctly

Hi all,
Thank you for making this great parse driver. The code is clean and works well.
Recently, I have been trying to retrieve the information in GST. I found that the value of alt_std_dev is always nan, and I added some log to figure out what happened.
Here is where I put the log in parse_nmea_sentence() in parser.py:
if sentence_type == "RMC":
    parsed_sentence["utc_time"] = convert_time_rmc(fields[9], fields[1])

# log for check GST
if sentence_type == "GST":
    rospy.logwarn(
            "nmea_sentence: %s" % nmea_sentence)
    rospy.logwarn(
            "parsed_sentence: %s" % parsed_sentence)
    rospy.logwarn(
            "fields: %s" % fields)

return {sentence_type: parsed_sentence}

And following is the output message:
[WARN] [1576155838.636761]: nmea_sentence: $GNGST,130358.60,36,,,,0.046,0.041,0.057*43
[WARN] [1576155838.638862]: parsed_sentence: {'semi_major_ellipse_std_dev': nan, 'alt_std_dev': nan, 'lon_std_dev': 0.041, 'semi_major_orientation': nan, 'ranges_std_dev': 36.0, 'semi_minor_ellipse_std_dev': nan, 'lat_std_dev': 0.046, 'utc_time':<PHONE_NUMBER>, 600000000)}
[WARN] [1576155838.641648]: fields: ['$GNGST', '130358.60', '36', '', '', '', '0.046', '0.041', '0.057*43']

It seems like the [*CheckSum] value should be removed before the parse to get correct alt_std_dev. Or did I do something wrong? Any advice would be appreciated.

Thanks for pointing out this bug in the parser. I haven't tried to recreate the error yet, but I took a stab at fixing it. Can you try out the change in pull request #87?

Hi @evenator,
Thank you for fix this so quickly. However, I don't have time to test it recently.  :(


RFC: blueprint sanity tests

From https://raw.githubusercontent.com/weldr/lorax/master/tests/cli/test_blueprints_sanity.sh:

list all blueprints & verify expected examples are there
save a blueprint & verify it creates .toml file in the local directory, toml file contains blueprint name, some expected packages
saving a non-existing blueprint exists with 1
push new blueprint - verify it is found in the output of blueprints list.  Also verified against internal location
blueprints show - verify that output matches the toml file contents verbatim
Start with a blueprint version 0.0.1, append a change to the toml file and push it again. Verify version is 0.0.2 - that is version was incremented automatically
blueprints delete - verify list doesn't contain the deleted BP, also verify against internal location
push a BP, delete it and then start a compose with the deleted BP name - expect it was not possible to do this, exit code 1. IIRC this exists because of a bug which allowed starting composes with deleted blueprints

Note: in osbuild-tests there's hardly any coverage for blueprints. I still haven't examined API tests in details.

I think, again, the behaviour here should be mostly verified in the API tests, which @bcl is looking into. We may do some more basic sanity tests in the cli tests, just to make sure the calls work at all (but I would not verify the details of the behaviour).

@teg I agree basically with all you said - checks on API level that actually cover the blueprints functionality seem fine to me, but we should also do at least some basic test for blueprint handling for composer-cli just to check it handles blueprints as expected.

https://github.com/osbuild/osbuild-composer/blob/master/internal/client/blueprints_test.go should cover all of these, I think.

@bcl Yes, blueprints_test.go covers this area thoroughly, but compared to the test we have for lorax-composer, it doesn't cover the integration with composer-cli. We need to cover composer-cli as well, as it's one of the user-facing tools to directly interact with osbuild-composer.
The only questions (IMHO) are if we should include those tests in the osbuild-composer test suite and how/when we should execute them - I think we should include them in osbuild-composer integration tests :).

@jikortus  see cmd/osbuild-tests/main_test.go - there are multiple commented out lines for the blueprints commands. The only one which is implemented is "blueprints list".
That's the place to add integration tests which go through the cli. There is very little (almost none) checking as to what the actual output of these commands is because that is verified in other layers of the test suite. If the commands don't error out then we accept that everything is fine.
See func TestSources in that file for an example of a group of tests which need a common setup.

IMO the remaining composer-cli testing is covered in https://github.com/weldr/lorax/pull/1007 and there is also test coverage in https://github.com/osbuild/osbuild-composer/blob/master/internal/client/blueprints_test.go.
The remaining/commented out sections of cmd/osbuild-tests/main_test.go will close any gaps. I will open a separate issue for the commented out sections.


Add percentile variance to standard output results

Compute variance between percentiles
Print it in normal mode
A good variance should be around 5-10%?

Not necessary after trust over time update


Migrate to Secure Login

This PR does the following:

Migrate from Retrofit to Ktor
Migrate from gson to kotlinx.serialization
Remove "save access/refresh token" in room database
Introduce preference store to store tokens
Unit test the network implementation
Some UI tests to test the login functionality (WIP)
Better network call setup - wrap calls inside a flowable sealed interface
Refactored/deleted lots of boilerplate code

This is excellent work.
I left some minor comments.
I also tested this, and it functioned like a charm. We still need to test the refresh token, though.

Yes, let's do that with Sam on the call


Assertion error in lesson ::: Applied Accessibility: Make Screen Reader Navigation Easier with the header Landmark

Describe your problem and how to reproduce it:
In the "Applied Accessibility: Make Screen Reader Navigation Easier with the header Landmark" lesson, when replaced the "div" tags with "header" tags, every assertion runs fine except the assertion about the existence of a "div" tag in the code. This error has been thrown in other lessons too.
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/responsive-web-design/applied-accessibility/make-screen-reader-navigation-easier-with-the-header-landmark
Tell us about your browser and operating system:

Browser Name: Firefox
Browser Version: 84.0.1 (64bits)
Operating System: Manjaro Linux (64bits)

Hello there,
I am unable to re-produce this. Do you have any browser extensions with access to the lesson page? If so, I recommend you disable them, and try again.

 There was a browser plugin messing up with the code! Thank you, Shaun.


Add aliased key code as QMK

I want to use the aliased key code as QMK. ( ex. KC_ENT, KC_SPC...)
I tried adding the aliased key code in QMK, but these are too large.
So, I gave up on adding some key codes at this time.

Thanks!


Account is unable to use wechat for web

Hi, I am getting "account is unable to use wechat for web" when I try to login. I think this is related to the web login issue here. Has this been resolved upstream? If so, what's required to fix this in the matrix bridge? Does the Docker image need to be updated and rebuilt?

What version are you using? I believe the latest version of the matrix-appservice has already used the latest version of the wechaty.
Can you follow https://wechaty.js.org/2021/02/20/google-cloud-shell-tutorials/ and see what's the result?

I see the same error when I run that tutorial. Maybe they blocked this method?

Are you using a WeChat account, instead if a WeiXin account?
Normally a WeChat account is registered by a non-China mobile phone number.
Web protocol is only confirm to work with WeiXin account, which is normally registered by a China mobile phone number.

Ahhhhh ok I think I am using a WeChat account
On Nov 10, 2021, 4:11 AM -0800, Huan (李卓桓) @.***>, wrote:

Are you using a WeChat account, instead if a WeiXin account?
Normally a WeChat account is registered by a non-China mobile phone number.
Web protocol is only confirm to work with WeiXin account, which is normally registered by a China mobile phone number.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Then try with a WeXin account then I believe everything will get work as expected!


Fixes to the Vue generator

Vue: Fixing value interpolation and prop enumeration

This (for now at least) requires that the props param must not be destructured, otherwise members plucked from the props won't be recognized.

Allow using innerHTML JSX-Lite prop (maps to React's dangerouslySetInnerHTML and Vue's v-html)
Added the Text Builder block

great stuff 👍

great stuff 👍


[WIP] Memory cgroup v2

This implements the memory cgroup v2 controller as part of ticket #78. It heavily tracks the behavior of the runc runtime. Due to lack of integrations test I tried to make the unit tests as encompassing as possible. Currently this is a work in progress until I find a way to reliably try it out.

This PR is closed in favor of  #141 which has a much cleaner history.


Remove special node handling in assigment

AFAICT, this logic has two behaviors when the assigned value is not a node:

Check if the value parses, perform this check with a class that doesn't exist
-- blow up
Don't even assign the value, always re-assign the existing node after this
no-op parse-check

After adding a spec around assignments, the logic was removed. The rest of the
suite remains green at this time as well.
/cc @codeclimate/review
This would replace https://github.com/codeclimate/codeclimate-yaml/pull/15

Ready for re-review

:+1:


TileMap YSort inconsistency for tiles with negative y-coordinate

Godot version:

Godot 3.3 RC 8 (same in 3.2.3 and many earlier versions including 3.1)
OS/device including version:

Windows 10, GLES 2/3.
Issue description:

I have two (or more) TileMaps, both are children of YSort. One TileMap for floor tiles with cell_y_sort property disabled, since floor should always be behind everything. Other TileMap(s) for objects etc. with cell_y_sort enabled. Non-floor tiles with negative y-coordinate placed over the floor get rendered behind floor tiles, while same tiles with positive y-coordinate are sorted on top of it. Cell size and tilemaps hierarchy doesn't matter as long as they are part of the same active YSort tree.
Expected non-floor tilemap(s) with enabled cell_y_sort to be always on top of floor tilemap with disabled cell_y_sort, no matter what coordinate tile has or at the very least all floor tiles to be consistently on top of non-floor tiles, not 50/50.
Floor:

Objects:

Together:

Steps to reproduce:

Create YSort (or TileMap with enabled cell_y_sort property)
Add 2 TileMaps as children, define tilesets
Enable cell_y_sort for one TileMap
Place cells on both tilemaps on top of each other, some with positive y and some with negative.
Observe different behavior.

Minimal reproduction project:

TileMapYSortNegativeTilesIssue.zip

What happens when you change your hierarchy to the following?
YSort:

YSort -> Floors
YSort -> Objects

@codecustard you mean if i do it like this?

Nothing really changes, as you can see... Since they are in the same YSort hierarchy.

I'd say that's expected/correct behavior. Since your floor tilemap has cell_y_sort disabled the parent YSort treats that floor tilemap as a single element (and floor's position is Vector2(0, 0), thus the result).

One TileMap for floor tiles with cell_y_sort property disabled, since floor should always be behind everything.

If it should be behind everything then why would you put it as a child of YSort? If you move floor before the YSort in the scene tree it works exactly as you want (floor is being rendered before YSort):

@kleonc

I'd say that's expected/correct behavior. Since your floor tilemap has cell_y_sort disabled the parent YSort treats that floor tilemap as a single element (and floor's position is Vector2(0, 0), thus the result).

Ah, i get it. I was under wrong impression that toggling cell_y_sort makes tilemap and all of its cells either participate in all ysort operations or don't, but in reality it only affects sorting of cells.

If it should be behind everything then why would you put it as a child of YSort?

The reason is because I want everything in one place. I'm making an isometric game. It consists of few self-contained sections (separate scenes) each having floor, walls, objects, characters, etc - except for floor everything is ysorted. Characters can move between different sections and sections should be sorted against each other too (their walls, objects, etc). So all sections have to be part of the same YSort hierarchy.
I fixed my problem by changing floor tilemap z-index to negative.
Thanks and sorry for cluttering tracker with my turns-out-not-a-bug issue. I guess my problem is rare enough use-case and doesn't require further discussion.


Continue to display Death charge timer even after spell has been activated

Is your feature request related to a problem? Please describe.
Since death charge is on a timer of 1 minute regardless of if the spell activates or not, it would be useful to continue to see this timer if the spell has already activated.
Describe the solution you'd like
Before feature:

Death charge is cast, Death charge infobox shows the time left until the spell runs out
An npc is killed, Death charge activates and returns special attack. Infobox disappears.
Death charge is on cooldown for the remainder of the 1 minute since it was initially cast

After feature:

Death charge is cast, Death charge infobox shows the time left until the spell runs out
An npc is killed, Death charge activates and returns special attack. Infobox continues to show (perhaps a different color?) for the remainder of the 1 minute timer, giving the player insight into when they will be able to cast it next
Death charge is on cooldown for the remainder of the 1 minute since it was initially cast

Describe alternatives you've considered
Timing it manually which is obviously dumb. Other
Additional context
I believe vengeance already works this way. Even after it's been "activated" (does damage to an opponent) the infobox continues to display.

This is actually already a feature. See GameTimer.java:88.


missing dependencies

some languages are not preinstalled in the image

sbcl
ruby
elixir
nim
perl6

dependencies installed. nim does not work currently. handled in #2


update(dockerfile): Dockerfile deletion

Reason for This PR
dockerfile is no longer needed for this repository
Description of Changes

File ./Dockerfile was removed

License Acceptance
By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.
PR Checklist

[x] All commits in this PR are signed (git commit -s).
[x] The reason for this PR is clearly provided (issue no. or explanation).
[x] The description of changes is clear and encompassing.
[ ] Any required documentation changes (code and docs) are included in this PR.
[ ] Any user-facing changes are mentioned in CHANGELOG.md.
[ ] All added/changed functionality is tested.

Friendly ping for @48d90782

Thank you @tarampampam

@tarampampam we could use a dockerfile for development workflow.
ex: debug and test without local go sdk and php setup.

Hey @litan1106, yeah, you might be right. What do you think @tarampampam ?

for example:

setup a dockerfile with golang:1.16-buster as base
add php to the dockerfile
setup Goland to run remote target via docker (or debug in vscode with docker)
debug/test in Goland (or debug/test in vscode)
Done.

@48d90782 sounds good! Can you create an issue for this with details (describe required features list)?


chore: added the start of the simplification process together with types

Description
This PR introduces simplification of types together with the general setup of the simplification process. It introduces both docs, tests and implementation.
Related issue(s)
Related to https://github.com/asyncapi/shape-up-process/issues/45

Thanks 🙇

Thanks 🙇


How to get the loss of a model?

Hi,
Sorry, I'm new to TensorFlow. How can I output a graph of the loss over time?
Is it possible to also get the best loss in order to compare two different models after n iterations? I want to train two tts models on the same dataset and compare the two
thanks!

PS: I would also like to train two differently formatted datasets with dc_tts and then compare how they fare - ie. using two different kinds of markup in each otherwise identical dataset

Just run tensorboard from logdir folder, with tensorboard --logdir=LJ01-1. Go to browser and type http://localhost:6006/ and search for loss.


Makes the min number of discoverable tablets with low rep lag configurable.

I signed it!

This is good work. If I understand correctly, this works better than the previous algorithm because of the added sort.
Can you add some unit tests? Since this is on the serving path, we need full code coverage: essentially, a test that shows that the sorting works as intended. You also have permission to delete tests that may have become obsolete.
Coverage command: go test -coverprofile=/tmp/c.out; go tool cover -html=/tmp/c.out

For sure @sougou I'll add some testing. I think the previous algorithm was basically a one pass find the min so not technically better... I needed the sort since I'm looking for potentially more than 1 tablet. I am assuming that the set of tablets is not too big.

@sougou So I'm running into trouble actually getting a sort to happen when we only need a minimum of 2 tablets. The other mean code will remove them slowly enough that the sort never matters.
Anyone know of a good way to modify flags for tests? Without making too much public?

Not sure about my testing but it is the best I could come up with for now. Comments are welcome.

LGTM


feat(enhancement): upgrade the ulimit configuration

These changes enable 1048576 openfiles for processes spawned via slurmscriptd.
Fixes #41

Going to land #40 soon so I can look at this. @jamesbeedy are you able to verify that the correct openfile limit is being applied using the test you mentioned in #41?

@NucciTheBoss awesome. yes, verified.

Great! I'm doing some work to address https://github.com/charmed-hpc/slurm-charms/issues/30 on the Slurm charms, so is it cool if I apply your patches there?
We can also add onto our integration test suite to verify the the correct ulimit config is being applied.


packaging: define installation components

The following components are declared:

runtime (libparameter and libremote-processor);
dev (everything necessary to develop a plugin or a client: headers, cmake
package files, ...);
c (the C bindings, i.e. libcparameter);
cdev (everything necessary to develop a client using the C bindings);
python (the Python bindings);
eng (the various tools, XML schemas, ...);
runtime-deps (on windows only: the msvc redistributable files).

Then, CPack can be used to generate separate packages.

This pull request is now managed with CodeReviewHub. Refresh.

[Current coverage][cc-pull] is 80.35%

Merging [#367][cc-pull] into [master][cc-base-branch] will increase coverage by +<.01%

@@           master       #367   diff @@
========================================
  Files         211        211
  Lines        7169       7169
  Methods         0          0
  Branches        0          0
========================================
  Hits         5760       5760
  Misses       1409       1409
  Partials        0          0

Powered by Codecov. Last updated by [1d51444...a62ad87][cc-compare]
[cc-base-branch]: https://codecov.io/gh/01org/parameter-framework/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/01org/parameter-framework/compare/1d51444ce870fe8fa8c373dd241d1fb5475c80f0...a62ad8739e3d3b71b3d0c9bd33f914012f2b2cde
[cc-pull]: https://codecov.io/gh/01org/parameter-framework/pull/367?src=pr

@krocard @tcahuzax @terahxluna Please review.


Adds bottom margin prop to typography components

Changes

Adds withMargin (default: false) option to typography components.
Updates the way to prop of the HtmlElement component works. Moved to a hash implementation for better performance.
Removes typography margin reset from globals as we're now doing that using the meyerweb resets.
Smaller fixes here and there.

Closes #22. Let's revisit the topic of double spacing once we get there.

Updated accordingly.