Fix build issue resulting from referencing an invalid git commit

Fixes #15. As documented in the issue, building the project in its current form will fail for some users (cargo's caching allows some of us to build). This is due to a now invalid commit hash being referenced in Cargo.lock.
Previously we were using commit f8c26aa4 from the build branch of svd-parser, but that doesn't seem to exist in the tree anymore. 1b5234d4 is quite similar (it looks like there was a rebase or something) so that can be used instead with only one small adjustment, which I have addressed.

Thanks @jessebraham, looks like that PR was merged, so I've applied your fix in master, using the current master commit.


Using ^ to search the beginning doesn't work properly

Info

Operating System: macos
Shell: zsh
Terminal: iTerm2
nvim --version:

NVIM v0.9.5
Build type: Release
LuaJIT 2.1.1703358377

fzf --version: 0.46.1 (brew)

[ ] The issue is reproducible with mini.sh (never tried, but I don't expect it to be an issue)

fzf-lua configuration

local fzf = require('fzf-lua')

local actions = fzf.actions
fzf.setup({
  'fzf-native',
  actions = {
    files = {
      ['default'] = actions.file_edit
    }
  },
  previewers = {
    git_diff = {
      pager = "delta --width=$FZF_PREVIEW_COLUMNS"
    }
  }
})

n('<LEADER>e', function()
  fzf.files({ cmd = 'fd' })
end)

n('<LEADER>E', function()
  fzf.files({ cmd = 'fd --no-ignore' })
end)

Description
I'm unable to search from the beginning of the string. See:

OTOH, if I include the folder icon in my search, it works:

I expect the use of ^ to work despite dev icons.

That’s because fzf doesn’t interpret the icon as not part of the string, if you want that or work add file_icons = false.

By using the same logic fzf-lua uses for parsing the file entries we can utilize fzf's --nth (with --delimiter) to be able to ignore the icons in the search:
local fzf = require("fzf-lua")
fzf.files({
  git_icons = false,
	fzf_opts = {
    ["--delimiter"] = string.format("[:%s]", fzf.utils.nbsp),
    ["--nth"] = "2..",
  },
})

Great plugin BTW. I use fzf a lot both in vim and out of it!
Thanks for the solution. I'll change my config to this and try it.
Since many users probably want this to work properly with icons, the default experience with icons should be sensible. This solution should be baked into the plugin.

https://github.com/ibhagwan/fzf-lua/commit/106554585dbb66171d75799bde4a30d31324cdf4
@bigH, should work for all file-type pickers (buffers, args, git-files, etc).

Thanks for doing this. I'll test the latest release soon.


Add email and username to the upsert script

Email and username needed for upsert script : https://github.com/nationalarchives/tdr-generated-graphql/actions/runs/8063060430/job/22024121463

Looks good. Thanks. So is this need for all the "upsert" scripts?

Yeah it's for the upsert script as I saw an error around not being able to identify who the user is. Not sure how this repo is different to the others as the other ones don't ask for a email or user


Failed task is no longer highlighted

Organization Name:
Microsoft
Short summary about the issue/question:
In new job detail page, failed task is no longer highlighted, then user can't glance over task status.

This is old page:

Brief what process you are following:

OpenPAI Environment:

OpenPAI version: v0.11

there should have been a container state field to replace the "exit code", indicating whether it fails or succeeds.

#2382

Closing as dup, will track in #2382


Some is* functions are missing from the types module

Not all available types can be checked with the is* functions contained in the types module. Some are missing.

Hello @daniele-rapagnani. I'd like to try to tackle this as my first open source issue! Can you clarify which types are missing? I see a list of MELON_TYPE_* defined in value.h, some of which do not have counterparts in types_module.c. Are theses the ones that need added? Thanks!

Hello @daniele-rapagnani. I'd like to try to tackle this as my first open source issue! Can you clarify which types are missing? I see a list of MELON_TYPE_* defined in value.h, some of which do not have counterparts in types_module.c. Are theses the ones that need added? Thanks!

Hey @NathanGugel, thanks for giving this a try!
Yes you are on the right track, the ones that are missing are:
MELON_TYPE_NATIVEPTR
MELON_TYPE_SMALLRANGE
MELON_TYPE_NATIVEIT
MELON_TYPE_SYMBOL
MELON_TYPE_RANGE

MELON_TYPE_FUNCTION can be ignored as it's only used internally and is not exposed to the language.
The only challenge here is that the isRange function should check for both MELON_TYPE_SMALLRANGE or MELON_TYPE_RANGE. The small range type is just a normal range value but packed inside an integer, while the normal range is a garbage collected object. This makes no difference to the user though, so isRange should return true for both types. This means that you can't use the existing macro melM_defineCheckType, you should write a new function based on the macro's content.
I also noticed that I forgot to add unit tests for the types module. If you want you can add them by simply adding a types.ms file in tests/fixtures/language in which you test the various is* functions and a corresponding types.out file with the expected output. You can take a look at the docs to understand how to create each type of value to be tested. MELON_TYPE_NATIVEPTR is the only value that can't be created from the language itself, you can leave it untested for now.
Although if you just want to add the is* functions and leave out the tests it's absolutely fine, I'll do it in another issue.

Hey @NathanGugel, thanks for giving this a try!
Yes you are on the right track, the ones that are missing are:
MELON_TYPE_NATIVEPTR
MELON_TYPE_SMALLRANGE
MELON_TYPE_NATIVEIT
MELON_TYPE_SYMBOL
MELON_TYPE_RANGE

MELON_TYPE_FUNCTION can be ignored as it's only used internally and is not exposed to the language.
The only challenge here is that the isRange function should check for both MELON_TYPE_SMALLRANGE or MELON_TYPE_RANGE. The small range type is just a normal range value but packed inside an integer, while the normal range is a garbage collected object. This makes no difference to the user though, so isRange should return true for both types. This means that you can't use the existing macro melM_defineCheckType, you should write a new function based on the macro's content.
I also noticed that I forgot to add unit tests for the types module. If you want you can add them by simply adding a types.ms file in tests/fixtures/language in which you test the various is* functions and a corresponding types.out file with the expected output. You can take a look at the docs to understand how to create each type of value to be tested. MELON_TYPE_NATIVEPTR is the only value that can't be created from the language itself, you can leave it untested for now.
Although if you just want to add the is* functions and leave out the tests it's absolutely fine, I'll do it in another issue.

Hey @daniele-rapagnani. Happy to give the unit tests a try as well after I finish adding the is* functions!
Should I make two pull requests or just one after I've finished both parts? (Still figuring out the github workflow)

Hey @daniele-rapagnani. Happy to give the unit tests a try as well after I finish adding the is* functions!
Should I make two pull requests or just one after I've finished both parts? (Still figuring out the github workflow)

That's great!
Making just one pull request would be better because you can associate it with this issue so that it's easier to see what issue caused what changes. Also the unit tests are meant to test the changes that you are going to introduce, so it's all part of the same task.

That's great!
Making just one pull request would be better because you can associate it with this issue so that it's easier to see what issue caused what changes. Also the unit tests are meant to test the changes that you are going to introduce, so it's all part of the same task.

@daniele-rapagnani Should NATIVEPTR be isPointer and NATIVEIT be isIterator to the user? What does the "native" mean?

@daniele-rapagnani Should NATIVEPTR be isPointer and NATIVEIT be isIterator to the user? What does the "native" mean?

Hey @daniele-rapagnani
I added the is* functions and am working on the unit tests. I realized that it would be nice if I could run them to make sure I'm writing them correctly and compare the outputs manually for now. However, I am not sure how to run melon scripts (is that what the .ms stands for?). I was able to download the mingw linker and build the project with cmake (though I cannot "make" since I am on Windows), but I don't know how to use the build folder to actually run a script. Any help would be appreciated! For now, I will do my best on the unit tests anyway.
Also, I'm not sure how to write tests for isProgram. The Program type is not in the docs or in the unit tests, so should I skip that one?

Hey @daniele-rapagnani
I added the is* functions and am working on the unit tests. I realized that it would be nice if I could run them to make sure I'm writing them correctly and compare the outputs manually for now. However, I am not sure how to run melon scripts (is that what the .ms stands for?). I was able to download the mingw linker and build the project with cmake (though I cannot "make" since I am on Windows), but I don't know how to use the build folder to actually run a script. Any help would be appreciated! For now, I will do my best on the unit tests anyway.
Also, I'm not sure how to write tests for isProgram. The Program type is not in the docs or in the unit tests, so should I skip that one?

@daniele-rapagnani Should NATIVEPTR be isPointer and NATIVEIT be isIterator to the user? What does the "native" mean?

Sorry @NathanGugel I missed this one! The "native" just means that it's implemented in C, iterators can be implemented in melon by overloading the next iterator operator on an object, but most core types have iterators implemented in C to make them as fast as possibile (for example arrays or ranges both have iterators implemented natively)
The NATIVEPTR type can be used to wrap a C/C++ pointer in a melon value so that it can be passed to the VM and back, for example because you want to wrap some C++ instance in a melon object and still be able to get the C++ instance back do do stuff with it. These values are opaque to the melon developer, which can't really do anything with them but still being able to test for them may be useful.

However, I am not sure how to run melon scripts (is that what the .ms stands for?)

Yes! Not very immaginative on my part, I know 😄

I was able to download the mingw linker and build the project with cmake (though I cannot "make" since I am on Windows), but I don't know how to use the build folder to actually run a script. Any help would be appreciated!

If you install MSYS2 along with MinGW you should have a unix like environment (with make and such) to build the project, if you don't run make in your build directory, you only have the Makefiles generated by cmake .., but still nothing has been built yet.
Regardless you have a couple of options here:

You can look inside the github action I use to build on windows to see how I do it
You can download a pre-built executable from a previous github action run, for example this one: https://github.com/daniele-rapagnani/melon/actions/runs/477177890. On the bottom you'll find the artifacts which are archives containing the pre-built executable. You can then simply run melon.exe yourfile.ms on the command line and it should work.

Also, I'm not sure how to write tests for isProgram. The Program type is not in the docs or in the unit tests, so should I skip that one?

The Program type represents the code inside a melon source file and it's a garbage collected object so it gets its own type. At the moment it's not exposed to the user in any way so I think you are right, we should skip it as well!

@daniele-rapagnani Should NATIVEPTR be isPointer and NATIVEIT be isIterator to the user? What does the "native" mean?

Sorry @NathanGugel I missed this one! The "native" just means that it's implemented in C, iterators can be implemented in melon by overloading the next iterator operator on an object, but most core types have iterators implemented in C to make them as fast as possibile (for example arrays or ranges both have iterators implemented natively)
The NATIVEPTR type can be used to wrap a C/C++ pointer in a melon value so that it can be passed to the VM and back, for example because you want to wrap some C++ instance in a melon object and still be able to get the C++ instance back do do stuff with it. These values are opaque to the melon developer, which can't really do anything with them but still being able to test for them may be useful.

However, I am not sure how to run melon scripts (is that what the .ms stands for?)

Yes! Not very immaginative on my part, I know 😄

I was able to download the mingw linker and build the project with cmake (though I cannot "make" since I am on Windows), but I don't know how to use the build folder to actually run a script. Any help would be appreciated!

If you install MSYS2 along with MinGW you should have a unix like environment (with make and such) to build the project, if you don't run make in your build directory, you only have the Makefiles generated by cmake .., but still nothing has been built yet.
Regardless you have a couple of options here:

You can look inside the github action I use to build on windows to see how I do it
You can download a pre-built executable from a previous github action run, for example this one: https://github.com/daniele-rapagnani/melon/actions/runs/477177890. On the bottom you'll find the artifacts which are archives containing the pre-built executable. You can then simply run melon.exe yourfile.ms on the command line and it should work.

Also, I'm not sure how to write tests for isProgram. The Program type is not in the docs or in the unit tests, so should I skip that one?

The Program type represents the code inside a melon source file and it's a garbage collected object so it gets its own type. At the moment it's not exposed to the user in any way so I think you are right, we should skip it as well!

@daniele-rapagnani Just wanted to give a quick update. I spent a few hours trying to figure out the cmake/msys2/mingw situation to no avail. I got to a point where I was able to use mingw32-make after generating MinGW Makefiles but ran into an error. It seems to be related to a linker file, but I double checked that I had the mingw x86-64 linker file that you put in the readme installed:
g++.exe: error: unrecognized command line option '-fuse-ld=lld'; did you mean '-fuse-ld=bfd'?
mingw32-make[2]: *** [CMakeFiles\disassembler.dir\build.make:124: disassembler.exe] Error 1
mingw32-make[1]: *** [CMakeFiles\Makefile2:168: CMakeFiles/disassembler.dir/all] Error 2
mingw32-make: *** [Makefile:148: all] Error 2

I also tried using the msys command line as you do in the git action but for some reason it can't find cmake (even though I added CMake/bin to MSYS's PATH environment variable).
I would use the pre-built executable except that I won't be able to test the new is* functions.
If I run cmake .. by default it gives a .sln file which seems to be for Visual Studio, but I can't figure out how to use this either.
All of my Google-ing doesn't seem to help haha-perhaps I should read all the docs from the beginning😄
I'm sure you have more important things to do than help me troubleshoot haha, so worst case, I will just upload the unit tests untested.

