Don’t build certain pages

Expected behavior and actual behavior
Expected not to build certain pages.
Currently get page templates being built as pages even though they are not part of the sitemap structure.
Steps to reproduce the problem (from a clean middleman installation)
I’ve created a custom template for the blog tags (config.rb line blog.tag_template = "content/blog_tag.#{locale}.html") so to create an index page for each of the tags listed in my articles.
In the built version I get a build/blog_tag folder generated along with all the index files to cover the total number of articles and tags. Pages in this case are in mixed order (I can see this from the title tag that shift randomly).
Appending the ignore: true in the frontmatter was in theory supposed to get this page/template being ignored, but this is not the case.
Additional information

Ruby version:  2.5
Middleman version:  4.2.1
OS version: Mac OS X

Can you restate the issue, I can't quite follow.
Where does locale come from?

@tdreyno locale is just a local array I created to contains my languages so I can initialise the blog with a do each. Nothing special.
The issue is with the template tag, contained in the content directory that is being generated.
Should I move that file outside the content directory, perhaps in a template folder?

I believe the file is auto-ignored, but you could try over-specifying the ignore.
ignore "content/blog_tag.#{locale}.html.erb"

So, you're booting up middleman with a specific locale specified? Or it's initializing the blog extension in a loop?

Can you post your config.rb please

# Per-page layout changes
page '/*.xml', layout: false
page '/*.json', layout: false
page '/*.txt', layout: false

###
# Helpers
###
helpers do
  # Additional Helpers method have been defined in the /helpers/ folder so they are
  # automatically registered when Middleman loads
end
###
# End Helpers #
###

# Methods defined in the helpers block are available in templates
set :locales, [:en, :it]

activate :i18n, :mount_at_root => false, :templates_dir => "content", :locales => config[:locales]

activate :autoprefixer do |config|
  config.browsers = ['last 2 versions', 'Explorer >= 9']
end

set :markdown, auto_ids: false
set :markdown_engine, :kramdown
set :markdown, :fenced_code_blocks => true, :smartypants => true

activate :syntax, :line_numbers => true

config[:locales].each_with_index do | locale |
  activate :blog do | blog |
    blog.name = "blog" + locale.to_s;
    #blog.sources = locale.to_s + '/blog/{year}-{month}-{day}-{engtitle}.html'
    blog.prefix = locale.to_s + '/blog'
    blog.permalink = "{category}/:permalink"

    blog.layout = "Blog-inner"
    blog.preserve_locale = true
    blog.summary_separator = /(READMORE)/
    blog.summary_length = 350
    blog.generate_day_pages = false
    blog.generate_month_pages = false
    blog.generate_year_pages = false#

    blog.paginate = true
    blog.per_page = 3
    blog.page_link = (locale.to_s == 'en' ? "page" : "pagina") + "-{num}"

    blog.taglink = "/{tag}/index.html"
    #blog.tag_template = "content/blog_tag.#{locale}.html"
    blog.tag_template = "content/tag.html"
  end
end

#custom options
$blog_comments_depth = 5
$blog_comments_per_page = 5
$bloc_recent_posts = 6

# Directory_indexes has to be activated after the blog
activate :directory_indexes

require "helpers/seo"
activate :seo_support

require 'helpers/readingT'
activate :reading_time

configure :development do
  activate :livereload, :apply_js_live => false, :apply_css_live => false
  set :protocol, "http://"
  set :host, Middleman::LiveReloadExtension.config.host
  #set :port, 80

  set :blog_dir, 'blog'
  set :css_dir, 'css'
  set :js_dir, 'js'
  set :images_dir, 'images'

  ##### BETTER ERROR #####
  require "better_errors"
  require "binding_of_caller"

  set :show_exceptions, true
  config.consider_all_requests_local = true
  use BetterErrors::Middleware
  BetterErrors.application_root = __dir__
  ##### END BETTER ERROR #####

  #activate :asset_hash

  # This is the module created to redirect to
  # the English version of the site for development purposes
  require_relative "helpers/RedirectDevelopment"
  # the file path relative to config.rb w/o .rb extension
  use RedirectRootToDefaultFolder

#  files.watch :source,
#    path: File.expand_path("./source/content/images/_blog"),
#    destination_dir: config[:images_dir]
end

# Build-specific configuration
configure :build do
  set :protocol, "https://"
  set :host, ("www.andreamoro." << (I18n.locale.to_s == "en" ? "co.uk" : "it"))
  #set :port, 80

  set :blog_dir, 'blog'
  set :css_dir, 'css'
  set :js_dir, 'js'
  set :images_dir, 'images'

  ignore '*blog_tag*'

  # For example, change the Compass output style for deployment
  activate :minify_css

  # Minify Javascript on build
  activate :minify_javascript

  # Minify HTML
  activate :minify_html
end

after_build do
  # Rename blog image folders
  config[:locales].each_with_index do | locale |
    RemapImagesDir(locale)
  end

# activate :rsync do |rsync|
#    rsync.production_server = "myapp.com"
#    rsync.staging_server = "staging.myapp.com"
#    rsync.path = "/path/to/middleman/app/on/servers"
#    rsync.user = "rsync_user"#

#    # Optional:
#    rsync.rsync_flags, "--compress --archive --delete -v"
#  end
end

In any case, the ignore function doesn't seem to be respected. As you can see from the image below  middleman sitemap - the file is parsed and a directory with its content is generated.

I had to put the following though, as the line you suggested with the "locale" variable was throwing an exception while the server was starting.
ignore "content/blog_tag.en.html.erb"
ignore "content/blog_tag.it.html.erb"

Wondering if it's actually the i18n extension reading and rendering those tags, not the blog auto-ignore. I don't remember the precedence of the two extensions, but maybe try flipping the order of their initialization. That way the page will be auto-ignored before i18n gets its hands on it.
Maybe an output path ignore for good measure at the end:
ignore "*/blog_tag/*"

@tdreyno thank for the hint.
Moving the i18n declaration under the blog statement works up to a certain degree. Yes the page is no longer shown, but then even the blog content disappear completely ... and in any case the tag pages are not shown :(
Any chance to investigate this a bit?
It will be frustrating creating an helper to delete a directory that should not be created in first instance?

There are a lot of edge cases here. Multiple blogs layered over each other, rather than siblings, localization being half controlled in blog and half in i18n, and the difficulty Middleman has communicating between two separate extensions.
If you, or others, discover the root of this bug, we'll happily merge it.
I acknowledge that this is an annoying bug, but for now, I suggest writing a cleanup script to run after build and remove the incorrect html files. The after_build hook can facilitate this.

Is this something I created somehow. Because I tried to get as close to the guidelines as possible, but had several issues.
If you see my first post on mm it's three years ago. Couldn't get a site published since then cause I can't get what I need :(


[BUG] input_file_name function returns URI encoded paths

Bug
Describe the problem
The input_file_name function returns URI encoded paths. This seems to either be a bug, or an awkward user experience.
Steps to reproduce
CREATE TABLE default.uri_test (part string, y long) USING delta PARTITIONED BY (part);
INSERT INTO default.uri_test VALUES ('a=b', 1);
INSERT INTO default.uri_test VALUES ('a:b', 2);
INSERT INTO default.uri_test VALUES ('a+b', 3);
SELECT *, input_file_name() FROM default.uri_test;

+----+---+-----------------------------------------------------------------------------------------------------------------------+
|part|  y|                                                                                input_file_name()|
+----+---+-----------------------------------------------------------------------------------------------------------------------+
| a+b|  3|    file:/tmp/4949515722967688984/uri_test/part=a+b/part-00000-30050065-24eb-4df0-b43d-363c1cc9416c.c000.snappy.parquet|
| a=b|  1|file:/tmp/4949515722967688984/uri_test/part=a%253Db/part-00000-a3d42cf8-58a1-47c6-b745-21a0666b69b2.c000.snappy.parquet|
| a:b|  2|file:/tmp/4949515722967688984/uri_test/part=a%253Ab/part-00000-9da7c976-38c9-4bfb-bf04-8595f97b7b85.c000.snappy.parquet|
+----+---+-----------------------------------------------------------------------------------------------------------------------+

Observed results
The paths are URI encoded.
Expected results
The expectation is for the paths to be plain URI decoded strings. For instance,
file:/tmp/4949515722967688984/uri_test/part=a%253Db/part-00000-a3d42cf8-58a1-47c6-b745-21a0666b69b2.c000.snappy.parquet should be file:/tmp/4949515722967688984/uri_test/part=a%3Db/part-00000-a3d42cf8-58a1-47c6-b745-21a0666b69b2.c000.snappy.parquet (i.e., %253D should be unescaped to %3D).
Further details

Environment information

Delta Lake version: 2.0.0
Spark version:
Scala version:

Similar behaviour was observed with a recent version of the Databricks runtime.
Willingness to contribute
The Delta Lake Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the Delta Lake code base?

[ ] Yes. I can contribute a fix for this bug independently.
[ ] Yes. I would be willing to contribute a fix for this bug with guidance from the Delta Lake community.
[ ] No. I cannot contribute a bug fix at this time.

@jkylling Thanks for opening the issue. Actually, this is a Spark behavior. Spark's input_file_name() function returns an encoded path. For example, if you try this with Spark's built-in file format such as parquet, you will see the same behavior.

@jkylling Thanks for opening the issue. Actually, this is a Spark behavior. Spark's input_file_name() function returns an encoded path. For example, if you try this with Spark's built-in file format such as parquet, you will see the same behavior.

Indeed, it looks like the string input_file_name() is based on always comes from a Hadoop Path/URI encoded path.
Created SPARK-41480 to clarify the documentation.


Could not find a version that matches with latest version of Mypy and Pylint

Follow up of #2785  (and probably other).
Got the same good-old "resolver issue" again today:

pylint 2.3.0 was released today on Pypi.
pylint 2.3.0 wants astroid>=2.2.0
astroid 2.2.0 want typed-ast>=1.4.0

In my pipfile I also have pylint = "*" and mypy = "*".
And latest version of mypy want typed-ast<1.4.0. So the resolver on pipenv update --clear fails witht the infamous error:
Could not find a version that matches typed-ast<1.3.0,<1.4.0,>=1.3.1

In short: as of today, latest versions of mypy and pylint packages have incompatible dependencies.
Fixing by add this in the Pipfile:
pylint = "<2.3.0"
typed-ast = "<1.4.0,>=1.3.1"
astroid = "<2.2.0"

To prevent update of pylint.
I understand digging into the resolver is a really complex task, and I tried it and failed at doing anything relevant, but still, I hope one good lad here find time to dig into this and find a way to have pipenv finds itself it should not take the latest version of pylint package in this case, and try with a previous version.
Expected behavior:

pylint version is updated, the resolver fails at the end with "Could not find a version" error.
previous versions available on pypi is tried, and so on, until the same version that were on Pipfile.lock. If not present, a depth limit may be set to avoid to try every version of pylint package
even in case of failure, pipenv update --clear should end in success (providing the lock file is still coherent) but add an explicit warning telling 2 packages have incompatible dependency rule in their latest version available on Pypi.

To reproduce in the future, and this in the Pipfile:
[packages]
pylint = "<=2.3.0,>=2.2.0"
mypy = "==0.670"

Expected behavior:

resolver chooses pylint==2.2.2
resolver chooses astroid==2.1.0
resolver chooses mypy==0.670

Bitten by this too. See also https://github.com/pypa/pipenv/issues/3569.

@gsemet
did you try to force to set mypy==0.660 in Pipfile?
I think after mypy pacakge updated in Feb 9, it's happening nowadays.

Yes, freezing version does the job.
My point is that the pipenv's resolver should be able to find out the solution itself. Given the rule pylint = "<=2.3.0,>=2.2.0" and mypy = "==0.670", the solution pylint==2.2.2 is correct.

@gsemet Thanks for the efforts. Due to some limitations, piptools' resolver, which is used by pipenv doesn't have the ability as you said. The pipenv team has a brand new resolver implementation passa which can do this job. It will replace the current resolver in the future.

good :) Tell me if you need help for testing it

