Allow to set Webadaptor Context via config

Allow the Webadaptor context (e.g. server / portal) to be set by config.
Right now it's hardcoded.
Therefor it's not possible to install a Webadaptor for Hosting-Server and GeneralPurpose-Server on the same machine.

Already possible
				   "ServerContext": "[servercontext]"
       }


Watch issue

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
Watch is not working.
What behavior are you expecting?
Steps to reproduce:

npm run watch

[17:05:51]  watch failed: A watch configured to watch the following paths failed to start. It likely that a file
            referenced does not exist: xxx\src\**\*.(ts|html|scss)
[17:05:51]  ionic-app-script task: "watch"
[17:05:51]  Error: A watch configured to watch the following paths failed to start. It likely that a file referenced
            does not exist: xxx\src\**\*.(ts|html|scss)

Which @ionic/app-scripts version are you using?
1.1.3
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)

Increasing timeout in watch.js to 10000 fixes the issue for me.

Fixed in 1.1.4, it's now configurable. See the readme for IONIC_START_WATCH_TIMEOUT property.
Thanks,
Dan


GitHub Learning Resource

Describe the solution you'd like
Add relevant GitHub training material to the Learning Resources page.

Added via https://github.com/rubrikinc/welcome-to-rubrik-build/pull/7


require multiple files in Elixir?

I cannot get multiple, e.g.,
defmodule Wandbox do
require Foo
Foo.hello()
end
will not work when I create a second file for the Foo.ex module.

Elixir in Wandbox is using .exs file, not .ex.
So if you want to load another .exs file, you should use Code.require_file/1.
https://wandbox.org/permlink/pGReSes2OIhy1GOh

However, this behavior is difficult to understand, so I will consider whether it can be solved.

Thank you.

Is there a convenient way to require external libraries, e.g., https://github.com/devinus/poison
?

It is impossible now. You can not use the internet with Wandbox.


fix(client): fix double quote in head tag

fixed #1174 https://github.com/vuepress-theme-hope/vuepress-theme-hope/issues/2448#issue-1461331094

You’d better explain why JSON.stringing is needed.

Closing as this is not a valid fix, if attr contain space, you fix can cause similar issue.

Closing as this is not a valid fix, if attr contain space, you fix can cause similar issue.

sure?  can you give me an example

You’d better explain why JSON.stringify is needed.

'some text with "double quote"' valid string
https://github.com/vuepress/vuepress-next/blob/8c345d827c758326429b20f37b6be95f0865c655/packages/client/src/setupUpdateHead.ts#L88
but here will transform that to "some text with "double quote""，it's invalid
so use  "" or ''  will generate invalid string,

<meta name="description" value="Some words here" />


Fix Lazarus support to eFirebase

As reported by Ernesto on #2 , there is a problem with eFirebase on Lazarus (FreePascal). I'll be reviewing it as soon as possible. The reasons for the issue are:

Some methods Delphi use are not compatible with Lazarus;
Some Units in Uses clause need to be reviewed.

I've already fixed the implementation, but I still need to check if everything is going to work as planned and I have plans to build a sample app in Lazarus to test if everything's fine and to make it easy for Lazarus (FreePascal) developers learn how to use it.

Everything's working just fine. Released version 1.0.5


add website link lamrium token

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address: 86UGuBBNSy6QDgYggDVLixCEjrwHAShRu8Vu3Lno7FiP
Token Name: LAMRIUM
Token Symbol: LAMRO
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token: https://lamrium.space
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

I can help resolve automerge issues.
Feel free to check out J-a-m-e-s-o-n/token-list#1


jobs failing with sigbus and unknown userid errors

Description of the bug
This may be two separate issues, but I'm having jobs fail either with a Java Runtime Environment SIGBUS (0x7) error, or with FATAL: Couldn't determine user account information: user: unknown userid [some number]
In both cases, the process that fails and the file associated with the error changes with each rerun.
With subsequent reruns, the process and sample that failed previously will complete, but a new process and sample will error out. Eventually the pipeline can be completed with multiple reruns.
The pipeline runs fine with the test profile.
From what I can figure out the Java Runtime Environment error has something to do with running out of memory in the java vm.
but I'm stumped on the userid error.
I have made a small modification to the modules.config file to add --no-model param to MACS. But I am not seeing any errors with MACS, and again, the test profile runs as expected.
Any help would be greatly appreciated!
Command used and terminal output
#!/bin/bash
#SBATCH --mem=8G
#SBATCH -t 8:00:00
#SBATCH -n 1
#SBATCH -o var/log/chip-%j.out
#SBATCH -e var/log/chip-%j.err

module load nextflow

nextflow -log var/log/.chipseq run nf-core/chipseq \
	-profile singularity \
	-c config/slurm.config \
	-resume \
	-params-file config/chip_params.yaml

Relevant files
bug.zip
System information
Nextflow: 23.04.2
Hardware: HPC
Executor: slurm
Container: Singularity
OS: RHEL8
nf-core/chipseq: 2.0.0

It turns out this is almost certainly an issue on the HPC side.
slurm / admins were causing jobs that were submitted to one partition to get redirected to a different partition.
By default, nextflow polls job status from the submitted partition, and if it can't find the job things get messed up.
We have been able to fix this by adding the following to our config file:
executor {
    name = "slurm"
    queueGlobalStatus = true
}

This will tell nextflow to poll for job status globally and not just within the submitted partition.
It would be great if this was either the default behavior from nextflow, or there was a more informative error message in cases where a job has been redirected.


not going to listen mode

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
followed guide, when alexa is spoke, i see response on console but it never shows "listening prompt" goes back to alexa is idle"
What is the expected behavior?
expect to see listening prompt
What behavior are you observing?
returns to idle prompt
Provide the steps to reproduce the issue, if applicable:
reinstalled
Tell us about your environment:
raspberry pi 4
PRETTY_NAME="Raspbian GNU/Linux 10 (buster)"
NAME="Raspbian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"
latest
  <x.y.z>

Tell us what hardware you're using:

[ ] Desktop / Laptop
[x ] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ x] Other - tell us more: raspbain buster

in the log attached, i can see where it goes to listening, it must fly by as i never see it in the console window
alexaerror.txt

after reviewing this, appears alexa is listening, and is responding, appears to be a audio issue.

I built using sudo, which put the sound settings under root i believe,


Rename

Filterable is taken on rubygems + this gem does more than filtering. We should rename prior to 1.0.

Should close as gem renamed?

Yup Thanks!


Wrong parsing result

var b= (a, b) => {

}

Fixed by https://github.com/IWANABETHATGUY/lang-rome-ast/commit/8134a49cb672290c930a8e6993c599833086ac37


Expose the raw hash of the object

raw_body was removed in #17

@syncrou Please review.  I think this will fill in one of the missing parts you need for update.

👍 LGTM


I UPDATED IT STILL DOESNT WORK youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.10.05. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ ] I've verified and I assure that I'm running youtube-dl 2018.10.05

Before submitting an issue make sure you have:

[ ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones
[ ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.10.05
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

my logs:
$ youtube-dl -x --audio-format aac --verbose https://www.youtube.com/watch?v=kmZhmIt_pYo
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-x', u'--audio-format', u'aac', u'--verbose', u'https://www.youtube.com/watch?v=kmZhmIt_pYo']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.06.05
[debug] Python version 2.7.14 - Darwin-17.7.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.4.1, ffprobe 3.4.1
[debug] Proxy map: {}
[youtube] kmZhmIt_pYo: Downloading webpage
[youtube] kmZhmIt_pYo: Downloading video info webpage
[youtube] kmZhmIt_pYo: Extracting video information
WARNING: unable to extract uploader nickname
[youtube] {43} signature length 42.46, html5 player vflsLe3jn
[youtube] kmZhmIt_pYo: Downloading player https://www.youtube.com/yts/jsbin/player-vflsLe3jn/en_US/base.js
ERROR: Signature extraction failed: Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1162, in _decrypt_signature
    video_id, player_url, s
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1073, in _extract_signature_function
    res = self._parse_sig_js(code)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1134, in _parse_sig_js
    jscode, 'Initial JS player signature function name', group='sig')
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 782, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
 (caused by RegexNotFoundError(u'Unable to extract \x1b[0;34mInitial JS player signature function name\x1b[0m; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1162, in _decrypt_signature
    video_id, player_url, s
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1073, in _extract_signature_function
    res = self._parse_sig_js(code)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1134, in _parse_sig_js
    jscode, 'Initial JS player signature function name', group='sig')
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 782, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 761, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 433, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1774, in _real_extract
    encrypted_sig, video_id, player_url, age_gate)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1172, in _decrypt_signature
    'Signature extraction failed: ' + tb, cause=e)