@daniele-rapagnani Just wanted to give a quick update. I spent a few hours trying to figure out the cmake/msys2/mingw situation to no avail. I got to a point where I was able to use mingw32-make after generating MinGW Makefiles but ran into an error. It seems to be related to a linker file, but I double checked that I had the mingw x86-64 linker file that you put in the readme installed:
g++.exe: error: unrecognized command line option '-fuse-ld=lld'; did you mean '-fuse-ld=bfd'?
mingw32-make[2]: *** [CMakeFiles\disassembler.dir\build.make:124: disassembler.exe] Error 1
mingw32-make[1]: *** [CMakeFiles\Makefile2:168: CMakeFiles/disassembler.dir/all] Error 2
mingw32-make: *** [Makefile:148: all] Error 2

I also tried using the msys command line as you do in the git action but for some reason it can't find cmake (even though I added CMake/bin to MSYS's PATH environment variable).
I would use the pre-built executable except that I won't be able to test the new is* functions.
If I run cmake .. by default it gives a .sln file which seems to be for Visual Studio, but I can't figure out how to use this either.
All of my Google-ing doesn't seem to help haha-perhaps I should read all the docs from the beginning😄
I'm sure you have more important things to do than help me troubleshoot haha, so worst case, I will just upload the unit tests untested.

I also tried using the msys command line as you do in the git action but for some reason it can't find cmake (even though I added CMake/bin to MSYS's PATH environment variable).

How did you install CMake? You can try installing it with pacman directly from MSYS to make sure it's available in an MSYS shell.

g++.exe: error: unrecognized command line option '-fuse-ld=lld'; did you mean '-fuse-ld=bfd'?
You should check that your g++ version is >= 9 because earlier versions did not support lld as a linker. Try to run a g++ --version

If I run cmake .. by default it gives a .sln file which seems to be for Visual Studio, but I can't figure out how to use this either.
Yes, those are Visual Studio Solutions, but at the moment I don't support MSVC (Microsoft's C/C++ compiler) so I don't think they would be of any use.

so worst case, I will just upload the unit tests untested.
It's not the best workflow but you can do that: if you open a new pull request with your changes the tests will be run by the github actions. If anything goes wrong you can download the prebuilt binary from that run (which includes your new commits) and figure out what's wrong.

I also tried using the msys command line as you do in the git action but for some reason it can't find cmake (even though I added CMake/bin to MSYS's PATH environment variable).

How did you install CMake? You can try installing it with pacman directly from MSYS to make sure it's available in an MSYS shell.

g++.exe: error: unrecognized command line option '-fuse-ld=lld'; did you mean '-fuse-ld=bfd'?
You should check that your g++ version is >= 9 because earlier versions did not support lld as a linker. Try to run a g++ --version

If I run cmake .. by default it gives a .sln file which seems to be for Visual Studio, but I can't figure out how to use this either.
Yes, those are Visual Studio Solutions, but at the moment I don't support MSVC (Microsoft's C/C++ compiler) so I don't think they would be of any use.

so worst case, I will just upload the unit tests untested.
It's not the best workflow but you can do that: if you open a new pull request with your changes the tests will be run by the github actions. If anything goes wrong you can download the prebuilt binary from that run (which includes your new commits) and figure out what's wrong.


Create null validator to handle unvalidated models

Using simple injector, bind unvalidated models to IValidator.
http://simpleinjector.readthedocs.io/en/latest/advanced.html

Null validator added with simple injector bindings


Implement routing of messages and observation pruning based on condition

User Story

As a receiver of test results potentially containing multiple conditions,
I want the messages I receive to only have the conditions I care about,
so that I can have an easier time processing the messages
Description/Use Case

In order to be able to route messages based on condition, we first need to be able to stamp various observations in a message with the condition(s) they represent. This was done as part of #12275. This ticket will build on top of that and add support for:

Routing a message based on the conditions that are contained within it and each receiver's condition settings
Pruning observations that are linked to conditions NOT in the receiver's condition settings

The tentative design for this is documented in the code to condition mapping design note under the section titled Filtering for condition.
Note, the design suggests where and how to implement this feature. This design should be used as a guide and NOT as a hard requirement. We should implement routing and pruning in a way that is most maintainable. This may mean refactoring current functionality.
Risks/Impacts/Considerations

The design note mentions the following:

During the routing step the results of the condition filter are evaluated and for observations which pass the filter a reference to that observation is added to the FHIR bundle endpoint resource. During the Translate step any observations not referenced in the endpoint resource of the FHIR bundle will not be added to the outgoing report to the receiver. This will include both observations where a mapping could not be found as well as observations representing a reportable condition that is not of interest to the receiver.

The decision to add an observation to the endpoint resource to indicate that is passes a filter is based on existing functionality - which will be removed in #12224. We will need to think through a new solution for this part and update the design note.
Dev Notes

Acceptance Criteria

[ ] ReportStream has the ability to route observations based on their mapped conditions
[ ] ReportStream has the ability to prune observations from a message to adhere to a receiver's selected conditions
[ ] pruning tested

[ ] verified we remove any orphaned resources after removing observations

[ ] routing tested

[ ] a message where all observations are to be filtered out does not get sent to a receiver

Hey team! Please add your planning poker estimate with Zenhub @arnejduranovic @jalbinson @JessicaWNava @jimmyfagan @thetaurean @JFU-NAVA-PBC


ERROr: Java_Home variable

I am not a programmer or developer, so I am already a tad frustrated, because I don't understand why this can't just exist as simple MSI or EXE, but I digress.
I get the Error:Java_Home is not set and no 'java' command could be found in your path error.

There are prebuilt binaries available on the releases page. You can download the *-windows-amd64.zip file, extract it, and run the EXE inside the bin folder.


Re-implemented spinner through the ProgressBar

Incompatible API change: spinner() returns the ProgressBar the methods of which are not mut.
with_download_template, with_spinner_temple, with_template(custom)
Spinner can be used in a multi-progress bar.
MultiBar::: stop, cancel, error(msg).

@cylewitruk Review

@cylewitruk FYI, I can't use the as_ prefix because of the linter complaints. as_ is a specific idiomatic prefix where &self should be a reference. Thus, with_download_template which is more explicit basically.

@cylewitruk FYI, I can't use the as_ prefix because of the linter complaints. as_ is a specific idiomatic prefix where &self should be a reference. Thus, with_download_template, which is more explicit.

Ah, no big deal, with works just as well :)


Allow Only Company Aircraft (Flights)

Is your feature request related to a problem? Please describe.
No, this is a new feature.
Describe the solution you'd like
Currently when a flight is created and/or imported without subfleet assignments, all subfleets a user is allowed to can be used for that flight. In a multiple airline environment this can cause unwanted results like Airline A's aircraft being used for Airline B's flights.
As a solution, we should have some control or at least some automation for this. Like a setting, which then loads only the subfleets of that particular flight's airline instead of loading all.
We do have rank/typerating based filters for subfleets so nothing more is needed, rest of system will continue working on as before.
Describe alternatives you've considered
Alternatively we can have some admin features, to auto assign airline subfleets to airline flights if they do not have any. With this we can also allow admins to delete all subfleet assignments of an airline etc. But these will require direct model / relationship table interactions and will be partly automated.

Accept the PR. I was about to work on this myself. I agree with the implementation.


Pie chart on dashboard disappears with many languages

@HackerWand writes,

When I use too many languages, the pie chart on the panel disappears

