Added new MXML (MX and Spark) snippets

Thanks for the contribution! I'm thinking that maybe we should make this into a separate extension. Not everyone using MXML is necessarily using Flex, since there is also Royale and Feathers. It would be good if these snippets were only presented to developers who actually want them.

Oh, I should have asked that in this issue https://github.com/BowlerHatLLC/vscode-nextgenas/issues/200
So there is no way in the extension to differentiate which snippets are provided based on if it is Flex, Royale or Feathers in use? If so, then it would be better to pack them in separate extension, and mention that in Wiki.

No, I don't think it's possible to enable only a subset of snippets. You always get the full list for any file in the language that they're designed for.

Done, as new extension.

Nice! I'll be sure to share it on social media next week.

Cool! It doesn't look too complicated. Maybe I will create such repository also for Royale. I will add it to my To Do list!


Ignore tag doesn't work in RecyclerView items

[ ✓ ] I have verified there are no duplicate active or recent bugs, questions, or requests
[ ✓ ] I have verified that I am using the latest version of Aesthetic (see the README)
[ ✓ ] I have given my issue description title
[ ✓ ] I have read over the documentation (before asking questions on how to do something).

Include the following:

Aesthetic version: 0.4.7
Device OS version: 7.1.2
Device Manufacturer: LG
Device Name: G2

Reproduction Steps
Set item's android:tag=":aesthetic_ignore" in XML or view.setTag(":aesthetic_ignore") in Java. I also tried adding tag to RecyclerView.
Expected Result
Aesthetic ignore view.
Actual Result
Aesthetic keep theming view.

Hey @HugoCastelani same issue i am facing inside listview item,
Have you found any solution? then please let me know.

Sorry, I didn't.

Applying the tag to the views you want to ignore in your list item layouts works just fine, just tried it. It can't go on the parent view, currently, it has to go on the exact view.


[495] Setup Vagrant for development

Story / Bug id: #495
Reference person: @riv2q
Description:
Adding application setup option via vagrant
New imports / dependencies:
psycopg2==2.6.2
What tests do I need to run to validate this change:
vagrant up
vagrant ssh
./manage.py runserver <IP_ADDRESS>:8000 --settings=volontulo_org.settings.dev_vagrant

Codecov Report

Merging #512 into master will not impact coverage.

@@           Coverage Diff           @@
##           master     #512   +/-   ##
=======================================
  Coverage   91.26%   91.26%
=======================================
  Files          53       53
  Lines        2073     2073
=======================================
  Hits         1892     1892
  Misses        181      181

Continue to review full report at Codecov.

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

I saw, that You @riv2q have fix some issues, that I pointed out at, but we still have ther few use of su - which doesn't look necessary.
That's the last issue from me and if You will resolve it I'm going to merge this branch.
Thanks and good work!

su - -c "pip3 install -r $PROJECT_DIR/requirements/dev.txt"

actually sudo is needed there, because lines
python3 $PROJECT_DIR/manage.py migrate --settings=volontulo_org.settings.dev_vagrant
python3 $PROJECT_DIR/manage.py loaddata $PROJECT_DIR/initial/data.json --settings=volontulo_org.settings.dev_vagrant

gives error that django is not found
mb i will debug it later

actually sudo pip3 install should stay with sudo in case we are not using virtualenv on virtual vagrant environment :)
https://stackoverflow.com/questions/27849412/permissionerror-with-pip3
https://stackoverflow.com/questions/39950254/permissionerror-errno-13-permission-denied-usr-lib-python3-5-site-package


Trip update event load

This PR updates performance_manager to ingest RT_TRIP_UPDATES parquet files, along with the following:

create lib/gtfs_utils.py to hold common gtfs processing functions.
update july_17_filepaths.json with RT_TRIP_UPDATES files
move path selection into event library files

Asana Task: https://app.asana.com/0/1202784843755682/1202798609780869

i had a few small things, but LGTM. i had a few small things in there and think a few comments could be really helpful for future us.


cargo watch should read a configuration file for actions if present

Context: I love cargo watch, and am currently invoking it as cargo watch -x check -x fmt -x doc -x clippy -x run. This is a bit of a handful to type, so I've saved that in a shell script, at the root for a project. I would prefer for cargo watch to just mean that, ideally in all my projects by default.
Proposal:

Have cargo watch, when not explicitly told what to do, look for configuration before defaulting to cargo watch -x check.ml.
Some sort of per-user config in the home directory could serve as a customizable default.
If neither is present, we should fall back to the current default behavior.

There's cargo-make for that kind of thing.

There's cargo-make for that kind of thing.

Thanks for the suggestion, I'll have a look at cargo make!

Thanks for the suggestion, I'll have a look at cargo make!


 Most of the time,
long methods contain nested loops or nested if statements,
and handle multiple tasks instead of one. One big reason
that students wrote long methods is that they tended to
place a new feature in an existing method rather than to cre-
ate a new one. Likewise, students prefer to modify existing
switch or if blocks rather than to implement polymor-
phism, since it is easier for students simply to add another
condition to the existing code than to understand an inheri-
tance hierarchy structure across files. This violates a rule of
object-oriented design—when you see a switch statement,
you should think of polymorphism [15].

Refactor Pseudocode
Algorithm constructTeamsBiddingInfo(unassignedTeams, signUpTopics)
Input: array unassignedTeams of teams, array signUpTopics of topics
Output: array teamsBiddingInfo with bids sorted by priority
allBids ← fetchBids(unassignedTeams, signUpTopics)
teamsBiddingInfo ← empty array

for each team in unassignedTeams do
    teamBids ← processTeamBids(team, allBids)
    sortedBids ← sortBidsByPriority(teamBids)
    append {team_id: team.id, bids: sortedBids} to teamsBiddingInfo

return teamsBiddingInfo

Algorithm fetchBids(unassignedTeams, signUpTopics)
Input: array unassignedTeams of teams, array signUpTopics of topics
Output: array allBids of bid records
return query all Bid records where team_id in unassignedTeams and topic_id in signUpTopics

Algorithm processTeamBids(team, allBids)
Input: team object, array allBids of bid records
Output: array topicBids for the team
topicBids ← empty array

for each bid in allBids where bid.team_id = team.id do
    append {topic_id: bid.topic_id, priority: bid.priority} to topicBids

return topicBids

Algorithm sortBidsByPriority(bids)
Input: array bids of bid records
Output: array bids sorted by priority
return bids sorted in ascending order of priority


Potential improvements to address prop to make it more customizable

Description
Address.tsx updates that allow for better customization of the card while retaining default values that work out of the box.

Address.tsx continued...

Possible usage of customizing the address card in an implementation.

Possible result of the implementation's customizations

Additional Information

[x] I have read the contributing docs (if this is your first contribution)
[x] This is not a duplicate of any existing pull request

Your ENS/address: JacobHomanics.eth

@carletex

We should try to standardize with the rest of the components (Balance, etc)

Gotcha, I will begin taking a look at those components.

The prop size is not being used in your changes and it's changing the look of existing components that use a different size. (e.g. TransactionTable.tsx)

This was one of my concerns, so thanks for pointing it out.
You can see that size is also still being used in the Address component.

So I am not sure about the best way to handle this change, especially since it affects other components as well. I will try brainstorming to think of a solution, but I don't have many potential answers at the moment.

wrapperClasses might be a better name than cardClasses

Heard. I will start using that terminology from here on out

For reference, this screenshot showcases a real example for the need to have this type of customizability. You can see that the Address component, normally, does not fit into my website's design - making things look out of place and clunky.

This screenshot shows the feature being implemented and allowing the developer to freely customize it to fit into their website's design as the design gets more complex and custom.

@rin-st @carletex I've implemented ya'lls feedback and made textClasses into textClassName and wrapperClassName.

@Hotmanics , thanks for your fast response! Some additional points.
I'm thinking do we really need wrapperClassName but still not sure
your example

Possible usage of customizing the address card in an implementation. <img alt="Screenshot 2024-03-11 at 3 46 41 PM" width="687" src="https://private-user-images.githubusercontent.com/32080359/311862530-9511f804-5d9e-4c44-8764-758d05e40722.png

pros

easier to customize

cons

all wrapper classNames from your example could be implemented with addional wrapper div when needed
should it be additional class names since it seems we always need flex items-center? And for adding one class we always need to pass default props + new class, for example flex items-center bg-red-500. Alternative: use wrapperClassName with default classes, not instead of them. flex items-center ${wrapperClassName}

Also noticed that when changing textClassName only text changes, but not avatar and copy icon. So proportions breaks
text-sm

text-4xl

Not sure if it planned behavior or not.
So it looks like these textClassName should be applied some way to avatar and copy/copied icons. Looks like using predefined size like it was before. Or we need also IconsClassName prop 🤯 .

Possible result of the implementation's customizations

For reference, this screenshot showcases a real example for the need to have this type of customizability.

Looks like for both cases you could use just needed wrapper div
tried on se-2 main page:

Connected Address:

    <Address address={connectedAddress} />

  <Address address={connectedAddress} />

result:

What do you think?

What do you think?

@rin-st
I see. I guess my React fundamentals were off point here which may invalidate most of the points of the PR. It never occurred to me that I could just wrap the component into an enclosing div to achieve the same result. That seems moreso the proper way to accomplish the customizability I am seeking. Are there any restrictions using this approach?
I do think the properties of the image, text, and copy button should be customizable and independant of eachother if the developer chooses so.
Gonna work on some commits to achieve that and will ping when I get an example made up.

Ok...so i think there still may be a need to define the wrapper class as something other than the hardcoded flex items-center as it is still too opinionated IMO. Additionally, as mentioned in my previous comment, it would be nice to independently change the properties of the avatar, text, and copy elements. Thus, the latest commit represents the potential path forward.
There is now BaseAddress and ScaffoldAddress components.
The philosophy is:
BaseAddress acts as the functional component with full customizability.
ScaffoldAddress wraps around BaseAddress, resulting in the functionality and developer workflow of the original Address component. It is opinionated.
Here is an example implementation of both of the components. You can see ScaffoldAddress stays as simple and clean to implement as Address originally desired. Meanwhile, for the extra customizability, you can use BaseAddress directly and pass in the appropriate parameters that are necessary for your project.

This example demonstrates the ability to stack each element and customize them as well if the developer desires that for their application. This was previously not possible. It will be more work and require more understanding on their part, but thats the price of customization IMO.

Thoughts and feedback? @rin-st

additional adjustments made to make the blockiesize more customizable.
example implementations:

example outcome:

I've also added for the ability to re-order the components that get rendered

I may be going overboard with these customizations, so let me know if I'm just overwhelming y'all lol.

Thanks for the changes!

I may be going overboard with these customizations, so let me know if I'm just overwhelming y'all lol.

A little bit 🙂 . Probably it's my bad since I wasn't clear what we need to do. And I thought to discuss first before writing code.

wrapperClassNames instead of flex items-center. I think flex items-center is enough for most of the cases. Don't know when and why let's say flex-col needed. Yes, wrapperClassNames could add flexibility, but adding to much freedom causes more bugs in code (both ours and user's) and difficulty of maintaining.
Different sizes of avatar/text/icons. Voting for one size prop for all of them. Again, don't know why to use let's say small text and large copy icon at the same time
Ordering. Interesting implementation 👍  but again, not sure who needs to change the order.

Thanks for the changes!

I may be going overboard with these customizations, so let me know if I'm just overwhelming y'all lol.

A little bit 🙂 . Probably it's my bad since I wasn't clear what we need to do. And I thought to discuss first before writing code.

Not your bad at all! My coding style is code first, ask questions later.

wrapperClassNames instead of flex items-center. I think flex items-center is enough for most of the cases. Don't know when and why let's say flex-col needed. Yes, wrapperClassNames could add flexibility, but adding to much freedom causes more bugs in code (both ours and user's) and difficulty of maintaining.

I'm sure we can find some scenarios if we think hard enough. The one that comes to mind is the example shown above where they are vertical. For example, you can have a page full of "profiles" in that fashion, rather than a list of, what feels to me, like "widgets". I think user complexity can stay fairly simple if not the same through the introduction of "ScaffoldAddress", which is the component that fits 95% of scenarios (Pretty much what the original "Address" component is). Then, if they want to dig deeper and explore a customizable option, then that option exists. It may not be easier, but they do have the option. Albeit, yes this does add more code management and complexity to the template repository that may not be needed.

Different sizes of avatar/text/icons. Voting for one size prop for all of them. Again, don't know why to use let's say small text and large copy icon at the same time

Thus, "ScaffoldAddress". As "Address" currently stands, there is little need to have those options, but once you start allowing customizations, then I can personally see a situation where there is small text and a large copy icon.

Ordering. Interesting implementation 👍  but again, not sure who needs to change the order.
It's a rough and dirty implementation, but it does the job. Glad to clean up later if these changes are something ya'll want to continue to consider.

If it's not clear, I'm obviously for as much customization as possible. I just ask the question "Why not?" The only reason that comes to mind in this situation is that it adds more complexity and management on the Repo's side.

I agree that the customizing is great!
But for me, mastering is to use minimal possible props that are enough for most cases. And add new props only when required. Like in your case: you need the possibility to update styles - you create pr with the functionality needed.

The only reason that comes to mind in this situation is that it adds more complexity and management

I think it's a big reason :)
Let's wait a bit what others think, but it can take time since part of core contributors are at EthLondon

But for me, mastering is to use minimal possible props that are enough for most cases. And add new props only when required. Like in your case: you need the possibility to update styles - you create pr with the functionality needed.

++

Thanks so much @Hotmanics for this !! Just created #786 out of this discussion, I think it's better if we first discuss their and then tackle this. Since then will have a better picture and also in line with considering other components 🙌
Really feel bad for closing this :( and appreciate the hard work but that's why we have this note while creating the PR :

But thanks again for contribution!! This led us to #786 and we could totally re-open this after #786 if we feel this is the way to go  🙌!


chore: dw-lisbon-go to 0.0.3

chore: Promote dw-lisbon-go to version 0.0.3

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Improvements For All Files

Formatted license file as a markdown file (this is legally sound, and is used elsewhere).

This, however, has prevented GitHub from automatically recognising it and applying that cool header on top. It uses Licensee for identification and outside of a certain tolerance, it doesn't recognise it anymore (even if the wording is the same). 😢
If you prefer that header provided by GitHub, I'll revert the file to GitHub's default CC0 template.

Added section in CONTRIBUTING.md pertaining to categories.
Added some new libs to the listing, as well as a CC0 badge up top.

I have fixed as much as is possible based on your feedback!

About the tables: I see why the tables in README.md are HTML, colspans are simply not available in Markdown. But the tables in CONTRIBUTING.md should be doable in GitHub flavored Markdown, which should be preferred.

I'll give it a shot - I had big trouble with GFM formatting it with weird cell alignment, etc. last time but maybe I'll be able to make it work.

Okay, this is the most I'm able to change in CONTRIBUTING.md. Keep in mind that the second table in that file uses colspan, so I can only really revert the outer structure of the first table to GFM. I also can't use GFM code blocks as they must be multiline for GFM to apply proper syntax highlighting, even with the keyword present, yet the table will then not recognise the row.

Ah, yeah, you're right - code highlighting in Markdown tables is a pain, I forgot about that.
This looks good overall now :+1:
Thanks for your time and effort :)


Consider to automate coverity analysis in CI

Consider automating Synopsis Coverity runs in CI on PRs and push to master. This is similar to what is done for Code Sonar.

See also #161 which could potentially fit in well with coverage statistic in sonar.

Also try to filter out Boost and STL code from the analysis, as these code artifacts are responsible for the only 9 remaining defects.

See also https://github.com/ckormanyos/wide-decimal/issues/143


Architecture: Interaction with mu-plugins

In WordPress, a plugin placed in the mu-plugins folder isn't considered active or installed, ie
in_array( 'some-mu-plugin/plugin.php', get_plugins() ) === false
is_plugin_active( 'some-mu-plugin/plugin.php' ) === false

In my experience, a plugin dependent on another has used a callback to check if it is present:
if ( function_exists( 'wc_get_products' ) ) {
    // WooCommerce is active, dependency met.
}

How will the plugin dependencies handle such a situation gracefully?

In all likelihood plugin dependencies won't handle this situation are really no built in resources, such as get_plugins(), that work within the mu-plugins folder. But saying that it shouldn't be necessary to have mu-plugins to do this check any longer.
Plugins don't have to be active to scrape the headers for a Requires Plugins header but they do have to be in wp-content/plugins.
Listing Requires Plugins: woocommerce will prevent the activation of the plugin that requires it until WooCommerce plugin is installed and active. Notification would be given both that there are plugin dependencies that are required for some currently installed plugins and in the event of an activation attempt that fails for this reason.

I don't think that mu-plugins is something that should be handled by the plugin-dependencies implementation...

There are three types of mu-plugins:

Those that are self-contained mu-plugins, that aren't "normal plugins"
Those that are plugins, but stored within mu-plugins.
Those that are simply loaders, which forcibly include a plugin within the "normal plugin directory"

Case 1 doesn't matter here I feel, as although I'm sure some would want to use it, it seems like a reasonable thing to skip.
Case 2 probably doesn't matter too much, because they don't get updates anyway, and authors are going to run into other issues.
Case 3 is an example where is_plugin_active() will fail to return truthful even though a mu-plugin is forcibly including it
3 can probably be worked around by filtering get_plugins to always have the plugin get loaded (rather than require'ing it), and would probably suffice as a work-around for those who want to use the functionality.. but even then, I doubt it matters much.. given plugin dependencies are for those who want things handled automatically and aren't likely to want to use mu-plugins.

@dd32 I think @costdev and I have written a loader that solves Case 3.
https://core.trac.wordpress.org/ticket/60504#comment:15
https://gist.github.com/afragen/9117fd930d9be16be8a5f450b809dfa8

Doesn't the Worker plugin from ManageWP install an MU plugin?

Yes and no. It does add a loader file to mu-plugins, because (from the loader's description):

to increase performance and reliability.

Also from the description:

It is automatically disabled when disabling the main plugin.

So this seems more like a file intended for loading ManageWP - Worker earlier than "normal" plugins. ManageWP - Worker itself is installed in wp-content/plugins, and is in active_plugins.
As such, it's detected as a dependency by Plugin Dependencies:


Decorum Install Typeface

The light typeface used in the installation guide is very hard to read which opposes the design-oriented experience.

A fix is in progress and will be released with the Beta 1 release


vertexai: added support for custom ids and sprinkled in some docstring updates

PR Description
PR implements the feature described as missing in https://github.com/langchain-ai/langchain-google/issues/435. The VectorStores are now able to receive custom document ids and use them to create/update the index and upload the documents to google cloud storage. I added a unit test that checks if the custom ids are used, instead of creating unique ids.
Additionally, i updated and formatted the docstrings in the file.

Relevant issues
https://github.com/langchain-ai/langchain-google/issues/435

Type

🆕 New Feature
📖 Documentation
✅ Test
Changes(optional)
_BaseVertexAIVectorStore.add_texts
tests/unit_tests/test_vectorstores.test_add_texts_with_custom_ids
Testing(optional)
Ran tests using make tests. All passed, 12 Warnings.
Note(optional)
The documentation on https://python.langchain.com/v0.1/docs/integrations/vectorstores/google_vertex_ai_vector_search/ could be updated to include the new feature.

What is missing to merge this PR?


Form Field Group [Pattern] using Stack Layout and FFG Context

Second implementation after discussion on 26/6 regarding Form Field Group
This uses Stack Layout to show how a group could easily be made with our layout components pretty cleanly
Uses Form Field Group context as a wrapper to specify label width, placement, and variant

(I talked to Liberty earlier) If this is more of a pattern thing and there is a way we can do without introducing additional context / dependency of code (e.g. const sharedProp = {position: 'left'}; <FormField1 {...sharedProp} /> <FormField2 {...sharedProp}/>) and not too much more code consumer going to write, then let's not introducing them.
When we get to look at app wide config / pattern, we may be able to do something at the salt provider level by then.

Looks good to me other @origami-z 's comment around whether we need to provide this vs leaving it to consumers

The legacy FormField (when label was at the top) used to align the label, value and helpertext.  I haven't seen the new design spec but wanted to check that this isnt still a requirement as in the examples added in this PR the value is not in alignment

Aside from what David has hilighted, everything else looks good


iFrames / render html

Lovely project, having a fun time creating a POC with it.
Any plans to add an iframe plugin so we can show embeds? Couldn't find any html plugins and didn't have much luck trying to create one using something similar to https://github.com/mdx-editor/editor/issues/95#issuecomment-1732225694

I'm doing some generic HTML support - check this thread: https://github.com/mdx-editor/editor/issues/92. Not sure how would that work with iframes/embeds, though.


Standard 5.1.0 is released

Just FYI!: https://github.com/feross/standard/commit/28e5f56d5d0317acdff89cd534ab31402de70b49

Actually, just made a pull request:
https://github.com/ricardofbarros/linter-js-standard/pull/43


Does this package support Python 3?

On 19 February 2017, Python 3 will be 3,000 days old!  We are interested to see if we can get at least 50% of the Top 5,000 PyPI packages to compatible with Python 3 by that date.  We are really close and given that this package is in the PyPI Top 5,000, we seek your assistance in pushing us over that threshold.
So, if this package already supports Python 3 then please consider adding an appropriate trove classifier "Programming Language :: Python :: 3" to this package's PyPI page so that tools can automaticly determine its Python 3 compatibility.

Yes, this package does support Python 3 and does have the Python 3 trove classifier, and has since early 2011. If these messages are automated, it might be worth double-checking the criteria.

You are correct @minrk ...  I got messed up by https://pypi.python.org/pypi/pyzmq-static


vite static assets does not work in build

https://vitejs.dev/guide/assets.html#importing-asset-as-url does not work in the build because URLs are not the same in dev and prod. And that's the root cause of https://github.com/electron-vite/electron-vite-react/issues/78
https://github.com/electron-vite/electron-vite-react/commit/71dd5078d99a8dacab297433de894e101307d286 is a hack to fix that.
However,
Something like./electron.png is actually not valid usage for vite. It's neither via public nor via dynamic import. It just happens to work in dev and prod because of the particular route of App.tsx.
The valid way to do this in vite is to use /electron.png. But that does not work in this boilerplate because the URL is not handled correctly.

Do you still prefer to use the files in the public directory?
e.g. import foo from '/foo.png'
This will only work with browsers based on the http protocol.
And Node.js can only use relative paths, absolute paths, bare modules
. /module or /User/name/module or module.
In other words, if the public path (/foo.png) is used, is this required? It can't be replaced by other means?

On Saturday, 12 November 2022 21:29:04 CST 草鞋没号 wrote:

Do you still prefer to use the files in the public directory?
e.g. import foo from '/foo.png'
This will only work with browsers based on the http protocol.
And Node.js can only use relative paths, absolute paths, bare modules
. /module or /User/name/module or module.
In other words, if the public path (/foo.png) is used, is this required?
It can't be replaced by other means?

Sorry for not making my point clear.
I think the evil is the inconsistency. You can write down a vanilla vite usage
and see everything is good in yarn dev while everything booms in yarn  build. dev and prod should have the same behaviour or the dev preview is
meaningless.
BTW, import foo from './assets/foo.png' will not work as expected in prod.
The image will be packed into dist/assets/foo.xxxx.png, but the imported URL
will be somewhere under dist, not dist/assets.

electron-vite-react changes Vite default behaviors?


Packaging for debian

Hi,
Can you package for Debian (specifically Jessie has systemd support)?  I see you packaged profile-sync-daemon.
Thanks,
John

Sorry, but since I do not run Debian, it would be a task to do this and properly test it.  My Makefile is good so you are just fine to build it on your own as detailed in https://github.com/graysky2/anything-sync-daemon/blob/master/INSTALL

But you made .deb for profile-sync-daemon (https://launchpad.net/~graysky/+ppa-packages).
Do the PSD and the ASD have such big differences?..

The debian build system is needlessly complex in my opinion and since it is not my native packing format, I would need to spend to some time in order to maintain it. Particularly the post/pre install scriptlets.  Move on over to Arch Linux and enjoy my packages if you want :grin:

I'm enjoying your packages on Arch more then 3 years, thanks!))
But my fiancée prefers Linux Mint..

I did the changes for packaging in Debian. Would you consider a pull request?

My problem is supporting changes down the road.  For example, if core changes to the script are ever needed, the package manager should stop asd before the update.  I am not savvy enough with the debian packager to do this.  With Arch, it is trivial.

I handled that case in the preinst file.   Do you want to link to my repo and I'll maintain the debian packaging?

I can.  Send me the info.

See:
https://github.com/johnarnold/anything-sync-daemon/tree/debian

I am happy to link your repo but the changes debian/* stuff really has no place for me since I cannot test nor maintain.

Sure that is fine.  Do you want a pull request to update the README, how about a link in the Wiki?  Just want to save other debian/Ubuntu users the trouble in the future, and maybe they'll find some bugs too :)

Sure thanks.


Bump ADTypes compat entry to include version 1

Will rerun checks once https://github.com/SciML/Optimization.jl/pull/738 has gone through 👍


Automatic renaming of columns and tables for @map and @@map

Problem

Hi 👋
First off, thanks for the amazing library! Working with Prisma has been a great experience!
Right now, when you use @map or @@map to rename a column or table Prisma automatically generates a DROP TABLE (at least for Postgres).
This can be mitigated by using the --create-only option for Prisma Migrate, but it still creates a chance of data loss.
Suggested solution

It seems like @map and @@map are unambiguously for renaming columns and tables, so ideally Prisma Migrate would automatically generate a ALTER TABLE instead of a DROP TABLE for renaming.
Alternatives

I see that there are planned improvements to Migrate renaming on the roadmap, however the roadmap only mentions improvements for renaming fields/models and does not mention @map or @@map renaming.
If renaming can't be done automatically, then it would still be a huge benefit to the developer experience (and safety of data) to have @map and @@map also use the upcoming renaming improvements.
Additional context

For context, our data team is specifically asking us to use snake_case for all fields and tables so we'll be using @map and @@map a lot.
An alternative feature request might be to support a global configuration of table/column casing, but that sounds like a problem specific to us and less practical than @map and @@map using ALTER TABLE.

An alternative feature request might be to support a global configuration of table/column casing, but that sounds like a problem specific to us and less practical than @map and @@map using ALTER TABLE.

You can find related requests to this under this label: https://github.com/prisma/prisma/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc+label%3A"topic%3A+introspection-configuration" We are not focussing on this right now though, so solving your actual problem is probably closer.

Just to make sure, can you show a before and after schema and the SQL migration with the unwanted DROP generated?

Sure!
Before
datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

generator client {
  provider      = "prisma-client-js"
  binaryTargets = ["native", "rhel-openssl-1.0.x"]
}

model Banner {
  id            String   @id @default(cuid())
  leadLine      String
  title         String
  subtitle      String
  imageUrl      String
  navigationUrl String?
  createdAt     DateTime @default(now())
  updatedAt     DateTime @updatedAt
}

After renaming columns/fields with @map
datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

generator client {
  provider      = "prisma-client-js"
  binaryTargets = ["native", "rhel-openssl-1.0.x"]
}

model Banner {
  id            String   @id @default(cuid())
  leadLine      String   @map("lead_line")
  title         String
  subtitle      String
  imageUrl      String   @map("image_url")
  navigationUrl String?  @map("navigation_url")
  createdAt     DateTime @default(now())
  updatedAt     DateTime @updatedAt
}

Generated migration
/*
  Warnings:

  - You are about to drop the column `leadLine` on the `Banner` table. All the data in the column will be lost.
  - You are about to drop the column `imageUrl` on the `Banner` table. All the data in the column will be lost.
  - You are about to drop the column `navigationUrl` on the `Banner` table. All the data in the column will be lost.
  - Added the required column `lead_line` to the `Banner` table without a default value. This is not possible if the table is not empty.
  - Added the required column `image_url` to the `Banner` table without a default value. This is not possible if the table is not empty.

*/
-- AlterTable
ALTER TABLE "Banner" DROP COLUMN "leadLine",
DROP COLUMN "imageUrl",
DROP COLUMN "navigationUrl",
ADD COLUMN     "lead_line" TEXT NOT NULL,
ADD COLUMN     "image_url" TEXT NOT NULL,
ADD COLUMN     "navigation_url" TEXT;

After renaming the table with @@map
datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

generator client {
  provider      = "prisma-client-js"
  binaryTargets = ["native", "rhel-openssl-1.0.x"]
}

model Banner {
  id            String   @id @default(cuid())
  leadLine      String   @map("lead_line")
  title         String
  subtitle      String
  imageUrl      String   @map("image_url")
  navigationUrl String?  @map("navigation_url")
  createdAt     DateTime @default(now())
  updatedAt     DateTime @updatedAt

  @@map("banner")
}

Generated migration
/*
  Warnings:

  - You are about to drop the `Banner` table. If the table is not empty, all the data it contains will be lost.

*/
-- DropTable
DROP TABLE "Banner";

-- CreateTable
CREATE TABLE "banner" (
    "id" TEXT NOT NULL,
    "lead_line" TEXT NOT NULL,
    "title" TEXT NOT NULL,
    "subtitle" TEXT NOT NULL,
    "image_url" TEXT NOT NULL,
    "navigation_url" TEXT,
    "createdAt" TIMESTAMP(3) NOT NULL DEFAULT CURRENT_TIMESTAMP,
    "updatedAt" TIMESTAMP(3) NOT NULL,

    PRIMARY KEY ("id")
);


Support runtime programs pagination

What is the expected enhancement?
The Qiskit Runtime service is adding pagination support for the GET /programs endpoint. Qiskit will need to support that once it becomes available.

The parameters are described here: https://runtime-us-east.quantum-computing.ibm.com/openapi/#/ .
@rathishcholarajan Is there additional information you're looking for?

@jyu00 Nope. Thanks!

5


Concert Bot Rules

Proposed changes:

Added RulePolicy to config, removed fallback & mapping policies
Implemented chitchat & nlu_fallback intents as rules

@b-quachtran thanks for submitting this - the same goes here as in Arjaans PR, the stories should also be converted to yaml and so should the nlu training data (if there is any)

@b-quachtran what about the stories for searching venues/concerts?

@akelad Oops sorry .gitignore was set to ignore changes in the concertbot/data folder -_-


feat: add a runtime for FrankenPHP with Symfony

Add a runtime for the worker mode of FrankenPHP. To be released this afternoon at ForumPHP!

Could you please rebase your PR now when #125 is merged?

@Nyholm rebased

Thank you!

https://packagist.org/packages/runtime/frankenphp-symfony

Thank you very much @Nyholm!


add List.splitIf builtin

Not sure about naming. Also, this may not be important if we have a nice parsing library to use, but looking at day 4, I would like to first split the string by new line, then split the produced list by empty lines. I feel that some sort of split function that uses a predicate would be quite useful for lists in general. Of course, this can be implemented with List.walk if needed.
CC: @ayazhafiz since he is looking into adding some sort of parsing library that may solve this issue in a nicer way and makes adding this unnecessary.

Yeah I think this is better done by a parsing library and not needed as a builtin. Here's what a parser for day 4 with a combinator I'm working on ends up looking like
Square : [Marked, Sq U64]
Board : List (List Square)
Input : [Bingo (List U64) (List Board)]

parseInput : _ -> Result Input _
parseInput = \input ->
    u64 = digits |> map Num.toU64
    square = u64 |> map Sq

    parseSepSquare =
        (opt spaces)
        |> dropBefore square

    parseBoardLine =
        oneOrMore parseSepSquare
        |> dropAfter newline

    parseBoard =
        newline
        |> dropBefore (many parseBoardLine)

    parseNumbers =
        u64
        |> sepBy (codeunit (Num.toU8 ','))
        |> dropAfter newline

    parseLines =
        const (\nums -> \boards -> Bingo nums boards)
        |> apply parseNumbers
        |> apply (many parseBoard)

    when parseBytes parseLines input is
        Ok a -> Ok a
        Err (Msg s) -> Err (Msg s)

My one concern with a solution like the one you wrote is that it is complex. I feel like a new user would not be able to think up something like that without many examples and some simplification. Using step by step direct splitting and cutting things up is much easier to think through.

My concern is also a bit with bloating the standard library. I don't feel
terribly strongly, but IMO the use case is relatively niche (parsers), and
as you mentioned, it can be implemented with List.walk. Moreover a
walk-based (or other) implementation is likely to be much more efficient
than splitIf, as it avoids intermediate lists.
On Mon, Oct 3, 2022 at 11:02 AM Brendan Hansknecht @.***>
wrote:

My one concern with a solution like the one you wrote is that it is
complex. I feel like a new user would not be able to think up something
like that without many examples and some simplification. Using step by step
direct splitting and cutting things up is much easier to think through.
—
Reply to this email directly, view it on GitHub
https://github.com/roc-lang/roc/issues/4176#issuecomment-1265683876, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE6GL6TFZK6LWSLWKRAKQUDWBL7P7ANCNFSM6AAAAAAQ3WJMXM
.

Though performance should be fine with seamless slices.


Diff is not Defined as an available token [feature]

Version and OS
<IP_ADDRESS> - python package
Is your feature request related to a problem? Please describe.
{{ diff }} is not a listed usable token in the table in the notification tab.

Describe the solution you'd like
I would like to see a list of all available pre-packaged tokens that could be used in the notification body.
Describe the use-case and give concrete real-world examples
When setting up a discord notification using {{ diff_full }} i was constantly hitting the 2000 charater limit and was not able to quickly identified what changed on the sites I was monitoring.
Additional context
If possible, could a note be added under the table to the file path where the token names are located?

use {{ diff_full }}


Dependabot doesn’t update package dependencies anymore 😒

With PR #208 we moved the package metadata to a declarative approach using the pyproject.toml file. Alas, it seems that Dependabot isn’t able to handle that correctly and so we track their progress in this issue:

https://github.com/dependabot/dependabot-core/issues/4683
https://github.com/dependabot/dependabot-core/issues/3290

@behnazh looks like Dependabot is back again… both referenced issues are merged and:

Closing because Dependabot now works for pyproject.toml files.


disabling of slack group if no one on shift isn't implemented, but in config

redesign flag:

disableSlackHandleTemporaryIfNoneOnShift: true --> optional: default is false

proposal:
new key name: slackHandleNoOneOnShiftStrategy
new values:

disable
if disable, sent a message to job specific infoChannel
nextOnShift

final_schedule - where user is in slack


v0.4.3

This is the final pull request for the provided branch. This post will be updated as I work out what to add in this feature
Planned
Safer external APIs (this means validating tables and types, Fragment tables will be weakly checked, all others will be strictly checked).
Tests, hopefully to finalise the API and publish the first proper release.
Maybe
Logging API, need to work out some API refinements and how to access it.
Unlikely
Sprint Runtime (although detatched from Catwork) wont be started until Catwork has passed its tests.

Update: 09/02/2024:

Added `Flags.DISABLE_NATIVE_SERVICE for runtimes that use their own native service controller


Warning when using pathMatch

Version
4.0.8
Reproduction link
https://jsfiddle.net/cwbe2yq3/
Steps to reproduce
Open JSFiddle
Open console
Click on "Home" link
What is expected?
No warning in console
What is actually happening?
Warning in console:
[Vue Router warn]: Path "/dev" was passed with params but they will be ignored. Use a named route alongside params instead.

Hi, @narwajea
This might be a workaround: use name and params in redirect instead of path.
see this:  https://jsfiddle.net/h8c6gojt/1
const router = createRouter({
  history: createWebHistory(),
  routes: [
  {
    name: "home",
    path: "/:env",
    component: Home,
  },
  { path: "/:pathMatch(.*)*", redirect: {name: "home", params: {env: "dev"}}},
  ],
})

@PeterAlfredLee thank you, it works indeed, as suggested in the warning message. But I think it is hiding the real issue because in that case, no warning should be displayed.

Hi @narwajea .  I think the warning is useful here.
A params can be parsed from the path: "/:pathMatch(.*)*". At the same time, you are using redirect, this will lead to a recursive call of router.resolve. The new call is using pathMatch(.*) as the params and using /dev as path. In this case, a new params can be parsed from /dev, which is different with the existing one "/:env". That's why we got a warning here.
In short words, a route like this:
{ path: "/", redirect: 'dev' }

will not generate any warnings. This is complicated. Hope this helps.
Maybe we can improve the warning message here. Maybe we can use a warning like Path "${(rawLocation as any).path}" was passed with params "${(rawLocation as any).params}" but they will be ignored. Please use a named route alongside params instead. WDYT? @posva


There is a issue with python dotenv restricting Azure Function usage with this SDK

Stability SDK is relying on python-dotenv package which is having or causing issue with its implementation as mentioned in the dotenv github issues here.
It would be helpful to overcome and fix this for the SDK.
Main issue:

Result: Failure Exception: ModuleNotFoundError: No module named 'main' Stack: File "/azure-functions-host/workers/python/3.9/LINUX/X64/azure_functions_worker/dispatcher.py", line 448, in _handle__invocation_request call_result = await self._loop.run_in_executor( File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run result = self.fn(*self.args, **self.kwargs) File "/azure-functions-host/workers/python/3.9/LINUX/X64/azure_functions_worker/dispatcher.py", line 691, in _run_sync_func return ExtensionManager.get_sync_invocation_wrapper(context, File "/azure-functions-host/workers/python/3.9/LINUX/X64/azure_functions_worker/extension.py", line 215, in _raw_invocation_wrapper result = function(**args) File "/home/site/wwwroot/HttpTrigger_jetson_01_test/init.py", line 43, in main container_client = adls.get_container_client(place_name) File "/home/site/wwwroot/adls.py", line 14, in get_container_client dotenv.load_dotenv() File "/home/site/wwwroot/.python_packages/lib/site-packages/dotenv/main.py", line 322, in load_dotenv dotenv_path = find_dotenv() File "/home/site/wwwroot/.python_packages/lib/site-packages/dotenv/main.py", line 275, in find_dotenv if usecwd or _is_interactive() or getattr(sys, 'frozen', False): File "/home/site/wwwroot/.python_packages/lib/site-packages/dotenv/main.py", line 272, in _is_interactive main = import('main', None, None, fromlist=['file'])

Thanks for reporting this! I poked around and I suspect what's going on is a bug in the azure functions python runtime, which is being fixed presently: https://github.com/Azure/azure-functions-python-worker/issues/1094
that said: I'm pretty sure dotenv isn't a critical dependency, so we may address it on our side as well.

@polavishnu4444 #66 may solve this, but I don't have an Azure setup to test - could you check out cnri.dotenv-optional and see if it works in your setup?


[Feature] : Adding dependabot for keeping the dependencies up to date

@roshan1337d I would like to work on this under gssoc'23

Go ahead, assigned this to you 👍


missing a license

every project should have a license, not just to protect yourself and the contributors legally, but also to tell people what they are allowed and not allowed with the project.
keep in mind that every contributor has to accept changes to the license (and thus also the addition of a license) because part of the code belongs to them.

Any license recommendations? Anything speaking against the WTFPL license?

Any license recommendations? Anything speaking against the WTFPL license?

I am a huge fan of the MIT license, I don't think the WTFPL is a really good choise, cause it doesn't state that there is no guarantee and no warranty, it also isn't really professional, is it?

@LeMoonStar @joblo2213 Do you, as current contributors to this project, agree on changing the license of this repository (https://github.com/TrojanerHD/AdventofCode-Bot) to the MIT license as stated in https://choosealicense.com/licenses/mit/?

@LeMoonStar @joblo2213 Do you, as current contributors to this project, agree on changing the license of this repository (https://github.com/TrojanerHD/AdventofCode-Bot) to the MIT license as stated in https://choosealicense.com/licenses/mit/?

yes, I agree to the use of the MIT license for this project.

MIT all the way 💯


[FEATURE] Ukrainian translation

Ukrainian translation (/uk)
I've been starting to translate docs to Ukrainian language. I will be happy everyone helpers)

I can look at this on the weekend
But for now we should use 'UA' as the country code in line with: https://en.wikipedia.org/wiki/List_of_ISO_3166_country_codes

I can look at this on the weekend
But for now we should use 'UA' as the country code in line with: https://en.wikipedia.org/wiki/List_of_ISO_3166_country_codes

To be honest can't agree with you - we talk about language code, not about country code. For example Japan has a jp country code, but Japanese language has a ja code. And Ukraine as a country has a ua code, but Ukrainian language has a uk code.

Ah true
Though it was the same :see_no_evil:

May be useful, or not:

https://www.mkdocs.org/user-guide/configuration/#lang
https://github.com/MihaiValentin/lunr-languages#lunr-languages-----

I can look at this on the weekend

Will try to finish translation of index page till weekends and will be happy your review)

@shidenko97 gentle ping for review:upside_down_face:

@shidenko97 gentle ping for review🙃

Re-check it for typos, please (In first 30 lines I've found 3)
I will try to finish my version today and we can combine it in one perfect version

I've closed #1790 in favour of #1791
@shidenko97 i think it's better to review this way: https://github.com/tiangolo/fastapi/pull/1363#pullrequestreview-421383447
I don't have that much time to go through it again and try to spot issues
I'll try review #1791 this week but no promises:pensive:

How can I contribute the translation? What is the right way?

@mezgoodle Considering creator of #1791 is gone, you can take his pull request and remake it (taking in the reviews from that pull request) and I'll review it. My review might be enough to merge it.

Do I need to create my own pull request?

@mezgoodle Yes, just create a new one.


How to tabulate three body contributions?

I've found the very helpful function to tabulate fitted potentials for pair_style table and got it working with standard lammps.  Now it seems that this function writes only the two body contributions.   The lammps docs lead me to belief that I should also be able to read in the three body contributions if I had them tabulated somehow.  I guess the three body plotting class already does this somehow internally, but I didn't want to poke in there too deeply yet.  Is there a way to quickly get this working or are the more involved problems that I am missing?
For now I suppose I will be able to use the UFCalculator for my testing, so it's not exactly urgent.

Might I suggest using the lammps implementation of UF3 (can be found here) which supports both two-body and three-body interaction? You will need to re-compile lammps but you will be able to read the fitted potentials directly into LAMMPS. We also have a GPU implementation of UF3 in LAMMPS that is much faster than other ML potentials implemented in LAMMPS.

Ah, didn't think to change the branch. Thanks, that should work perfectly!


How to support api versioning with Path Parameter

There is issue with the apis which is containing path parameters. While we are trying to access such apis using hapi-api-versioning its not detecting path parameter.
Due to that its throwing 404 error.
Ex, client is requesting this endpoint /api/user/{userId} (/api/user/62). So its path parameter {userId} is not detected in server.ext('onRequest', ...).
Eventually it's ending up with 404 error. Which is more likely not handled in hapi-api-versioning.
Seeking support. Thank you in Advance

Hello! Can you please provide some more detail about what you are trying to do, stating what the current behavior is, and what the expected behavior is?
It may be helpful to look at this diagram about the Hapi request lifecycle https://futurestud.io/downloads/hapi/request-lifecycle
In onRequest the path parameters haven't been parsed yet, which is why this package uses custom parsing to extract the api version from the path parameter.


nhentai.net "Non-retriable status code 403 while downloading" error code on all new albums.

Ripme version: 1.7.95
Java version: openjdk version "11.0.14" 2022-01-18 - OpenJDK Runtime Environment (build 11.0.14+9-suse-1.1-x8664) - OpenJDK 64-Bit Server VM (build 11.0.14+9-suse-1.1-x8664, mixed mode)
Operating system: OpenSUSE Tumbleweed Version = 20220207
Exact URL you were trying to rip when the problem occurred: https://nhentai.net/g/394674/ "(NSFW)"
Please include any additional information about how to reproduce the problem: Entering the above link in the URL field of the RipMe GUI, and clicking on the Rip button.

Expected Behavior
All the 26 pictures in the album to be downloaded
Actual Behavior
Ripping the album will result in 100% error rate, and no pictures from the album are downloaded. The GUI throws the "Non-retriable status code 403 while downloading" error for each entry in the album. Interestingly this is only applicable to the recently added albums which are showcased under the New Uploads section of the front page of the website. Older albums such as https://nhentai.net/g/765/ "(NSFW)" still download fine with no issues what so ever.

This isn't pretty, but a quick fix:
129c129
<             imageURLs.add(el.attr("data-src").replaceAll("t\.n", "i.n").replaceAll("t\.", "."));

        imageURLs.add(el.attr("data-src").replaceAll("https://t([^<]*)\\.nhentai", "https://i$1.nhentai").replaceAll("t\\.", "."));

Any chance we might be able to see this, or a better, and more permanent fix for this in an update anytime soon?


image size for training is different from that for validation?

Sorry if I miss something, but I wonder why the training data only use image size of 224, but validation data use different image size for different setting of EfficientNet. I did not see this difference in the official tensorflow implementation.

The training code is not yet complete. When complete, it will of course have the same image size as validation.
Training is trickier than it seems, as the original repo uses a number of TensorFlow-specific operations/tools (e.g. model weight EWA). We're working on it, though!


Introduced the printing functions for action, cost, activation, residual, contact, and impulse models

This PR extends the preliminary work proposed in #932.
To improve the printing operator, this PR introduces the virtual function print.
Additionally, the PR includes tests for each operator (i.e., it just runs the operator)
Below, I provide details about how different classes are printed:
Action models
IntegratedActionModelEuler {dt=0.01, DifferentialActionModelContactFwdDynamics {nx=37, ndx=36, nu=12, nc=12}}
ActionModelImpulseFwdDynamics {nx=37, ndx=36, nc=9}

Shooting problem
ShootingProblem (T=108, nx=37, ndx=36, nu_max=12)
  Models:
    0: IntegratedActionModelEuler {dt=0.01, DifferentialActionModelContactFwdDynamics {nx=37, ndx=36, nu=12, nc=12}}
    1: IntegratedActionModelEuler {dt=0.01, DifferentialActionModelContactFwdDynamics {nx=37, ndx=36, nu=12, nc=12}}
    2: IntegratedActionModelEuler {dt=0.01, DifferentialActionModelContactFwdDynamics {nx=37, ndx=36, nu=12, nc=9}}
    3: ActionModelImpulseFwdDynamics {nx=37, ndx=36, nc=9}

Activation models
ActivationModelQuad {nr=3}
ActivationModelQuadraticBarrier {nr=36}

Residual models
ResidualModelCoMPosition {cref=[0.011, 0, 0.46]}
ResidualModelContactFrictionCone {frame=LF_FOOT, mu=0.7}, ActivationModelQuadraticBarrier {nr=5}}}
ResidualModelControl

Stack of cost function
CostModelSum:
  Active:
    LF_FOOT_frictionCone: {w=10, CostModelResidual {ResidualModelContactFrictionCone {frame=LF_FOOT, mu=0.7}, ActivationModelQuadraticBarrier {nr=5}}}
    LH_FOOT_frictionCone: {w=10, CostModelResidual {ResidualModelContactFrictionCone {frame=LH_FOOT, mu=0.7}, ActivationModelQuadraticBarrier {nr=5}}}
    RF_FOOT_frictionCone: {w=10, CostModelResidual {ResidualModelContactFrictionCone {frame=RF_FOOT, mu=0.7}, ActivationModelQuadraticBarrier {nr=5}}}
    RH_FOOT_footTrack: {w=1e+06, CostModelResidual {ResidualModelFrameTranslation {frame=RH_FOOT, tref=[-0.32, -0.2, 0.1]}, ActivationModelQuad {nr=3}}}
    comTrack: {w=1e+06, CostModelResidual {ResidualModelCoMPosition {cref=[0.011, 0, 0.46]}, ActivationModelQuad {nr=3}}}
    ctrlReg: {w=0.1, CostModelResidual {ResidualModelControl, ActivationModelQuad {nr=12}}}
    stateBounds: {w=1000, CostModelResidual {ResidualModelState, ActivationModelQuadraticBarrier {nr=36}}}
    stateReg: {w=10, CostModelResidual {ResidualModelState, ActivationModelQuad {nr=36}}}
  Inactive:

Stack of contacts
ContactModelMultiple:
  Active:
    LF_FOOT_contact: {ContactModel3D {frame=LF_FOOT}}
    LH_FOOT_contact: {ContactModel3D {frame=LH_FOOT}}
    RF_FOOT_contact: {ContactModel3D {frame=RF_FOOT}}
  Inactive:

Stack of impulses
ImpulseModelMultiple:
  Active:
    LF_FOOT_contact: {ImpulseModel3D {frame=LF_FOOT}}
    LH_FOOT_contact: {ImpulseModel3D {frame=LH_FOOT}}
    RF_FOOT_contact: {ImpulseModel3D {frame=RF_FOOT}}
  Inactive:

@jcarpent and @proyan thanks for the feedback. I have defined the print function as a public member. I still keep the same name as I like its simplicity (as other simple names such as calc and calcDiff).
@proyan -- it is indeed a good idea to extend this approach for data structures. if I haven't been too busy I would push it in this PR.


Windows GPU random initialization not working

Using tf.truncated_normal (and possibly the other random functions too) on my GPU (GTX 1060) with the new Windows build produces only zeros whereas running it on my CPU works as expected.
Example code:
with tf.device('/cpu:0'):
    with tf.Session() as sess:
        # Something random, as expected
        print("CPU:", sess.run(tf.truncated_normal([5])))

with tf.device('/gpu:0'):
    with tf.Session() as sess:
        # All zeros
        print("GPU:", sess.run(tf.truncated_normal([5])))

I used the same setup as here with CMake 3.6 and rolled back to https://github.com/tensorflow/tensorflow/commit/41ba1e0e6fbf443f1a972ff0130ba6741b9b7a50 to fix the issue with eigen not compiling.
// Also it seems the result of any calculation involving a tf.Variable becomes 0, while just running / printing the variable itself shows its correct value (CPU works fine though).

@mrry @guschmue for any insight

let me take look. I still have a bunch of gpu fixes - working on a pr for those but not sure of this one is part of it.

Works on my box but I have some idea why its not working for you: on windows there might be some issue that it doesn't handle correctly if the exact cuda version is not compiled in. By default we compile with cuda 3.5+5.2 but a 1060 is on 6.1. I belief on linux it would downgrade you to 5.2 but on windows it does something undefined. Let me debug this a little - should not be to hard to find.
If you need a solution right away: you can edit the CMakefile and add 6.1 to the list of cuda capabilities.

Alright I'll try it tomorrow and let you know if it worked, thanks for your help and great work!

If you change your CMakefile: search for all occurrences of 5.2 and add a 6.1. I should have a better solution by tomorrow.

I added 6.1 everywhere and it worked. I also tried neural style which worked too.

OK, closing this for now. Thanks much @guschmue !

I found the reason why on windows we are not falling back to 5.2 if we find a 6.1 gpu. Going to send a PR so in the future it will behave just like linux.


Adding support for component('button/button.ejs', {}) syntax for EJS

N.B. This PR should preserve the existing syntax too (as legacy) in the ejs-engine
Request from the app team.
They had a few concerns about the existing syntax:
      <%- include('components/input/input.ejs', { ...props });

Their concerns were:

'include', though it conforms to ejs's native syntax, doesn't do what a generic include would do

The 'Component' prefix is not necessary, and too verbose

Here is our proposed syntax:
<%- component('button/button.ejs', { text: props.text, level }); %>

@bglw it won't let me make a PR to patch-1. Could we merge patch-1 to master

@bglw it won't let me make a PR to patch-1. Could we merge patch-1 to master

@bglw Good idea! Will do.

@bglw Good idea! Will do.

Here's the step in ruby to convert it. Path + last section of path + extension
https://github.com/CloudCannon/bookshop/blob/master/plugins/jekyll-bookshop/lib/jekyll-bookshop.rb#L37-L38

Here's the step in ruby to convert it. Path + last section of path + extension
https://github.com/CloudCannon/bookshop/blob/master/plugins/jekyll-bookshop/lib/jekyll-bookshop.rb#L37-L38

@bglw  I think I have implemented this now!: https://github.com/Robbie-Cook/bookshop/commit/3997eaed90ef4adf181eaadf9dd8009d10783c39
Also, I am supporting the old import syntax just in case.

Released as 1.6.2

Released as 1.6.2


Magisk module onboarding for Android

Description
When generating certificate files, a Magisk module is generated as well for easier Android onboarding. To get the certificate accepted system wide on Android it needs to be in the system certificate store, but the /system partition is no longer writable. For rooted phones with Magisk a module is an easy way to circumvent this. The module is generated with the correct key, and is then served in the onboarding add-on.
Small other things:

The tests for the onboarding add-on test more headers
Content-disposition: inline was changed to attachment as we are serving a download.

I do wonder, it seems certificate generation is not automatic for the test suite when testing the onboarding add-on, should create_store() not be run when testing instead of committing the certificates to git? I now just added the Magisk module as well.
Checklist

[x] I have updated tests where applicable.
[x] I have added an entry to the CHANGELOG.

I am not sure why the ubuntu-18 check was cancelled (I think that is what is holding this up?)

Cool, done! I dont know what is going wrong with the coverage?

Ah, it's not just coverage. The Windows tests are failing: https://github.com/mitmproxy/mitmproxy/runs/7975503357?check_suite_focus=true


Casting elements using "as" and correct Typescript type

We are using the property as to mimick styled-components api for casting an element-type to one or the other.
We have used ElementType from react for this but are uncertain now if its the correct type to use as its flagged as "any" value in reality.
Casting components from one element type to another also poses a problem as the props type is not changed/updated to reflect the new set of native attributes a component has. Most notable example of this is when Button is casted from button to a when href is defined. You are unable to set any of the a properties as type is still set to button.
Affected components Button, List, Typography, Breadcrumbs.Breadcrumb

related issue #1308

Based on the testing @oddvernes did in #1897, maybe we should add the polymorphic type in utils for use in other needed components.
Based on this reddit thread, I feel like this implementation was clean:

Types: https://github.com/biswarup35/material-you/blob/v0.1.x/packages/material-meeu/src/types/PolymorphicWithRef.d.ts

Usage in component: https://github.com/biswarup35/material-you/blob/v0.1.x/packages/material-meeu/src/Button/Button.tsx

Will be fixed by #2338


Feature Request: Please add support for Mermaid syntax in Markdown documents

It would be really great if the extension would support Mermaid syntax for Markdown documents (syntax highlight, linting and preview). That would make it much easier to work with diagrams included in Markdown files that are used for Wiki pages that are published through Azure Repos (support for Mermaid has been added to Azure DevOps in Sprint 158).
AB#277639

Not sure that anything but docs.microsoft.com is the target support for the docs authoring extensions. There are a number of open issues that boil down to the same question about supporting other markdowns and publishing systems (ADO, GitHub, mostly)

This is out of scope for the foreseeable future.


[WIP] Support DAG-based conversations

Cyclic conversation models are also supported but cycles are dropped from the navigation stack as soon as they occur

This PR is WIP and is missing tests. Any hints on how to write them are most welcome.

Tests also have a downside that they slow devs down, and today we have to prioritize speed. I think we can allow ourselves to wait until we start hitting real problems because of lack of tests.


Policy Assignment User assigned Managed Identity was not working as expected

Hi,,
Can someone support on the issue while passing the user assigned manage identity in the policy assignement
Issue:

Error: Error in function call
--
04-Jun-2024 23:01:56 |
04-Jun-2024 23:01:56 | on /home/vscode/.terraform.cache/singtel-dev/modules/enterprise_scale/modules/archetypes/locals.policy_assignments.tf line 30, in locals:
04-Jun-2024 23:01:56 | 30:     filepath => jsondecode(templatefile("${local.custom_library_path}/${filepath}", local.template_file_vars))
04-Jun-2024 23:01:56 | ├────────────────
04-Jun-2024 23:01:56 | │ local.custom_library_path is "../../../../platform/components/lib"
04-Jun-2024 23:01:56 |
04-Jun-2024 23:01:56 | │ local.template_file_vars is object with 25 attributes
04-Jun-2024 23:01:56 |
04-Jun-2024 23:01:56 | Call to function "jsondecode" failed: invalid character 'f'.

23:01:56 | Call to function "jsondecode" failed: invalid character 'f'.
This is the issue - it is not valid JSON

Hi Matt,
I have passed like below the policy assignment. Can we have a quick call ?
@.***
Regards,
Madhulika
From: Matt White @.>
Sent: Wednesday, 5 June 2024 1:49 am
To: Azure/terraform-azurerm-caf-enterprise-scale @.>
Cc: Madhulika Sarangam @.>; Author @.>
Subject: Re: [Azure/terraform-azurerm-caf-enterprise-scale] Policy Assignment User assigned Managed Identity was not working as expected (Issue #967)
[CAUTION: External email] Do not click links or open attachments unless you recognize the sender and know the content is safe.

23:01:56 | Call to function "jsondecode" failed: invalid character 'f'.
This is the issue - it is not valid JSON
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/terraform-azurerm-caf-enterprise-scale/issues/967#issuecomment-2148087571, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A6BXQH5EHFK5ABVEJ446KGLZFX4Z5AVCNFSM6AAAAABIYXJ4YSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBYGA4DONJXGE.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Request: docking-station

Name
LooLzzz/docking-station
Website
https://github.com/LooLzzz/docking-station
Official SVG file for icon
https://raw.githubusercontent.com/LooLzzz/docking-station/main/docking-station-app/src/app/icon.svg
Additional Comments
No response

Added in 0.4.1.


Derpibooru 3+ tags search still fails

Bionus, you can probably skip to the TL;DR below, it's a similar problem.
What steps will reproduce the problem?

Add the API_KEY to the password field in derpibooru's options
Input a search with 3 or more tags formatted for proper derpibooru searching.  For example: "faves.gte:250 AND upvotes.gte:350 AND explicit AND animation"
Alternate option, input 3 or more tags formatted for proper derpibooru searching while attempting to for + separation.  For example: "faves.gte:250+AND+upvotes.gte:350+AND+explicit+AND+animation"

What is the expected behavior? What do you get instead?
I expect that Grabber would load the page

https://derpibooru.org/search.json?key=HEkmp3xZorgfgyNJzCNq&page=1&q=faves.gte%3A250+AND+upvotes.gte%3A350+AND+explicit+AND+animation&nocomments=1&nofav=1
on one of those two inputs.  This page has an image list as you'd expect.

What happens instead is that Grabber loads

https://derpibooru.org/search.json?key=HEkmp3xZorgfgyNJzCNq &page=1&q=faves.gte%3A250 AND upvotes.gte%3A350 AND explicit AND animation&nocomments=1&nofav=1

or it loads

https://derpibooru.org/search.json?key=HEkmp3xZorgfgyNJzCNq &page=1&q=faves.gte%3A250%2BAND%2Bupvotes.gte%3A350%2BAND%2Bexplicit%2BAND%2Banimation&nocomments=1&nofav=1
Neither of these pages have any images, so it displays nothing.

Please provide any additional information below
TL;DR: The difference between what it does and should do is that the separate tags should be + separated, but are being space separated!  Trying to manually work around this is impossible since + characters are parsed to %2B.  Changing to + separation at some point for derpibooru should fix this.
As an aside, mainwindow.cpp line 2380, inside mainWindows::loadTag, possibly change from
	if (tag.startsWith("http://"))
to
	if (tag.startsWith("http://") || tag.startsWith("https://")) ?
It wasn't until I read that line that I realized why I couldn't force Grabber to download the correct URL I listed above.  Unfortunately, forcing it like that breaks tag downloading, so not helpful for me.

Actually, I tried single tag searchs and tag

accidental close, apologies

It's working find for me though. Isn't it just because you added a space at the end of your API key? + and   should be interchangeable in encoded URLs, so much that some browsers even edit them out or don't escape them at all.

As an aside, mainwindow.cpp line 2380, inside mainWindows::loadTag, possibly change from
if (tag.startsWith("http://"))
to
if (tag.startsWith("http://") || tag.startsWith("https://")) ?

Done, were tags from HTTPS source not opening on click? 😱

HUGE thank you for the new URL downloader! 👍 I was afraid I'd have to wait another few weeks for this to be solved, but it's allowing me to bypass this problem entirely by making the URLs I need by hand. Thank you thank you thank you.

Yeah, that's exactly why I did it. As well as occasional requests for downloading some source-specific pages, for which I don't really feel like adding UIs yet. I may add a new special "URL" tab though (in addition to the current "tags" and "pool" tabs).

Bump? Looks like your issue was a space at the end of your API key (from what you copy-pasted in your post).
Can I consider the issue solved? :smile:


Add head link to build current development version.

Thanks @mclehman! I merged your commit but squashed it into the previous one to keep the PR clean.


Uploading of annotations results in all frames being keyframes

Annotations dumped in CVAT XML 1.1 for videos and uploaded for a similar task. However after uploading all frames are keyframes even though XML has information on which frames should be keyframes.
This makes it tedious to make changes to a track as each frame has to be edited

@azhavoro , need to investigate. If both tasks are in interpolation mode and dumped annotations for videos it is a bug.

@Hiddenfire21 @nmanovic @azhavoro
Bug is confirmed.


WIP: Add S.raw

Related to #55 a draft.
Cover just String and Object.
todo:

[x] number
[x] Boolean
[x] Integer
[x] Array
[x] integration with AJV (e.g nullable)
~Sanitaze input~
~Evaluate to support string as input~ ( postponed )
[ ] More recursion scenarios
[ ] Coverage

Good work!

🙏 Still, work to do 👻

@jnv and @firasdib can you test it and tell me if it covers your use cases? 🙏

@aboutlo Sure, let me check on Monday. Thank you!

I'll try to find time this week, but it's looking good!


add option to track parameter errors

It is sometimes useful to be able to track parameter errors as well as parameter values. (I also modernized a few loops.)

Not all parameters that can be tracked have errors - eg this functionality can be used to keep track of external model parameters (mass points etc) in the output tree. What would be the interpretation there? I can see users getting confused if its just to use the default of 0.

That's a fair point; I had not thought much about how this would interact with external parameters. A few possibilities:

Modify extArg interface to allow specifying an error value. (This might be complicated.)
Add a note in https://cms-analysis.github.io/HiggsAnalysis-CombinedLimit/part2/settinguptheanalysis/#extra-arguments that the "error" is assumed to be 0.

Either way, I realized I should also add this option in https://cms-analysis.github.io/HiggsAnalysis-CombinedLimit/part3/runningthetool/#common-command-line-options, so I'll update the PR with that.


Add color values and ability to setup with highlight groups

This PR adds colors for the devicons as well as creating highlight groups if the setup function is called.
A couple of things to note:

The colours are largely from github (github/linguist) ultimately but via stealing the json from chadtree.vim where these colors are stored in json.

I made the logic dumber but simpler and I hope less potentially error prone i.e. rather than using a regex to derive the names for a highlight group, I just manually (vim macro) gave each group a name. This means we just check the name and concatenate DevIcon and use that as the name for the highlight group.

I've also added a few more icons.

Example result using api in nvim-bufferline.lua

NOTE: Tried this with nvim-tree.lua and nothing seems to be broken.

Open question
vim-devicons returns a default icon  if there is none for a filetype. I personally think this is more consistent behaviour than returning nil. I can add this here if you agree

thanks, we'll see if it breaks something but i think it should be allright :)


Bug: missing defined lib so no update

We need to check for each version, if the lib is defined in the pom.xml
Because, when the lib is missing, the version is not updated by dependabot
See https://github.com/jhipster/jhipster-lite/blame/main/src/main/resources/generator/dependencies/pom.xml

If I am not mistaken imported boms are also not updated via dependabot (e.g. testcontainers). Or must the version property follow a certain naming schema, e.g. testcontainers-bom.version in this case?

yes, something to do here :D

If I am not mistaken imported boms are also not updated via dependabot (e.g. testcontainers). Or must the version property follow a certain naming schema, e.g. testcontainers-bom.version in this case?

FYI this was due to missing <type>pom</type> in dependency declaration, that was preventing dependabot to find new version (since it's searching a jar):
pdater | 2024/02/27 00:33:31 INFO <job_792541647> Checking if org.testcontainers:testcontainers-bom 1.19.4 needs updating
  proxy | 2024/02/27 00:33:31 [403] GET https://repo.maven.apache.org:443/maven2/org/testcontainers/testcontainers-bom/maven-metadata.xml
  proxy | 2024/02/27 00:33:31 [403] 200 https://repo.maven.apache.org:443/maven2/org/testcontainers/testcontainers-bom/maven-metadata.xml
  proxy | 2024/02/27 00:33:31 [405] HEAD https://repo.maven.apache.org:443/maven2/org/testcontainers/testcontainers-bom/1.19.6/testcontainers-bom-1.19.6.jar
  proxy | 2024/02/27 00:33:31 [405] 404 https://repo.maven.apache.org:443/maven2/org/testcontainers/testcontainers-bom/1.19.6/testcontainers-bom-1.19.6.jar

@pascalgrimaud Bounty claimed: https://opencollective.com/generator-jhipster/expenses/194292


[IOTDB-6030] Improve efficiency of ConfigNode PartitionInfo takeSnapshot

Use BufferedOutputStream(with 32MB buffer) instead of FileInputStream in PartitionInfo
Before Optimization:
2023-06-27 09:52:50,996 [0@group-000000000000-StateMachineUpdater] INFO  o.a.i.c.p.e.ConfigPlanExecutor:489 - [ConfigNodeSnapshot] Finish to take snapshot for org.apache.iotdb.confignode.persistence.partition.PartitionInfo, time consumption: 441904 ms

After Optimization:
2023-06-27 14:25:56,211 [ForkJoinPool.commonPool-worker-115] INFO  o.a.i.c.p.e.ConfigPlanExecutor:490 - [ConfigNodeSnapshot] Finish to take snapshot for org.apache.iotdb.confignode.persistence.partition.PartitionInfo, time consumption: 8330 ms

coverage: 44.561% (-0.006%) from 44.567% when pulling 9f83245ea9abeca8f10a7357530a6cb6cf8f031a on Optimize-ConfigNode-take-snapshot into dc931102e48643f41e5a7b8d8aaf083dc422272a on master.


fix(build): Use correct package name in DKMS config file

Commit d3a3d5bf0f4afcbbbf22fbdcf98b5527e03c18e1 renamed
PACKAGE_NAME to DRIVER_PACKAGE_NAME. While doing so, it missed
the template substitution in driver/dkms.conf.in
Signed-off-by: Grzegorz Nosek<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area driver-kmod

/area driver-ebpf

/area libscap

/area libsinsp

/area tests

/area proposals

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gnosek
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 991deef8fbc90d2bbeea4c7797c65b14a7b8fecf


[FLINK-16629] Ignore Streaming bucketing end-to-end test with Hadoop 2.4.1

What is the purpose of the change
The stream bucketing end to end test does not work on Hadoop 2.4.1 because it is lacking the truncate() method of Hadoop --> We need to exclude it.
Verifying this change
Validated here: https://dev.azure.com/georgeryan1322/Flink/_build/results?buildId=208&view=logs&j=1f3ed471-1849-5d3c-a34c-19792af4ad16

Thank you for the review!


[2020-03-11] master => live

New CA added (Lucas Santos)

Docs Build status updates of commit c228cc8:
:warning: Validation status: warnings

File
Status
Preview URL
Details

advocates/index.html.yml
:warning:Warning
View
Details

advocates/lucas-santos.yml
:white_check_mark:Succeeded
View

advocates/map.yml
:white_check_mark:Succeeded
View

advocates/media/profiles/lucas-santos.png
:white_check_mark:Succeeded
View

advocates/toc.yml
:white_check_mark:Succeeded
View

advocates/index.html.yml

Line 0: [Warning] Invalid file link:(~/tweets).
Line 0: [Warning] Invalid file link:(~/map).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add social icons for fellows

At the moment, it only displays the fellow name and location. To help people network, we should add social icons to each of the fellow cards.

[x] Add LinkedIn icons to the card
[x] Add Twitter icons to the card
[x] Add Resume icons to the card
[x] Add GitHub icons to the card

@grace-omotoso I am working on the issue #3 (adding profile page for each fellow), I thought it also includes creating a card and adding social icons, So I added it there itself. I hope that it's okay with you.

okay, that is fine. I have assigned it to you.

Hey, @grace-omotoso  this issue is resolved in above merged PR. You can close this issue.


Update to 4.2

Hello! Can you update to the latest stable version ? And how can we help you to keep this project always updated?

Hello. I will try to update to v4.2 when i have some free time which should be soon! As for helping, you can contribute by adding new functions, qol features or bug fixes.
Cheers

Sorry, i've been busy with life and havent been able to update to 4.2. I'll maybe finish by the end of this year :sweat:

Sorry, i've been busy with life and havent been able to update to 4.2. I'll maybe finish by the end of this year 😓

I'm glad to hear that!

Hey, seems someone else has made bindings for 4.2 here: https://github.com/rc-05/haxe-raylib
Maybe you can collaborate? I don't see the point of having three different raylib libraries for Haxe :sweat_smile:

That would be nice. If @rc-05 is willing to then sure!

Sorry for not responding earlier, sure we can merge the libraries so that we eliminate repetition

Awesome, should make things easier for everyone.

A pr would be appreciated! (sorry for not replying early, im really busy with school currently :sweat:)

A pr would be appreciated! (sorry for not replying early, im really busy with school currently sweat)

PR done, check #34

It should be version 4.2 rn i have modified and merge #34. if any functions are need and missing, feel free to make another issue. Closing this now


add galley e2e test

This currently tests the configuration validation service. Additional test cases will be added overtime to cover other aspects of Galley.
copy of https://github.com/istio/istio/pull/5047

@ayj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
fb3a161beb35885cd67e26b5fa6408f52ca45d6a
link
/test istio-presubmit

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

@ayj: PR needs rebase.

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

@ayj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
fb3a161beb35885cd67e26b5fa6408f52ca45d6a
link
/test istio-presubmit

prow/istio-unit-tests.sh
0d5291619da1e20ca2a0b731a302b5e3422649e0
link
/test istio-unit-tests

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

@ayj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
0d5291619da1e20ca2a0b731a302b5e3422649e0
link
/test istio-unit-tests

prow/istio-presubmit.sh
0d5291619da1e20ca2a0b731a302b5e3422649e0
link
/test istio-presubmit

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

@ayj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
305fc890980161522fa952ead328432e120b7803
link
/test istio-presubmit

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

@ayj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
2203b989ad66cd251187a9be68aaa48c8a70460e
link
/test istio-presubmit

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

@ayj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
024b43fce4843e19496e2e4da98174ef73da2466
link
/test e2e-bookInfo

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

@ayj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
024b43fce4843e19496e2e4da98174ef73da2466
link
/test e2e-bookInfo

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
024b43fce4843e19496e2e4da98174ef73da2466
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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

@ayj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
024b43fce4843e19496e2e4da98174ef73da2466
link
/test e2e-bookInfo

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
024b43fce4843e19496e2e4da98174ef73da2466
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
024b43fce4843e19496e2e4da98174ef73da2466
link
/test istio-pilot-e2e

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

@ayj: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
024b43fce4843e19496e2e4da98174ef73da2466
link
/test e2e-bookInfo

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
024b43fce4843e19496e2e4da98174ef73da2466
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
024b43fce4843e19496e2e4da98174ef73da2466
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
024b43fce4843e19496e2e4da98174ef73da2466
link
/test e2e-simple

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

/test istio-pilot-e2e

@ayj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
4b4fcdb426892fbf725100afe07ebabb0c1aa929
link
/test istio-pilot-e2e

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

@ayj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
d45a9d64faeab961ec0b51c729df159b900f38c6
link
/test istio-pilot-e2e

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

Unrelated test failures in prow/istio-pilot-e2e.sh
I0604 20:10:22.727] --- FAIL: TestAuthNJwt (116.17s)
I0604 20:10:22.728]     --- PASS: TestAuthNJwt/b->a[200] (0.67s)
I0604 20:10:22.728]     --- PASS: TestAuthNJwt/c->a[200] (0.69s)
I0604 20:10:22.728]     --- PASS: TestAuthNJwt/a->b[200] (0.88s)
I0604 20:10:22.728]     --- PASS: TestAuthNJwt/a->b[200]#01 (0.68s)
I0604 20:10:22.728]     --- PASS: TestAuthNJwt/c->b[200] (0.67s)
I0604 20:10:22.729]     --- PASS: TestAuthNJwt/d->b[200] (0.72s)
I0604 20:10:22.729]     --- PASS: TestAuthNJwt/a->c[200] (0.75s)
I0604 20:10:22.729]     --- PASS: TestAuthNJwt/a->c[401] (0.72s)
I0604 20:10:22.729]     --- PASS: TestAuthNJwt/b->c[401] (0.67s)
I0604 20:10:22.729]     --- PASS: TestAuthNJwt/d->c[200] (0.67s)
I0604 20:10:22.729]     --- FAIL: TestAuthNJwt/a->d[200] (84.81s)
I0604 20:10:22.729]     	pilot_test.go:148: try again
I0604 20:10:22.729]     --- PASS: TestAuthNJwt/b->d[401] (0.68s)
I0604 20:10:22.729]     --- PASS: TestAuthNJwt/c->d[200] (0.73s)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ayj, hklai
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [hklai]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ayj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
d45a9d64faeab961ec0b51c729df159b900f38c6
link
/test istio-pilot-e2e

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


KafkaReceiver does not account for consumer group joining

Description
Following up the discussions in this issue Kafka-produce and consume JSON message, Assert fails for 'consume' operation
 KafkaReceiver currently does not track if the consumer group joining has finished.
The Problem Statement
If the consumer group join takes longer than maxNoOfRetryPollsOrTimeouts * pollingTime the consumer will not process any messages and report an empty record set. This situation also aften expects the pollingTime to be adjusted while running in different hardware configuations such as inside a docker or a standalone Jenkins or laptop etc, which is not a very good user/automation experience.
Solution
AC1 :
A fix could be to modify the receiving part, that the consumer tries to successfully join the consumer group and if that fails err out with an explanation.
AC2:
As explained here

Do a first poll outside of the loop to trigger the consumer group rebalancing
Ignore any runs if there is no assignment for that consumer

i.e. Do not increase maxNoOfRetryPollsOrTimeouts if there is no assignment
i.e. Increase the maxNoOfRetryPollsOrTimeouts only if there was an successful assignment

Available in 1.3.24. Yet to release.

Released.
See Release Notes here.


Would like to enforce HydroBase dependency for datasets

Content Suggestion
It would be good if DMI command files could indicate which version of HydroBase is needed for the dataset, to prevent errors.

I have suggested a new feature for TSTool and StateDMI that would use special comments like:
#@dependsOn

To indicate a dependency on an input file, software version, etc.  This could be done with HydroBase as well.  It involves software enhancements and adopting some new conventions.


[BUG] Cannot set parent: node already has a parent

Describe the bug
We use Liveblocks on a react app with live text input updates. The initial contents of this text input are fetched from a param value in the URL.
Eg.
/play/<text-input-id>
This text-input-id is then used to retrieve associated data from the database. There is also a dropdown input that lets you select valid text-input-id on selecting, we update the route history as well /play/<new-text-input-id>
Everything works as expected, however, there is a strange edge case where Liveblock fails with the following error:
index.js:194 Uncaught Error: Cannot set parent: node already has a parent
    at LiveObject._setParentLink (index.js:194:1)
    at LiveObject.update (index.js:1953:1)
    at LiveObject.set (index.js:1849:1)
    at createOrUpdateRootFromMessage (index.js:2747:1)
    at index.js:3247:1
    at doNotBatchUpdates (index.js:2637:1)
    at WebSocket.onMessage (index.js:3205:1)

To Reproduce
Steps to reproduce the behavior:

Got to /play/<text-input-id>
Inspect that the initial data is loaded correctly.
Change the ID from the dropdown to a new ID. The route is now updated to /play/<new-text-input-id>
Inspect that the initial data is loaded correctly.
Switch back to the old ID. The route is now updated to /play/<text-input-id>
Check the console for the above error.

Expected behavior
The input value should be updated to the old value fetched from the database.
Environment (please complete the following information):

react - v18.2.0
@liveblocks/client - ^0.18.5",
@liveblocks/react - ^0.18.5",

Happy to provide any additional context to help understand the root cause here.

Hello @chaitanyya — thanks for writing up this bug report. That error message is an internal error that should never happen under normal circumstances. We'd be interested in understanding how this code path gets triggered as much as you do.
To that end, do you happen to have a small reproduction of this issue, for example, a minimal example that showcases this bug on, say, Codesandbox, by any chance? Once we have a way to replicate it, it'll be easier for us to take a deeper look. Thank you so much!

Thanks for the update; I'll prepare a minimal example based on our implementation and share it asap.

Hey, I may have found the issue here. What is happening is the room_id changes when the route changes and a new id is provided to <RoomProvider /> component.
Probably a dumb question, but that should not break things, right? Providing a new room id should switch to a new room?
const editorId = useUrlKey(); // This gets updated when the route updates

....

<RoomProvider
  id={editorId}
  initialPresence={{}}
  initialStorage={{
    someData:
      new LiveObject() <
      LivePattern >
      {
        ...sampleData,
      },
  }}
>
  <ClientSideSuspense fallback={<Loader />}>
    {() => <SuspensedWrapper>{children}</SuspensedWrapper>}
  </ClientSideSuspense>
</RoomProvider>;

Probably a dumb question, but that should not break things, right? Providing a new room id should switch to a new room?

 Actually, a good find. This may be something we need to fix indeed. Thanks for bringing it to our attention.
If that theory is correct, you should be able to work around it for now by also passing the roomId to a key prop to the RoomProvider, like so:

<RoomProvider
  key={editorId}  // 👈
  id={editorId}
  ...
>
  ...
</RoomProvider>

Doing so will cause React to unmount-then-remount the new room instead of letting the RoomProvider do the switching.

While working on a CodeSandbox demo, it became clear that the component did not remount with a new id. :)


Wordpress on AppEngine does not list all availble regions

It lists only 4 (e.g. in europe only europe-west1) while AppEngine is available in many more regions.

AppEngine constantly introduce new regions for the cloud services, so perhaps allowing the users to enter the cloud instance connection name instead of offering a selection of regions maybe more practical to maintain the wordpress-helper script.

This should now be fixed: https://github.com/GoogleCloudPlatform/php-tools/blob/master/src/Utils/Project.php#L25


confusing error messaging around query parameter expectations (for a new user)

I was recently working on a fix in an application I didn't develop and am not familiar with wiremock
The fix to our issue was to add a query parameter to a URL that previously didn't have one.
It turned out we had wiremock tests that asserted on the URL and began to fail. The error message provided didn't help me understand the issue and was very confusing (to me).
Versions of things
testCompile "com.github.tomakehurst:wiremock:2.6.0"
testCompile "org.mockito:mockito-core:1.+"
assertJ 3.9.1
junit 4.12
What did I see
On running a test I was given the following output
com.github.tomakehurst.wiremock.client.VerificationException: No requests exactly matched. Most similar request was:  expected:<
GET
/dashboard?registrationFlow=linkAccountSuccess
> but was:<
GET
/dashboard?registrationFlow=linkAccountSuccess
>

It isn't clear from this what has failed. And as I haven't used wiremock or written these tests didn't give me any onward journey.
I can't share our exact code because it's overly complicated. But will try and make time to make a replication of the issue.
In short we had
        stubFor(get(urlPathEqualTo("/dashboard"))
                .willReturn(aResponse()
                        .withStatus(303)
                        .withHeader("Location", redirectUrl)));

and then after the body of the test
WireMock.verify(1, WireMock.getRequestedFor(
          WireMock.urlPathEqualTo("/dashboard")));

When if failed I changed it to
WireMock.verify(1, WireMock.getRequestedFor(
          WireMock.urlPathEqualTo("/dashboard?registrationFlow=linkAccountSuccess")));

But I needed to change it to:
WireMock.verify(1, WireMock.getRequestedFor(
          WireMock.urlPathEqualTo("/dashboard"))
          .withQueryParam("registrationFlow", equalTo("linkAccountSuccess")));

What would have been better
It would have saved me 10 minutes of googling if the test output had been
com.github.tomakehurst.wiremock.client.VerificationException: No requests exactly matched. Most similar request was:  expected:<
GET
/dashboard?registrationFlow=linkAccountSuccess
> but was:<
GET
/dashboard?registrationFlow=linkAccountSuccess
>

It looks like this URL path includes a query parameter. You might want to use withQueryParam in your assertion. See: http://wiremock.org/docs/request-matching/

I can see in this situation that it would be better to a) indicate which URL match operator was in use, and b) show the relevant part of the request URL, rather than always showing the whole thing.
Do you feel like creating a PR to fix?
BTW, 2.6.0 is a pretty ancient version. Although the latest (2.25.1) doesn't improve this particular issue, there are plenty of things it does better, so I'd recommend updating.


OAP-VectorizedRleValuesReader.java

Spark 2.1 & 2.2：
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedRleValuesReader.java
OAP:
src/main/spark2.1/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedRleValuesReader.java
src/main/spark2.2/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedRleValuesReader.java
@@ -37,13 +37,13 @@ import org.apache.spark.sql.execution.vectorized.ColumnVector;
  *  - Definition/Repetition levels
  *  - Dictionary ids.
  */
-public final class VectorizedRleValuesReader extends ValuesReader
+public class VectorizedRleValuesReader extends ValuesReader
     implements VectorizedValuesReader {
   // Current decoding mode. The encoded data contains groups of either run length encoded data
   // (RLE) or bit packed data. Each group contains a header that indicates which group it is and
   // the number of values in the group.
   // More details here: https://github.com/Parquet/parquet-format/blob/master/Encodings.md
-  private enum MODE {
+  protected enum MODE {
     RLE,
     PACKED
   }
@@ -59,13 +59,13 @@ public final class VectorizedRleValuesReader extends ValuesReader
   private BytePacker packer;

   // Current decoding mode and values
-  private MODE mode;
-  private int currentCount;
-  private int currentValue;
+  protected MODE mode;
+  protected int currentCount;
+  protected int currentValue;

   // Buffer of decoded values if the values are PACKED.
-  private int[] currentBuffer = new int[16];
-  private int currentBufferIdx = 0;
+  protected int[] currentBuffer = new int[16];
+  protected int currentBufferIdx = 0;

   // If true, the bit width is fixed. This decoder is used in different places and this also
   // controls if we need to read the bitwidth from the beginning of the data stream.
@@ -582,7 +582,7 @@ public final class VectorizedRleValuesReader extends ValuesReader
   /**
    * Reads the next group.
    */
-  private void readNextGroup()  {
+  protected void readNextGroup()  {
     int header = readUnsignedVarInt();
     this.mode = (header & 1) == 0 ? MODE.RLE : MODE.PACKED;
     switch (mode) {

The code doesn't change from Spark 2.1 to Spark 2.2, this is also why we use tag #Spark API change: 2.1 & 2.2, so we just need to keep a half of this? Also #846

About spark2.3, changes are the same.

About spark2.3.2, changes are the same.