Multistep reaction capabilities

As a result of our discussions about how to best handle multi-step/stage reactions, this adds that capability.
Use cases are as follows:

For a multi-step reaction sequence with complete intermediate isolations and characterizations:

A string reaction_id field is added to the CompoundPreparation message. This is to be used when the preparation type is SYNTHESIZED and the preparation of that compound is also recorded in the database. Note that this is not to be used when the preparation merely follows a previously described process; the isolated product of the previous reaction should be what is used, physically, in this reaction.

For a one-pot reaction sequence with no intermediate isolation:

For all but the final step (i.e., reaction steps without analysis/characterization of the crude product):
Reactions have a single ReactionOutcome defined at the appropriate reaction_time.
A putative product structure (key intermediate) can be defined as a ReactionProduct, with no supporting analytical data used to confirm its identity (i.e., analysis_identity is empty); yield, purity, and selectivity would be undefined as well.

For all but the first step (i.e., reaction steps that use the output of a previous reaction step without complete isolation and characterization):

A new repeated CrudeComponent crude_components field is added to the ReactionInput message to indicate that crude (or partially worked up) was used in the reaction.

The key additions in this PR are as follows:

Addition of a CrudeComponent message to the schema and a repeated CrudeComponent crude_components field for a ReactionInput
Addition of a reaction_id to CompoundPreparation to be used when SYNTHESIZED
Validations and validation tests to ensure that cross-referenced reaction IDs are all defined within a single dataset
Addition of a validate_ids keyword argument to validate_message so that we can enforce well-formed IDs only after assigning them through ord_schema.updates
Addition of an update_dataset function in updates.py that is used by process_dataset.py, which ensures that when reaction IDs are assigned, we still preserve any cross-references that existed within the Dataset.
Addition of a test for update_dataset which confirms that cross-references are preserved.

FYI @antonkast-google this should not break the editor, but simply introduces additional fields that will not be captured by the current form

No problem. I will add the new fields in the editor.


fixs issue mem leak with LoadModel / OBJ and issue with -1 material i…

…ndex
as promised....

@chriscamacho thanks! :D


chore(main): release 1.1.0

:robot: I have created a release beep boop
1.1.0 (2022-10-17)
🧹 Miscellaneous

deps: update all non-major dependencies (#8) (1f01f30)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/apollo-server-integrations/apollo-server-integration-h3/releases/tag/v1.1.0 :sunflower:


Location set to "" for time.Time fields apart from created at and update at

Your Question
I am using Postgres and when I declare a struct  with a field of type time.Time, the location is set to "" even that I set the timezone in the connection. In the gorm configuration I changed the nowFunc to set the timezone, but this only works for the created at and updated at fields, not for other time.Time fields. As an example:
type TestStruct struct {
  ID                     int
  CreatedAt       time.Time
  UpdatedAt      time.Time
  StartTime.       time.Time
}

And I use the following gorm config:
gorm.Open(postgres.New(postgres.Config{
		Conn: gc.Db,
	}), &gorm.Config{
		NowFunc: func() time.Time {
			return time.Now().UTC().Truncate(time.Second)
		},
	})

The issue with this is that, if I create the TestStruct with an UTC location, and then I try to do assert.Equal with the one I retrieve form the database, the only difference is the loc between expected.StartTime (loc is nil) and actual.StartTime (loc is "")
The document you expected this should be explained
I only found this post in Stackoverflow related to my question (https://stackoverflow.com/questions/71019353/how-to-set-default-timezone-on-gorm-time-time-data-type)
Expected answer
I think all the time.Time fields should have the timezone set to declared timezone in the connection, or maybe I am missing something?

Have you tried changing your application's default timezone(You need to put this code before using any time package functions)?
os.Setenv("TZ", "Europe/Istanbul)

yes, I tried and still get loc = ""

I get this issue too. @saeidee why was this closed?


wip: Update diff action

Fixes #529
Signed-off-by: bryanl<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 883

121 of 200 (60.5%)  changed or added relevant lines in 7 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.2%) to 63.052%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/util/strings/strings.go
0
6
0.0%

pkg/clicmd/diff.go
5
13
38.46%

pkg/cluster/object.go
0
13
0.0%

pkg/diff/diff.go
63
79
79.75%

pkg/cluster/show.go
0
17
0.0%

pkg/actions/diff.go
30
49
61.22%

Files with Coverage Reduction
New Missed Lines
%

pkg/clicmd/diff.go
3
6.54%

Totals

Change from base Build 882:
0.2%

Covered Lines:
8831

Relevant Lines:
14006

💛  - Coveralls


refactor keypresses

Mildly messy. could be better.

OMG.
Took me like a week but it's done as of: adb2cb7e3634dbc61f1cb9277ddd21389f123627
For the record "mildly messy" is a hhhhhhhhhhhhhhuuuuuuuuuuuuuge undestatement.


Update github-activity-instructions.Rmd

Adding a note on mac vs windows for copy and paste as well

looks good! - just noticed that we used "find" instead of "Phind" - might want to check the rest of the instructions.

just checked this file... don't see any other "find" references where we mean phind


Azure Core TokenCredential is not getting group membership changes

Package Name: azure-core
Package Version: 1.27.0
Operating System: Ubuntu
Python Version: 3.10

Describe the bug
When I use Privileged Identity Management to get membership to a group, my calls to get_token for a TokenCredential are returning a token that does not include the new group membership
To Reproduce
Steps to reproduce the behavior:

Have PIM capabilities to a group
Run a script that uses the get_token
PIM your self to a group
Run a script that uses get_token

Expected behavior
When we examine the token from step 4, we would like for its membership information to include the group.
At the very least, is there a way for us to invalidate the token and request a new one via the python SDK?
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Thanks for reaching out.
I am not quite clear what you want to achieve.
Could you give more details?
If possible, maybe some code snippets to repro the issue?

Thanks for reaching out.
I am not quite clear what you want to achieve.
Could you give more details?
If possible, maybe some code snippets to repro the issue?

I think I'm confused by something and maybe you could help me understand.  I think our problem might be in the Azure CLI.  We have users log into the azure cli using --use-device-code and then we pull their credentials from .config/azure/credentials.json  I am wondering where the token is created.  If it's on CLI login then I don't think there's much that can be done here but if it's from somewhere else then I'd like to see if there's a way to automatically refresh the token to get the users current group membership.

Could you share how you want to use it?
e.g. a Python script? a command line? something else?

We use it in python.  We are creating a list of TokenCredential providers that we call get_token on.
We create an AzureCliCredential and a DeviceCodeCredential

I asked the question on Stack Overflow.  I think there's still an area for improvement on the CLI but for the time being I'm going to pursue this by checking the user's groups via the MS Graph API on the server side: https://stackoverflow.com/questions/78235099/is-there-any-way-to-get-a-new-azure-cli-token-without-logging-out


Internationalization: Create useLocalizedStrings hook and format function

User Story
As a user, I want Platform.Bible to have the equipment in place to be able to display the UI in my language so I can work effectively.
Description
Depends on #738

[x] Create a hook that wraps the localization service and gets strings as passed into the hook. While loading the localized strings, just pass back an empty string if the input was just one key or an object with empty strings if the input was multiple keys.
[x] Create a formatting utility function in platform-bible-utils that replaces {stuff} - format(stringToFormat, replacers: { [key: string]: value: string }): string-ish. Also needs to remove backslash \ on \{ and \} in case there are escaped curly braces.

Can probably adapt git.util.ts's formatGitErrorTemplate if you want
ex:

console.log(format('Hi, this is {name}! I like \{curly braces\}!', { name: 'Jim' }));

output:

Hi, this is Jim! I like {curly braces}!

You may be able to use useData to make this hook easier.
Testing Ideas
Test the new string utility function, formatReplacementString, in a webview. You can use the example above or any example of your choice. I would recommend including a word to replace in unescaped curly braces (e.g. {name}) and a word in escaped curly braces that should not be replaced (e.g. {unreplaceable}).
Tests can now replace
usePromise(
useCallback(() => {
return papi.localization.getLocalizedString({
with
useLocalizedStrings
Note: if you use useLocalizedStrings, you will need to wrap the keys array in useMemo and the locales array in useMemo.

Test Repo Steps:

Added te.json file in the C:\Repos\paranext-core\assets\localization\te.json

 {
  "%submitButton%": "సమర్పించండి",
  "%Book.Gen%": "ఆదికాండము"
}

Updated my test extension to use the useLocalizedStrings hook on the web-view

const [localizedStrings] = useLocalizedStrings(
    useMemo(() => ['%submitButton%', '%Book.Gen%', '%wordList%'], []),
    useMemo(() => ['te', 'fr', 'en'], []),
  );

  const localizedString = localizedStrings['%submitButton%'];
  const localizedGenesisString = localizedStrings['%Book.Gen%'];
  const localizedFallbackString = localizedStrings['%wordList%'];

Confirmed that the text is displayed in Telugu and English is displayed as fallback

These test steps look great. Thanks, Roopa!


MGDCTRS-348 Reduce the exposed attributes for Debezium connectors in the catalog

https://issues.redhat.com/browse/MGDCTRS-348

@ani-sha looks good except for some properties. I added comments where we need to change based on the meeting notes and I asked Gunnar about some properties where I was not sure if we should include or exclude them.

LGTM @ani-sha, thx a lot! I also added your spreadsheet to the meeting notes document.
Could you please rebase your branch and re-build the descriptors? I had to change something around the data_shape property. Then we are good to merge the PR! Feel free to do squash and merge yourself then.


[CP] Enable logging a warning when the user opts out of using Impeller.

Issue Link
https://github.com/flutter/flutter/issues/144439
Target
stable
PR Link
https://github.com/flutter/engine/pull/51914
Changelog Description
If the user explicitly opts out of Impeller on iOS, display a warning saying that the option is going to away in an upcoming release.
Impacted Users
Users who have explicitly opted out of using Impeller on iOS.
Impact Description
Explicitly opting out of using Impeller on iOS will emit a log warning the developer that the option is going to go away.
Workaround
Don't opt out of Impeller on iOS. If there is a workaround for a specific Impeller issue, the developer is instructed to file that issue on the issue tracker.
Risk
low
Test Coverage
yes
Validation Steps
The tests in both engine and iOS harnesses should verify this.

CP LGTM


Common error which scanning the IP

First, make sure that none of the open and closed issues is about the same issue as you are describing, and make sure to check the frequently asked questions in the README file.
Then, replace the parts of this template that are between  with the data relative to your issue.
If you're reporting a bug, use the template below. Otherwise, delete this template and write your issue normally.
Context
Please select one:

[ ] I use the docker image ullaakut/cameradar
[ ] I use my own build of the docker image
[ ] I use the pre-compiled binary
[ ] I use my own build of the binary
[ ] None of the above / I don't know

I use the docker image ullaakut/cameradar
Please select one:

[ ] I use a specific version:
[ ] I use the latest commit of the master branch
[ ] I use the latest commit of the develop branch
[ ] I use a forked version of the repository:
[ ] I use a specific commit:

Latest version downloaded 3 days before
Environment
My operating system:

[ ] Windows
[ ] OSX
[ ] Linux
[ ] Other

Windows
OS version:
OS architecture:
10, 64 bit
Issue
I followed the instruction step by step mentioned in the portal "https://www.hackers-arise.com/post/ip-camera-hacking-hacking-ip-cameras-with-cameradar". I checked Shodan for the IP's "RTSP/1.0 has_screenshot:true" as well and could trace approximately 50 IP's  with the command docker run -t ullaakut/cameradar -t xxx.xxx.xxx.xxx. The out put for all the scan ends with "Streams were found but none were accessed. They are most likely configured with secure credentials and routes. You can try adding entries to the dictionary or generating your own in order to attempt a bruteforce attack on the cameras". However I could see different out put in the logs, sometime password, routes etc

Kindly help whether I do something wrong or the tool is behaving like this. Thanks.
What was expected

What happened

Logs
If your issue is with Cameradar's binary or docker image, please run it with -v to print verbose logs, and paste them here:
<cameradar logs>

Hi @vishprat18 ! Thanks for opening this issue :) The logs indicate what credentials Cameradar attempted to use, and show that it was unsuccessful with each attempt. Like the output explains, this means that the cameras are either protected with secure credentials, or that the dictionary we have in Cameradar is slightly outdated and doesn't cover this brand of Cameras.
If you do find out what the credentials are, feel free to open a pull request against this repository to add them 👍


flex_vector does not respect type alignment requirements

Essentially, flex_vector can (always?) allocates objects at addresses that do not align with the type's alignment requirements.  I uncovered this while using Eigen with immer, which causes some nasty segfaults when there are alignment violations on vectorized types.
I added a repro case here: https://github.com/sissow2/immer/commit/9d295b829c37dbabdb96b4f703ba56492734e76c .  The sanity check with std::vector passes always, while the immer::flex_vector variant always fails :
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
flex_vector-alignment is a Catch v2.13.7 host application.
Run with -? for options

-------------------------------------------------------------------------------
direct alignment
-------------------------------------------------------------------------------
/home/sissow2/projects/immer/test/flex_vector/alignment.cpp:32
...............................................................................

... snip ...

/home/sissow2/projects/immer/test/flex_vector/alignment.cpp:41: FAILED:
  CHECK( v[i].is_aligned() )
with expansion:
  false

===============================================================================
test cases: 2 | 1 passed | 1 failed
assertions: 8 | 4 passed | 4 failed

This might be related to https://github.com/arximboldi/immer/issues/228, but the reporter's repro case is for a non-vectorized type so I'm not sure it's the same issue.
I might be able to create a PR with a fix, but wondering if you have any ideas before I dive into it.
Also: I'm really liking this library!  This and lager have been making my life so much easier.  Also thanks for providing a nix-shell environment, it made creating this repro case a breeze.

Thanks a lot! Yes had issues with Eigen before which made me suspect for some time that this was the case. I somehow thought that I fixed this in the past but it is clearly still there. Thanks for the nice words and for finding a small repro case!

So I did a bit of digging on this.
First, the current state.  It looks like you've definitely thought about this before, since in rbts::node you wrap T in a std::aligned_storage_t.  This does correctly change the alignof(node) to the appropriate value.  Unfortunately, it looks like this mechanism is defeated because the buffer itself is initialized by a placement-new expression on an unaligned block of memory returned by MemoryPolicy::HeapPolicy::allocate.  I think this is a loophole that erases the alignment requirements posed by the aligned_storage_t and therefore node.
I can see a couple solutions here:

in rbts::node, add some machinery around leaf_t::buffer that over-sizes the buffer (based on alignment requirements), and adjust rbts::node::leaf() to apply an offset from the beginning of this buffer to ensure alignment.
Make the allocators alignment-aware somehow.  I don't have a specific proposal here.

If you want I can take a pass at either of these options, or something else that you have in mind.  Or I can leave it be; I have a workaround I can use at the call site (turning off alignment optimizations for eigen types I store in immer containers).

Hi @sissow2!
Your assesment seems very much on point. Thanks a lot for looking into this. Either of the two approaches sounds good. Maybe the latter could be better as to keep the data-structure code simpler, but happy to take the other route if you feel like it's actually not much effort.
I don't have much time to look into this over the next few days, so if you wanna start the work it would be he greatly appreciated.
Also: can you reproduce this issue with some of the other data-structures?

Re other data structures: I updated the repro case to check all the data structures, as well as different alignments: https://github.com/arximboldi/immer/compare/master...sissow2:immer:sissow2-alignment-flex-vector-bug.  Here's what I found:

with 16-byte alignment, I only observed failures with vector and flex_vector.  Notably, vector<box<T>> was fine :thinking:
with 32-byte alignment, I observed failures with all data structures (intermittently)

The natural alignment of the compiler in nix-shell is 16 bytes.  Since this is the case, it makes sense at the 32-byte case has more failures.  What's surprising to me is that there are failures at all for the 16-byte case.  Digging even deeper, it looks like the RBTS-based structures will use an optimized free list for their heap, while HAMTS structures do not do this optimization and will use debug_size_heap<cpp_heap> (which is 16-byte aligned).  The 8 bytes of misalignment is therefore unique to the free list, which makes sense because free list nodes start with a pointer.
Re the allocator solution:  Writing an allocator that is guaranteed to return aligned pointers seems to be quite the exercise, but fortunately c++17 added overloads to operator new (see 3) that accept a desired alignment as input.  Working backwards from this, the API would change to HeapPolicy::[de]allocate(size_t size, std::align_val_t alignment).  This unfortunately would bump the c++ version from 14 to 17.  Some potential workarounds:

Can do some #define magic to only enforce alignment in cpp_heap if C++17 is used
Can replace cpp_heap with a custom alignment-aware implementation to work with C++14 as well.  I'd need to do a bit of research on the state-of-the-art here though.

I do no think we need the C++17 alignment API since we use our own allocator API?
Memory allocated with new is alligned by default to alignof(std::max_align_t). It is us (me :sweat_smile:) that broke the alignment with the metadata that heaps like debug_size_heap and free_list_heap. I think changing the HeapPolicy API and fixing those heaps adaptors should be enough.

Hey, it just hit me I forgot to communicate back here-
I found a workaround that I can use in my own codebase, so I de-prioritized this as I needed to move on.  I don't know when I'll have time to get back to this, so I just wanted to let you know not to wait on me!
Again, sorry for leaving this in limbo for so long.

No worries, I wasn't waiting for you. Thanks for checking in!


Eyepatches for your other eye

About the PR
Adds the ability to flip eyepatches over to the other eye! No more having to deal with one eye always being the greedy one who gets to see!
Why / Balance

Some people want their eyepatch on the other eye
Technical details

Using FoldableClothing in the same way as the soft hats do
Media

https://github.com/space-wizards/space-station-14/assets/83733158/da666925-8689-483a-b552-b912f7a7455f

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: You can now flip your eyepatches to the other eye! Yarrrrr!

Can't you just mirror the layer instead

here is a video of the bug
https://github.com/space-wizards/space-station-14/assets/83733158/c0fa5eb0-fa42-42da-bc9c-c9ebbb47f81c

here is a video of the bug... I need some help with this https://github.com/space-wizards/space-station-14/assets/83733158/c0fa5eb0-fa42-42da-bc9c-c9ebbb47f81c

I... uh... did not know that this would not go through and have no idea how to get it to go through. Here is the code I added to SpriteComponent.cs
    [DataField("mirrored")]
    private bool mirrored = false;

    /// <summary>
    ///     Mirrors the layer through a negitive scale.
    /// </summary>
    [Animatable]
    [ViewVariables(VVAccess.ReadWrite)]
    public bool Mirrored
    {
        get => mirrored;
        set
        {
            if (value)
            {
                Scale = new Vector2(-Scale.X, Scale.Y);
            }
            else
            {
                Scale = new Vector2(MathF.Abs(Scale.X), Scale.Y);
            }
            mirrored = value;
            UpdateLocalMatrix();
        }
    }

here is a video of the bug... I need some help with this space-wizards/space-station-14/assets/83733158/c0fa5eb0-fa42-42da-bc9c-c9ebbb47f81c

I... uh... did not know that this would not go through and have no idea how to get it to go through. Here is the code I added to SpriteComponent.cs
    [DataField("mirrored")]
    private bool mirrored = false;

    /// <summary>
    ///     Mirrors the layer through a negitive scale.
    /// </summary>
    [Animatable]
    [ViewVariables(VVAccess.ReadWrite)]
    public bool Mirrored
    {
        get => mirrored;
        set
        {
            if (value)
            {
                Scale = new Vector2(-Scale.X, Scale.Y);
            }
            else
            {
                Scale = new Vector2(MathF.Abs(Scale.X), Scale.Y);
            }
            mirrored = value;
            UpdateLocalMatrix();
        }
    }

What's wrong with setting the scale in yaml?

Yay! I got it! Took me way too long because I forgot to edit the meta files and remove the icon_flipped before testing!


(#979) Add support for new transifex client

The old transifex-client that was previously supported has stopped working some time ago, the new executable that replaces it is not fully compatibly with how we set up the api token when connecting to transifex.
This new executable also adds support for new global arguments where the token can be specified with, and as such this PR updates cake.recipe to be compatible with this new executable.
The changes are made directly here in Cake.Recipe instead of in Cake.Transifex for the global argument, as it won't be backported to be supported on older versions of Cake.
fixes #979

@AdmiringWorm thank you for getting this fixed up!


Valhalla: avoid locations/polygons incorrectly formatted in Directions request

The avoidLocations and avoidPolygons DirectionsOptions are not formatted as Valhalla expects them to be.

When running with dryRun: true in order to see what's going on:

I dug into the built code; the issue is here:

The Valhalla API expects the avoid locations parameters to be formatted identically to the directions location parameters (object with lat/lon properties).  https://github.com/valhalla/valhalla-docs/blob/master/turn-by-turn/api-reference.md
I am passing an array, so I've edited only the top portion of this segment of code:

Or perhaps something more general:
params.exclude_locations = avoidLocations.map(([lat, lon]) => ({ lat, lon }));

That fixed the issue for me (for avoidLocations only; would also need the same in avoidPolygons). Dry run result:

And the final success from the valhalla public server:

I wanted to write a PR but I'm not sure how this would be handled from the typescript/interfaces perspective; but please let me me know if you'd like me to have a go at it.
Cheers!
EDIT: just saw the _buildLocations function.

Oops, yeah that’s the „kids diseases“:) We’d be happy with a PR, as you wish. FWIW, avoid polygons does not take location objects but a plain array of coordinates (essentially a open/closed ring), as that’s not supposed to snap to road segments but describe polygons.

I've made the changes on a local branch but I'm afraid I've not submitted a PR before. Do I need to fork the repo in order to push to a branch for comparion and PR creation?


Not an issue: Couple of questions

Hi Florian,
First off, thanks for the great work. I have an free FEM code and want to integrate your mesh generation code in it. I have couple of questions:
1- is it possible to use TQMesh to mesh domain with internal material lines (not internal hole)? in FEM most of the time different materials is separated by lines.
2- is it possible to specify number of division on the boundary lines (i.e. number of segments) and make sure program uses the specified points and prevent it from generating new points on the boundary line?
3- currently I'm using "Triangle" library developed by prof. Shewchuk. Using this code I'll be able to specify marker for each closed region in the domain and program will assign the same id to the mesh inside the region. The same can be done for the initial nodes and edges. I see that in your code user will be able to do the same for the boundary by specifying the marker but can it be done for the region and etc?
thank you again.
-Roozbeh

Hey Roozbeh,
thanks for your interest! Regarding your questions 1 & 2:
Internal lines and the specification of the actual number of vertices on boundary segments is currently not supported.
But this sounds really useful - so I will try to implement it :-)
Concerning question 3:
Currently it is only possible to create a domain with one single exterior boundary - this basically means, you can only create one single enclosed domain. I am planning to extend this for multiple domains and in that case, it will also include the possibility to specify markers for elements or interior edges.
Again - thanks for your interest :-)
Greetings
Flo

ssible to specify number of division on the boundary lines (i.e. number of segments) and make sure program uses the specified points and prevent it from generating new points on the boundary line?

Could I ask if we can specify the number of segments on the boundary now? (regarding the question 2)

Hey Zqinghaohh,
So far, it is not possible to specify the number of segments on a boundary.
But thanks for your comment - I actually forgot to put this idea on my to-do list and I will try to implement it as soon as possible.
Greetings
Flo


Bump numpy because of meshzoo in minreq

As all meshzoo release prior to 0.10 have been deleted and 0.10 advertise itself as compatible with numpy 1.20+ only, this is what was triggering failure on MIN_REQ.
To fix that we can either bump minreq numpy to 1.20 (which I do here), or drop meshzoo and corresponding examples.
Numpy 1.20 was published in january 2021, Nep 29 suggest supporting numpy 1.20+ starting June 21, 2022.

Hmmm, even if the min suggested numpy support is changing in June, I can still see merits for pinning the broadest support we can tolerate.  So I have a slight preference for dealing with meshzoo rather than bumping numpy here, but defer to @jni

I don't have any preferences. We can also just skip meshzoo test on min_req, which is a 3rd approach.

We can also just skip meshzoo test on min_req, which is a 3rd approach.

We could do this but then we should leave a note to add them back in after we move to numpy 1.20+. June 21, 2022, doesn't feel that far away

Ok, let's see if #4329 works for you then.

CLosed by #4329


Add multiple key mappings, use time stamps instead of sleep

Fix #226.

@eapache could you please review it ?
I don't feel confortable merging it without a quick review.
Thanks in advance!

I understand the issue this is trying to fix (and I left a few comments) but honestly I wasn't quite sure how this PR affects the logic.

'''bundle exec rake''' passing on my machine.
The logic does not change except when when you press a letter starting a special char.
In that case, we wait up to '''timeoutlen''' for the user to complete the special key.

Looks pretty good to me now.


Release timeline for v1.0.3 / v1.1.0

It looks like there has been some great progress since v1.0.2 (https://github.com/jpype-project/jpype/compare/v1.0.2...master). Indeed, at this point, I'm excited to say that, for the libraries that I support, master is better than any release that has been made of JPype to date (it properly does JVM cleanup and it doesn't core-dump! :partying_face:).
It looks like there hasn't been an effort to maintain a patch version branch for v1.0.x, which suggests the intention is to go straight for v1.1.0, though it also looks like some of the work that has been merged isn't yet finished (the android stuff?).
All of this leads me to ask the question - is master in a state where it could be released, and if not, is the v1.1.0 milestone reflective of what is necessary for the release, or just the wishlist?
I'd love to be able to drop support in my libraries for JPype v0.x, but at this point there isn't a release of v1.x which I can reliably use (FWIW I systematically test against master and am very happy with it).

I don’t think we are that far from being able to release a 1.1.   There are a few PRs on the queue I would like to see in, but we could cut a 1.1 release by next week.

Ok, let's remove this in favour of #861


beet play: output from CLI music player not displaying

This is probably my CLI ignorance. I want to use mplayer with beet play.
In my config files has these lines (Debian):
play:
    command: "/usr/bin/mplayer -playlist"

The issue is that mplayer display does not show up in the terminal window, although the music does begin to play, ex: beet play beatles. It's just a blank terminal display.
If I spawn another terminal it works fine, but that is not what I want to do, I want to use the same terminal for CLI output from mplayer.
play:
    command: "terminator -x /usr/bin/mplayer -playlist"

How would I get mplayer to display in the window where invoked the beet play command?
Love the program, I finally feel I have control over my unwieldy music library (no thanks to itunes...).

Thanks for the report! To check whether this might be related to mplayer itself, can you try changing the command to a dummy value temporarily? For example, just command: "echo" should print the filenames.

It does the same for cvlc and nvlc.
When I run it with command: "echo" it outputs Playing 2 tracks the returns control.
Here is the version info
beets version 1.3.10
plugins: embedart, lastgenre, chroma, duplicates, play, fetchart, fromfilename, info

Current Config File:
directory: /media/TRAVELER/music-clean
library: /media/TRAVELER/musiclibrary.blb

import:
    move: yes
    copy: no
    log: /media/TRAVELER/beetslog.txt

plugins: chroma duplicates play embedart fetchart fromfilename lastgenre info

play:
    command: "terminator -x /usr/bin/mplayer -shuffle -playlist"

Thanks so much for your help! My favorite command is rm, get rid of all this perfectly tagged junk music.

Thanks for checking! I looked again and the play command is indeed suppressing the subcommand output. You can see it if you use verbose mode, i.e., beet -v play. Would you want to see it every time? If so, we can make this a feature request, either for default behavior or for a flag.

I just pushed commit 13e47472 which turns '\n' and fixes writing playlists with files with non-ascii characters.
@sampsyo running in verbose mode is not the same, for in that case all of stdout gets written on stderr once the play session has ended. What I think is desired with on-the-fly full ("normal") interaction which the launch command, i.e. it should use stderr stdout and stdin. Such a change would be pretty simple: just don't fill stdout and stderr in subprocess.Popen call in beets.util.check_output().
Questions are:

should stdout and stderr become parameters of check_output ou should a new function be created?
should this behaviour be the default one or enabled with a flag/a config option?

Oops, and thanks for that; I made the same change but evidently forgot to push. :flushed:
Yes, it's definitely not the same as piping the subcommand straight through to the tty; I just wanted to see if it was close enough. It may be that nobody really needs full interactivity; they just need to debug by looking at the output if something goes wrong.
If we do this, though, I would lean toward it being the default since it seems harmless enough.
And you're right about needing changes to beets.util.check_output. Without looking too closely, I would guess that a new function would be simpler (as the name suggests, that function is really meant for capturing output), or perhaps just a passthrough flag rather than individual stdout and stderr parameters. It may be easier for the plugin to use subprocess directly.

I worked a bit on it and included a bunch of might-not-be-desired changes. See the PR.


Crash on 1.16.4 when creating new world

Crash Report
Description: Exception in server tick loop
org.spongepowered.asm.mixin.transformer.throwables.MixinTransformerError: An unexpected critical error was encountered

I believe this is a crash because the mod is fixing something that Forge also just recently fixed. So basically Forge is now doing what this mod used to do? Maybe? I could be wrong. But it means you can just delete this mod and everything will work fine?


ci: block PRs until automerge label is added, other ci cleanup

Overview

Removed unused next branch setup
Use PAT instead of default token in actions so we can trigger the automerge action in more scenarios
Check commit message instead of commit author since automerge commits will come from our normal bot user

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

🚀 Styleguide deploy preview ready!https://6050c1802d17f3f5a455544f--gamut-preview.netlify.appDeploy Logs


Image extension not included in exported Pascal VOC XML files

Describe the bug
When exporting PascalVOC, XML does not include file extension in brick-front
To Reproduce
Try exporting to PascalVoC and look at all generated XMLs
Expected behavior
File extension should be included

OS: Mac OS
Label Studio Version: 1.4.1post1

It's fixed here, but not yet release:
https://github.com/heartexlabs/label-studio-converter/pull/100


Add i18n support

want to add internationalization support. I can translate the simplified Chinese part.

Added to Roadmap.

Done, you can write your translation in lang_pack.ts.


TS symbol search is broken

latest 1.44.0 insider
open the VS Code project
use symbol search, it is broken 9 of 10 cases

oh, also using 3.9.0-dev.20200304

Also with TS 3.8.3. I see the server returning the correct info so need to look into what's going on here


Decode.andMap?

Is there a reason the Decode module doesn't expose an andMap? I found myself needing this in an app I'm working on, and ended up just defining it inline:
decodeAndMap = Decode.map2 (\f x -> f x)

...not a big deal to have to do that, but was there a reason for the omission? Thoughts on including it?

None of the libraries I make expose andMap. There was discussion about it a while back, but approaches like https://package.elm-lang.org/packages/NoRedInk/elm-json-decode-pipeline/latest/ seemed like a better way to do that sort of thing. Rather than having a generic "andMap" concept that doesn't really make sense on its own or in context, having specific APIs for each use case seemed better.

Thanks for the explanation. That makes some sense; in my case I wanted it as part of lower level plumbing in a module that exposes its own mapN functions, so I see the argument for leaving it out of the API that normal users have to sift through.


Use Minio instead of s3 for testing

@navinpai uff, finally got it to working.
I have also raised an issue with minio about double slash test case (case14)

I still see multiple test cases failing: https://travis-ci.com/bloomreach/s4cmd/builds/73840004 ... The test passes with OK but there are exceptions being thrown:
API: GetObject(bucket=miniobucket, object=case13/source/011) Time: 15:09:04 UTC 05/18/2018 RemoteHost: <IP_ADDRESS>:33688 UserAgent: Boto3/1.7.19 Python/2.7.14 Linux/4.4.0-101-generic Botocore/1.10.19 Error: 'bytes=0--1' does not have a valid last byte position value        1: cmd/object-handlers.go:135:cmd.objectAPIHandlers.GetObjectHandler()        2: cmd/api-router.go:73:cmd.(objectAPIHandlers).GetObjectHandler-fm()        3: /opt/go/src/net/http/server.go:1947:http.HandlerFunc.ServeHTTP()
Can you take a look at this?

If minio differs in behaviour from s3, does it really make sense to use minio for testing? Wouldn't testing with s3 directly be more preferable (though the issue of access keys still needs to be figured out).
Minio errors are being ignored, and we have a minio specific test case as well in the test script. So even if tests pass, we have miniocmd instead of s4cmd, since CI is not testing with s3 at all. Just don't want to end up with a case where a code change works perfectly with minio, but breaks with s3, and I'm unsure how we can be sure of that.

Yes, there is a chance that the code which works with minio can be broken in s3 due to incompatibilities. And one way to work around it is to execute run_tests with s3 before upgrading a version.
Using minio would reduce the barrier of entry for contribution and the test case which has a dependency on s3 uses double slashes in the path and the probability of it breaking is less.
If we don't use minio we need to use companies s3 access keys and they will stop working with account termination.

closed it due to incompatibilities between s3 and minio.


D2: Module Level Integration

Integration with Team 3
4/5 features done

Integrated with the following
Archived Projects

Charts Summary

Generate Report

Charts Individual


Deprecate future encoder

import cats.effect.IO

import scala.concurrent.ExecutionContext.Implicits._
import org.http4s._
import org.http4s.dsl.io._
import org.http4s.implicits._
import cats.implicits._

import scala.concurrent.Future
val service: HttpService[IO] = HttpService[IO]{
  case GET -> Root =>
    BadRequest(Future({
      println("hi1")
      "zz"
    }))
}

val service2: HttpService[IO] = HttpService[IO]{
  case GET -> Root =>
    Ok(Future({
      println("hi2")
      "zz"
    }))
}
//repl doesn't pick up our syntax :(
(service <+> service2).run(Request[IO](GET)).getOrElse(Response.notFound).unsafeRunSync

This prints both hi1 and h2. This is potentially terrible for business logic on top of the fact it clearly violates RT.
We need to deprecate EntityEncoder.futureEncoder[F]

Yeah, do it.  It exists for convenience, but I don't think the danger was properly appreciated.

Closed by #1759.


RichTextLabel StyleBox shadow is not fully displayed due to Clip Contents being enabled by default

Tested versions
Doesn't work in 4.2 or 4.3 rc
System information
Windows 11
Issue description
The RichTextLabel shadow gets cut off by its own size.

Steps to reproduce
If you create a Rich Text label and then set a styleBoxFlat style to it, and then add a shadow and offset it, the shadow gets clipped by the textbox's bounds. This doesn't happen with labels or panels.
 I feel this is an easy fix

This is due to the same cause as https://github.com/godotengine/godot/issues/32996. RichTextLabel has a built-in scrollbar system that requires Clip Contents to work, so it's always enabled (even if the scrollbar is not visible).

so question, is the requirement to make the exception for the shadows every where (like styleBoxflat, textOutline) ?

or just not enable it from the start.

This would break rendering in RichTextLabels that require any amount of scrolling to have their full content seen, since the scrolling would no longer hide content located outside the node's rect.

This would break rendering in RichTextLabels that require any amount of scrolling to have their full content seen, since the scrolling would no longer hide content located outside the node's rect.

understood so not recommended. After testing it out with just basic toggling the clip_contents in RichTextLabel one bottom extra line was out of bounds for scrolling content (there are more below)

false
|
true


coloring in dark themes is partially broken

continuation of #2
another few places seems need fixing. looks like the same issue.
here are some screenshots with theme Tomorrow Night Eighties selected via Tinted Theming: Activate Theme
command palette - unreadable selected item

quick switch project - ditto

(various) input fields, eg try Find & Replace or View > Show Console - wrong border comparing to fields themselves

@JamyGolden could you take a look?

I've created a fix for this and merged it: https://github.com/tinted-theming/tinted-sublime-text/pull/5
Can you check to see if it looks good for you?

@JamyGolden mostly yes, except Quick Switch Project panel is broken now, either transparent or broken render, check this out

checked it with Adaptive theme - works fine
and too low contrast in Find'n'Replace's Where field hint - but i dunno, maybe it's just specific themes' issue

Thanks! I've fixed it, it should look not-broken now :sweat_smile:

I've just merged a fix which increases contrast between font and input background in text fields: https://github.com/tinted-theming/tinted-sublime-text/pull/7

@JamyGolden now it looks like it should, nice! also noticed you've fixed base fg color from base04 to base05 - just like it should be, finally. thanks!


Documentation needs clarification (or API inconsistencies)

I was in the process of writing a ~python~ client for the API when I noticed an inconsistency. The documentation above shows that the query can be made without a trailing slash. When querying the API without a trailing slash for tafseer, a 301 Permanently Moved is returned with a redirect to the same URI with a trailing slash appended.  See here:
% curl http://api.quran-tafseer.com/tafseer/1/2/1 -vvv
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to api.quran-tafseer.com (<IP_ADDRESS>) port 80 (#0)
> GET /tafseer/1/2/1 HTTP/1.1
> Host: api.quran-tafseer.com
> User-Agent: curl/7.54.0
> Accept: */*
>
< HTTP/1.1 301 Moved Permanently
< Server: nginx/1.14.0 (Ubuntu)
< Date: Tue, 26 Feb 2019 03:10:21 GMT
< Content-Type: text/html; charset=utf-8
< Content-Length: 0
< Connection: keep-alive
< Location: /tafseer/1/2/1/
< Vary: Accept-Language, Cookie
< Content-Language: en
<
* Connection #0 to host api.quran-tafseer.com left intact

Some clients (like the library used by cURL above), don't follow redirects by default and might break here. Should the documentation be clarified to require the trailing slash? If so, my only nit would be that the tafseer resources seems inconsistent with the quran resources given the latter don't require a trailing slash in their URIs.
jazakumallah khair for your work on this project

Thanks, @amerghazal for reporting this, I will fix it.
BTW I made a Python client if you would like to contribute to it as well.

thanks! i saw that after i had gotten started - i'll look into it iA. i'm happy to submit a PR if you can tell me whether you prefer addressing the API or the documentation.

@amerghazal If you mean by addressing the API is this bug, then no I will do it. it is an easy fix. I want to help more in the documentation part/

sounds good, JAK


Proguard removing necessary fetcher classes

I'm forced to add this to my proguard rules otherwise Kamel refuses to load due to not being able to find classes through reflection
-keep class io.kamel.** { *; }

Is there a more accurate ruleset I can use rather than marking the whole lib to keep?

I finally just tried out using enabling minify and you're right. I'm just going t remove reflection in the next release and use the same method as every other non-jvm target.

fixed by a3720b8566c73de1589d7cd66306f127294f8c25


Add Fish until next level to Fishing Overlay

I already mentioned this before, just use XP tracker sidebar


Make subcommands for packaging (and deploy)

My 2cs feedback on UX that I got while following the tutorial.
Looking at how packaging works now, I think we can make subcommands to make it more self-explainable. E.g. now:
$ mlem pack rf pip -c target=build/ -c package_name=example_mlem_get_started
⏳ Loading model from .mlem/model/rf.mlem
💼 Written `example_mlem_get_started` package data to `build`

With subcommands
$ mlem pack pip --help
Pack model to Python package that can be distributed with PyPi

Arguments:
    MODEL     Path to model
    NAME        Package name
    TARGET    Folder to save build

Examples:
    $ mlem pack pip rf example_mlem_get_started build/

Pros of this approach: easier to see specific args in --help that are required for each packaging solution, harder to do it wrong.
Cons of this approach: requires either writing subcommands for each packaging type (then it should be part of extension mechanism) or generating subcommands automatically from extensions (may be not user-friendly UX since it's hard to fine-tune it).
The same point probably could be made for deploying mechanics, although I didn't explore how it could be made more intuitive.

As another thought, I think mlem types packager pip for now will be often a complementary command to mlem pack pip. Should we mention this in mlem pack --help? Or suggest checking out mlem types packager X when user calls mlem pack pip with wrong args/options?

If you specify config for packaging you don't specify type that is why model argument goes before type argument, since type argument is optional.
mlem types works with all MlemABC, so it should be mentioned everywhere I guess. But more important is to add doc links

closed by #363 🚀


add POST /eth/v1/validator/sync_committee_subscriptions

https://hackmd.io/@QYHAVYiHRg65pdI_CEm7Eg/syncommitteeapi#POST-ethv1validatorsync_committee_subscriptions
notable differences:

sync_committee_index became an array: sync_committee_indices, as multiple subnets per validator is possible.

is_aggregator is not really needed in this instance as far as we can tell, and potentially won't be known at the time of making the subscription.

added until_epoch so that the BN can unsubscribe based on this request.

is_aggregator is not really needed in this instance as far as we can tell, and potentially won't be known at the time of making the subscription.

@rolfyone Could you explain in more detail why it's not needed?

is_aggregator is not really needed in this instance as far as we can tell, and potentially won't be known at the time of making the subscription.

@rolfyone Could you explain in more detail why it's not needed?

The beacon node is required to subscribe to the subnet regardless of whether the validator is an aggregator or not, so there's no change in beacon node behaviour.  That provides the stable backbone of peers on the subnet.  Also given sync committee periods are 256 epochs long (might have even changed to 512?) a validator would have to sign 256*32 = 8192 slots to check if it was an aggregator and it almost always will be at some point during the sync committee period.

@rolfyone until_epoch  means until the start that epoch or the end of that epoch?

The intent was that you'd unsubscribe once that epoch was past, so if it's 'untilEpoch' 20, you unsubscribe at the first slot of 21.
https://github.com/ethereum/eth2.0-APIs/pull/143
I'm not the biggest fan of that wording, but the PR is up.


Add stack run list command

Add a command to list the runs for a specific stack.
This is a subtask of https://github.com/spacelift-io/spacectl/issues/33

This exists


[fix][misc] Support .nar files without explicit directory entries

Motivation

while working on #19165, I noticed that a test failed. The reason was that the jar file used in the test was no longer an invalid nar file and could also be loaded as a nar file (related to https://issues.apache.org/jira/browse/MJAR-281 being fixed). The difference with #19165 changes was that META-INF/ directory entry is listed in the jar file before META-INF/MANIFEST.MF file.

it's better to accept .nar files where the directory entry might be missing

Modifications

Create the directory for a file entry while extracting the .nar file.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/lhotari/pulsar/pull/137

Since we will start the RC version of 3.0.0 on 2023-04-11, I will change the label/milestone of PR who have not been merged.

The PR of type feature is deferred to 3.1.0
The PR of type fix is deferred to 3.0.1

So drag this PR to 3.0.1


Fix for issue 14 - checksum failing on Ubuntu for kernel > 5.7.2

Hi Ildella,
Your contribution is welcome but you have to add a condition that adapt the behavior since v5.7.2
Once done, I will be happy to accept your MR
Best

Something like this?

I fixed it https://github.com/usbkey9/uktools/commit/bb8fb2b32860bd615f47549e9a8b2c90a3a851be


Better error handling when attachment extension is invalid

Follow-up of #1055

@Benbb96 - You should have permissions to raise PR's directly in Helpdesk going forward inatead of via a fork. Let me know if not.


Minor changes (task #5352)

Table alias sorting columns.
Pass entity instance to sub-element.

Codecov Report

Merging #502 into master will increase coverage by 0.01%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master     #502      +/-   ##
============================================
+ Coverage     29.33%   29.34%   +0.01%
- Complexity      909      911       +2
============================================
  Files            84       84
  Lines          3095     3104       +9
============================================
+ Hits            908      911       +3
- Misses         2187     2193       +6

Impacted Files
Coverage Δ
Complexity Δ

src/Event/Controller/Api/RelatedActionListener.php
0% <0%> (ø)
13 <0> (ø)
:arrow_down:

src/Event/Controller/Api/IndexActionListener.php
70.73% <100%> (+2.31%)
16 <0> (ø)
:arrow_down:

src/Event/Controller/Api/BaseActionListener.php
27.65% <14.28%> (-0.92%)
39 <4> (+2)

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 e3c5c86...53c87df. Read the comment docs.


Add support for ip_range range datatype

It looks like an implementation of the ip_range range datatype that was introduced in 5.5 is needed.

https://www.elastic.co/guide/en/elasticsearch/reference/5.5/range.html
https://www.elastic.co/guide/en/elasticsearch/reference/5.5/release-notes-5.5.0.html#feature-5.5.0
https://github.com/elastic/elasticsearch/pull/24433

Thanks for opening @arledesma, we'll look to get these in the next 5.x and 6.x releases 👍


standardise cosmos-db API version

@asalkeld ptal

looks good, but one of us will have to rebase..


Refactor topology node processor builder

Refactor topology node processor builder
Building processor node must implement Cloneable interface for guarantee one processor instance by stream task.
But state store instances are shared between each stream task ===> It's not correct.
So this issue is for tracking, that we must refactor processor creator with factory which garantee one processor instance and state store instance by stream task.

https://github.com/LGouellec/kafka-streams-dotnet/commit/7a6b9e389ec5c136d1df0567ac68cacb5653401f


PHP version and install oscommerce pb

There the information when I tried to install osc.
Serveur HTTP : 	Apache/2.4.12 (Ubuntu)
Version PHP : 	5.6.11-1ubuntu3.1 (Zend : 2.6.0)
Notice: Use of undefined constant DB_SERVER - assumed 'DB_SERVER' in /home/www/test/ms234/includes/OSC/OM/Db.php on line 32
Notice: Use of undefined constant DB_SERVER_USERNAME - assumed 'DB_SERVER_USERNAME' in /home/www/test/ms234/includes/OSC/OM/Db.php on line 36
Notice: Use of undefined constant DB_SERVER_PASSWORD - assumed 'DB_SERVER_PASSWORD' in /home/www/test/ms234/includes/OSC/OM/Db.php on line 40
Notice: Use of undefined constant DB_DATABASE - assumed 'DB_DATABASE' in /home/www/test/ms234/includes/OSC/OM/Db.php on line 44
Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[HY000] [2005] Unknown MySQL server host 'DB_SERVER' (2)' in /home/www/test/ms234/includes/OSC/OM/Db/MySQL.php:49 Stack trace: #0 /home/www/test/ms234/includes/OSC/OM/Db/MySQL.php(49): PDO->__construct('mysql:dbname=DB...', 'DB_SERVER_USERN...', 'DB_SERVER_PASSW...', Array) #1 /home/www/test/ms234/includes/OSC/OM/Db/MySQL.php(22): OSC\OM\Db\MySQL->connect() #2 /home/www/test/ms234/includes/OSC/OM/Db.php(60): OSC\OM\Db\MySQL->__construct('DB_SERVER', 'DB_SERVER_USERN...', 'DB_SERVER_PASSW...', 'DB_DATABASE', NULL, Array) #3 /home/www/test/ms234/includes/OSC/Sites/Shop/Shop.php(27): OSC\OM\Db::initialize() #4 /home/www/test/ms234/includes/OSC/OM/SitesAbstract.php(29): OSC\Sites\Shop\Shop->init() #5 /home/www/test/ms234/includes/OSC/OM/OSCOM.php(56): OSC\OM\SitesAbstract->__construct() #6 /home/www/test/ms234/includes/OSC/OM/OSCOM.php(27): OSC\OM\OSCOM::setSite(NULL) #7 /home/www/test/ms234/install/includes/application.php(24): OSC\OM\OSCOM::initialize() #8 in /home/www/test/ms234/includes/OSC/OM/Db/MySQL.php on line 49

Development version is under construction
comment out the following two lines in install/includes/application_top.php
  //  OSCOM::initialize();   //  OSCOM::setSite('Setup');

Yes I know, just for information

Thanks for the report, this has already been fixed.


graphql how to get dynamic fields data

I have a table like : Table<address, Table<String, address>>
by graphql, how to get the inner table data?
thx.

@enoloo it's not very clear what you mean by that data structure? What were the queries you tried for fetching the dynamic field objects and did not work?

Hey @enoloo , do you have some example ids I can use? You may be able to reach it by owner { dynamicFields { value { asMoveObject { dynamicFields { value - note, pseudo-code - but would need to verify


Hide your friends

Investigate D1189 and see if it cannot improve the quality of CNL. Specifically, does this easy to load on operator overload resolution?

After some study, not sure this is helpful.


Fix minor bug in mesh mesh intersection

One of the intersection bugs happens due to a mistake when setting up isometries.

@sebcrozet Fixed the bug with isometries not being applied correctly.


Option for running without emit

Hello!
Tried out this package in my project and really like the speed of type checking, but noticed that by default it's generating and writing files to the file system in my source code.
Is there a way to run this where the files are checked in memory, or maybe in a separate cache folder?
Thanks for your work on this!

Thanks, glad you like it.
Are you using the compilerOptions.noEmit option in your tsconfig.json? That should prevent the output you're seeing. See the following from one of the text fixtures (in particular line 8):
https://github.com/policyfly/vue-script-tsc/blob/de94d62bedfe21f61f0de53130433958fb817673/tests/fixtures/tsconfig.clean.json#L2-L9

That was the problem! Adding "noEmit": true to my tsconfig.json solved it. Thank you for the quick response! I'll go ahead and close this but leave it around in case others run into this problem like I did.


UI Gekko Importing Data abort option

Hey Mike,
when i import data from a Exchange, and i define the range of 2017-01-01 00:00:00 to 2017-03-09 12:00:00.
it tells me wrong time range. that is probably just an issue of me.
but, if i define a wrong timerange it still trys to import this range and therefore will never stop to do so. a button to abort the scan would be nice. even for a scan that hast started if you incase made a mistake or something.

If I use those dates it works great, can you post a screenshot of the error?

Sorry, but now im also not able to reproduce it.
excuse me, fault on my side.

No problem, I am happy people are testing it out!

Hey, So I accidentally ran 2 data imports of pretty much the same range within the UI, I'd definitely like to second the option for an 'abort' button on the /imports page under the 'currently running imports' section as well as on the actual import's own page.


[Build] Use actual output path instead of virtual nodes.

Removes the use of virtual nodes when generating llbuild manifest file.
using virtual nodes might cause issues in incremental builds.
Bug: https://bugs.swift.org/browse/SR-1708

Woo awesome, I will take a look when possible...

rebased

I'm not quite sure what to do here...
I think this might be only the tip of the iceberg for missing dependencies, and we need to clean up the model for products and how the build manifest is constructed. It is hard for me to detangle that from this PR...
Is there a test case we can show this fixes? If so I am fine with taking it, if not I think I would rather audit and fix this area from the ground up, but that is more involved and means fixing parts of the model first.

I agree Build should be cleaned up, closing this PR.


Adding a fluxible guide

Example of testing Fluxible components

@jsmey great write up, we really appreciate the effort. I'm not sure we want to include this though; if you look at our current guides we really only provide official guidance on build systems or testing environments. If we added a guide for fluxible, we couldn't really argue against adding it for other misc. libraries (Redux, Relay, Mobx, etc).
I think if anything we'd want to add a guide that addresses the general case of testing HOCs.

I agree. This seems like it would be a great blog that you could post on your own! But for the docs I think general guides are better than specifics. That was the goal of #472

I'm going to close this out then, I agree with @blainekasten, it'd be a great blog post!


remove delivery

Background
No other CTG pipeline have delivery build into them.
Problem/Issue
No consistency if some pipelines do deliver, others do not. Delivery should all be made in a consistent way for all pipelines.
What have I done
Removed delivery.
Check list
I have:

[ ] Tested with Stub run
[ ] Tested on our HPC

Tested branch on ls4 and analysis successfully got to the end and created the web_summary.html file.


change println content cause cargo test failed

Problem
If println is used in test and cargo clean is not run, using cargo test directly will cause the built binary to run incorrectly(signal: 9, SIGKILL: kill)
Note that the first cargo test will not cause an error, it will only cause an error after modifying the content of println
Steps

Copy below code and run cargo test

fn main() {}

#[cfg(test)]
mod tests {
    #[test]
    fn test_foo() {
        println!("hello");
        assert_eq!(1+1, 2);
    }
}

Change the println!("hello"); to println!("hello 2");
Run cargo test found the binary (signal: 9, SIGKILL: kill)

Possible Solution(s)
Do not use the println macro, or run cargo clean after change.
Version
cargo 1.64.0-nightly (8827baaa7 2022-07-14)

I found out that the problem might be zld, it doesn't appear after removing the zld config


A second breakfast first pass to add repl focus cues to the wash repl

Intended to close #106.
I plan to revisit this functionality, I think it would look nice to bind the output panel and the logging panel to when alt or cmd KeyEvents are detected so those key modifiers are more intuitively tied to the output/logging windows for scrolling.
I ran the tests that failed prior without issue so I expect this one to be in the clear.

Are we saying there's a test we expect to fail?

Looks like this needs cargo fmt run against it

For some reason I don't see the Slow blink still but it might just be my terminal, the cyan is plenty to distinguish slightly_smiling_face
Same comment as @autodidaddict , you just need a cargo fmt for this to pass. Thank you for doing this!

Thanks for letting me learn among all of you. It's good for me.


Setting not persisting for UWP

Failure to fill out this information will result in this issue being closed.
Bug Information
Version Number of Plugin: 3.1.1
Device Tested On: Local Machine
Simulator Tested On:
Version of VS: 2017
Version of Xamarin: 4.9.0
Versions of other things you are using:
Xamarin.Forms <IP_ADDRESS>0555
Steps to reproduce the Behavior
I have the following code in App.xaml.cs
var name= CrossSettings.Current.GetValueOrDefault("NAME", "");
CrossSettings.Current.AddOrUpdateValue("NAME", "John Doe");
name= CrossSettings.Current.GetValueOrDefault("NAME", "");
I added a call to Application.SavePropertiesAsync() to no avail.
Expected Behavior
NAME Setting should persist between runs
Actual Behavior
values are stored while running, but on every restart the first GetValueOrDefault returns blank. This only occurs on UWP. The data is persisted when tested on Android.
No exceptions or errors are observed.
Thank you
Code snippet
Screenshots

RESOLVED!
My error. I had been having other issues and set the Debug option in settings for
"Uninstall and then re-install my package. All information about the application is state is deleted"


Update devcontainer image hash: 136a67a

Automated changes by create-pull-request GitHub action

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

Comparison is base (136a67a) 67.12% compared to head (f15f1f0) 67.51%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1412      +/-   ##
==========================================
+ Coverage   67.12%   67.51%   +0.38%
==========================================
  Files         231      231
  Lines       20844    21023     +179
==========================================
+ Hits        13992    14193     +201
+ Misses       6852     6830      -22

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


fix i18n entry

What type of PR is this? (check all applicable)

[ ] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] ♻ Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert
[x] 🌐 Internationalization / Translation

Description
fix i18n entry (Project menu -> save as)

.i18n-update-push

.i18n-update-pull


ModuleNotFoundError: No module named 'typing_extensions'

Current Behavior
Executing import pymovements as pm in a new environment (after running pip install -e .) fails due to missing module type-extensions.
Suggestion: Add typing_extensions as a dependency. Currently it is only an optional test dependecy.
Expected Behavior
Importing the package should not fail.
Minimum acceptance criteria
Importing the package should not fail in a newly created environment.
Failure Information (for bugs)
Steps to Reproduce

git clone repo + cd
conda create -n pymovements
conda install pip
pip install -e .
Run import pymovements as pm

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

Project Version / Commit: pymovements-0.17.3.post26+git.8bcbb161
Operating System: linux

Failure Logs
Traceback (most recent call last):
  File "/data/home/shubi/pymovements/playground/10_min_tutorial.py", line 1, in <module>
    import pymovements as pm
  File "/data/home/shubi/pymovements/src/pymovements/__init__.py", line 26, in <module>
    from pymovements import plotting
  File "/data/home/shubi/pymovements/src/pymovements/plotting/__init__.py", line 35, in <module>
    from pymovements.plotting.traceplot import traceplot
  File "/data/home/shubi/pymovements/src/pymovements/plotting/traceplot.py", line 32, in <module>
    from typing_extensions import TypeAlias
ModuleNotFoundError: No module named 'typing_extensions'

Checklist

[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed
[x] I have provided sufficient information for the team

I was able to reproduce this with both python 3.11 and 3.12

I also reproduced the bug using virtualenv instead of conda

So strange. We removed it in #460 when we dropped support for python 3.7.
We must have thought that typing_extensions are part of python by default?
Anyway, the issue is resolved with #684.
Thank you for reporting @OmerShubi and thanks again for resolving the issue @SiQube


Proposed fix for #115

Changed to calculate from position of left/right-most note, rather than from first/second

Ah. Haven't written checks for this yet. Can confirm that it doesn't break the existing tests, and that the box is 1) consistent width in hand drawn and 2) present in simple. Works on my machine (ha).

All tests pass (but sadly that doesn't check visual layout), and visually checked myself. It looks good.


Initial implementation for multiple outputs

This is sort of a phase one to create an outputs configuration in order to send metrics to more than InfluxDB and begins to address https://github.com/influxdb/telegraf/issues/38.
I'd love some feedback here!

@jipperinbham Hi JP, if you want to fix the 2 comments I made and the branch conficts, then I can test this out and merge it!

@sparrc I've updated the import paths to point to influxdb and rebased from master. Please let me know if something isn't building or running properly.

@jipperinbham It looks like there may have been a merge problem, see ci failure above:
./config.go:220: undefined: ErrInvalidConfig Action failed: go build

sorry about that @sparrc, should be good to test now

@jipperinbham no prob!
This looks good to me, I like the design and fits with telegraf well.
Two questions, and maybe I'm just not 100% understanding the code, but should the sample and test TOML files be changed from [influxdb] to [outputs] ?
2nd: is there any validation that a valid output was specified? not that it needs to validate that the output destination is connectable, but maybe some validation that the configured output exists as an "out plugin" in telegraf?
PS: you can find those files in etc/config.sample.toml and testdata/influx.toml

re: the sample TOML files, I held off on updating those in case there were some changes I needed to make but will do that now
re: output validation, I'm pretty sure the following covers it
https://github.com/jipperinbham/telegraf/blob/outputs-phase1/agent.go#L83-L86

@jipperinbham I think you've accidentally added the telegraf binary to your commit, once that's removed I think the PR is good to go :+1:

@jipperinbham I'm anxious to get this merged, are you done with this PR? I think last we left it, you needed to update the Connect() function?

@sparrc yes, it should be good to go. Sorry I forgot to /cc you on that last commit.
I couldn't fully test because I hit the line protocol change for deprecating ints but all the config and tags looked good.

@jipperinbham I tried to merge this in but had some major conflicts, so I had to revert it, unfortunately I don't think that I'm able to re-open this PR.
Sorry for the hassle, but could you please rebase your changes and re-open a new PR?

@sparrc no problem, I'll rebase and get things cleaned up and submit a new PR

thanks @jipperinbham! Just to warn you, that is probably going to be a pretty hairy merge/rebase, there have been some pretty large changes in agent.go and config.go.
Let me know if you need a hand and we can walk through it together on Google hangouts if need be

It looks like the issue is here, https://github.com/influxdb/telegraf/commit/29e8ce68e44daf4b5e5af6c9deb533bddcab912a, which I'm not entirely sure how to handle since we moved decided to move the Tags to the InfluxDB output and out of Config.
Not entirely sure how to proceed with the changes needed...

@jipperinbham yes, this is what I was afraid of, I think I may have been wrong to suggest moving that into InfluxDB, and that the Tags should in fact stay in Config.

no worries, I can easily look back through the git commits to move it back, it also makes cleaning up these merge conflicts possible without touching what others have added.


Allow Sequence nodes to execute new connections

This is an update to the Sequence node, based on what we discussed shortly on Discord.
This adds a new flag to Sequence, which makes it track which connections it executed previously. If new connections are added or old ones are changed, it can then execute the changed connections when loading a save game.
Reasoning: This allows the developer to add new logic into existing flowgraphs, without causing issues in savegames. Without this feature, if a player has saved the game and new logic is added into a graph, there is no mechanism to ensure the logic is ran when the game is loaded. With this change, the developer can simply update the sequence connections and new nodes will execute after the game is loaded.
I don't know if this is the optimal/best way to go about it, but I figured I'll open this PR for any ideas/discussion since I had this code anyway.
Example scenarios:
Add a new connection:

Developer creates a sequence and enables the save pin execution flag.
Developer adds one connection to the node
Player plays, the sequence is executed, player saves the game
Developer adds a new connection to the sequence node
Players load game, since the save flag was enabled, the newly added connection is executed, allowing the player to get the new features

Replace a connection
Same as above, but instead of adding a new pin, the developer can replace an existing connection with a new node.
The code stores the FGuid of the connected node, so it can detect when the connected node changes.

Submitted change to the Sequence node: bSavePinExecutionState is now True by default. Thanks again! :)


System.IO.File' does not contain a definition forWriteAllBytes'

@InvexGames hi! it seems im facing issue regarding Assets/MaterialUI/Scripts/ShadowGen.cs(292,40): error CS0117: System.IO.File' does not contain a definition for WriteAllBytes'
how can i fix it ? any solution please ?

Hello,
What platform are you targeting?

android . can you help me how to fix it ?

What version of Unity are you using?

using unity version 5.2.0
Pada tanggal 6 Jun 2016 22.23, "Yohan Hadjedj"<EMAIL_ADDRESS>menulis:

[image: Boxbe] https://www.boxbe.com/overview This message is eligible
for Automatic Cleanup<EMAIL_ADDRESS>Add cleanup rule
https://www.boxbe.com/popup?url=https%3A%2F%2Fwww.boxbe.com%2Fcleanup%3Fkey%3D5A%252Fla8td%252FNQAva%252FqpuN5nGY5NivHJpnroSV5nW8AF7I%253D%26token%3D1xcDzxDj8JBfSpnd0uotuP85PWZS21EWdoTso%252BTuJZmMhy2zQ44%252BpMXcJTZ1IlpJzfzpHJIwj6NHOugVfxoy2wjwL%252BBJ3iYIyRV2v45J3bXo21eRJZmR4K9CvMTCrL169w%252FLWfLVtrwsBnX09gkszg%253D%253D&tc_serial=25664448379&tc_rand=109247157&utm_source=stf&utm_medium=email&utm_campaign=ANNO_CLEANUP_ADD&utm_content=001
| More info
http://blog.boxbe.com/general/boxbe-automatic-cleanup?tc_serial=25664448379&tc_rand=109247157&utm_source=stf&utm_medium=email&utm_campaign=ANNO_CLEANUP_ADD&utm_content=001
What version of Unity are you using?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/InvexGames/MaterialUI/issues/56#issuecomment-223992158,
or mute the thread
https://github.com/notifications/unsubscribe/AK5RKwKc_cpLDnYr7JIplTnZ2uNHtgx6ks5qJDsXgaJpZM4IuHkH
.

Hum... are you sure you're on Android?
Seems like you're targeting WebPlayer that is not supported.
Go to File -> BuildSettings and switch to Android platform.

Yes, using for my android.
Pada tanggal 7 Jun 2016 22.55, "Yohan Hadjedj"<EMAIL_ADDRESS>menulis:

[image: Boxbe] https://www.boxbe.com/overview This message is eligible
for Automatic Cleanup<EMAIL_ADDRESS>Add cleanup rule
https://www.boxbe.com/popup?url=https%3A%2F%2Fwww.boxbe.com%2Fcleanup%3Fkey%3DEZkjXH8mylAB0z26OneXXex8lHgSx5UtAumhqrSC%252BuE%253D%26token%3DBPJiLhe70%252FHhxU3QXTcZhAYCl%252Fd1Kx9XvX1NVwz%252BxjoC3hDFkOyVS%252FiwmaxPR84Y8qw7YNfUa6%252FvPiLaQuPWFuR6RyfDbSlDa6sSz96dV%252FLRgGEGjWHfROV4gWwDuUdmWsR5a6jkCX92LCNGDTvjAg%253D%253D&tc_serial=25676952179&tc_rand=630801703&utm_source=stf&utm_medium=email&utm_campaign=ANNO_CLEANUP_ADD&utm_content=001
| More info
http://blog.boxbe.com/general/boxbe-automatic-cleanup?tc_serial=25676952179&tc_rand=630801703&utm_source=stf&utm_medium=email&utm_campaign=ANNO_CLEANUP_ADD&utm_content=001
Hum... are you sure you're on Android?
Seems like you're targeting WebPlayer that is not supported.
Go to File -> BuildSettings and switch to Android platform.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/InvexGames/MaterialUI/issues/56#issuecomment-224325923,
or mute the thread
https://github.com/notifications/unsubscribe/AK5RKyEMc6646rQ7UzIIsmTBEdeI0GHgks5qJZRbgaJpZM4IuHkH
.

@nah0y  the unity version is 5.3 . should i download only the beta ?

yep i already download the android platform thanks @nah0y  . now my new problem is when i click ADD OPEN SOURCES i go to the  folder where i put what im download. then the android platform is .EXE but the only choices there to import is .UNITY and ALL FOLDERS? question is how can i import the android platform ?

I don't understand a single thing of what you said, sorry lol


Prepare Solidus Digital for Solidus 3.0

Update this extension so that it works with Solidus 3.0 by making sure the specs pass.

@derrickp and myself are going to pick this up.

Ha, I pulled it down to look at what was up with the factory loading. 😅

Looks like this was closed by #20 - thanks! 🎉


Use different middleware for different routes with App Engine

It's already fairly clear how to set up different middleware for different routes when using standard Go, however, when it comes to using App Engine, it's not clear to me.  In App Engine, we have to pass in a *web.Mux to a http.Handle in the init().
It would be possible to do it with something like this:
func init() {

m1 := web.New()
m1.Use(middleware1)
m1.Post("some/route", handler1)
http.Handle("/some/route", m1)

m2 := web.New()
m2.Use(middleware2)
m2.Post("some/other/route", handler2)
http.Handle("/some/other/route", m2)

...
}

While that would work well if the middleware acts on families of routes (such as "admin", or "user"), in cases where the middleware needed isn't as related, it would result in having calls like this for almost every handler, which would get messy.
Is there a clean way in App Engine of setting up a single *web.Mux to have different middleware groups that can be applied on a per route basis?  Alternatively is there any way of inlining middleware in the m.Post(), m.Get(), etc?

The short answer is that you can wrap individual routes by providing a
handler that returns a compatible type - a web.HandlerFunc or web.Handler -
e.g. mux.Get("/", SomeMiddleware(SomeHandler)) where SomeMiddleware might
look like func(c web.C, w http.ResponseWriter, r *http.Request) { ... }
Middleware in general isn't wholly designed to apply to single routes
(middleware generally is a way to action the same logic over many routes),
which is why it's a little less obvious.
On Wed, Jan 6, 2016 at 12:35 AM John Beckett<EMAIL_ADDRESS>wrote:

It's already fairly clear how to set up different middleware for different
routes when using standard Go, however, when it comes to using App Engine,
it's not clear to me. In App Engine, we have to pass in a *web.Mux to a
http.Handle in the init().
It would be possible to do it with something like this:
func init() {
m1 := web.New()
m1.Use(middleware1)
m1.Post("some/route", handler1)
http.Handle("/some/route", m1)
m2 := web.New()
m2.Use(middleware2)
m2.Post("some/other/route", handler2)
http.Handle("/some/other/route", m2)
...
}
While that would work well if the middleware acts on families of routes
(such as "admin", or "user"), in cases where the middleware needed isn't as
related, it would result in having calls like this for almost every
handler, which would get messy.
Is there a clean way in App Engine of setting up a single *web.Mux to
have different middleware groups that can be applied on a per route basis?
Alternatively is there any way of inlining middleware in the m.Post(),
m.Get(), etc?
—
Reply to this email directly or view it on GitHub
https://github.com/zenazn/goji/issues/175.

This is unfortunately pretty hard to do within the confines of Goji's abstractions—like Matt said, Goji's middleware are designed to apply across all of your routes, and don't work well for a more pick-and-choose style.
You might find it a bit easier to use a standalone middleware / http.Handler-wrapping abstraction. Thankfully, the interfaces here are pretty straightforward to use, but unfortunately I'm not sure of a good off-the-shelf library that you can drop in.
A brief note on AppEngine in particular: if you're looking to avoid the built-in net/http mux as much as you can, I'd suggest registering a Goji mux like so:
root := web.New()
http.Handle("/", root)

This allows you to "get out" of net/http as soon as possible, and let Goji handle all of your application's routing.


Not able to build on Ubuntu 14.04

Hi,
I tried to build in a fresh docker container with ubuntu:14.04 image according to https://github.com/hgomez/obuildfactory/wiki/How-to-build-and-package-OpenJDK-8-on-Linux, but got stuck on libfreetype6-dev not being recognized. I have also got to the same error trying to build in a debootstrap environment according to http://blog2.vorburger.ch/2014/06/build-your-own-jdk-at-home.html .
sudo apt-get update
sudo apt-get install git mercurial zip bzip2 unzip tar curl
sudo apt-get install ccache make gcc g++ ca-certificates ca-certificates-java
sudo apt-get install libX11-dev libxext-dev libxrender-dev libxtst-dev libxt-dev
sudo apt-get install libasound2-dev libcups2-dev libfreetype6-dev
sudo apt-get install build-essential ruby ruby-dev
sudo apt-get install openjdk-7-jdk
sudo gem install fpm
mkdir ~/openjdkathome
cd ~/openjdkathome
git clone https://github.com/hgomez/obuildfactory.git

#  the above commands did their work, then:

export JAVA_HOME=/usr/lib/jvm/java-7-openjdk-amd64
export PATH=$JAVA_HOME/bin:$PATH
XUSE_NEW_BUILD_SYSTEM=true XBUILD=true ./obuildfactory/openjdk8/linux/standalone-job.sh
.....
.....
checking cups/ppd.h usability... yes
checking cups/ppd.h presence... yes
checking for cups/ppd.h... yes
configure: error: Could not find freetype! You might be able to fix this by running 'sudo apt-get install libfreetype6-dev'.
configure exiting with result code 1
make: *** No rule to make target `images'.  Stop.
can't find java into JDK /root/openjdkathome/sources/openjdk8/build/linux-x86_64-normal-server-release/images/j2sdk-image, build failed

(Needless to say, libfreetype6-dev is installed)
sudo apt-get install libfreetype6-dev
Reading package lists... Done
Building dependency tree
Reading state information... Done
libfreetype6-dev is already the newest version.
0 upgraded, 0 newly installed, 0 to remove and 5 not upgraded.

This might be perhaps related to:

http://mail.openjdk.java.net/pipermail/build-dev/2013-October/010874.html
http://mail.openjdk.java.net/pipermail/build-dev/2013-October/010856.html

Same happens to me using Ubuntu 16.04 Beta2 (Ubuntu Xenial Xerus), but works fins using Linux Mint 17.3

No activity/reports on this ticket, feel free to reopen a new one

Ran into this issue too, using Ubuntu 14.04 x86 (yeah, i know, it's outdated...).
To expand a bit on mcfongtw's comment, the configure file is under sources/openjdk8 and I had to run:
bash configure --with-target-bits=32 --with-freetype-include=/usr/include/freetype2/ --with-freetype-lib=/usr/lib/i386-linux-gnu
Building worked afterwards.


Update PBS Go Java Feature List

Hey Scot, is this a docs change or is there a code PR as well?

@MartianTribe This is a docs change. Catching up on features we previously added.


Update RetroWatchArduino_no_button.ino

PROGMEM const char* const weekString[] = {"", "Sun", "Mon", "Tue", "Wed", "Thu", "Fri", "Sat"}; PROGMEM const char* const ampmString[] = {"AM", "PM"}; Changing this will remove many issues

avrdude: stk500_recv(): programmer is not responding
avrdude: stk500_getsync() attempt 1 of 10: not in sync: resp=0xee
avrdude: stk500_recv(): programmer is not responding
avrdude: stk500_getsync() attempt 2 of 10: not in sync: resp=0xee
avrdude: stk500_recv(): programmer is not responding
avrdude: stk500_getsync() attempt 3 of 10: not in sync: resp=0xee
avrdude: stk500_recv(): programmer is not responding
avrdude: stk500_getsync() attempt 4 of 10: not in sync: resp=0xee
avrdude: stk500_recv(): programmer is not responding
avrdude: stk500_getsync() attempt 5 of 10: not in sync: resp=0xee
avrdude: stk500_recv(): programmer is not responding
avrdude: stk500_getsync() attempt 6 of 10: not in sync: resp=0xee
avrdude: stk500_recv(): programmer is not responding
avrdude: stk500_getsync() attempt 7 of 10: not in sync: resp=0xee
avrdude: stk500_recv(): programmer is not responding
avrdude: stk500_getsync() attempt 8 of 10: not in sync: resp=0xee
avrdude: stk500_recv(): programmer is not responding
avrdude: stk500_getsync() attempt 9 of 10: not in sync: resp=0xee
avrdude: stk500_recv(): programmer is not responding
avrdude: stk500_getsync() attempt 10 of 10: not in sync: resp=0xee
Failed uploading: uploading error: exit status 1


Slugbuilder build with private buildpack repo fails on builder v2.6.0+

Due to the changes introduced by #444 / #447 the environment variable SSH_KEY is no longer passed to the sludbuilder environment, however this variable is required by the build.sh script in slugbuilder to be able to clone buildpacks from private git repos, so this variable needs to be whitelisted.
See https://github.com/deis/slugbuilder/blob/master/rootfs/builder/build.sh#L102
This is related to issues #454 and #455.

Note: For other people running into this issue, you can work around it by editing the deis-builder deployment and changing the version of the builder image back to v2.5.5 until a fix is available.

fixed in https://github.com/deis/slugbuilder/pull/126


fix: add win32 paths support to getCallerInfo()

PR for the issue https://github.com/storyofams/next-api-decorators/issues/236

@leeuwis all done, please re-review and merge


My abuild plugin can be debugged and run normally in versions before 1.92, but it cannot run in subsequent versions (1.92-1.95)

#

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.92.2,1.93.1,1.94.2,1.95.3
OS Version: ubuntu22.04

Steps to Reproduce:

Open abuild development project
F5 Debugging
Debug console shows an error, the abuild plugin cannot start

My abuild plugin can be debugged and run normally in versions before 1.92, but it cannot run in subsequent versions (1.92-1.95)
The abuild plugin is a plugin I developed for Linux compilation and development. It has been working well in versions before 1.92 and can maintain the display of the debugging window in versions 1.92-1.94. However, the plugin cannot be started, and the debugging window in version 1.95 exits directly, mainly reporting NLS related errors.
For example, reporting an error in 1.94.2:
Error: !!! NLS MISSING: 523 !!!
at Ny (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:29:4480)
at p (/usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:29:4372)
at /usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:31:83929
at ModuleJob.run (/home/yk/GIT/vscoodePlugIn/abuild/lib/internal/modules/esm/module_job.js:222:25)
at async ModuleLoader.import (node:internal/modules/esm/loader:316:24) {stack: 'Error: !!! NLS MISSING: 523 !!!
at Ny (fi…ort (node:internal/modules/esm/loader:316:24)', message: '!!! NLS MISSING: 523 !!!'}
arg0 =
Error: !!! NLS MISSING: 523 !!!\n    at Ny (file:///usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:29:4480)\n    at p (file:///usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:29:4372)\n    at file:///usr/share/code/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:31:83929\n    at ModuleJob.run (node:internal/modules/esm/module_job:222:25)\n    at async ModuleLoader.import (node:internal/modules/esm/loader:316:24) {stack: 'Error: !!! NLS MISSING: 523 !!!\n    at Ny (fi…ort (node:internal/modules/esm/loader:316:24)', message: '!!! NLS MISSING: 523 !!!'}
t @ /usr/share/code/resources/app/out/bootstrap-fork.js:16:1498

Extensions are not impacted by ESM.


Get-PSResource doesn't support prerelease tags

Find-PSResource works fine:

Get-PSResource for the same installed module does not unless you explicity drop the tag. The behavior should be consistent, recommend implementing -prerelease for Get-PSResource as well as allowing it to match on tagged version strings.

Thanks, just confirmed that I am still seeing this issue


Intellisense (auto-complete) not working in for-of

Issue Type: Bug
steps to reproduce:
//create an array of any type:
let my_arr:MyType[] = [];

// bring a for-of snipet
// forof (enter)  becomes:

for (const iterator of object) {
}

// the cursor will be positioned in "iterator" variable. Intellisense can't help there (OK)
// hit tab one time and the cursor will go to the "object" variable. (OK)
// start to write "my_" and Intellisense is completelly silent in this section
// the expected behavior is the auto-complete box appears with "my_arr" as an option. (or any variable in the scope)
// is not a too problematic bug because I can hit ctrl+space to force Intellisense to come out, but is a little anoying since the rest of the code works without ctrl+space  (like on IFs)
VS Code version: Code 1.46.0 (a5d1cc28bb5da32ec67e86cc50f84c67cc690321, 2020-06-10T08:59:04.923Z)
OS version: Linux x64 5.3.0-59-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-3630QM CPU @ 2.40GHz (8 x 2372)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
0, 0, 0

Memory (System)
15.59GB (7.76GB free)

Process Argv
--no-sandbox --unity-launch /home/luis/devenv/i3track/apis/maxtrack/workunit

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

Extensions (2)

Extension
Author (truncated)
Version

xml
Dot
2.5.0

vscode-docker
ms-
1.3.0

Try setting "editor.suggest.snippetsPreventQuickSuggestions": false

Thanks for the sugestion but it isn't a good solution.
Now when I start typing in the iterator part (I am naming a fresh new variable there) snippets suggestions pops and if I hit tab to go to the object part the snippet suggestion is accepted breaking the code.
The IF statement now suffers for the same problem.
All I wanted was that the "for-of" worked like the "if" statement in regard to code-suggestions.
But thanks for the try.
I can continue hitting ctrl+space in the object sesction. No problem.

/cc @jrieken for feedback on snippet behavior


svdtools: 0.1.23 -> 0.2.5

Description of changes

svdtools was rewritten in rust for v0.2
svdtools is a set of tools to handle vendor-supplied, often buggy SVD files.
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Is there a reason you're using fetchCrate instead of fetchFromGitHub?


Update tests env names

The original test_envs.py will fail due to the incorrect name Humanoid in the agent_list. This PR aims to update the agent list and task list to be the same as README

Thanks for your contribution!


convert to typescript

Is there any reason this project is not written in typescript?
I'd be happy to take that on

I think I'd support that. What @shawntabrizi would say?
Before that though I'd prefer https://github.com/paritytech/polkadot-launch/pull/28 to be merged 😄

I think I'd support that. What @shawntabrizi would say?
Before that though I'd prefer https://github.com/paritytech/polkadot-launch/pull/28 to be merged 😄

FWIW, it's merged now

OK @shawntabrizi Can I go ahead and convert the code to typescript?

Of course, go for it :)


Q: e-library offering books for free

How could we classify an e-library that offers books for free? like this one here (FIY 'kutub' in Arabic means 'books'): https://www.kutub.info/library/book/21784
The texts on the site are probably describing the books with the intent to get the potential readers interested in them and thus they are most likely book blurbs and would be classified as Description with intent to sell. This is of course a bit contradictory, since the books are not sold, but I'd probably annotate them as described.

edited IP-ds


Global scope game state storage

Storing the state of the game in global scope is a bad practice
https://github.com/rando-idiot/Terminus.JS/blob/bbd663f897dfff103f71800e38337470a8ad2b7d/terminus.js#L31-L41
users have full access to these variables this way, this leads to a really messy environment

i can do this by typing this in the console:
this
Window https://rando-idiot.github.io/Terminus.JS/

basegain: 1

baseup: function baseup()
begin: function begin()

beginbought: 0

configYML: function configYML()

configymlbought: 0

doctype: function doctype()

doctypebought: 0

helloworld: function helloworld()
index: function index()

indexbought: 0

info: function info()
infshop: function infshop()

infunlock: 0

points: 0

push1: function push1()
push2: function push2()
shop: function shop()
sleep: function sleep(milliseconds)
stepfour: function stepfour()
stepfour2: function stepfour2()

stepfouradd: 1

stepone: function stepone()
stepone2: function stepone2()

steponeadd: 0

stepthree: function stepthree()
stepthree2: function stepthree2()

stepthreemult: 1

steptwo: function steptwo()
steptwo2: function steptwo2()

steptwomult: 1

update: function update()

updateloop: 1

upgbonus: function upgbonus()

upgbonusvar: 1

upgpriceboost: 0

upgstage: 0

this is fixed by using let instead of var

What is the difference between "let" and "var"?

ok 👍 will fix, mostly had it like this for testing purposes but ill def fix this from now on

fixied


Bug with test reporting and Portal middleware

Definitely an edge case: in HoneySQL, there's a test that uses an unhashable Object to test a specific equality check and it "breaks" Portal.
Repro case here: https://github.com/seancorfield/portal-bug-repro
I tracked it as far as the clojure.test interceptions that Portal's middleware puts in place. Hope the repro case is helpful!

Thanks for the repro case! I think https://github.com/djblue/portal/commit/486bcc59ed8dff56a4deec3a3d27bf93c1f5ba92 should resolve the issue.

Nice! Thank you.


[CS2103-T14-1] SeniorLove

SeniorLove is an application for social workers from volunteer centres to manage contacts and profiles of the elderly with no or limited family support, as well as to contact the elderly regularly to check in on their well-being.

Codecov Report

Merging #18 (21728ef) into master (ba53b8c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #18   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 ba53b8c...21728ef. Read the comment docs.


exports from ".d.ts" files should be dropped as it was before 3.9

Bug Report
exports from ".d.ts" files should be dropped as it was before 3.9
there wil be no javascript file out of ".d.ts" file, so there should not be a "require" from not exisiting file.
🔎 Search Terms
export retained
exports
🕗 Version & Regression Information
this is valid starting 3.9 version, it is documented here:
https://www.typescriptlang.org/docs/handbook/release-notes/typescript-3-9.html#export--is-always-retained
but it breaks a lot of code, because of exports from ".d.ts" files.
⏯ Playground Link
Playground link is not possible because multiple files needed.
💻 Code
index.ts:
// ... here other normal exports from ".ts" files
export * from './foo'

foo.d.ts:
export interface Foo {}

🙁 Actual behavior
the compiled javascript contains require('./foo') but there is no such file.
🙂 Expected behavior
require('./foo') should be dropped from javascript file

This was an intentional change. You can use export type instead if you don't want the require in the emitted code.

This was an intentional change. You can use export type instead if you don't want the require in the emitted code.

This was an intentional change. You can use export type instead if you don't want the require in the emitted code.

how exactly? what should I put instead of export * from './foo'; and export all type definitions from it?
as far as I know export type * from './foo'; is also not possible?
yes I can put for example export {Foo} from './foo' but the point is that we have a large project and this change breaks it...

This was an intentional change. You can use export type instead if you don't want the require in the emitted code.

how exactly? what should I put instead of export * from './foo'; and export all type definitions from it?
as far as I know export type * from './foo'; is also not possible?
yes I can put for example export {Foo} from './foo' but the point is that we have a large project and this change breaks it...


Introduce get_transaction_merkle_proof RPC

Change Description
Merkle proof is very basic feature used in inter-blockchain communication. It can be done in application level, but it would be helpful provided by RPC API.
Consensus Changes

[ ] Consensus Changes

API Changes

[x] API Changes

/v1/chain/get_transaction_merkle_proof is added.
Documentation Additions

[x] Documentation Additions

API needs to be documented.

@swatanabe-b1
Thank you for your review. Is there any conventional way to test RPC in this repository? I couldn't find any tests for other RPCs, so I'm not sure where to put my test in.

It should probably go in tests/.  You can add a getProof method in tests/Node.py.

This should really be handled in the SDKs instead as a new RPC endpoint.

@swatanabe-b1 This proof can be acquired from the result of /v1/chain/get_block with full list of transactions, but I thought that getting whole block costs too much for getting a proof of one transaction. Can you let me know what SDK you mentioned means? (eosjs or history-tools?)


feature: adding function for remove all previous register keys

Did you check the docs?

[X] I have read all the which-key.nvim docs

Is your feature request related to a problem? Please describe.
Adding function for remove all previous register keys would make it easier to work with lazyvim or lunarvim style configurations.
Thanks a lot
Describe the solution you'd like
a function style :
local wk = require("which-key")
wk.clean()
Describe alternatives you've considered
no one
Additional context
No response

It would also be nice to be able to do this on a buffer specific basis. For example if I have a filetype/buffer type that I want to treat as an independent "mode" then I could clear out the all of the which-key menu items for that buffer and add back only the things relevant to that buffer to give me a very focused view of what I can do.

I'd love to see a function to clear the mappings as well.
There is a workaround for now, but it is only cosmetic.
We can hide Keybindings, that we don't want to see with which_key_ignore
it has a few drawbacks though. Let's say, for example, we load keybindings based on the active buffers file type.
1.) Each Mapping must be aware of all other existing mappings, so it can hide the keys, it does not need.
2.) The Hidden keys are still active. So if you hit the keys on accident, they still work.

@mehalter check the install section of the new docs.
<leader>? is what you want.
It's basically a help window for any plugin.


Remove usage of myget NuGet feeds since 1.0.3 has been released

@naamunds

Replaced with https://github.com/dotnet/dotnet-docker-nightly/pull/143