get_result should return an error if multiple rows are returned

Versions
All
Problem Description
RunQueryDsl provides several methods for accessing query results:

execute (returns number of rows affected)
load/get_results (returns all rows)
get_result (returns a single row)
first (returns the first row)

I expected that get_result would return an error if multiple rows are returned (to differentiate it from first). However, it seems that it will just return the first row. Other ORMs (SQLalchemy, Django ORM) will raise an exception in this case.
This is dangerous: if code is using get_result then it clearly expects a single row to be returned, and if multiple rows are returned, it's a sign that the query may be wrong, or accidentally missing filters which should be present. If this expectation is invalid, then diesel should report an error sooner rather than later.
Unfortunately, this is a breaking change to fix. Personally I think that's still worth it due to the danger not fixing it poses. However, I would also be OK with the documentation being updated to call out this fact, and a new method being added to assert that a single item is returned.

I'm not sure if I would really return an error in this case, is it can sometimes be the desired behavior (thought then you should probably just use .first in the first place as it appends LIMIT 1 to your query.) Adding a note to the documentation sounds definitively like a good idea.
Theoretically we can also generate an error message for the case of an result set for more then one element there in the next version, as it will be diesel 2.0 anyway. So if someone adds some convincing arguments here I would accept a PR doing that.

I agree with @Diggsey and the main reason has to do with having an explicit intention. Having a method to make explicit the intention of wanting to get one and only one row from an update makes for clearer and stricter API while still supporting all use cases. Then .first, .get_result, .get_results all have different specific use cases.


Update Set-CsTeamsMeetingPolicy.md

add all supported parameters

@MartinRinas - This PR https://github.com/MicrosoftDocs/office-docs-powershell/pull/12075 requested to make the same changes as you. The author of the changes provided more context on the definition of each parameter. I approved the attached PR and will close this one.


tool: add skopeo, crictl into dev dockerfile

Signed-off-by: Zhou Liang<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zhouliang121 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


SOAP to REST API publisher notification improvement

Problem
The publisher shows the following message when we try to create SOAP APIs (using WSDL).

This is valid for the "Generate REST APIs" option. For passthrough (passing through SOAP messaging instead of JSON), since the API Manager doesn't do any mediation, this might not be relevant. Even with Passthrough, with the API Management features, there can be use cases we can address through the API Manager.
Solution
Suggest providing this notification only when someone selects the "Generate REST APIs" option.
Affected Component
APIM
Version
4.2.0-Beta
Implementation
No response
Related Issues
No response
Suggested Labels
Affected/4.2.0-Beta

The message has been updated through U2 as follows.

Fixes with the PR: https://github.com/wso2/apim-apps/pull/488


Require Developer Certificate of Origin (DCO) signoff on all commits?

Somewhat related to https://github.com/OpenRefine/OpenRefine/issues/6422#issuecomment-1984389601, I'd like to suggest that we require DCO sign-off on all commits and document what we mean by it in the context of the OpenRefine project.
https://wiki.linuxfoundation.org/dco
https://stackoverflow.com/questions/1962094/what-is-the-sign-off-feature-in-git-for
Proposed solution

Add DCO info to our developer docs explaining what sign-off implies for us
Socialize info with developer community
Enable DCO checking in Github

I've never liked this extra effort and didn't think in OpenRefine (without any commercial bent connected to it, unlike Linux Foundation or Oracle projects, etc.) that a DCO would even be needed for us?  Maybe I'm not the right one to comment on this.
@wetneb I'd be curious what or how CS&S (essentially, our legal team) thinks about this, if we should have DCO signoffs or not?  Can you email them?

We require developers to respect licenses and intellectual property rights and only submit code that they own or have the rights to. The DCO is a lightweight mechanism for them to positively affirm that. We've had cases in the past where people have submitted things without sources available or with incompatible licenses. The DCO would prompt them to think first.
I'm a little disappointed that this didn't at least spark a discussion, but I don't feel strongly about it.


Rails Admin not functioning with nginx

Describe the bug
Rails admin gem works perfectly fine with localhost but when we put it to our server using nginx then it gives doesn't take any new input for tables and doesnt allow editing also. Also we cannot export the data, when we try doing so it gives internal server error.
Reproduction steps
Host a rails application through nginx as your server and check for rails admin functionalities
Expected behavior
It should work as fine as it was with localhost.
Additional context

rails version: <IP_ADDRESS>
rails_admin version: 3.0
rails_admin npm package version:
full stack trace (if there's an exception)

Add any other context about the problem here.

Basically what this means is you're not setting up Nginx or other related components correctly. If you think that RailsAdmin is to be blame, you should give a reasonable explanation on why that's not the case.


IDE support for all Bazel-built particles

Discovery: Adding the platform = ['wasm'] flag to the BUILD rule turns off IDE sync.
This PR slightly changes the shape of the Particle interface and adjusts the BUILD rules to bring about Android Studio (or IntelliJ) support for Particles.

What is the implication of removing platform = ['wasm']?

In B(l)aze(l) land, it means that we build both Wasm and Jvm targets. At the moment, the only way the IDE gets the providers it needs for the generated code is through the Jvm build pathway.


Many class/struct member variables can be moved to initializer list

Godot version:

master branch commit 41d1dba35fc851258d5aabad819d09f6a43e324c
Issue description:

There are many class/struct member variables that can be moved to an initializer to avoid creating objects twice.
The performance enhancement of doing this is negligible for primitive types and for compilers that already do this optimization by default (like GCC), but a performance improvement might be noticeable on compilers that do not do this since on some cases, generally speaking, creating non-primitive objects twice could hurt performance.
Here's a list of the member variables that can be moved. This file does not include the variables I moved in #24241. It's an easy patch for anyone who'd like to contribute.
Steps to reproduce:
To generate attached list in linux you must have cppcheck installed. Then:

cd $godot_repo
cppcheck --enable=performance --std=posix . 2> performance_check.txt
fgrep 'initialization list' performance_check.txt | fgrep -v '[thirdparty/'

I'd like to try this.

I'd like to try this.

@Cartogy Since there's refactoring going on in the master branch, I'd wait until it's more stable.

@Cartogy Since there's refactoring going on in the master branch, I'd wait until it's more stable.

Understood. I will keep my eye on its progress.

Understood. I will keep my eye on its progress.

Closing in favor tracking this issue in:

#43636


[WIP] Add Pipeline

Add Pipeline as an ember-engine to 2.0
Here is related Issue

Revert yarnl.lock; I don't know what http://registry.npm.taobao.org/ but I don't want 5000 lines of lock changes to load stuff from an unknown and insecure registry.

Many hardcoded english strings need to be moved to translations.


To improve the performance use bundle info from the index.db

What should be done here?
Unmarshal and write in the tmp dir in the disk the bundle files from bundle column (operatorbundle table) when this info exist instead of downlad and extract the bundle from the image using the bundle path.
Why is required to write the bundle?
That is only required to run the scorecard. So, it should be only load in memory when the scorecard checks will not be executed to reduce the io effort required.

@ecordell I tried to do it, however, I am unable to Unmarshal the data from the database to do the structure  github.com/operator-framework/api/pkg/manifests.Bundle

@ecordell I am facing issues to Unmarshal that for github.com/operator-framework/api/pkg/manifests.Bundle. Do you know any place that uses this info from the index.db for I am be able to check how that has been used?

@ecordell I have been trying to parse the manifests from the database. Do you know if some project does that? Have we any API implementation to help on that?

It shows outdated/done


Most of the changes to get the survey working

Defaults to amd64, I overlooked this hoping this is already implemented and I'm just not passing in the correct values to change architecture.
compute:

architecture: ppc64le
controlPlane:
architecture: ppc64le

Missing defaultMachinePlatform, however may not need defaultMachinePlatform.
defaultMachinePlatform:
type: christy-ocp-40g
May be missing MetaData, however these may populate already I just havent tested them.
metadata:
creationTimestamp: null
name: christy-pvc
Signed-off-by: wolf<EMAIL_ADDRESS>

one more: pkg/asset/installconfig/powervs/platform.go:124:27: syntax error: unexpected newline, expecting comma or }

@clnperez Ill take a look, but this seemed to work before I made the PR.  If that is the case, could you help me understand what is different between our two builds so that I can avoid these types of errors in the future?  Perhaps different versions if go expect different things?  Or maybe I missed something when I merged session.go?  Oh... I need to update region.go now that session.go updated ssn.session to ssn.Session... will do that in a bit, or in the morning

since it doesn't compile, i think you just missed something when you merged the code. it's not due to different versions of go for sure. but yah the last bit with session vs Session is b/c of Mike's PR.

to get the right arch:
diff --git a/pkg/types/defaults/machinepools.go b/pkg/types/defaults/machinepools.go
index 7896ae451..e23882265 100644
--- a/pkg/types/defaults/machinepools.go
+++ b/pkg/types/defaults/machinepools.go
@@ -3,6 +3,7 @@ package defaults
 import (
        "github.com/openshift/installer/pkg/types"
        "github.com/openshift/installer/pkg/types/libvirt"
+       "github.com/openshift/installer/pkg/types/powervs"
        "github.com/openshift/installer/pkg/version"
 )

@@ -11,6 +12,8 @@ func SetMachinePoolDefaults(p *types.MachinePool, platform string) {
        defaultReplicaCount := int64(3)
        if platform == libvirt.Name {
                defaultReplicaCount = 1
+       } else if platform == powervs.Name {
+               p.Architecture = "ppc64le"
        }
        if p.Replicas == nil {
                p.Replicas = &defaultReplicaCount

Any updates here @23TNC?

LGTM
We need to compile a list of cleanup items and put those in a followup PR, then tackle the region removal, adding some more env vars, etc.


fix: skip close on bundles

Closing on each bundle will cause slowness in Bigtable read. In the future we'll migrate BigtableRead to SplittableDoFn to clean up the resource properly.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Discussed offline and let me remove the milestone for now, RC1 is targeted by the end of today


ENH: Adds new transformer for AMRFinderPlusAnnotationsDirFmt to qiime2.Metadata

closes #5
depends on #4

Adds new transformer from AMRFinderPlusAnnotationsDirFmt to qiime2.Metadata.
Now all output files by amrfinderplus actions can be used with metadata tabulate

Set up an environment
mamba create -y -n q2-amrfinderplus -c https://packages.qiime2.org/qiime2/2024.10/metagenome/staged/ -c conda-forge -c bioconda --strict-channel-priority qiime2 q2-types q2-feature-table q2cli q2-metadata ncbi-amrfinderplus

Run it locally

First, clone the repo and checkout the PR branch:

git clone https://github.com/bokulich-lab/q2-amrfinderplus.git
cd q2-amrfinderplus
git fetch origin pull/6/head:pr-6
git checkout pr-6
pip install -e .

Download test files
PR-102.zip

Test it out!

qiime metadata tabulate --m-input-file amr_all_mutations_contigs.qza --o-visualization amr_all_mutations_contigs.qzv
qiime metadata tabulate --m-input-file amr_all_mutations_feature.qza --o-visualization amr_all_mutations_feature.qzv
qiime metadata tabulate --m-input-file amr_all_mutations_sample_mags.qza --o-visualization amr_all_mutations_sample_mags.qzv
qiime metadata tabulate --m-input-file amr_annotations_contigs.qza --o-visualization amr_annotations_contigs.qzv
qiime metadata tabulate --m-input-file amr_annotations_feature.qza --o-visualization amr_annotations_feature.qzv
qiime metadata tabulate --m-input-file amr_annotations_sample_mags.qza --o-visualization amr_annotations_sample_mags.qzv

open them with

qiime tools view amr_annotations_sample_mags.qzv

Hmm I do not understand how that works.
I rebased it onto the commit ee66471 becuase thats the last one that the both branches share. I thought that was the way to go but that does not work.
should I rebase it on the newest commit on the other branch?

Ah, sorry, my comment was incorrect - you should rebase on top of main as we want to see the diff between this branch here and the main branch, without taking into account the other changes. If you think, though, that this branch and the one you started it from are too entangled, it may not work - in this case it will be easier to first merge the other PR and then rebase this one on top of main. Which do you think is easier?

Yeah lets merge the other one first. I cant get it to work with the rebase.

Hi @misialq
I tried to use the genome_dict that I created also here but realized that that doesnt work because in the AMRFinderPlusAnnotationsDirFmt there are suffixes in the filenames so the ids are not correct anymore.
Should i just create a separate genome_dict for AMRFinderPlusAnnotationsDirFmt or leave it like this?

Hey @VinzentRisch, could you please include some example of what the genome_dict returns now and what you want to get? I'm not sure I quite see what you mean 😞

Im sorry
So Amrfinderplusannotationdirfmt does not have a genome_dict function i just wanted to ask if you think it makes sense to add one that i can use it here for the transformer
But I mean for now maybe its okay without because I am not using the format as an input anywhere else.
Could you have a look at this today? then Liz can add the plugin to the distro tomorrow hopefully

No I'm gonna do the genome_dict anyway for the Amrfinderplusannotationdirfmt so I can use it here for the transformer.


fix(deps): update dependency query-string to v6.13.3

This PR contains the following updates:

Package
Type
Update
Change

query-string
dependencies
patch
6.13.2 -> 6.13.3

Release Notes

sindresorhus/query-string
v6.13.3
Compare Source

Fix the TypeScript type for the .stringifyUrl() method  b15f945

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

:tada: This PR is included in version 6.0.0 :tada:
The release is available on:

v6.0.0
GitHub release

Your semantic-release bot :package::rocket:


Fix: default value of rate-limit and unnecessary content of access logs.

Description
Fix the default value of sending 5 times requests in 1 minute to 60 times in 1 minute for the rate limit and refactor the access log message.
We rename the AuditLoggingMiddleware to AccessLogMiddleware, and change the module name from audit-log to access-log.
The access log message become to below format:
2022-09-12 10:15:26.997  INFO  [ACCESS_LOG:b995f7d9-9147-4456-b59c-e07b4d16d7a3] --> <IP_ADDRESS> -- "PUT http://ripe-celebration.org" -- size: 15.93kb
2022-09-12 10:15:26.998  INFO  [ACCESS_LOG:b995f7d9-9147-4456-b59c-e07b4d16d7a3]  -> header: {"X-Agent":"test-school-client"}
2022-09-12 10:15:27.000  INFO  [ACCESS_LOG:b995f7d9-9147-4456-b59c-e07b4d16d7a3]  -> query: {"sortby":"score"}
2022-09-12 10:15:27.001  INFO  [ACCESS_LOG:b995f7d9-9147-4456-b59c-e07b4d16d7a3]  -> params: {"uuid":"9d086934-3156-4e22-8faa-10f52ce1eefa"}
2022-09-12 10:15:27.002  INFO  [ACCESS_LOG:b995f7d9-9147-4456-b59c-e07b4d16d7a3] <-- status: 200 -- size: 85.2kb
2022-09-12 10:15:27.002  INFO  [ACCESS_LOG:b995f7d9-9147-4456-b59c-e07b4d16d7a3]  <- header: undefined

Commit Message

977e858 fix(serve): default value of rate limit and unnecessary content of audit logs

set default max to 60 in rate-limit middleware.
change audit-log middleware message format.
update test cases.

916da3e - chore(serve): rename audit-log to access-log

rename AuditLoggingMiddleware to AccessLogMiddleware.
rename option name from audit-log to access-log.
rename AUDIT to ACCESS_LOG in getLogger function.

Codecov Report
Base: 92.08% // Head: 92.06% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (916da3e) compared to base (8bb4910).
Patch coverage: 92.59% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #70      +/-   ##
===========================================
- Coverage    92.08%   92.06%   -0.02%
===========================================
  Files          215      215
  Lines         2980     2987       +7
  Branches       346      350       +4
===========================================
+ Hits          2744     2750       +6
- Misses         172      173       +1
  Partials        64       64

Flag
Coverage Δ

extension-dbt
97.43% <ø> (ø)

extension-debug-tools
98.11% <ø> (ø)

integration-testing
96.42% <ø> (ø)

serve
88.80% <92.00%> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...es/serve/src/lib/middleware/rateLimitMiddleware.ts
81.81% <85.71%> (-3.90%)
:arrow_down:

...es/serve/src/lib/middleware/accessLogMiddleware.ts
93.75% <93.75%> (ø)

packages/core/src/lib/utils/logger.ts
57.50% <100.00%> (ø)

packages/serve/src/lib/middleware/index.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Hi @oscar60310, The PR has been fixed, please check it, thanks!


Come up with json/file structure!

Come up with json/file structure!

Does following structure make sense for storing messages? @mattstrick
Structure
{
  "scopeOfMessages": {
    "messagesPurpose": {
      "messageId": {
        "en_US": "Message copy for en_US locale"
      }
    }
  }
}

Examples
{
  "emailValidation": {
    "info": {
      "emailExample": {
        "en_US": "Your email should look like<EMAIL_ADDRESS>        "ru_RU": "Ваш адрес электронной почты должен быть похож на<EMAIL_ADDRESS>      }
    },
    "error": {
      "invalidEmail": {
        "en_US": "Invalid email address",
        "ru_RU": "Неправильный адрес электронной почты",
      }
    }
  }
}

What about a label for a form element?  Or pluralization support?

why not : ) @deepeeess
i was thinking about making it so you can plug it in with i18next ex. https://www.i18next.com/translation-function/plurals for example
mostly i just want to have stepping stones for library of messaging that covers majority of use cases so one can stop breaking their head with how message should be worded.

also: good list of locales https://stackoverflow.com/a/28357857/201244

What would be a good list of things to cover besides forms, for quick project bootstrap?
Here what I have come up with forms for example, I think there are few other things to cover plus not sure this is right format. Do not group them too tightly but...
{
	“forms”: {
		“inputLabels”: {},
		“inputHints”: {},
		“accessibilityLabels”: {},
		“validationErrors”: {},
	}
}

Sent with GitHawk


Missing documentation for wnut_17 (ner_tags)

On the info page of the wnut_17 data set (https://huggingface.co/datasets/wnut_17), the model output of ner-tags is only documented for these 5 cases:
ner_tags: a list of classification labels, with possible values including O (0), B-corporation (1), I-corporation (2), B-creative-work (3), I-creative-work (4).
I trained a model with the data and it gives me 13 classes:
"id2label": {
    "0": 0,
    "1": 1,
    "2": 2,
    "3": 3,
    "4": 4,
    "5": 5,
    "6": 6,
    "7": 7,
    "8": 8,
    "9": 9,
    "10": 10,
    "11": 11,
    "12": 12
  }

  "label2id": {
    "0": 0,
    "1": 1,
    "10": 10,
    "11": 11,
    "12": 12,
    "2": 2,
    "3": 3,
    "4": 4,
    "5": 5,
    "6": 6,
    "7": 7,
    "8": 8,
    "9": 9
  }

The paper (https://www.aclweb.org/anthology/W17-4418.pdf) explains those 6 categories, but the ordering does not match:
1. person
2. location (including GPE, facility)
3. corporation
4. product (tangible goods, or well-defined
services)
5. creative-work (song, movie, book and
so on)
6. group (subsuming music band, sports team,
and non-corporate organisations)

I would be very helpful for me, if somebody could clarify the model ouputs and explain the "B-" and "I-" prefixes to me.
Really great work with that and the other packages, I couldn't believe that training the model with that data was basically a one-liner!

Hi @maxpel, thanks for reporting this issue.
Indeed, the documentation in the dataset card is not complete. I’m opening a Pull Request to fix it.
As the paper explains, there are 6 entity types and we have ordered them alphabetically: corporation, creative-work, group, location, person and product.
Each of these entity types has 2 possible IOB2 format tags:

B-: to indicate that the token is the beginning of an entity name, and the
I-: to indicate that the token is inside an entity name.

Additionally, there is the standalone IOB2 tag

O: that indicates that the token belongs to no named entity.

In total there are 13 possible tags, which correspond to the following integer numbers:

O
B-corporation
I-corporation
B-creative-work
I-creative-work
B-group
I-group
B-location
I-location
B-person
I-person
B-product
I-product


Update viz.py

When calling the functions or methods that have show_plot as argument, there should also be an option to save the plot to a file.
Milsent pls check it, I'm not sure,  that I haven't forgotten anything

Sure.
Just running short on time now, but I will definitely check it!
Cheers

I have merged as it is, but I will continue to add saving figus functionality in a separate branch called feature/save_fig.
Still some work to be done there.
I intend to allow the possibility for the user to provide other args to plt.save_fig(), such as dpi...


Enabling rendering on nbviewer by persisting state of anywidget in Jupyter Notebook

@manzt  and i have been experimenting with anywidget + @carbonplan/maps in https://github.com/manzt/carbonplan. The notebook works perfectly fine when connected to a live kernel. however, i am facing an issue with persisting the widget's state within the notebook and having external services like nbviewer render the initial state of the widget.
to illustrate the problem, i have shared a sample notebook at https://nbviewer.org/gist/andersy005/f8bca6c542135a75ab3e11203eada3a1. as you can observe, the last cell of the notebook is not being rendered.
any guidance on how to resolve this issue?
https://github.com/manzt/anywidget/assets/13301940/d4902396-7795-4de8-b37b-5a2a784b75bd
Cc @katamartin

Howdy! I'm a bit new to how widgets persist state in the notebook so for my own sake (and maybe others) I'm going to document what I see in the notebook. I can see the widget model in the output:

There is no other reference to the widget model in the source of the notebook and nothing for nbviewer to render (other than that text/plain entry).
When I try the simplest widget in JupyterLab, the IntSlider I'm also not seeing the persistence.

It looks like there's a setting that has to be enabled in JupyterLab to automatically save widget state

However, that doesn't render in notebook viewer. nbviewer.org/gist/rgbkrk/d11992979381414b32d18b16be2d64cf

I'm not sure how nbviewer is implemented, but nbconvert works for rendering widgets to HTML. https://github.com/flekschas/jupyter-scatter/issues/81#issuecomment-1638951995

@rgbkrk / @manzt, thank you for looking into this and suggesting workarounds. i was able to convert the executed notebook to HTML and then served it from an s3 bucket: https://carbonplan-share.s3.us-west-2.amazonaws.com/leap-demo.html

Awesome! @andersy005 if you use ipywidgets.jslink instead of ipywidgets.link you can also link the inputs on the client side (i.e., sliders and dropdowns will still work in the HTML-only version).

thank you very much, @manzt! the jslink changes work :) with the exception of any selection widgets (dropdown, radiobuttons, select, etc), and this appears to be a well known issue upstream:

https://github.com/jupyter-widgets/ipywidgets/issues/2382

however, i'm satisfied with dropping the dropdown widget for the time being: https://carbonplan-share.s3.us-west-2.amazonaws.com/leap-demo.html

Great! I'm going to mark this issue as resolved!

Ah, actually realizing this doesn't address that things don't seem to be working in nbviewer. Still need to look into that, but it doesn't seem like any widgets render with nbviewer.

Seems related:

https://github.com/jupyter/nbviewer/issues/987

@kdheepak this is pretty perplexing to me. I ran the notebook for myself locally and yes, it doesn't seem to work with quarto. However, the logs say that first it is looking for anywidget v0.9.* and then failing on anywidget v2.0.0.
I also am seeing different behavior in different browsers. My best guess is that this is somehow coming from quarto and how it has set up its HTML widget manager. I'm able to extract the widget metadata from the generated index.html, so the state looks like it is all there:
cat index.html | \
  sed -n '/<script type="application\/vnd.jupyter.widget-state+json">/,/<\/script>/p' | \
  sed '1d;$d' | \
  jq '.state |= map_values(del(.state))'
# {
#   "state": {
#     "0418f9b758f04d7889cfc9f4332f42b9": {
#       "model_module": "anywidget",
#       "model_module_version": "~0.9.*",
#       "model_name": "AnyModel"
#     },
#     "cacac92874f74116b4888983a3a056ac": {
#       "model_module": "@jupyter-widgets/base",
#       "model_module_version": "2.0.0",
#       "model_name": "WidgetModel"
#     },
#     "caf9424537bd43af875312b056669671": {
#       "model_module": "@jupyter-widgets/base",
#       "model_module_version": "2.0.0",
#       "model_name": "LayoutModel"
#     },
#     "e7509faff7254447995e244fd217a612": {
#       "model_module": "@jupyter-widgets/base",
#       "model_module_version": "2.0.0",
#       "model_name": "WidgetModel"
#     }
#   },
#   "version_major": 2,
#   "version_minor": 0
# }


[Event Request] Codeunit 5060 DuplicateManagement InsDuplContIndex - OnBeforeOnInsertSearchString

[IntegrationEvent(false, false)]
local procedure OnBeforeOnInsertSearchString(var Contact: Record Contact; DuplSearchStringSetup: Record "Duplicate Search String Setup")
begin
end;
As we need to fill some more Information in the "Cont. Duplicate Search String" table, because we differentiate between Person or Company Contacts, we need an event like mentioned above.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[Task]: Create a mongo db docker image that hosts the seed data for running local perf tests.

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
The developers should be able to connect local backend server to this docker image to get a setup with seed data for running local perf tests.
The mongo db docker image should have a snapshot of the seed data that is generated using seed data scripts in our load testing infra github repo.
The purpose of this docker image is to reduce the time needed to spin up a fresh db state with seed data. This will also ensure reusability of this database in other scenarios as well.
The docker image can be hosted on our dockerhub account

we will not be using grafana tooling anymore for telemetry or load testing so do not need this for now

This task is unrelated to grafana and new relic.


Revert "flutter_tools: Remove unused //ignore comments; change flutter-specific ones to "ignore_for_flutter""

Reverts flutter/flutter#45917
It broke forbidden_imports_test, which uses a custom ignore directive.
https://cirrus-ci.com/task/5320647412023296?command=main#L820

Landing tbr @jonahwilliams

LGTM


add Gradient Bandit agent

Hope I understood how we work with Git repo correctly :)

I dont think we have agreed on a very specific way when it comes to working with the repo 👀


AAA: Unban Aegislash

https://www.smogon.com/forums/threads/almost-any-ability.3587901/page-19#post-7518170
It was unbanned in September last year..... nobody unbanned it

Actually, TI unbanned it the same day in https://github.com/Zarel/Pokemon-Showdown/commit/c682712ca516f3c33a7e47a34dfb5119a9f1f94b and you forgot to unban it when you started inheriting the OU banlist in https://github.com/Zarel/Pokemon-Showdown/commit/86fbb45acce592ba9211ce0f1880caf1e2b2469c#diff-51750d8b764d10b745cbaea560131ce8R498. 😅

ah, must've been an oversight. whoops


Feat/cherry orchid farm

Updated satRgbFinal4.* and satRgbFinal4_1000ft.* files provided by @llhouse2.
The "broad" and "local" land-use proportions for this new location were added to the landcover-nj-2023.csv file.

@johnbradley did you want to pull and deploy this in the VM?


".NET regular expression source generators" - wrong code snippets in article for .NET 8

Type of issue
Other (describe below)
Description
This article seems to be a bit of a mess, at least when reading the .NET 8 "pivot".
https://learn.microsoft.com/en-us/dotnet/standard/base-types/regular-expression-source-generators?pivots=dotnet-8-0#inside-the-source-generated-files
https://learn.microsoft.com/en-us/dotnet/standard/base-types/regular-expression-source-generators?pivots=dotnet-7-0#inside-the-source-generated-files
In the section Inside the source-generated files of the article, it talks about two code examples.

"Consider the source generator output for one of the primary functions from the (a|bc)d expression:"

in the .NET 8 version of the article, it shows generated code for (abc|def)
I think here the article doesn't have any prose specific to the example, and the (abc|def) example would be fine too, if introduced as such? I'm unsure why there's these two differing source gen regexes being used

"Even when backtracking is involved, the structure of the backtracking becomes part of the structure of the code, rather than relying on a stack to indicate where to jump next. For example, here's the code for the same generated matching function when the expression is [ab]*[bc]:"

in the .NET 8 version of the article, it shows the generated code for (abc|def) again

"You can see the structure of the backtracking in the code, with a CharLoopBacktrack label emitted for where to backtrack to and a goto used to jump to that location when a subsequent portion of the regex fails."

all of this only exists in the actual [ab]*[bc] source of course 🐈

Page URL
https://learn.microsoft.com/en-us/dotnet/standard/base-types/regular-expression-source-generators
Content source URL
https://github.com/dotnet/docs/blob/main/docs/standard/base-types/regular-expression-source-generators.md
Document Version Independent Id
76b98e7b-6acc-cb43-da8e-475a8b4ed5a1
Article author
@IEvangelist
Metadata

ID: 56298202-fa5d-363d-3c52-adbf5256b184
Service: dotnet-fundamentals

@stephentoub could you please help me find the right person to take a look?

@stephentoub could you please help me find the right person to take a look?

Presumably @IEvangelist as its author?

Sorry, I've been a bit distracted lately with .NET Aspire content. I'll get to this ASAP.


Enable releasing from Github

We did it: https://github.com/gadomski/stac-asset/actions/runs/5224985565


AirBit Miner – HIP19 3rd Party Manufacturer Application

AirBit Miner want to join the production of Helium miner.
Our Application : https://github.com/airbitminer/HIP/blob/master/0019-third-party-manufacturers/airbit-miner.md
If you are interested in this project, please contact us :
Email<EMAIL_ADDRESS>

Thanks for your proposal, someone from the manufacturing committee will be in touch.

Waiting for DeWi Identity~
Please contact us if any question by<EMAIL_ADDRESS>

@Sophi Hello, we have send our DeWi KYC Identity to your email~

@airbitminer I haven't seen it, can you retry?

@Sophi We have try, and a new email has sent, if you haven't seen it, it maybe in spam folder, thanks ^_^


Remove custom full-drain event

Remove the full-drain event from the Buffer and instead close as appropriate internally. This event is currently only used to end() the buffer. This does not change any of the outside behavior otherwise.
While not used anywhere in React's ecosystem, this is still a BC break because others may actually rely on this event. However, this event has never been documented anyway, so chances are nobody will be affected by this BC break.

Ping @reactphp/core, this targets the upcoming v0.5.0 milestone :shipit:


Fixes after Weld 1.x -> 2.x upgrade

+1 to merge.
*** NOTE : DO NOT MERGE UNTIL ALL WELD RELATED PR's ARE APPROVED ***


Reformatting of code

The code could be reformatted for readability. Updates could include uniform line spacing and improved Pydoc comments for the functions.
The messages displayed to the user throughout the program could also be updated to be a little more clear and intuitive.

@connorkendrick  lets talk https://gitter.im/instantnews/Lobby


Raise a descriptive error when a required class is missing

Currently when Mockery has been configured not to allow mocking non-existent methods, attempting to mock a non-existent class gives a confusing error. It complains that it can't mock methods that don't exist on the specified class, but actually the error is that mockery can't find the referenced class to mock it and verify the methods it's allowed to mock.
This commit changes that behaviour so that attempting to mock a class that doesn't exist throws an exception that explicitly says that the specified class doesn't exist, however it only does so while mockery is configured not to allow mocking of non-existent methods.
I'm not entirely certain that this is the right way to make this change, so feedback is appreciated. Also appreciated is feedback on if this change is considered desirable; no hard feelings if it's not!

Coverage increased (+0.01%) to 76.021% when pulling df8d34dcfba269902075ab821899a494ce727674 on rbone:raise_descriptive_error_on_missing_class into 15dc926d6f805edbcd58d0c422590de5fd5533c4 on padraic:master.

Good enough for me, thanks!


Remove manual symbols in Crystal with automatic casting enums?

I'm personally not sure if there are other examples, but #6427 could be easily replaced with an enum listing all the colours instead.
This has several advantages:

if a user enters in a wrong colour, the error is a compile-time error, not a runtime error
the code for colorize can be much cleaner

However, this might require an overhaul of the entire module since a lot of code in that module is dependent on the symbols themselves - for example, this section of code:
{% for name in COLORS %}
  def {{name.id}}
    @fore = ColorANSI::{{name.camelcase.id}}
    self
  end

  def on_{{name.id}}
    @back = ColorANSI::{{name.camelcase.id}}
    self
  end
{% end %}

As said in the beginning, I'm not sure if there are any other examples of enums being manually duplicated as symbols within the compiler, but if they are I think they should be converted fully to enums as well, to conform with #6074.

I think that doing that would help to stress more the the auto casting of symbols to enum feature.
There are some corner cases that were reached in that PR and I think some intents to use the auto caste were reverted.
But I think the main usage of the autcasting for symbols to enum are in method calls, to avoid the user to type. The moment you try to assign it to variable, and pass it around it might do things harder to follow. Because, for example, in the opening comment snippet you might assume @fore is Symbol and not a ColorANSI.

I would personally revert the autocasting feature because it's incomplete (it breaks in the playground) and I can't think of a way to fix that.

@asterite the problem is how the playground instruments code, not in the feature.

@bcardiff What exactly do you mean by the last paragraph?

I don't know for the removal of automatic casting in general, but at least converting auto-magically from Symbol to Enum is confusing:

we don't see any Symbol overload in the API docs, but we can still pass one instead of an enum
if we pass a random Symbol, the compiler even tell use we can't use symbols:

Error in line 2: no overload matches 'MyClass.new' with type Symbol
Overloads are: - MyClass.new(num : SomeEnum)

this may be a backslash for incremental compilation

@j8r I would have to disagree with some of your points.
For your first point, the fact that there’s no documentation doesn’t matter - symbols are a shorthand in this case, a part of Crystal’s design as opposed to being an overload to a function. Documentation in this case would make more sense in something like the Crystal guidebook.
For your second point, this could be fixed by something like adding an extra section to the error message:
Instead of “:foo”, do you mean:

- :foop (Bar::Foop)
- :foob (Baz::Foob)
- :goo (Quux::Goo)

Another way this could be fixed is by using a special syntax for this type of shorthand, maybe something like :.foo or :foo:, but it’d increase the learning curve and implementation complexity of this feature.

How a beginner is suppposed to know this @Qwerp-Derp ? I find this feature to obfuscate code comprehension: the enum is replaced by a symbol: MyEnum::Val => :val – we lost MyEnum.
This yields to methods acception symbols instead of an enum.

This feature needs to be documented in the language reference. But only when we're certain about if and how it should stay. Right now, it should be considered experimental, hence the missing documentation. We want to try it out and see how it works while discussing additional features. For example there is a proposal to remove symbols entirely from the language, which would make the current symbol syntax exclusively used to enum autocasts.
@Qwerp-Derp Your idea to improve the error message sounds great. Would you mind opening a separate issue for that?

This yields to methods accepting symbols instead of an enum.

No symbol is ever accepted by this feature. It's only literal symbol syntax that is expanded to the full enum member name when it matches one that fits. It's completely type safe.

@straight-shoota If I've properly understood, SymbolLiteral could be replaced with a StringLiterral (even CharLiteral) then?


Trigger page rendered event

Does anyone know how to trigger page rendered event?
I want when certain page loaded and rendered succesful, and it will trigger an event for me to do extra steps.

You can listen for the pagerendered event and attach a handler; refer to https://github.com/mozilla/pdf.js/blob/f6bc93402ef35675ce61a278e3abe6b93d7fd393/web/app.js#L1287.


wrong control check for <IP_ADDRESS>

name: "SCORED | <IP_ADDRESS> | PATCH | (L1) Ensure Network security Configure encryption types allowed for Kerberos is set to AES128 HMAC SHA1 AES256 HMAC SHA1 Future encryption types"
win_regedit:
path: HKLM:\Software\Microsoft\Windows\Currentversion\Policies\System\Kerberos\Parameters
name: SupportedEncryptionTypes
data:<PHONE_NUMBER>
type: dword
when:
rule_2_3_11_4
tags:
level1-domaincontroller
level1-memberserver
rule_<IP_ADDRESS>
patch

Addressed with PR#24 and release 1.2.1


Update components-contrib.

Description
Update components-contrib.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close:
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Please run ‘go mod tidy’

Did run that already.


ASB lock renewal not isolated per subscription

Expected Behavior
When subscribed to multiple ASB topics, only messages belonging to that subscription entity should be renewed.
Actual Behavior
Currently lock renewal attempts to renew all locks on every subscription which leads to the following error:
time="2020-06-17T15:58:57.4641242+01:00" level=error msg="azure service bus error: error renewing active message lock(s) for topic B, %!(EXTRA common.Retryable=unhandled error link 6b69e1d7-9b98-43c0-8977-582d3bde1a71: status code 410 and description: The lock supplied is invalid. Either the lock expired, or the message has already been removed from the queue. Reference:860b9c70-97da-4438-a39b-a362860bb97a, TrackingId:50419ebc-96fb-42da-a30a-ff6c8451afac_B1, SystemTracker:daprdev:Topic:b|node-subscriber, Timestamp:2020-06-17T14:58:57)" app_id=node-subscriber instance=XPS15 scope=dapr.contrib type=log ver=edge

Steps to Reproduce the Problem
Subscribe to multiple topics and cause sufficient back pressure for message locks to require renewal.

I’ll fix this up ASAP please assign me

Assigned


Fix Get-SystemPreferredUILanguage spelling error

PR Summary
Fixes Get-SystemPreferredUILanguage being mispelled as Get-SystemPrefferedUILanguage.

Fixes #3279
Fixes #3532

PR Checklist

[X] Descriptive Title: This PR's title is a synopsis of the changes it proposes.
[X] Summary: This PR's summary describes the scope and intent of the change.
[X] Contributor's Guide: I have read the contributors guide.
[X] Style: This PR adheres to the style guide.

@tiburd
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


[libigl] update to v2.4.0

Library name: libigl
New version number: 2.4.0
Other information that may be useful (release notes, etc...)
https://github.com/libigl/libigl/releases/tag/v2.4.0
This version perfectly work on Mac ARM too so please enable the port on Mac ARM

Will be fixed by PR https://github.com/microsoft/vcpkg/pull/27963

Duplicate of issue: https://github.com/microsoft/vcpkg/issues/23354


Optional import "Configuration Data" broken in SP12 and SP14

Importing the optional import "Configuration Data" overwrites ItemTypes from previous packages, causing the folder navigator to not load with this error "Failed to get the tgc_Apply to Item Types ItemType".

@EliJDonahue Contacted Rolf, "The 'Configuration Data' part was more for backward compatibility.  Just mark is as “discontinued” new users should not need this."  We can update this in the readme/folder name to resolve this issue.


android devices not emitted by ScanBleDevices

I am using the example1 to list the available Bluetooth devices. But I am not getting android devices, Mac devices.
is there anything I am missing ??
Thanks in advance

Hey,
thanks for using the library!
For questions about usage please select one of the desired options

non-commercial — head to StackOverflow tag #rxandroidble
commercial — drop an email to<EMAIL_ADDRESS>for more info


deviceGroupApiUrl not defined in config file

Type of issue

[X] Bug
[ ] New feature
[ ] Improvement

Description
The deviceGroupApiUrl variable, used to request device groups, is used in the code but not defined in the config file.
Furthermore, this line when adding a group resolves to http://iotpcsdemo.azureedge.net/uiconfig/v1//api/v1/devicegroups/<new group name>, which seems wrong to me (is the device group saved under uiConfigApiUrl or deviceGroupApiUrl? Should there be a double slash (v1//api) in the URL)?
Steps to reproduce
Use device groups (e.g. create a group).
Current behavior
Failure to connect to the endpoint (404).
Possible solution
deviceGroupApiUrl should be defined in config.js. Endpoint to create devices should be fixed.
Context and Environment

Operating System: Windows 10
GitHub branch: master

The deviceList component has been removed. Closing.


Tagging contacts

I haven't reviewed the code yet, but I see CI failing. Can you please fix it first?


plugin-client-redirects: Support external URLs

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Description
We migrated a chapter describing the API from a plain markdown file into the project's source code API code comments. Then, we generated the API reference pages with the Solidity doc generator and host them on another domain (api.docs.oasis.io versus docs.oasis.io).
But, for backward compatibility, we always redirect any removed/renamed pages to a new URL using https://docusaurus.io/docs/api/plugins/@docusaurus/plugin-client-redirects. To make the redirect work in our case, we would need to use something like this:
        {
            to: 'https://api.docs.oasis.io/sol/sapphire-contracts/',
            from: '/dapp/sapphire/precompiles',
        },

, but this yields
[ERROR] ValidationError: "redirects[44].to" is not a valid pathname. Pathname should start with slash and not contain any domain or query string.

It would be nice to have support for redirecting to external websites. For now, we just use a placeholder file with "This chapter has beem moved to..." content.
Has this been requested on Canny?
No response
Motivation
See above.
API design
No response
Have you tried building it?
No response
Self-service

[ ] I'd be willing to contribute this feature to Docusaurus myself.

Please respect our issue template, mention the docusaurus version used and always try on the latest version.
3.0 supports this: https://github.com/facebook/docusaurus/pull/9171


Neymar's first issues

Neymar has started to write code, I hope everyone can exchange code and football skills...

eae menino Ney
tmj
joga muito

eae menino Ney
tmj
joga muito

soy,I can't understand your voice....


Deterministic encryption

Deterministic IV allows completely deterministic behavior that enables two things:

Complete set of test vectors to use in alternative implementations.
Black box auditability. A device that uses truly unpredictable numbers (e.g. random IV in AES-CBC, or random k nonce in ECDSA signatures) may potentially leak secret data via those numbers. Also, faulty RNG may lead to critical vulnerabilities. Fully deterministic behavior prevents these kinds of invisible backdoors or bugs.

Note that identical messages are having identical ciphertexts. Usually it's either not critical, or messages already include some sort of timestamps, sequence numbers or unique identifiers. However, if your protocol does not include unique identifiers within plaintext and it's critical to not allow someone to learn that two messages are identical, then you should pass Random.getRandomBuffer(16) (or better, an alternative deterministic IV) as a second argument to encrypt method.

Bitcore-compatible ECIES is implemented in CoreBitcoin, an Obj-C Bitcoin toolkit: https://github.com/oleganza/CoreBitcoin/commit/a92d38dc1bfd95de201bdd2186b024c2c7f6f4e6
The same test vectors are used.

:+1:

+1.  I will review with the team.

Sounds great! Thanks for the contribution!!
On Sun, Mar 29, 2015, 16:53 Eric Martindale<EMAIL_ADDRESS>wrote:

+1. I will review with the team.
--
Reply to this email directly or view it on GitHub
https://github.com/bitpay/bitcore-ecies/pull/29#issuecomment-87463247.

LGTM


Fix stray let keywords

As Phaser 3 is supposed to be in ES5 those should be vars.
Noticed when using webpack -p and it complained about them 😋

#3220, #3221


sources.yaml needs to be updated

https://github.com/base16-project/base16-builder-node/blob/master/lib/sources.yaml need to be updated and pointed to the base16-project repos, but those haven't yet been created. @belak what is the plan on doing that? Is it just a matter of creating the repo and bringing across the git history or is there more to it?

Spec 0.10.0 drops those as a concept. All the schemes were moved into https://github.com/base16-project/base16-schemes and builders are encouraged to focus on building a single templated repo at a time, removing the need for the *-sources repos.

Makes sense, thanks :+1:

Wait, I need to read 0.10.... so now we don't futz with git at all?  If they just need themes then I'd prefer to just have instructions telling them to how to fetch/update from git manually and let the builder do ONLY building.

Pretty much - the best way forward would be to either bundle the schemes with a release or provide a way to download them, but it's limited to just the schemes repo now (not the source repo plus separate repos for every single set of schemes and the same for templates). You could have it clone the repo to a common location for convenience.
Most template maintainers only want to deal with one template - there's no need to clone all the templates just to build their project.
It also has the nice side-effect of simplifying builders.

Most template maintainers only want to deal with one template - there's no need to clone all the templates just to build their project.

Fully agree with this one.


Keras Model Import Problem:  NullPointerException

Issue Description
When I try to import my pre-trained keras tensorflow model, I encounter a NullPointerException.
` String path = new ClassPathResource("my_model.h5").getFile().getPath();
MultiLayerNetwork modelAndWeights = KerasModelImport.importKerasSequentialModelAndWeights(path);`
I am not sure if it is related  #9524  and #8827
Version Information
Keras Version: 2.7.0
Tensorflow Version: 1.15.0
Keras backend: tensorflow
Relevant versions
dl4j-examples version: 1.0.0-SNAPSHOT
Platform information (OS, etc): Windows 10
Additional Information
Stack trace
Exception in thread "main" java.lang.reflect.InvocationTargetException 	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 	at java.lang.reflect.Method.invoke(Method.java:498) 	at com.intellij.rt.execution.CommandLineWrapper.main(CommandLineWrapper.java:66) Caused by: java.lang.NullPointerException 	at org.deeplearning4j.nn.modelimport.keras.layers.normalization.KerasBatchNormalization.<init>(KerasBatchNormalization.java:124) 	at org.deeplearning4j.nn.modelimport.keras.utils.KerasLayerUtils.getKerasLayerFromConfig(KerasLayerUtils.java:257) 	at org.deeplearning4j.nn.modelimport.keras.KerasModel.prepareLayers(KerasModel.java:225) 	at org.deeplearning4j.nn.modelimport.keras.KerasSequentialModel.<init>(KerasSequentialModel.java:110) 	at org.deeplearning4j.nn.modelimport.keras.KerasSequentialModel.<init>(KerasSequentialModel.java:57) 	at org.deeplearning4j.nn.modelimport.keras.utils.KerasModelBuilder.buildSequential(KerasModelBuilder.java:326) 	at org.deeplearning4j.nn.modelimport.keras.KerasModelImport.importKerasSequentialModelAndWeights(KerasModelImport.java:218) 	at org.deeplearning4j.examples.KerasImport.main(KerasImport.java:15) 	... 5 more
When converting BatchNormalization, the error occurred, it's details.

I am so sorry for my bad Englsih, can you help me solve the problem?
Thanks!

@rogerlee24 it looks like this is a separate namespace for model weights. Could you confirm how this model was buitl?

@rogerlee24 it looks like this is a fairly recent change to the file format: https://github.com/keras-team/keras/blob/master/keras/saving/hdf5_format.py#L668 I guess now the weights are saved in  their own group than with the layers. I'll have to look further in to this.

@agibsonccc
it's a simple model.

@rogerlee24 "it's simple" is not really what I'd be looking for. After a bit of digging it turns out that there is a change in the model format saved by keras that we'll need to handle. Luckily another user gave me a model and I was able to find the problem. I'll have a fix up in the next day or 2. No matter how simple a model is if keras changes its way of saving models a bug will occur :) try downgrading and seeing if that helps. I think this is an issue with keras 2.7.0 or above.

@agibsonccc  sry, I misunderstood what you mean. Thanks for your  guidance!

@rogerlee24 sure no problem. Generally when I mean mdoel, , I usually would want the model file so I can attempt to load it. When working with source code, I have to set it up in my ide, double check the configuration and may not even be running the right version.
It just creates a lot of work when trying to understand a problem. Thanks for your patience as I get this resolved.

Closing in favor of https://github.com/eclipse/deeplearning4j/issues/9524 - The OP did confirm this was the same issue.

I have faced the same problem and I added it to build the model. This solved my issue.
model = models.Sequential()


Replacing text_data with a translated string will being replaced again if its the same as another one

Replacing トウカイテイオー to 帝王 in name.csv

Then it replaces the translated 帝王 to Teio・Emperor in title.csv

Solution I think: can use the category id to determine the string location, a single csv file replaces only the string within a single category

we can use the category id to determine the string location, a single csv file replaces only the string within a single category

I've thought about this, but decided against it since it increases code complexity quite a bit and I'd like to keep simple.
I'll think about it and implement it when I have some time.
For now, as a workaround, you can keep a backup copy of the original master.mdb and run the tool only with the fresh file.


Unable to remove old/removed light from AL

If you remove a light entity from HA that was previously a part of Adaptive Lighting group, you can't remove it from the group, nor can you do any changes to the group (a "device" in HA implementation). Any change to the the group leads to User input malformed: light.[MISSING_ENTITY] is not a valid option for dictionary value @ data['lights']. You then have to delete the AL group/device and set it up again without the missing light entity.

Same, subscribing

Yep, been a thing for a while I've noticed.

Same here, I have repaired my Zigbee lights and renamed entities and now I can't remove the old ones

+1

To workaround the issue I joined a spare light to the group with the old entity_id and removed it from the configuration. That saved me a lot of tweaking to the new configuration.
Most friendly solution (IMHO) would be to populate the dropdown with all entries already in the configuration regardless of them being present. Optionally they could be highlighted in red or something to attract attention.

If you navigate to the integrations page, there is an "Adaptive Lighting" integration where it will list all groups. You can select the old ones and delete it from there.

If you navigate to the integrations page, there is an "Adaptive Lighting" integration where it will list all groups. You can select the old ones and delete it from there.

The configuration page for the doesn't list the old groups if the entity is gone from HA, that's the crux of this issue. You can't save the configuration if an entity is no longer valid, but you can't remove the entity from the configuration because it doesn't show up to be unchecked.

I left the following comment in #287 (since it seems to have received more attention):
This is the same as https://github.com/basnijholt/adaptive-lighting/issues/278 (which gives a much better description), https://github.com/basnijholt/adaptive-lighting/issues/236, https://github.com/basnijholt/adaptive-lighting/issues/220 and seems to have started with 2021.11's dropdown list (see https://github.com/basnijholt/adaptive-lighting/issues/213), which is addressed in https://github.com/home-assistant/frontend/issues/10608 - unfortunately without any obvious reaction for almost one year.

I've posted a PR to fix this issue.
https://github.com/basnijholt/adaptive-lighting/pull/350

Closed by #350 thanks to @danaues :tada:


Add sensitive data encryption (Without risk)

In different systems this libs can cause problems? Ex: Funtoo Linux or Slackware or Centos.
+/usr/lib/gcc/i586-suse-linux/4.5-xxxxxxx

I use Openssl.
Openssl function was integrated in php since 4.0.5 .
And if you want to port my work in the C parser, just use libssl ;)
Openssl is build in all linux !


Add fish instructions for setting the env vars

Minor request to help people using the fish shell to get this setup if they aren't familiar with their config file. This helps since the syntax for setting fish vars is different from POSIX compliant shells.

Yay, thanks! I know nothing about fish!

So far it is working in fish without a hitch for me. I'll be sure to open an issue/PR if anything regresses in that direction though.
@paulirish will be happy about this one. :smiley:

this is so rad. :D  and yay for fish compat! 💻 🐠  🐠


Remember GitHub authentication on Windows 10

I see that it is possible to remember the GitHub authentication on Linux (#2562). How would I do this on Windows?
There are no errors on the GitHub Authentication output, only info. It does state "Adding session found in keychain".
VS Code details:
Version: 1.57.0-insider (system setup)
Commit: 217f1a2f60a197807de78af362ecb832100fda96
Date: 2021-06-02T06:36:06.263Z
Electron: 12.0.9
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

@Fydon this should just work on Windows. A few questions:

How often are you being prompted to sign in?
Can you share a screen shot of where you're seeing the sign in prompt?
How are you launching vscode?
What version of the GitHub Pull Requests and Issues extension do you have installed?

Thank you for responding.

Every time I restart my PC
The same prompts from #2562. I could add some screenshots when I next restart if you'd prefer?
I'm launching from the Windows Start Menu
I'm using version v0.26.0 of GitHub Pull Requests and Issues

@Fydon just to further clarify: if you just close vscode and re-open it (no PC restart), you do not get the prompt?

@alexr00 Yes I found that a bit strange too, so I restarted and it appears to be remembering the authentication now.
I've been logging in every day for some time so I'm not sure what changed. Something I noticed is that I wasn't able to login through Accounts before (where Settings Sync is, image shown below). I would go through the Authorization process, but it would continue to show the 1 by Accounts. I would only be able to log in through the extension directly and it would only log in in one window. Now I can both log in through Accounts and logging in in one window updates all windows. Maybe this is because I'm using the Insider build of VS Code. Sorry to have bothered you with this.


Certificate issue

Good morning,
I've tried to execute the script but I receive the following error:
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in _run_code
File "/python-classdojo-archive/dojo_archive/cli.py", line 81, in
main()
File "/python-classdojo-archive/dojo_archive/cli.py", line 73, in main
run(config)
File "/python-classdojo-archive/dojo_archive/cli.py", line 50, in run
client.login()
File "/python-classdojo-archive/dojo_archive/client.py", line 60, in login
response = self.on_response(self.session.post(self.config.login_url, json={
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/python-classdojo-archive/venv/lib/python3.12/site-packages/requests/sessions.py", line 637, in post
return self.request("POST", url, data=data, json=json, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/python-classdojo-archive/venv/lib/python3.12/site-packages/requests/sessions.py", line 589, in request
resp = self.send(prep, **send_kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/python-classdojo-archive/venv/lib/python3.12/site-packages/requests/sessions.py", line 703, in send
r = adapter.send(request, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/python-classdojo-archive/venv/lib/python3.12/site-packages/requests/adapters.py", line 517, in send
raise SSLError(e, request=request)
requests.exceptions.SSLError: HTTPSConnectionPool(host='home.classdojo.com', port=443): Max retries exceeded with url: /api/session (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self-signed certificate in certificate chain (_ssl.c:1000)')))
make: *** [run] Error 1

Thank you for raising the issue. I can't reproduce it.
I could suggest to try a lower Python version, e.g. Python 3.9.
Although I tried running it with Python 3.12 as well and it was working.
The update on a stackoverflow post seem to suggest that it could be caused by configuration in /etc/ssl.

Closing due to inactivity


Hotfix to set restrict flag to false to mitigate downloadpackage extr…

…act failure
Task name: DownloadPackageV0/V1
Description:
Documentation changes required: (Y/N)
Added unit tests: (Y/N)
Attached related issue: (Y/N)
Checklist:

[ ] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

Do we understand the security scenario this break was intended to address so we know it's safe to default to un-restricted mode?

Do we understand the security scenario this break was intended to address so we know it's safe to default to un-restricted mode?

Still investigating on the security scenario. There was a sev2 last night, Edgar pushed out a hotfix to revert decompress zip version back to 0.3.0

Do we understand the security scenario this break was intended to address so we know it's safe to default to un-restricted mode?

Still investigating on the security scenario. There was a sev2 last night, Edgar pushed out a hotfix to revert decompress zip version back to 0.3.0

The original justification from the decompress-zip PR was "This has significant security implications when decompressing files from untrusted users." I don't think that's a concern for us. We can assume the person who controls the build definition has permission to place files anywhere on the agent

Do we understand the security scenario this break was intended to address so we know it's safe to default to un-restricted mode?

Still investigating on the security scenario. There was a sev2 last night, Edgar pushed out a hotfix to revert decompress zip version back to 0.3.0

The original justification from the decompress-zip PR was "This has significant security implications when decompressing files from untrusted users." I don't think that's a concern for us. We can assume the person who controls the build definition has permission to place files anywhere on the agent

Right, I am trying to repro a scenario where the check will fails.

I'm not convinced that's necessary. If it takes much time to repro it, I would just release this instead. We can be confident that if the agent allows files to be written to a path, it's acceptable for the build owner to do that.


Azure OpenAI API Keys don't seem to be working

Describe the bug
Azure OpenAI API Keys are not recognized.
To Reproduce

Configure Continue to connect to Azure OpenAI as below

   models=Models(
        default=OpenAI(api_key="--", model="gpt-4", server_info=OpenAIServerInfo(
            api_base="https://---.openai.azure.com",
            engine="--",
            api_version="2023-03-15-preview",
            api_type="azure"
        ))
    ),

Try to chat with it
Error: openai.error.AuthenticationError: Incorrect API key provided: XXX.

Happens with both the primary and secondary keys, I can only assume it's not connecting to the proper endpoint.
Setting the continue.OPENAI_API_KEY setting in VSCode as low-key suggested by the docs didn't improve things.
Expected behavior
I'm able to chat with the model :).
Environment

Operating System: MacOS Ventura
Python Version: 3.11.4
Continue Version: v0.0.305

@vladiliescu It turns out this was a mistake in our documentation, and the kwarg names should be openai_server_info instead of server_info. I've tried with the exact setup, saw that it failed, then made that change, and it worked. Updating documentation now. And giving a look to your other issue next!

Thanks @sestinj, I can confirm that it works!


chore(deps): bump graphql-lookhead from 1.3.0-beta.2 to 1.3.0

As per the title

:tada: This PR is included in version @graphql-gene/plugin-sequelize-v1.2.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[fix][starrocks] fix NPE

Purpose of this pull request
close #6524
in some source connector, it not set db name, so will get NPE here

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Could you add a test case for this?

@Hisoka-X in the current code, there has the NPE check https://github.com/apache/seatunnel/pull/6540/files#diff-ab5c2b43dd112473304501aa777993ab9cd746f3a12b6606f87523d92b5e9086R117. In this method i think the author is missing to check.

Since this is a bug, then we could have the means to reproduce it. It is best to use test cases to reproduce the bug, so as to ensure that there will be no regression in the future.

Thanks @liunaijie for update test case! I suggest use fake source connector to produce table with only table name. There are example https://github.com/apache/seatunnel/blob/cdfb5735acfbd281e01bf34393870fa573c5516c/docs/en/concept/schema-feature.md#table. It will be more simple and stable.

Could you remind check other connector like doris/jdbc/file/s3 too (open another pr is fine)? Thanks

Please fix ci

Could you remind check other connector like doris/jdbc/file/s3 too (open another pr is fine)? Thanks

I checked other connectors, they all have null value check.
But when i check the code, i find StarRocksCDCSinkIT is disabled, the disable reason is has some issue unfixed @Hisoka-X

Could you remind check other connector like doris/jdbc/file/s3 too (open another pr is fine)? Thanks

I checked other connectors, they all have null value check. But when i check the code, i find StarRocksCDCSinkIT is disabled, the disable reason is has some issue unfixed @Hisoka-X

Yes, could you try to reopen it?


Feature/Property Grid Filtering

adding feature to filter records on the property-grid component using FilteringPropertyDataProvider, as well as using HOC to support unified selection.

@aruniverse I think it should be.


Update vcpkg installation instructions

The connector libraries such as sqlite3 and mysql which can come with sqlpp11.
They are named as sqlpp11-connector-sqlite3 and sqlpp11-connector-mysql in vcpkg.
In order to make them more clearer to be used, I add these information to readme.
Related #313

Hi Roland,
I added these information just now. Could you please help review?
Any suggestion is welcome.
Thanks,
Nancy

Thanks!


Feature Request: const fields

If you have a const field
public const string Bar = "foo";

It would be great if it generated a not nullable enum with only one value ("foo"). Now it is ignored.
When OpenAPI.NET starts supporting version 3.1, it could be generated as a constant value https://github.com/OAI/OpenAPI-Specification/issues/1313
Also if there is some workaround on how to rewrite the string field/property to an enum with one value now, it would be appreciated if you shared this knowledge👍

Workaround is to create an attribute. Something like
[AttributeUsage(AttributeTargets.Property)]
public class SwaggerConstTypeAttribute : Attribute
{
    public SwaggerConstTypeAttribute(string enumValue)
    {
        Value = enumValue;
    }

    public string Value { get; }
}

Annotate your desired class
public class Foo
{
    [SwaggerConstType("const-value")]
    public string Type { get; set; }
}

Create SchemaFilter
public class SwaggerConstTypeFilter : ISchemaFilter
{
    public void Apply(OpenApiSchema model, SchemaFilterContext context)
    {
        foreach (var property in context.Type.GetProperties())
        {
            var schemaPropertyName = PropertyName(property);
            // This check ensures that properties that are not in the schema are not added.
            // This includes properties marked with [IgnoreDataMember] or [JsonIgnore].
            if (model.Properties?.ContainsKey(schemaPropertyName) != true)
            {
                continue;
            }

            if (property.CustomAttributes.Any(
                    attr =>
                    {
                        var t = attr.AttributeType;
                        return t == typeof(SwaggerConstTypeAttribute);
                    }
                ))
            {
                var constAttribute = property.GetCustomAttribute<SwaggerConstTypeAttribute>();
                model.Properties[schemaPropertyName].Enum = new List<IOpenApiAny>
                {
                    new OpenApiString(constAttribute.Value)
                };
                continue;
            }
        }
    }

   // if you are using camelCasePropertyName
    private static string PropertyName(MemberInfo property)
    {
        return new CamelCasePropertyNamesContractResolver().GetResolvedPropertyName(property.Name);
    }
}

And register the filter in
services.AddSwaggerGen(
    c =>
    {
        ...
        c.SchemaFilter<SwaggerConstTypeFilter>();
        ...
    }
);


where is the URDF

Hi, your work is really helpful. But I need the urdf path transfer parameter, could you please tell me where it is saved? I checked sit-packgs and did not find it.

The way to get the URDF path currently is to import the robot description as a submodule and get the path from it, for example:
$ python
Python 3.12.3 | packaged by conda-forge | (main, Apr 15 2024, 18:38:13) [GCC 12.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from robot_descriptions import jvrc_description
>>> jvrc_description.URDF_PATH
'/home/your_user/.cache/robot_descriptions/jvrc_description/urdf/jvrc1.urdf'

Unlike example-robot-data where all descriptions are packaged and installed to site-packages, in robot_descriptions the descriptions are cloned to a local cache at runtime. You can set the location of this cache via the ROBOT_DESCRIPTIONS_CACHE environment variable.


[Experimental]: Frontload docs support

Description
Attempt to add support for front loading MD/MDX docs before API docs. It is intended to work as follows:

Define beforeApiDocs plugin option as array of relative paths to MD/MDX docs to frontload
MD/MDX docs read in a processed, i.e. proper doc metadata is added
MD/MDX docs are processed through createData and generateSidebar

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate)

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes if appropriate.
[ ] All new and existing tests passed.

Currently having issues with getting versionMetadata right.


Add option of devshellOverlays for devshell

This issue-based on https://github.com/divnix/digga/pull/11/commits/88e55af866ae2916b6fbdb120262be2783f8fe9a
Sometimes, I would like to use some packages from overlay to deveshell packages.  So, I added this option here.
https://github.com/GTrunSec/digga/commit/d93852269b0e32ed00b0dd1c721c7c2fa7574b96
Exp:
   devshellModules = [ ./devshell.toml ];
   devshellOverlays = [ sops-nix.overlay ];
   #devShellOverlays= shareOverlays;

This makes sense.
We can work around it by not using devshell.toml but a more flexible devshell.nix and import that passing along self.pkgs.
On the other hand, I have to say, devshellOverlays is just quite convenient.
And if overlays is all one needs, we can stick with the (simpler) toml format.
we might consider, though:
devshell = { modules = []; overlays = [];};

devshell = { modules = []; overlays = [];};
+1, That would be good one.

I'm a little confused about this request. Any overlay added to your default channel(nixos) is added to the pkgs that your devshell is built on.
What is the goal for devshellOverlays?
I'm thinking:
devshell = { modules = []; overlays = []; externalModules = [ ]; };

I believe this would achieve the same thing:
channels.nixos = {
  overlays = [ sops-nix.overlay ];
}

and sops-nix should be in "part of" your devshell - as in not installed but available.
If that doesn't happen, then we should fix that.

To prevent the additional API argument I would prefer #22, but let me know if I'm missing a good warrant for devshell.overlays.

I had to look this up again:
The current devshell consumes nixpkgs, the only way to conform this to nixos is by using digga.inputs.nixpkgs.follows. But still, overlays of nixos are not applied.
We did this for the following reasons:

the digga lib by itself depends on nixpkgs for some things. In that sense it's not a pure lib.
those things have either to do with tests or with devshell.
we agreed on nixpkgs dependency on the digga level for only tests / devshell

I think we would want to uphold that conceptual separation by continuing to using nixpkgs.
On the other hand, a default channel is host specific configuration within the hosts namespace. I am a bit wary that it would be legit to semantically reuse it in another namespace (devshell).
Therefore, I think creating a devshell namespace is the most normalized form of implementing this. We might analyze though, if we have similar needs for tests:
When oberserving the maxime of dev/ci parity, it should always be possible that devshell is a wrapper around the ci, tests environment. That is any devshell overlays should be a superset of those eventually used for tests. Note, that tests currently only use pkgs.lib and build-support which is essentially lib-like, as well, no "real" dep on a package.
Hence, I think it is ok to reuse the devshell namespace for the pkgs that also gets supplied to tests, and in general to the entire pkgs-lib.

We already use the "default channel" for tests and shell. All I want to do in #22 is just preserve the overlays for that channel.
I disagree on separating the idea of a "default channel" between hosts and devshell. Then we would have to re-implement the concept of channels within devshell. channels are a global thing that should be used across digga.
Currently we pick the default channel by looking at hostDefaults.channelName which makes a lot of sense since channels used for hosts would be the channels you would likely want for your devshell. I would prefer to encourage the concept of the default channel across digga.
Now we could change that by adding a top-level defaultChannel argument. But I would like the "default channel" to be a concept preserved across hosts and devshell. Splitting it up is confusing for a user and in implementation.

Also devshell.overlays is anyways going to be applied to the default channel in any version of its implementation. The only way to prevent that is to re-create a channels concept for devshell.

I looked at rhe wrong code. :smile:
Look and feel of https://github.com/divnix/digga/blob/5ef9b8cabbc10c9b4fe5534107224c7241c63b3d/src/mkFlake/default.nix#L90
is good.
The distinction to digga standalone use is made in the way pkgs-lib get's pkgs in  flake.nix vs how it's invoked in mkFlake.

Thanks so much, the append overlay has been solved my problem.


[BUG] Datatable doesn't work in Laravel 9

Describe the bug
When using the x-adminlte-datatable tag in Laravel 9 my app crashes.
I get Illuminate \ Contracts \ Container \ BindingResolutionException:
Target class [JeroenNoten\LaravelAdminLte\Components\Tool\Datatable] does not exist.
Expected behavior
My table should appear.
Environment

Item
Version

Laravel
9.1.0

Project
3.8.0

OS
Ubuntu 20.04

PHP
8.1.2

You use a wrong namespace.
Please use:
JeroenNoten\LaravelAdminLte\View\Components\Tool\Datatable

Where do I change that? The only thing I've done is put the x-adminlte-datatable tag in my Blade template and it worked in Laravel 8, I've never put any namespace anywhere...

Oh, sorry. Seems the problem was in my compiled views. A simple artisan view:clear solved the problem, everything's working now again!


How to Set Time to Chinese Time

How to set time to Chinese time? I'm in China so I want to set the Beijing time.

The README config section specifies the time_zone = "America/New_York" flag.


Rename from_type() to sparse()

Consider:
    centrality = Vector.dense(FP32, n, 0)
    paths = Matrix.dense(FP32, ns, n, 0)
    frontier = Matrix.from_type(FP32, ns, n)

does renaming from_type to sparse seem more readable @szarnyasg @marci543?

I didn't have problem with the current name, but sparse seems to be more consistent and is closer to the MATLAB syntax. I think the change would be beneficial.

Agreed, I like sparse.

Done, this will be released shortly, I also renamed from_random to random.


Firebase Auth errors

Hi,
I am getting 4 errors when I try to run the firebase sample app.  I am using XCODE Version 9.2 (9C40b)
ios/Classes/FirebaseAuthPlugin.m:71:41: error: incompatible block pointer types sending 'void (^)(FIRAuthDataResult *__strong, NSError *__strong)' to parameter of type 'FIRAuthResultCallback _Nullable' (aka 'void (^)(FIRUser * _Nullable __strong, NSError * _Nullable __strong)')
            signInAnonymouslyWithCompletion:^(FIRAuthDataResult *dataResult, NSError *error) {

ios/Classes/FirebaseAuthPlugin.m:104:41: error: incompatible block pointer types sending 'void (^)(FIRAuthDataResult *__strong, NSError *__strong)' to parameter of type 'FIRAuthResultCallback _Nullable' (aka 'void (^)(FIRUser * _Nullable __strong, NSError * _Nullable __strong)')
                                 completion:^(FIRAuthDataResult *dataResult, NSError *error) {

ios/Classes/FirebaseAuthPlugin.m:132:37: error: incompatible block pointer types sending 'void (^)(FIRAuthDataResult *__strong, NSError *__strong)' to parameter of type 'FIRAuthResultCallback _Nullable' (aka 'void (^)(FIRUser * _Nullable __strong, NSError * _Nullable __strong)')
                             completion:^(FIRAuthDataResult *dataResult, NSError *error) {

ios/Classes/FirebaseAuthPlugin.m:189:43: error: incompatible block pointer types sending 'void (^)(FIRAuthDataResult *__strong, NSError *__strong)' to parameter of type 'FIRAuthResultCallback _Nullable' (aka 'void (^)(FIRUser * _Nullable __strong, NSError * _Nullable __strong)')
                                   completion:^(FIRAuthDataResult *dataResult, NSError *error) {

ios/Pods/Headers/Public/FirebaseAuth/FirebaseAuth/FIRAuth.h:431:63: note: passing argument to parameter 'completion' here
                       completion:(nullable FIRAuthResultCallback)completion;

flutter Doctor
[✓] Flutter (Channel dev, v0.5.7, on Mac OS X 10.12.6 16G1314, locale en)
    • Flutter version 0.5.7 at /Users/Jeff.Gardner/git/flutter
    • Framework revision 66091f9696 (3 weeks ago), 2018-07-09 12:52:41 -0700
    • Engine revision 6fe748490d
    • Dart version 2.0.0-dev.63.0.flutter-4c9689c1d2

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).
      If Android SDK has been installed to a custom location, set $ANDROID_HOME to that location.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✗] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[!] IntelliJ IDEA Community Edition (version 2017.2.5)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[✓] IntelliJ IDEA Ultimate Edition (version 2017.2.6)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 19.1
    • Dart plugin version 172.4343.25

[✓] Connected devices (1 available)
    • iPhone X • DA6CA3F5-26F8-4583-8FEE-40B86D284FBA • ios • iOS 11.2 (simulator)

pubspec YAML
name: firebase_auth_example
description: Demonstrates how to use the firebase_auth plugin.

dependencies:
  flutter:
    sdk: flutter
  firebase_auth: ^0.5.15
  google_sign_in: ^3.0.4

flutter:
  uses-material-design: true

Any ideas been struggling with this for a while.

Have you created your own firebase project with authentication?
Your googleAppID is different from the example?
I don't know in android but it requires SHA1.

Couldnt find the same error here.

Have you tried running pod update in ios/?

I did try pod update I do get an error saying can't connect to github when I run pod update however if I delete PODS and run pod install everything works fine.

I am only trying on IOS and I did create a new project and download the GoogleService-info.plist.

I was able to run pod update finally now I am getting this error
5.5.0 - [Firebase/Core][I-COR000003] The default Firebase app has not yet been configured. Add [FIRApp configure]; (FirebaseApp.configure() in Swift) to your application initialization. Read more: https://goo.gl/ctyzm8.
5.5.0 - [Firebase/Analytics][I-ACS023007] Analytics v.50100000 started
5.5.0 - [Firebase/Analytics][I-ACS023008] To enable debug logging set the following application argument: -FIRAnalyticsDebugEnabled (see http://goo.gl/RfcP7r)
Syncing files to device iPhone X...
*** First throw call stack:
(
0   CoreFoundation                      0x000000010cd9512b __exceptionPreprocess + 171
1   libobjc.A.dylib                     0x000000010c429f41 objc_exception_throw + 48
2   CoreFoundation                      0x000000010ce0a245 +[NSException raise:format:] + 197
3   Runner                              0x0000000109b0e438 -[GIDSignIn signInWithOptions:] + 242
4   Runner                              0x0000000109b0ae24 -[GIDSignIn signIn] + 64
5   Runner                              0x0000000109a69312 -[FLTGoogleSignInPlugin handleMethodCall:result:] + 2114
6   Flutter                             0x000000010a203d4a __45-[FlutterMethodChannel setMethodCallHandler:]_block_invoke + 118
7   Flutter                             0x000000010a21be3a _ZNK5shel<…>
Lost connection to device.

Sounds like you are missing https://github.com/flutter/plugins/blob/4acdcc1cdd73fd74fdadba452a37e708ae895731/packages/cloud_firestore/example/lib/main.dart#L12 but the app shouldn't crash from that

The fix for our project was to update the versions of any packages that depend on google_signin to their latest version to accommodate the signature change that those errors are complaining about.

After more struggling and following the tutorial I got it to generate the following error
MissingPluginException(No implementation found for method init on channel plugins.flutter.io/google_sign_in)
Any ideas how to solve that one?

I got it working I just added [GeneratedPluginRegistrant registerWithRegistry:self]; to my appDeligate and everything works fine now.

i got the same error.

@buttonsquared Seems the issue resolved for you? Are you ok if I close the issue?
@lihui2333 Do you still have issues? Could you create a separate issue and providing the details of your issue?

@buttonsquared
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/1121. Any further collaboration will be done there.


Cloudcashbox documentation

CloudCashbox documentation

Hey @Bluette-C-Riviere, two general thoughts:

Please use lower-case filenames with dashes
You'll need to add the newly added files to toc.js


New Adapter MoMa

Adapter for monitoring and maintenance

Here some small review feedback:

If on state/object/message change is not needed, remove it
you register a new Admin Tab called "Moma" but I can not find anything in that page ... AM I miss something or is it no Tab? Then please remove that from io-package
In readme you write it is also pssible to update stuff, but I could not find any code for this?

The admin tab is in place now and does update/reboot the machine

welcome


FFT at using lower sample rate audio input

There's project that uses Processing to analyze brain waves that I'd like to port over to use P5js but the frequency range of the brain waves I want to monitor are in the range of 8 - 30 Hz.  The current FFT function using 44 kHz audio input samples and limited to 1024 point FFT doesn't give much resolution down at that lower frequency range so I'd either need to do FFT's with significantly higher size (4096 or higher?) or filter and resample the audio input before it's sent to the FFT function so that it provides useful results at that very low range of frequencies.
My understanding is that the FFT function is limited to a max size of 1024 samples, is there a way to resample the audio input to a lower sample rate to, for example,  2.2 kHz rather than 44 kHz?

#617 is a similar question.
I dont know about resampling the audio input, but as I said in #617 I think you should be able to use values between 16 and 16384 for the p5.FFT bins.


Update styles.css

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

Before: https://main--gw22-aesthetics-scoring-franklin--chicharr
After: https://test-styles-remove--gw22-aesthetics-scoring-franklin--chicharr

hdjqdjqdq


No overcommit ratio config nor host resource reservation

Required information

Distribution: Ubuntu
Distribution version: 22.04
The output of "lxc info" or if that fails:

Kernel version: 5.15.0-67-generic #74-Ubuntu
LXD version: 5.12-c63881f
Storage backend in use: dir

Issue description
As far as I can see, LXD may not have an idea of setting a per-host overcommit ratio for CPU/memory/disk resources.
For example, it seems that one can create a big VM exceeding host resources as long as it's within the project quota.
Let's say:
Project A: limits.cpu=200 limits.memory=800GiB
Project B: limits.cpu=10 limits.memory=200GiB
and if Project A used almost all resources in the cluster, it looks like Project B can still create a VM with 200GiB on 128GiB host for example, and potentially can kill a VM from Project A or a host process based on OOM killer.
Steps to reproduce
As a minimal reproducer:

create a VM with bigger memory than a host:

lxc launch ubuntu:jammy test-vm-1 --vm -c limits.memory=64GiB on 32GiB system for example

check the size of recognized memory by the VM

lxc exec test-vm-1 -- free -h -> 64GiB

run a memory allocation test in the VM

lxc exec test-vm-1 -- stress-ng -m 1 --vm-bytes 50G --timeout 60

Then OOM will be observed on the host:
Apr 06 14:42:55 t14 kernel: Out of memory: Killed process 220836 (qemu-system-x86) total-vm:69043096kB, anon-rss:51520kB, file-rss:0kB, shmem-rss:22018520kB, UID:999 pgtables:44320kB oom_score_adj:0

Marking this as something we may consider looking into later.
If you need guarantees around CPU and memory, you want to use CPU pinning and hugepages, as anything else can indeed be quite seriously overcommitted. We're not likely to find a way to prevent or even calculate an overcommit amount. That's because while it's doable for VMs, it is not for containers and as you can have both, it makes the value ultimately useless.
That said, there are some obvious things that we likely should check prior to instance startup and block starting instances that we know will immediately cause an OOM situation or the like.

Similar to https://github.com/canonical/lxd/issues/8682


The default values for options and ports are not exposed through the Web API

From<EMAIL_ADDRESS>on July 10, 2012 18:01:50
What steps will reproduce the problem? 1. Start Pipeline 2 in local mode
2. GET http://localhost:8181/ws/scripts/daisy202-to-epub3
3. Inspect the "mediaoverlay" and "compatibility-mode" options What is the expected output? What do you see instead? The "mediaoverlay" and "compatibility-mode" options returned are:
<option desc="Whether or not to include media overlays and associated audio files (true or false)." mediaType="" name="mediaoverlay" ordered="true" required="false" sequence="false" type="boolean"/>
<option desc="Whether or not to include NCX-file, OPF guide element and ASCII filenames (true or false)." mediaType="" name="compatibility-mode" ordered="true" required="false" sequence="false" type="boolean"/>

They correctly have the @required="false" attribute set, indicating that the WS knows that a default value is present. However, the default values are not exposed.
It was agreed upon when talking about the [ http://code.google.com/p/daisy-pipeline/wiki/F2F201204_Oslo#Script_Metadata_Cleanup Script Metadata Cleanup] that the default values should be exposed, but I don't think we described concretely how. Maybe simply @default="defaultvalue"?
Original issue: http://code.google.com/p/daisy-pipeline/issues/detail?id=188

See https://github.com/daisy/pipeline-framework/pull/90


Uninstall and remove deprecated SMS Reportback modules

Once all SMS Reportback functionality has been moved out of Drupal, cleanup time.

[x] conductor
[x] conductor_sms
[x] dosomething_sms

Spoke with @jonuy -- we should be no longer sending SMS Reportback requests to the endpoints defined by the dosomething_sms module.
I'm going to disable all of the endpoints defined by DoSomething SMS via the admin configuration page, and we'll monitor to see if we're in a good spot to begin disabling, uninstalling, and removing these modules from the codebase.

This issue is ready for dev!

All modules are disabled and uninstalled on the international production.

Can we run this on US too?

Affirmative!

Done.


task: log experience & verbosity

TF_LOG_PROVIDER=INFO terraform {apply,refresh,destroy}

What is the cruft at the top?

mTLS stuff
[WARN] Truncating attribute path of 0 diagnostics for TypeSet:

Repeats Reading messages thrice (apply,refresh) and five times (destroy) (!)
Doesn't indicate how many steps

s#Reading DefaultVPC...#Checking status [1/10] DefaultVPC#
s#Creating KeyPair...#Creating [6/9] KeyPair#

Done! - what is done? There's a brief pause after this message so clearly not everything is "done"

2022-04-01T15:47:23.371+0100 [INFO]  provider: configuring client automatic mTLS
2022-04-01T15:47:23.467+0100 [INFO]  provider.terraform-provider-iterative: configuring server automatic mTLS: timestamp=2022-04-01T15:47:23.467+0100
2022-04-01T15:47:23.544+0100 [INFO]  provider: configuring client automatic mTLS
2022-04-01T15:47:23.648+0100 [INFO]  provider.terraform-provider-iterative: configuring server automatic mTLS: timestamp=2022-04-01T15:47:23.647+0100
2022-04-01T15:47:23.713+0100 [INFO]  provider.terraform-provider-iterative: 2022/04/01 15:47:23 [WARN] Truncating attribute path of 0 diagnostics for TypeSet: timestamp=2022-04-01T15:47:23.713+0100
2022-04-01T15:47:23.720+0100 [INFO]  provider: configuring client automatic mTLS
2022-04-01T15:47:23.810+0100 [INFO]  provider.terraform-provider-iterative: configuring server automatic mTLS: timestamp=2022-04-01T15:47:23.809+0100
iterative_task.cask: Refreshing state... [id=tpi-24l72zv7lgna7-5knn2kw7-2q66guxt]
TPI [INFO] Reading DefaultVPC...
TPI [INFO] Reading DefaultVPCSubnet...
TPI [INFO] Reading Image...
TPI [INFO] Reading Bucket...
TPI [INFO] Reading Credentials...
TPI [INFO] Reading SecurityGroup...
TPI [INFO] Reading KeyPair...
TPI [INFO] Reading Credentials...
TPI [INFO] Reading LaunchTemplate...
TPI [INFO] Reading AutoScalingGroup...
2022-04-01T15:47:55.780+0100 [INFO]  provider.terraform-provider-iterative: 2022/04/01 15:47:55 [DEBUG] AutoScaling Group State: running: timestamp=2022-04-01T15:47:55.780+0100
TPI [INFO] Done!
TPI [INFO] Reading DefaultVPC...
TPI [INFO] Reading DefaultVPCSubnet...
TPI [INFO] Reading Image...
TPI [INFO] Reading Bucket...
TPI [INFO] Reading Credentials...
TPI [INFO] Reading SecurityGroup...
TPI [INFO] Reading KeyPair...
TPI [INFO] Reading Credentials...
TPI [INFO] Reading LaunchTemplate...
TPI [INFO] Reading AutoScalingGroup...
2022-04-01T15:48:11.872+0100 [INFO]  provider.terraform-provider-iterative: 2022/04/01 15:48:11 [DEBUG] AutoScaling Group State: running: timestamp=2022-04-01T15:48:11.872+0100
TPI [INFO] Done!
TPI [INFO] Reading DefaultVPC...
TPI [INFO] Reading DefaultVPCSubnet...
TPI [INFO] Reading Image...
TPI [INFO] Reading Bucket...
TPI [INFO] Reading Credentials...
TPI [INFO] Reading SecurityGroup...
TPI [INFO] Reading KeyPair...
TPI [INFO] Reading Credentials...
TPI [INFO] Reading LaunchTemplate...
TPI [INFO] Reading AutoScalingGroup...
2022-04-01T15:48:24.994+0100 [INFO]  provider.terraform-provider-iterative: 2022/04/01 15:48:24 [DEBUG] AutoScaling Group State: running: timestamp=2022-04-01T15:48:24.994+0100
TPI [INFO] Done!
TPI [INFO] aws m in us-west
TPI [INFO] LOG 0 >> -- Logs begin at Fri 2022-04-01 14:35:09 UTC, end at Fri 2022-04-01 14:48:17 UTC. --
TPI [INFO] LOG 0 >> Apr 01 14:35:42 ip-172-31-29-139 systemd[1]: Started tpi-task.service.
TPI [INFO] Status: running 🟡
2022-04-01T15:48:26.708+0100 [INFO]  provider.terraform-provider-iterative: 2022/04/01 15:48:26 [WARN] Truncating attribute path of 0 diagnostics for TypeSet: timestamp=2022-04-01T15:48:26.708+0100

What is the cruft at the top?

mTLS stuff
[WARN] Truncating attribute path of 0 diagnostics for TypeSet:

Those messages can't be disabled: TF_LOG_PROVIDER is meant to print internal debug information for both the provider code and its gRPC connection with Terraform Core. See https://github.com/iterative/terraform-provider-iterative/pull/423#pullrequestreview-901140497 and https://github.com/iterative/terraform-provider-iterative/pull/467#discussion_r839961885 for context.

Repeats Reading messages thrice (apply,refresh) and five times (destroy) (!)

This can and should be fixed: see https://github.com/iterative/terraform-provider-iterative/pull/471#issuecomment-1084786646 for context.

Doesn't indicate how many steps

s#Reading DefaultVPC...#Checking status [1/10] DefaultVPC#
s#Creating KeyPair...#Creating [6/9] KeyPair#

This can be fixed, would you like a pull request?

Done! - what is done? There's a brief pause after this message so clearly not everything is "done"

That brief pause is not releated to infrastructure management. As soon as the Done! message appears, all the operations have been completed.

can we get rid of the unicode characters? Does not render well on default Ubuntu terminal

Yes!

Closing in favor of #487