I came from #132 by @AlexandruBuhai, since that one is closed.
For those who seek for a better Pie chart, you can try my little python program Visualize WakaTime Statistics on GitHub Gist.

It is also interactive, and a lot fancier than the default ones.


Blog Creation And Post Tech Issue

Assignment
Points
Grade
Evidence

Baseline Grade
55%
55%

Max Grade
90%
90%

Pull Request (Integration)
1
.8
Turned in the first pull request late and we were not prepared

Relevancy Checklist (Peer)
2
2
Left comments on the team we peer reviewed and received helpful comments that made me realize what I needed to improve

Lesson Style (Period)
1
.88
I felt that our lesson was solid however we made a crucial mistake in the javascript versions of our code

Homework, Popcorn Hacks
1 x 5
.88 x 5
Our popcorn and homework hacks were also very solid however the previously mentioned mistake of javascript took a tole on our credibility

Individual Contribution
1
.9
I was the scrum master of my group and coordinated roles to each of the scrums of my team and attempted to make sure we got everything in by our deadline

Personal Notebooks / Blogs
1
.87
The majority of my homework was turned in on time and I received sold grades on them

Total
11

10.57

Skill
Points
Grade
Evidence

Work Habits (Analytics)
1
.89
Work diligently towards the end

Team Planning (Issue)
1
.9
Completed on time with all the teammates issues on there

Presentation Memories
1
1
First time working on a project with peers and learning how to coordinate roles

Grading and Feedback
1
1
Received feedback from peer grading comments and used their suggestions to change our page

Beyond Perfunctory
1
.9
I understood my role in the project and made sure I expressed that in my presentation

1: I have shown my understanding of the units because of my popcorn and homework hacks. I have consistently been getting grades that prove my understanding.
2: I will create a table in my blog summarizing my experience. I will link my popcorn and homework hacks from my teachings which will show my understanding.
3: I show I’m fully engaged by writing down key takeaways from each lesson and regularly using my work to ensure continuous improvement.
4: I can guarantee my work will be original as my understanding will reflect that.
Personal Blog

Cookie clicker and table.  Link to topics.  Mostly simple links.  Discussed coordinating responsibility and doing fact checks.  Review and correct.  He made an adventure game with AI and NPC.

Glows

Has notes on each individual topic through a detailed table, and has links to the homework and popcorn hacks of each teach.
Can get a more in depth view to the lesson page as well if he is confused.
Favorite memory was organizing peers and collaborating.
Learned to fact check and validate whatever is being taught.
Also has a final project (adventure game), that uses all skills learned in the lessons.

I liked your navigational bar and how you had a little summary of what you learnt as well. I also liked how you linked It to your homework's for applications. I liked how you talked about what you learnt, and how you came across problems and got over obstacles. I liked your final project, such as how you used nesting conditionals.

I like the details of the code and the overall theme used however I feel that you could improve the SCSS styling and maybe prepare more for the presentation. Your code is really awesome and shows a great implementation of all the lessons done so far and I like the creative idea behind the code.

I like how your website is formatted with the big squares of each lesson that we learned and you added notes to each square that we can see when we hover over them. I like your adventure game, it's very unique. Good job Kush!

I like your homepage and utilization of dropdown. I completely agree with what you said about fact-checking to make sure you are teaching the right things. I also like your game and that you had interactions with an NPC.

Hi Kush,
I liked the table you had with all the notes and links. Overall, great project but you could've made it a bit more unique.

I love the dropdown menus on your lessons which reflect your learnings. Linked to the lessons page. You have a really cool final project, which basically displays a cave adventure game.

I liked how you showed the lesson you did and what your memories of the team teach were and I also liked your cookie clicker game which is looks pretty good. I also liked your final project which looked pretty good

Hi Kush,
I really like your format in making your summaries page, I like how you took the example mort gave, used similar features but made it unique to yourself. I also like how you talked about your favorite memories.

I liked how you presented your final project, as it implements everything you have learned so far, and I like the overall format of your website.

I liked how you included your lessons page, and how you talked about your favorite memories.
You went in depth into what you did, and your homework and popcorn hacks.
For your final project you made a game (a story game), where you could make choices in the game. I thought that was really nice.

Made a cookie clicker and an overview of all the lessons when you hover over your 6 boxes it gives a summary and when you click it, it gives you the link to the actual team lessons. made a game where you have an adventure and you could have a unique story line, good job!

Hi Kush, great job on the dropdown notes for your lessons. I liked how you explained how you had to overcome obstacles when creating your lesson. Also, I how creative and unique your game idea was.

He had a summary with buttons. His favorite memory was to coordinate with this group member. Overall, I liked the formatting of the homepage and the adventure game.

Hi Kush, I liked your table with the notes of all the lessons and the links to them. I liked the adventure game as well.

I liked your final project and it was really interesting and i liked your buttons. His favoriate memory was also important to the learning process

The adventure game you made was very creative and showed that you had an understanding of the topics. The drop-down menu was also nice and made the whole blog very organized.

I liked how you explain your final project as it was descriptive and showed examples of every lesson. Your favorite memory was really good as it showed what you had a lot of fun in. I love how you made a little box menu being able to go into any blog notebook or see a short description.

I liked your final project and adventure game it was very interesting.


Include info about the injected secret when describing a binding

I'd like to be able to always use svcat and never fall back to kubectl. 😀 So it would be helpful to display information about the associated secret when looking at a binding:
$ svcat describe binding testdb
  Name:        testdb
  Namespace:   default
  Status:      Ready - Injected bind result @ 2018-03-01 16:39:52 +0000 UTC
  Secret:      testdb-creds
  Instance:    testdb

Secret Data:
  database-host:  61 bytes
  database-name:  10 bytes
  password:       16 bytes
  port:           4 bytes
  username:       47 bytes

Dibs


npm start fails to launch app - react-scripts start error

Describe the bug
Running npm start from the CLI results in an error. Something around react-scripts start.
Did you try recovering your dependencies?
Yes dependencies have been attempted to be recovered multiple time in multiple apps in multiple locations
npm version - 6.12.1
Which terms did you search for in User Guide?
Did not search the User Guide, just googled stackoverflow and github issue entries.
Environment

(paste the output of the command here.)
Steps to reproduce
(Write your steps here:)

npx create-react-app hello-world
cd hello-world
npm start

Expected behavior
The app would start
Actual behavior
The app failed to start
Reproducible demo
https://github.com/brandnn1/reactappfail
Instructions to reproduce error -
Run npm start
2019-11-04T19_50_14_437Z-debug.log

cmd isn't accessible in your path, which you can read more about fixing here. It's a long-standing issue in the open package, which we use to open the browser window.


deploy-web-service.ipynb used deprecated function

When running the deploy-web-service.jpynb, I get this error:
model = run.register_model(model_name = "pi_estimate", model_path = "outputs/pi_estimate.txt")
Deprecated, use RunHistoryFacade.assets instead.

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

ID: 5ae9c8cf-5f68-9baa-1660-118208034381
Version Independent ID: 48d350ca-0ef8-3500-57ae-a5ec6c36ac79
Content: Quickstart through Azure portal - Azure Machine Learning service
Content Source: articles/machine-learning/service/quickstart-get-started.md
Service: machine-learning
GitHub Login: @hning86
Microsoft Alias: haining

@olavt Thanks for your feedback, I will investigate it and get back to you soon.

@olavt - thanks for your report. We are investigating a fix for this. Can you try again now?

still the error is coming When running the model = run.register_model

Thanks for your reports. It was fixed in release 1.0.6 of the SDK late yesterday. Please try again.

@j-martens  Thanks.
@olavt We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.

Hi Team,
I was in vacation for couple of days and today i tried to validate the fix,
but still getting the same error. Below is the screen shot.
[image: image.png]
Let me know if you need more details.
Bala
+91-9686989873
On Fri, Dec 21, 2018 at 10:14 PM YutongTie-MSFT<EMAIL_ADDRESS>wrote:

@j-martens https://github.com/j-martens Thanks.
@olavt https://github.com/olavt We will now proceed to close this
thread. If there are further questions regarding this matter, please
respond here and @YutongTie-MSFT https://github.com/YutongTie-MSFT and
we will gladly continue the discussion.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/20876#issuecomment-449438033,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWj1TkSqq8GAnK0d2OS2zqzllmzm_sYeks5u7Q_7gaJpZM4ZW3iE
.

--
The great end of life is not Knowledge but the action

Hi, @j-martens
This is the example I tried from Microsoft Ml tutorial
But not able to deploy it as a service. I have faced above issue.
model = run.register_model(model_name = "sklearn-mnist", model_path = "") Deprecated, use RunHistoryFacade.assets instead.  not worked.
If so, what would be the proper way to do that? Thank you.

Hi Team,
Please let me know if we have any timeline for next release or fix?
On Wed, 26 Dec 2018, 07:54 Josée Martens<EMAIL_ADDRESS>wrote:

That message is an error. The register_model function works; in the next
release we will be fixing the deprecation warning.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/20876#issuecomment-449889847,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWj1Tj1Ux8INwSweaRFF4T4Szxek4remks5u8t3PgaJpZM4ZW3iE
.

We have bimonthly release cadence, so early January. No patches will be issued for this issue as it's minor and does not affect user scenarios


feat: add some extra logging to further diagnose performance issues

Let me know if I've missed anything or you'd like any additional information logged here

@paulswartz As evidenced in the existing code in this file, those details are something I'm just learning now and good to know! 🙇
@zovt These extra bits of timing and including the alert ID seem good to me. The one extra tiny thing I might suggest considering is adding the alert ID to the "Scheduling distinct notifications" log line. I'm thinking it might be interesting to correlate the alert with the number of notifications that go out.


UP-4957: CORS Filter

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines
[ ] tests are included
[x] documentation is changed or added

Description of change

https://issues.jasig.org/browse/UP-4957

I'm planning to pull this one down and kick the tires a bit today, FWIW.

If feasible, I'd like to merge this tonight.

@bjagg -- can we connect a bit and talk about what problem this change solves?  I wasn't tracking any needs that this change addresses.
I'm not saying that it doesn't do something valuable -- or that we don't want it -- I just can't connect it to any of the items we've previously discussed.

https://www.owasp.org/index.php/Cross-Site_Request_Forgery_(CSRF)_Prevention_Cheat_Sheet documents how CORS is the first part in prevention.

