Example notebooks

The data isn't actually in the repository, so this fails to execute. I think adding the 1k pums data sample you refer to would be fine.
It would be better to use the var_names at the top when specifying column names, and keys.
Your var_names have an extra pid column.
If trivially achievable, we should opt to use types that make this work on debug builds.
For now, explicitly mention that the input data should have no column names. This column metadata should go into an accompanying file in the data dir. Maybe something like public.json? This will make the "Notice the first element was 0" text outdated, as it won't be imputed as zero.
Your col_names vary across cells.
With the resize transform you can make the chain complete:

chain = (
    # Convert data into Vec<Vec<String>>
    make_split_dataframe(separator=",", col_names=var_names) >>
    # Selects a column of df, Vec<str>
    make_select_column(key="age", T=str) >>
    # Cast the column as Vec<Int>
    make_cast(TI=str, TO=float) >>
    # Impute missing values to 0
    make_impute_constant(0.) >>
    # Clamp age values
    make_clamp(20., 50.) >>
    make_resize_bounded(lower=20., upper=50., length=100, constant=20.) >>
    make_bounded_mean(lower=20., upper=50., n=100, T=float) >>
    make_base_laplace(scale=1.0)
)

res = chain(data)
print(res)

I pushed patches for all but 5, 6, 8, 9.

1. The data isn't actually in the repository, so this fails to execute. I think adding the 1k pums data sample you refer to would be fine.

2. It would be better to use the var_names at the top when specifying column names, and keys.

3. Your `var_names` have an extra pid column.

4. If trivially achievable, we should opt to use types that work on debug builds. Changing column names to var_names achieves this.

5. For now, explicitly mention that the input data should have no column names. This column metadata should go into an accompanying file in the data dir. Maybe something like public.json? This will make the "Notice the first element was 0" text outdated, as it won't be imputed as zero.

6. Your col_names vary across cells. There is a situation where an analyst only cares about the first k columns, so specifying a list of length k avoids loading up the rest of the columns after column k. Not sure if this is something you want in a basic notebook though, it seems super situational.

7. With the resize transform you can make the chain complete:

chain = (
    # Convert data into Vec<Vec<String>>
    make_split_dataframe(separator=",", col_names=var_names) >>
    # Selects a column of df, Vec<str>
    make_select_column(key="age", T=str) >>
    # Cast the column as Vec<Int>
    make_cast(TI=str, TO=float) >>
    # Impute missing values to 0
    make_impute_constant(0.) >>
    # Clamp age values
    make_clamp(20., 50.) >>
    make_resize_bounded(lower=20., upper=50., length=100, constant=20.) >>
    make_bounded_mean(lower=20., upper=50., n=100, T=float) >>
    make_base_laplace(scale=1.0)
)

res = chain(data)
print(res)

1. You're using the laplace mechanism, but not enabling floating point, which is probably why you have every invocation of the laplace commented.

2. I'm seeing a lot of repetition at the end of the basic data analysis file.

Thanks for all the comments and edits!
Regarding (1), I meant to ask if adding data to git was okay for the repo, thanks for going ahead with it. On (8), I thought I had enable_features("floating-point") at the top. Mentioned above, I must have done something in git that caused the repetitions at the end.


D(P)G elements support in MixedBilinearForm

This PR introduces support for D(P)G elements in mixed bilinear forms, i.e. discontinuous finite elements in forms with different trial and test spaces. It should be stressed that this is something different from the current trace integrators in mixed forms, which assume one of the spaces to be continuous, giving a trace face element directly. In particular, there are these changes:

New interior and boundary face integration in MixedBilinearForm.
Extension of DGTraceIntegrator for the new mixed DGs, including integral elements following #4088 .
New mixed elements integration in non-linear forms.
Methods for assembly of the face matrices for the mixed forms (Compute(Bdr)FaceElementMatrix()), including also the const qualifier following #4082 .
Methods for retrieval of the mixed face integrators from MixedBilinearForm (Get(B)FBFI())

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#4123
@najlkin
@tzanio
@sohailreddy + @tzanio + @psocratis
5/11/24
10/4/24
⌛due 6/1/24

PR Checklist

[ ] Code builds.
[ ] Code passes make style.
[ ] Update CHANGELOG:

[ ] Is this a new feature users need to be aware of? New or updated example or miniapp?
[ ] Does it make sense to create a new section in the CHANGELOG to group with other related features?

[ ] Update INSTALL:

[ ] Had a new optional library been added? If so, what range of versions of this library are required? (Make sure the external library is compatible with our BSD license, e.g. it is not licensed under GPL!)
[ ] Have the version ranges for any required or optional libraries changed?
[ ] Does make or cmake have a new target?
[ ] Did the requirements or the installation process change? (rare)

[ ] Update continuous integration server configurations if necessary (e.g. with new version requirements for each of MFEM's dependencies)

[ ] .github
[ ] .appveyor.yml

[ ] Update .gitignore:

[ ] Check if make distclean; git status shows any files that were generated from the source by the project (not an IDE) but we don't want to track in the repository.
[ ] Add new patterns (just for the new files above) and re-run the above test.

[ ] New examples:

[ ] All sample runs at the top of the example source file work.
[ ] Update examples/makefile:

[ ] Add the example code to the appropriate SEQ_EXAMPLES and PAR_EXAMPLES variables.
[ ] Add any files generated by it to the clean target.
[ ] Add the example binary and any files generated by it to the top-level .gitignore file.

[ ] Update examples/CMakeLists.txt:

[ ] Add the example code to the ALL_EXE_SRCS variable.
[ ] Make sure THIS_TEST_OPTIONS is set correctly for the new example.

[ ] List the new example in doc/CodeDocumentation.dox.
[ ] If new examples directory (e.g.examples/pumi), list it in doc/CodeDocumentation.conf.in
[ ] Companion pull request for documentation in mfem/web repo:

[ ] Update or add example-specific documentation, see e.g. the src/examples.md.
[ ] Add the description, labels and screenshots in src/examples.md and src/img.
[ ] In examples.md, list the example under the appropriate categories, add new categories if necessary.
[ ] Add a short description of the example in the "Extensive Examples" section of features.md.

[ ] New miniapps:

[ ] All sample runs at the top of the miniapp source file work.
[ ] Update top-level makefile and makefile in corresponding miniapp directory.
[ ] Add the miniapp binary and any files generated by it to the top-level .gitignore file.
[ ] Update CMake build system:

[ ] Update the CMakeLists.txt file in the miniapps directory, if the new miniapp is in a new directory.
[ ] Add/update the CMakeLists.txt file in the new miniapp directory.
[ ] Consider adding a new test for the new miniapp.

[ ] List the new miniapp in doc/CodeDocumentation.dox
[ ] If new miniapps directory (e.g.miniapps/nurbs), add it to MINIAPP_SUBDIRS in the makefile.
[ ] If new miniapps directory (e.g.miniapps/nurbs), list it in doc/CodeDocumentation.conf.in
[ ] Companion pull request for documentation in mfem/web repo:

[ ] Update or add miniapp-specific documentation, see e.g. the src/meshing.md and src/electromagnetics.md files.
[ ] Add the description, labels and screenshots in src/examples.md and src/img.
[ ] The miniapps go at the end of the page, and are usually listed only under a specific "Application (PDE)" category.
[ ] Add a short description of the miniapp in the "Extensive Examples" section of features.md.

[ ] New capability:

[ ] All new public, protected, and private classes, methods, data members, and functions have full Doxygen-style documentation in source comments. Documentation should include descriptions of member data, function arguments and return values, template parameters, and prerequisites for calling new functions.
[ ] Pointer arguments and return values must specify whether ownership is being transferred or lent with the call.
[ ] Any new functions should include descriptions of their intended use e.g. for internal use only, user-facing, etc., along with references to example code whenever possible/appropriate.
[ ] Consider adding new sample runs in existing examples to highlight the new capability.
[ ] Consider saving cool simulation pictures with the new capability in the Confluence gallery (LLNL only) or submitting them, via pull request, to the gallery section of the mfem/web repo.
[ ] If this is a major new feature, consider mentioning it in the short summary inside README (rare).
[ ] List major new classes in doc/CodeDocumentation.dox (rare).

[ ] Update this checklist, if the new pull request affects it.
[ ] Run make unittest to make sure all unit tests pass.
[ ] Run the tests in tests/scripts.
[ ] (LLNL only) After merging:

[ ] Update internal tests to include the new features.

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

@najlkin Is there an example we can test this?

Well, it originates from my fork of MFEM like many other things, so I definitely used that somewhere in past. Currently, I am relying on this branch in the HDG branch darcy-hdg-dev. It might look like a lot of new code, but actually there is not much new, just copied and slightly modified, so relatively safe. Specifically, compare additions to MixedBilinearForm::Assemble() with BilinearForm::Assemble() and the same for the Compute*FaceElementMatrix methods. Then with DGTraceIntegrator::AssembleFaceMatrix(), just compare the classical and this mixed version.

@psocratis , maybe look at ex5-hdg.cpp in darcy-hdg-dev. I am using new DGNormalTraceIntegrator there, but it is very similar. Actually, maybe it would be a good idea to include it here. What do you think?

@psocratis , maybe look at ex5-hdg.cpp in darcy-hdg-dev. I am using new DGNormalTraceIntegrator there, but it is very similar. Actually, maybe it would be a good idea to include it here. What do you think?

@najlkin , Can you provide a sample run? I tried ./ex5-hdg.cpp -m ../data/star.mesh with and without -dg and the results are different. Are they supposed to be the same?

@psocratis that is a good point 😄 , it seems the BCs are not handled generally enough. I should return to that in the HDG branch. Anyway, that is not related to this PR. What definitely works is a simple square domain, .e.g.: ./ex5-hdg -nx 20  😉 .

Folks, I have decided to remove the mixed elements methods in NonlinearForm, because there is no matching MixedNonlinearForm, so it perhaps should a part of a bigger effort to add that. Besides that, I will use a dedicated interface in #4350  for the HDG faces, so this I will not be needed for my purposes anyway. Being this said, I believe we can move forward with this PR :wink:.

I just removed virtual and added override as there is now consensus on it (see https://github.com/mfem/mfem/pull/4399 ) 😉 .

ping @tzanio

Merged in next for testing...


STT0050: update time format in catalog for each department.

Description
we need to change time format in catalog with a uniform standard format in catalog for each department. This will help us in calculation of audio duration uploaded and duration of audio still left to upload.
all the catalog sheets are here:
stt catalogs
Completion Criteria
All department catalog with standard time format to show audio duration.
Implementation
Subtask

[x]  create script to change time format and push the script to repo.
[x] update time format in NS department catalog.
[x]  update time format in NS_P department catalog.
[x]  update time format in TT department catalog.
[x]  update time format in CS department catalog.
[x]  update time format in MV department catalog.
[x]  update time format in AB department catalog.
[x]  update time format in PC department catalog.
[x]  update time format in HS department catalog.
[x]  update time format in NW department catalog.

running the script on all departments.

youtube time format and regex time format update done. drive link time format need to download audio to get timer.


Home assistant skill did not work until I twiddled the settings

Describe the bug
I paired my out-of-the-box mark ii to mycroft.ai, configured the home assistant skill. Everything was working. Then I flashed my Mark II with the dinkum sandbox image. After pairing the home assistant skill was not working. The intents "Turn off ..." were not being recognized.
I browsed to the HA skill under https://account.mycroft.ai/skills , and saw that it was configured. But it wasn't working.
So I twiddled the ip address to be a wrong value, saved, then edited it again back to the correct value.
This caused the Mark ii dev instance to pull down fresh settings, and then the HA skill worked out of the box.
Expected behavior
If I pair a new Mycroft, the already configured skills should be configured after pairing. I should not have to twiddle settings to force a skill config update.
Environment (please complete the following information):

Device type: Mark ii
OS: dinkum sandbox image
Mycroft-core version: 20221108-mark2_dinkum.img.gz

I've been wanting to connect my home assistant. I'll have to try this.


Anonymous user authentication not support

Unless I am mistaken it is not currently possible to use anonymous user authentication.
https://www.firebase.com/docs/web/guide/login/anonymous.html

This is a Firebase javascript web API feature that you linked to, not a Firebase REST API feature. As far as I can tell, this library only supports Firebase REST API features currently.

Ahh my bad :stuck_out_tongue_winking_eye:


Add exception counts to metrics

Want the metrics dictionary to store how many times each kind of exception occurred so that we get a clearer picture about how and why an approach is failing.

@ Rohan's question, I don't think anything has to change on the analysis/plotting side of things, since those scripts all have a list of keys to index into the metrics dict with, and these new items just aren't in that list of keys so they are otherwise ignored.


Remove Bluebird library

Change-type: minor

@karaxuna Don't mind the node16 check failure, I haven't migrated this library to be compatible with node16 yet. I'll open a draft PR in the main repo to test this branch's beta package. But from what I can see, things look good. This is making me think we should do drop Bluebird from other repos as well.

Everything passed in https://github.com/product-os/jellyfish/pull/7570: Results. LGTM 🚀


feat: add support for CHANGE_MODEL actions

Ensure all instantiated models are put in model cache by making it part of construction instead of explicit addition.
Add ConversationModelState to store which model is active for conversation.
Consolidate placeholder card renderings for actions that have no visual effect on dialog.
Fix what looks to be bug with not storing turnContext

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Font axis (ital, wght) are wrong

It looks like the Google Fonts Module is handling font axis in the wrong way. I'll give you an example:
'Roboto Condensed': {
  wght: [300, 400, 700],
  ital: [400],
},

This families definition generates the following path: https://fonts.googleapis.com/css2?family=Roboto Condensed:ital,wght@0,400;1,300;1,400;1,700&display=swap
wght and ital attributes are correctly assigned. BUT compared to Google Fonts Selection it should generate https://fonts.googleapis.com/css2?family=Roboto Condensed:ital,wght@0,300;0,400;0,700;1,400&display=swap
In my opinion this is a Google Fonts API issue. Seems like they flipped wght and ital attribute internally. If you open the given CSS file Roboto Condensed Bold is just generated for the italic style but not (like expected) for the normal one.

Hi @bart
Fixed on v1.1.5

As I said, I think this is more a Google Fonts issue than a plugin issue. But maybe "solving" this in your plugin prevents other people from having the same question like I had. On the other hand if they just copy it blind from Google Fonts frontend they might expect the exact same issue because then it's double flipped :)
Thanks for your effort!


jQuery: add March 2024 update

I had already submitted to ossf so it was not late, but I'll add that in an edit.

Added here: https://github.com/jquery/alpha-omega/pull/10
I put the note in the 2024 README, since I think it applies to all 2024 monthly updates.


All dotfiles are ignored accross the whole repository by default

We recently did a setup of Saleor as a clone of the repository. We then created our own SPA for the storefront within a subdirectory at the root level. We had multiple bugs causes by both inattention and the rules present in Saleor .gitignore file.
The rule that caused issues is the one on the second line (.*) which ignore all dot files across the whole repository. We ended-up having our own .eslintrc.js, .env, … files being ignored resulting in bugs on the CI for example.
I've removed this rule to fix our issues but did not see any other file appearing as unstaged in Git. Now I'm wondering what is the purpose of this global blacklist especially since all dotfiles are whitelisted explicitely right after this rule in the gitignore.
To avoid others falling in the same trap, I'd ask to reconsider these rules, either by removing them or at least by prefixing them with a slash to only target the root directory. I'd be happy to submit a MR once I got your opinion on that.

Many tools we use locally create their own dotfiles at the root level of the repository. It's easier to whitelist the selected few files that we do want to be tracked than it is to explicitly blacklist all of the other files.

I do believe it's a bit overkill to blacklist all the dotted files, even if there is a lot of cases where we don't want them.
I still do believe we should blacklist hidden (dotted) folders as most of the time they are create by IDEs–but iirc there are some cases where we want them, e.g. Elastic Beanstalk; in that case, one can simply force-add them. That shouldn't be an issue for deploying when the tool is using gitignore to ignore files (even if most of the time they provided their own file that we can add as well).

I like to think that editors and user-specific files should be ignored by the users themselves in their own global .gitignore. But I guess it's not something everyone know and having to deal with it every once in a while when people submit PRs is just wasting time.
Would you agree on prefixing all the existing rules with a slash then?

I have them ignored globally but I believe I'm in the minority here.


Orientate error when compression

When compressing an image on IOS ( my case ios 15 ) the orientate auto make vertical become horizontal and image has been cropped,

Still have the same bug in v1.0.17

Please provide the image, the device you use, the browser version and the steps to reproduce.

@Donaldcwl Version using: "browser-image-compression": "^1.0.12"
The problem is only on IOS, I check IPhone 6 (ios 12) and Iphone Xr (ios 15): the same issues

@Donaldcwl Version using: "browser-image-compression": "^1.0.12" The problem is only on IOS, I check IPhone 6 (ios 12) and Iphone Xr (ios 15): the same issues

The version in package.json file does not necessarily be the installed version according to NPM. Please console.log the version to see the actual version you are using:

@Donaldcwl Version using: "browser-image-compression": "^1.0.12" The problem is only on IOS, I check IPhone 6 (ios 12) and Iphone Xr (ios 15): the same issues

The version in package.json file does not necessarily be the installed version according to NPM. Please console.log the version to see the actual version you are using: console.log('imageCompression version', imageCompression.version)

I'm having the same issue. After logging it using imageCompression.version, I'm getting the version as 1.0.14 (which is the same as indicated by my package.json file).

@Donaldcwl Version using: "browser-image-compression": "^1.0.12" The problem is only on IOS, I check IPhone 6 (ios 12) and Iphone Xr (ios 15): the same issues

The version in package.json file does not necessarily be the installed version according to NPM. Please console.log the version to see the actual version you are using: console.log('imageCompression version', imageCompression.version)

I'm having the same issue. After logging it using imageCompression.version, I'm getting the version as 1.0.14 (which is the same as indicated by my package.json file).

Please upgrade to version 1.0.17 and try again.


Improper exception handling demonstration

On this page https://docs.microsoft.com/en-us/dotnet/api/system.windows.controls.contentcontrol.contenttemplateselector?f1url=https%3A%2F%2Fmsdn.microsoft.com%2Fquery%2Fdev15.query%3FappId%3DDev15IDEF1%26l%3DEN-US%26k%3Dk(System.Windows.Controls.ContentControl.ContentTemplateSelector);k(VS.XamlEditor);k(SolutionItemsProject);k(TargetFrameworkMoniker-.NETFramework,Version%3Dv4.7)%26rd%3Dtrue&view=netframework-4.7.1
There is an example with following code:
catch             {                 return null;             }
The behavior of this code does not match the expectations. For example, catched OutOfMemoryException or NullReferenceException do not mean we should provide empty Template. Better to rewrite that as
'catch(FormatException)
{
return null;
}'

Thanks @jinek, that's a good suggestion. Do you wanna try to submit a PR for this? Otherwise, we can handle it.

Yes, would be great. Let me do this.

@mairaw May I also refactor it a little bit, like removing Application.Current.MainWindow; and adding numberTemplate and largeNumberTemplate properties instead.
Also what I see is code does not match the behavior description. There is nothing said about cases when "null" template will be used. As I see it is better to remove some null returns instead of DataTemplate. If we want to explain some exception handling technique it's better to rethrow the exception ( https://github.com/jinek/docs/blob/52d3ef051f577fe16347082413a89d1dbb19ac6a/samples/snippets/csharp/VS_Snippets_Wpf/ContentControlTemplateSelector/CSharp/Window1.xaml.cs ) or not to catch it at all as exception handling is out of the example subject:

In this way (picture demonstrated) reader can get information more easily as it shows twice less code.

Feel free to refactor and will review your suggestions. Thanks @jinek!


EmptyDataError while load_vcf_as_df

I was trying to run iSAFE on a SNPEff annoted vcf, and get such error. Can you tell me where might go wrong?
Reference (REF) allele is considered as ancestral allele and alternative allele (ALT) is considered as derived allele. Strongly recommend to provide an ancestral allele file if it is available.
warnings.warn("Ancestral allele file (--AA) is not specified. Reference (REF) allele is considered as ancestral allele and alternative allele (ALT) is considered as derived allele. Strongly recommend to provide an ancestral allele file if it is available.")
Traceback (most recent call last):
File "/home/yinglu/.local/lib/python3.8/site-packages/isafe/bcftools.py", line 58, in load_vcf_as_df
df = pd.read_csv(StringIO(os.popen(cmd).read()), sep='\t', header=None)
File "/home/yinglu/.local/lib/python3.8/site-packages/pandas/io/parsers.py", line 610, in read_csv
return _read(filepath_or_buffer, kwds)
File "/home/yinglu/.local/lib/python3.8/site-packages/pandas/io/parsers.py", line 468, in _read
return parser.read(nrows)
File "/home/yinglu/.local/lib/python3.8/site-packages/pandas/io/parsers.py", line 1057, in read
index, columns, col_dict = self._engine.read(nrows)
File "/home/yinglu/.local/lib/python3.8/site-packages/pandas/io/parsers.py", line 2061, in read
data = self._reader.read(nrows)
File "pandas/_libs/parsers.pyx", line 756, in pandas._libs.parsers.TextReader.read
File "pandas/_libs/parsers.pyx", line 771, in pandas._libs.parsers.TextReader._read_low_memory
File "pandas/_libs/parsers.pyx", line 827, in pandas._libs.parsers.TextReader._read_rows
File "pandas/_libs/parsers.pyx", line 814, in pandas._libs.parsers.TextReader._tokenize_rows
File "pandas/_libs/parsers.pyx", line 1951, in pandas._libs.parsers.raise_parser_error
pandas.errors.ParserError: Error tokenizing data. C error: Expected 43 fields in line 9, saw 53
The number of fields varies as sample size changes.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/yinglu/.local/bin/isafe", line 11, in
load_entry_point('isafe==1.1.1', 'console_scripts', 'isafe')()
File "/home/yinglu/.local/lib/python3.8/site-packages/isafe/isafe.py", line 172, in run
df, dfreq, dfI, Need_Random_Sample = get_snp_matrix(chrom, region_start, region_end, args.input, args.AA,
File "/home/yinglu/.local/lib/python3.8/site-packages/isafe/bcftools.py", line 169, in get_snp_matrix
df = get_combined_vcf(chrom, region_start, region_end, case_vcf, cont_vcf=cont_vcf, case_IDs=case_IDs, cont_IDs=cont_IDs)
File "/home/yinglu/.local/lib/python3.8/site-packages/isafe/bcftools.py", line 83, in get_combined_vcf
df, sample_IDs = load_vcf_as_df(case_vcf, chrom, region_start, region_end, samples=samples)
File "/home/yinglu/.local/lib/python3.8/site-packages/isafe/bcftools.py", line 59, in load_vcf_as_df
except pd.io.common.EmptyDataError:
AttributeError: module 'pandas.io.common' has no attribute 'EmptyDataError'

Could you please send me a vcf file with this errors? The first error should be related to the vcf format and the second one is probably related to your python environment. In case you need my email<EMAIL_ADDRESS>

I noticed that I didn't filter all indels and something like   "*       TACAACAA" exists in my vcf file. This may be the vcf format  part.

This error were caused by using un-phased vcf as input or lack of genotype imputation. By using beagle 5.2 and convert vcf to .hap file. This error was avoided.


Updating to the latest .NET Core release

This PR updates all netstandard2.0 to netstandard2.1 and all netcoreapp2.1 -> netcoreapp3.0.

I'll merge this into the feature branch despite the failing build since the build will be restored with the PR containing the self-contained language server.


Add first-timers-only-app to website

Checklist

[x] If you're adding a listing for your app, be sure your app is in the /_apps/ folder before opening this Pull Request.
[x] All comments in frontmatter need to be removed.
[x] Performs a useful action through the GitHub API that solves an existing problem for developers: {explain the action here}
[x] Is original: for example, it does something not already done by an existing Probot app
[x] Has tests
[x] Has documentation
[x] Is open source
[x] Has a license
[x] Has a code of conduct
[x] Has someone willing to at least minimally maintain them for the near future: {your maintainer's name}

@hiimbex I made the changes! Let me know if there is still any error.


Tag search/autocomplete

Is your feature request related to a problem? Please describe.
Currently, users have to scroll through a potentially long list of tags to find the ones they wish to select.
Describe the solution you'd like
A search-box where the list of tags is filtered by the content of the search box as you type.
Describe alternatives you've considered
None
Additional context
Inherited from #31

Resolved in e74fee31cb8c5a46141dfb30b85f247c92b3f048, albeit without auto-completion.


[firebase_dynamic_links] Have OnLinkErrorException extend PlatformException

Firebase dynamic links creates a new error object. This should extend PlatformException (it effectively carries the same signature) so that it can be thrown by the app if needed.
class OnLinkErrorException extends PlatformException {
  OnLinkErrorException._(String code, String message, dynamic details):
      super(code: code, message: message, details: details);
}

Fixed with #15


Dev high order component

According to #300 , use high order component to replace mixin.
I only modified createContainer but remain composeWithTracker.js unchanged, but I think composeWithTracker.js, Mixin.js and connectMeteor API can be removed in the future and also remove react-komposer and react-mixin dependencies.
By the way, I could not test the modification through the example in this project, so I ported the example with create-react-native-app and added the build process with babel to test.

OK, I will fix the typo and send another PR to makes the commit more clear.
By the way, I tried to move componentWillMount to componentDidMount  and componentWillUpdate  to componentDidUpdate and got some error. I think I should take some time to trace how the Tracker trigger the rendering process in React

Thank you for your fantastic work!
I hope you don't mind that I've fixed the typo directly in your branch.
I've also QA this inside our application, and everything looks fine.

Thank you


Card Series Year

Describe the solution you'd like
With the original XTC plugin, changing the series of a card only updates all future cards.  Currently existing cards retain their prior series (unless added to a deck) and can still be used to complete the sets they are for.  Aside from ensuring a similar functionality in your fork, I would also like to suggest a separate NBT value for the card's year.
The resource pack I am developing for the plugin just changes the card's back texture each year.  I also am planning on making holiday series cards, but the normal schedule function you added should work nicely for that.  With the holiday series, the front textures will be different from normal but the back I would like to have the year's card back.  The reasoning is that some players collect the cards and seem to enjoy trying to complete all sets each year, not just turn them in for the reward.
So a card might have:
isYearly=true
which sets an NBT value of year=current.year
Then card can be a part of the default, Halloween, etc. series and still get the year's back texture.
Describe alternatives you've considered
Currently, this requires a global search and replace for the year in the config at the turn of each new year.
Additional context
Add any other context or screenshots about the feature request here.
Not important to the discussion, but here is a prototype I made for a Valentine's Day series card

I'm not really sure about this one.
We could just create a different series for every year. i.e. halloween2022, chrimstas2022 etc.
Perhaps we could add an "additional nbt" feature that allows you to add whatever nbt key values you want.

Apologies for the delayed reply, life happened.
Changing series each year was the manual option I mentioned above, which is what server admins currently need to do to see the new card backs.
I was hoping for an auto-incrementing year, either in Lore or as a hidden NBT tag. I suppose I could work out a BASH script to run as a cronjob parse the config and change the year using sed, but this might be a bit difficult for other server admins to implement. The only issue then would be ensuring that the Deck database tracks the original series data and doesn't update old cards to the new series designated in the config.

Closing the issue for now, if I decide to implement this I will re-open this issue.


[IO-711] Use Objects.requireNotNull for fail fast method/constructors

Fixes https://issues.apache.org/jira/browse/IO-711

@garydgregory Please review and mark the cases you don't want to change for compatibility and/or design reasons. I'll change then all occurrences accordingly.

@garydgregory Please review and mark the cases you don't want to change for compatibility and/or design reasons. I'll change then all occurrences accordingly.

@garydgregory Please review and mark the cases you don't want to change for compatibility and/or design reasons. I'll change then all occurrences accordingly.

Hi @boris-unckel
Sorry but I don't want to take the time to review each change in each of 31 files. If you do not want to revise the PR to minimize changes and duplication of null checks, that's fine as well. I might get to these kinds of changes later. Thank you for your contributions so far though :-)

@garydgregory Please review and mark the cases you don't want to change for compatibility and/or design reasons. I'll change then all occurrences accordingly.

Hi @boris-unckel
Sorry but I don't want to take the time to review each change in each of 31 files. If you do not want to revise the PR to minimize changes and duplication of null checks, that's fine as well. I might get to these kinds of changes later. Thank you for your contributions so far though :-)

@garydgregory

Fixed the formatting (is there a official eclipse configuration you can recommend?)
Removed single parameter checks in case of direct usage,
Kept single parameter checks when prior (different) Object creation happens (see case above)
Kept multiple parameter checks
Removed double checks in constructors, even if objects are created
Kept all checks in constructors, where fail will be in post constructor usage of methods

@garydgregory

Fixed the formatting (is there a official eclipse configuration you can recommend?)
Removed single parameter checks in case of direct usage,
Kept single parameter checks when prior (different) Object creation happens (see case above)
Kept multiple parameter checks
Removed double checks in constructors, even if objects are created
Kept all checks in constructors, where fail will be in post constructor usage of methods

Coverage increased (+0.1%) to 89.121% when pulling cb1751782b038c10b4219fa5208cfeccfc627634 on boris-unckel:utilize_objects_requirenotnull into 8665a70d1ea00939d12c7a22ff6c8a8124924acd on apache:master.

Coverage increased (+0.1%) to 89.121% when pulling cb1751782b038c10b4219fa5208cfeccfc627634 on boris-unckel:utilize_objects_requirenotnull into 8665a70d1ea00939d12c7a22ff6c8a8124924acd on apache:master.

@boris-unckel
May you please rebase on master?


cyber_launch relative_map failed

Describe the bug
When I run cyber_launch start /apollo/modules/map/relative_map/launch/relative_map.launch, I received the following error. I was working this issue for 2 days, please help me out.

I installed the cyber-dev and map-dev by buildtool install cyber-dev and buildtool install map-dev

Desktop (please complete the following information):

OS: [ubuntu 18.04]
Browser [firefox]
Version [apollo8.0]

Try to change this build file
You must add     alwayslink = True, two times.
And also rebuild project.
cc_library(
name = "navigation_lane_lib",
srcs = ["navigation_lane.cc"],
hdrs = ["navigation_lane.h"],
copts = MAP_COPTS,
deps = [
"//cyber",
"//modules/common_msgs/localization_msgs:localization_cc_proto",
"//modules/common_msgs/map_msgs:map_lane_cc_proto",
"//modules/common_msgs/perception_msgs:perception_obstacle_cc_proto",
"//modules/common_msgs/planning_msgs:navigation_cc_proto",
"//modules/common/math",
"//modules/common/util",
"//modules/common/vehicle_state:vehicle_state_provider",
"//modules/common/vehicle_state/proto:vehicle_state_cc_proto",
"//modules/map/relative_map/common:relative_map_gflags",
"//modules/map/relative_map/proto:relative_map_config_cc_proto",
"@com_google_absl//:absl",
],
alwayslink = True,
)
cc_library(
name = "relative_map_component_lib",
srcs = ["relative_map_component.cc"],
hdrs = ["relative_map_component.h"],
copts = MAP_COPTS,
deps = [
":relative_map_lib",
"//cyber",
"//modules/common/adapters:adapter_gflags",
"@com_github_gflags_gflags//:gflags",
],
alwayslink = True,
)

It works! cheers!

@AlexandrZabolotny sorry to bother. When I run cyber_launch start /apollo/modules/map/relative_map/launch/relative_map.launch, I got "invalid localization input" error. Then I checked /apollo/localization/pose channel, it has "pose" data, so I have no idea what‘s wrong. Can u help me?
![20230228-140141](https://user-images.githubusercontent.com/38745125/221767671-68e04d84-5220-4f

d1-a3ee-54c63691ad6a.png)

@AlexandrZabolotny sorry to bother. When I run cyber_launch start /apollo/modules/map/relative_map/launch/relative_map.launch, I got "invalid localization input" error. Then I checked /apollo/localization/pose channel, it has "pose" data, so I have no idea what‘s wrong. Can u help me?


Building plugin

With the 5.1.0 release we are introducing FillExtrusionLayer. One of the features possible with this, is showing buildings on top of the map by using OSM data. Atm, integrating this feature into your map requires developers to write boiler plate code. Similar to our Traffic plugin, I'm proposing to have a simple facility API so developers aren't required to hook into the runtime API and can show buildings on a map by just calling a single method.
Note that Google however exposes this functionality directly in their SDK. Since this feature for us would be a high level implementation on top of a low level implementation. I'm suggesting putting it into a plugin instead.
WIP can be followed in https://github.com/mapbox/mapbox-plugins-android/pull/26
cc @mapbox/android @pveugen @ericrwolfe

This landed


Add os specific config.spire.steamdir test

Added test for default steamdir on windows and linux, with intentional failed test on any other system.

This test will fail on linux as the config file only includes the windows path.

Woo!  All tests passing!


how to use in vue touer?

https://github.com/SimulatedGREG/electron-vue

I don't use Vue and cannot provide info how to integrate it with any kind of boilerplate solution. This module serves the files as if they were served by a server and not file://, which enables actually changing the URL instead of using hash. How to integrate it into electron-vue is really up to you to figure out. You could try asking on the electron-vue issue tracker.


Update fetch to 5.7.7

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

note the DL has a habit of timing out (it always evades my script because of this)


Fix UnbouldLocalError and let {h,v}hea be no-op when no {v,h}mtx is present

I believe that was the spirit of this code, do have recalc() do nothing if the metrics tables are missing
Fixes https://github.com/fonttools/fonttools/issues/3289

I wonder if fontBuilder should issue a warning (or even an error) if one attempts to call setup{Horizontal,Vertical}Header before setup{Horizontal,Vertical}Metrics?

or maybe it's not about the order in which the methods are called but the requirement that both Header and Metrics methods are called before compiling the font (which is when the recalc() is attempted)

Yes, I also don't think it's about the order. I'm not sure how fontbuilder could enforce such things.

I'm not sure how fontbuilder could enforce such things.

yeah, maybe just documenting it better would work


New feature: wrangle query sizes

Here's how database queries work right now, as pseudocode:
def start_queries(repo_list, query_list):
    for query in query_list:
        run_query.delay(repo_list, query)

@celery_task()
def run_query(repo_list, query):

    // runs query on Augur db for each repo in repo_list at once
    data = postgres(query=query, "where repo_id in {repo_list}")

    for repo in repo_list:

        // cache the data per-repo in Redis, keyed by repo_name+query_name hash
        redis.cache(hash(repo.name + query.name), data[repo_id == repo])

In plain english, len(query_list) tasks are scheduled to run, each getting data for all considered repos at once. If the data for all repos is too large for openshift to handle, the celery worker handling this task is killed.
We see the effects of this architecture in these issues:
#440
#450
#429

We could arbitrarily break queries into smaller subsets of repos, but that wouldn't necessarily fix the problem if we hard-coded something like "collect the data for 20 repos at a time" because the query itself could be very large, and 20 repos would have a lot of data anyway.
The ideal solution is to know how much data will be returned by a query, per repo, and break up the query greedily into something like 256MB chunks of repos, and if a repo itself accounted for more than 256MB, to paginate that query into smaller chunks.

The pseudocode for possible perfect solution would look like:
def start_queries(repo_list, query_list, size_requirement):
    for query in query_list:
        sublists_that_fit = find_sublists(repo_list, query, size_requirement)
        for sublist in sublists_that_fit:
            run_query.delay(sublist, query, paginate=(len(sublist)==1))

def find_sublists(repo_list, query):
    // ask Postgres for the size of the result per repo in repolist,
    // greedily package repos together if the sum of their result size
    // is below some configured threshold.
    // IF single repo result size > threshold, make it its own sublist.

    return sublists

@celery_task()
def run_query(repo_list, query, paginate=False):

    if paginate:
         // query and store data in pages
         return

    // runs query on Augur db for each repo in repo_list at once
    data = postgres(query=query, "where repo_id in {repo_list}")

    for repo in repo_list:

        // cache the data per-repo in Redis, keyed by repo_name+query_name hash
        redis.cache(hash(repo.name + query.name), data[repo_id == repo])

Unsure if the perfect solution would be tenable, would also love to find a better solution than current "perfect" solution.

@JamesKunstle How many repos do you normally expect iin a repo_list?
Even if the queries are small for a single repo I would consider creating a single celery task for each repo. Those jobs would be distributed to workers. Unless there is a known issue of overhead for celery jobs. I would start with that.
Then each repo query function would check the total number of records available and have a predefined limit to cause pagination (ex: 100k).  Then the parent repo query could create celery sub tasks for each set of paginated records.
Does that make sense?

@codekow Yeah that makes sense. We could pack multiple repos into workers scheduled with 128MB or 64MB of ram though. Maybe that's unnecessary optimization in a relatively resource-abundant setting, though.

Here's an example of how to use streaming to query a result from database and chunk the result
so that the memory usage of the querying worker is always relatively low. I think this is slower than
querying all at once, but it's far more resilient to OOM.
Chunksize is a tunable hyper-parameter that we can mess with based on the memory we want to
schedule a worker with.
The chunk_dataframe is treated as an iterable, so the final result could be fewer than chunksize
rows.
connection_string = "..."

engine = salc.create_engine(
    connection_string
)
conn = engine.connect().execution_options(
    stream_results=True)

for i, chunk_dataframe in enumerate(pd.read_sql(query="...", conn, chunksize=10000)):
    cache.write_rows(table=table_name, source=chunk_dataframe)

If you can get the total size of the query then you can split the whole query into chunks and schedule those chunked queries on the worker pods. Horizontal auto scaling should help do the rest for big queries.

Solution: query pagination from Augur, storing data in a Postgres cache.


Send summary run stats to stakeholders after run processing

For every run, after processing, email all the invitees for the Re: [Weekly call #A] Sequence and run QC meeting to provide, for the run:
fraction/percent that passed depth-gte-10 criterion
fraction/percent that are accepted
(I suggest also adding number that passes indels)

Check urgency on this -- I can see how it would be nice to have, but I don't think it should block data release etc. Would be good to know if there is a specific use case and if it has to be a push email rather than users pull from directory known to all stakeholders (which is much easier).

I think this is now obsolete


Entering a locker and buckling makes you teleport to 0,0 and enter noclip

Steps to reproduce: Spawn a locker and a chair next to it, enter the locker and close it, click the chair.

you are welcome

Before I get banned from your GIThub: Putting yourself into a locker and destroying it makes you invincible


last_locked should be unsigned

That is the only change, since every other index is unsigned, but it is signed in the one case where it doesn't need to be.
The rest of the function shall be left alone.

cc @etcwilde

cc: @compnerd

While this change is not wrong, it really isn't doing anything useful here either. I'm closing this PR.
Please only make PRs which are meaningfully fixing something that is broken.


Incorrect log-spaced sampling

In Generator1D, when the method is set to log-spaced or log-spaced-noisy, the sampling domain actually goes from 10^t_min to 10^t_max instead of from t_min to t_max.
https://github.com/odegym/neurodiffeq/blob/a60cfd326f8aef37633c36619df8d995e4e9b644/neurodiffeq/generators.py#L110
https://github.com/odegym/neurodiffeq/blob/a60cfd326f8aef37633c36619df8d995e4e9b644/neurodiffeq/generators.py#L113

The names t_min and t_max are misleading


Wanted quest, suggesting go black knight base talking Lord draquarius instead Mage of zamorak first

I had to go all the way back into taverley dungeon, so the order incorrect

.


It was not possible to have an iframe with fixed width and height attributes

I have just added compatibility at the iframe components to have a fixed width and height, since I need it for the app I'm currently working at.

I just moved the height and width logic on the iframes to two functions, giving priority to the width and height attributes at the markers.
But I'm thinking that it would probably be better to call them "elementsWidth" or something like that...

Hi,
I'm really trying to limit the amount of props available in this module. This is kinda getting out of hand, and I fear it might be overwhelming for someone who hasn't tried the plugin yet and is reading the readme for the first time.
Anyway, I don't think a specific prop is needed for this feature. I'm currently improving the iframe renderer so that you can set static dimensions for your iframes inside the tagsStyleand classesStyles props.
The logic I'm implementing is like so  :
Height : HTML attribute > classesStyles.yourClass.height > tagsStyles.iframe.height > 200 (fallback value)
Width : HTML attribute > classesStyles.yourClass.width  > tagsStyles.iframe.width > staticContentMaxWidth (your device width, by default).
This way, an iframe rendered without any dimensions will be in full width and 200px of height. You can override this at the markup level with the HTML attributes, and finally, you can style this just like any other component with tagsStyles and classesStyles.
See : https://github.com/archriss/react-native-render-html/commit/a142300567dbaf01d9d3a70d8ebbeccbee27b47a


Base README.md

Working on the French translation of the base README file

Hi, I am bhumikatewary, a contributor to OpenForce 2022. I would like to work on this issue. I would be making a PR as soon as I am done with resolving the issue. Thank you

I have made a PR:  Updated README.md #356
Please review and approve it


Remove v2.FS from v2.Path context

pfio.v2.Path shall be independent from pfio.v2.FS implementations such as S3 and so on as it includes too much information to represent generic paths and its manipulations. In this PR, it deprecates giving pfio.v2.FS to the constructor of pfio.v2.Path, and add a new method .scope() to FS for calling file access methods from Path such as open and exists.
Before:
with from_url('s3://bucket/path/to/dir') as s3:
  p = Path('path/to/file', fs=s3)
  p.open().read()

After this PR, we can do this:
p = Path('path/to/file')
with from_url('s3://bucket/path/to/dir') as s3:
  with s3.scope():
    p.open().read()

Current style (above) remains supported for 2.7.x, but will be removed in 2.8.

Closing in favor of https://github.com/pfnet/pfio/pull/331 .


now server owners can regulate what this can use

before this tweak, there was no way for server owners to regulate what the wire_user could interact with that didn't affect players pressing E
what if i was a server owner and didn't want players to pick up money with a wire_user?

an example of usage would be
local nah={
spawned_money=true,
spawned_weapon=true,
}
hook.Add("WireUse","example",function(self,trace,ply)
if nah[trace.Entity:IsValid()and trace.Entity:GetClass()]then
return false
end
end)

Not a fan of custom hooks since they're undocumented and can introduce inconsistent behavior to wire, e.g. people will start wondering why picking up money no longer works anymore. I'd just remove/restrict the user entity from the server.
The code does look fine, but I just don't think it's the correct solution.

imo it would make more sense to use the 4th parameter of use to distinguish users (looks like it's made for this kind of stuff).
Haven't seen anyone checks it so the user could just sent some identifier (Players using sends 0, wiki recommends 1, so the user could probably use some magic value).

The best way imo to prevent nearly all such exploits is to simply check the distance to the player. If the player is using an entity while standing on the opposite side of the map, you know something's up.
Also optionally check line of sight using a trace from the player's eyes so they can't use things through thin walls.
No need for a new hook, or changes, at all honestly.

@Divran the problem is you can't distinguish user uses vs player uses and he wants only player uses to pickup money.

I like adosika's idea better than the new hook idea. I also just had an idea that you can debug.getinfo in the PlayerUse hook to see if the source was from wire user.

hook.Add("PlayerUse","A",function()
    if debug.getinfo(3, "S").short_src == "addons/wire/lua/entities/gmod_wire_user.lua" then
        print("hi")
    end
end)

yeah but if the player is in range and has line of sight then does it really matter

Yeah, either do @Divran or my suggestions, or remove the user entity/tool.

I mean I can't say I'm against adding it
I don't think we can use the 3rd or 4th input of E:Use(...) though. Those enums don't match what we are trying to do. It would make more sense to add a new parameter to the PlayerUse hook call just above the line this PR modifies. Like this for example
if not hook.Run( "PlayerUse", ply, trace.Entity, "WireUser" ) then return false end

Ideally the 2nd argument of E:Use should be the user, according to the wiki. But also according to the wiki, that could cause issues with badly written addons.

I don't see how it would cause issues with badly written addons. However, how would it affect the PlayerUse hook? I don't think that would change at all would it?

I wouldn't be against adding the extra parameter to the hook call if they did that instead.

While it would be correct, changing the call to E:Use wouldn't allow an easy way to stop the Use from going through. You'd have to change the source of whatever entity you want to disallow. Or in the case of engine entities, you can't disallow it.
Either an extra argument to PlayerUse or a custom hook, cause either way people would have to modify their addon to take advantage of what we're adding.


Homework 4

Drift Video Link: https://video.drift.com/v/abZKpyBGjcCDc1xAHH9hfhOtuNhTqTOm7Fxyt6fxBHI/
Screenshots:
Question 1:

Question 2:

Question 3:

Question 4:

Metric
Value

alert_status
:x: ERROR

bugs
0

code_smells
11

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
0

code_smells
7

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


GDScript : default value of an exported variable can be modified at run time

Godot version:  v3.2.2.stable.official
OS/device including version: Windows 8.1

Issue description:
When using export var variable : Type = default_value, if default_value is a reference, it can be modified at run time. At the creation every variable will point to the same value. It means that one can modify variable of other instances of the class. It will also affect the value of variable at the creation of a new instance.
Steps to reproduce:
minimal code to reproduce
class_name CustomObject
extends Object

export var dict : Dictionary = {}

extends Node

func _ready():
	var a = CustomObject.new()
	a.dict["key"] = "test"
	a.free()
	var b = CustomObject.new()
	print(b.dict)
	b.free()

output : {key:test}
Minimal reproduction project:
minimal-project-export-default.zip

Related to #20436, might be duplicate of another issue but can't find it right now. CC @ThakeeNathees @vnen @bojidar-bg


feat: implement topology for all pool types

Changes

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

#1225  👈
#1224
#1223
#1222
#1221
#1213
#1212
#1210
main

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

Join @MasterPtato and the rest of your teammates on  Graphite


Tensorboard naming convention

I have question about parametrising the name of experiment which occur in Tensorboard. Current one look like that:
{Guild-id} {script name} {parameters} {name given in script like save dir}

I have more tham 20 parametes what make such description unreadable. I would like to have just ID and save dir. It is possible using code from master?

Master is basically 0.7.0.rc9 - so no. But this is a good idea. What you're seeing there is the run label - Guild includes values like 'save dir' because they are flag values. Guild otherwise doesn't know anything about the types of flags or what they mean. It's just the label.
But as TensorBoard doesn't really give you a good way to name runs in the list, we generate directory names using ID, operation, and label.
I think we could add a --run-name option (or something like that) were you could specify how runs are named. E.g.
$ guild tensorboard --name-name .id =save-dir'

This is a funny looking pattern, but it uses Guild's column spec syntax to specify a pattern that includes the run ID and the flag value for save-dir. E.g. this spec is used for specifying compare columns.
Given the need to release 0.7 ASAP, this feature would be deferred until the next minor release (e.g. 0.7.1) but I think it would be a nice change.
What do you think?

Okay - here's a proposal for 0.7 which will be in master shortly:
The guild tensorboard command will accept a new --run-name-flags option, which is a list of flag names (comma or space delimited). So to accomplish what you're looking for would be something like this:
$ guild tensorboard --run-name-flags save-dir

I'm assuming the info "name given in script like save dir" is in a flag. If the value isn't in a flag, this scheme won't work.

A new option --run-name-flags is available for the tensorboard command. You can provide a list of flags (comma or space delimited) to specify which flags are used in the label (using the standard NAME=VAL, space delimited convention). You can specify an empty string to drop use flags altogether.
This will be available in the next release.

The fix is now generally available in 0.7.0. If this is still an issue, please re-open and we'll sort it out. Thanks again for the report!

Is there a way to reset the --run-name-flags config? It seems like I once ran it with --run-name-flags "model_type" and that worked, but now without the flag instead of reverting to the regular behaviour (displaying all flags) it became the empty behaviour (--run-name-flags "").
(Less related: space delimited --run-name-flags "model_type covar_module" did not work.)

Thanks for pointing out the bug there - neither space nor comma delimited lists work. This is fixed in master now.
I can't recreate the behavior you're describing with --run-name-flags being stuck in the empty behavior. To create a ran the Get Started train.py example three times with x=[1,2,3]. Then I used --run-name-flags x to confirm that only x is used. Then I omitted --run-name-flags and confirmed that both x and noise are used.
Guild doesn't cache anything that could cause the behavior you're describing so what you're seeing is surprising.

Thank you for checking it out! Now that I look closely, it is actually the label that is interfering, as without any --run-name-flags the run name is:

7a542ffa 1D:train-np 2020-09-15 18:04:01/summaries

and with the empty --run-name-flags "" the run name has the label "physionet_np_grid" appended to it:

7a542ffa 1D:train-np 2020-09-15 18:04:01 physionet_np_grid/summaries

What you're seeing there is the intended behavior. The use of --run-name-flags overrides the default behavior, which is to show the label at that position.


failed to build native extension.

Hi,
I switched laptops and I can't get vorax to work.
Error detected while processing /home/rahul/dotfiles/.vim/bundle/vorax4/autoload/vorax/ruby.vim:
line   24:
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.
Error detected while processing /home/rahul/dotfiles/.vim/bundle/vorax4/autoload/vorax/sqlplus.vim:
line    9:
E117: Unknown function: vorax#ruby#SqlplusFork
Error detected while processing function vorax#sqlplus#Connect..<SNR>178_PrepareCstr:
line    3:
E117: Unknown function: vorax#ruby#ParseConnectionString
E15: Invalid expression: vorax#ruby#ParseConnectionString(cstr)

Never mind. Figured it out.
vorax was gem installed in a rvm ruby and gemset. I installed it in the ruby version vim was built with and it works fine now.


⚠ ClamAV has degraded performance

In bb0893a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 106 ms

Resolved: ClamAV performance has improved in 98b8bc6 after 24 minutes.


overflow issue with ScrollArea

I am using ScrollArea for ma side navigation bar. And, I am encountering an issue. First I have a question. The overflow is hidden when i created the ScrollArea with CLI, npx shadcn-ui@latest add scroll-area
 <ScrollAreaPrimitive.Root
    ref={ref}
    className={cn('relative overflow-hidden', className)}
    {...props}
  >
    <ScrollAreaPrimitive.Viewport className="h-full w-full rounded-[inherit]">
      {children}
    </ScrollAreaPrimitive.Viewport>
    <ScrollBar />
    <ScrollAreaPrimitive.Corner />
  </ScrollAreaPrimitive.Root>

I have tried overwriting it from the NavBar, it did not worked.
 <ScrollArea className="space-y-[10px] mt-[20px] text-primary-foreground text-[14px] overflow-visible">
        {navItems.map((item, index) => (
          <li
            key={index}
            className={`pl-[15px] @[113px]/sidebar:mr-[10px] ${
              checkActiveUrl(item.link) &&
              "relative before:content-[''] before:bg-primary before:h-[50px] before:w-[8px] before:absolute before:top-[50%] before:left-0 before:translate-y-[-50%] before:rounded-e-[8px] text-primary z-[10]"
            }`}
            onClick={() => handleLinkClick(item.link, item.title)}
          >

so i tried removing the overflow-hidden from the main ScrollArea component. Yet, it does not work.
 <ScrollAreaPrimitive.Root
    ref={ref}
    className={cn('relative', className)}
    {...props}
  >
    <ScrollAreaPrimitive.Viewport className="h-full w-full rounded-[inherit]">
      {children}
    </ScrollAreaPrimitive.Viewport>
    <ScrollBar />
    <ScrollAreaPrimitive.Corner />
  </ScrollAreaPrimitive.Root>

Is this intentional behavior of ScrollArea or its a bug? If its intentional then this issue ends here. If not then:
My Objective
I intend to make a sidebar navigation which is scrollable (i have only 4 routed for now). Also its collapsible. In its collapse state, when the nav links are hover, I intend to show a helper text which must overflow outside the ScrollArea. But the ScrollArea is hiding the overflow.
Here is what i meant

I am using ul for now. I only have four navigation links for now. I have more links to add, then i will be needing the ScrollArea

I, also am facing another issue (I recently found out). I am using the ScrollArea to make the div on the right scroll able (the section with hello text). The helper text, namely Dashboard goes behind the right div. I have tried using z-index to move in front but the issue persists.

Were you able to get this resolve?

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Various buttons not functioning

First, I love that you're taking the time to make a nice-looking File Explorer
When right-clicking on a column name, the Edit Columns and Size All Columns to Fit buttons do nothing. Also, when I go into Layout Mode and try to change to icons, nothing happens, even after restarting Files UWP.

OS Version: Windows 10 1903 18362.418
App version: v0.5.0

Thanks for the great feedback. These features aren't implemented yet, so their buttons act as placeholders for now.

This is a duplicate of https://github.com/duke7553/files-uwp/issues/8

These options are now available. You can download the latest version of Files from our website https://files.community/download


console.log statement in constructor

Thanks for this module.. works well..
Can you remove the console.log statement on this line?

any chance of publishing a patch release so that I can use this package from npm without the console.log statement..

waiting for latest npm release with this console.log removed

Urgently, Need the minified version without the console.log
Thanks for helping in advance.


Dest folder was outdated

Hello,
I've added updated the dest folder and added some information to README about how to use onRespondComplete.
Thanks,
Bulut.

Hi,
Just pinging for merge request.
Thanks.

Nice one, thanks!


remove fortmatic and use v4 for walletlink (coinbase)

@midgerate where is the v4 change at?

@samuveth I just removed walletlink from personalSign, that way it will by default use typed data

@midgerate  Voting on single choice proposals doesn't work for me anymore with Coinbase wallet (WalletLink)

@samuveth strange. Are you sure it was working when you tested it the last time?

@midgerate can you test yourself? https://www.coinbase.com/wallet

@samuveth @bonustrack added walletlink back for personal sign


It doesn't work as expected in Google Chrome

Hi @Nolanus, first of all, thank you for your time and for bringing to us this library.
Have you noticed an strange behaviour in the last versions of Google Chrome?
I'm trying to use your library in a project of mine and I can't achieve to make it work properly in Google Chrome. I also tried your demo with the same result.
The problem is that the scroll does not move correctly to the element.
I'm using Google Chrome Versión 53.0.2785.143 m (64-bit).
It works perfectly in Firefox Developer Edition 51.0a2 (2016-10-03) 32-bit and Internet Explorer 11 11.0.9600.18450.

Hi,
thanks for reporting your problem. Unfortunately I'm not able to reproduce it. Tried the mentioned Chrome version in OS X and Windows 7, both work as expected.
Are there any errors or logs on the console?

I have the same problem, it appears since the update to 2.0.0 final.
No logs/errors inside the browser.

:+1: Same problem ! :disappointed:

Same problem. I am on Windows 10 64bit, no errors, works in Firefox and worked in Chrome until Chrome update (now on Version 53.0.2785.143 m). Also works fine in android versions of both Firefox and Chrome.

The key is High-DPI screens. For me in Chrome is doesn't work on a 4K screen, but is does work on a 1080p screen. It works correctly in the other desktop browsers.
I don't know why, but it's true.

I can confirm issue in Chromium (53.0.2785.143 Built on Ubuntu, running on Ubuntu 16.04 (64-bit)).
In firefox it works as expected.
I'm using 3.2.1 version.
There is no any errors in the console.

Jep,
same here,
3 screens of 2560 x 1440 (so that is below 4K I think).
Works on firefox and edge.
Does not work on chrome 54.
There seems to be exactly one 'jump' or 'frame' which is executed, after that, none anymore...

Hi,
I have exactly the same issue on chrome. Did someone find how to fix it ?
Or we just have to wait for an update of the module or chrome ?
Thanks

I'll try to test the scrolling on a high dpi screen using chrome, however I assume it is a problem in the browser, as it works in the others (or chrome is even more strict and stays to the rules, whereas all others apply some "compatibility" logic?!).

@Nolanus I don't think that the issue in the hight DPI. I have old display https://www.amazon.co.uk/Acer-V223HQ-21-5-Widescreen-Monitor/dp/B001GH1QAE#productDetails

@Invis1ble, thanks for the feedback. Sorry for asking again, but may you try http://nolanus.github.io/ng2-page-scroll/ with Windows 10 and latest Chrome (54.0.2840)? Does it work in "Incognito Mode" without any AddOns? Do you have english locale?

I have tried:
with Windows 10 and latest Chrome (54.0.2840.87)
with and without incognito mode: Both work.
nl_be locale I think.
If I try my code: dev version of https://github.com/AngularClass/angular2-webpack-starter/pulse then it does not work.

@Nolanus Sorry, I can't test it on Windows. Also, the demo works fine in Chrome. I'll try to debug my case later. Maybe the reason in the url's anchor (#some-anchor) that I change before scrolling.

I've just updated to v3.2.3 (from v3.2.1) and it fix my issue.

Jep, works as a charm.


Make sure column names in schedules match ISO standard

Use the ISO standards (FIXME: which ones?) as the basis for naming the schedules. Check each column name to see if it is the correct name.
These column names must be updated in the documentation (demand.md and possibly the appendix)

A quite diverse notation of variable names was found in 30% of the data. It still works like this, but it would be great to have a standad notation


PHPUnit Test Listeners are technically read-only (not enforced). We need another means of manipulating suite execution order and suite filtering.

It sounds like PHPUnit 7.0 will spell the end for the current approach...
See https://github.com/sebastianbergmann/phpunit/issues/2477
Note: Current implementation is https://github.com/humbug/phpunit-extensions

Updated a bit to see how this actually develops. Presumably something will have to be added to allow test/suite level write access before cutting off the Listener API. There is a TestDecorator extension point documented, but I can't find any way of injecting that at present. We certainly can't ask users to add code somewhere (and how would we use if we did from outside the testing loop where we belong?).

Opened https://github.com/sebastianbergmann/phpunit/issues/2660


Node set*Property doesn't work with a DerivedProperty

I ran into this while instrumenting Graphing Quadratics...
Node options visibleProperty and setVisibleProperty are documented as taking a value of type {TinyProperty.<boolean>|Property.<boolean>|null}.   Nothing here about it needing to be settable, no comments, no assertion. So I expect it to work with DerivedProperty, which is a subclass of Property.  But pass in a DerivedProperty and it will fail hard, downstream.  Node requiresvisibleProperty to be settable.
There's not a single call to isSettable in Node.  So I suspect that pickableProperty, enabledProperty, and inputEnabledProperty may also have this problem.  Do any of them work with a DerivedProperty? Which ones need a check for isSettable?
Better yet... Is there something that can be done to make Node accept DerivedProperty for these Properties? I'd love to pass a DerivedProperty visibleProperty to Node.  If I can't pass a derived visibleProperty to Node, then I have to set visiblePropertyOptions: { phetioReadOnly: true } and manage the Node's visibility myself.  I'm doing that 14 times in Graphing Quadratics, 10 times in Natural Selection,...

Slack:
Jonathan Olson  5:30 PM
Allowing Node options visibleProperty to be a DerivedProperty would remove a lot of the need to make visibleProperty read-only. I’m not sure why Node needs to be setting it’s visibleProperty - maybe to synchronize with its options.visible value?
5:30
I wasn't aware of that constraint
5:30
Presumably we could let the setVisible fail out if you've passed in a DerivedProperty to control it?
5:31
I've used DerivedProperty a lot for fill/stroke, and it would be a shame not to be able to use that for visibility
Jonathan Olson  5:31 PM
What happens when you try to pass a DerivedProperty?
Chris Malley  5:32 PM
I’ll have to re-create a failure, give me a minute…
Chris Malley  5:40 PM
OK.  So Node does support DerivedProperty.  But the various related set calls (setVisible, etc) are not failing out.  So I had accidentally left a call to this.visible = in a listener, and it was failing way downstream.  So… If DerivedProperty is indeed supported, then all corresponding setters need to verify isSettable.  I’ll put this info in the issue.
Jonathan Olson  5:41 PM
Wait, would derivedProperty.set( value ) not fail out on its own?
Chris Malley  5:43 PM
Yeah, I guess I can figure that out from the stack trace.  Not the most helpful, but that seems to be the JS way.
So.... DerivedProperty is indeed supported, and we're not going to add more error checking.  Closing.


Speed up pileup

At the moment we use bamtools for pileup which is fairly slow. We could probably get some seed-up either by using @stevenhwu's 'bamtools utils' code to speed up parsing of reads or DNG's approach which uses htslib

"bamtools utils" is in the develop branch, using htslib is too large a change to consider in the near-term


SDXL dual prompt example

I have been able to use Compel for SDXL and SDXL-Refiner with a single prompt.  I'd like to test sending different prompts to the two text encoders for SDXL.
Is this supported yet?  If so, does someone have an example of it?

you could spin up two separate Compel instances and use torch.cat() to concatenate the embeddings. you'll need the embedding to have a .shape of [1, 77, 2048] for it to work.
but to be clear, the base model at least already uses both text encoders, as does i believe the refiner. there's not one encoder for base and one for refiner. base and refiner both use both encoders.

base and refiner both use both encoders

Refiner only uses the second encoder, but its the same as the base.
See https://huggingface.co/stabilityai/stable-diffusion-xl-refiner-1.0/tree/main and https://github.com/huggingface/diffusers/blob/ea5b0575f8f91b76f32fb6f6930c0bc30e42865e/src/diffusers/pipelines/stable_diffusion_xl/pipeline_stable_diffusion_xl_img2img.py#L298

Thanks!  If anyone else is following and looking for an example:
compel1 = Compel(
    tokenizer=self.pipe.tokenizer,
    text_encoder=self.pipe.text_encoder,
    returned_embeddings_type=ReturnedEmbeddingsType.PENULTIMATE_HIDDEN_STATES_NON_NORMALIZED,
    requires_pooled=False,
)

compel2 = Compel(
    tokenizer=self.pipe.tokenizer_2,
    text_encoder=self.pipe.text_encoder_2,
    returned_embeddings_type=ReturnedEmbeddingsType.PENULTIMATE_HIDDEN_STATES_NON_NORMALIZED,
    requires_pooled=True,
)

conditioning1 = compel1(prompt1)
conditioning2, pooled = compel2(prompt2)
conditioning = torch.cat((conditioning1, conditioning2), dim=-1)

image = pipeline(prompt_embeds=conditioning, pooled_prompt_embeds=pooled, num_inference_steps=30).images[0]

is this resolved? can i close it?

Might want to include the snippit in documentation, but yes I think we're good.

thanks!


make BlockLauncher unload GUIs when in multiplayer

@zhuowei you realllly need to make BlockLauncher unload GUIs when in multiplayer.
I can't play a single game of survival games without seeing a person using a gui that adds items to their inventory. (Also MiNET should sync inventory's on the server so people can't do this but MiNET isn't yours)

So..? Thinking about it?

I assume it's a priority to Zhuowei. It's especially important because people can use it to fly.

And use it for many other things


Failed to import - blockly

Failing module

GitHub: https://github.com/google/blockly
npm: https://www.npmjs.com/package/blockly

import<EMAIL_ADDRESS>
Error message
After running deno run I got this:
error: Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'createBlockDefinitionsFromJsonArray$$module$build$src$core$common')
    at https://esm.sh/v135/blockly@11.1.1/denonext/blocks.js:4:1024
    at https://esm.sh/v135/blockly@11.1.1/denonext/blocks.js:4:923
    at https://esm.sh/v135/blockly@11.1.1/denonext/blocks.js:4:983
    at https://esm.sh/v135/blockly@11.1.1/denonext/blocks.js:4:459
    at https://esm.sh/v135/blockly@11.1.1/denonext/blocks.js:6:48221

Additional info

esm.sh version: v135
Deno version: 1.45 - 1.46.3

thanks! i can confirm this also doesn't work on dev branch, i will look into it!


i converted it to pytorch...

Hi,
First off thanks for posting this! I couldn't find a better way to contact you, so I hope this is ok.
I'm excited to use it, but I ran into some issues. I tried several times and just couldn't get it to run. i know it's something with my configuration, I just can't figure it out.
So I converted the project to use Pytorch and I got that working. I'm wondering if I can get a copy of the .raw files for the example folder, so I can see how accurate my conversion is?
My plan is to do a visual inspection first, with your vis.py script, then if that pans out, compare the volumes generated.
My hope is that it is accurate, so I can post the conversion for others to use, giving you credit of course.
Thanks,
-Andrew

Cool. That's great.
The Python version(s) will never produce the same results as what are found in the paper as I have not been able to find an isosurface algorithm which works quite as well as the one built into MATLAB. However, we can certainly compare the volumes. If you email me<EMAIL_ADDRESS>I will email back with the raw files produced by the Python version.
If we are sure it is working correctly, I'll gladly link to it from the README.


findWindowHandle won't work when the title is set via binding

The method findWindowHandle  is not working when the titleProperty is set via binding.
One idea: check if titleProperty.isBound() and throw an execption
Another idea: provide a second method for this case:
public static WindowHandle findWindowHandle(final Stage stage, final StringBinding titleBinding) {
        if (Platform.getOSType() != Platform.WINDOWS) {
            return null;
        }

        final var searchString = "stage_" + java.util.UUID.randomUUID();
        if (stage.titleProperty().isBound()) {
            Objects.requireNonNull(titleBinding, "TitleBinding must not be null");

            stage.titleProperty().unbind(); // unbind current binding
            stage.setTitle(searchString); // still use the search string for a unique id

            final var hwnd = User32.INSTANCE.FindWindow(null, stage.getTitle());
            stage.titleProperty().bind(titleBinding); // reset the binding

            if (hwnd != null) {
                return new WindowHandle(hwnd);
            }
        } else {
            final var title = stage.getTitle();
            stage.setTitle(searchString);
            final var hwnd = User32.INSTANCE.FindWindow(null, searchString);
            stage.setTitle(title);

            if (hwnd != null) {
                return new WindowHandle(hwnd);
            }
        }

        return null;
    }

Thanks, i will look into it. Though i should mention there are better alternatives now:
https://github.com/dukke/FXThemes
https://github.com/xdsswar/shared-fx-jni-src

Well at least FXThemes will need to add --add-opens/--add-exports to your vm plus you can only change the gui "after" show. You version do not have such problems :-).

😃
The fix (tryFind method now throws an exception) in the 0.2 branch. Not yet tested.

New version is now the main branch. Be warned that I used this opportunity to remove the dash from the artifact id and the underscore from the package name.


Disable italics on numbers and strings.

I'm using LazyVim and I've been trying to get Lush working for a while now. Any tips for how to disable the italics for numbers and strings? Trying to follow this guide: https://github.com/mcchrish/zenbones.nvim/blob/v3.1.0/doc/zenbones.md#extendoverride-highlights
-- lua/plugins/colorscheme.lua
return {
  {
    "mcchrish/zenbones.nvim",
    dependencies = { "rktjmp/lush.nvim" },
    init = function()
      vim.g.neobones_darkness = "stark"
      vim.g.neobones_lighten_cursor_line = 8
      vim.g.neobones_lighten_comments = 40
      vim.g.neobones_italic_comments = false
      vim.api.nvim_create_autocmd({ "ColorScheme" }, {
        pattern = "neobones",
        callback = function()
          local lush = require("lush")
          local base = require("neobones")
          local testNum =<PHONE_NUMBER>0
          local specs = lush.parse(function()
            return {
              Constant = { base.Constant, gui = "NONE" },
              Number = { base.Number, gui = "NONE" },
            }
          end)
          lush.apply(lush.compile(specs))
        end,
      })
      vim.cmd("colorscheme neobones")
    end,
  },
  {
    "LazyVim/LazyVim",
    opts = {
      colorscheme = "neobones",
    },
  },
}

@dannyfritz I'm able to reproduce the issue and I think the issue is with lush. It works with using plain nvim_set_hl api.
vim.api.nvim_create_autocmd({ "ColorScheme" }, {
    pattern = "neobones",
    callback = function(params)
        require("mcchrish.statusline").setup(params.match)
        local base = require(params.match)
        vim.api.nvim_set_hl(0, "Constant", { fg = base.Constant.fg.hex })
        vim.api.nvim_set_hl(0, "Number", { fg = base.Number.fg.hex })
    end,
})

Which makes it simpler actually. Alternatively, you can also use nvim_get_hl and nvim_set_hl combo to modify highlights too.

@mcchrish thank you. The snippet you provided everything I need.

That seems to be working for me:
Constant { base.Constant, gui = '' },


React PropTypes Warning

I'm receiving the following warning:

Warning: Accessing PropTypes via the main React package is deprecated. Use the prop-types package from npm instead.

Install

yarn add prop-types

Use
import PropTypes from 'prop-types';
instead of
import React, {PropTypes} from 'react';

This should be fixed on devel.


User.Read permission always appears on a consent screen for any delegated flow, even if it is not on the app registration.

The User.Read ( Microsoft Graph permission ) is the default permission required for any delegated sign-in flow.  So, even if that permission is removed from the app registration, an admin consent ( or user consent ) for any other scope will, like the offline_access scope, always be presented for consent.  Documentation should indicate this as the question has come up and this information about the user.read scope always being present is not coming up in the docs!  Thanks!

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

ID: 0e056c00-a2a1-7019-fb36-0243819a7b99
Version Independent ID: a7a0b942-5e83-6e60-c9e8-32e0225de250
Content: Microsoft identity platform scopes, permissions, & consent
Content Source: articles/active-directory/develop/v2-permissions-and-consent.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

Thanks! I made a pull request to update the documentation. https://github.com/MicrosoftDocs/azure-docs-pr/pull/189045
I'll leave this open for review by the content author, @rwike77


Switch/button inputs suffer from noticeable delay between physical hardware and sim response

Fixed with v0.2


LuisGen cannot create a C# class when json has sub entities

Tool
Name: LuisGen
Version: 2.2.0
OS: Windows 10
Describe the bug
LuisGen doesn't create a C# class when I use a exported LUIS file that has sub entities.
To Reproduce
Steps to reproduce the behavior:

Create Sub Entity Under Entity

Publish the LUIS App, Export the .json file
Use Luis gen to create a C# class using LUISGen {Your export json file} -cs {your C# file name}
You get below error.

Any log messages given by the failure

Unhandled Exception: System.ArgumentException: Can not convert Object to String.    at Newtonsoft.Json.Linq.JToken.op_Explicit(JToken value)    at System.Dynamic.UpdateDelegates.UpdateAndExecute1[T0,TRet](CallSite site, T0 arg0)    at CallSite.Target(Closure , CallSite , Object )    at LUISGen.Utils.EntityApply(JObject entity, Action1 action) in D:\a\1\s\packages\LUISGen\src\Utils.cs:line 94
at System.Dynamic.UpdateDelegates.UpdateAndExecuteVoid5[T0,T1,T2,T3,T4](CallSite site, T0 arg0, T1 arg1, T2 arg2, T3 arg3, T4 arg4)
at LUISGen.CSharp.Entities(Object app, Writer w) in D:\a\1\s\packages\LUISGen\src\CSharp.cs:line 161
at System.Dynamic.UpdateDelegates.UpdateAndExecuteVoid3[T0,T1,T2](CallSite site, T0 arg0, T1 arg1, T2 arg2)
at LUISGen.CSharp.Generate(String description, Object app, String className, String space, String outPath) in D:\a\1\s\packages\LUISGen\src\CSharp.cs:line 289
at System.Dynamic.UpdateDelegates.UpdateAndExecuteVoid6[T0,T1,T2,T3,T4,T5](CallSite site, T0 arg0, T1 arg1, T2 arg2, T3 arg3, T4 arg4, T5 arg5)
at LUISGen.Program.Main(String[] args) in D:\a\1\s\packages\LUISGen\src\Program.cs:line 182`

Expected/desired behavior

The command must create a C# LUIS Recognizer class.

Versions

Microsoft.Bot.Builder.AI.Luis 4.11.1
Microsoft.Bot.Builder 4.11.1

@Chitma you're right, when trying to use LUISGen CLI with a LUIS model that contains subentities, you'll get the error that you have posted. This is because the CLI is a legacy CLI now, the Bot Framework team has moved onto replacing the multiple, separate CLIs we had with the new BF CLI. You can see note of this switchover in the LUISGen readme as well as in the BF CLI Porting Map. LUISGen, I believe should only work for v2 of LUIS API, however the current API version is v3.

To generate a strongly typed C# class from an exported v3 LUIS model use bf luis:generate:cs command in BF CLI.

Install BF CLI (docs): npm i -g @microsoft/botframework-cli
Generate strongly-typed C# class: bf luis:generate:cs -i <YOUR-EXPORTED-LUIS-MODEL> -o <OUTPUT-FILE>

Example:

bf luis:generate:cs -i .\ash-pizza-subentities.json -o AshPizza.cs

@Chitma you're right, when trying to use LUISGen CLI with a LUIS model that contains subentities, you'll get the error that you have posted. This is because the CLI is a legacy CLI now, the Bot Framework team has moved onto replacing the multiple, separate CLIs we had with the new BF CLI. You can see note of this switchover in the LUISGen readme as well as in the BF CLI Porting Map. LUISGen, I believe should only work for v2 of LUIS API, however the current API version is v3.

To generate a strongly typed C# class from an exported v3 LUIS model use bf luis:generate:cs command in BF CLI.

Install BF CLI (docs): npm i -g @microsoft/botframework-cli
Generate strongly-typed C# class: bf luis:generate:cs -i <YOUR-EXPORTED-LUIS-MODEL> -o <OUTPUT-FILE>

Example:

bf luis:generate:cs -i .\ash-pizza-subentities.json -o AshPizza.cs


Add canonical links for cross-posted documents

Fixes: #999
Changes:

Add canonical link for Keycloak blog, zh-cn and en-us.
Add canonical link for Dapr blog, zh-cn and en-us.
Add canonical link for OpenTelemetry blog, zh-cn only, since en-us were already done in https://github.com/apache/apisix-website/pull/995/

@juzhiyuan @SkyeYoung @nfrankel @SylviaBABY @hf400159 please take a look when available, thanks.

@yzeng25 The intent is great, but I don't know if the generation engine takes the <head> into account. If yes, then perfect!

@yzeng25 The intent is great, but I don't know if the generation engine takes the <head> into account. If yes, then perfect!

I just checked.
#995 did this and here is the preview.

This pull request did similar things and they show up in preview environment. Here are a couple of examples.

Perfect!!!


Use constant for ha config type label

What this PR does / why we need it:
Uses constant for ha config type label. Cleanup.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

/test pull-extension-shoot-oidc-service-e2e-kind


NotImplementedError: Unsupported ONNX ops of type: ConstantOfShape

Hi
I'm trying to export onnx to core ml and I'm getting the following error:
`---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call last)
 in ()
----> 1 mlmodel = onnx_coreml.convert(model_onnx)
2 # mlmodel.save('models/pytorch_lstm12_1000_300619.mlmodel')
~/anaconda3/envs/tf_p36/lib/python3.6/site-packages/onnx_coreml/converter.py in convert(model, mode, image_input_names, preprocessing_args, image_output_names, deprocessing_args, class_labels, predicted_feature_name, add_custom_layers, custom_conversion_functions, onnx_coreml_input_shape_map)
457     '''
458     if not add_custom_layers:
--> 459         _check_unsupported_ops(graph.nodes)
460
461     err = ErrorHandling(add_custom_layers,
~/anaconda3/envs/tf_p36/lib/python3.6/site-packages/onnx_coreml/converter.py in _check_unsupported_ops(nodes)
143     if len(unsupported_op_types) > 0:
144         raise NotImplementedError("Unsupported ONNX ops of type: %s" % (
--> 145             ','.join(unsupported_op_types)))
146
147
NotImplementedError: Unsupported ONNX ops of type: ConstantOfShape`

I will be sending PR this week

OK, waiting.

@nitsanp ConstantOfShape support has been added with https://github.com/onnx/onnx-coreml/pull/424
Could you please give a try?

@nitsanp ConstantOfShape and LSTM is now supported in CoreML 3
Note that, you will need to use Mac OS 10.15 beta
please use disable_coreml_rank5_mapping=True flag while converting to enable nd-tensor conversion path

I had a similar issue with ConstantOfShape, LSTM, Gather, and setting add_custom_layers to true solved it for me. I have not tested the converted model yet, but at least it converts without error message.

Thank you @dnnagy, when I set add_custom_layers=True I don't get that error anymore, but I still get an error when it arrives to the LSTM layer:
ValueError: array split does not result in an equal division

@bhushan23 I don't understand where to find this flag

@bhushan23 I don't understand where to change this flag.

@nitsanp I can convert my model to .mlmodel format, but when I try to integrate it in my app, i get an error in Xcode:

I think, we cannot use LSTMs currently, because it was introduced in CoreML 3, and is currently available only in Xcode 11 beta and macOS Catalina beta versions.

Thank you, I hope it will be available soon.

@bhushan23 I don't understand where to change this flag.

@nitsanp you can pass disable_coreml_rank5_mapping=True to convert method here https://github.com/onnx/onnx-coreml/blob/master/onnx_coreml/converter.py#L337
This option disables rank5 mapping and allows usage of ops with nd tensor support i.e. newly introduced layers in CoreML
I see, it's not mentioned in README. Will make a change to specify what this option does.

@bhushan23
OMG it worked!!!!! thank you so much!!!!!

closing as support for ConstantOfShape is added

I am having the same issue in https://github.com/onnx/tutorials/issues/150
Could someone help me that? I already pass disable_coreml_rank5_mapping=True but the error persist.

@edjust please use latest onnx-coreml beta https://github.com/onnx/onnx-coreml/#new-beta-onnx-coreml-converter-with-core-ml-3
pass target_ios='13'
these ops are supported with target_ios='13'


Add SearchForLanguageAndCarrierConstants example

Based on Perl library example in PR 9 (still under review).

The Perl example appears to be complete now, but if there are further changes there I'll incorporate them before merging this one.


allow for basic auth

nginx-proxy allows for basic auth.
https://github.com/jwilder/nginx-proxy#basic-authentication-support
For this to work, the respective volume has to be added.

Thanks


files are just don't considered in the whole repo

Preliminary Steps
Please confirm you have...

[x] reviewed How Linguist Works,
[x] reviewed the Troubleshooting docs,
[x] considered implementing an override,
[x] verified an issue has not already been logged for your issue (linguist issues).

Problem Description
Files are not considered for language detection.

URL of the affected repository:
https://github.com/darkRaspberry/Forest
Last modified on:

Expected language:
All files are (including in docs folder) should considered for languages %

Detected language:
none in docs folder only.

I have locally tested and found that docs are not considered or linguist can't see docs files

i have moved files to different place still no response

As you've correctly identified, your files are not being considered for the language stats because they're in the docs folder and thus being determined by Linguist as documentation thanks to:
https://github.com/github/linguist/blob/5da91555329a1371a602f10a9719439219d280de/lib/linguist/documentation.yml#L12

I have moved files to different place still no response.

It should do, though you could have checked before the background job that does the analysis had run. Your commit history doesn't show any commits that moved the files either.

I have locally installed linguist and it doesn't return anything. (It works with another repo).

Did you commit your changes? Linguist only acknowledges committed changes:
$ git clone -q git@github.com:darkRaspberry/Forest.git
$ cd Forest/
$ github-linguist . --breakdown

$ git mv docs another-name
$ git commit -m 'Rename docs to another-name'
[main b3a0ec5] Rename docs to another-name
 7 files changed, 0 insertions(+), 0 deletions(-)
 rename {docs => another-name}/404.html (100%)
 rename {docs => another-name}/CNAME (100%)
 rename {docs => another-name}/index.html (100%)
 rename {docs => another-name}/static/assets/css/css.css (100%)
 rename {docs => another-name}/static/assets/img/gatewayTree.png (100%)
 rename {docs => another-name}/static/assets/js/transit.js (100%)
 rename {docs => another-name}/static/assets/js/ui.js (100%)
$
$ github-linguist . --breakdown
76.75%  JavaScript
18.63%  HTML
4.62%   CSS

HTML:
another-name/404.html
another-name/index.html

CSS:
another-name/static/assets/css/css.css

JavaScript:
another-name/static/assets/js/transit.js
another-name/static/assets/js/ui.js

$

You can also implement an override to force your docs directory to be considered, but there's a catch: you're also using a path that matches a "vendored" path...
https://github.com/github/linguist/blob/5da91555329a1371a602f10a9719439219d280de/lib/linguist/vendor.yml#L242
... so you need to implement two overrides if you're going to keep the docs directory name. Carrying on from the illustration above:
$ git revert HEAD
[main 571ec80] Revert "Rename docs to another-name"
 7 files changed, 0 insertions(+), 0 deletions(-)
 rename {another-name => docs}/404.html (100%)
 rename {another-name => docs}/CNAME (100%)
 rename {another-name => docs}/index.html (100%)
 rename {another-name => docs}/static/assets/css/css.css (100%)
 rename {another-name => docs}/static/assets/img/gatewayTree.png (100%)
 rename {another-name => docs}/static/assets/js/transit.js (100%)
 rename {another-name => docs}/static/assets/js/ui.js (100%)
$ github-linguist . --breakdown

$
$ echo '* -linguist-documentation' > .gitattributes
$ echo '* -linguist-vendored' >> .gitattributes
$ git add .gitattributes
$ git commit -m 'Implement Linguist overrides'
[main 06243c5] Implement Linguist overrides
 1 file changed, 1 insertion(+), 1 deletion(-)
$ github-linguist . --breakdown
76.75%  JavaScript
18.63%  HTML
4.62%   CSS

HTML:
docs/404.html
docs/index.html

CSS:
docs/static/assets/css/css.css

JavaScript:
docs/static/assets/js/transit.js
docs/static/assets/js/ui.js

$

Thanks


Include timecode header support

The two uint header fields (utc_seconds, utc_fraction) should be included in the message header. This ideally would only be when a certain command line flag is set.

Added in d583bb4


{Network} az network private-endpoint: Added Device Update for IoT Hub support to Network Private Endpoint

Related command
No new command. Added Device Update for IoT Hub support to Network Private Endpoint
Description
Added Microsoft.DeviceUpdate/accounts support to Network Private Endpoint
Testing Guide
See the added test and recording

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Network


how to set

I have llama3:8b-instruct-q5_K_M model with ollama
How to set it with config, its unclear

tried above but get

you need to add a ollama- prefix, run ask list to view the list of models

thanks that worked


Do not refer to ActionMailer::Base in initializers

This causes issues with autoloading (invoking behaviour that has been
deprecated in Zeitwerk when certain stars align, in our case if we add
ActionText, c.f. https://github.com/rails/rails/issues/36546)
Instead of adding configuration to ActionMailer::Base, set it on
Rails.configuration.action_mailer instead.

Review app deployed to https://teaching-vacancies-review-pr-3521.london.cloudapps.digital