ExtractorError: Signature extraction failed: Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1162, in _decrypt_signature
    video_id, player_url, s
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1073, in _extract_signature_function
    res = self._parse_sig_js(code)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1134, in _parse_sig_js
    jscode, 'Initial JS player signature function name', group='sig')
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 782, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
 (caused by RegexNotFoundError(u'Unable to extract \x1b[0;34mInitial JS player signature function name\x1b[0m; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

OK... my issue was the old version
$ youtube-dl --version
2017.06.05

solution on osx:
$ brew upgrade youtube-dl
==> Upgrading 1 outdated package:
youtube-dl 2017.06.05 -> 2018.10.05
==> Upgrading youtube-dl
...

after again checked the version

$ youtube-dl --version
2018.10.05

and everything worked fine!

A linked issue is: https://github.com/rg3/youtube-dl/issues/17935

Please close this issue! Not valid!


[5.7-04252022] Fix API Changes highlight cut off for relationship section in S viewport

Rationale: Fix a regression caused by the content style redesign where API changes highlight box is cut off on the right in S viewport
Risk: Low
Risk Detail: Change only affects small viewport when API change is turned on
Reward: High
Reward Details: Fixes regression and broken UI
Original PR: https://github.com/apple/swift-docc-render/pull/313
Issue: rdar://93517250
Code Reviewed By: @dobromir-hristov
Testing Details: Manual test in browser

@swift-ci test


How to train on my own image dataset?

I want to perform transfer learning to train the model with the pretrained weights on my own small image dataset. Let me know how to go about it.

I want to perform transfer learning to train the model with the pretrained weights on my own small image dataset. Let me know how to go about it.

Hi , I'm also working on my own dataset recently but don't know how to make my own HOI dataset. I'll be very grateful if u told me how to annotate my own dataset or what tools can be used to annotate HOI dataset. Thanks a lot.

I want to perform transfer learning to train the model with the pretrained weights on my own small image dataset. Let me know how to go about it.

Hi , I'm also working on my own dataset recently but don't know how to make my own HOI dataset. I'll be very grateful if u told me how to annotate my own dataset or what tools can be used to annotate HOI dataset. Thanks a lot.

Have you received a response? I also want to implement it on my own dataset, but I don't know how to annotate it.


Add documentation of variable types

I believe it's mostly correct but some are missing as I wan't quite able to figure them out. I hope someone more familiar with the project could finish it :)

Thanks for the contribution. I believe it will save many people's time.

Happy to help! :)


Branching strategy request

Hey,
I think the project could benefit from (at least) a develop branch.
This could be where you merge the PRs to before any final updates that make it ready for release (on main).

Makes sense to me. I've created a "develop" branch for us to use as suggested.


refactor(tasks.py): reduce dependency

put the dependency packages to tasks.

@william-silversmith I just added a few comment lines. not sure it is what you expected or not.

Hmm, I meant more like write why we are moving these imports in particular and not others. I can see that we're importing intern from the lines themselves.

@jingpengw Would you mind rebasing this on master? I think it will fix the travis problem.

@william-silversmith there are a bunch of conflicts. you should rebase your quantizedaffinities first?

oh my bad. you're right

rebased to your updated quantizedaffinities branch

maybe related
https://stackoverflow.com/questions/563022/whats-python-good-practice-for-importing-and-offering-optional-features
https://softwareengineering.stackexchange.com/questions/262697/is-it-safe-to-catch-importerror-when-trying-to-import-optional-modules

@jingpengw what shall I do with this?

cloud volume was split out, I think we can close this.


Update UndoRedo description

It was outdated after EditorUndoRedoManager was added.
UndoRedo is now general-purpose class, not tied to the editor.

add_do_method() and the likes now use Callables, so this needs updating in the example too.

Ok updated, but I don't know about the C# example.
I also added some examples to do/undo reference methods.

Nice! Wait I gotta ask though, don't these solutions leave orphan nodes? Or is this generally impossible to avoid with UndoRedo?

Thanks!

Cherry-picked for 4.0.2.


[ConstraintGraph] Don't try to contract edge of parameter bindings with inout attribute

Currently edge related to the parameter bindings is contracted
without properly checking if newly created equivalence class has
the same inout & l-value requirements. This patch improves the
situation by disallowing contraction of the edges related to parameter
binding constraint where left-hand side has inout attribute set.
Such guarantees that parameter can get inout type assigned when
argument gets l-value type.
Resolves: rdar://problem/33429010

@swift-ci please clean smoke test

@rudkx This version uses intersection of relational constraints associated with type variables to determine priority. Easily extendable to cover subtype, just needs another case statement...

@swift-ci please smoke test

@swift-ci please clean smoke test

@rudkx It looks like foo2([].bar { $0 == "c" }!) failing test shouldn't have type-checked at the first place, it did because we used argument type set by == (String, String) -> Bool overload, we can't really assume that the type of array is [String] just because operator == matched, WDYT?

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@xedin In theory I believe that really should type check under our bidirectional type checking. I think I mentioned before sometimes single-expression closure bodies do have an impact on what we select outside of the body.
Out of curiosity, what happens when you wrap that expression in another closure and change the array literal to something like [$0], without providing a parameter to that outer closure to help clarify what $0 is?
If that works, then perhaps it's reasonable to break the case where it's truly an empty literal...?

@rudkx Same thing, it looks like I have de-prioritized right-hand side binding too much, I'll try one more thing before moving in to other things...

After looking at the example:
import Cocoa
class C {
  init(a: CGFloat) {}
}
[0.2].map { C(a: $0) }

I'm certain now that it's not safe to postpone binding of the right-hand side of the BindParam nor it's safe to do that in presence of implicit conversions, even if the bindings where discovered on the right-hand side, it's only safe to attempt them using left-hand type variable.


Convert the data pipeline to GEE

Still a work in progress. Core code and notebooks live in the GEE directory

Merging the fork into a gee-pipeline branch on the main repo. More discussion to come when it comes time to merge gee-pipeline into main


META - Design a better JupyterLab launcher

Background
The JupyterLab launcher works well for traditional workflows where users work locally, more likely within a conda environment (or similar). Thus they'd end up with a handful of kernels (at most).
However, once that workflow breaks, as is what conda-store enables: local or remote development with many available kernels, the launcher becomes unusable by most folks:

The current card approach (Python or Kernel logo + kernel name) becomes redundant as one cannot distinguish kernels from that context
As more kernels and environments get added, the cards end up taking all the available space (and are repeated for notebooks and consoles)
Right now, the end-user cannot customize the launcher to adapt to their needs
There was a previous attempt at an enhanced JupyterLab launcher, but it is based on the same cards approach, and the significant improvement is achieved by merging the console and notebooks cards (plus it seems to be unmaintained now)

### Tasks
- [ ] Define requirements for such an extension
- [ ] Design: wireframes and mockup
- [ ] Dev: extension development
- [ ] Define requirements for user customisation

Thanks for opening this issue! We've discussed this a few times, but I did not realize we already had this issue to point folks to. :)
cc @dharhas - so you're notified of activity as this overlaps slightly with discussions in Nebari/JATIC spaces

Specific requests that have come up:

Need ability to Search in Launcher screen for kernels
Need ability to Search for environments in the kernel selection
Mike - We can create a workspace which customizes layout. It is saved as a json. We can push this to users so every launch will be the same. Will be in the 4.2 release.

Also relevant: Adding notebook "favorites" in the sidebar will be available in nebari soon

From @krassowski :
I know that this issue is about redesign, but I will also leave some implementation notes that might help with planning:

Need ability to Search in Launcher screen for notebooks

Searching for recently used files/notebooks would be easy to implement and might be accepted in JupyterLab core. Searching for arbitrary file on the server which may have hundreds of thousands of files will have a high maintenance cost because it has performance implications (and increases surface for denial of service attacks).
It feels to me that general purpose "search for a notebook" might be better placed in the file browser rather than launcher.
However, searching for a tile in the launcher sounds like be a good idea, especially if the search box was hidden by default (e.g. only showing when user clicks on a discrete search icon or presses a key combo). This is because most users will not benefit from a new search box (most will only have a few tiles), but some users with many kernel tiles (or even the "recently used file tiles" if we decide to include those) will.

Need ability to Search for environments in the kernel selection

This can be implemented in jupyterlab-nebari-mode extension by replacing (extending) the default provider of ISessionContextDialogsinterface (replacing @jupyterlab/apputils-extension:sessionDialogs plugin). It has full access to kernel specs, so any kernel-specific metadata that we might want to display (and search in) can be passed via kernel spec.
A basic search function could be contributed to the JupyterLab core by rewriting the default selection dialog. However, some advanced users are strongly opinionated on the existing sections in the kernel selection list so it would be easier to contribute back an improvement which does not change that too much.
For reference, this is the existing kernel selection dialog:

There will be "recent" notebooks available on the left sidebar coming native to jlab officially in 3-4 months but we could use this functionality in an extension that already exists.

For reference this refers to PR https://github.com/jupyterlab/jupyterlab/pull/15483

We can create a workspace which customizes layout. it saves as a json. We can push this to users so every launch will be the same. Will be in the 4.2 release.

For reference, this refers to PR https://github.com/jupyterlab/jupyterlab/pull/15946
On the launcher itself, there are two upstream tickets in JupyterLab already:

https://github.com/jupyterlab/jupyterlab/issues/3795 exploring redesign options
https://github.com/jupyterlab/jupyterlab/issues/12190 proposing to make the launcher more modular/ customisable

On the last point, it is worth considering if we should allow users to just drag widgets around the launcher to customize the layout it to their liking, especially if we are adding more optional things.

random thoughts.
One confusion I have seen in new users is that they expect the launcher to be a file-browser when they see lots of Jupyter icons (one for each python enviornment) they mistake it for a list of their Jupyter notebooks.
The two things I use the most in the launcher is

launch a new jupyter notebook
launch a terminal

Things I have never used:

new text/markdown/python file (I use nano/emacs from terminal or codeserver for these). But I can see why these should potentially stay
Show Contextual Help (I actually don't know what this does)
"Consoles"

We have had a client who customized the launcher to include shortcuts to favorites or example folders.
Specifically for Jupyter I feel like we need a two step launcher -> 1. Select Jupyter, 2. Choose environment
It would be nice to also do this for Terminal as well -> 1. Select Terminal 2. Choose environment

I’m following up upon request to bring you some very first passes at how the launcher could be adjusted to address the pain points listed above. These are not visually styled to match JupyterLab’s launcher at this point, only to make sure we are in agreement about the information the launcher needs, how that information should be oriented, and how the launcher should behave. Feedback on those aspects would be most helpful at this point!
Designs will also prioritize aligning with base JupyterLab’s launcher over being designed exclusively for conda-store-ui.
All directions replace the launcher’s cards with table-style lists. All directions also include a Search bar at the top that allows launcher contents to be filtered by text.
Option 1-1
Combines the Notebook and Console sections into one table and allows users to go kernel by kernel when selecting what they want to open. The Other sections become buttons to the side (or below, on narrower screens) to open new file types not related to kernels.

Option 1-2
Preserves the sections (Notebook, Console, and Other) already in the launcher, but replaces the cards with tables. Like the file browser, the whole row is a button to open the relevant session with a given kernel. The sections are now collapsible.

Option 1-3
Combines all launcher items into one table. The table is filterable by the search bar and by checkboxes to change what populates the table. While filtering came up when I was onboarding to this project, I was not clear what filtering would be most useful for this context; these are just example ideas that could be replaced.

Next steps
Feedback I’ve gotten in meetings means I’m going to move forward with the main concept of Option 1-2 for now, though other feedback on this issue is welcome and may change the way I take that next step. Thank you!

I have another update! Moving forward with feedback from option 1-2, I’ve changed the following.

Added a “starred” section that the user can populate with options from any of the other categories so they are quick to find.
Labeled the icon category as Kernel to clarify the purpose of the icons.
Added the caret icon to the columns of the table to mimic sorting like the File Browser does.
Added a column to “star” options in each of the other categories.
Added a scroll bar/limited height to each of the sections so that scrolling on the whole launcher is limited and users only have to interact in-depth with the section they want.

In action, that looks like the following.
Option 2-1 for JupyterLab
As a one-column list (for narrow screens or all if we choose it).

As a two column list.

Option 2-1 for conda-store-ui
We haven’t identified too much that we want to add uniquely to the conda-store-ui use of the launcher, but to demonstrate how other metadata can be added to this UI I’ve added a namespace option to the following mockups.
As a one-column list (for narrow screens or all if we choose it).

As a two column list.

Feedback on the last round of mockups pointed out that problems with the current card-based launcher approach are not only to handle large numbers of kernels and/or environments and/or namespaces in a way that helps the user better find the one they want, but also to help with “decision clutter” too. These new designs cover a kind of branching need of “I just want to open something so I can get to work with the fewest clicks possible” and “I know exactly what I want to open with the fewest clicks possible.”
Changes are as follows:

Added a title to the launcher to clarify what the launcher does (and that it’s not the file browser)
Added the current directory for users to orient where they are and where they may be adding new files or starting new work.
Changed section levels to be split by starting without a kernel and starting with a chosen kernel. These are still collapsible. Things originally in “Other” have been moved to
Sections in Open New By Type are cards again! They are smaller cards with shorter text, but back to being cards for easy spotting.
Favorites have been added to Open New By Type because they can be any category with or without kernels.
Sections in Open New By Kernel are still tables. Rather than each being their own section, they are grouped with subheadings for Notebooks and Consoles.

Option 3-1
Keeps everything from the previous options, but reorganizes it as mentioned above.

Option 3-2
Option 3-1 with Starred removed.

Option 3-3
Option 3-1 with Starred removed and no cards.

I will keep updating, next time with kernel selection updates as well. Thanks for reading and giving feedback!

I'm following up with a summary of the feedback the last round of mockups received for record-keeping purposes. Let me know if you have any questions!

Move search to Open New By Kernel section because that is where search is most needed. Make the empty field text be "Search for Kernel."
Add a title to the launcher (other than the tab name title). Maybe "Launch New…"
Label the file path as "Current directory"
Change VSCode icon to match Code Server
Add Python File to Open New By Type cards (@krassowski already did this above!)
Move Contextual Help to the top of page/in its own section since it's more a global setting than a new file.

We are going to pause here on design iterations and work on implementation. Once we get a POC in place and have some user experience to work with, we may revisit design. Thanks @isabela-pf for the great work!

Regarding the kernel selector there is also some designs in https://github.com/jupyterlab/jupyterlab/issues/14717 but I don't think they get us closer to what we want to achieve.

One more snapshot:

Should we move this issue over to the implementation repo - https://github.com/nebari-dev/jupyterlab-new-launcher/

Probably but since they are in different orgs, we can't do a "transfer". We'd just have to create a new issue and copy the relevant info.

Not sure whether to go for
(a) clicking on a row checks a checkbox (and then you press Select), or
(b) clicking on a row starts the kernel

I'd prefer fewer clicks when possible so I'd pick (b)

One blocker on using more metadata from conda is the PR to conda-nb-kernels being stuck.

Can you link to that work for reference?
Also, for the kernel selector, I think we talked about getting info from the metadata which would allow us to add some organizational UI niceties if they are available (e.g. namespaces in nebari or whatever other grouping people would like to use). Is that affected by the blocked work you mentioned?
Design for the conda env selection on jhub apps for reference:

The namespace will be available as a column. I guess you are suggesting having a filter by environment above the table, or in context menu over column?

I was responding to your "Select Kernel" UI. I wasn't aware that would be part of this work, but if it is, I thought I'd share the designs we had for that. If the namespace will be available as a column there as well, that will work too.

Yes, we need this component as well.

Should we move this issue over to the implementation repo - https://github.com/nebari-dev/jupyterlab-new-launcher/

I think this is a good time to evaluate the initial implementation (you can try it on binder: https://mybinder.org/v2/gh/nebari-dev/jupyterlab-new-launcher/main?urlpath=lab), start opening individual issues over on https://github.com/nebari-dev/jupyterlab-new-launcher and maybe close this issue.
The next step for me is to investigate improvements specific to conda-store (which columns to show, whether to split some metadata into multiple columns, think about filtering by environment etc) and implement changes based on feedback.

I agree! Great work!

Moving discussion over to https://github.com/nebari-dev/jupyterlab-new-launcher/issues/4 and closing this one!


Implement logging to file

#76 Added make file, and logger

Only small nitpicks. :+1:

Current coverage is 38.64%

Merging #102 into master will decrease coverage by -1.45% as of de9cab1

@@            master    #102   diff @@
======================================
  Files           24      24
  Stmts          656     709    +53
  Branches         0       0
  Methods          0       0
======================================
+ Hit            263     274    +11
  Partial          0       0
- Missed         393     435    +42

Review entire Coverage Diff as of de9cab1

Powered by Codecov. Updated on successful CI builds.

:+1:


Re-assign conditions

Description:
I noticed that at night the yahoo component was reporting sunny and yesterday on a clear day it was reporting fog.  Checking against what the actual API was providing (https://developer.yahoo.com/weather/documentation.html) there was a bit of misalignment.  The below table (gdrive link) shows the changes (highlighted in yellow)
https://drive.google.com/file/d/1Tpt9cftm_fkuMC_ChitftsLijKT1oVzF/view?usp=sharing
Related issue (if applicable): fixes #
N/A
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
N/A
Example entry for configuration.yaml (if applicable):
N/A no changes
Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
N/A, I think this is a simple enough change that a code review can suffice.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io
N/A

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.
N/A

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.
N/A, I think this is a simple enough change that a code review can suffice.

Nice!


Backup test

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug

Hi, owner of backup, I upgrated Compute API version recently. Some tests are marked record only and can not pass when I rerun them. So I skip them. Please help rerun them.
https://github.com/Azure/azure-cli/pull/14212
{Compute} Bump version disks 2020-05-01, compute 2020-06-01
To Reproduce

Expected behavior

Environment summary

Additional context

Backup

routing to appropriate team

@sambitratha : Can you please look into this or assign somebody else accordingly?

hi @qwordy  sure i'll take a look.

@sambitratha : So, is the required work done and can we close this?

Thank you so much!


Fix broken link to KEDA deployment

Description
Fix broken link to KEDA deployment in Autoscaling a Dapr app with KEDA
Issue reference
Closes #710

Thanks!

Happy to help!


App Revamp

Refactor:

Code Reorganized

Broke down the few big grand classes (esp views.py) into various smaller files.
Redid the folder structure setup for flask application.

Better Schema

Properly set up database schema relationships, backrefs, cascade for better consistency and less orphan rows.
Completely replaced application-level join and cascading delete rules with database-level constraints.

Documentaion

Wrote docstrings for most of the routes and helper functions.
Rewrote README up to date

Feature:

Multi-offering Feature

User can now login using Canvas account and the app should fetch a list of their enrolled courses (we call it offerings). Depending on the enrollment type (staff or students) of the offering they click into, they are able to work with the offering to different levels.
This allows only one central app (and one central db) to be hosted by us and used across all students or staff.

Multi-exam feature

Instead of fixing one course and one exam using a fixed set of environment variables (in the 2022 version of the app) we allow adding, updating and deleting exams for a single offering.

Replaced 61A Auth / Okpy Auth with Canvas auth

All routes and protected behind a Canvas login using url convertor
Authorization and permission checks (course enrollment, enrollment types) are all fetched from Canvas data.
User also do not need to login as staff and student separately - the system does not discriminate user groups - instead, we only do a course-specific permission check.

Canvas API

Added a wrapper for Canvas API that supports fetching user's courses / courses' users.
Wrapper allows stubbing API, which can be toggle from an env var MOCK_CANVAS
Refactored Canvas ID to be effective primary key as much as possible (replace db id, email etc)

Dev Login

Build a dev login and seeding data for two user scenarios (Yu as 169TA 168Student, Jimmy as 169Student, 168TA). This is useful when Canvas API is stubbed.

Frontend Improvement

Improved a few Jinja templates: base, student, students, to display more info
Revamped navbar to be more logical and breadcrumb-like
Add login/user status to navbar

Deleting student Feature

Added deleting single student feature
Added deleting all students feature

Devops:

Setup Env Var

Separated configuration classes for different environments (development, testing, production)
Added .env.sample

Dependencies

Figured out a working set of deps and pinned all deps
Bumped up Python to 3.10

Code Quality

Added formatter (autopep8).
Added and configured linter (flake8).

Testing

Set up testing and coverage framework with PyTest.
Set up UI/ end-to-end testing with selenium.
Added relevant flask tasks.

CI

Set up GitHub workflow for: linting
Set up GitHub workflow for: testing, coverage
Set up GitHub workflow for: auditing (with safety, pip-audit)
Set up CodeQL

I think we should use some "f-strings" in Python,

Okie! It seems that f-string is the new preferred way, better than str.format in performance, and definitely better in style than concatenation.
I will do a global replacement sometime.

Yeah, no rush - more just like a factor of iteratively improve the app. :)


Couldn't run the rake task

bundle exec rake jshint:copy_config
rake aborted!
Don't know how to build task 'jshint:copy_config'
(See full trace by running task with --trace)

+1

+1

@ysihaoy I'm sorry, I don't use this project any longer. Would you be interested in maintaining it?


Description and title not getting appeared on shared link in linkedin

When passing props title and description to social sharing component , it is not getting displayed in linkedIN. It is taking page title tag to show title rather than title prop that has been passed.
CODE :
              <social-sharing url="https://github.com"
                   v-bind:title="'Event : ' + event.title"
                   v-bind:description="'Event Name : ' + event.title + '\n' + 'Event Date : ' + formatDate(event.date) + '\n' + 'Event description : ' + event.event_description + '\n'"
                   v-bind:quote="'Event Name : ' + event.title + '\n' + 'Event Date : ' + formatDate(event.date) + '\n' + 'Event description : ' + event.event_description"

                  inline-template>

                   <network network="facebook">

                    </network>
                     <network network="twitter">

                    </network>
                       <network network="linkedin">

                    </network>
                    <network network="email">

                  </network>

              </social-sharing>

Hello @srishti-sqm,
Facebook/Twitter/LinkedIn use their own graph API to fetch the data used in the sharing dialog, to customize the shared content you need to add meta tags on your page.
More info here #69

But How can we update the title and url(dynamic title and url link to share) each time linkedin is clicked to share. And even it is stripping query params in url prop in linkedin share.


I cannot upload bedrock worlds from <IP_ADDRESS>

[I cannot upload bedrock worlds from <IP_ADDRESS>

https://drive.google.com/file/d/1gCThfltkW-flccpkF3E6lzqfPMNEmLD2/view  ]

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

ID: 9f2fb27f-0ce1-9541-e617-7fe8148f8297
Version Independent ID: 6777fa19-93ad-1984-5fd8-a9adc3dc3a3a
Content: Chunker - Overview
Content Source: creator/Documents/ChunkerOverview.md
Product: gaming
GitHub Login: @neonerz
Microsoft Alias: v-jeffreykim

Thanks for reporting this issue. Currently, Chunker.app has a file size limit which is exceeded by the file you're attempting to use.
We plan to significantly raise this limit by the end of November, which should resolve this issue for you. We will also improve the error messaging where possible to prevent this confusion.


[Hexagon] Don't use {} initialization with FastRPC structures

The data members in FastRPC structures aren't guaranteed to remain in the same order. Replace aggregate initialization with direct, member-by-member initialization.

@csullivan : This could fix the signature verification error you were seeing.

Thank you @kparzysz-quic @csullivan the PR has been merged


[SPARK-37936][SQL] Use error classes in the parsing errors of intervals

What changes were proposed in this pull request?

In the PR, We propose to throw ParseException from below methodswith the error classes:

moreThanOneFromToUnitInIntervalLiteralError
invalidIntervalLiteralError
invalidIntervalFormError
invalidFromToUnitValueError
fromToIntervalUnsupportedError
mixedIntervalUnitsError

MORE_THAN_ONE_FROM_TO_UNIT_IN_INTERVAL_LITERAL - moreThanOneFromToUnitInIntervalLiteralError
INVALID_INTERVAL_LITERAL - invalidIntervalLiteralError
INVALID_INTERVAL_FORM - invalidIntervalFormError
INVALID_FROM_TO_UNIT_VALUE - invalidFromToUnitValueError
UNSUPPORTED_FROM_TO_INTERVAL - fromToIntervalUnsupportedError
MIXED_INTERVAL_UNITS - mixedIntervalUnitsError
New error classes are added to error-classes.json.
New test suite QueryParsingErrorsSuite  for checking the errors has been created.
Why are the changes needed?

Please refer SPARK-37935 - Migrate onto error classes
Does this PR introduce any user-facing change?

Yes
How was this patch tested?

By running new test suite:
build/sbt "test:testOnly *QueryParsingErrorsSuite"

@MaxGekk Could you please review this PR?

Can one of the admins verify this patch?

Closing this PR and creating a new one


Update workflow for deploying docs from main repo

🧰 Task
Since #5216, we have docs in the github.com/napari/docs repo. As we were planning this move, we thought that this meant that docs need to be built in that repo and then deployed from there. But actually, we can just build in either place:

in the docs repo, we clone the current napari/napari@main, and then build the docs with the current main or PR within that repo, and (optionally) deploy.
in the main repo, we just clone the current napari/docs@main, then build the docs with the current main or PR, and then optionally deploy.

Since both repos have deploy keys to napari.github.io, both can do this. As far as I can tell, you won't get into race conditions here — later PRs in one will contain the latest commit in the other one, even if that one has not yet finished building. It could happen that one build hangs, but that can happen already — a manual workflow trigger would solve this, or we can just wait till the next PR.
The advantage of this is that we don't need to do the remote workflow trigger, which requires my personal access token, which is yucky.

So, is the idea that this build docs workflow from the napari/napari repo would build and deploy to napari.github.io on every PR, include those that don't touch the docs? If that's the case, copying the build docs workflow from the napari/docs repo (with very minor changes) would be enough.

Every merged PR, yes. 😅 I think that should work? And yes, I think the changes are just related to which checkout goes in which directory but otherwise the action is basically unchanged I think. 🙏

SO the idea is to replace: https://github.com/napari/napari/blob/main/.github/workflows/deploy_docs.yml
which is triggering the deploy in napari/docs with a deploy step in the build workflow?
https://github.com/napari/napari/blob/main/.github/workflows/build_docs.yml
So basically what we have in napari/docs now:
https://github.com/napari/docs/blob/main/.github/workflows/build_and_deploy.yml

that's my idea, yes. I don't know however whether it is the right idea. 😅 If the last few weeks have taught us anything it's that the interaction of many workflows and various race conditions can have unexpected consequences. 😅

I think the latest changes in the napari/docs workflows (napari/docs#462 and napari/docs#463), which allow deploying from triggered workflows, should be enough to resolve what this issue was meant to resolve? So I think actually we can close this. Please reopen if I'm mistaken. 😅


BUG

Describe the bug
Cannot install xinference from git repository. It throws an error.
ChefBuildError

  Backend operation failed: HookMissing('build_editable')

  at ~/.local/share/pypoetry/venv/lib/python3.11/site-packages/poetry/installation/chef.py:152 in _prepare
      148│
      149│                 error = ChefBuildError("\n\n".join(message_parts))
      150│
      151│             if error is not None:
    → 152│                 raise error from None
      153│
      154│             return path
      155│
      156│     def _prepare_sdist(self, archive: Path, destination: Path | None = None) -> Path:

To Reproduce
Install the xinference from

3.11
git latest version
Full stack of the error.

ChefBuildError

  Backend operation failed: HookMissing('build_editable')

  at ~/.local/share/pypoetry/venv/lib/python3.11/site-packages/poetry/installation/chef.py:152 in _prepare
      148│
      149│                 error = ChefBuildError("\n\n".join(message_parts))
      150│
      151│             if error is not None:
    → 152│                 raise error from None
      153│
      154│             return path
      155│
      156│     def _prepare_sdist(self, archive: Path, destination: Path | None = None) -> Path:
Note: This error originates from the build backend, and is likely not a problem with poetry but with xinference (0.12.2.post1+12.g2a64f4b

Minimized code to reproduce the error.

Expected behavior
Install xinference

@vikrantrathore Could you paste out your command that you pip install from git?

The error is through poetry which uses pip behind. The command is poetry add git+https://github.com/xorbitsai/inference within a local project.
Once the installation fails at installing xinference, I can go activate the venv by poetry shell and then install using pip install git+https://github.com/xorbitsai/inference. It works, but then there is an error of Web UI which is using npm and need to run
npm install
npm run build

This will build the UI. Also vllm used is 0.4.3 and wanted to try 0.5.0 from the git given it supports gemma-2 9B and 27B but couldn't do it due to other dependencies.
Also sglang support gemma-it but when I use sglang worker for gemma-it model it throws an error model not supported so was trying to install it from git directly to see if can use gemma-it with sglang.

Ok after doing a second try by removing the vllm from the git it started to work now,

It seems the error still persist when I try to install it from local directory like poetry add -e ../inference/.["all"], probably need to test more extensively with poetry.


change peerDeps to allow for react 17

Is there any hard reason for it? It seems to work fine but npm is not liking the requirement when run under a react 17 codebase. Thanks in advance.

Is there any reason for not updating your app to React 18? I'd like to keep the peerDeps at 18+

It's a huge codebase as platform for 30+ apps and the changes (in particular, dev mode hook rerenders etc) cause a huge risk, don't think we can update any time soon.
Worst case, I may fork and republish this as a namespaced package and somehow keep it in sync with yours

I see, I'd waant to keep the peerDeps at 18+ so feel free to fork!

Were you able to make it work with React 17?
I want to use this with react 16 🫠

For anyone whose interested:
This can be achieved via adding "override" field in the package.json:
"overrides": {
    "sonner": {
      "react": "$react",
      "react-dom": "$react-dom"
    }
  }

This can ensure whenever you upgrade your react version, this works as well without the need to fork.


Change how the inventory is opened and closed

Currently, the inventory is opened via InventoryUIClientSystem in onToggleInventory. For seemingly no reason it is closed in InventoryScreen in onBindEvent.
The closing section in InventoryScreen appears to be redundant as when onBindEvent is removed the inventory opens and closes as normal. I am aware that I might simply be missing something which explains the logic behind this.
Also, removing onBindEvent also allows for other screens to be opened and closed alongside the inventory screen, as I've done in my fork with potionStatusScreen (See files PotionStatusScreen and PotionStatusUISystem)
(Also, I'm sorry if this is the wrong place for this, but I didn't think it fit on the forum)

Can one of the admins please verify this patch?

This is a fine place for this and I'm very curious how it works out as we have a known issue about opening/closing some screens - see https://github.com/Terasology/JoshariasSurvival/issues/15 for the primary example and some more thoughts.
Might take a little extra testing to figure this one out (unless maybe @portokaliu can spot it immediately)
@GooeyHub: add to whitelist

Hooray Jenkins reported success with all tests good!

I've tested this out with a few interaction screens and it seems to work the same as before.
onBindEvent seems to be called at a higher level in NUIManagerInternal whenever a raw input event is received. However, both pieces of code seem to accomplish the same thing.
This onBindEvent code was added in 3c02cfb0f4f688cad207b33c6adefbd063315e93 along with ContainerScreen even though the part about nuiManager.toggleScreen() already existed then. Additionally InventoryScreen seems to be the only class that actually implements onBindEvent
I'm not exactly sure what all this means, but just throwing what I found out here. 🙂

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Tested this out along with #2703 and inventory stuff seems working as well and better (no double e) than before :-)
@jellysnake do you have your potions UI piece ready? @iojw mentions in #2703 that we might want to make any one interaction screen opening first close all other ones which may well make sense. Would your stuff count as a background element like the toolbar? Which would sound sensible.


Document .keys()

Hi,
I noticed .keys() is already implemented but not yet documented (c.f., #440)

Thanks for your review!


Certain posts aren't showing up on the Group map

Posts from members in the Holochain Advocates Community aren't showing up on the Group Map. Certain members have posted events and offers, though only members are displaying.

looking through all the posts in the group i dont see any ones with locations, can you double check with Anita @KrishaTerranIO ?

after talking to anita it seems this is not actually broken


My Spder Background is complete black

Description
What steps will reproduce the problem?

When I installed anaconda after when I try to use spyder I can not see anything except black screen.
I have read:
https://github.com/ContinuumIO/anaconda-issues/issues/9597
https://github.com/spyder-ide/spyder/issues/7327
I tried : conda install pyopengl
from anaconda prompt
Versions

Spyder version: 3.3.1
Python version: 3.7.0
Qt version: 5.9.6
PyQt5 version: 5.9.2
Operating System: Windows 2008ServerR2

Dependencies
pyflakes >=0.6.0  :  2.0.0 (OK)
pycodestyle >=2.3 :  2.4.0 (OK)
pygments >=2.0    :  2.2.0 (OK)
sphinx >=0.6.6    :  1.7.9 (OK)
rope >=0.9.4      :  0.11.0 (OK)
jedi >=0.9.0      :  0.12.1 (OK)
nbconvert >=4.0   :  5.4.0 (OK)
pandas >=0.13.1   :  0.23.4 (OK)
numpy >=1.7       :  1.15.1 (OK)
sympy >=0.7.3     :  1.1.1 (OK)
cython >=0.21     :  0.28.5 (OK)
qtconsole >=4.2.0 :  4.4.1 (OK)
IPython >=4.0     :  6.5.0 (OK)
matplotlib >=2.0.0:  2.2.3 (OK)
pylint >=0.25     :  2.1.1 (OK)

This should be solved in our next release (3.3.2). In the meantime, please close Spyder, open the Anaconda Prompt, run there:
conda install qt=5.6 pyqt=5.6 sip=4.18

and try again.

@ccordoba12 Thank you so much, it solved my problem, now I can see IDE.
Was it because of I had 2008ServerR2 and something was related to my graphic card?

I think it's something related to the drivers of your graphics card, or maybe to your DirectX version. Let's hope it's fixed in our next version.


How to update/ sync schema changes or remove RDS data source added through add-graphql-datasource?

Which Category is your question related to?
Graphql API / RDS Aurora
Amplify CLI Version
4.46.1
What AWS Services are you utilizing?
Auth, Storage, Graphql , Video
I have added a RDD data source using amplify api add-graphql-datasource , which has successfully generated schema and respective resolvers. After that I modified the RDS mysql database running sql statements through aws console and ran amplify api add-graphql-datasource  and amplify push gives me following error, If I undo the changes amplify push works fine otherwise timeout as seen below:

⠼ Uploading files...✖ An error occurred when pushing the resources to the cloud

Your socket connection to the server was not read from or written to within the timeout period. Idle connections will be closed.
An error occurred during the push operation: Your socket connection to the server was not read from or written to within the
timeout period. Idle connections will be closed.

So right now I am struggling to find out how to

Delete RDS datasource , since I am using Dynamodb  already and dont want to delete that.
Sync changes either from RDS database or update RDS database through changes to models in rds.graphql

thanks

Hi @sakhunzai
This seems like a network problem from the error , can be due to large file size.
Currenlty there is no flow in cli to remove the RDS database . For the workaround you can try these steps:

Modify the schema so that it doesnt contain RDS related data
Delete the RDS stack generated by amplify cli present in api/<apiName>/stacks/<RDSstack>

This will remove RDS connection with API.
Let me know if this works for you?

Hi @akshbhu,
Long story long, I needed to remove RDS after I added it via amplify api add-graphql-datasource and have followed the suggestions you'v provided. The RDS stack file came back.
In addition, I ran a search for the keyword "rds" in the ../backend/ folder and deleted anything relevant found within those files, including lines in:
nested-cloudformation-stack.yml (2 files)
backend-config.json
cloudformation-template.json (2 files)

after that push failed to run, I again deleted those entries since they were added back. I searched in the rood project folder for rds and found something relevant in the
team-provider-info.json

file and deleted those lines.
Finally, after that push didn't work, I again went through the list of files above then deleted all types that had @model directived in my:
schema.graphql

file.
Obviously that errored with "no top-level..", but I added a small @model type and pushed without error. Then I added all other @model types back one-by-one and I think that seemed to work (pushed without errors). Probably could add them back all at once but at that point I was too deep in the woods and wanted to step out carefully.
So, after all of that, I would say a remove-graphql-datasource would be a valuable feature add! ;)
-bp


network_server example - Callback example wrong

From my understanding and trying to use nanopb, it would seem the network_server example code does not set the callback in the way that is now the preferred method. Ie, it seems that it assumes that the function
-> bool [messageName]_callback (pb_istream_t *istream, pb_ostream_t *ostream, const pb_field_iter_t *field)
is used, with no need to set the callback pointer. However it seems that nanopb actually uses a  pb_callback_t structure to define the callback for value types that require callbacks
-> [messageType].value.funcs.encode = &callback;
where callback is
-> bool (*encode)(pb_ostream_t *stream, const pb_field_t *field, void * const *arg);
The example should be fixed as it's very confusing to persons new to the library.

The way used in the example is actually the preferred way since 0.4.0: https://jpa.kapsi.fi/nanopb/docs/whats_new.html#callbacks-bound-by-function-name
At least maybe. The pb_callback_t will remain supported and may be easier for some use cases. Sorry if it is confusing, I have a terrible lack of time & motivation towards improving the documentation.


Full width

Hey,
How could the player use all avaible space ?
Using:
{
  width: '100%',
  height: '100%'
}

doesn't force the player use all the container's space. (it is even smaller)
Thanks for any answer

https://github.com/tjallingt/react-tjube/blob/master/src/components/Player/Player.jsx/
https://github.com/tjallingt/react-tjube/blob/master/src/components/VideoAppPlayer.css#L1-L8
In combination with the highlighted CSS applied to your YouTube component you should be able to get it full screen.

Thank you for this!  Just used and worked like a charm.  And also thanks for maintaining this, it's really great!


TimeMapper without Twitter?

In light of the chaos at Twitter, we can't take it for granted that everyone has or wants to have an account there, or that they'll even exist in 2023. What prospects are there for a way to sign in to TimeMapper that are independent of Twitter? Thank you.

I did some work to make it possible to use TimeMapper without an Twitter account and there is now an experimental version where you can use TimeMapper even without hosting your spreadsheets on Google. Suppose that you have a google spreasheet, you can visualize it as a timemapper using:
https://aivuk.github.io/TimeMapper2/?url=URL_TO_A_CSV_OR_GOOGLE_SPREADSHEET
Example:
https://aivuk.github.io/TimeMapper2/?url=https://docs.google.com/spreadsheets/d/1V94J2fN9bAA6mSw_o1cIOVsP7EYOdn2jQChshGRCi3M/pub?output=csv
Instead of using Google Spreadsheet you could also provide an URL  where your CSV is openly accessible:
https://aivuk.github.io/TimeMapper2/?url=https://raw.githubusercontent.com/aivuk/TimeMapper2/master/docs/battles.csv
In that way an user can use TimeMapper without any account and share the link above. The only thing that we would miss is a page listing all the TimeMaps created by an individual user.


COUNT based triggers fail with an unknown column error

When adding a trigger with a query that has a calculation of op = "COUNT" and no column it fails with the following error: 400 Bad Request: unknown column

Ah, the issue seems to be that we set an empty string in column, while this field is a *string and should be set to nil.
Here we assign this field: https://github.com/kvrhdn/terraform-provider-honeycombio/blob/main/honeycombio/data_source_query.go#L122
The Terraform SDK will return the default value for properties that are not set, in this case that's "".
This column field is set to json:omitempty: https://github.com/kvrhdn/go-honeycombio/blob/main/query_spec.go#L28
But since it's a pointer, "" is considered a non-empty value.
I should be able fix this later today. Thanks for the testcase!


confusing and annoying names

example: SignslRHub should be renamed to Hub as it can be part of the SignalR namespace, SignalR should also be renamed t o Client

Fixed in v0.2.


CP_System_GetWindowFocus - Add Documentation

CP_System_GetWindowFocus is a new function added in response to Issue #22.
We need to add documentation to the Wiki.

Complete
https://github.com/DigiPen-Faculty/CProcessing/wiki/System#CP_System_GetWindowFocus


allow back-references in String#sub & #gsub

Provides #684
As was suggested, a named argument is used as a flag to enable back-reference scanning. Literal interpretation of the replacement string remains the default.

Thinking about this, I'm not sure if the argument is clear on the call side, it's verbose without any information about what it does unless you're familiar with the method. Maybe we can find a different method name? Keep sub, gsub as is, add rsub and grsub that does the replacement pattern matching? Admittedly not great names, r would be a mnemonic for "replace" or "references".
In any case I'd like to see feature parity for sub here :)

Good points. I'll amend with the quotes and new method names.

Restored #sub and #gsub to their original implementations, added #rsub and #grsub, placed double-quotes around substrings in docs, and rebased to get CI passing again.

LGTM, I'd like a second opinion on this though.

I like back references so :+1: for the feature! I'm not sure about naming, though. I'd like @asterite to chime in.

I think we can add this, but I would like just gsub and sub, which would check back references automatically, unless explicitly told not to do so (so the happy powerful path is turned on by default, but you can make it faster in case it's a bottleneck in your app). The implementation in this PR is OK but can be improved a lot in terms of performance. Right now with this code:
time = Time.now
200_000.times do
  "foofoofoofoo".gsub(/o/, "xyzxyzxyzxyzxyzxyzxyzxyzxyzxyzxyzxyz")
end
puts "gsub: #{Time.now - time}"

time = Time.now
200_000.times do
  "foofoofoofoo".grsub(/o/, "xyzxyzxyzxyzxyzxyzxyzxyzxyzxyzxyzxyz")
end
puts "grsub: #{Time.now - time}"

I get:
gsub: 00:00:00.7161840
grsub: 00:00:01.8086300

I was able to reduce it to:
gsub: 00:00:00.7063900
grsub: 00:00:00.7627250

If the replacement string has a "\0" in it, the times for grsub go from 1.67s to 1.13s.
For this, grsub first checks if there's a slash in the replacement, otherwise there's no need to check for backreferences over and over. Then I used a Char::Reader, because indexing a string is super expensive. Finally, I avoid creating intermediate strings and just append to the only string that's being generated (I added a bunch of code for this, and refactored it).
So, if you'd like so, I can continue working and this and make backreferences work by default, without different overloads.
However... are backreferences that useful? I never used them in my life. And right now you can do them anyway, although in a bit longer way:
# proposed
 "foo".rsub(/o/, "x\\0x")

# current way
"foo".sub(/o/) { "x#{$~[0]}x" }

# proposed
"foo".rsub(/(o)/, "x\\1x")

# current way
"foo".sub(/(o)/) { "x#{$1}x" }

So this functionality is already present, you just need to do it in a slightly different way.

@asterite here is my use case. I can't use blocks, because they wouldn't access the $~ variables, thought I guess I could have plural(/(hive)\Z/i) { |m| "#{m[1]}s" } and a macro could abstract it away.

@asterite sorry, you're right, my implementation is not well optimized. Do you want me to keep working on this? It sounds like you've already put significant effort into the implementation, should I update the docs and specs to match the API you have in mind?
I'm not strongly in favor of the feature being accepted, but if it's rejected, then #684 can certainly be closed.

+1 for back references enabled by default, this is what I would expect. And I favour it much over using "magic globals".

@bjmllr Oh, no need to say sorry :smile_cat: What I meant is that using your implementation I was able to optimize it a lot very quickly (I reused 95% of your implementation), so the possibility of supporting back references by default sounds like a good idea.
I'll submit a separate pull request that basically does what you did, but slightly faster, and does it with sub and gsub, with an option to turn it off. I didn't submit any code because I'm still missing the "\k" case.
@ysbaddaden how does that "\1" work? Shouldn't that be "\1"?

@ysbaddaden As a side note, you can assign to $~ to forward its value: https://play.crystal-lang.org/#/r/tga
It's a bit contrived, but reading from $~ reads from the variable that was assigned because of a method call inside the current method (basically the variable in scope). But writing to $~ defines it as a variable in the caller. The same applies to $?.
The compiler implements this like this: if we assign to $~ inside a method, the method receives an extra $~ variable as a pointer argument. If a method calls that method, it declares that variable and passes a pointer to it. You can check that with --emit llvm-ir.

Closing in favor of #2253


Use [package] in Cargo.toml

This is what various tools expect, like cargo outdated.

Interestingly enough: cargo itself does accept both variants.

What's the difference between the two keywords?

Sorry for merging it that quickly.
package is the one that is mainly used by cargo and is also the one a cargo new will generate.
I suspect [project] is there for legacy reasons (seems there are test cases on the cargo repo for it). I right now don't have time to track it down further.

Ok thanks. No problem for the merge, as long as you know what you're doing :)

"Know what I'm doing" based on "it still compiles and is what cargo does", so yeah :)

Ah ah, at least it still compiles—could be worse ;) Thank you both!


ADD missing ARO mappings to manual curation

Additions:

argannot_curation: (Phe)cpt_strepv:U09991:AAB36569:1412-1948:537 -> ARO: 3000249
megares_curation: MEG_2114 -> ARO: 3000249, MEG_2430 -> ARO: 3000016, MEG_985 -> ARO: 3000229, MEG_2865 -> ARO:3000387, MEG_7974 -> ARO:3000076
sarg_curation: AM180355.1.gene2260.p01 -> ARO: 3000250
resfinder_curation: dldHA2X_1_AL939117 -> ARO:3003970, grdA_1_QJX10702 -> ARO:3007382, EstDL136_1_JN242251 -> ARO:3000387
resfinderfg_curation: UDP-N-acetylmuramoyl-tripeptide--D-alanyl-D-alanine ligase|KF629588.1|pediatric_fecal_sample|CYC -> ARO:3003970

Right now, the resfinder table has a column for Description, but none of the other ones do. Ideally, this would be more common, but maybe we can start adding descriptions for the new entries going forward?

Looks good. Can you rebase on the current main?

I have gone ahead and rebased the commits. It should be ready to merge now


NOISSUE -  Add proplet docs

What type of PR is this?
This is a documentation update

What does this do?
Adds Proplet docs

Which issue(s) does this PR fix/relate to?
No issue

Have you included tests for your changes?
No

Did you document any new/modified features?
Yes

Notes

Branch from #1 and add it at the appropriate place

resolved here: https://github.com/absmach/propeller-docs/pull/3


批曰。編譯有誤

Describe your issue/suggestions
批曰。「「吾嘗觀「「易經」」之書」」

編譯得
/*"易經"*/

而應得
/* 吾嘗觀「「易經」」之書 */

另外，建議增加塊注釋(block comment)
長批曰。
    ……
    ……
    ……
長批至此。

Checklist
Before you create this issue, Please make sure ALL the following items are checked

[x] If this is a feature request, it's NOT listed Here
[x] If this is a bug report, it's NOT listed Here

Hi @cuixiping ,
Nested 「「 are not supported yet, but will be among my very next things to add.
I think once we support that, then we already have block comments:
注曰「「
一行
一行
又一行
三四五六七八行
」」

Thanks!

Now nested comment is supported: 32b0f3abd1beb55cd28c369ad74b79e677248cc7
You can nest comments as much as you like:
批曰。「「「吾」嘗觀「「「易」經」」之「「「書」」」」」

However, the 「」's need to be matched even inside the comment. Otherwise there is no way for the parser to see where the comment ends.
Thanks!

Great!

Fixed in v0.1.2. Closing now. Feel free to reopen it if you got any questions.


The Chrome Store install link doesn't work

The Chrome Store install link in the docs returns a 404.

Thanks. I have fixed the link


[VectorDistribution] Fix 0-rank vector.broadcast distribution

Fixes: https://github.com/iree-org/iree/issues/18955
Also removes locally carried revert

Do we have any tests for rank 0 vectors/broadcast

added test


If file is missing, Webpack stays in error state even after it is created

This has bugged me for months because it requires restarting dev server which is taking a loong time.
If I accidentally mistype a require, Webpack will enter an invalid state:
ERROR in ./src/scripts/atoms/CheckLink.jsx
Module not found: Error: Cannot resolve 'file' or 'directory' ./CheckLink.less in /Users/dan/Documents/Projects/whatever/src/scripts/atoms
 @ ./src/scripts/atoms/CheckLink.jsx 11:0-27
webpack: bundle is now VALID.

Now even if I add this file, Webpack won't ever see it, even though it will trigger recompilation.
I have to restart dev server for changes to take effect.

I'm not sure if this happens every time though.
I'll try to give exact steps to repro later today.

I know this bug... It's caching the fs access...

can you test if this still occurs in<EMAIL_ADDRESS>

I will on Monday. Thank you.

It may also be related to the jsx-loader. The sass-loader for example had this problem, because we were unable to mark the dependencies correctly. When a syntax error occurred, node-sass just didn't report the malformed file.

This is fixed in 1.5 indeed, thank you!
@jhnns How to reproduce this scenario? I think missing files are now picked up by Webpack but I'm not sure about errors involving syntax errors—can you give me steps to check that?

Write a SyntaxError, correct it and check if the file is compiled by Webpack again.

This has been working for me for quite a while, but I'm using NoErrorsPlugin. Not sure if there are any problems without it..


alignment of text

Our facilities text should be centrally aligned. I want to fix this issue
Please assign this issue to me under gssoc-extd and hacktoberfest.

@Kajalmehta29 It's alignment is absolutely correct. The division is further divided into two division and the text is aligned with the text of 2nd division. I appriciate your effort but try to find some other issues.


ADD: Possibility to also browse files on camera

When pairing to do keygen/keysign, please add file picker so it is possible to browse files and pictures. Like that it is possible to do keygen/keysign with shared QR codes and user has full flexibility.
Reason: If a user sharse a QR over airdrop from iPhone/iPad the files can't be imported into the app as the photo gets save in downloads and with the button we just have access to the gallery.

https://github.com/vultisig/vultisig-ios/pull/755


TransferManager::RetryUpload() does not check for unexisting file.

While TransferManager::UploadFile() appears to check for the existence of a file and returns a FAILED status immediately, a subsequent call to TransferManager::RetryUpload() does not appear to do this, and then the transfer incorrectly finishes with status COMPLETED. My expectation is that when attempting to upload a file that does not exist, the transfer should always end with status FAILED, no matter how many times RetryUpload() is called, if ever. This is using the overloads of UploadFile() and RetryUpload() that take the file path as a string. Please see comments inline in the code example below to see how this happens.
Minimal code example that reproduces the issue
using namespace Aws::S3;
using namespace Aws::Client;
using namespace Aws::Transfer;

std::string filename = "does-not-exist.zip";

std::string accessKeyId = "key";
std::string secretAccessKey = "secret";
std::string sessionToken = "token";
std::string bucketName = "bucket";
std::string region = "us-west-2";
std::string keyName = "keyname";

Aws::SDKOptions options;
Aws::InitAPI(options);

Aws::Auth::AWSCredentials credentials;
credentials.SetAWSAccessKeyId(accessKeyId);
credentials.SetAWSSecretKey(secretAccessKey);
credentials.SetSessionToken(sessionToken);

ClientConfiguration client_config;
client_config.region = region;
auto client = Aws::MakeShared<S3Client>("S3Client", credentials, client_config);

Aws::Utils::Threading::DefaultExecutor executor;
TransferManagerConfiguration config(&executor);
config.s3Client = client;
auto transfer_manager = TransferManager::Create(config);

// This is OK.
// UploadFile() checks for the existence of the file in CreateUploadFileHandle().
auto transfer_handle = transfer_manager->UploadFile(
    sourceFile,
    bucketName,
    keyName,
    "application/zip",
    Aws::Map<Aws::String, Aws::String>()); // no metadata

// Just wait, don't abort.
transfer_handle->WaitUntilFinished();

// OK, it failed.
bool success = transfer_handle->GetStatus() == TransferStatus::COMPLETED;
printf("TRANSFER HANDLE STATUS: %d", transfer_handle->GetStatus()); // 4, FAILED. This is OK.

// This doesn't appear to check for the existence of the file.
transfer_handle = transfer_manager->RetryUpload(sourceFile, transfer_handle);
transfer_handle->WaitUntilFinished();

// And now it succeeds, even though the file (does-not-exist.txt) still doesn't exist.
printf("RETRY STATUS: %d", transfer_handle->GetStatus()); // 5, COMPLETED. But the file still doesn't exist.

Aws::ShutdownAPI(options);

What platform/OS are you using?
Windows
What compiler are you using? what version?
Visual Studio 2017 with v140 (2015) toolset.
What's your CMake arguments?
Defaults except for:

BUILD_SHARED_LIBS = false
FORCE_SHARED_CRT = false

Will look into it and have it fixed as soon as we can.
Thank you

Awesome, thanks! I would pull some merge requests myself, but I am not yet fully acquainted with your SDK, so for now I'm just reporting what I can spot :)

Hi this bug should have been fixed, https://github.com/aws/aws-sdk-cpp/commit/8082ace481931523a8ed8967f1cda163443dcee9
Reopen if you still face this problem


Successfully patched

Will update later.

fixed in 1.1.1


chore: point dev to default arlocal endpoint

Was this just to bump the firebase preview?
We can probably close this identical PR/branch now: #347

Just me testing things. Still getting problems with arlocal, so closing this until open another PR and have a ticket to track it.


Additional Templates

Currently on react, vue and vanilla are added in the library. Ideally, we should support a wide range of templates.

[ ] preact
[ ] angular
[ ] svelte
[ ] react-typescript
[ ] static (no bundling at all?)

The long term feature would be to dynamically fetch template data from codesandbox and avoid hardcoding the files and dependencies.

Typescript templates are much needed, they are becoming increasingly popular. A great feature would also be loading user-defined templates or even loading them from a Github repository.

closing this as we want to integrate the template fetching as soon as possible


WFC3 README not updated on front end

The WFC3 REAME was updated in the repo with the new notebook under "Point Spread Function". However, those changes were not reflected on the front end site. Any reasons why?

Hello @FDauphin, it appears that the gh-storage has not been updated to reflect changes on the gh-pages. To address this, could you kindly create a ticket for the CI team to investigate? You can visit https://stsci.service-now.com/dmd, select SPB notebook-ci, and create the ticket there. Additionally, please tag me in the ticket so that I can stay informed about the progress. Thank you!

Issue has been addressed.

gh-storage branch is only used to store notebooks with generated outputs, readme's etc while they may be there from the initial branch creation are note used.


The remove function works on other objects, but not text

I can remove objects with engine.remove(obj), but when I try this on text, it doesn't work.

I agree with this change but it's not longer relevant to the new Sprig system.


coloredlogs is imported but is missing in requirements.txt

Rasa NLU version (e.g. 0.7.3):
master
Used backend / pipeline (spacy_sklearn, ...):
Operating system (windows, osx, ...):
Issue:
coloredlogs is imported but is missing in requirements.txt and the latest docker image.
https://github.com/RasaHQ/rasa_nlu/blob/67d5be534eff6ffb227886fe2f10e6dd7475ee85/rasa_nlu/utils/__init__.py#L289

yeah that doesn't seem right! could you please create a PR?

PR #948 was merged


HTR build failure in CI

Reported by @mariodirenzo:

All my CIs based on the CMake build of Legion have been failing with errors like
/home/gitlab-runner/legion-debug-cmake/runtime/realm/gasnet1/gasnetmsg.cc:1665:
 undefined reference to `gasneti_thunk_tm'

The error appeared after https://gitlab.com/StanfordLegion/legion/-/merge_requests/1427 was merged.

Mario, can you provide any additional details on the specific CMake command line, and the system (OS, CUDA, CMake, etc. versions)?

@mariodirenzo Please retest this on latest master, we've merged various fixes to the CMake build that I think are likely to resolve this.

I've been told that CI is passing now, so closing. (If this is not true, feel free to reopen.)

Unfortunately, I haven't been able to get a build of Legion using CMake after https://gitlab.com/StanfordLegion/legion/-/merge_requests/1427 was merged.
If the runtime is built properly, I get this error
[0 - 7fd0f5de7c40]    0.000000 {5}{gex}: Failed to load gex wrapper at librealm_gex_wrapper.so

when executing the code.
I do not have any time to debug this until mid-October. I'll get in touch when I will have further info

@mariodirenzo - if you can provides details to reproduce I can debug further. I don't see any cmake builds in the CI (the link I have been using for it  - https://lc.llnl.gov/gitlab/stanford-psaap/ci/-/pipelines)

One config that fails with cmake

Build failure with cmake + gasnet1 on lassen (different from what's reported above)

the same options work on sapling
the same options but without cmake work on lassen
the scripts and output from the builds (with/without cmake) are on sapling -> /scratch2/seemah/lassen_cmake_issue/
build.sh - build script for both, bad.txt is the build output with cmake, good.txt is without cmake
cmake version 3.23.1

@seemamirch I don't have access to the internal CI link you posted.  Can you provide the logs of the issue?  If you're seeing the following error, it is likely because you're building with the gasnetex wrapper enabled somehow, which requires either an environment variable set to be able to locate the wrapper library, or it needs to be available in a library search path (e.g. LD_LIBRARY_PATH).  If you do not want to use the gasnetex wrapper, then you need to disable it's use (by default I believe it is not used, so I believe you would be enabling it yourself or through another part of the build system, maybe legate or something?)
If you can provide the full log including the cmake command line used to configure Realm, and the build command and build output, I might be able to reproduce the issue and help resolve it for you.  Thanks!

@seemamirch I don't have access to the internal CI link you posted. Can you provide the logs of the issue? If you're seeing the following error, it is likely because you're building with the gasnetex wrapper enabled somehow, which requires either an environment variable set to be able to locate the wrapper library, or it needs to be available in a library search path (e.g. LD_LIBRARY_PATH).

[0 - 7fd0f5de7c40]    0.000000 {5}{gex}: Failed to load gex wrapper at librealm_gex_wrapper.so

If you do not want to use the gasnetex wrapper, then you need to disable it's use (by default I believe it is not used, so I believe you would be enabling it yourself or through another part of the build system, maybe legate or something?)
If you can provide the full log including the cmake command line used to configure Realm, and the build command and build output, I might be able to reproduce the issue and help resolve it for you. Thanks!

I've moved the files to https://sapling2.stanford.edu/~seemah/lassen_cmake_issue/) so you can view them.
my issue is without gasnetex
The CI link is not relevant for this issue
@mariodirenzo may benefit from your comments above (with gasnetex)

Okay, looking at "bad.tzt", I see the following line:

/usr/WS1/mirchandaney1/legion_rc/runtime/realm/realm_config.h:90:2: error: #error Shared memory not supported on GASNET1
#error Shared memory not supported on GASNET1

This has to do with the shared memory support added several releases ago, and the logic to enable this is here in the cmake build system:
https://github.com/StanfordLegion/legion/blob/4a03402467547b99530042cfe234ceec2cd31b2e/CMakeLists.txt#L822
Which, if gasnet1 is enabled, then this should be disabled.  So either your configure command is enabling this feature and overriding the default, or something is wrong with the logic here.

As an update, I've been able to get a successful CI of HTR++ binding the latest master from scratch and setting GEX_BUILD_SHARED and Legion_USE_GASNETEX_WRAPPER to OFF. From my point of view, we can close the issue

It's possible there was a config issue with my cmake build on lassen - I can't reproduce it. Since @mariodirenzo is also ok with the cmake build of HTR++ - closing this issue


Attempted Migration to Version 1.6

DedicatedServer(Update).zip
Hello. I've been using this mod to run a Stardew server for 10+ people and as the 1.6 update came around, the server has broken and the mod no longer worked.
Rather than downgrading back to 1.5.6 I decided to try updating the mod's code to work with 1.6's new framework.
I have updated the mod as best as I can and swapped out deprecated code with the new ones that came with the update. The Solution has been rebuilt successfully with the Net6.0 Framework and the updated NuGet package from PathosChild.
SMAPI will attempt to load the mod when run and fail as of now rather than being skipped due to incompatibility. The issue is currently on line 55 of the StartFarmStage.cs file and line 20 of the HostAutomatorStage.cs file.
With my limited understanding it seem the mod as it is currently written is incapable of creating and loading a save file and is also unable to execute the pause command while no other player is around.
I have removed Crop Saver, Demolish and Build commands as I couldn't figure out how to get them working.
Event ID's have been converted to string format, along with pet breed selection.
I have also changed the ready check codes as the original code no longer worked.
Any help on fixing the mod would be greatly appreciated! And I hope my contribution here has helped with the migration should you have been planning to do so as well.

Hello, the Mod should work with this version v1.1.0-beta:

This version implements additional functions, please check the description in branch Develop.
The branch develop and main from the fork cannot be merged, as the merge into this repository (#17) is still open.


fix(release): update semantic release version and put release setting…

Re-testing semantic release branch, using 16.x beta

:tada: This PR is included in version 1.14.0-prerelease.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Azure support for .NET 5 Function Apps (Pipeline and portal)

I got here from https://docs.microsoft.com/en-us/azure/azure-functions/dotnet-isolated-process-guide
I've looked around but I'm not really sure where this should go, sorry if this is the wrong place.
I have been fighting Azure DevOps (ADO) with a test .NET 5 isolated Function App for a while now, trying to get it to work through ADO. I know it's not officially supported, and I don't expect it to work; I am trying to hack something together to preserve our workflow, otherwise .NET 5 is basically dead for us. I've been trying to deploy files/zips directly, use dotnet/VS publish commands & profiles, blah blah and nothing works so far. (Deploying straight from VS works as described in the docs.)
Relatedly, I needed to create the Function App through VS, there was no way I found to do it through the portal (I suspect the API works and is what VS uses).
My question is: is this support intended to happen? Is it underway, planned or on a roadmap? Is there portal support for .NET 5 isolated FuncApps planned as well?
Any routing help or pointers to communication that confirms this support is planned would be appreciated.

It's certainly possible to deploy with DevOps, but I don't think any of the built-in tasks do this today. I'm assigning to @anthonychu to work on a sample/docs for this.
And yes, the portal work is planned -- but we have no ETA for this at the moment.

@josh-endries I tried this out and it works. I used a Windows consumption app.
The steps described here should work.
There is one added detail that you must set the FUNCTIONS_WORKER_RUNTIME app setting to dotnet-isolated. You can either manually do this in the portal, or set it in the AzureFunctionApp@1 task (see below).
Here's the pipeline I tested with:
trigger:
- azure-devops

pool:
  vmImage: ubuntu-latest

steps:
- task: DotNetCoreCLI@2
  inputs:
    command: publish
    arguments: '--configuration Release --output publish_output'
    projects: '*.csproj'
    publishWebProjects: false
    modifyOutputPath: false
    zipAfterPublish: false
- task: ArchiveFiles@2
  displayName: "Archive files"
  inputs:
    rootFolderOrFile: "$(System.DefaultWorkingDirectory)/publish_output"
    includeRootFolder: false
    archiveFile: "$(System.DefaultWorkingDirectory)/build$(Build.BuildId).zip"
- task: PublishBuildArtifacts@1
  inputs:
    PathtoPublish: '$(System.DefaultWorkingDirectory)/build$(Build.BuildId).zip'
    artifactName: 'drop'

- task: AzureFunctionApp@1
  inputs:
    azureSubscription: 'my-service-connection-name'
    appType: 'functionApp'
    appName: 'my-function-app-name'
    package: '$(System.DefaultWorkingDirectory)/**/*.zip'
    appSettings: '-FUNCTIONS_WORKER_RUNTIME dotnet-isolated'
    deploymentMethod: 'runFromPackage'

+ @ggailey777 for thoughts on how to incorporate this into docs.

Leaving this open to track the documentation improvements.

@ggailey777 , I'm going through some of these older issues to see what needs follow up. Is this something we already have documented?
Thanks!

I missed this in our recent revision of the Pipelines article. 🙁 Will there be similar issues for Python v2 and Node v4 apps?

Closing older issue. The doc state should be accurate and cover the supported deployment options for the isolated worker model.


Fix Scaffolder Prepare step when templates on disk

Hey, I just made a Pull Request!

Template entity location should be a directory name and resolve from there
And actually copy to the path that the templater expects /template rather than /checkout as there is no need for the /checkout folder.

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)

Tugboat has finished building the preview for this pull request!
Link:

https://pr4516-tlb7t2dncxfwviyjc1skrmnvykby4blr.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6026fb1d65f817cc53ef9d7a

Tugboat has finished building the preview for this pull request!
Link:

https://pr4516-tlb7t2dncxfwviyjc1skrmnvykby4blr.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6026fb1d65f817cc53ef9d7a


Disabling showLittleSticksPlug in settings.js doesn't work

I have set showLittleSticksPlug: false, in settings.js
Yet the Little Sticks plug in the footer remains active with the button Get this template still showing.
Am I missing something with removing this?
Thanks

Let me take a look @KitAdams, I recently changed some of the plug code so it could be an oversight from that.

@KitAdams I am not having any issues with on the latest code of the theme. If I set showLittleSticksPlug to false in the settings.js file it hides it from the footer for me?

@jaydanurwin - thanks for testing!
I am still seeing this:

Let me retry cloning this and see if it occurs still.

@KitAdams Did you still have issues after re-cloning?

@jaydanurwin - all working after recloning this on my linux machine.
It may have been a Windows issue as this was where I first was using this.


Add native simd vector support

This will take moving to MemorySegments and FloatBuffers.
Then c code using jextract

See PR for results. It's not gonna work.


Use Rust formatting

Given the increasing differences there are with the original repository (+ it seems it has been abandoned?), we may want to apply rustfmt to be consistent with the rest of our codebase, and insert back the rule in the CI jobs.

Any thoughts on this @hadjiszs @Freyskeyd? The upstream repo is clearly abandoned, we've made two major updates to the original FROST construction, bringing us quite away from it, and some portions of the codebase really hurt my eyes..

I'm OK to apply a Rustfmt if the upstream repository is abandoned. And yes, if we clean up the code style, we need to enforce these rules by using fmt Github actions.


[MM-41409] - Make status dropdown header clickable and open profile settings

Summary
Make status dropdown header clickable and open profile popover
Ticket Link
https://mattermost.atlassian.net/browse/MM-41409
Release Note
NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

@matthewbirtch updated

Thanks @nevyangelova
Should we also make the header selectable if user opens the menu using keyboard shortcuts?
Currently, tabbing through menu options is not highlighting the header.

@jgilliam17 its a good point but I am not sure if its a part of this PR. This will require some refactoring on the menu header so I would keep this PR for the cosmetic changes.

Thanks @nevyangelova - I'll create a follow up for keyboard shortcuts
E2E report looks good, no PR related failures
Can you please add a new e2e for this?
I created a test case in Zephyr MM-T4914
copying steps here, please feel free to edit as necessary I can update Zephyr later

Click on user avatar on global header
Verify menu opens
Hover over menu header
Verify mouse pointer changes to hand
Click on the header
Verify click on header opens Profile settings modal

One more thing, I know we have a ticket somewhere for this, in dark themes username is not using full opacity when name is not set

Follow up ticket https://mattermost.atlassian.net/browse/MM-45716

Looks good @nevyangelova. I'm going to approve - I'm thinking we probably want to add the same hover state that the rest of the menu items have with the background color. Other than that, we're good!

@nevyangelova a reminder to add the hover state as well

Thanks @nevyangelova
Tested manually - looks good
One issue - new test is failing on this line cy.get('#accountSettingsModal').should('exist');
See full E2E report here

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/220680

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/220699


Custom Nested form no longer working after upgrading from version 5.12.7 to version 6.0.4

Question
We have a custom type for that gets two pieces of data that is usable in our formly forms. This custom field type has been working well for us until we updated to the new ngx-formly version 6.0.4. The last version we had that it was working was version 5.12.7.
After updating to version 6.0.4 this field type renders and looks like it is working but it is not passing the inputed value to the model, like the other types.
Other dependency info of packages we are using:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>I have my issue here in this stackblitz. In the stackblitz I have my custom type and the configuration of it in the module. my form calls it with the name and it shows up in the form along with a basic text input.
When you fill out the form and sumbit it you can see that the model does not have the values for the phone numbers it is only seeing the values for the text input.
My stackblitz code example
I am not sure what I am doing wrong. Any direction would be super helpful

related to #3455, will make sure to provide fix it in the next minor release.

This issue has been fixed and released as part of v6.1.7 release.
Please let us know, in case you are still encountering a similar issue/problem.
Thank you!