... Although it is usually trivial to spoof any header from a browser using JavaScript, it is generally impossible to do so in the victim's browser during a CSRF attack, except via an XSS vulnerability in the site being attacked with CSRF. More importantly for this recommended Same Origin check, a number of HTTP request headers can't be set by JavaScript because they are on the 'forbidden' headers list. Only the browsers themselves can set values for these headers, making them more trustworthy because not even an XSS vulnerability can be used to modify them.
The Source Origin check recommended here relies on three of these protected headers: Origin, Referer, and Host, making it a pretty strong CSRF defense all on its own.

Added ability to disable CORS filter. Will be disabled by default to keep default browser restrictions of "SAME ORIGIN" in place.

My latest commit replaces the new flag with the optional filter decorator I wrote. Also ran goJF successfully.


Visual Studio Test deleting trx files and other artifacts on task completion rather than waiting until pipeline completes

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: Visual Studio Test
Environment
If using Azure Pipelines, provide the account name, team project name, build definition name/build number:  I can provide this information privately.
Agent
Private, Windows 10 Pro, Agent Version 2.163.1
Issue Description
The "Visual Studio Test" task deletes all trx files and artifacts from the agent machine upon completion of the task, preventing us from accessing the information with subsequent tasks, such as uploading test results or reporting them to slack.  The tooltip for the test results field indicates the TestResults directory is only cleaned at the beginning of the test run and at the end of the pipeline, not the end of the task.

@chuck-hunter are you using Azure Devops Server Onprem or the hosted Azure Devops Service? What version of the vstest task are you using?
Please provide the full zip logs with system.debug = true so that we can get all the info we need. The latest version of the task does not clean up the results at the of the task.

@chuck-hunter gentle ping.

Closing due to no response, please re-open with requested data.


Feedback

Thought I should create a general feedback issue so people don't have to open one for each minor concern.
I haven't looked at all the individual userstyle directories yet - but 2 are wrongly categorized as userstyles:

DuckDuckGo (it's just loading the theme into your cookies at the moment)
Stable Diffusion Web UI (it is CSS, but it's available as an actual Extension for the host, essentially a Marketplace)

We should update them in the ports.yml as agnostic - the goal for DDG at least is that we someday have an actual userstyle...

Oh lol, yeah I just basically took the ports.yml and generate code and modified it
Agreed with both

Also I think we really need to use git lfs.
My git push is capped at like 500 kibs and it's taking forever to push it all

Also @nekowinston, I think you would have enjoyed this process :laughing:

Good point, I hadn't thought of that previously. 95MB is pretty concerning if we're going to have multiple people maintaining this.
Pretty funny how the largest Repos are all filled with PNGs, while the WEBP barely contribute to the size.
Since we're essentially starting out from tabula rasa with this repo, we should again consider getting rid of screenshots in individual repos. Neovim, our highest-profile repo aside from the main one, already isn't following our screenshot guidelines...

Also I think we really need to use git lfs.
My git push is capped at like 500 kibs and it's taking forever to push it all

Good point, I hadn't thought of that previously. 95MB is pretty concerning if we're going to have multiple people maintaining this. Pretty funny how the largest Repos are all filled with PNGs, while the WEBP barely contribute to the size.
Since we're essentially starting out from tabula rasa with this repo, we should again consider getting rid of screenshots in individual repos. Neovim, our highest-profile repo aside from the main one, already isn't following our screenshot guidelines...

Well I don't think it's bad when they're individual repositories right, it's just when it's all combined... we have this. I personally quite like the screenshots but yeah I get both sides of this.
Is there a nice middle ground?
It's just taken me 10 minutes to force push everything, we definitely can't keep it this way that's for sure

Woops, the actions workflow has committed deno_cache, probably should have saw that one coming

Is there a nice middle ground?

Yeah, I've been working on something toolbox/catwalk related, I think we could also solve the screenshot storage problem with that. I'm weary of just chucking stuff onto the GitHub CDN for screenshots - I think it's alright in edge cases, but I would hate for us to depend on GitHub for everything.
The idea is to have a web interface for catwalk, to remove the install step. When combined with a GitHub login & S3 storage, we could self-host it.

Woops, the actions workflow has committed deno_cache, probably should have saw that one coming

What we don't put up with for those fast workflows...

Time to remove everything from GitHub and chuck to it big Jeff :sunglasses:
(All seriousness, sounds good for stopping our reliance on GitHub's CDN. I wasn't feeling that great even when I was referencing GitHub links in the generate code from here to catppuccin/catppuccin/resources/generate lol)

In the short term, there are probably 3 immediate solutions

Remove individual Latte, Frappe, Macchiato, and Mocha screenshots and just keep the catwalk screenshot?
Use Git LFS (I don't know how impactful this is) + ImgBot image compression?
Remove all the screenshots entirely?

Just by running
find . -name "*.png" -exec convert -quality 95 {} {}.webp \;

I'm able to reduce the size of all current assets to 27MB:
$ find . -name "*.webp" -print0 | du --files0-from=- -hc | tail -n1
> 27M     total

How could I forget the 4th solution:

winston's brain

I've managed to cut it down to 16MB.
Quick note: youtubemusic seems to store some PNG files which it uses in its stylesheet. So I'm not converting those to webp:
# convert & delete PNG files
find . -name "*.png" -exec sh -c 'convert "$1" "${1%.png}.webp"' _ {} \; -exec rm {} \;
# reset youtubemusic
git restore styles/youtubemusic/assets/icons/ && rm styles/youtubemusic/assets/icons/*.webp

$ find . -name "*.webp" -print0 | du --files0-from=- -hc | tail -n1
> 16M     total

We should re-initialize the repository at some point though, since my .git folder now north of 160MB.

Yeah it'd be great if you want to do that since you have the images locally and I wouldn't want to take any credit :heart:
Lovely work!

I have committed all converted WEBP images, and I think I also rewrote most of them to use relative URLs, e.g. assets/preview.webp instead of https://raw.githubusercontent.com/**/*.png.

The DDG userstyle is almost done. I just need to add some finishing touches.


Migrate Pollution Assessment Stage1 notebooks & environment from WikiSRAT

For the Stage 1 Pollution Assessment work, we developed a series of Jupyter Notebooks in the root folder of TheAcademyofNaturalSciences/WikiSRATMicroService repository, in part because we leveraged the WikiSRAT API.
This work, however, ballooned into a massive number of additional files in the data/ and figs/ folders and elsewhere. These many files and commits have substantially confused the core purpose of that repo.
Let's move all those files into this repo.
@sjordan29 & @mikecampagna, I just invited you to join the WikiWatershed organization, so that you can help with this.
I'll take a closer look and try to take a first stab at migrating those files over.
I'll then delete them from the WikiSRAT repo, if that make sense.

Completed with commit 12139deaa6ab98b478092222c9798fc41b89ab86
Link to commit where these files were removed from parent repo:
https://github.com/TheAcademyofNaturalSciences/WikiSRATMicroService/commit/b4606d567a704113ac97a4be4cd464629ab0eb0c
Commit history is preserved in this branch: https://github.com/TheAcademyofNaturalSciences/WikiSRATMicroService/tree/archive_PollutionAssessmentStage1


Add how-to for upstream devs testing using ACS Engine

What this PR does / why we need it: This PR provides documentation on how to deploy a private Kubernetes build using ACS Engine.  We need it so that new devs working on upstream features in Azure can get started more easily.
Which issue this PR fixes: None
Special notes for your reviewer: None
Release note:

NONE

@itowlson,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

cc @khenidak


[pigeon]: SwiftCompile normal arm64 Compiling SwiftPlugin.swift

When building for add-to-app, when swift class is generated by pigeon, the SwiftCompiler complains about the internal structs, classes and protocols, as shown below:
/Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/SwiftWebmoto
                     rsNativeApiPlugin.swift:39:17: error: method cannot
                     be declared public because its result uses an
                     internal type
                         public func getNativeConfigs() ->
                         NativeConfigArgs? {
                                     ^
                                     ~~~~~~~~~~~~~~~~
                     /Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/ApiHost.swif
                     t:51:8: note: type declared here
                     struct NativeConfigArgs {
                            ^
                     /Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/SwiftWebmoto
                     rsNativeApiPlugin.swift:60:14: error: property
                     cannot be declared public because its type uses an
                     internal type
                       public var cartenArgs : CartenArgs?
                                  ^            ~~~~~~~~~~
                     /Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/ApiHost.swif
                     t:17:8: note: type declared here
                     struct CartenArgs {
                            ^
                     /Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/SwiftWebmoto
                     rsNativeApiPlugin.swift:61:14: error: property
                     cannot be declared public because its type uses an
                     internal type
                       public var nativeConfigArgs : NativeConfigArgs?
                                  ^                  ~~~~~~~~~~~~~~~~
                     /Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/ApiHost.swif
                     t:51:8: note: type declared here
                     struct NativeConfigArgs {
                            ^
                     /Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/SwiftWebmoto
                     rsNativeApiPlugin.swift:71:15: error: method cannot
                     be declared public because its parameter uses an
                     internal type
                       public func setCartenArgs(args: CartenArgs) {
                                   ^                   ~~~~~~~~~~
                     /Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/ApiHost.swif
                     t:17:8: note: type declared here
                     struct CartenArgs {
                            ^
                     /Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/SwiftWebmoto
                     rsNativeApiPlugin.swift:75:15: error: method cannot
                     be declared public because its parameter uses an
                     internal type
                       public func setNativeConfigArgs(args:
                       NativeConfigArgs) {
                                   ^
                                   ~~~~~~~~~~~~~~~~
                     /Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/ApiHost.swif
                     t:51:8: note: type declared here
                     struct NativeConfigArgs {
                            ^
                     /Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/SwiftWebmoto
                     rsNativeApiPlugin.swift:100:15: error: method cannot
                     be declared public because its result uses an
                     internal type
                       public func getCartenArgs() -> CartenArgs? {
                                   ^                  ~~~~~~~~~~
                     /Users/p22.matheus/dev/webmotors.posvenda.flutter/pa
                     ckages/webmotors_native_api/ios/Classes/ApiHost.swif
                     t:17:8: note: type declared here
                     struct CartenArgs {

This is still not a complete example; it's not clear what the code producing these errors is trying to do it for it fits into your app. Are you trying to make Pigeon-generated types part of the public API of some other code?
If so, that's probably not something you should be doing in the first place, since Pigeon-generated code is subject to change.


Fixes the broken link and outdated plugin name

I am using Spring Native 0.11.3 for testing, I get below two small issues:

Broken link
https://docs.spring.io/spring-boot/docs/2.6.6/reference/html/spring-boot-features.html#boot-features-container-images-buildpacks
-> https://docs.spring.io/spring-boot/docs/2.6.6/reference/html/container-images.html#container-images.buildpacks

Outdate plugin name
native-image-maven-plugin -> native-maven-plugin

@moarychan Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@moarychan Thank you for signing the Contributor License Agreement!

Hi @moarychan, thanks for the contribution!


Support Deployments with a "branch & merge" strategy

When using a git flow like branching strategy with feature branches and a main (or release) branch in which they get merged, Deployments won't show up in Jira after the merge.
If I push a commit with issue key KEY-1 into the feature branch, the deployment shows up connected with the KEY-1 issue in Jira.
When I merge the feature branch into main, the deployment that is triggered from the main branch, doesn't show up in Jira (i.e. it's not being connected with the KEY-1 issue.
I guess we need to implement some smart logic that crawls all commits that have been made on the feature branch, but are not yet in the main branch, and check those commits for issue keys and then add those issue keys to the deployment

+1

I came across this issue after reading this line https://github.com/atlassian/github-for-jira/blob/075c4339968fdaafc781ff581acf5a32398f4da7/src/transforms/deployment.ts#L67
If i am reading this correctly, then only the last commit is considered when processing deployments, isn't it?
I want to highlight another use case were this is quite unfortunate:
We have some environments that can be updated only during certain maintenance windows (i.e. once a week). Usually there are five to ten commits on our "main" branch between each deployment to these environments.  Each commit has an jira issue key.
However, as only the last commit is parsed for jira issue keys, at most one jira issue gets the right deployment state. And all others issues show as "not deployed".

Hey @jakila and @eialanjones. Good news, we've just rolled an update for this out to all customers. Please let us know if this now works as expected.

That's great news!
I just did a quick check and it seems to be working for us. Will take a few more days (and a few more deployments) to verify it is working as expected.
But already now the deployment view in our Jira looks more complete than ever before 🚀
A big "Thank You" to atrigueiro, rachellerathbone and everyone else working on the topic!

That's great to hear @jakila 🎉

Hi, we are experiencing a similar issue.
We have a single branch (main) which we deploy to, and we mark each commit with the task id in the name.
On every commit to the main branch we deploy to our development environment. The workflow will then wait for a review /approval before deploying to staging, and then the same for production.
All our tasks are displayed as deployed to development as expected, but only the last one to go staging/prod will be updated as released to the correct environment.

Hey, we just had our first production deployment with this Jira for Github integration. We expected the deployment status to change for all relevant issues but nothing happened.
I investigated the code and found this: when looking for the commit messages since the last successful deployment, you return undefined when there hasn't been a successful deployment yet. Since this was our first deployment only issue keys from the last commit messages were extracted (merge commit). The next deployment will extract issue keys from all deployments since our first deployment. Thus, the deployment status for the issues in the first deployment will never be updated.
https://github.com/atlassian/github-for-jira/blob/main/src/transforms/deployment.ts#L67
Suggestion: return all commit messages when there are no previous successful deployments. Is this feasible or would you run into scalability issues?

Hey @palexcast and @pdyck if I'm understanding correctly, it looks like your issues are related to deployments but are different from the resolved issue in this thread. To make sure issues stay focused on one problem, and to make sure your messages don't get lost in this thread, I've done ahead and opened up new issues for your separate concerns:

@palexcast https://github.com/atlassian/github-for-jira/issues/1041
@pdyck https://github.com/atlassian/github-for-jira/issues/1042

Please feel free to add any additional information &/or screenshots to your respective issues.

@jakila I'm going to go ahead and close this issue. If you notice anything not working as expected regarding the support of issue keys multiple branches + commits, please feel free to reopen this and reach out again.


unsubscribe can async

unsubscribe can async, will make server fast close.

It's really not worth the optimization


Add function withoutDuplicates at extension Sequence

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 4.
[x] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[x] I have added a changelog entry describing my changes.

          1 Warning

:warning:
The source files have been modified. Please consider adding a CHANGELOG entry if necessary.

          4 Messages

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

:book:
iOS: Executed 535 tests, with 0 failures (0 unexpected) in 10.562 (12.029) seconds

:book:
tvOS: Executed 510 tests, with 0 failures (0 unexpected) in 2.018 (2.440) seconds

:book:
macOS: Executed 377 tests, with 0 failures (0 unexpected) in 0.939 (1.270) seconds

  Generated by :no_entry_sign: Danger

Please note that your tests are failing on Linux. @LucianoPAlmeida can you please help with that? Any idea why it's failing?

Please note that your tests are failing on Linux because you've used IndexPath in your tests, which are part of Foundation, not the standard library. Please use a different struct/class. Don't import Foundation.

Thank you for contributing to SwifterSwift! I've invited you to join the SwifterSwift GitHub organization - no pressure to accept! If you'd like more information on what that means, check out our contributing guidelines. Feel free to reach out if you have any questions! 😃


Update gleam, and use a dynamic GL function table.

cc @sotaroikeda

Blocked on https://github.com/servo/gleam/pull/106

So gl::Gl is a trait, and thus the new code works with the trait object, right?
Is there any benefit of using the unsafe raw pointers versus just Rc/Weak?

@kvark That's right. It avoids some refcount churn. In this case it seemed easy enough to do it, but I'd be fine also with Rc. It may make sense to share gl function tables across threads though, that's something that Rc would prevent, but we don't use that at all I think so it may be ok.

Oh, and I hadn't been following the gleam PR btw, the generic proposal looks ok to me. It's kind of sad to use dynamic dispatch, but I was taking that for granted already. Couldn't we make Gl a static table and add convenience methods to it instead? I did something similar for clang-sys for bindgen (see the functions struct in https://github.com/KyleMayes/clang-sys/pull/45). It'd be a matter of adding all the glue that would be inlined and would just look like:
#[inline]
fn flush(&self) {
    (self.functions.expect("glFlush not loaded"))()
}

Couldn't we make Gl a static table and add convenience methods to it instead?

It might be better for performance. But  I want to handle it as another issue, since we want to enable Graphics branch soon.

@emilio

It avoids some refcount churn.

The extra refcount work would only apply to resource creation/deletion, right? I wouldn't be too concerned and preferred the code clarity (and less unsafety) instead.

It may make sense to share gl function tables across threads though

Could you explain why we'd need that? It sounds like a hack.

Couldn't we make Gl a static table and add convenience methods to it instead?

Possible, although I'd rather want to see static dispatch here in the first place. Considering the deadlines, this will have to wait.

On Wed, Feb 15, 2017 at 09:43:07PM -0800, Dzmitry Malyshau wrote:

@emilio

It avoids some refcount churn.

The extra refcount work would only apply to resource creation/deletion, right? I wouldn't be too concerned and preferred the code clarity (and less unsafety) instead.

Right, and that's a fair point.

It may make sense to share gl function tables across threads though

Could you explain why we'd need that? It sounds like a hack.

Didn't have a specific use case in mind, but yeah, I guess context
creation time is not a big deal.
I could imagine us sharing the same function hooks or something, but
yeah, as I said I don't expect it to be super-useful.

Couldn't we make Gl a static table and add convenience methods to it instead?

Possible, although I'd rather want to see static dispatch here in the
first place. Considering the deadlines, this will have to wait.

That's fair, as I said I don't love dynamic dispatch, but we can
probably live with it.

@emilio thanks!
I've just published gleam-0.3.0, so let's have this PR updated.

@emilio, thank you!

Done, feel free to take a look, let me know when do you want to merge this.

What is a current status?

Whoops, totally forgot about this, there it is.

Thanks for the review @kvark! Landed and published as 0.7.0

Thank you!


Support Apple Podcast

https://www.reddit.com/r/DataHoarder/comments/1g4342q/comment/ls214xm/

Would be great for podcast reading if we can specify the audio source.

As Apple Podcast is supported by https://github.com/yt-dlp/yt-dlp, this should require very little work.
I have just played with it a bit - yt-dlp renders the description of Apple Podcasts with a little different structure, which trashes the prompts that yt2doc feeds into Whisper. But this issue should be very easy to fix.
Should be done in a day or two.

WIP: https://github.com/shun-liang/yt2doc/pull/18

Done: https://github.com/shun-liang/yt2doc/commit/385140ed2061131e7ddcf28dfd375f2628820e75


chore: bump sqllogictest to 0.9.0

Signed-off-by: xxchan<EMAIL_ADDRESS>

test files need to be updated. want https://github.com/risinglightdb/sqllogictest-rs/issues/114 😇


Temperature sensor I2C driver

Reading back temperature values from the TMP175 temperature sensor.
Correctly converting them into celsius units.


IFormFile is generated with OpenApi3 artifacts while using --serializeasv2

Installed package:
"Swashbuckle.AspNetCore" Version="5.0.0"
When serializing a controller method with IFormFile with the following statement:
dotnet swagger tofile --serializeasv2 --output bin/swagger.json $(OutputPath)$(AssemblyName).dll v1
the generated output in the swagger.json is the following(excerpt)
         {             "in": "formData",             "name": "FormFile",             "required": true,             "type": "string",             "format": "binary"           }
The OpenApi documentation states the followingen:
https://swagger.io/docs/specification/data-models/data-types/#file
I would expect the type to be of "file".
Is this a bug wih the parameter --serializeasv2? Or are we doing something wrong? Sadly the generated output does not work correctly with AutoRest since it doesn't support OpenApiV3.
Thank you!

I am also affected by this issue. Any idea how to work around this?

Worked around this by adding
c.MapType(typeof(IFormFile), () => new OpenApiSchema() { Type = "file", Format = "binary" });
to AddSwaggerGen.

@RisaI Thanks for your solution, I think you can further remove 'Format = "binary"' because OpenApiV2 doesn't know about it.

Closing as external.


[BUG] actions correctly generated in TS-Type but not in JS-File

Is there an existing issue for this?

[X] I have searched the existing issues

Nature of Your Project
JavaScript, TypeScript
Current Behavior
I created a Entity with a simple (void) action:
entity IH_Items          as projection on ih.IH_Items
        actions {
            action myAction();
        }

The generate TS-Type-Stub looks (correctrly) like this:
export function _IH_ItemAspect<TBase extends new (...args: any[]) => object>(Base: TBase) {
  return class IH_Item extends Base {
      [...]
      static readonly actions: {
        myAction: { (): any, __parameters: Record<never, never>, __returns: any, kind: 'action'}
      }
  };
}

But in the generated JS-Type this action is missing:
function _IH_ItemAspect(Base) {
    return class IH_Item extends Base {
        [...]
        static actions;
    };
}

Expected Behavior
Action is also generated for JS-Type-File.
Steps To Reproduce

create an entity with a (void) bound action
run cds-typer

Environment
- **OS**: Windows + Linux
- **Node**: v20.13.1
- **npm**: 10.8.0
- **cds-typer**: 0.21.0
- **cds**: 7.9.1

Repository Containing a Minimal Reproducible Example
No response
Anything else?
No response

No Bug. Problem solution in #252


Basic Tauri UI

Very basic setup of ui. I only get errors on the frontend if the state is different than the backend. Could probably just catch from the invoke and revert the state.
I think the types should be auto-generated from rust and written to a file so we can just import the interface instead of writing the types inside App.tsx (prone to errors). There's a way to do this in C#/.NET core, not sure if rust can do it.
Also I get this error if I don't build and just run dev. Not sure if it should be fixed b/c prod is always built.
Main.rs line 95
proc macro panicked
message: The `distDir` configuration is set to `"../dist"` but this path doesn't exist

Also I'm using the Prettier extension. It's good to have a consistent auto-formatter.


Noobs Partition is Different

The Noobs partitioning scheme is a little different, which means that the editing of cmdline.txt and /etc/fstab doesn't work.
In Noobs, the root partition is /dev/mmcblk0p6 - unsure if there is an automatic way to discover if you're running under noobs or not....

Yeah, at the time I elected to just write this for non-NOOBS installations, but we've had enough inquiries that it seems worth looking into. No promises, but I'll investigate.

Simplest way might be to

Replace mmcblk0p2 with mmcblk0p[2-9] in the sed command on line 135 - there should only be a single reference to an mmc partition in cmdline.txt

Line 141 in /etc/fstab might be trickier, but you should still be able to match the line with something like:

/mmcblk0p[2-9]\s+/\s+ext4/
(ie any mmc partition from 2 to 9 followed by whitespace then / then whitespace then ext4 - it made sense in my head, but I haven't tested it)
Obviously I'm assuming that mmcblk0p1 is going to be /boot in every case....

n00bs itself has a problem with this setup. AFAIK it is programmed to install to the SD; updating the boot text file after overwriting the partition.


Fix casting partial date input for Spark

Bug description
When casting string as date, Spark throws error or returns null for 123, but works for 2020. While in Velox, 123 and 2020 both work. We can follow-up in a separate PR to fix Spark cast.
scala> spark.sql("select cast('123' as date)").show(false)
+-----------------+
|CAST(123 AS DATE)|
+-----------------+
|null             |
+-----------------+

scala> spark.sql("select cast('2020' as date)").show(false)
+------------------+
|CAST(2020 AS DATE)|
+------------------+
|2020-01-01        |
+------------------+

System information
CMake Version: 3.16.3
System: Linux-5.4.0-173-generic
Arch: x86_64
C++ Compiler: /usr/bin/c++
C++ Compiler Version: 11.1.0
C Compiler: /usr/bin/cc
C Compiler Version: 11.1.0
CMake Prefix Path: /usr/local;/usr;/;/usr;/usr/local;/usr/X11R6;/usr/pkg;/opt
Relevant logs
No response

https://github.com/facebookincubator/velox/pull/9357#issuecomment-2048798144


Bugfix for ignored pings

https://github.com/Storj/dataserv-client/issues/106

Travis please repeate the unit test?


Make recursive_generator [[nodiscard]] like generator

Using recursive_generator I was caught off guard by the lack of a [[nodiscard]] specifier, allowing a foolish typo to persist in my code for weeks. This change enables the same warnings generator enjoys, likewise with the change to shared_task enabling warnings like for task.

Thanks for the  PR. Looks good!


Deprecate bin/ resources

Changes

Comments

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

#247  👈
#246
#245
master

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

Join @forstmeier and the rest of your teammates on  Graphite


QEMULauncher takes 100% cpu usage even when no VM is started

Describe the issue
QEMULauncher takes 100% cpu usage even when no VM is started
Configuration

UTM Version: 3.1.5 (53)
OS Version: 12.3.1 (21E258)
Intel or Apple Silicon? M1

Crash log
I did a sample of the process when this was happening, hopefully it will be enough.
https://sbarnea.com/ss/Sample-of-QEMULauncher.txt
Debug log
Upload VM

So it looks like the VM is still running (SPICE is updating the display). However, if you say "no VM is started" do you mean you've closed all the open VM windows?

I have had it once. Just quit the UTM VM manager (the main app) and if there are still qemu'ish processes running (running top or ps -ef | grep qemu , kill them using killall -9 qemu or kill -9 <pid of qemu>.
I did that and after restarting UTM wil a VM the problem did not occur anymore. Probably a transient glitch.


Order Size Invalid

I get this message: "Exception: 200-{"code":"300000","msg":"Order size invalid."}"
However, I am trading $50 in USDT to buy ETH. There is no minimum order on KuCoin for $50.
What is the problem?

Hey, I don't understand why that is the case with Kucoin too. Please try a higher value or a different pairing.
It's difficult to integrate the min quantity size because the pairing and the coin needs to be determined before checking.

Thanks for your response. To further explore, I went through the front-end web site of KuCoin and bought 1 USDT of ETH with a market order just fine. The minimum ETH buy at KuCoin is .0001. So the minimum is 27c USD. I tried the Simple Pump and Dump program with $50 and it said order size too small. I know it connects to my account fine because it displays my balance. Any other possible suggestions?

Here is the error message:
? Please select trade configuration?  market-trade-one
Your USDT balance is 48.87420854
Please ensure Config is correct before proceeding
? Please enter coin for Pump?  eth
Traceback (most recent call last):
File "bot-kucoin.py", line 290, in
asyncio.run(main())
File "/usr/lib/python3.8/asyncio/runners.py", line 43, in run
return loop.run_until_complete(main)
File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
return future.result()
File "bot-kucoin.py", line 230, in main
buy_order = market_order(selected_coin_pair, 'buy')
File "bot-kucoin.py", line 144, in market_order
order_id = trade.create_market_order(selected_coin_pair, 'buy', clientOid=str(uuid.uuid4()), size=str(buy_qty))
File "/usr/local/lib/python3.8/dist-packages/kucoin/trade/trade.py", line 164, in create_market_order
return self._request('POST', '/api/v1/orders', params=params)
File "/usr/local/lib/python3.8/dist-packages/kucoin/base_request/base_request.py", line 93, in _request
return self.check_response_data(response_data)
File "/usr/local/lib/python3.8/dist-packages/kucoin/base_request/base_request.py", line 110, in check_response_data
raise Exception("{}-{}".format(response_data.status_code, response_data.text))
Exception: 200-{"code":"300000","msg":"Order size invalid."}
Here is my CONFIG:
################# KUCOIN CONFIG ####################
trade_configs:
market-trade-one:
#Coin pairing for trade
pairing: USDT
#Currently, only market is available, keep unchanged
order_type: market
#Amount from wallet to use for trade buy; range 0 to 1
buy_qty_from_wallet: .1
#Profit margin multiplier; 0.5 equals 50% gain
profit_margin: 0.001
#Amount from wallet to use for trade sell; range 0 to 1
sell_qty_from_wallet: 1
#Refresh interval in milliseconds to check market for current price
refresh_interval: 100
#Timeout in milliseconds if profit margin doesn't reach intended value; 25 seconds
sell_fallback_timeout_ms: 25000

OK it is the math.floor function in the market_order function, rounding my amount to the nearest integer down to 0. I changed math.floor to round() to 4 decimals and this worked.
However, I ran it, and it successfully placed the order, but does not seem to have sold after several minutes, when I expected it to sell after 25 seconds or less. Any ideas?

I was able to programmatically get both the a) max number of decimals and round down from there and b) get  the minimum for the particular pair. It's very easy to implement and highly recommend this is added. It's only a few lines of code
symbol_list = client.get_symbol_list()
symbol_list = [e for e in symbol_list if str(e["symbol"]) == str(selected_coin_pair)]
symbol_list[0]['baseIncrement']
symbol_list[0]['baseMinSize']
Then you can

have a condition for base MinSize

And just round down the buy_qty and sell_qty using this function,  where decimals is the length of baseIncrement - 3:

def round_down(value, decimals):
factor = 1 / (10 ** decimals)
return (value // factor) * factor

Hi, this is great! I'll implement it in the Node version of the bot


query and update animals table

### Query and update animals table
- Implimentations

Update the animals table by setting the species column to digimon for all animals that have a name ending in mon.

Update the animals table by setting the species column to pokemon for all animals that don't have species already set.

Verify that changes were made.

Commit the transaction.

Verify that changes persist after commit.

screenshot of the results

Delete all animals born after Jan 1st, 2022.

Create a savepoint for the transaction.

Update all animals' weight to be their weight multiplied by -1.

Rollback to the savepoint

Update all animals' weights that are negative to be their weight multiplied by -1.

Commit transaction

screenshot of the results

How many animals are there?

How many animals have never tried to escape?

What is the average weight of animals?

Who escapes the most, neutered or not neutered animals?

What is the minimum and maximum weight of each type of animal?

What is the average number of escape attempts per animal type of those born between 1990 and 2000?

Hey @citec-47 Kindly note that you will have to update the screenshots for the transactions so as to reflect the changes made


back up and install Atom packages from a list

Export a list of your currently installed Atom packages from APM with atom-package-backup, save it with your dotfiles and install from it later with atom-package-install.

Oh cool, yeah I can get behind that.


Rates

Hello, i would like to know what the ratelimits are.

This is all about Error 429.
All functions in API have different limits.

Limits are tracked on an individual application or user level, depending on the type of token used in the request.

200 calls per hour.

That’s what official API documentation said. And that’s true. But it’s about official API.
Private API’s limits are different. Most function need 5-7 sec pause between requests, but some functions need longer delays. Just because of spam usage, all we know this as mass actions.
A few years ago functions didn’t not limited so high, but all last year and I think think this year also Instagram will fight with spammers more actively.
Restrictions started coming very actively since Facebook bought Instagram.
This Cambridge Analytics and HYP3R incidents also added more oil to fire.

Does this module respect the rate limits (especially for the feeds), or do I have to implement that myself?

Does this module respect the rate limits (especially for the feeds), or do I have to implement that myself?

must implement that yourself

Okay, thanks!

since this is the first result in google, I will comment here:
so I wanted to autoaccept following requests and here is the story:
first, I set up random interval between checking requests list to 5-7 seconds. after just 2 minutes, instagram throw an error and required me to add phone number.
I decided to add it and increase interval to 30-60 seconds (random) between checking followers list.
just 2 hours later it crashed and throw 429 error. I was able tomanually get requests list through app so I think the right interval would be about 1-2 minutes (random with float-precious seconds) and 1-5 seconds random between accepting each request in a loop
if anyone has more info on ratelimit based on their experience, please comment here.


Git Error: Could not parse object on Bundle command

Hi,
I was setting up rails repo using vagrant. I was running bundle command inside /vagrant/rails/.
Then this is the error I got:
fatal: Could not parse object '7e3c714a09ae2b38d2620835e794150d8857cd49'.
Git error: command `git reset --hard 7e3c714a09ae2b38d2620835e794150d8857cd49` in directory /home/ubuntu/.bundle/ruby/2.4.0/rb-inotify-7e3c714a09ae has failed.

Additional Info:
System:      Mac OS X El captain 10.11.2
Vagrant:      1.9.1
VirtualBox: 5.1.12 r112440

I created a new VM and with an up to date master branch bundle install inside vagrant/rails succeeded.
I am going to close on the assumption that your session founded some kind of Git hiccup unrelated to the dev box itself.


GPM IMERG recipe issues

In https://github.com/pangeo-forge/staged-recipes/issues/5 we get one HDF5 file per half hour, which has several fields, all on the same lat/lon grid, but with different data types (some in float32 and some in int16). This means that we want to concatenate some of them along the time dimension in chunks of 4 half-hours, and others in chunks of 8 half-hours, to get the 100MB chunk size.
I believe the NetCDFtoZarrSequentialRecipe recipe doesn't support heterogeneous data types yet, am I right? If that is true, would it make sense to implement inputs_per_chunk as a dictionary of the form name: inputs_per_chunk?
We need to authenticate to the HTTP server with a HTTP user and password. Is it implemented in the NetCDFtoZarrSequentialRecipe recipe? May be related to #53.

This means that we want to concatenate some of them along the time dimension in chunks of 4 half-hours, and others in chunks of 8 half-hours, to get the 100MB chunk size.

Yes, if you want to insist on having constant-memory chunks, then you would want to do this. In my experience, it's best to have the different variables in a dataset all have the same chunk size in terms of dimensions. Otherwise you have to do some rechunking if you want to do calculations on multiple variables. Having some chunks be 50MB, others 100MB is really not a problem IMO.
Nevertheless, it would be good to have the ability to customize chunks more. That is also discussed in #50.
xref #53 for credentials

This means that we want to concatenate some of them along the time dimension in chunks of 4 half-hours, and others in chunks of 8 half-hours, to get the 100MB chunk size.

Yes, if you want to insist on having constant-memory chunks, then you would want to do this. In my experience, it's best to have the different variables in a dataset all have the same chunk size in terms of dimensions. Otherwise you have to do some rechunking if you want to do calculations on multiple variables. Having some chunks be 50MB, others 100MB is really not a problem IMO.
Nevertheless, it would be good to have the ability to customize chunks more. That is also discussed in #50.
xref #53 for credentials

I believe the NetCDFtoZarrSequentialRecipe recipe doesn't support heterogeneous data types yet, am I right?

Just to clarify, it does support heterogeneous data types. ✅

I believe the NetCDFtoZarrSequentialRecipe recipe doesn't support heterogeneous data types yet, am I right?

Just to clarify, it does support heterogeneous data types. ✅


Improve the clustermesh status command

This PR includes a set of improvements for the cilium clustermesh status command, to simplify troubleshooting and reduce CI flakiness:

Always check the readiness of the clustermesh-apiserver deployment.
Report a node as not ready also when the configuration has not yet been detected (as the secret modification has not propagated yet).
Report explicit errors about why a node is not ready, based on the additional information introduced in https://github.com/cilium/cilium/pull/26788 (e.g., etcd connection, cluster configuration retrieval, data synchronization).
Remove duplicate checks and fix inaccuracies in output messages.

Please refer to the individual commit messages for additional information.
Fixes: https://github.com/cilium/cilium-cli/issues/1773
I'm currently marking this PR as ready for review as it makes sense that the review occurs in parallel with that of https://github.com/cilium/cilium/pull/26788, so that possible modifications can be synchronized. The commit with the go.mod changes will be dropped before merging.

It looks like you forgot to remove the commit aptly named "TO DROP: use forked version of cilium"

Yeah, the commit is still there because otherwise the CLI doesn't compile. This is the reason for the dont-merge/blocked label being set, as it depends on https://github.com/cilium/cilium/pull/26788. At the same time, I wanted to mark this one ready for review so that they can be reviewed in parallel, and possible changes can be propagated if necessary.

We're now checking the health of the Deployment, but it may be the case that the Service is separately not able to receive traffic (e.g. an LB Service has not been assigned an IP). Is the idea here that if nodes have synced then the Service must be reachable?

There's actually a service readiness check, but it is hidden inside the statusAccessInformation function (which in turn calls extractAccessInformation). The inner function returns an error (thus triggering the retry mechanism) if no service IP is found. This is also the reason for the removal of the explicit service readiness check in https://github.com/cilium/cilium-cli/pull/1834/commits/218fc9a24ec9cf0d3630e909255888ea69a063d5, as it was already tested earlier.


Support ExternalDocs for autorest.powershell

Support ExternalDocs for autorest.powershell

if externalDocs is added to operation, its content will be added to three parts:

the attribute of c# cmdlet

the result of Get-Help {cmdlet-name}

the  related links section of docs/MDs

See generated details: https://github.com/Azure/azure-powershell/blob/demo/Search/src/Search/docs/New-AzSearchService.md#related-links

Support adding externalDocs by customization

see https://github.com/Azure/azure-powershell/blob/demo/Search/src/Search/docs/Get-AzSearchAdminKey.md#related-links

if externalDocs is added to parameter, its content will be added to parameter description:

if externalDocs is added to property, its content will be add to the xml comment of property:

See generated details from demo projects: https://github.com/Azure/azure-powershell/blob/demo/Search/src/ContainerService/docs/New-AzContainerServiceComponent.md#-hockeyappid

/azp run


Create command fails if permissions are too limited

Description of the issue

When running the create command, you get an error and a prototype is not successfully created
Steps to reproduce the issue

On a managed mac, run npx govuk-prototype-kit create
Actual vs expected behaviour

actual:
[redacted] juggling-licence % npx<EMAIL_ADDRESS>create

Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y
Creating your prototype

added 380 packages in 5s

33 packages are looking for funding
  run `npm fund` for details
sh: /Users/[redacted]/Documents/prototypes/juggling-licence/node_modules/.bin/govuk-prototype-kit: /usr/bin/env: bad interpreter: Permission denied
/Users/[redacted]/.npm/_npx/fb72d33463aacf66/node_modules/govuk-prototype-kit/lib/exec.js:43
          reject(new Error(`Exit code was ${code}`))
                 ^

Error: Exit code was 126
    at ChildProcess.<anonymous> (/Users/[redacted]/.npm/_npx/fb72d33463aacf66/node_modules/govuk-prototype-kit/lib/exec.js:43:18)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:1091:16)
    at ChildProcess._handle.onexit (node:internal/child_process:302:5)

Node.js v18.12.1

in addition, this popup appears:

Environment (where applicable)

Operating system: macos (managed)
Browser:
Browser version:
GOV.UK Prototype Kit version: 13.1.0, 13.0.1

To get this working I ran the following on a non-managed laptop:

npx govuk-prototype-kit create downloadable-prototype-kit
cd downloadable-prototype-kit
echo "require('./node_modules/govuk-prototype-kit/lib/dev-server').runDevServer()" > dev.js
rm -R node_modules

I then created a zip of the downloadable-prototype-kit directory and sent that to the person with a managed laptop.  They could unizip it and run:

cd WHEREVER_THEY_UNZIPPED_IT
npm install
node dev.js

There are a few reasons why this isn't an ideal situation but for a short-term work around it would work to get someone started and the kit seems to work well after following these steps.  If we end up needing a solution like this longer term I'd like to look at the contents of the dev.js file so it's not referencing specific files/details inside the module.  Something like require('govuk-prototype-kit').commands.startDevServer() would be achievable.

To be clearer about the longer-term solution: This problem comes up because we're using npx to create the kit and to start it.  If that isn't working for people then we can go back to providing a download, the steps I've laid out above could be used to generate that download but I'd want to refine step 3 (the file that is created) before I'm willing to commit to longer term support, the current approach is suitable as a workaround to get someone started who would otherwise be blocked but there's a chance it would break in later releases and need fixing.

Tried to reproduce on another GDS managed mac and it worked - the issue did not occur. Will speak to IT team

In this case, the problem resolved after installing xcode command line tools

reopening as it happened again

I'm wondering if this happens because we try to run git, if we ran which instead it might not trigger

currently have 2 people with this issue

we fixed the kit running today by changing the package.json script to:
scripts": {
    "dev": "node node_modules/govuk-prototype-kit/bin/cli dev",
    "serve": "govuk-prototype-kit serve",
    "start": "govuk-prototype-kit start"
  },
...

as in, calling node explicitly as opposed to using node via the shebang

just a thought, I think this is because nothing is allowed to 'execute' outside of whats allowed (installed apps). In this case, without the node command added, whats being executed is node_modules/govuk-prototype-kit/bin/cli. With the command, node is being executed, which is allowed.

a little more logging that we added recently:
npx govuk-prototype-kit create test1
Creating your prototype
 - Installing dependencies
 - Setting up your prototype
sh: /Users/[redacted]/Documents/[redacted]/test1/node_modules/.bin/govuk-prototype-kit: /usr/bin/env: bad interpreter: Permission denied
/Users/[redacted]/.npm/_npx/1998d3cc354829cb/node_modules/govuk-prototype-kit/lib/exec.js:46
    reject(new Error(`Exit code was ${code}`))
        ^
Error: Exit code was 126
  at ChildProcess.<anonymous> (/Users/[redacted]/.npm/_npx/1998d3cc354829cb/node_modules/govuk-prototype-kit/lib/exec.js:46:16)
  at ChildProcess.emit (node:events:513:28)
  at maybeClose (node:internal/child_process:1091:16)
  at ChildProcess._handle.onexit (node:internal/child_process:302:5)

I think changing line 247 of bin/cli to:
await spawn('node', ['node_modules/govuk-prototype-kit/bin/cli', 'init', runningWithinCreateScriptFlag, installDirectory, `--created-from-version=${kitVersion}`, ...(getArgumentsToPassThrough())], {

might fix it. As mentioned above, this executes node (which is allowed) rather than /bin/cli which is not
it works on my machine but obviously mine is not locked down

found the previous PR where we fixed this last time, it was to call node directly instead of executing a file in the prototype folder:
https://github.com/alphagov/govuk-prototype-kit/pull/1039

had another case of this on support

@nataliecarey to review this on Monday 18th as this needs to be resolved urgently

The solution that's worked for everyone I've heard back from is to run xcode-select --install and then they should be able to use the kit.

Today I caught up with the only person I knew still had this issue.  They are no longer having the issue.  To the best of my knowledge xcode-select --install resolves the issue.


Sop Scatter V2 missing

Hello
thank you again for this fantastic tool. I use it everyday.
action scatter node in Sops only creates old scatter node not the new v2.
scatter
scatter::2.0
thank you

Should be fixed now, sorry for taking forever to do this
Release - v0.2.0


add flipper-api documentation

I was looking at using the flipper-api but found that it lacked documentation so while documenting it for my own purposes, I decided it'd be better to contribute upstream. I've documented all of the API endpoints that currently exist.
Note: I wasn't sure if a Usage section was necessary or not as it'd be virtually the same as that of flipper-ui in terms of mounting, etc. so I left that out for now. But it is probably worth noting in some way as the /flipper-api path used in the documentation examples assumes a mount point so I'm curious what your thoughts are on that.
Thanks!

@jnunemaker thanks for your feedback. I've added some documentation on mounting and pointed to the Flipper::UI usage section as it's really comprehensive. I also modified the URLs and examples.
As more endpoints are added, do you think having this on a single page will be acceptable or is it worth splitting out to separate pages per endpoint?

As more endpoints are added, do you think having this on a single page will be acceptable or is it worth splitting out to separate pages per endpoint?

I usually lean toward a single page until it hurts.


feat(cli): prompting the user to use info.title for their installed SDK

🚥 Resolves #676

🧰 Changes
This slightly changes the installation process to now prompt the user if they would like to install their SDK using the info.title value from the API definition they supplied:

Note that this work only applies if they supply a non-ReadMe API Registry value to npx api install:
npx api install --identifier buster demo.json # will continue to use "buster"
npx api install @developers/v2.0#17273l2glm9fq4l # will continue to use "developers"
npx api install demo.json # will prompt the user if they want to use `info.title`

Revised this a bit and now there's just a single identifier question with a new initial value based on info.title if it's present.


[docs] Add an announcement blog post for v0.23.0

\cc @DevExpress/testcafe-docs
Announcement blog post - the same text as in the changelog except for the first line.

Test will fail with the following link
https://devexpress.github.io/testcafe/documentation/using-testcafe/command-line-interface.html#--sf---stop-on-first-fail
But it will be alive as soon as we publish the website.


LDFLAGS mapping breaks on msvc

As seen on here/ci:
stderr=```cl : Command line error D8021 : invalid numeric argument '/Wl,-LD:\a\cargo-c\cargo-c\example-project\target\.\release'

Hello,
Thanks for the issue. I've no idea yet how MSVC is supposed to handle LDFLAGS. I'll check. Is it blocking for you?

I just passed everything through CFLAGS.

OK. So I've time to find a proper solution, which starts by having a Windows VM somewhere :-D!

For reference, the correct way to pass arguments to the linker via MSVC is via "/link". All arguments after /link are passed to the linker.
https://learn.microsoft.com/en-us/cpp/build/reference/compiler-command-line-syntax?view=msvc-170

It looks like this issue was addressed by the wasmer project by forking this repository into https://github.com/wasmerio/inline-c. The package to use on cargo is https://crates.io/crates/wasmer-inline-c/0.1.2

Be careful. wasmer-inline-c was a fork of this project, that is pointing to https://github.com/fschutt/wasmer-inline-c-rs, which is now dead. The first release was on September 2022. The attributions have even been corrected to reflect it was a fork.
Wasmer is trying to steal this project from me, and has created another fork https://github.com/wasmerio/inline-c (which apparently is a copy of https://github.com/fschutt/wasmer-inline-c-rs).
All this is unclear and I'm exchanging emails with Syrus Akbary (CEO of Wasmer) to try to protect my work. As soon as I've pointed him out that https://github.com/fschutt/wasmer-inline-c-rs was dead, he has created https://github.com/wasmerio/inline-c.
I don't want this fight. Syrus is claiming the ownership after 3-4 years. This is non-sense…

It's even better than that.
So Syrus has put https://github.com/fschutt/wasmer-inline-c-rs back online at https://github.com/wasmerio/inline-c. It's interesting to read the commit history.

First commit, https://github.com/wasmerio/inline-c/commit/1edacf7d9c7914598aa9ed070587a5e4a877190a, we see that fschutt (who was working at Wasmer at that time) has stolen my code. This first commit copies the inline-c-macro code, even with my name as the author (https://github.com/wasmerio/inline-c/commit/1edacf7d9c7914598aa9ed070587a5e4a877190a#diff-8f8d4d6e0fe204c781061cda8ccf902914990001e641e61ca2c7c181e386d4dbR6-R8). This first commit copies the inline-c code (the run.rs and assert.rs module has been inlined, that's mostly it, https://github.com/wasmerio/inline-c/commit/1edacf7d9c7914598aa9ed070587a5e4a877190a#diff-b1a35a68f14e696205874893c07fd24fdb88882b47c23cc0e0c80a30c7d53759).
Second commit (2 days later), https://github.com/wasmerio/inline-c/commit/ef0fe1c46a696c16c402f49e26fa7c799d496fc1. fschutt has clarified the author to restore my name,
Third commit, https://github.com/wasmerio/inline-c/commit/0f0b3a6c3b57ea2a0af03a76044392765ca764fa, nothing was compiling so stuff had to be fixed.

Note that the LICENSE.md file not been copied, here is my LICENSE, https://github.com/Hywan/inline-c-rs/blob/master/LICENSE.md.
Was Syrus aware? Probably, as he is co-owner on crates.io, https://crates.io/crates/wasmer-inline-c:

At the time of writing (September 14th, 2024), he has not updated crates.io yet.

The "wasmer-inline-c" fork, as far as I can tell, works only for windows and removed the ability to run inline C/C++ code. Wasmer's c-api code switches between the original inline-c and the fork based on platform detection.
Ideally there'd be one inline-c repository (Hywan/inline-c-rs) and it would have full support (execution included) for Windows. I'd be surprised if wasmer wouldn't switch to such a thing if that were in place. Would you be willing to accept a patch adding Windows support? I'm confident I could do it. We could also add a GitHub workflow that will verify this works as part of CI.

I would definitely be happy with a patch for Windows support yeah. Feel free to provide one :-).


Add Google Nearby Messages

Google Nearby Messages API for iOS is missing.

VS bug #732440

@rubo I have added it as an enhancement and assigned it to @Redth

It's been two years now. Any updates on this?

Any update on this?

@waletoye @givod As the latest release is dated August 2018, looks like it's abandoned by Google. With this said, I see no reason for the Xamarin team to implement bindings for this.
For this particular library, you can create bindings yourself with Visual Studio without any problems or deep knowledge of Objective-C. I'd tried so that two years ago to test the library in a Xamarin app.

Hello @rubo, I currently tried creating binding with objective Sharpie for this cocoapod library, when I try the command "Sharpie pod init Google Nearby Messages", it doesn't work, it takes forever, sometimes it returns with not found, I use Sharpie 3.4 on xcode 12. If you have a working binding you did, please share, cos I am way beyond my deadline. Thank you

@givod Here you go. It's the assembly only as I lost the source. As it was for testing purposes only, the namespace has been left intact. Anyway, I hope it will be helpful for you.
For more details, you can check it, for example, with dotPeek.

@rubo, thank you, will give it a try.


Roll Engine from 59778ae91c1f to 74b70aa59001 (2 revisions)

https://github.com/flutter/engine/compare/59778ae91c1f...74b70aa59001
2021-12-09<EMAIL_ADDRESS>Roll Skia from b4d01cbe41b1 to 5d1d92c505c0 (1 revision) (flutter/engine#30238)
2021-12-09<EMAIL_ADDRESS>Merge NDK and licenses into Android dependencies script (flutter/engine#30230)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Add adverts to Test3 About page

@PatReynolds to get data ad slot numbers

@Vino-S  has code for implementation


feat: Class editor: allow DnD to rearrange properties

@finos-cla-bot[bot] check

@finos-cla-bot[bot] check

@finos-cla-bot[bot] check

@finos-cla-bot[bot] check

@finos-cla-bot[bot] check

@finos-cla-bot[bot] check

@cla-bot[bot] check

@cla-bot[bot] check

@TheJuanAndOnly99 seems like it still didn't work. Does @xannem need to repush or something?

@cla-bot[bot] check

hi @akphi I think the bot is having issues. Give me a few minutes and I'll solve it

@cla-bot[bot] check

@akphi sorry about the issues. should be all set now!

@cla-bot[bot] check

@xannem are you able to close this PR and open a new one? The cla-bot got stuck on this one. But you are now covered and this wont happen again.

Thanks @TheJuanAndOnly99. Things look good now!


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provid

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.1
Time: Thu Mar 17 2016 10:50:04 GMT-0700 (Pacific Daylight Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\mmkhan\AppData\Local\GPMDP_3\app-3.0.1\resources\app\build\inject\GPMInject\interface\mouseButtonNavigation.js:8:8.
    at BrowserWindow.callIntoRenderer (C:\Users\mmkhan\AppData\Local\GPMDP_3\app-3.0.1\resources\atom.asar\browser\lib\rpc-server.js:201:19)
    at emitTwo (events.js:92:20)
    at BrowserWindow.emit (events.js:172:7)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #470