sp_configure user_options does not mention contained databases

sp_configure will not work for user options when a database is contained on the server. The fact that contained databases removes the ability to set defaults doesn't seem to be mentioned anywhere else either. I suspect because contained dbs is abandonware.

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

ID: 8c6695aa-46cb-5317-8712-ce0820214a6a
Version Independent ID: f514c661-41e7-c2b4-1efd-66d3f9da898a
Content: Configure the user options Server Configuration Option - SQL Server
Content Source: docs/database-engine/configure-windows/configure-the-user-options-server-configuration-option.md
Product: sql
Technology: configuration
GitHub Login: @markingmyname
Microsoft Alias: maghan

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

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

@markingmyname -- please look into this issue.

#assign @WilliamDAssafMSFT

@WilliamDAssafMSFT
any thoughts?

@twiggy The nature of contained databases is to be portable between SQL Server instances and insulated from any instance-level settings. The behavior of sp_configure here is intentional as it configures instance-level options.  Some options, like MAXDOP, can be configured at the database level via the ALTER DATABASE SCOPED CONFIGURATION option. For more information on different features for contained databases, see Modified Features (Contained Database).


feat: use time_to_change_lane

Description
横移動量ベース(旧)からカットインまでの時間ベース(新)に変更

https://star4.slack.com/archives/C03S84LDJGG/p1684717842809799
Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Codecov Report
Patch coverage has no change and project coverage change: -13.56 :warning:

Comparison is base (d7973a1) 13.55% compared to head (a4bdf2f) 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@               Coverage Diff               @@
##           beta/v0.8.0    #510       +/-   ##
===============================================
- Coverage        13.55%   0.00%   -13.56%
===============================================
  Files              199       4      -195
  Lines            18792     873    -17919
  Branches         10299       0    -10299
===============================================
- Hits              2547       0     -2547
+ Misses           13307     873    -12434
+ Partials          2938       0     -2938

Flag
Coverage Δ

differential
0.00% <0.00%> (-13.56%)
:arrow_down:

Impacted Files
Coverage Δ

...map_based_prediction/map_based_prediction_node.hpp
0.00% <ø> (ø)

...based_prediction/src/map_based_prediction_node.cpp
0.00% <0.00%> (ø)

... and 201 files with indirect coverage changes

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


Add bibliogaphy

Add a bibliography section to the end of the document.
The bibliography should contain the list of articles listed in the Google Doc.
The bibliography lives above the copyright.

@LP3436 take care of this ticket. If there is a section in Google Docs, then add it here. If not, add the Readings in Canvas to this section.

@LP3436 if you have any question, then please respond.

@LP3436  If your Google Document does not have a bibliography section, then add the papers from the Reading section in Canvas.
For example the HuBMAP 2019 Nature paper can be found here.
If you visit the page at the top

click on Cite this Article
which will lead you to