To confirm, is the below the official "solution" until passa is incorporated?
[packages]
pylint = "<=2.3.0,>=2.2.0"
mypy = "==0.660"

we have the OK to incorporate passa, but we have a lot of legwork to do first cleaning up other issues.  The issue here is just that piptools doesn't backtrack to resolve conflicts after it makes a decision.  Passa handles that issue

Today a pipenv update failed because it

Could not find a version that matches typed-ast<1.4.0,<1.5.0,>=1.3.1,>=1.4.0

Do I still need to use the following version restrictions, or are there newer versions of these packages I can use?

Since the original issue was resolved, close it now.

yes, it can be closed

Indeed, the current incompatibility is solved, but the pipenv resolver is still not able to handle a similar situation in the future... hope the new version will support it, it is one of the big show stopper that makes people (not me, but some other) in my company not happy when I want to convince them to use pipenv. Even if bare “pip install” actually hides the issue (it does not crash because there is no big resolver, it is the lastest dependency that « wins » the game), at least it works and does not block them in their work

@gsemet Yes, this issue is only one of a large number of similar resolution issues in the backlog.


view_actions() hook, to avoid showing table actions on view pages that don't work

Spotted this on Datasette Cloud:

All those plugins forgot to check if the table was actually a view, and break if you attempt to use them!
Need something in Datasette Core to help plugins not make that mistake.

Some options to add to Datasette:

await db.is_view(name) - to save you from needing to do if name in await db.view_names()
Option: don't show table actions on the view page at all - this would at least prevent the default mistake, it's probably the best way to do it
Could pass an is_view=True parameter to the plugin hook?

Documentation: https://docs.datasette.io/en/latest/plugin_hooks.html#view-actions-datasette-actor-database-view-request

This is wrong:
https://latest.datasette.io/fixtures/simple_view

Should say "View actions", not "Table actions".

... which is weird, because in the tests it seems to have the correct label.

... no, it's working. I hit the demo page before the demo had finished deploying.
Now https://latest.datasette.io/fixtures/simple_view looks like this:


group members inventory marked as success when groups failed

Actual output:

Resource Name
Status
# in Snapshot

organizations
SUCCESS
1

folders
SUCCESS
12

folder_iam_policies
SUCCESS
27

projects
SUCCESS
100

appengine
SUCCESS
31

backend_services
SUCCESS
16

bigquery_datasets
SUCCESS
0

buckets
SUCCESS
74

buckets_acl
SUCCESS
224

cloudsql
SUCCESS
1

firewall_rules
SUCCESS
434

forwarding_rules
SUCCESS
65

instance_group_managers
SUCCESS
25

instance_groups
SUCCESS
41

instance_templates
SUCCESS
34

instances
SUCCESS
167

project_iam_policies
SUCCESS
744

service_accounts
SUCCESS
180

groups
FAILURE
Unknown

group_members
SUCCESS
Unknown

org_iam_policies
SUCCESS
478

@blueandgold can you confirm if this is fixed?


shell: uart: Handle RX ring buffer full case

Scenario where RX ring buffer is full and cannot accept
more data was not handled. In that case byte should be
dropped. Such situation may occur when long command is
pasted (exceeding ring buffer size).
Signed-off-by: Krzysztof Chruscinski<EMAIL_ADDRESS>

Codecov Report

Merging #11399 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #11399   +/-   ##
=======================================
  Coverage   48.37%   48.37%
=======================================
  Files         265      265
  Lines       42188    42188
  Branches    10137    10137
=======================================
  Hits        20408    20408
  Misses      17703    17703
  Partials     4077     4077

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 a7d1ce6...764c269. Read the comment docs.


Better link printing

Wrangler.toml is updated with my own deployed details. You should change it or not commit that to the main repo.

looks promising


Allow DependencyInjection in Normalizer-Methods

This issue is meant more for discussion than for immediate action

If we have an event, that has e.g. a doctrine entity as a property, then it currently is not possible to get access to the data inside that event.
My aggregate only saves the ID to the payload. In my Normalizer I then retrieve that ID and turn that into an actual object. My thinking is, to use ProxyObjects that are going to be resolved, if the aggregate actually needs more than the ID.
Currently I am adding a static function to the entity that creates an entity with just its ID but from there I could actually never repopulate the entity data.
To support Injection, we'd just need a ContainerAware AttributeEventMetadataFactory, that, when calling normalizer() would then also add the necessary properties.
Is this Idea something useful for others?

Could you perhaps provide an example Normalizer? I'm not sure if I understand the problem correctly. Also, could you provide how the event looks like?

As of my understanding what you are doing is to retrieve the data for the event in the normalizer. This should not be done. The normalizer is only there to transform the event into a data format so it can be saved in for example a database.
You should put all the data you need in the event at the point you are creating and dispatching it. If you gather the data delayed changes already could have happened which then leads to an invalid event and thus to an invalid state.
We plan to update the normalizer api with https://github.com/patchlevel/event-sourcing/pull/346 for 3.x of the library.
With these changes it would also be harder to allow DI in the normalizer as we think that there is no usecase for that. But if you can provide a valid usecase we will have a look into it.

Sorry for replying late, and happy late new year
I am using normal entities and event sourced models side-by-side.
Lets say I have a Member Model as a normal entity and some WorkingHistory as an event model. Whenever the API saves something into the WorkingHistory, I do not want to save the entirety of User into the WorkingHistory-Table, that would completely bloat that table and the serialization would take forever.
I do agree, the normalizer might be actually the wrong place to populate the object as it would do that to every event and not just my most current one – but if the member data is only retrieved on access (proxied), then I should have no issue there and can fully interact with the result from my Aggregate or Projection.
This is how it currently looks like:
public function __construct(
        #[Normalize(new WorkingHistoryJsonNormalizer())]
        public readonly WorkingHistoryEntry $entry,
        #[Normalize(new MemberNormalizer())]
        public readonly Member              $member,
    )
    {
    }

public function denormalize(mixed $value): Member|null
    {
        if ($value === null) {
            return null;
        }
        if (is_string($value)) {
            // this just returns an empty Member Object with only a uuid
            return Member::fromUuid(Uuid::fromString($value));
        }
        throw new \Exception('some error');
    }

Sorry for the late reply, was on vacation.
If I'm understanding everything correctly then you are doing here something really wrong. You have a doctrine entity Member which can be updated, these updates are tracked via WorkingHistory which is an Aggregate but itself has no functionality besides tracking events of a general event which is maybe called MemberWasUpdated. This is something we are not recommending to do, since Member and WorkingHistory are itself not atomic anymore and need each other to really have value.
In my opinion you should not do this kind of combination. In this case, I would recommend to either migrate Member to be a Aggregate with meaningful events where the eventstream is then the working history. Or the other way around and migrate the WorkingHistory to a logging concept.
Not allowing DI in the normalizers is also a guard to prevent such kind of misusage. If using the normalizers correctly only for transforming the event into a saveable format there is no need for extra services.
I hope i understood your usage correctly tho :D


Python: AttributeError: 'Kernel' object has no attribute 'create_semantic_function'

Description:
I am trying to add 'create_semantic_function' in my simple chatbot code, but it is showing me an attribute error that 'no attribute create_semantic_function'
Code:
productnamer = kernel.create_semantic_function(prompt, temperature=0.9)
Versions am using:
python==3.11.5
semantic-kernel==0.9.1b1
openai==1.13.3

@Sarfaraz021, please note that since the beta release, create_semantic_function has been replaced with create_function_from_prompt. You can examples here.

Is this function depreciated it?
Can't find create_function_from_prompt from example or source code.

Hi @minkyu-choi07, please refer to our API docs or code samples for the latest methods. Now that we're at v1.0.0+, we won't have any more breaking changes.
Possible methods are:

add_function(...)
add_functions(...)
add_plugin(...)
add_plugins(...)

You can find examples of this throughout our samples.


Add Multi-Language Support Using Language Translator

Implement a language translation API to allow dynamic switching between multiple languages.
Add a language selector (dropdown or buttons) to the UI for easy language switching.

Thank you for the suggestion; this feature isn't required.


Could not launch Sandbox on MacOS Catalina due to Vulkan problem

~/Lumos (dev) $ ./bin/Debug/Sandbox
[02:38:25] LUMOS: Lumos Engine - Version 0.2.2
[02:38:25] LUMOS: Initialised JobSystem with [8 cores] [8 threads]
[02:38:25] LUMOS: Initializing System
[02:38:25] LUMOS: Creating window - Title : Sandbox, Width : 1280, Height : 800
[02:38:26] LUMOS: Initialised GLFW version : 3.3.0 Cocoa NSGL EGL OSMesa
[02:38:26] LUMOS: Battery Info - Percentage : 94 , Time Left 9540s , State : On Battery
[02:38:26] LUMOS: Vulkan : 1.0.108
[02:38:26] LUMOS: GPU : AMD Radeon R9 M370X
[02:38:26] LUMOS: Vendor ID : 4098
[02:38:26] LUMOS: Device Type : DISCRETE GPU
[02:38:26] LUMOS: Driver Version : 0.2.1843
[mvk-error] VK_ERROR_FEATURE_NOT_PRESENT: vkCreateDevice(): Requested feature is not available on this device.
[02:38:26] LUMOS: [VULKAN] VKDevice::Instance()->GetDevice() vkCreateDevice() failed!
Segmentation fault: 11

But all tests passed:
~/Lumos (dev) $ ./bin/Debug/Tests
[02:39:11] LUMOS: Lumos Engine - Version 0.2.2
[02:39:11] LUMOS: Initialised JobSystem with [8 cores] [8 threads]
[02:39:11] LUMOS: Initializing System
[02:39:11] APP: Vector2 Test Passed
[02:39:11] APP: Vector3 Test Passed
[02:39:11] APP: Vector4 Test Passed
[02:39:11] APP: Reference Test Passed
===============================================================================
All tests passed (259 assertions in 5 test cases)

Reproduced on master and dev branch

Think it's to do with the Vulkan version. Works for me and I'm using 1.0.85. Got this last time I tried to update it. I'll try again to a more update to a more recent version.

Seems like on newer Vulkan/Moltenvk versions commenting out 			//deviceFeatures.shaderCullDistance = VK_TRUE;
Before vkCreateDevice fixes the issue.

 See previous logged error.
[20:30:58] LUMOS: [SceneManager] - Scene switched to : Physics Scene
Segmentation fault: 11

Looks like it is my hardware/MoltenVK problem. Anyway, thank you for answers.

Hmm strange the shader not compiling on a different moltenvk version. I've submitted a potential fix.

Fix helped! Thank you.


Launch application at boot time

http://stackoverflow.com/questions/10428510/how-to-start-launch-application-at-boot-time-android

http://stackoverflow.com/a/10428665/354173

Thx
On Tuesday, 26 January 2016, Jo Grimstad<EMAIL_ADDRESS>wrote:

http://stackoverflow.com/a/10428665/354173
—
Reply to this email directly or view it on GitHub
https://github.com/literacyapp-org/literacyapp-android/issues/20#issuecomment-174962118
.


Modal fills screen

When I double click to edit a text box, the ckeditor modal pops up as intended however it fills the screen completely to the point where it is unusable. i.e. the save buttons are off the window and I have to reduce my font size to even see the edge. Any ideas? Perhaps something I can add to CKEDITOR_SETTINGS?

The issue was being caused by the menu being shown in the base template.
<ul> {% show_menu 0 1 100 100 %} </ul>

@zsoobhan  I have encountered this issue as well, How did you fix it ?

@bxm156 this happened so long ago and my comment is somehow less than helpful. I no longer have access to the code base but if memory serves me right, the problem was with the show_menu flag being used twice. Once in the base_template and once in the inherited one.
Suggestion:
Try removing the show_menu flag in all the templates being used and see if that fixes the issue.
If that works, please document it in this thread (better than I did).

@bxm156 this happened so long ago and my comment is somehow less than helpful. I no longer have access to the code base but if memory serves me right, the problem was with the show_menu flag being used twice. Once in the base_template and once in the inherited one.
Suggestion:
Try removing the show_menu flag in all the templates being used and see if that fixes the issue.
If that works, please document it in this thread (better than I did).


Some revisions are not converted to links to GH commits

For example:

https://github.com/pjsip/testproject12/issues/1516

Needs to look into difference of for example what's different  #2173 and #1516

One difference is the successful ones have leading "r" before the ref number

@bennylp Not really, #2173 both are converted successfully
r5928
5930

Problem caused by incomplete revmap file - current revmap file only from trunk. Need to generate further revmaps for all branches

New revmap generated


get_or_create method

Correct me if there's a better way, but I found myself copying and pasting the following code (or something similar) over and over again :
# Given the strings `database_id` and `collection_id` and a relevant `client` (instance of `DocumentClient`)
try:
    database = next(data for data in client.ReadDatabases() if data['id'] == database_id)
except StopIteration:
    database = client.CreateDatabase({'id': database_id})
database_link = database['_self']
try:
    collection = next(data for data in client.ReadCollections(database_link) if data['id'] == collection_id)
except StopIteration:
    collection = client.CreateCollection(database_link, {'id': collection_id})

So, how about a get_or_create_sometypemethod in the DocumentClient class of pydocumentdb/document_client.py, like the one in Django QuerySet class? Then, the code above would be much shorter and more readable like this:
# Given the strings database_id and `collection_id` and a relevant `client` (instance of `DocumentClient`)
database = client.get_or_create_database(database_id)
database_link = database['_self']
collection = client.get_or_create_collection(database_link, collection_id)

p.s. The link to guidelines for contributing to this repository is broken.

I fixed the link to contribution guidelines. Feel free to submit a PR for this or else we would add it to our backlog and get to it in near future.

Closing this issue due to age. Active development of our Python SDK has moved to https://github.com/Azure/azure-sdk-for-python. We have also released 4.0 which includes many updates and new features. If this issue still needs our attention, please reopen it in the new repository.


Fix CMake Warnings for Postgres client libs [issue#595]

Refer to https://github.com/cmu-db/peloton/issues/595#issuecomment-291380166

@CHUYilei @jarulraj Travis acts weird sometimes. I restarted the build and it seems to be going.

Coverage increased (+0.01%) to 67.782% when pulling 44d1c93133f65790a3d4e2b472cd202a9ae646c3 on hanli32:test into dd8fe018fc12c7ee2dc964ab72407c6f66df7d9d on cmu-db:master.


Async world (un)loading

All platforms block the async (un)loading of worlds and chunks. To prevent developers from having to access the main thread manually when (un)loading targets, make sure these implementations are automatically transferred to the main thread.

Possible with current core API.


RAC-5431 vmslave booking && recovery

add vmslave booking recovery system to keep sandbox vmslave safety.
docs: https://rackhd.atlassian.net/wiki/spaces/RAC1/pages/23101688/SH+Jenkins+Sandbox
@panpan0000 @PengTian0

BUILD on-build-config #605 : FAILURE

@changev , I believe you are doing RAC-5431. please add this in PR title.

@changev , Wrong rebase ....
your PR contains 2 of my commit..

General comments:
your solution sounds workable if people follow the process to book/unbook the vmslave if he/she is aware of the impact of change he/she will make.
but how about this situation ?  someone create a script/pipeline which has destructive impact to vmslaves  but he/she is not well aware of that, so he/she didn't follow the book/unbook procedure.
example , "change network setting in a new script" .  then the impact to the vmslave will persist .
following your idea, I was thinking of taking snapshot after vmslave being configured/re-config, and this/those snapshot(s) can be used to restored if a destructive impact happens ? ( the negative consequence will be the vmslave will lost the workspace data it had)

@panpan0000 Thanks for your advice.
Keep a resident snapshot of every vm is good for security. But it increase the spending.
Every maintainer(now me) must know this external infomation, they must delete the old resident  snapshot and create a new one(maybe use a jenkins job) after every time vmslave configurating.
This is a long time work.
In other words, "don't follow the book/unbook procedure and destroy a vmslave" may be rare case.
However, IMO this is a cost/benefit tradeoff. Both is OK to me.

Ok.
there's 2 cases:
(1) in someone's PR, the modification of CI scripts will do destructive impact to vmslave. and when this PR goes into PR Gate, the harm will be done.
(2) people books a vmslave, and do any change, then some automation helps him to restore the vmslave .
@changev's PR address the point 2.
point (1) is hard to do. will not be in short term scope.


Spike: Ability for enrolling app to know which apps can approve new enrollments

Is your feature request related to a problem? Please describe.

Currently apkam approvals happen through already enrolled apps which have enrollment widget implemented.
There are few drawbacks of this approach

User needs latest app version which has approval feature
User has to remember previously enrolled apps inorder to approve new enrollment request

Describe the solution you'd like

Enrollment listing and approval feature from registrar/admin web application.
Explore design and implementation approach in this ticket

Describe alternatives you've considered
No response
Additional context
No response

@murali-shris I've had some conversation with @cconstab about this over the weekend. We should discuss again in architecture call, ideally tomorrow (Tuesday)

These are the possible solutions we have discussed so far

Store enrollment details in public hidden key. Enrolling app will lookup this public hidden key to view list of apps that has approving privilege.

On enrollment approval, store enrollment details in a public hidden key.
On enrollment revoke, remove the enrollment entry from the value using enrollmentID
e.g
Key : public:_enrollments@alice
Value:  // each value in list will contain enrollment id, app name , device name
Pros:

Simple to implement
User doesn’t have to remember already enrolled/onboarded apps

Cons:

Internal enrollment details exposed via a public key
If there are no prior approved enrollments, then this key will be empty. This scenario will arise when an approving app has not updated to latest version which has enrollment widget functionality.

Approve enrollments through registrar web application

Use the existing registrar web application (my.atsign.com) to view/approve/revoke enrollments. This requires use of an intermediary secondary server @registrar to notify the web application and communicate with @alice
Pros:

Single web application to control enrollments, onboarded apps, etc.
User doesn’t have to remember already enrolled/onboarded apps

Cons:

Complex to implement. Have to introduce @registrar secondary. Changes  in client and server code
Passing of encryption keys required for new enrollments through @registrar is complicated.
Additional entry point to access @alice secondary

An existing onboarded/enrolled app with enrollment widget uptake can approve enrollments

After user enrolls a new app, display a message to user e.g “Use one of the onboarded/enrolled apps”
Pros:

Simple to implement.App that needs to approve enrollments has to uptake enrollment widget

Cons

User has to remember already enrolled/onboarded apps
Already  onboarded app may not have updated to latest version and cannot see enrollment notifications
Already enrolled app, may not have sufficient privilege to approve new enrollment

Headless app that auto approves enrollment

#TODO

New admin mobile app

#TODO

Another idea...
Store the list of M/APKAM devices+apps in a self-key
Then if someone does not know what device to approve the enrollement in then any atKeys for the atSign can be used to share that list.
This reduces the attack surface to just devices/apps that have any set of keys for the atSign and gives the person some good clues as to which app/keys they can use to approve the M/APKAM request with.
We could also add this functionality to the CLI tools...
Pros:

all done in atSign space and should be simple

Cons

Perhaps some cognitive load for people but also lots of flexibility.

Discussion from arch call:
Consider combination of approach 6 and additional atsign manager app

Moving to backlog since it is not actively worked on. Will revisit once APKAM widget changes are complete


Minor compilation fixes

Firstly, observe that aws-creds doesn’t compile without default
features enabled.  This is because Credentials::refresh method which
calls Credentials::default which is only defined if http-credentials
feature is enabled.  Change the code (and related tests) so that the
former method is present only if that feature is enabled.
Note that even with that, rust-s3 doesn’t compile without default
features enabled.  This is because rust-s3 depends on aws-creds with
the above bug present.  This will be fixed once the dependency is
updated.
Secondly, fix minor compilation warnings caused by conditional
compilation leading to some dead code.

@mina86, thanks for raising, fixed in 0.33


CASMTRIAGE-6649 CASMTRIAGE-6778 CASMTRIAGE-6596 1.5 Net docs

Description
CASMTRIAGE-6649
CASMTRIAGE-6778
CASMTRIAGE-6596

Checklist

[ ] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[ ] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[ ] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

backport release/1.5

/backport release/1.5


dora - more intuitive tags multi-select

The default django's multiselect widget for the entry forms is not intuitive and looks pretty 1990's.
Can there be a more elegant way of multi selecting tags?
perhaps: https://medium.com/@alfarhanzahedi/customizing-modelmultiplechoicefield-in-a-django-form-96e3ae7e1a07

Use this: https://developer.snapappointments.com/bootstrap-select/


Use a link validator for markdown docs

Verification

[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

Provide a detailed description of the proposed feature
I've noticed some invalid links within the documentation (i'm about to submit a merge request for some links), however invalid links can easiely be prevented by adding a link validator such as https://github.com/marketplace/actions/markdown-link-check
What is the motivation for the feature?
Prevent invalid links getting into documentation
How will the feature be relevant to at least 90% of Homebrew users?
It affects the integrity of all documentation
What alternatives to the feature have been considered?
I haven't seen any proposed alternatives by others, nor can i think of a better solution than the one i suggested. Open to suggetions.

Thoughts on adding this as a local command, maybe brew doc-check-links or brew style --links or similar? An alternative I thought about is adding a CI that runs on edits to the documentation, but it would be cumbersome for someone to have to fix links in unrelated parts. And it wouldn’t be that useful if it only checks the added lines.
It would be easier if there’s already Homebrew formula that checks link rot, and have the dev command (something similarly to how brew style uses shellcheck). I don’t have a specific tool in mind though.

We already have html-proofer (bundle exec rake test) but it seems it's not run in the existing docs workflow for some reason.

I tested using markdown-link-check as a job just to see how many broken links we're dealing with, the majority are broken due to sharding issues (the rest were false positives since i didn't fine tune the link test parameters)
Looks like https://github.com/Homebrew/brew/pull/15891 , https://github.com/Homebrew/brew/pull/15904 will take care of those sharding link issues.

The only broken internal link is https://github.com/Homebrew/brew/blob/3c8b4949baefb1f8166749ff1a3f0665afadedda/docs/How-To-Organize-AGM.md?plain=1#L147

The only external broken link is https://github.com/Homebrew/brew/blob/3c8b4949baefb1f8166749ff1a3f0665afadedda/docs/Formula-Cookbook.md?plain=1#L796C6

I'm not familiar with html-proofer but since it's already implemented maybe this is best to continue using. Since external links can change sporadically, maybe these should be non-failing events within CI since it could interfere with future merge requests for unrelated issues.

I'm not familiar with html-proofer but since it's already implemented maybe this is best to continue using.

Yes, this would be best to use.

maybe these should be non-failing within CI since it could interfere with future merge requests for unrelated issues

I think it's still worth failing here so they can be addressed rather than ignored.

Thoughts on adding this as a local command, maybe brew doc-check-links or brew style --links or similar?

Yeh, I'd rather we didn't add a new command for this.


.region() as Region, .regionName() as String; also PIP and AvailabilitySet interface restructuring

this way, resource.region() will return the more intuitive/discoverable strongly typed Region enum constant, but in case someone cares about handling a future region that's not currently in the constant, they can fall back on .regionName().
With the latter point in mind, this change does NOT change the internal use of region, i.e. I first renamed previously existing .region() to .regionName() and did not change the automatically refactored existing impls to refer to the new .region(), but kept them as referring to .regionName().

Can one of the admins verify this patch?

backing out for now, as Anu raised an interesting question about the organization approach to these interfaces


TPC-C benchmarking results

In the Performance Benchmarking with TPC-C article there is no recommendations about what to do if result is unsatisfactory.
And what would such result mean in principle. Сan someone please clarify this question?

@rmloveland, assigning to you since you updated these docs for 20.2.


bug

You can't move Windows when you open the app, but you can move Windows when you open Settings

good

good


Not working on MSI Computer X99A GAMING PRO CARBON A

Compile from source, when I ran the examples the following was returned:
./msi-rgb 206487a9 206487a9 10325476 -ir -ig -ib -d 5
error: could not set the colour
caused by: The sI/O chip identifies as c911, which does not seem to be NCT6795D
caused by: --ignore-check flag, which would skip the check, is not specified (may be dangerous); also try `--base-port
Changed the base port to 2e as suggested in the help page, the following was returned:
./msi-rgb 206487a9 206487a9 10325476 -ir -ig -ib -d 5 --base-port 2e
error: could not set the colour
caused by: The sI/O chip identifies as ffff, which does not seem to be NCT6795D
caused by: --ignore-check flag, which would skip the check, is not specified (may be dangerous); also try --base-port

As the error message says:

caused by: The sI/O chip identifies as c911, which does not seem to be NCT6795D

If a visual inspection of the board reveals presence of a NCT6795D chip, please re-open the issue.


adding updated TF example using NGC and latest MPI Operator

resolves issue #77
Signed-off-by: John Lockman<EMAIL_ADDRESS>

@j0hnL can we rename the file? Looks like it's called k8s-TensorFlow-NvidiaNGC-esnet50-multinode-MPIOperator.yaml.
Should be: k8s-TensorFlow-NvidiaNGC-resnet50-multinode-MPIOperator.yaml or perhaps all lowercase:  k8s-tensorflow-nvidia-ngc-resnet50-multinode-mpioperator.yaml

updated to all lowercase


binstar-token invalid

Currently circle-ci builds are failing with the following message:
Traceback (most recent call last):
  File "/opt/conda/lib/python3.6/site-packages/binstar_client/commands/upload.py", line 133, in add_release
    aserver_api.release(username, package_name, version)
  File "/opt/conda/lib/python3.6/site-packages/binstar_client/__init__.py", line 377, in release
    self._check_response(res)
  File "/opt/conda/lib/python3.6/site-packages/binstar_client/__init__.py", line 216, in _check_response
    raise ErrCls(msg, res.status_code)
binstar_client.errors.NotFound: ("release version=u'0.2.5' does not exist", 404)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/conda/bin/anaconda", line 6, in <module>
    sys.exit(binstar_client.scripts.cli.main())
  File "/opt/conda/lib/python3.6/site-packages/binstar_client/scripts/cli.py", line 88, in main
    description=__doc__, version=version)
  File "/opt/conda/lib/python3.6/site-packages/binstar_client/scripts/cli.py", line 66, in binstar_main
    return args.main(args)
  File "/opt/conda/lib/python3.6/site-packages/binstar_client/commands/upload.py", line 242, in main
    args=args)
  File "/opt/conda/lib/python3.6/site-packages/binstar_client/commands/upload.py", line 180, in upload_package
    add_release(aserver_api, args, username, package_name, version, release_attrs)
  File "/opt/conda/lib/python3.6/site-packages/binstar_client/commands/upload.py", line 138, in add_release
    create_release(aserver_api, username, package_name, version, release_attrs)
  File "/opt/conda/lib/python3.6/site-packages/binstar_client/commands/upload.py", line 42, in create_release
    aserver_api.add_release(username, package_name, version, [], announce, release_attrs)
  File "/opt/conda/lib/python3.6/site-packages/binstar_client/__init__.py", line 417, in add_release
    self._check_response(res)
  File "/opt/conda/lib/python3.6/site-packages/binstar_client/__init__.py", line 216, in _check_response
    raise ErrCls(msg, res.status_code)
binstar_client.errors.Unauthorized: ('Authorization token is no longer valid', 401)
Traceback (most recent call last):
  File "/opt/conda/bin/upload_or_check_non_existence", line 122, in <module>
    main()
  File "/opt/conda/bin/upload_or_check_non_existence", line 111, in main
    upload(cli, meta, owner, channel)
  File "/opt/conda/bin/upload_or_check_non_existence", line 62, in upload
    env=os.environ)
  File "/opt/conda/lib/python3.6/subprocess.py", line 291, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['anaconda', '--quiet', '-t', 'binstar.token', 'upload', '/feedstock_root/build_artefacts/linux-64/eqcorrscan-0.2.5-np111py27_0.tar.bz2', '--user=conda-forge', '--channel=main']' returned non-zero exit status 1.

./ci_support/run_docker_build.sh returned exit code 1

This is the same for travis (and likely for appveyor).
I think this is related to this issue.
This release is not totally urgent as we have released via pypi, but it would be good to get it updated @conda-forge/core

The token has been revoked. Please see this comment for details.

The tokens got automatically updated; if you want to release 0.2.5, you can merge a PR with an empty commit git commit --allow-empty -m "empty commit to force build", or you can go ahead and just release 0.2.6.

Released 0.2.6. Thanks guys!


Change the Common project file to be SDK Style

Converted it using the Project2015To2017.Migrate2019.Tool
Command line: dotnet migrate-2019 wizard
More information about SDK style projects at https://docs.microsoft.com/en-us/dotnet/core/project-sdk/msbuild-props
This resolves a part of #116 and is a step on the way to .NET 5. There are at least two other project files that needs to be converted but starting with this one.

:x: Build ServiceBusExplorer 5.1.0-common-proj-to-sdk-style.37.build.850 failed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/f0bc6855bd by @ErikMogensen)

:white_check_mark: Build ServiceBusExplorer 5.1.0-common-proj-to-sdk-style.38.build.851 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/155eb87a07 by @ErikMogensen)

@ErikMogensen @SeanFeldman approved 😃


Initial implementation of friendlier errors

I'll add a proper description later, this PR is just for testing the CI steps.
Related: #2107
TODO:

[x] assert that documentation exists (via pytest)
[x] make all of this optional (flag and checking for imports)
[ ] Don't fail bad if ast parsing fails (for example with dynamic code)
[ ] don't add libcst as main dependency
[ ] redirect from errors.strawberry.rocks

Future release:

[ ] Errors in wasm (if libcst works)

Maybe:

[ ] custom flake8 plugin to disable pytest.raises with StrawberryExceptions

/pre-release

Pre-release
:wave:
Releasing commit [1c694722e2c8ac628ec8fb603029a1adf0c67aa2] to PyPi as pre-release! :package:

Pre-release
:wave:
Pre-release 0.127.0.dev.1661177678 [1c694722e2c8ac628ec8fb603029a1adf0c67aa2] has been released on PyPi! :rocket:
You can try it by doing:
poetry add strawberry-graphql==0.127.0.dev.1661177678

/pre-release

/pre-release

/pre-release

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

This release...

Here's the preview release card for twitter:

Here's the tweet text:
🆕 Release (next) is out! Thanks to @patrick91 for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)

/pre-release

/pre-release

/pre-release

/pre-release

/pre-release

/pre-release

/pre-release

/pre-release

from discord:
we should make this error better, if not already done here:
import strawberry
from strawberry.scalars import JSON
strawberry.union(
    name='NamedUnion',
    types=(
        # some other types
        JSON,
    ),
)

AttributeError: 'ScalarWrapper' object has no attribute 'name'. Did you mean: 'ne'.

/pre-release

/pre-release

/pre-release

/pre-release


Error when using "Returning Into" differs from when it is not been used

The syntax without "returning" returns the constraint error, which is the correct problem with the insert.
stmt, _ := conn.Prepare(`INSERT INTO foo ( name ) VALUES (:1)`)
if _, err := stmt.Exec("boo"); err != nil {
        fmt.Println(err) // ORA-00001: unique constraint ( uq_foo ) violated
`}`

The syntax with "returning" returns a TCC error.
stmt, _ := conn.Prepare(`INSERT INTO foo ( name ) VALUES (:1) RETURNING id INTO :2`)
var id int64
if _, err := stmt.Exec("boo",  sql.Out{Dest: &id}); err != nil {
        fmt.Println(err) // TCC error: received code 19 during response reading
`}`

I have no idea what this TCC error means, but it looks like it is trying to take the ID without checking if the insert was successful, I didn't check the code it is just my own suspicion, I will try to check how the code does to take the last ID and see what is happening over there.

this is because insertion error
no problem. would you share trace log to fix this issue?

I sent you the logs by email.

ok I will try to simulate the condition and find a solution

fixed in last release v2.5.32


Hubert finetuning hyperparameters

❓ Questions and Help
Hi, I try to finetune the released pretrained Hubert model with the config examples/hubert/config/finetune/base_10h.yaml and then decoding with 4-gram LM, but I cannot reproduce the number in the paper.
Could you release the finetuning hyperparameters to reproduce the reported results in https://arxiv.org/abs/2106.07447?
Thanks very much.

I met the same problem recently. Did you reproduce the number eventually?


Remove Membership Factory

What changed?
membership.Monitor is used instead of MembershipMonitorFactory

Why?
To simplify upcoming changes for pluggable membership options.

How did you test it?
Unit tests/local test

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 755450f2-768b-4a78-95c7-5c50a425a628

17 of 44 (38.64%)  changed or added relevant lines in 7 files are covered.
80 unchanged lines in 17 files lost coverage.
Overall coverage decreased (-0.02%) to 56.911%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

common/membership/rpServiceResolver.go
0
1
0.0%

cmd/server/cadence/server.go
0
2
0.0%

common/membership/rpMonitor.go
8
13
61.54%

common/config/ringpop.go
0
19
0.0%

Files with Coverage Reduction
New Missed Lines
%

common/persistence/sql/sqlExecutionStore.go
1
62.69%

common/task/weightedRoundRobinTaskScheduler.go
1
89.64%

service/history/execution/mutable_state_task_refresher.go
1
73.82%

client/history/client.go
2
40.3%

client/history/metricClient.go
2
45.94%

common/persistence/executionManager.go
2
76.57%

common/persistence/statsComputer.go
2
96.43%

service/history/handler.go
2
48.42%

common/cache/lru.go
3
90.73%

service/matching/matcher.go
3
91.06%

Totals

Change from base Build fc6b58f8-7517-4e22-ad79-b3b4ec4f0150:
-0.02%

Covered Lines:
82339

Relevant Lines:
144680

💛  - Coveralls


[ini] Update ini parsing and format

Update the parsing and format of ini input file
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #240. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/240-202006261441340.61521196365356-f13f62f749bd49bbbac7fd0e3d450a965538e968/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/240-202006261727390.1798210144043-b4aaedc189ceb0d09f9d0fbb254763bd3882a7cf/.

Could you elaborate As-is / To-be of this PR if possible?
Thanks!

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/240-202006301241000.86276292800903-1d5d16b0618ee3a74fc9bf5721d5c48bdb47d084/.


Streamlink integration.

Closes #118

It may not be possible to do this with pyO3, as packages aren't really supported, as far as I can tell.

Closing until a better solution is found


refactor: standardize file arrays used in check functions

@David-Pena I need your input on this
So all of our checks builds an array containing some objects
For example
cyclomaticComplexityFiles: { fileName: string, cyclomaticComplexity: number }[]
deepIndentationFiles: {filePath: string, message: string}[]
elseConditionFiles: { fileName: string, elseCount: number }[]
functionSizeFiles: {filename: string, funcName: string}[]
magicNumberFiles: { filePath: string, message: string }[]
parameterCountFiles: { filename: string, funcName: string, paramsCount: number }[]
plainScriptFiles: string[]
longScriptFiles: { fileName: string, scriptLength: number }[]
shortVariableNameFile: {filename: string, variable: string}[]
tooManyPropsFiles: { fileName: string, propsCount: number }[]
elementSelectorsWithScopedFiles: { filename: string, selector: string}[]
implicitParentChildCommunicationFiles: { filename: string, message: string }[]

It is messy :vomiting_face:
The purpose of these file arrays is only to build up the offenses array what is typed nicely.
The offenses objects uses the filename (what should be called filePath BTW), and the message from these file arrays.
So my suggestion is to have filePath: string, message: string in these file arrays.
If we go like this all message coloring can happen in the file arrays, and offenses just use them as they are in the message, apending the 🚨 emoji.
For example implicitParentChildCommunication already doing this.

Agreed 🤝
Yesterday updating the tests I realized how different we have that interface. So filePath: string, message: string  sounds good to me

Great.
I roll up my sleeves

needs some brain:

[ ] directiveShorthands
[ ] simpleComputed


There are no attributes for 'eduPerson' Object Class defined in Embedded LDAP

Requirement
A set of new attributes are needed to use in embedded LDAP for the feature of InCommon Federation Compliance for WSO2 IS. Those attributes are defined under eduPerson Object Class Specification. The existing LDAP doesn't support this object class but, it is required to be supported.

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


In Selenium 3.8.1 PhantomJS marked as deprecated webdriver

Hi,
I am very sorry that writing to issues instead another place (please moved to right place or close the issue).
I like PhantomJS and used it with Selenium as webdriver but since Selenium 3.8.1 PhantomJS is deprecated and recommend us to use either Chrome or Firefox in headless mode.
Is it OK? What do you think about it? Thanks.

@manlix Ive talked about this with Selenium's devs, but they are rather uncooperative, so there is not much I can do about this. Sorry

Many users felt abandoned when sad news started flowing about phantomJS stopping and may have pushed for that. If releases star flowing again, it may help Selenium reconsider.

@ylavoie yes. Let them direct all their anger at Selenium. Not PJS. Because PJS relies on Selenium heavily....

Due to the development being suspended until further notice (#15344), it is unlikely that we have the time to ensure and verify that the latest Selenium will work correctly with PhantomJS. Thus, the characterization is correct.
When the development is resumed again in the future, we will revisit the issue.


Couldn't find OpenFermion.Hamiltonians Pkg

Hii i want try this ex. code, but still error when after adding OpenFermion pkg, so where Hamiltonians  ?

Hi @basernapairs ,  I believe this problem is related to where PyCall is looking for our python packages.  PyCall defaults to a python build that comes in Conda.jl.  Following the docs on PyCall I was able to set up a new environment, install openfermion, point my PyCall build to that envs python, and then successfully import OpenFermion.
I used the following after making a new env with a fresh OpenFermion build.
julia> ENV["PYTHON"]="<path to your python3.x"
pkg> add PyCall  # or Pkg.build("PyCall") if you have it already

Then close your julia console. Reopen and try importing OpenFermion.  I was able to see the modules from inside the Julia console so I assume they are usable.


Add description of the use of NEST

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


fix to remove standard port numbers

Hi,
i'm using evcc behind a reverse proxy (kubernetes ingress). callback urls for oauth gets malformed, due adding default port numbers, e.g. https://evcc.localdomain:443.
In my case, mercedes-benz developer portal doesn't accept default port numbers like 443 for https which results in invalid callback urls.
Maybe this PR could help others, but it's not perfect. An configurable external base uri for callbacks could be the better approach.
Kind regards,
Simon

Could you please gofmt the PR?

An configurable external base uri for callbacks could be the better approach.

Absolutely!


Error when starting CustomSender sample project

Hi,
The error I get is: "Could not load file or assembly 'Microsoft.AspNet.WebHooks.Common' or one of its dependencies. Strong name signature could not be verified.  The assembly may have been tampered with, or it was delay signed but not fully signed with the correct private key."
Any ideas? Should I configure something in Visual Studio?
Thanks in advance!

Can you run build.cmd from the command line? That will fix it

AzureStorageEmulator.exe was not installed.
Thanks!

Hello, even tho I ran the build.cmd, I still got the same error.
Solved it by removing the "Microsoft.AspNet.WebHooks.Common" reference and installing it again using:
Install-Package Microsoft.AspNet.WebHooks.Common -Pre

@b4rj - This would not work for me. I am trying to DEBUG my final changes for OneDrive. I followed your steps and now I am getting Microsoft.AspNet.WebHooks.Receivers.OneDrive. In my case, I can't remove or add NugetPackage for OneDrive (because it does not exist). @HenrikFrystykNielsen - any suggestions?


Mamba end-to-end perf is much slower on CI than lab machines

I am seeing a large difference between end-to-end performance (specifically in this pipeline) on lab machines and CI machines. From my investigation - it seems that this is mainly because the CI machines are considerably less powerful than the lab machines I tested.
This isn't a huge deal for catching performance regressions, but if we are going to start automatically reporting performance numbers from CI it would be nice if these matched what we observe on development machines since they are best case.
For example, here are some numbers I collected recently:

tt-metal-large-bm-e09cs01
yyz-lab-124

End-to-end performance (tok/s)
286.1462
389.4207

CPU
AMD EPYC 7352 24-Core Processor
AMD Ryzen 9 7900X 12-Core Processor

RAM (GB)
24
64

Here is a comparison between the two CPUs: link.

fyi @tt-rkim

so what is the action item on this issue?

@esmalTT I think the developer ask here is to have the exact same machine setup on CI as we do on dev.
Unfortunately that's untenable, and something that we've been pretty clear on from the beginning. What are your guys' thoughts on having different thresholds depending on the CI fixture?

@tt-rkim @TT-billteng Yeah having something comparable (or just better) perf-wise would be ideal.

What are your guys' thoughts on having different thresholds depending on the CI fixture?

For now we use the lowest observed threshold between all of the tests we are running, but ideally we'd do something like that. This method would be the best way to catch perf regressions, but this particular issue is more about the absolute measurements.
I guess maybe the core issue is more about us not having a standardized host machines we use to report our official benchmarks (like the ones in the README).

We don't have much control over what's on DevInfra or what's on cloud, so I think the way forward here is to do something to tailor measurements for specific environments, or in this case using the fixture to grab a different threshold.
Unfortunately, I have to characterize the ask here as having metal infra team to convince both cloud and internal devinfra to have the same machines. I don't think it's worth the hassle, and we should practice more defensive programming.
It's not like all dev machines people will run mamba on will be the same.
Let us know what you think

I am seeing a large difference between end-to-end performance (specifically in this pipeline) on lab machines and CI machines. From my investigation - it seems that this is mainly because the CI machines are considerably less powerful than the lab machines I tested.
This isn't a huge deal for catching performance regressions, but if we are going to start automatically reporting performance numbers from CI it would be nice if these matched what we observe on development machines since they are best case.
For example, here are some numbers I collected recently:
tt-metal-large-bm-e09cs01	yyz-lab-124
End-to-end performance (tok/s)	286.1462	389.4207
CPU	AMD EPYC 7352 24-Core Processor	AMD Ryzen 9 7900X 12-Core Processor
RAM (GB)	24	64
Here is a comparison between the two CPUs: link.

is there actually only 24 GB RAM on CI BM?

Take this up with cloud / dev infra, we don't control the type of machines they buy

Hmm yeah I just noticed there. 24GB... that's interesting. Not sure where you got that from...
We have 100s of GB of RAM on the BM.

If we see no movement or proposed action item from your team by the end of this week on this issue, we will close it.

Hmm yeah I just noticed there. 24GB... that's interesting. Not sure where you got that from... We have 100s of GB of RAM on the BM.

Yeah my bad - its actually 240 GB.

If we see no movement or proposed action item from your team by the end of this week on this issue, we will close it.

Based on the above, I don't think there's any immediate action on you guys. I am fine to close this while we consider how we want to standardize our model perf benchmarks.


Support per compaction service metrics

In #1605 compaction services with multiple thread pools were added.  Having metrics for the queue size and number of running compactions would be very useful. Not sure of the best way to do this, but it would be nice if the metrics included the compaction service name and thread pool/executor name within the service.
compaction.task.<service name>.<executor name>.queued=<num queued>
compaction.task.<service name>.<executor name>.running=<num running>

Having access to this information will be very important for tuning compaction service configuration.

I have implemented this at the following branch.
https://github.com/keith-turner/accumulo/tree/accumulo-1612
Plan to submit a PR after #1605 is merged.

Keith I took a cursory glance and have a question that I hope you can answer of the top of your head.  In CompactionExecutorsMetrics, what is the cardinality of the CompactionExecutorId?
I'll try to look at this in more detail later.

what is the cardinality of the CompactionExecutorId?

That is the number of compaction executors across compactions services configured by users.  With the default configuration there will be 7 :

Compaction service for root tablet has 2 executors
Compaction service for metadata tablets has 2 executors
Compaction service for user tablets has 3 executors

Would the ids persist and be consistent across restarts?  The reason that I'm asking is that some metrics systems will create a separate time series for each metric "name"  Things that are unbounded (which seems this is not) cause issues - if the id are not consistent, then performing metric queries of a time-series that would cross restart boundaries could be misleading.  (Anyone / System querying over a time range may not be aware of the restart)
If that would be the case, then rolling the values up into root, metadata and users may be more meaningful to a metrics system.  If there is still interest in each ceid at runtime, then maybe logging?

Would the ids persist and be consistent across restarts?

Yeah.  The ids are based on user provided names in configuration.  For example the following configuration will create a compaction service named cs1 with three executors named small,medium,large`.
tserver.compaction.major.service.cs1.planner=org.apache.accumulo.core.spi.compaction.DefaultCompactionPlanner
tserver.compaction.major.service.cs1.planner.opts.executors=[{"name":"small","maxSize":"16M","numThreads":3},{"name":"medium","maxSize":"128M","numThreads":2},{"name":"large","numThreads":2}]

This would result in the following metrics :

prefix_cs1_small_running
prefix_cs1_small_queued
prefix_cs1_medium_running
prefix_cs1_medium_queued
prefix_cs1_large_running
prefix_cs1_large_queued

Not sure what the actual prefix would be, would need to dig around in the code.


Bug: Auto-plot new opens all plots when loading new db-file

Found this bug when loading a new db-file:
When the checkbox "Auto-plot new" is enabled in the main plottr window and a new db-file is loaded, it will load all the experiments from that db-file.
For now it is easy to circumvent by unclicking the box, but I assume this is unwanted behaviour.

Fixed in #85


spicedb migrate does not output help text

When spicedb migrate is invoked, no help text is provided.
This becomes even more confusing when I forget to specify the revision in the command. This outputs no information, and exists with status code 1.
spicedb migrate \
    --datastore-conn-uri='<URI>' \
    --datastore-engine="mysql" \
    --log-level="debug"

The fix here would be to validate the input arguments and fail with the help text, if the revision is not provided/valid.

Thanks for the report! I've opened https://github.com/authzed/spicedb/pull/1101 to address it


Replace the big number with Infinity

This PR

Fixes a bug

Describe the changes below:
I think it makes more sense to use the Infinity definition than to use a large number instead.

This won't work, I'm afraid:
let counter = Infinity;
counter = counter - 100;
console.log(counter); // outputs 'Infinity'

You can't use it as a seed for a decrementing value, as we do in all those cases.
We could use Number.MAX_SAFE_INTEGER instead, but there's no real point, it just takes up more bytes for no benefit in this case.


Upgrade criteo branch to 4.9.3

What remains to do:

[ ] Validate with our SDK
[ ] Ensure kubeconfig provides the right certificates for validation

I have a counter proposal: review/test v4.9.3-criteo1, which is criteo branch rebased on v4.9.3.


chore: Support new structure of web features data.json v0.10.0

The Web Features repo changed the structure of the data.json in this PR: https://github.com/web-platform-dx/web-features/pull/1060
Instead of being a map of features, it is now an object with three top level keys:

features
groups
snapshots

This is a breaking change and has caused us to start ingesting features as if they had IDs: features, groups, snapshots
This commit fixes that by updating the defs.schema.json to be the latest version (v0.10.0). This file comes from the web features repo.
After auto-generating the code for that schema file, this commit adjusts the code to use it.
We previously had v0.6.0 of the schema. And since then, some optional fields have become required fields. So you will see the removal of some nil checks (since now the field is not nullable).
This change does NOT ingest the groups or snapshots yet because this change alone is pretty big. Instead, this change keeps the same functionality. Only adjusting for the new data format.

Can you guard against future schema changes via assertions or similar? It's better to not display any information if the schema has changed than to silently ingest unexpected data.

Can you guard against future schema changes via assertions or similar? It's better to not display any information if the schema has changed than to silently ingest unexpected data.

There are a few things we can do

Define our own "Minimally Complete Feature":

In this particular instance, we could create a clear, internal definition of the essential fields and their types that constitute a valid feature for our application. This acts as a contract between the data source and our ingestion process.
Implement validation checks (assertions, type checks) during parsing/ingestion to ensure that every incoming feature adheres to this contract.
If a feature violates this contract (e.g., missing required fields, unexpected types), raise an explicit error to halt processing and prevent the ingestion of incomplete or malformed data.

Incorporate Schema Versioning (Preferably Semantic Versioning) - long term:

Advocate for adding a schemaVersion field (or similar) to the data.json in the Web Features repo. This version should ideally follow semantic versioning (MAJOR.MINOR.PATCH) to clearly communicate the level of change.
On our end, store the expected schemaVersion that our application supports. During ingestion, compare the incoming schemaVersion against our supported version. If the version uses semantic versioning, we can easily ignore patch and minor updates.
Previous works: Kubernetes does a similar idea where all resources  have a top-level apiVersion key. (but it does not follow semver).

I can make an issue and talk about number 2 with @foolip

I'd like to suggest a simpler solution, which is that we release web-features 1.0.0 and from there bump the major version for any breaking changes, which would include any schema changes that aren't pure additions. Would that work?


Table first row color different than other rows.

Just typed a full description and then Github gave an error 500. Ok. Here we go again.
Basically target is the same as this : https://docs.lvgl.io/master/widgets/table.html#simple-table
Event class is missing some functions : get_target(), get_draw_part().
Underlying the drawpart there are structures that enable to change cell rectangles and labels. For which no Rust equivalent has been defined so far.
Can this be exposed ?

I've been trying to figure out a way to do this safely for a while but I honestly have no idea. I think for now using the lvgl_sys unsafe bindings is needed; I just don't see a way to be able to do this safely easily. There probably is a way but I worry about get_target() specifically since it can give multiple Objects which all point to the same lv_obj_t and thus can cause race conditions.

Ok, I see , thanks for the feedback. I'll see if using lvgl_sys is workable.


Slideshow shrinks and grows with image size

When scrolling through images in slideshow mode, if the image sizes change (e.g. 256x256 to 1024x1024 and back) the slideshow window changes size as well, causing a jerky user experience and the metadata to slide around.
https://tools.pds-rings.seti.org/opus#/instrumentid=Cassini+ISS&timesec1=2015-255T03:18:10&timesec2=&view=browse&browse=gallery&colls_browse=gallery&page=1&gallery_data_viewer=true&limit=100&order=time1&cols=ringobsid,planet,target,phase1,phase2,time1,time2&widgets=timesec1,instrumentid&widgets2=&detail=
Select the first thumbnail and click on the slideshow (two arrows) icon. Then use left and right arrow keys to scroll through and notice annoying behavior.
I think we probably want to solve this by making the window always large enough to hold the largest image returned by the current search. The largest image size can be returned from the "Greater Size in Pixels" query.

Images are now force to be a particular size.


Speech API giving 0 results

I'm using Google Cloud Speech to text api in Java.
I'm getting 0 results when I call speechClient.recognize
pom.xml:
    <dependency>
    	<groupId>com.google.cloud</groupId>
    	<artifactId>google-cloud-speech</artifactId>
    	<version>0.80.0-beta</version>
    </dependency>

Java code:
import java.io.FileInputStream;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.util.List;
import com.google.api.gax.core.FixedCredentialsProvider;
import com.google.auth.oauth2.GoogleCredentials;
import com.google.cloud.speech.v1.RecognitionAudio;
import com.google.cloud.speech.v1.RecognitionConfig;
import com.google.cloud.speech.v1.RecognitionConfig.AudioEncoding;
import com.google.cloud.speech.v1.RecognizeResponse;
import com.google.cloud.speech.v1.SpeechClient;
import com.google.cloud.speech.v1.SpeechRecognitionAlternative;
import com.google.cloud.speech.v1.SpeechRecognitionResult;
import com.google.cloud.speech.v1.SpeechSettings;
import com.google.protobuf.ByteString;

public class SpeechToText {

	public static void main(String[] args) {

		// Instantiates a client
		try {

			String jsonFilePath = System.getProperty("user.dir") + "/serviceaccount.json";
			FileInputStream credentialsStream = new FileInputStream(jsonFilePath);
			GoogleCredentials credentials = GoogleCredentials.fromStream(credentialsStream);
			FixedCredentialsProvider credentialsProvider = FixedCredentialsProvider.create(credentials);

			SpeechSettings speechSettings =
					SpeechSettings.newBuilder()
						.setCredentialsProvider(credentialsProvider)
						.build();

			SpeechClient speechClient = SpeechClient.create(speechSettings);

			//SpeechClient speechClient = SpeechClient.create();

			// The path to the audio file to transcribe
			String fileName = System.getProperty("user.dir") + "/call-recording-790.opus";

			// Reads the audio file into memory
			Path path = Paths.get(fileName);
			byte[] data = Files.readAllBytes(path);
			ByteString audioBytes = ByteString.copyFrom(data);

			System.out.println(path.toAbsolutePath());

			// Builds the sync recognize request
			RecognitionConfig config = RecognitionConfig.newBuilder().setEncoding(AudioEncoding.LINEAR16)
					.setSampleRateHertz(8000).setLanguageCode("en-US").build();

			RecognitionAudio audio = RecognitionAudio.newBuilder().setContent(audioBytes).build();

			System.out.println("recognize builder");

			// Performs speech recognition on the audio file
			RecognizeResponse response = speechClient.recognize(config, audio);
			List<SpeechRecognitionResult> results = response.getResultsList();

			System.out.println(results.size()); // ***** HERE 0

			for (SpeechRecognitionResult result : results) {

				// There can be several alternative transcripts for a given chunk of speech.
				// Just use the
				// first (most likely) one here.
				SpeechRecognitionAlternative alternative = result.getAlternativesList().get(0);
				System.out.printf("Transcription: %s%n", alternative.getTranscript());
			}
		} catch (Exception e) {
			System.out.println(e);
		}
	}
}

In the code above, I'm getting results.size as 0. When I upload the same opus file on demo at https://cloud.google.com/speech-to-text/, it gives output text correctly.
So why is the recognize call giving zero results?