Use that text as bibliography. To add it as a bullet point in streamlit_app.py you can type
text = ```
* HuBMAP Consortium. The human body at cellular resolution: the NIH Human Biomolecular Atlas Program. Nature 574, 187–192 (2019). https://doi.org/10.1038/s41586-019-1629-x

st.write(text)


CI/CD Setup Instructions may be missing some steps

I am going through the steps in file tfengine/generated/cicd/README.md.  I am not able to call terraform successfully through google cloud build trigger.
In cicd directory, I ran terraform apply which ran successfully.
It generated files:
tfengine/generated/cicd/configs
-rw-r--r-- 1 xxshambm xxshambm 1276 Jul 24 16:14 terragrunt.sh
-rw-r--r-- 1 xxshambm xxshambm 2411 Jul 24 16:14 tf-apply.yaml
-rw-r--r-- 1 xxshambm xxshambm 2604 Jul 24 16:14 tf-deletion-check.sh
-rw-r--r-- 1 xxshambm xxshambm 2295 Jul 24 16:14 tf-plan.yaml
-rw-r--r-- 1 xxshambm xxshambm 1651 Jul 24 16:14 tf-validate.yaml

I committed those configs into my github repo, but the triggers were looking for the config files in live/cicd/configs and not cicd/configs.    Did I miss something in the README or is a step needed to copy/move those files?
After moving the files to live/cicd/configs. I was then able to execute the triggers on pull request or manually.
They fail due following error:
Step #1 - "Init (CICD)": The directory has no Terraform configuration files. You may begin working
Step #1 - "Init (CICD)": with Terraform immediately by creating Terraform configuration files.
Finished Step #1 - "Init (CICD)"
Starting Step #2 - "Speculative plan (CICD)"
Step #2 - "Speculative plan (CICD)": Already have image (with digest): gcr.io/cloud-foundation-cicd/cft/developer-tools@sha256:02b06198f1da423183937b60493bdaa20dedf36b1a852a1d7fbb5a537fd943fd
Step #2 - "Speculative plan (CICD)":
Step #2 - "Speculative plan (CICD)": Error: No configuration files

My live directory does not have any terraform files.

There are two cicd/ directories, one is at the root of your output directory (i.e. tfengine/generated/cicd) and the other one is under live (tfengine/generated/live/cicd). The difference is tfengine/generated/cicd should be run once manually, and tfengine/generated/live/cicd is managed by CICD itself.
Can you let me know what's the directory structure of your generated configs after running tfengine? The Cloud Build Triggers work together with the Terraform configs and directory structure configured by tfengine and it assumes there is a /live folder and also a live/cicd folder inside. If you use tfengine to generated configs, the directory structure should be set up for you.

Thanks.  I had TERRAFORM_ROOT as "live" in the trigger which was wrong and now changed it to ".".   This fixed the issue of not being able to run.  I ran into another issue.

docs tell you to download and install terraform which is used to create the initial state file, but Google Cloud Build has a slightly older version of terraform so the run fails because you manually created the state with a new version.

Error loading state: state snapshot was created by Terraform v0.12.28, which is newer than current v0.12.26; upgrade to Terraform v0.12.28 or greater to work with this state

And then the READMe in cicd... Should it mention to copy cicd/confgs/* to live/cicd/configs so triggers can see those files?

Thanks. I had TERRAFORM_ROOT as "live" in the trigger which was wrong and now changed it to ".". This fixed the issue of not being able to run. I ran into another issue.

docs tell you to download and install terraform which is used to create the initial state file, but Google Cloud Build has a slightly older version of terraform so the run fails because you manually created the state with a newer version.

Error loading state: state snapshot was created by Terraform v0.12.28, which is newer than current v0.12.26; upgrade to Terraform v0.12.28 or greater to work with this state

Let me update the Docker image used in CICD so it's using the up-to-date Terraform.

And then the READMe in cicd... Should it mention to copy cicd/confgs/* to live/cicd/configs so triggers can see those files?
You don't need to copy/paste any cicd/ directories. They are two separate cicd/ directories. Cloud Build triggers are configured to look for Cloud Build configs at {TERRAFORM_ROOT}/cicd/configs/, not in {TERRAFORM_ROOT}/live/cicd/configs/

@xingao267 it seems like when there is a new Terraform binary release it will cause this version discrepancy. Can we update the devops instructions to use the same docker container when running from local machine?

@umairidris deploying bootstrap and cicd dirs locally from a container is not difficult but might make the process a little bit more complicated. docker will need to be installed. And to run it, users will need to mount their gcloud credentials, source configs and output local terraform state files to the container. Sent https://github.com/GoogleCloudPlatform/healthcare-data-protection-suite/pull/477 instead to pin the terraform version in the readme files.

@xingao267 makes sense, let's pin the versions so they are consistent throughout.
@careb2b FYI we're working on simplifying the cicd directory layout as part of #397.

@xingao267  Yes, it was due to be having TERRAFORM_ROOT set to live incorrectly.   After reapplying it was ok and I did not need the config files in live/cicd.
I noticed the triggers were updated for later terraform version and I was able to apply now.
I will close this.


Ibmc poc ptests

What type of PR is this?
/kind code-refactoring
What does this PR do / why we need it:
Refactoring to test execution of periodic tests  in cluster hosted in IBM Cloud
Which issue(s) this PR fixes:
Fixes #4827
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[ ] Documentation

[ ] Update changelog

[x] I have read the test guidelines

How to test changes / Special notes to the reviewer:
Periodic tests should be executed

@rnapoles-rh are you still working on this? Do we still need this to move ahead with move to IBM Cloud?

@rnapoles-rh are you still working on this? Do we still need this to move ahead with move to IBM Cloud?

No, this changes are not required here. Closing it


Consider name change for OptionT<T>

I'd like to consider changing OptionT<T> to something less "bad" - for the lack of a better term - before we go 1.0.  This is partly because it forces Result<T, E> to be ResultT<T, E> for consistency purposes, but also because I just don't like OptionT<T> at all.
In addition, if we were to move forward with #40 (which I'd really like to do in some form or fashion, but maybe not until after 1.0). I'd hate to be forced into naming it ValidationT<T, E> just to match OptionT<T> and ResultT<T, E>.
My only good suggestion at this point is Optional<T> (keeping Some<T> and None), but we could also switch to the other naming convention which is Maybe<T> with Just<T> and Nothing as the two variants.  I'm not as fond of the Maybe naming convention, but if you guys think that's better than crow-barring our way through this problem just to use a variant of the word "Option", then I could be convinced to change it.
Do you guys have any thoughts on this?
cc @brycehipp @ryanguill

I think changing it to Optional is fine.  I don't think we should switch to Maybe with the different api - there are subtle differences and I don't see any benefit in changing, especially at this point.
Why did we go with OptionT initially?

Why did we go with OptionT initially?

I wanted this library to be something that could be used in the browser as well as other contexts, but sadly Option is already a global in the browser.  This doesn't completely prohibit us from using Option, but since most of the time you'd want to import Option/Result directly (so you could write Option.some(1)) instead of just importing nullshield as a whole (and having to write nullshield.Option.some(1)) it'd be pretty annoying to be forced into the latter in the browser.
I couldn't think of a good way around this at the time, so I just stuck a T on the end of the name and went with it.  Thinking back now Optional is a much better option that I either simply didn't think of or rejected for some unknown reason.
However, if you can think of a better solution to this problem, I'm all ears.

So, summing up here, just to be clear about the result (heh) of making this change:

Before
After

OptionT<T>
Optional<T>

ResultT<T, E>
Result<T, E>

ValidationT<T, E>*
Validation<T, E>*

*(assuming we decide to build this one)
Seem good to you guys?
I'm slightly annoyed by the fact that Optional is now an adjective and they both used to be nouns.  I would have liked for it to all be symmetrical in that aspect, but I guess it's not a big deal.

LGTM.  Think of them as containers and all of them are adjectives.  Optional Container, Result Container, Validation Container.

In total agreement of the name change.
Thinking of them as containers definitely helps wrap my mind around them too.

Awesome! I'll get to work on this asap. Thanks for the feedback here guys!


fix(react-scripts): use react-scripts' own postcss dependencies

Fixes #12177
When using npm 8, in some cases there are no postcss-flexbugs-fixes in the node_modules directory, instead they appear in node_modules/react-scripts/node_modules:
ls -l node_modules/react-scripts/node_modules | grep postcss

Therefore, we should use react-scripts' own postcss plugin dependencies, as well as postcss implementation.

Should be fixed in https://github.com/facebook/create-react-app/pull/11751 already

This is not fixed in v5.0.1. I get error Loading PostCSS "postcss-normalize" plugin failed: Cannot find module 'postcss-normalize' if I try to use css module file from node_modules.


feat(x/gov): remove No With Veto, and set default pass threshold to 2/3 and quorum to 25%

Removes NWV Option
Sets default pass threshold to 2/3 (more precisely 0.667)
Sets default quorum to 25%

Changing the default params was pretty straightforward so I did it alongside the NWV removal

I did leave no with veto in v1beta1


Document usage of -t "" to create fixed field length output in bcp Utility

It would be helpful if the documentation for  "-t " option explicitly showed that field_term can be a double-quoted empty string ("") to instruct BCP to export to fixed-field length format (i.e. no field separators).  A google search reveals people have asked how to use BCP to export to fixed field length files and this simple answer of specifying (space)"" after the -t option does not come up as a solution very often. The examples in the ms documentation show using hex notation( 0x),  escaped character notation (\t), and a single character directly following the -t option (-t,) .   But, never is an example shown, such as "-t """.

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

ID: 86bd520a-7468-fd55-c357-b0f645a917ef
Version Independent ID: 3dc9dee5-afcb-8287-4441-e84ff717186a
Content: bcp Utility - SQL Server
Content Source: docs/tools/bcp-utility.md
Service: sql
Sub-service: tools-other
GitHub Login: @markingmyname
Microsoft Alias: maghan

@Rokartt4209 -- thank you for your detailed feedback.
@markingmyname -- please look into this issue.

@Rokartt4209 We recently updated this article with a refreshed look and feel, and new examples. Do you mind taking a look and letting us know if this issue you reported still needs inclusion?

@rwestMSFT thank you for addressing this.
@Rokartt4209,
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion, and we will reopen the issue. To see what the community is wanting, please visit the SQL Server User Feedback site or the Q & A site.

@markingmyname
Having read the updated documentation I still would not know how to make BCP write a fixed field length file.  It would be most helpful if there was a specific section on how to do this, such that the file HAS NO FIELD SEPARATORS and each record is terminated with cr/lf. There may be multiple ways of doing it.   I just asked chat.openai.com and I got an answer that says to write the output query to use CAST or CONVERT on each column to CHAR(##) where ## varies for each desired column length and to use the "-c" option on BCP.   This answer is probably wrong, assuming when we say "fixed field length" that we do NOT want any type of field separator character(s).  From the documentation it seems this would still place a tab character between each field.  This answer also differs from what I previously got from chatGPT (which I've noticed is increasingly incorrect on its answers) of using:
-t ""
What I ultimately did for creating fixed field length output files with cr/lf as record terminator is to define "export" tables in my database where all columns had a fixed CHAR(##) data type. I populated the "export" tables with queries to format all the data into the desired CHAR(##) columns and then used  BCP with an option like:
-n -C RAW -r\n -T -S


Update common.props

Actually strong name the assemblies

looks good

I did a search for "SignAssembly" in the corefx repo and noticed there were no matching results in the code. I found https://github.com/dotnet/corefx/pull/2194 which seems to imply true is the default in that repo. Anyone know why? Is there an equivalent to the KoreBuild repo that sets this to true?


Testfiles in lessmsi-v2.0.0

The 'TestFiles' folder (76.5MB) in https://github.com/activescott/lessmsi/releases/download/v2.0.0/lessmsi-v2.0.0.zip is not needed for release.

Thanks again for reporting @raddyst !


Xenial package uses different ports

Ubuntu Xenial packages uses ports 80/443, Trusty uses ports 8080/8443.

Should we document the new ports, revert to the old ports, and/or make them configurable via /etc/kontena-server.env e.g. PUBLISH_HTTP/HTTPS=80/443?

I think we should use the old ports, easier to document and does not confuse users too much (and they allow to install master + agent on the same machine). It might be also a good idea to make them configurable.


ipython 2.4.1

Ready for review.

:+1:

2.4.1 <= x < 2.5
Would be better, unless we think we will go to 3.0 on very short order
On Mar 1, 2015 8:33 PM, "Joshua Shorenstein"<EMAIL_ADDRESS>wrote:

[image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/pull/888#issuecomment-76654171.

My understanding is we're going to be on 3.0 as fast as we can, and this is a stopgap.

@squirrelo I already made @wasade suggestion and the test pass. I think is fine to validate that range and simply change to 3.0 once our code supports it.

Works for me.

Changes Unknown when pulling 8d6061b346352b445c3a06351bda93734e62bce9 on antgonza:ipython-2.4.1 into ** on biocore:master**.


WIP - Suggested changes to the utils code

Some suggested changes to improve the code flow and to support per-service log-level tweaks

I should not have mixed up so many areas into this PR. My bad.
Broken out into more reviewable chunks
Refactoring play_SOME_sound() to be DRY and adding a bus arg to support remote playing:
https://github.com/OpenVoiceOS/ovos-utils/pull/188
Adding a "logging" config section: https://github.com/OpenVoiceOS/ovos-utils/pull/189
Improving the sound executable scanning: https://github.com/OpenVoiceOS/ovos-utils/pull/187


A quick summary of major difference from urho3d since the split a few years ago?

Aside from obvious new third party module and the editor.

Hey, thanks for taking interest :) I wrote a page of things i managed to remember. Couple may be missing, but hopefully nothing of importance.
rbfx and Urho3D differences

Thanks!
Actually there's still some design decisions I wanted to learn more about:

Why was eastl introduced in place of urho's string/containers, and also instead of any other game oriented libraries out there? was it objectively better?
Context now holds a few cached pointer to a few other subsystems. And also now friend with Engine class. Was this necessary?

(I was trying to backport rbfx editor back in to vanilla urho3d but there were more breaking changes in urho engine than expected, and are now going back and forth between continuing or just straight up adopt rbfx. So this will help a lot.)

Why was eastl introduced in place of urho's string/containers, and also instead of any other game oriented libraries out there? was it objectively better?

EASTL is maintained (at EA's own pace anyway) and more tested than urho's custom containers. Besides it implements stl so previous knowledge of stdlib is applicable. It also does SBO in multiple places so we are looking into way less allocations by using easlt::string alone. I am not aware of another as complete stl implementation as eastl (well maybe stlport, but it is long unmaintained by now). With esatl we also get a predictable performance across platforms, something we cant count on when using stl implementations that come with compilers. There are some pretty cool optimizations geared towards games too. Its just perfect for our usecase.

Context now holds a few cached pointer to a few other subsystems. And also now friend with Engine class. Was this necessary?

Not really necessary.  It is true that hashmap lookup is cheap, but lookup we dont do is even cheaper still ;) As for friend - it is not necessary yes. It is a leftover from earlier developments. Ill get rid of it.

I was trying to backport rbfx editor back in to vanilla urho3d but there were more breaking changes in urho engine than expected

If you really want to do that you can use source code from a month or few back where it used old urho3d containers. That would make porting significantly easier, but not trivial still. You would also be looking into maintaining editor yourself and getting updates from rbfx would be increasingly difficult as time goes on due to lots of changes between two codebases.

99a64dd - friendship ended with engine
Oh okay


Use a better semver library to handle version parsing to deal with version that starts with a character v

Signed-off-by: Mike Ng<EMAIL_ADDRESS>

/lgtm

/lgtm


How is the sequence ID calculated in an efficient manner

Hello,
In your excellent paper, a key asspect used is the sequence identity between the artificial and any known natural sequences.
May I ask how this sequence identity is calculated in an effective manner? As it requires to screen all the databases for each sequences.
Many thanks in advance

These values are calculated using the MMseqs2 tool to find the closest matches between the generated sequences and the protein databases. We report the identity to the top database hit for each generated sequence.


Translate contribute/review/ into Japanese

Closes #23945.

/assign @inductor

Page preview: https://deploy-preview-23946--kubernetes-io-master-staging.netlify.app/ja/docs/contribute/review/

/approve

/lgtm


import error @schema-utils

when i build tsconfig.json with ”tsc -b tsconfig”
node_modules/webpack/types.d.ts:91:10 - error TS2305: Module '"schema-utils/declarations"' has no exported member 'validate'.

91 import { validate as validateFunction } from "schema-utils";
            ~~~~~~~~

Found 1 error.

i fixed at node_modules\webpack\types.d.ts from
import { validate as validateFunction } from "schema-utils"
to
import { default as validateFunction } from "schema-utils/declarations/validate";

Sorry, not related to webpack


feat: support unnest in FROM clause

Which issue does this PR close?

Closes #9324.
Rationale for this change
Support queries like select * from unnest([1,2,3]).
The unnest table function is recognized by sqlparser-rs as a table factor.
To support it, first convert it into an Unnest logical expression, and then create the corresponding Unnest logical plan.
What changes are included in this PR?
Support unnest in FROM clause
Are these changes tested?
Yes
Are there any user-facing changes?
No

CI failed due to issue #9358.

Thank you @jonahgao  and @jayzhan211


go get -u github.com/rakyll/hey

go get -u github.com/rakyll/hey
package golang.org/x/net/http2: directory "/Users/user/gowork/src/golang.org/x/net/http2" is not using a known version control system
package net/http/httptrace: unrecognized import path "net/http/httptrace" (import path does not begin with hostname)

hey only works with Golang 1.7 and above.


advent

In GA accent, it doesn't use "ago", it uses "edge". I also noticed that in the CUBE dictionary (modern british?), it uses "edge" as well. http://cube.elte.hu/index.pl?s=advent&t=&syllcount=&maxout=&wfreq=0-9&grammar=

Sure, good suggestion. I'll include it next time I update the ReadLex.


Icarus 'assign' statements don't work

Probably related to #96 .
http://www.edaplayground.com/x/4nt
http://www.edaplayground.com/x/96k
Given a very simple and (apparently) identical testbench written in verilog and in python, the output is different if there is an 'assign' style asynchronous assignment. Run the code and look at the EPWave window to see the output.
Can someone please run these under a non-icarus simulator to see if the problem is in icarus or in cocotb?

I need to check in more detail, there is a bug in Icarus which is fixed in newer versions which meant that values assigned outside of a process weren't updated.

I can confirm this works on Modelsim and Riviera so it is the Icarus bug.
You could try either compiling the latest version of Icarus, or switch to the free version of Modelsim.

Not a bug in Cocotb, closing.


How I disable the HMR temporarily?

What problem does this feature solve?
The problem I have is that I don't know how to disable Hot Module Replacement ( aka HMR) in vue-cli service. I tried to follow this thread but I didn't gone too far: https://stackoverflow.com/questions/52766802/how-do-i-disable-hot-module-replacement-with-vue-cli-service-serve.
PS.: The purpose of this is that I want to continue the development of my application and the designer that is in another computer, seeing my machine in my ip will style the application. When I save the application, the HMR cleans the css of the inspector and the designer lost the CSS he did in it.
Hope for a better comprehension. Suggestions are welcome.
What does the proposed API look like?
I guess other people might suffer from same issue.

I don't think such workflow makes sense and I'm strongly against this approach.
You and the designer may better not share the same dev server. Build a snapshot version and serve the dist folder for them.

Nonetheless, hot reload can be disabled with the following config:
module.exports = {
  devServer: {
    hot: false,
    liveReload: false
  }
}

(The console message still presents but there won't be actual module replacements happening)

@sodatea, thanks for the help!

invalid configuration...always has the ws://localhost:11117/sockjs-node/009/umr1d0xa/websocket

So,
module.exports = {
  devServer: {
    hot: false,
    liveReload: false
  }
}

disables hot reload for devServer mode, but does not disable HMR of webpack - still have those /info requests in the browser.
Another solution:
 chainWebpack: config => {
    config.module
      .rule('vue')
      .use('vue-loader')
        .loader('vue-loader')
        .tap(options => {
          options.hotReload = false
          return options
        })
  }

does not work in my case because my project uses 'raw-loader' instead of vue-loader, and it does not have hotReload  option.
Another solution:
chainWebpack: config => {
    config.plugins.delete('hmr');
},

did not work.
One more solution:
vue-cli-service serve --mode production
did not work, still HMR is activated.
One more solution:
process.env.NODE_ENV === 'production'

this finally works, yay - no more /info requests... but there's a problem - what if we need NODE_ENV like staging, testing and want to disable HMR for those? How to solve this correctly?

Guys, someone can tell me what configuration files you talking about? I tried to put those config snippets to my webpack config, but nothing happens. But since nobody told me which configuration file I should edit, I honestly can't test these on my instance.

I found that only this worked for me to completely disable the live-reload
module.exports = {
  devServer: {
+   webSocketServer: false
}

(my issue was that running over HTTPS in a container would break the app when it tried to access insecure ws:// sockets)

I found that only this worked for me to completely disable the live-reload
module.exports = {
  devServer: {
+   webSocketServer: false
  }
}

(my issue was that running over HTTPS in a container would break the app when it tried to access insecure ws:// sockets)

After searching far and wide, this is was the solve for me. Changes are still detected upon saving which recompiles our app but the annoying "HMR" messages no longer spam the console.
Thanks @tpluscode!


Improve is_camera robustness, primarily thread safety fixes

Summary of changes:

Removed ros::spinOnce() call at the end of PublishLoop(). There is a comment to explain why. I consider this change experimental and could probably be convinced to revert it... perhaps there's something I'm not understanding about the ROS nodelet threading model.
Switched from using ROS timers (which run callbacks in a thread from the nodelet manager's thread pool) to PeriodicTrigger instances (which are used to invoke functions in the same thread as PublishLoop()). This avoids potential thread safety issues. V4L2 is supposed to be thread-safe but the community wisdom seems to be that it is vendor-specific and you shouldn't count on it. Beyond strict thread safety, a more subtle point is that synchronizing events to happen at a particular point in the frame loop is likely to make the behavior more reliable by avoiding rare corner cases. (For example, what happens if the syscall to adjust the exposure is executed while the camera is in the middle of grabbing a frame? We can eliminate cases like these by syncing the syscall to only happen at a particular point in the loop.)
Changed newer ioctl() calls to use xioctl() wrapper, consistent with the rest of the driver.
Added guards that skip the exposure parameter adjustment syscalls if the actual value was not modified. It's great to avoid syscalls when possible because they typically require a context switch, which adds some latency. (There was also a comment that suggested making too many exposure adjustment calls reduces the frame rate, so it may have more impact than just a context switch.)
Fixed a bug in modulo arithmetic. The C++ modulo operator preserves the sign of its first operand. You might think that (i - 1) % k always evaluates to a number between 0 and k - 1, suitable for indexing into a length-k ring buffer, but in fact it evaluates to -1 when i is 0 and causes an invalid memory reference. The formula should be (i + k - 1) % k.

This PR is marked as draft because it should definitely be tested on the granite table before being merged into develop. The testing should include applying manual exposure adjustments like we have done in past ISAAC procedures. I don't think there is a huge rush on this. There may be other changes we want to make and test together.

I tested this PR. The nav_cam crashes consistently (and then the entire nodelet group follows) when auto_exposure is set to true in the cameras.config file since startup (not if set to false and then true after starting). (consistently= crashed 3/4 trials)
As a sanity check to make sure this is a problem with this PR I tested develop in the same circumstances and in the 3 trials, the fsw didn't crash once.
So there's something fishy happening here at startup...

I tested this PR. The nav_cam crashes consistently (and then the entire nodelet group follows) when auto_exposure is set to true in the cameras.config file since startup (not if set to false and then true after starting). (consistently= crashed 3/4 trials) As a sanity check to make sure this is a problem with this PR I tested develop in the same circumstances and in the 3 trials, the fsw didn't crash once. So there's something fishy happening here at startup...

Thanks for checking, Marina! Were you able to get any further information, such as a backtrace from a core dump, or any kind of error message preceding the crash? (If not, I could still try to comb through the logic again to see if there's an obvious mistake.)

This PR is broken, closing and capturing the intent in #722


bugfix(TDI-38698) : Correct return variable name

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-38698
What is the new behavior?
Fixed return variable name.
Please check if the PR fulfills these requirements

[X] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[X] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[X] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:


Results are not correctly handled when ligand binds

results folder is empty when a simulation converges to a bound structure. Attached is the python traceback that is dumped to the standard output/error file:
ligbinder.python.log

reporting bugs have been fixed.
also, biasing power has been redefined to take into account the total number of leaves for each branch.
please install again in portal and give it a try.
if everything works fine I'll merge the fix to master


[FLINK-14952][network] Solve the issue of exceeding memory limits when using mmap blocking partition

What is the purpose of the change
As reported by users in mail list, the container would be killed by yarn because of exceeding physical memory limits. It is mainly because the memory usage of mmap in BoundedBlockingSubpartition is not capped and not accounted by configured memory limits, but yarn would track this memory usage and kill the container once exceeding some threshold.
To solve this issue, we adjust the default option of taskmanager.network.bounded-blocking-subpartition-type from auto to file for safety usage. And users can still enable mmap way via configuration if they make a proper adjustment in yarn configuration to avoid container killing.
Brief change log

Generates the respective html files for previous changed configurations
Adjusts the default option of blocking subpartition type from auto to file

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run travis

Thanks for the review @pnowojski . I updated the codes for addressing the comments.


GraphiQL.Menu not working

When using GraphiQL.Menu everything of the menu that is outside the toolbar, like the dropdown will be cut off.

This regressed with #579. If i remove the overflow-x it works again.
/cc @asiandrummer

The menu should be fixed now.

@pleunv: yep, as of v0.11.8.
Thanks for the report, @danez.


Remove bluebird dependency

We use it for one thing. We should rip it out of that one place to avoid having to include it.

Believe this can be closed now.

indeed.


fix: LEAP-846: Fix date filtering in virtual tabs

previously we used packJSON to serialize filters for virtual tabs; this has some problems, in that it seems to get rid of hyphens, and has bugs that result in wrong datetimes if multiple datetimes are selected in the virtual tab. Instead of using packJSON to accomplish this, switch to gzip (by way of pako) and base64 serialization (by way of js-base64) to compactly serialize filters for URLs.
An example of an old packJSON url with a couple filters:
/projects/278389/data?tab=Y2zxDxw4fbr3g041r8bW101GVJPVlHhj4BpPOsyOpRsPOe7andGNPlHXiuWsco.UZL0JwRBzwDVm0wtasks.03gRMYJUYVy0NvDq8vPqQoRvQe9lessGNp9VnGWAOG0Gdm0VmHBe0du0T00VW1q01-05tm004Q2T.Cw48coRqRsOK0012EMiz6jcDJ00

and the same with gzip, though with 6 filters:
projects/38/data?tab=H4sIAAAAAAAAA62SQWvCQBCF_8uco4zG9rDnXgrFk6cWKWN2ItNmNyE7ESXkv3dii9aDlYKwh919b7_3GLYHFa0YHDxxSV2lkEEplXKbwPVQ1PGji4VKHc1C0ZssysHEt_7HaML3ximlz-S6xpOyf6eRVTfcktajqeKU7GZHVTfmzXG-mGA-meUrXDhEW1NEfDWLHppjI8OoBIYhu1NWjnfI2hwus2xGShJ_5eUn7Isk_QNpQJpKoC3fQgIlX57rPh_fXAeLvwRKPKN6CHZ0swwC7a3scKIuu7Ax1nVs0fJ_xo2PK3y4Me61xdet51bi1j7VevgCXznVcZICAAA

for backwards compatibility, we first attempt to deserialize with gzip, and if that fails, fall back to packJSON.
This fixes numerous issues with virtual tabs, and most saliently, allows date filtering to work within them.
The URLs above can be used for testing. Verify that deserialization is successful for both the virtual tabs given, but that the packJSON one results in a urlJSON.js:15 falling back to packJSON. message on the console and an error like the below, originating from a wrongly formatted date filter.

/git merge develop


[Bower] install certain version not working

Doing this
bower i jquery#1.11.2 --save

returns zsh: no matches found: jquery#1.11.2. But when I switch to bash, it works as expected.

It interprets the # as a globbing character, you have to wrap it between single commas (') or put noglob before bower. You can also define an alias to use noglob by default: alias bower='noglob bower'

Ah I see, it does solve my problem. Thank you @mcornella.


Best practice for generating less?

Hi Robert,
Think this is particularly interesting for people who want to mask php files with the less content header in order to pass WP options or similar data through a less file (which isn't possible with the standard less integration), i.e.
@primarycolor: <?php echo get_option('primarycolor'); ?>;

Have you done any testing in that direction or had any experiences there yet?
Cheers!

Duh, just saw the readme had updated since the last time I saw it, my bad!

Alright Noel,
I think I pushed an update a little while ago, there's a filter that you can use to pass extra variables in. Check the readme nearer the bottom. I might be mistaken and I've got some code to push.
Anyway short answer it's doable!
Robert O'Rourke
sanchothefat.com
+44 7816 329424
On 22 Jan 2012, at 07:53, Noel<EMAIL_ADDRESS>wrote:

Hi Robert,
Think this is particularly interesting for people who want to mask php files with the less content header in order to pass WP options or similar data through a less file (which isn't possible with the standard less integration), i.e.
@primarycolor: <?php echo get_option('primarycolor'); ?>;

Have you done any testing in that direction or had any experiences there yet?
Cheers!

Reply to this email directly or view it on GitHub:
https://github.com/sanchothefat/wp-less/issues/1


chore(main): release butler-sheet-icons 3.6.1

:robot: I have created a release beep boop
3.6.1 (2024-06-02)
Bug Fixes

Add support for QSEoW 2024-May (85acb6f), closes #555

Miscellaneous

deps: Update dependencies (5e3a393)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ptarmiganlabs/butler-sheet-icons/releases/tag/untagged-f51ce012c315434f0de9 :sunflower:


Sign out message have to move one line down

Sign out message has to come down one line down to appear fully without overlapping.

Duplicate of #30


Create Google Project Management: Professional Certificate

you don't have to create 3 different file to add it , please add it in readme file as well as in allcourses.json file


Error: The tuning parameter grid should have columns sigma, C

Hi,
I downloaded scPred by install_github and constructed SingleCellExperiment object using my own data. But at trainModel step, it turned like:
"Error: The tuning parameter grid should have columns sigma, C"
Any idea about this error? The only difference between my script and tutorial is that SingleCellExperiment object.

Hi @CongqianyouYz, thanks for your feedback! This error should be solved with commit c716bbe (current state of the repo). Let me know if you still experience some issues.
José


Page added Announcements

In announce.html there is some sample data to show what it will look like but there is also a for loop provided to iterate through all the dynamic data.

backend support needed

@harryvineeth I get an error saying announce.html does not exist on this branch. Please add the file and push.
Also why has represent.html been added to this PR?

@harryvineeth Commit the announce.html file. I'll create the views for this and we will finish this off.


Make goto-definition go to a signature declaration if possible

Fixes #9269 and #10074

For the record, calls and news are not the only things that undergo overload resolution. Tagged templates and decorators do as well.
If it's not too hard, it'd be ideal to keep the behavior consistent between them.

Can you also add a test for going to definition on a call where overload resolution failed?
declare function doStuff(n: number): number;
declare function doStuff(s: string): string;

/**/doStuff(true);

Ideally, the language service should still jump to one of the overloads.

Other than the comments, :+1:


The class 'FirebaseAuth' doesn't have a constructor named 'instance'  (after creating GoogleSignIn())

I get this error if try to create new FirebaseAuth instance after creating GoogleSignIn()
But it works fine if created before GoogleSignIn()
Screenshot

pubspec dependencies
cupertino_icons: ^0.1.0
shared_preferences: ^0.4.1
charts_flutter: ^0.2.0
image_picker: 0.4.1
google_sign_in: 3.0.2
firebase_analytics: 0.3.3
firebase_auth: 0.5.5
firebase_database: 0.4.6
firebase_storage: 0.3.0
Run your application with flutter run and attach all the log output.
flutter analyze errors
error - The class 'FirebaseAuth' doesn't have a constructor named 'instance' at lib\main.dart:34:42 - new_with_undefined_constructor
error - Expected to find '(' at lib\main.dart:34:50 - expected_token
Flutter Doctor
[√] Flutter (Channel master, v0.3.6-pre.50, on Microsoft Windows [Version 10.0.16299.402], locale en-PK)
• Flutter version 0.3.6-pre.50 at C:\fluttersdk\flutter
• Framework revision c813eabf50 (11 hours ago), 2018-04-27 22:55:51 -0700
• Engine revision 613aae032f
• Dart version 2.0.0-dev.50.0.flutter-cd0c4e4fe1
[√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
• Android SDK at C:\Users\Muhammad Zeeshan\AppData\Local\Android\sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-27, build-tools 27.0.3
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)
• All Android licenses accepted.
[√] Android Studio (version 3.1)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 23.2.2
• Dart plugin version 173.4700
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)
[√] VS Code, 64-bit edition (version 1.22.2)
• VS Code at C:\Program Files\Microsoft VS Code
• Dart Code extension version 2.12.0
[√] Connected devices (1 available)
• TRT L21A • VWD7N17706001493 • android-arm • Android 7.0 (API 24)

For more information about diagnosing and reporting Flutter bugs, please see https://flutter.io/bug-reports/.

You're using the static ìnstance field of the FirebaseAuth class, so you don't need the new keyword as you're not creating a new instance. With final _firebaseAuth = FirebaseAuth.instance;, the code should work.

I assume @simolus3's comment fixes this issue, right?


Bug: handlers on touch/mobile don't work as expected

I've experienced this with other handlers, but most recently with handleEnd. I have a handleEnd function that does nothing but console.log, yet it breaks expected behavior on touch devices. Likely due to an event getting canceled when the callback is fired.
Repro: https://codesandbox.io/p/sandbox/fk-drag-touch-repro-forked-fy4pz9?file=%2Fpackage.json%3A12%2C38
Video:
https://github.com/formkit/drag-and-drop/assets/13283469/17074301-0a62-49d1-a150-90ab849f60d1

@wesharper Ah yes, when you define handleEnd, you are actually overriding the core event handler itself. The idea is to import the original handleEnd and reinvoke it before or after whatever logic you are trying to add on: sandbox. We plan in the future to implement events you can listen to instead of needing to override the handler itself.

Ah, makes sense. Thanks for the feedback. Perhaps there could be an onEnd or onHookName pattern that doesn't override default behavior but acts more as a way to fire off your own logic?

@wesharper Ah yes, when you define handleEnd, you are actually overriding the core event handler itself. The idea is to import the original handleEnd and reinvoke it before or after whatever logic you are trying to add on: sandbox. We plan in the future to implement events you can listen to instead of needing to override the handler itself. Probably would be worth adding an example of this to the docs as other people have been confused by this as well.

Thanks for mentioning this @sashamilenkovic, I ran into a similar issue with performTransfer as specifying it caused the default transfer functionality to break, which was unexpected. Event handlers / listeners would be a much better solution or this needs to be more widely documented as myself and others are wasting a lot of time figuring this out

As of v0.2.0 There are now both global and parent scoped event listeners: https://drag-and-drop.formkit.com/#events


any inference code or something to check the model

After some investigation, I replicated inference code using the same goal with supplied one/multiple snapshots and history actions.
It is not working very well on zero-shot situations.

@truebit Can you publish your inference code? I would appreciate it!

@truebit Please share the inference code if possible.

I have been working on writing the inference code, here is what I could achieve till now. I wrote a function to produce the processed input for an image and the goal. However, now I am not sure on how to use that as an input on a pretrained model.
This is the code that I wrote to process the image file and the goal:
`
import action_type, action_matching
import tensorflow as tf
import numpy as np
from tqdm import tqdm
import json
import jax.numpy as jnp
import argparse
import pickle
import torch
import tensorflow as tf
from PIL import Image
from transformers import AutoProcessor, Blip2Model

device = "cuda" if torch.cuda.is_available() else "cpu"
model = Blip2Model.from_pretrained("Salesforce/blip2-opt-2.7b", torch_dtype=torch.float16)
model.to(device)
processor = AutoProcessor.from_pretrained("Salesforce/blip2-opt-2.7b")

def _decode_image(
    example,
    image_height,
    image_width,
    image_channels,
):
    """Decodes image from example and reshapes.

    Args:
        example: Example which contains encoded image.
        image_height: The height of the raw image.
        image_width: The width of the raw image.
        image_channels: The number of channels in the raw image.

    Returns:
        Decoded and reshaped image tensor.
    """
    image = tf.io.decode_raw(
        example.features.feature['image/encoded'].bytes_list.value[0],
        out_type=tf.uint8,
    )

    height = tf.cast(image_height, tf.int32)
    width = tf.cast(image_width, tf.int32)
    n_channels = tf.cast(image_channels, tf.int32)

    return tf.reshape(image, (height, width, n_channels))

def parse_image(
    image_file_path
):
#     parsed_episode = []

    goal = "How to login?"
    step_id = "123"
        # episode_id = ex.features.feature['episode_id'].bytes_list.value[0].decode('utf-8')
    output_ep = {
        "goal": goal,
        "step_id": step_id
    }

    img = Image.open('sample.png')

    image_height = img.height
    image_height
    image_width = img.width
#     image_channels = img.getChannel()
    with torch.no_grad():
        inputs = processor(images=img, return_tensors="pt").to(device, torch.float16)
        image_features = model.get_image_features(**inputs).pooler_output[0]
        image_features = image_features.detach().cpu()
    output_ep["image"] = image_features
    output = []
    output.append(output_ep)
    parsed_episode = []
    parsed_episode.append({"episode_id":123, "data":output})
    return parsed_episode

def parse_args():
    parser = argparse.ArgumentParser()
    parser.add_argument('--dataset', type=str, default='general')
    parser.add_argument("--split_file", type=str, default="dataset/general_texts_splits.json")
    parser.add_argument('--output_dir', type=str, default='dataset')
    parser.add_argument('--get_images', default=True, action='store_true')
    parser.add_argument('--get_annotations', default=True, action='store_true')
    parser.add_argument('--get_actions', default=True, action='store_true')
    parser.add_argument('--file_path', type=str, default='sample.png')

    args = parser.parse_args()
    return args

if __name__ == '__main__':

    args = parse_args()
    print('====Input Arguments====')
    print(json.dumps(vars(args), indent=2, sort_keys=False))

    all_parsed_episode = parse_image(args.file_path)

    with open(f"{args.output_dir}_test_val.obj", "wb") as wp:
        pickle.dump(all_parsed_episode,wp)

`

Hi friends,
We’ve got AutoUI running and tested its end-to-end performance in our recent paper. You can find the inference code here
https://github.com/Berkeley-NLP/Agent-Eval-Refine/tree/main/exps/android_exp/models/Auto-UI

Hi friends,
We’ve got AutoUI running and tested its end-to-end performance in our recent paper. You can find the inference code here
https://github.com/Berkeley-NLP/Agent-Eval-Refine/tree/main/exps/android_exp/models/Auto-UI

Great job thanks will try that 👍 any insights in how good it works for zero shot approaches?


Geocoder does not work

On Work, enter a case. Duplicates are detected, but the geocoder does not return any values:

Seems to have been a new API requirement


Add simple example

The idea is to use this example from the new website homepage. We'll provide a link to open this on the Playground from the website homepage.

It is actually not included in the release. This was an unintentional release caused by the release automation initial run. 1.0 version of the spec was released a long time ago. The release mentioned by the bot was removed.

Not included in release 2.1 yet, 2.1 was mistakenly triggered by our automation.


PWM on broken on Arduino

It has no PwmPin.
    self._add_pins(
        [DigitalPin(self, location)
                for location in range(detected_digital_pins)] +
        [AnalogPin(self, 'A%s' % location, resolution=10)
                for location in range(detected_analog_pins)]
)

PR #82 closes this Issue.


New Functions on PAGE 6510

Make Function ZeroLineExists External and Global
Make Function NextExtryNo External and Global
Make Function TestTempSpecificationExists External and Global
Make Function CalculateSums External and Global
Create New function GetEdtableVariables

Create New Function GetVisibleVariables

Create New Function GetVariables

Create New Function SetVariables

PAG6510_011519.txt

Please note that this is a typical scenario to be solved through a Reclassification Journal.
In the Warehouse (P7365->P6550) it can be easily solved but a couple of issues preventing it in "basic" Item Tracking (P393->P6510):

If the bug in TestTempSpecificationExists is solved (that is: check in Reclass is performed against "New Lot No." instead of "Lot No."), then you can create several lines in Page6510 with Identical "Lot No." and different "New Lot No." and the user can see them all at once.
If not, the lot has to be split in the journal, and the Item Tracking defined separately for each line. I already requested a fix for this in #1346

If variable FormRunMode is made accessible, then you can build your scenario on top of "FormRunMode::Reclass" (which allows everything to be seen and edited). I've also requested that we get a new function to expose this variable (GetFormRunMode) in #1347

The action @MatthewMcCarthy7  is talking about is going to be enabled on Item Tracking Line Page only if the user opens it from Sales or Transfer Order, so that they can create one or more new lots from the single existing on without leaving the page.
Currently we are building Reclass journal with the item tracking in the background and posting it.
This is really user-friendly as it would save them time and clicks to going through reclass journal and assigning newly generated lots back to sales or transfer order.
I am sure #1346  and #1347 would improve item tracking functionality for reclass journal but we need this functionality for Sales and Transfer order.

But these are 2 seperate transactions: first you must split the lot and post your changes, so that the new lots exist in stock before you can link them to an order.
You say that you've already solved generating ReclassJnl in the background, so it seems that the problem to solve is to allow the user to "review the new Lots created and modify what was created and then save the values".
To me it sounds like a question of timing rather than modifying the standard:
If you let the user revise the split lots BEFORE you post them (that is: with P6510 in Reclass mode), then it fits more easily within the existing design:
You (background) generate the Reclass, open P6510, let him/her revise/confirm, you close P6510 and (background) post the Reclass and leave the user with P6510 where the order is the Soure Rec an the new (revised) lots may be linked to the order (where no second editing is necessary)...
That approach doesn't increase the number of clicks needed for the task.

That is exactly we are doing. We are doing 2 separate transactions, and yes our current functionality is saving a number of clicks by posting reclass in the background when they close the page.
When User clicks this action button on Item Tracking Lines, they can select From Lot (existing) and To Lot (New Lot), how many new lots they want and quantity per lot.
After that when the user closes the P6510 we generate reclass in the background and assign new lots to P6510, so yes it is fitting more easily with the existing design.
We need requested events/functions to covert this mods to extinction. We need to access those temp variable so that we can insert new/changed lots into those temp variables and give it back to BC to execute its standard workflow. Plus these events/functions will also be useful in the future as it would increase access to temp variables of P6510.
We have it as a button because the user would not always split the lot from sales and transfer orders.
I hope now you would understand, why we are asking these events/function to Microsoft.
@JesperSchulz, please let us know if you have any update on this request.
Thank you.

I hope I haven't offended you with my input - by no means it was my intention to step on your toes and obviously Microsoft is who decides.
I was just trying to point out that you can figure out visibility & editability without adding more code to to the already "messy" code of standard Item Tracking, since these properties depend on well defined conditions and are controlled through calls to SetControls.
When extending Item Tracking, whether it be adding new attributes to LN/SN or adding new scenarios/flows, they basically must obey the same criteria as is used for SetControls.
Therefore I personally don't see any added value in fx GetVisibleVariables and GetEditableVariables.
As to GetVariables and SetVariables I also believe there are other alternatives to hold on to the newly created lot numbers (e.g. single instance codeunit).
Anyhow, I'll step out of the way.

Thank you for your inputs, We need fx GetVisibleVariables and GetEditableVariables because the controls we are adding depends on the different conditions.
We also need GetVariables and SetVariables to hold temporary variables and would also be useful for others.

Hello,
I was checking in on the status of this request.

Hello,
Is there any update on this request? This request is holding us up from being able to submit our app to Appsource.

This is done.  Closing Ticket


pmb2_description.pmb2_launch_utils no longer exists

Dear PAL team,
This code seems to be no longer valid:
https://github.com/pal-robotics/tiago_robot/blob/8bf78ce642a5b62bab7ccfc5b4883d72693b4ba3/tiago_description/tiago_description/tiago_launch_utils.py#L16
Any idea to fix it?
Thanks!!

Hello @fmrico,
Thank you for writing to us. Regarding the tiago_launch_utils.py the method still exists in the following file:
https://github.com/pal-robotics/pmb2_robot/blob/58cba9be1a5ca8b3a043d93924da252ab6f8991f/pmb2_description/pmb2_description/pmb2_launch_utils.py#L18
We hope this helps to resolve your issue,
Best Regards,
PAL Robotics Team.

Hi @saikishor
I have just solved it by using https://github.com/pal-robotics/pmb2_robot/tree/07e0fcd82cc0d21e0a03cb020bd6d69d5be5574d
Thanks anyway!!
Francisco


[Bug?] Opening vrchat through VRCX makes hands unusable

Im on Index with Index controllers
If I open a new instance of a world / Join a friend through VRCX and launch in VR, my controller bindings are broken.
I cant use fingertracking and some buttons dont work properly.

This isn not a VRCX issue, this happens when you launch any VR game outside of SteamVR since it doesn't load the actual controller binding profile but a fallback profile.

Running vrchat through Steam will solve it I think, but I don't know how to run it through Steam. 😅

steams game shortcuts use the steams game app ID to launch them so maybe steam://rungameid/438100// --no-vr as an example starting vrchat with no vr

yep just tested that by pasting it into my web browser but it does ask for confirmation

question is can pypy change it so launching vrchat uses the steam:// method, but also able to pass parameters to vrchat like what world to join, with what instance privacy, etc.

seems its possible using a vrchat launch link vrchat://launch?Id=wrld_3259761f-5dee-4b86-80e1-00f9178da209&instanceId=24330~private(usr_f4495a87-599d-41e4-96c9-b88e419befcc)~nonce(7d5918b1-5ff5-4ceb-ea17-8fb293cbff77)~canRequestInvite
but doesnt work well as that requires its own intergration protocol from vrchat which is added when you first install vrchat steam doesnt understand this protocol so it wont recognize it as a parameter

the protocol im speaking off is the vrchat first time installer that asks for install when you first install the game install.exe

it seems possible
[My Computer\HKEY_CLASSES_ROOT\steam\Shell\Open\Command]
"C:\Program Files (x86)\Steam\steam.exe" -- "%1"

there you go

also check this one
https://developer.valvesoftware.com/wiki/Steam_browser_protocol


Make testing with the speaker more convenient

I was just testing a PR and wanted to play something through a speaker. The speaker doesn't wait for playback to finish. The tutorial states that you have to do the following:
done := make(chan bool)
speaker.Play(beep.Seq(streamer, beep.Callback(func() {
	done <- true
})))

<-done

It would be nice to have a helper method to do this:
speaker.PlayAndWait(streamer)
speaker.PlaySync(streamer) //or

This method must work with multiple streamers, just like speaker.Play accepts multiple streamers.
Notes for myself:

In what other scenario's is beep.Callback used?
Make the tutorial explain beep.Callback's uses in other ways.

I marked this as a good first issue because the scope is limited to a very small part of Beep and not a lot of knowledge of the package is required. However, it does deal with concurrency which makes it a little more difficult to implement.


[addon-info] Need another way to pull description from docgen info

Bug or support request summary
I created components with docgen description comments like:
/** This is the description */
class Button extends React.Component {
  // ...
}

And then I create a story named "Secondary Button", when I click "Show Info" I expect to see "This is the description" from my component somewhere on the page, but I do not.
If I rename the story to just "Button" then I do see "This is the description".
The problem seems to be in the _getComponentDescription() function, which is comparing the displayName of the component to context.story, which appears to be the story name.
It seems like having a secondary condition here might be useful. One approach might be adding another option to the parameters for info like:
{
  info: {
    descriptionClass: Button
  }
}

Another option would be to allow passing the class where we can currently pass a string description (ie. text), ie:
{
  info: {
    text: Button
  }
}

As a bonus, with multiple components, this would allow us to tell which component to get the description from.
Thoughts?
Steps to reproduce
Create a story for a component that has a docgen description comment, where the story name is not the same as the component's displayName property and you will see this issue.
Please specify which version of Storybook and optionally any affected addons that you're running
❯ yarn list --pattern @storybook
yarn list v1.10.1
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>│  ├─<EMAIL_ADDRESS>│  ├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>

Marking this as a todo for #4341

This looks similar to https://github.com/storybooks/storybook/issues/4643


Dynamic gRPC call credentials

Hi, I'm trying to make a gRPC call using the google_grpc service so I can add credentials to the requests. Specifically I'm trying to use the access_token call credential option, but it only accepts a string. However I'd like to set the access_token value dynamically by using the response body from an HTTP call in a Lua script. Is this possible? I would think that the access_token should not have to be hardcoded into the envoy config

cc @htuch  @JimmyCYJ who may be able to provide a better answer.
I think the closest I can find is gRPC Security Token Service (STS) to get call credentials.  https://github.com/envoyproxy/envoy/blob/master/api/envoy/config/core/v3/grpc_service.proto#L132

Hey, thanks for responding. Yeah I saw the sts_service option, but it looks like that is for token exchange only, while the token I am trying to get is through the Client Credentials grant. So at this moment I guess this isn't possible?

The other possibility is using a metadata credentials plugin, https://github.com/envoyproxy/envoy/blob/master/api/envoy/config/core/v3/grpc_service.proto#L200, to perform the call and then populating metadata (which is basically HTTP/2 headers) with the values where the access token is expected. Lua is not supported today fo metadata credentials plugins.


Problems with copynumber analysis

Hi,I wanted to express my appreciation for your incredible tool.
While utilizing the T1K-copynumber function, I encountered an issue. Specifically, when running the command python3 t1k-copynumber.py -g XXX_genotype.tsv --nomissing KIR3DL3,KIR2DL4,KIR3DP1,KIR3DL2, I observed that in the output file generated by copynumber, several framework genes, such as KIR2DL4, have a log-ratio of 0.00, resulting in a copynumber of 0.
However, upon examining the genotype.tsv file for these genes, I noticed that the abundance of KIR2DL4 is relatively high and the quality score is 60. I am curious about how to interpret this situation and whether there is a solution to address it.
The sample genotype.tsv file and copynumber output file are shown in the figure.
I appreciate your time and assistance in resolving this matter.
huang

The script finds there are two copies for 2DL4 "2DL4*005 2 0.00", the number after the allele (abundance in the original genotype.tsv file), is the number of copies. Though based on your screenshot of the abundance, I think the log-likelihood difference with the next likely copy number should be larger, at least shouldn't be 0.00. Could you please paste the genotype.tsv as a text and I will look into this issue. Thank you.

Thank you for your reply.
This is the genotype.tsv file mentioned above.
t1k_kir_genotype_tsv.txt

Thank you for sharing the file. The "quality_score" column in the copy number output is the difference of the likelihood, and in this case both likelihood are very small, so the difference becomes less than 0.01. I have changed the value to the difference of log-likelihood (which I think I used this value at some point), so the difference is more obvious now.

Thank you very much for your assistance. With the adjustments made to the copynumber function, I have successfully resolved the issue mentioned earlier.


automate generation of implementation scorecard page

add functionality to the list of classes to implement to automatically mark what is already implemented

done


More binary package fixes

Zappa now checks package version when deciding if it has the correct lambda-package, or which wheels version to pull down. Before you would silently get the latest version no matter what version you had in your virtual env. Fixes https://github.com/Miserlou/Zappa/issues/800 (again) and probably https://github.com/Miserlou/Zappa/issues/841

Changed how the progress bars work when downloading manylinux packages. You now get one per package. There is also some logging to let you know whether a lambda-packages or wheels version of the package is used, along with which version. Here an example:

zappa package test
Calling package for environment test..
Downloading and installing dependencies..
Downloading cffi-1.10.0-cp27-cp27mu-manylinux1_x86_64.whl
100%|████████████████████████████████████████████████████████████████████████████████████████████████| 393K/393K [00:00<00:00, 2.34MB/s]
Using lambda-packages binary for psycopg2 2.6.1
Downloading grpcio-1.3.0-cp27-cp27mu-manylinux1_x86_64.whl
100%|██████████████████████████████████████████████████████████████████████████████████████████████| 5.31M/5.31M [00:01<00:00, 4.49MB/s]
Downloading protobuf-3.3.0-cp27-cp27mu-manylinux1_x86_64.whl
100%|██████████████████████████████████████████████████████████████████████████████████████████████| 5.72M/5.72M [00:01<00:00, 4.08MB/s]
Packaging project as zip..

Refactored the binary package code into one loop.

Tested on both Python 2.7 and Python 3.6

Coverage decreased (-14.2%) to 57.743% when pulling 375dc8f5ca67be4f82e5dfdb67d0e9ff9b4ebfb6 on nikbora:master into 78bb301b43755a8627f5080386f966909db4eeec on Miserlou:master.

Coverage decreased (-14.2%) to 57.743% when pulling 375dc8f5ca67be4f82e5dfdb67d0e9ff9b4ebfb6 on nikbora:master into 78bb301b43755a8627f5080386f966909db4eeec on Miserlou:master.

Coverage decreased (-14.2%) to 57.743% when pulling 375dc8f5ca67be4f82e5dfdb67d0e9ff9b4ebfb6 on nikbora:master into 78bb301b43755a8627f5080386f966909db4eeec on Miserlou:master.

Coverage decreased (-14.2%) to 57.743% when pulling 375dc8f5ca67be4f82e5dfdb67d0e9ff9b4ebfb6 on nikbora:master into 78bb301b43755a8627f5080386f966909db4eeec on Miserlou:master.

Coverage decreased (-14.2%) to 57.743% when pulling 375dc8f5ca67be4f82e5dfdb67d0e9ff9b4ebfb6 on nikbora:master into 78bb301b43755a8627f5080386f966909db4eeec on Miserlou:master.

Coverage decreased (-14.2%) to 57.743% when pulling 375dc8f5ca67be4f82e5dfdb67d0e9ff9b4ebfb6 on nikbora:master into 78bb301b43755a8627f5080386f966909db4eeec on Miserlou:master.

Coverage decreased (-0.2%) to 71.729% when pulling b3baf5edb9c5e9035f0733c6e1116c968099b70d on nikbora:master into 78bb301b43755a8627f5080386f966909db4eeec on Miserlou:master.

What do you think about making the progress bars per package only show if the user passes '-v' ? I have a lot of packages and this can get really long.

Not a bad shout. I shall see how easy it would be to pass a verbose logging
flag around. @miserlou any thoughts on this?
On Thu, 11 May 2017, 18:47 Bryan Marty<EMAIL_ADDRESS>wrote:

What do you think about making the progress bars per package only show if
the user passes '-v' ? I have a lot of packages and this can get really
long.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Miserlou/Zappa/pull/843#issuecomment-300848956, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACQOB-xrjCe7GK1jHhLsM07hPhkdtK4Yks5r4zuYgaJpZM4NXmsZ
.

You can use --quiet to silence output.

Coverage increased (+0.2%) to 72.145% when pulling c1550cf864dda55aeab23e53a6647875af1d644c on nikbora:master into 78bb301b43755a8627f5080386f966909db4eeec on Miserlou:master.

@Miserlou 👍
I also discovered some unit tests were not quite doing what they were meant to so also changed those and added a couple more.


CircleAvatar radius property sets the diameter.

I created a CircleAvatar with a radius of 32.0 inside of a Container with height of 64.0, and the CircleAvatar only took up half of the Container's height.  Setting radius to 64.0 made the CircleAvatar match the container's height.

Oops.


Flesh out contributor guide with content that we no longer support

We no longer maintain content on our blog that is older than a year. It would be good to create a page in our docs contributor guide about content that is out of scope to support. I'll start off that PR with some feedback from Jennifer Rondeau.

@jaredbhatti  was any PR open for this issue ?

@DanyC97 We do have a guide for this, however, as you can see, it is there as a guidance for people who triage and categorize issues. One improvement I can think of is to copy this notice to the contributor's guidance, e.g.

https://kubernetes.io/docs/contribute/start/#file-actionable-issues , and
https://kubernetes.io/docs/contribute/start/#improve-existing-content

/remove-lifecycle rotten

/priority backlog
Is that OK?


Made ColsVisibility container inherit font-size, expand linked filters unit tests

make ColsVisibility container inherit font-size
expand linked filters unit tests

Codecov Report

Merging #504 into master will increase coverage by 0.25%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #504      +/-   ##
==========================================
+ Coverage   87.73%   87.98%   +0.25%
==========================================
  Files          38       38
  Lines        4395     4395
  Branches      329      329
==========================================
+ Hits         3856     3867      +11
+ Misses        519      508      -11
  Partials       20       20

Impacted Files
Coverage Δ

src/tablefilter.js
92.95% <0%> (+0.29%)
:arrow_up:

src/modules/checkList.js
89.06% <0%> (+3.23%)
:arrow_up:

Continue to review full report at Codecov.

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

👍


Importing Default using "import" statement not working

Having a hard time understanding the documentation regarding import declarations.  The documentation on the website [https://webpack.js.org/api/module-methods/](module methods) makes mention to use imports, referring to MDN's spec, which matches the es6 [http://2ality.com/2014/09/es6-modules-final.html](es6 spec on imports).
In my export file I have...
export default class_builder(.....)
Which I am then importing (as per MDN and es6 spec)...
import MyClass from 'classfile'
Webpack is then resulting is declaring the MyClass as a Module and not as the default item, which seems contradictory to the MDN and ES6 specs that it refers too. (since one should use import * as MyModule from 'classfile' for Module importing.
I understand that the export default method is not favored, but have I missed something? was there a change that hasn't been clearly reflected on the website docs?
Using...

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I encourage you to ask your question on StackOverflow instead. We try to use the issue tracker for bug reports and feature requests, as well as some discussions, but this appears to be a very localized question.
I’m closing for these reasons.

Figured it out today, seems that it was Chrome Inspector was interpreting my import statement as a Module.  After using the good old console.dir(MyClass) I got the expected object, so it was Chrome that was misleading me.   It seems that the object was missing the method that was throwing the initial error.
So happy to close this.


Hall subscription woes

breathes out ...Okay.
Recent downtime (or rather, un-federation) of urbit-meta ~was~ is being caused by federation-related subscriptions being dropped. ~wanzod and ~binzod weren't listening to their parent anymore, ~samzod's parent wasn't listening to it anymore, and ~marzod had gotten completely disconnected.
Despite the fact that the developer docs never mention "for every %peer you make, be ready to handle a ++quit" (which, honestly, should be drilled as soon as you learn to do subscriptions), hall implements a generic ++quit to handle all subscriptions that don't need any special logic for re-subscribing.
++  quit  ::  dropped subscription
  |=  wir/wire
  ^-  (quip move _+>)
  :_  +>
  ?.  =(src.bol our.bol)  ~
  [(wire-to-peer wir) ~]

Note how it's ignoring the quit if it originated from a foreign ship. They may want to end our subscription for any number of reasons (we got banned, our subscription range has ended) that are totally legit. We only re-sub on locally caused quits, because those happen when gall says "you're holding up the queue, get out".
Or so I thought. Federation subscriptions don't get quit by hall manually, so they should only be caused by local gall. And yet the federation subscriptions just end, without re-subscribing.
(As an aside, if it were to actually re-sub, it'd still be terrible because subbing for federation currently always returns all data, which is slow and redundant and may or may not push down to subscribers depending on circumstances.)
Have added printfs to hall so that when ++quit fires, it prints out the origin and the affected wire. Curious to see if this tells us anything interesting. Expectation: foreign src.bol... Still won't tell us why though.

I think @cgyarvin said you should always retry on a quit, and only handle
"banned" type errors on resubscription? Though I can see checking the range
in the wire and not retrying if we actually have all the messages, as an
optimization
(Or rather, it's a corner case of "reopen the range subscription minus the
first however messages we've gotten")
On Wednesday, 13 December 2017, Fang<EMAIL_ADDRESS>wrote:

breathes out ...Okay.
Recent downtime (or rather, un-federation) of urbit-meta was is being
caused by federation-related subscriptions being dropped. ~wanzod and
~binzod weren't listening to their parent anymore, ~samzod's parent wasn't
listening to it anymore, and ~marzod had gotten completely disconnected.
Despite the fact that the developer docs never mention "for every %peer
you make, be ready to handle a ++quit" (which, honestly, should be
drilled as soon as you learn to do subscriptions), hall implements a
generic ++quit to handle all subscriptions that don't need any special
logic for re-subscribing.
++  quit  ::  dropped subscription
|=  wir/wire
^-  (quip move +>)
:  +>
?.  =(src.bol our.bol)  ~
[(wire-to-peer wir) ~]
Note how it's ignoring the quit if it originated from a foreign ship. They
may want to end our subscription for any number of reasons (we got banned,
our subscription range has ended) that are totally legit. We only re-sub on
locally caused quits, because those happen when gall says "you're holding
up the queue, get out".
Or so I thought. Federation subscriptions don't get quit by hall manually,
so they should only be caused by local gall. And yet the federation
subscriptions just end, without re-subscribing.
Currently investigating, any insights would be appreciated.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/498, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABxXhgl8QLwQ4Ke9fem2jGJw2INLhJ_Cks5s_9vMgaJpZM4RAm4e
.

Right, because if you get kicked/errored out as a direct result of the %peer, rather than an event happening to it down the line, it doesn't go back into ++quit, but just hits the ++reap instead.
That makes sense.

Yeah, ++reap is the handler for actual errors, ++quit just means "the
remote queue filled up or the local queue filled up or in general your
subscription as a piece of transient nonlocal state has met its end" and is
not a value judgement of any sort
On Wednesday, 13 December 2017, Fang<EMAIL_ADDRESS>wrote:

Right, because if you get kicked/errored out as a direct result of the
%peer, rather than an event happening to it down the line, it doesn't go
back into ++quit, but just hits the ++reap instead.
That makes sense.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/498#issuecomment-351461226, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABxXhptW7xx9csmvxc5qRFHHEYB61wolks5tAAfCgaJpZM4RAm4e
.

I'd say that's it a bit confusing then that userspace applications can also %quit incoming subscriptions. If you implemented your ++quit right, sure, it's going to end up at ++reap eventually. But taking your definition of ++quit as truth, it might be more semantically coherent if I could %pull an incoming subscription whenever, say, we ban someone from a channel.

By all means %pull their subscription if this's is useful for server
bookkeeping, just don't use this as the sole information channel.
On Wednesday, 13 December 2017, Fang<EMAIL_ADDRESS>wrote:

I'd say that's it a bit confusing then that userspace applications can
also %quit incoming subscriptions. If you implemented your ++quit right,
sure, it's going to end up at ++reap eventually. But taking your definition
of ++quit as truth, it might be more semantically coherent if I could %pull
an incoming subscription whenever, say, we ban someone from a channel.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/498#issuecomment-351464925, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABxXhjfjdmf5sv0iFgHM8dXXlsGjqiVkks5tAArRgaJpZM4RAm4e
.

Er, by all means %quit incoming subscriptions*
Client-side %pull should not result in a ++quit hopefully
On Wednesday, 13 December 2017, Anton Dyudin<EMAIL_ADDRESS>wrote:

By all means %pull their subscription if this's is useful for server
bookkeeping, just don't use this as the sole information channel.
On Wednesday, 13 December 2017, Fang<EMAIL_ADDRESS>wrote:

I'd say that's it a bit confusing then that userspace applications can
also %quit incoming subscriptions. If you implemented your ++quit right,
sure, it's going to end up at ++reap eventually. But taking your definition
of ++quit as truth, it might be more semantically coherent if I could %pull
an incoming subscription whenever, say, we ban someone from a channel.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/498#issuecomment-351464925, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABxXhjfjdmf5sv0iFgHM8dXXlsGjqiVkks5tAArRgaJpZM4RAm4e
.


add todo app ASA sample to awesome list

Please fill out this template! There are three different types of contributions, feel free to delete the checklists that are not applicable to your contribution type.
If you are submitting a new azd template to the gallery
Fill this out if you want your template to be added to the awesome-azd gallery!

[x] Added an entry to https://github.com/Azure/awesome-azd/blob/main/website/src/data/users.tsx that includes:

[x] Template title - A short title that reflects the local application stack that someone could use to get their application on Azure (e.g. "Containerized React Web App with Java API and MongoDB")
[x] Description - 1-2 sentence description of the architecture (e.g. Azure services) or solution that is defined by the template.
[x] Architecture Diagram or Application Screenshot - Used as display image for gallery card. The image should include all services and their connections (example). You should add the image to the website/src/data/images/.
[x] Link to Author's GitHub or other relevant website - Used for attribution
[x] Author's Name -  Name to credit on the gallery card
[x] Link to template source - Link to the template GitHub repo
[x] Tags - One or more tags representing the template. Provide at least 1 tag for programming language used and at least 1 tag for Azure services integrated. Also tag the IaC provider (Bicep or Terraform). If you don't see a relevant tag for your template? Feel free to add one!

[x] In the PR comment, if you can also add a link to the PR where you made your repo azd compatible this will allow us to provide feedback on your template and speed up the review process.
https://github.com/Azure-Samples/ASA-Samples-Web-Application/pull/14

Hi @gkulin , @hemarina , @rajeshkamal5050, @savannahostrowski and @jongio , would you help to review this PR? Thanks!

@v-xuto Could you run tests with this template?

Sure. We will start testing this template today.

@hemarina When testing this template Azure-Samples/ASA-Samples-Web-Application, run azd up, get error as follow, the error same to https://github.com/Azure/azure-dev/issues/2478#issuecomment-1633610145:

.

Hi @v-jiaodi , thanks for helping test the sample, and for the postgres error, it's an kown issue for postgres and you could bypass it by choosing region eastus instead of eastus2.

@yiliuTo This issue no longer exists using region eastus.

@v-jiaodi cool! and are there any other issues found out?

@v-jiaodi cool! and are there any other issues found out?

There are no other issues.

@jongio and @hemarina could you help to review this pr? thanks

@hemarina About the issue ParameterOutOfRange: The value of the 'Version' should be in: [] when creating resource DBforPostgreSQL, whether we consider some improvements, the following two options can be referred to：

Add a relevant tip in the Readme: When using postfreSQL within the playground subscription (or microsoft tenant's subscriptions), the location uses other location (like: eastus) instead of eastus2.
Improve the error message here, like Subcription *** is not allowed to provision in "East US 2".

Looks ok


Limitation in Reading View?

Thanks for your effort! I really like your plugin.
Is there a way to increase the number of "customized" codeblocks?
Im using the Multicolumn plugin as well (https://github.com/ckRobinson/multi-column-markdown)
Editing mode (fine):

Reading mode (only 4 codeblocks are customized)

Attachments: The note.

Thank you.
Scratchbook.md

Solved it by reloading the plugin.

There is no limit how many code blocks will render. I don't know this plugin, but this is how it looks on my machine:

Try changing to a different theme, or disabling custom CSS (if applied) to see if it changes.
Also note, that there is also a problem with problem with Multicolumn plugin. It renders the document endlessly. I disabled my plugin and I saw this:

Even if I did not scroll up and down in the document the number before:
[Violation] Forced reflow while executing JavaScript took <N>ms
and
[Violation] 'setInterval' handler took <N>ms
kept increasing. Which is not good, since it causes extra load for Obsidian.

Solved it by reloading the plugin.

You were faster than me :D

Thanks for the very detailed reply tho.
Especially the endless rendering is something i was not aware of.
Really appreciate it.

Anytime :)


gravity well is broken

Description

singulo like wells that have radial acceleration pull things into a line 1 tile below it
gravity anom like walls that have tangential acceleration pulls things into a tile 1 tile below and to the right of it

banana peel is radial well, the glasses get pulled into a line

grav anom pulls all the items into the corner

supermatter grenades are also ruined by this

ah, here's the duplicate issue from a month ago
https://github.com/space-wizards/space-station-14/issues/28608
they've seriously been broken for over a month....

https://github.com/space-wizards/space-station-14/pull/27443 unintentionally caused by this I guess

_physics.ApplyLinearImpulse(entity, (displacement * baseMatrixDeltaV) * scaling, body: physics);

to

_physics.ApplyLinearImpulse(entity, Vector2.Transform(displacement, baseMatrixDeltaV) * scaling, body: physics);


Sema: Remove TR_NonEnumInheritanceClauseOuterLayer [4.0]

Description: This fixes a source compatibility issue uncovered by the ProcedureKit project in the source compatibility test suite. We were incorrectly rejecting protocols that inherit from typealiases containing protocol compositions. This was not tested, but it worked in 3.1 and people now rely on this behavior.

Origination: The regression was introduced with the associated type where clause, where some defensive checks against malformed inheritance clauses were introduced. It appears that these defensive checks are no longer necessary.

Risk: Low, at worst we will crash on an invalid inheritance clause where we used to diagnose, but it appears that none of our compiler crasher tests regressed from this.

Tested: New test added to cover the case used in ProcedureKit.

Bug: https://bugs.swift.org/browse/SR-4855

Reviewed by: Huon Wilson

@huonw You originally added this flag (to fix some crashers from regressing, IIRC), but I'm assuming you won't be particularly upset if it goes away. Do you mind reviewing this for 4.0?

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - fcab2f87863827c76e612b6e92e3b23df6191a7c
Test requested by - @slavapestov

I'm assuming you won't be particularly upset if it goes away

I'm particularly non-upset that it's no longer necessary to stop crashers crashing. 👍

@swift-ci Please test

@swift-ci Please test


test: implement test for macro substitution

closes #297

Because of the branch name, the CI has all fails. Reviewers please test the tests locally


Remove warning for concurrency limit override

@ixti thanks for the review. What about the failing tests, should we merge this anyway? They look completely unrelated

I'll take a look asap


[HOLD for payment 2024-08-14] [$250] Track expense - App returns to confirmation page when tapping back button on category page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.71-0
Reproducible in staging?: Y
Reproducible in production?: Y
Found when validating PR : https://github.com/Expensify/App/pull/41465
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Precondition:

Account has no workspace.

Launch New Expensify app.
Go to FAB > Track expense.
Track a manual expense.
In self DM, tap Categorize it.
Select a category.
On confirmation page, tap on the back button.
On category list page, tap on the back button again.

Expected Result:
App will return to self DM.
Actual Result:
App returns to confirmation page.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/f686b437-8347-4bce-a700-2be2741f4642

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01824a5f82b4736dcc
Upwork Job ID:<PHONE_NUMBER>607385088
Last Price Increase: 2024-05-28
Automatic offers:

tienifr | Contributor | 102600461

Issue OwnerCurrent Issue Owner: @anmurali

We think this issue might be related to the #vip-split.

I can't reproduce this.
https://github.com/Expensify/App/assets/14225673/f2796596-2f2c-4b51-bc09-b7cac59b95e8

Thank you for the clarification @gijoe0295 - I am able to reproduce it.

Thanks @gijoe0295 -I am able to reproduce it now.

Thanks for the proposal @gijoe0295 and @tienifr
I cannot reproduce the issue mentioned in the OP as well as the issue in Share it with my accountant. The only issue I can reproduce is 1. Categorize it > 2. Select workspace> 3. Select a Category > 4. Press on Category in Confirmation Page > Instead of opening a new Category Page, you will be send back to the Category Page from Step 3. I am not sure if it's an issue or intended behavior.
Can any of you reproduce any other issues? If yes, Can you please share reproductions steps and on which platform it can be reproduced?

Thank you @gijoe0295 and @tienifr
I can reproduce the issue now.

I think you might have missed the pre-condition in the OP.

Yup. I missed the pre-condition.

Thanks for your proposal @gijoe0295

FYI We cannot set the participantsAutoAssigned in getTrackExpenseInformation as https://github.com/Expensify/App/issues/41783#issuecomment-2098932814 because participantsAutoAssigned is a client-side only properly, once the user logged out and logged back in, it will not be available and the issue will happen again.

I agree with @tienifr that 2nd part of your proposal might not work once user  log out and log back in.

Thanks for the proposal @tienifr
Can you please add more details to the first alternative solution in your proposal.  I want to see how it works.

Thanks for the update @tienifr
Alternative solution 1 from @tienifr's propsoal looks good to me.
🎀 👀 🎀 C+ reviewed

@srikarparsi - do you agree with https://github.com/Expensify/App/issues/41783#issuecomment-2141466185

Sorry for missing this, was OOO last week. @tienifr's alternative looks good to me as well :)

@sobitneupane can you complete the checklist whenever you have a chance please

@tienifr the offer extended expired. So I resent a new one - https://www.upwork.com/nx/wm/offer/103712326

Regression Test Proposal:

Log in with an account with no workspace
Create a track expense
Press Categorize it
Press Back
Verify app dismisses the RHP
Press Share it with my accountant
Press Back
Verify app dismisses the RHP

Do we agree 👍 or 👎

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ]  [@sobitneupane] The PR that introduced the bug has been identified. Link to the PR:

This issue was probably present with the addition of the feature to categorize the tracked expense.

[ ]  [@sobitneupane] Determine if we should create a regression test for this bug.

Yes.

[ ]  [@sobitneupane] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

https://github.com/Expensify/App/issues/41783#issuecomment-2324103115

@anmurali please post a payment summary for @tienifr when you get a chance.

Payment Summary

[ ] $250 to @tienifr approved, please request on ND

@anmurali I will also require the payment summary to request money in newdot.

$250 approved for @tienifr

@anmurali Bump for the payment summary.

Bump @anmurali


Fix various platform samples

Part of WildernessLabs/Meadow.Core#296 to fix issues after various folders were moved and renamed.

I've got the Meadow.Linux.Samples and Meadow.Windows.Samples solutions building locally. They aren't part of the CI build, apparently, but at least they are back to building again with the latest API and dependency changes.

@ctacke Mind giving the latest changes another review? I ended up reverting the M.Linux.Samples build, but I plan to put that on my backlog.


fix: typo in Overview.md

Minor change typo (Elixir instead of Exilir) in Overview.md
Signed-off-by: BSS ZU<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Revise RuleAtomData#compareTo

Eliminates the implicit assumption for two compared RuleAtomDatas to have the same variables set and instead works off the occurring variables of the rule.

Proposed change made obsolete by better solution.


[Patch develop] Change CI's develop link

[Patch develop] Change CI's develop link
As paddle's develop package occurs compile error, change CI's version to a valid link (0526's package)

The nightly build package contains CINN, which results in CI's error, close this PR


Camera permissions added

Does this have sense? I need it to get WebRTC (getUserMedia) working with XWalk third party engine.
https://github.com/crosswalk-project/cordova-plugin-crosswalk-webview

how to invoke permission since Android M permissions system for WebRTC getUserMedia ?

WebRTC usage is not related to camera plugin. You should perhaps use a hook to add the permission to your manifest.

Cordova CI Build has one or more failures.
Commit     - Link
Dashboard - Link

Builder Name
Console Output
Test Report
Device Logs

Windows 8.1 Store
Link
Link
Link

Windows 10  Store
Link
Link
Link

Windows 8.1 Phone
Link
Link
Link

iOS
Link
Link
Link

Android
Link
Link
Link

I agree with @nikhilkh
This should be added using a hook or your own plugin that just adds those permissions
Can you close the PR?


JPA: Allow expressing COUNT DISTINCT for multiple fields

Currently, to express COUNT(DISTINCT field) for a single field, I can write uniqueResult(q.field.countDistinct()), but there is no syntax for multiple fields like COUNT(DISTINCT field1, field2, ...).

Hello,
sorry for ressurrecting this issue, but is there any workaround? I don't wanna loose the JPAQuery good things because of this.. =/
I have some querys that uses Sum() and group by. When i use the .fetchCount() method it generates a query with count(distinct colum1, column2, column3 ...). I also tried to do a subquery but JPQL doesn't support ...