Hey, just a quick debug first.
Do any of the existing samples work for you as is?
Then once, we know your environment is setup correctly, we can debug the code. :)
(I'll take a look at the code now, but just a first initial question)

Also you might need to change the AudioEncoding, because your file is a .opus, you should probably use:
AudioEncoding. OGG_OPUS

Going to close this, but please feel free to reopen if your issue isn't solved by the above. :)


No 'sys.dm_os_loaded_modules' in Azure SQL DB?

This page states that it applies to Azure SQL Database. However, when I try to execute the SQL statement:
SELECT
		mod1.name,
		mod1.description
	from
		sys.dm_os_loaded_modules  as mod1
	where
		mod1.description = 'XTP Native DLL';

I get an error:
Invalid object name 'sys.dm_os_loaded_modules'.

I have tried the above statement both in the master database and in one of my application databases.

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

ID: 0e73faff-9baf-2c59-2039-61c0a957d581
Version Independent ID: a13e94c7-98dd-f801-4e4d-b5cfa061cab5
Content: Native compilation of tables & stored procedures - SQL Server
Content Source: docs/relational-databases/in-memory-oltp/native-compilation-of-tables-and-stored-procedures.md
Product: sql
Technology: in-memory-oltp
GitHub Login: @LitKnd
Microsoft Alias: kendralittle

@epiphany-it -- thank you for your feedback.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@LitKnd -- please look into this issue.

Hi @epiphany-it ,
Thanks for reporting this issue to us.
I can reproduce the issue. The article for sys.dm_os_loaded_modules itself applies to SQL Server and Azure Synapse, and doesn't apply to Azure SQL Database, which I believe is accurate.
Reviewing this, I think there are a few things that could use refreshing in the article:

As you point out, we should clarify that the sys.dm_os_loaded_modules DMV is not available, and perhaps provide an alternate way of querying memory optimized tables and procedures for SQL DB.
I also see that there's a code sample in that article which enables In-Memory OLTP and adds a filegroup. This isn't necessary or possible against an Azure SQL Database, it's automatically enabled if you are using a service tier which supports it, so we should clarify that.
In-Memory OLTP is supported by Azure SQL Managed Instance, which we don't have in the 'Applies to'.

We should also scan related articles for similar issues.
I have created a work item to get this work scheduled. I don't have a hard date for when this will be scheduled, but it's something we have ready to be prioritized as our team has availability.
Thanks again for reporting this. I am going to close out this issue since we now have the work item for tracking, but you can still respond to the issue even after it is closed.
Cheers,
Kendra Little from the Database Docs Team
#please-close


Text-to-Image Alignment Performance of the ELLA-SDXL Model

First of all, I would like to express my sincere gratitude for your open-source model ELLA, which is truly remarkable. I have been closely following your team's work, and as soon as the model was released, I couldn't wait to test it. I evaluated the text-to-image alignment performance of the ELLA-SD1.5 model using GenEval. Compared to the original Stable Diffusion 1.5, ELLA-SD1.5 demonstrated a 7 percentage point improvement in text-to-image alignment, and the improvement was even more significant when compared to Salesforce's diffusion-DPO  method.
I noticed that Stable Diffusion 3 has adopted GenEval to evaluate its text-to-image alignment performance. Therefore, I would like to inquire whether your team has plans to release the GenEval evaluation scores for ELLA-SDXL. This would enable us to compare the performance of ELLA-SDXL relative to SD3 on a unified scale.

model
Overall
single
two
counting
colors
position
color_attr

SD1.5
42.34
95.62
37.63
37.81
74.73
3.50
4.75

SD1.5-DPO
43.00
96.88
39.90
38.75
75.53
3.25
3.75

ELLA-SD1.5
49.94
94.69
55.81
36.56
77.32
14.75
20.50

SDXL
55.63
98.12
75.25
43.75
89.63
11.25
15.75

SDXL-DPO
58.02
99.38
82.58
49.06
85.11
13.50
18.50

ELLA-SDXL

DALL-E 3
67.00
96.00
87.00
47.00
83.00
43.00
45.00

SD3 best
74.00
99.00
94.00
72.00
89.00
33.00
60.00

I can't wait! Thanks for the research!

Thanks a lot for your job and research! @xiexiaoshinick


Switch ControllerRegistration controller to controller-runtime

How to categorize this PR?

/area dev-productivity scalability
/kind enhancement
What this PR does / why we need it:
Refactor the ControllerRegistration controller to controller-runtime.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/4251
Special notes for your reviewer:
Generally, we want to follow this cookbook while refactoring existing controllers:

Add documentation
Add integration test based on envtest
Switch controller to controller-runtime

Release note:

NONE

/assign

/lgtm

/approve


Version API doesn't return the SDK version

🐞 Bug Report
Affected Services [REQUIRED]
Version API
Is this a regression?
No
Description and Minimal Reproduction [REQUIRED]
Version API doesn't return the SDK version. Sample responses:
{
    "apiVersion": "v2",
    "version": "0.0.0",
    "serviceName": "device-onvif-camera",
    "sdk_version": "0.0.0"
}

The makefile here is missing the setting of the SDK version
https://github.com/edgexfoundry/device-onvif-camera/blob/main/Makefile#L22-L23
That is done here properly:
https://github.com/edgexfoundry/device-rfid-llrp-go/blob/main/Makefile#L21-L23
Note this looks to be an issue will many or all the Go Devices and App services
🌍  Your Environment
Deployment Environment:
EdgeX Version [REQUIRED]:
Anything else relevant?

@weichou1229 , this is a better example of what should be in the makefile.
https://github.com/edgexfoundry/device-rfid-llrp-go/blob/main/Makefile#L21-L23
I update the description with this link.


add support for cice6 dumping intermediate restart in CICE_plug

This PR allows coupled model with CICE6 to dump intermediate restarts if RESTART RECORD parameter is on in AGCM.rc. This is a feature required by coupled DA system.
There is also minor tweak of regrid.py utility.
This PR depends on https://github.com/GEOS-ESM/GEOSgcm/pull/672

Forcing.


Remove intermediary copy steps

All Next.js page/function files are initially copied by next-on-netlify to out_functions and out_publish.
This plugin then copies all those files from out_functions and out_publish to the site's Functions/publish directories.
If those are lots of files, this additional copy step might end take lots of build time. Instead, we could remove that intermediary step:

Make the main function exported by netlify-on-netlify accepts an options object so users can customize the location of out_functions and out_publish:

https://github.com/netlify/next-on-netlify/blob/a9dade3824d1288770d967cc6f57f238d5947367/index.js#L7
const nextOnNetlify = ({ functionsDir = "out_functions", publishDir = "out_publish" } = {}) => {

Pass constants.FUNCTIONS_SRC and constants.PUBLISH_DIR to next-on-netlify
Remove the additional copy step :tada:

https://github.com/netlify/netlify-plugin-nextjs/blob/84c6f7f4ccfd6d814bd642d0a27921ef49b67c25/index.js#L74
As a follow-up, we can also expose those to next-on-netlify CLI.
What do you think @lindsaylevine?

i think this is a great callout. especially with the CLI, it'll avoid making those additional dirs locally :). let's do it!

Note: several users have asked for it in https://github.com/netlify/next-on-netlify/issues/36 and https://github.com/netlify/next-on-netlify/issues/73

yeah, long awaited ask. was gonna just wait til we fully transitioned off NoN, but this is great. i'll take care of it shortly :)

Note: several users have asked for it in netlify/next-on-netlify#36 and netlify/next-on-netlify#73

to-do for me before closing this issue:
reply to these issues (and any other relevant ones?) about the plugin and this functionality

to-do:
add specific test to next-on-netlify for functionsDir and publishDir


Adding and dropping columns to active-set matrix S inefficient

As discussed, adding and dropping columns from the active-set matrix S is especially inefficient. Because S is only used through products with vectors, it seems plausible that we could replace the explicit matrix S with an implicit linear map.
Roughly, if active is the vector of indices pointing to active variables, the linear map would compute the product S*x via the method
    # product S*x
    function Sprod(A, x, active)
        xx = zeros(size(A,2))
        xx[active] .= x
        return A*xx
    end

(and a corresponding method for the product S'*y).
So instead of copying and deleting individual columns of A into S, we would just keep track of the active indices and use the linear map Sprod to compute the products. The tradeoff is additional products with A. (Currently, there is only one product with A at each iteration used to pull a column from A into S, ie, this line.)
To do:

[ ] Implement Sprod and SprodT
[ ] Wrap into a linear map
[ ] Verify that QRupdate works with linear operators
[ ] Create a benchmark problem and profile against current version

Am I understanding correctly that if A is m x n then S will be m x #active?
If this is the case, then for m >> n one could preprocess by replacing ||A x - b||^2 with ||R x - Q^T b||^2. This has a large initial cost (full QR factorization) but in practice this is often negligable compared to the everything else we are doing.

Am I understanding correctly that if A is m x n then S will be m x #active?

Indeed.

If this is the case, then for m >> n one could preprocess by replacing ||A x - b||^2 with ||R x - Q^T b||^2. This has a large initial cost (full QR factorization) but in practice this is often negligable compared to the everything else we are doing.

But ASP never solves the full m-by-n least-squares problem ||Ax-b||^2. Instead, it solves the thin m-by-k problem defined by the current active set, ||Sx_s-b||^2 via Q-less QR, ie, ||Sx_s-b|| = ||Rx_s Q^Tb|| where S=R'R (using the corrected semi-normal equations, which should be accurate for S with conditioning up to 1e12.

I am missing something. According to your first paragraph you are saying that S is m x #active. but in your second paragraph S is #active x #active?

There's a typo in my earlier response -- sorry, that probably led to the confusion.
At each iteration that is not stationary with respect to the current active set of $k$ variables, ASP pulls a new column $a$ from the full matrix $A$, adds it to the active set, and updates a Q-less QR factorization:

identify new column $a$
update the current $k\times k$ triangular factor $R$ to the $(k+1)\times(k+1)$ triangular $\bar R$, and update the  $m\times(k+1)$ active-set matrix $\bar S$ so that

$$\bar R^T\bar R = \bar S^T\bar S, \quad \bar S:=\begin{bmatrix} S & a \end{bmatrix}.$$

solve the $m\times (k+1)$ least-squares problem $$||\bar Sx_{\bar S}-b||_2$$ via the seminormal equations $$\bar R^T\bar R x_s = \bar S^Tb$$

Steps 2-3 are carried out in a numerically stable way by QRupdate.jl. Note that QRupdate doesn't actually need the updated matrix $\bar S$. In particular, it only requires as input $R$, $S$, $a$, and $b$. My main point is that $S$ doesn't need to be an explicit matrix because QRupdate only requires the products $Sv$ and $S^Tw$ for vectors $v$ and $w$, and so $S$ can in principle be a function that computes these products.

Good news: QRupdate now has in-place versions of its add- and delete-column routines! Huge savings, particularly in allocations.
The above discussion on potential savings from implicitly maintaining $S$ still hold.

@tinatorabi  -- could you please post a MWE here that reproduces this performance bottleneck? Probably a random problem of sufficient size will be enough.


v2.5.0

The 2.5.0 release contains the following new features:

Adaptive Data Rate for EU_863_870 band
Downlink Queue
Device Descriptions
TLS support in MQTT Client

API Changes:

Add description to device (Handler)
Add region to Uplink and Activation Metadata (gRPC)
Add schedule field to Downlink Message (MQTT)

Database changes (migration):

Implemented data migration functionality (Discovery, Networkserver, Handler)
Add _version to all models (Discovery, Networkserver, Handler)
Use downlink queue instead of next_downlink (Handler)

Other changes:

Publish more (error) events to MQTT
Check arguments in ttnctl

Coverage increased (+0.7%) to 74.354% when pulling 1c4c62ac5d7cef26cf7c269849d0f2e41c5d1dcd on develop into 4a3648d798c59488edb2239bdc49fa0edc259529 on master.

Coverage increased (+0.7%) to 74.354% when pulling 1c4c62ac5d7cef26cf7c269849d0f2e41c5d1dcd on develop into 4a3648d798c59488edb2239bdc49fa0edc259529 on master.


Allow using Guardian with Phoenix 1.3.0-rc

Just an update to mix deps and updated mix.lock.
Also, test executed without issues with Phoenix 1.3.0-rc.

Please see #273 for proper resolution.


TS-5082: define IOBufferReader::is_read_avail_more_than with TS_INLINE keyword

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1254/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1149/ for details.

Hmmm, it seems we are incredibly inconsistent here. Why singling out this one place for this change? Do we really need TS_INLINE in general? I notice we turn off inlining via #define in some places, but then I'd argue that we should not use it at all in those place, no?
I think what I'm saying is

Why not remove TS_INLINE ? I.e. just use inline.

In those places where TS_INLINE is used, but #define'd to nothing, we just remove TS_INLINE.

Maybe also make an effort to reduce the use of inline in general, and let the compiler deal with it.

I'm fine landing this as-is in this patch, but maybe file another Jira for a general cleanup here?

From my understand:

The TS_INLINE defined as inline in lib/ts/ink_apidefs.h only for libts, for libts.so these functions declared by TS_INLINE is inline function.
And TS_INLINE defined as empty in iocore/*/Inline.cc, for a sub module static libs (*.a) these functions declared by TS_INLINE is NOT inline function.

I don't know the reasons about the TS_INLINE macro, this case only fix the compile error.

A function of Event System and it is defined with TS_INLINE, e.g. bool is_read_avail_more_than()

It is compile into net/Inline.o and it is not inline function if it is called from Net Sub-system.
It is compile into eventsystem/libinkevent.a and it is inline function if it is called from Event System (internal calls).

@zwoop

Do I have to backport it to 6.2.x and 7.0.x ? @zwoop


Implement new functionalities

#87

Coverage decreased (-4.009%) to 84.211% when pulling beac10a261d9083760e1ac080c1560bf3dc138df on zdrawku:master into 6e1e17b294ca50d1cea3597f705674ba6e08783b on Infragistics:master.

Coverage increased (+4.9%) to 93.158% when pulling 8adef1b17a0eaa76dbb6e82448d81c9cd2f55e72 on zdrawku:master into 6e1e17b294ca50d1cea3597f705674ba6e08783b on Infragistics:master.


feat: port CategoryTheory.Monoidal.Discrete

[x] depends on: #2466

This PR/issue depends on:

leanprover-community/mathlib4#2466
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib4#2466
By Dependent Issues (🤖). Happy coding!

bors merge

bors merge

bors merge

bors merge


How to set-up React-Flexbox-Grid

I am having issues to set-up https://github.com/roylee0704/react-flexbox-grid
Errors:
Processing Step: SCSS compilation
Unable to compile /Users/PolGuixe/Documents/Developer/Meteor/Apps/tests/Zetoff-Sandbox/node_modules/react-flexbox-grid/doc/app/components/markdown/style.scss
Line: 14, Column: 11

and:
Errors prevented startup:

   While processing files with nathantreid:css-modules (for target web.browser):
   /Users/PolGuixe/Documents/Developer/Meteor/Apps/tests/Zetoff-Sandbox/node_modules/node-sass/lib/index.js:420:22: CSS modules
   SCSS compiler error: file not found:
   (/Users/PolGuixe/Documents/Developer/Meteor/Apps/tests/Zetoff-Sandbox/node_modules/highlight.js/styles/github-gist.scss)

How it should be properly set up?

CSS Modules is trying to process react-flexbox-grid's doc folder, which includes a sample app. You need to exclude this folder from processing. You most likely only care about the content of the src folder, so I recommend you also ignore the spec and test folders.
You can do this using the "ignorePaths" option (see Package Options) for more details.
Something like this should do the trick:
package.json
"cssModules": {
  "ignorePaths": [
    "node_modules/react-flexbox-grid/(doc|spec|test|)/"
  ]
}

@nathantreid now the errors had stopped but the package is not working neither.
This is my configuration:
"cssModules": {
    "extensions": [
      "scss"
    ],
    "globalVariables": [
      "node_modules/react-toolbox/lib/_colors.scss",
      {
        "theme-building": true
      },
      "client/theme/toolbox-theme.scss",
      "node_modules/flexboxgrid/dist/flexboxgrid.css"
    ],
    "ignorePaths": [
      "node_modules/react-flexbox-grid/(doc|spec|test|)/"
    ]
  }

If I include css from in extensions, then I get errors from a lot of other files that I will probably have to ignore.
Do you know which should be the configuration? Sorry I am a total CSS module noob.

Can you elaborate on "the errors had stopped but the package is not working neither."? Are you getting any errors now (if it builds successfully, also check the browser console).
Can you post a repro so I can take a look?
This should work without  css file parsing, as the files in globalVariables are read in directly.

Basically the components did not have any class name. The class property was empty.

OK, I've created a working example.
It looks like the only big change is that you need to enable css file processing, so you will need to ignore the other files that are causing you errors (or if they are legacy CSS files that you application needs, use the passthroughPaths option to exclude them from CSS modules processing but still include their CSS in the output).
I also configured the flexboxgrid processing using ignorePaths to only process the dist/flexboxgrid.css file to help keep processing times low.

Hi your example is no longer working with the latest version of react-flexbox-grid 1.1.4. I am going to have a play though and see if I can't get it working.
Problem seems to be that now react-flexbox-grid is creating a different pattern for the class names than before. I don't have much experience using css modules but they look awesome and I'm sure there must be a way to get it to work. I'm using react-felxbox-grid in my non-meteor projects just really hoping to get the latest version working in Meteor now too.


add docs for cli and contributing guide

Codecov Report

Merging #28 (a975c95) into main (54b08dd) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #28   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           28        28
  Lines         3234      3234
  Branches       187       187
=========================================
  Hits          3234      3234

Impacted Files
Coverage Δ

src/webdav4/cli.py
100.00% <ø> (ø)

tests/conftest.py
100.00% <100.00%> (ø)

tests/server.py
100.00% <100.00%> (ø)

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 54b08dd...a975c95. Read the comment docs.


Schedule action requests with correct session

Use privileged session to insert action-request contracts for future execution as non-privileged sessions are not allowed to create action-request contracts.
Change-type: patch
Signed-off-by: Josh Bowling<EMAIL_ADDRESS>

@balena-ci rebase

@balena-ci I self certify!


Support to specify a shorten name for compression codec

#66
This PR adds the support for shorten names for compression codecs and added a CompressionCodecs class instead of the implicit function as its use is nor recommended.

Current coverage is 89.20%

Merging #67 into master will decrease coverage by -0.02% as of b7f6ecf

@@            master    #67   diff @@
=====================================
  Files           12     13     +1
  Stmts          622    630     +8
  Branches        86     87     +1
  Methods          0      0
=====================================
+ Hit            555    562     +7
  Partial          0      0
- Missed          67     68     +1

Review entire Coverage Diff as of b7f6ecf

Powered by Codecov. Updated on successful CI builds.

I am going to merge this.

Merged.