[LIVY-757] Add support for Active Directory through LDAP

What changes were proposed in this pull request?
In this PR I implemented and ability to configure custom LDAP user DN pattern as and alternative to base-dn option, for cases when username attribute isn't uid (which is now hardcoded in Livy).
So now instead of specifying baseDN you also can specify user DN pattern by setting the following property in livy.conf:
livy.server.auth.ldap.user-dn-pattern = sAMAccountName=%s,ou=People,dc=example,dc=com

I decided to provide an ability to specify pattern rather than just add an option to configure user attribute, because it seems to be more common way to do this:

https://github.com/cloudera/hue/blob/18cd309/desktop/conf.dist/hue.ini#L485
https://github.com/apache/zeppelin/blob/5dd5a65/conf/shiro.ini.template#L46
https://cwiki.apache.org/confluence/display/Hive/User+and+Group+Filter+Support+with+LDAP+Atn+Provider+in+HiveServer2#UserandGroupFilterSupportwithLDAPAtnProviderinHiveServer2-hive.server2.authentication.ldap.userDNPattern

On the other hand, as Livy already has an option to set baseDN, it would be more logical just to add an option to set custom user attribute.
So I opened another PR with an alternative fix for this issue: https://github.com/apache/incubator-livy/pull/330
How was this patch tested?
Manual, since unit tests seems to be broken.

I've realized that the current LdapAuthenticationHandler in Livy is exact copy of LdapAuthenticationHandler from Hadoop which also does not support different ways to use LDAP.
So my fix would help only with my case but it's not enough to properly implement different LDAP configurations so it's not worth merging.


test: (STONEINTG-398) Add Happy Path coverage for namespace-backed envs

Description
This addition to the mvp-demo test suite's Chapter 2 deals with executing an IntegrationTestScenario on a Snapshot inside an ephemeral environment. The test makes sure that the snapshot passes the testing and gets marked as "passed".
Issue ticket number and link
STONEINTG-398
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/test all

/test all

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test all

/test all

/retest all

/test all

/test redhat-appstudio-e2e

/test all

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test all

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test all

/test all

/test all

/test all

/test all

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test all

/retest-required

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

The latest CI run failed on integration.go test due to timing out on waiting for the DTCLS resource to not exist in the cluster. DTCLS exists in the cluster because of rhtap-demo test that was running in parallel.
There are 2 solutions (that I can think of) for this:

Increase the timeout from 5 mins to 10 mins?
Delete the DTCLS early on within the rhtap-demo.go test (currently it gets deleted within the AfterAll block)

/test redhat-appstudio-e2e

/test all

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e


[SPARK-13702][CORE][SQL][MLLIB] Use diamond operator for generic instance creation in Java code.

What changes were proposed in this pull request?
In order to make docs/examples (and other related code) more simple/readable/user-friendly, this PR replaces existing codes like the followings by using diamond operator.
-    final ArrayList<Product2<Object, Object>> dataToWrite =
-      new ArrayList<Product2<Object, Object>>();
+    final ArrayList<Product2<Object, Object>> dataToWrite = new ArrayList<>();

Java 7 or higher supports diamond operator which replaces the type arguments required to invoke the constructor of a generic class with an empty set of type parameters (<>). Currently, Spark Java code use mixed usage of this.
How was this patch tested?
Manual.
Pass the existing tests.

Can one of the admins verify this patch?

Although I personally like this change, I'm less certain about it since it does not add any functional improvement or avoid a potential bug. It does add a little readability and only touches the relatively fewer Java files. I'd like to wait for others to give an opinion.

Thank you for review, @srowen .
I agree. This has a little limited benefit; only small portion Java and no functional improvement.
Actually, I started this PR to improve docs/examples (28 of 61 files) for simplicity and eventually hoped to apply all code to keep it consistently.

The conflict will be resolved via #11567 . After #11567 is committed, I will update this.

+1 on the code changes, which simplifies the Java code and shows some benefits from dropping Java 6, but the changes to the pom file should be discussed in a separate PR.

add to whitelist

ok to test

Test build #52696 has started for PR 11541 at commit 78e3d32.

Test build #52697 has started for PR 11541 at commit 78e3d32.

Thank you, @mengxr !
I will remove pom.xml and checkstyle part from this PR soon.

Hi, @mengxr .
I removed the change on build related files. Now this PR contains pure Java and md files.
Also, the description of PR/JIRA issue is updated accordingly.

Test build #52699 has started for PR 11541 at commit 2e20843.

Test build #52697 has finished for PR 11541 at commit 78e3d32.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52697/
Test PASSed.

Test build #52699 has finished for PR 11541 at commit 2e20843.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52699/
Test PASSed.

Hi, @mengxr .
The latest commit passes the test.
Could you review this again please?

Test build #52696 has finished for PR 11541 at commit 78e3d32.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52696/
Test PASSed.

Merged build finished. Test PASSed.

The latest one is at commit 2e20843, and all three builds (including that) passes.

LGTM, merging to master. This is now just straightforward use of the diamond operator only.

Thank you, @srowen . Yes, it is.


update performance on az_span api

Updates from @JeffreyRichter  for az_span_copy and az_span_append()
this changes are meant to improve performance

Some general questions:

How do we know these are improvements? What are we measuring?
What is the impact/magnitude of these changes?

These code changes make the code smaller, faster, simpler, and more maintainable. The changes were measured by the azure rtos team in Shanghai. They would still like additional improvements but I'm not sure how deliver them. One idea is to make the functions inlineable.


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Incompatibility with static serialization formats

Currently it is not possible to deserialize most time items using formats that require the layout to be known, such as bincode or postcard due to the call to deserialize_any.
The fix should be rather straightforward by calling deserialize_tuple instead if the serde-human-readable feature is not enabled. I can follow up with a PR if this change is desirable.

Ah, I didn't realize this. I'll get a patch put together tonight, most likely.

Hmm. There's an interesting problem here. I guarantee the following:

Deserializers for human readable formats will fall back to the binary format if the human readable format fails to deserialize.

(from the 0.3.6 release notes)
I don't see how this could be implemented for a generic deserializer while upholding this guarantee, as the deserialize_X methods all require ownership of the deserializer. In hindsight, I probably shouldn't have worded it so strongly. In practice, I suspect no one is actually relying on this fact. I might be willing to overlook the minor breakage. I'll have to think about it.

with no fallback required

The current behavior has a fallback. If serde-human-readable is enabled, then the human-readable format will be tried first. If that fails, the binary format is tried. I don't believe that is possible in a non-self-describing format, as the deserializer requires ownership to be passed.

I'm talking about the case when serde-human-readable is not enabled.

Sure, but that's not the issue. When it is enabled, the current behavior should not break. That is my concern that I don't see a way around.

I see, well, making it work without the feature enabled would satisfy my use-case at least, I suspect most people who serialize to binary formats wouldn't bother with that feature anyway.
I don't see a way to fall back either for all cases, even untagged serde enums seem to use deserialize_any.

True. I could conditionally call the method directly, but it doesn't solve the issue entirely. I'll do that later.

Should be all set on main!


[Bug] set barcodeFormats on Android

Hi, thanks for creating a great package on code scanning. I have tried it on my testing app and found out it works very good except setting formats on Android device.
I set barcodeFormats: [BarcodeFormat.code39, BarcodeFormat.code128, BarcodeFormat.qr_code] to test whether it can scan only the 3 types of code above. In iOS device, it works properly, but Android can only scan the qr_code type, which is the last element specified on barcodeFormats array.
It should due to .setBarcodeFormats in below only set the formats given from the latest call.
https://github.com/Wayaer/fl_mlkit_scanning/blob/697de6f8cc366c476f10885b2e6de99163bdb5be/android/src/main/kotlin/fl/mlkit/scanning/FlMlKitScanningMethodCall.kt#L102-L121

Thank you for your question. I'll fix it as soon as possible

Thanks for fixing this. It works perfect now.


Update ga-yaml-parser step in gpuCI updating workflow

Looks like the auto-bump of ga-yaml-parser in https://github.com/dask/dask/pull/9553 changed the behavior of the action, making it so we haven't been running the gpuCI updating workflow in a while; this PR makes modifications that should unblock it.

[ ] Closes #xxxx
[ ] Tests added / passed
[ ] Passes pre-commit run --all-files

Yeah, I recently noticed that the Dask gpuCI images hadn't been rebuilt in a while and just resolved the failures in https://github.com/rapidsai/dask-build-environment/pull/54, so we might see some failures crop up in Dask/Distributed that weren't caught over the past few days

Are both the dask and distributed main branches now in a state where they're using rebuilt images?


Custom error message shape

Consider the following schema:
export const FileToUpload = z
  .instanceof(File)
  .refine((file) => file.size < FILE_SIZE_LIMIT, {
    message: `File size should be less than ${FILE_SIZE_LIMIT_MB} MB`,
  });

I'm using react-intl package from FormatJS for providing i18n and would like to be able to return a translation object instead of instead of a string, like:
export const FileToUpload = z
  .instanceof(File)
  .refine((file) => file.size < FILE_SIZE_LIMIT, {
    message: {
      id: 'uploaded_file_size',
      defaultMessage: `File size should be less than {max} MB`,
      values: { max: FILE_SIZE_LIMIT_MB },
    },
  });

The first solution that naturally comes to mind is this:
const useFileTuUploadSchema = () => {
  const intl = useIntl();

  return z.instanceof(File).refine((file) => file.size < FILE_SIZE_LIMIT, {
    message: intl.formatMessage({
      id: 'uploaded_file_size',
      defaultMessage: `File size should be less than {max} MB`,
      values: { max: FILE_SIZE_LIMIT_MB },
    }),
  });
};

Unfortunately, it doesn't solve my issue, because I need access to the actual translation object, not a formatted string (I render all translations in a custom component that allows admins to edit translations on the frontend)
Interestingly enough, returning a translation object instead of a string seems to work fine, but it's a terrible hack and would require shutting typescript errors, so do you have any suggestion here?
Thanks a lot!

Is this what you are looking for?
const FILE_SIZE_LIMIT = 123456789
const FileToUpload = z.instanceof( File ).superRefine( ( file, ctx ) => {
    if ( file.size < FILE_SIZE_LIMIT ) return true
    ctx.addIssue( {
        code: 'custom',
        message: `File size should be less than ${ FILE_SIZE_LIMIT } MB`,
        params: {
            id: 'uploaded_file_size',
            max: FILE_SIZE_LIMIT,
        },
    } )
} )

If you found my answer satisfactory, please consider supporting me. Even a small amount is greatly appreciated. Thanks friend! 🙏
https://github.com/sponsors/JacobWeisenburger

Perhaps this could also help:
https://youtu.be/JLJyWpOSMgA?si=TGuuAej2JZEBMf05

Is this what you are looking for?
const FILE_SIZE_LIMIT = 123456789
const FileToUpload = z.instanceof( File ).superRefine( ( file, ctx ) => {
    if ( file.size < FILE_SIZE_LIMIT ) return true
    ctx.addIssue( {
        code: 'custom',
        message: `File size should be less than ${ FILE_SIZE_LIMIT } MB`,
        params: {
            id: 'uploaded_file_size',
            max: FILE_SIZE_LIMIT,
        },
    } )
} )

Oh yeah, that'll do, thank you!


Adressleiste -Versenden-

Ich kann von meiner Defi Wallet nichts versenden (ETH BTC DeFi )   man kann auf der Adressleiste nichts eintragen, und auch die Höhe leiste funktioniert nur mit max.!

Hi @Bikefire1201 , thanks for your feedback, could you send your app log & binary log please?

Hi @Bikefire1201 , v2.1.4 has released, kindly update and see if your issue still persist.
https://github.com/DeFiCh/app/releases/tag/v2.1.4
If yes, please follow the guide above and send your app logs. Thank you.

Hi @Bikefire1201 , update v2.1.6 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, you may reopen this issue with new information (include screenshots) and your app logs. Thank you.

Hi @Bikefire1201 , update v2.1.6 is out https://defichain.com/downloads/.
Do give a thumbs up to this comment if your issue has been resolved, otherwise, you may reopen this issue with new information (include screenshots) and your app logs. Thank you.


Update Pages “PestControlPage”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
1212f2d761e490d047a03a6e042ebbf2ff531081

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/656fc15c7cf90e000791c8fb


feat: add function to fetch user playtime

This is my first time making a module PR and a typescript PR so you can bet that I had a lot of fun figuring out why stuff didn't work.
const {
	exchangeNpssoForCode,
	exchangeCodeForAccessToken,
	makeUniversalSearch,
	getUserPlayedTime,
} = require('psn-api');
const fs = require('fs');

async function thisWillTestPlaytime() {
	const access = await exchangeNpssoForCode(
		'PUT-YOUR-NPSSO-CODE-HERE'
	);
	const auth = await exchangeCodeForAccessToken(access);
	const userList = await makeUniversalSearch(auth, 'PUT-THE-PSNID-YOU-WANT-TO-TEST-HERE', 'SocialAllAccounts');
	const userId = userList.domainResponses[0].results[0].socialMetadata.accountId;
	const timePlayed = await getUserPlayedTime(auth, userId);
	fs.writeFileSync('./timePlayed.json', JSON.stringify(timePlayed));
}

thisWillTestPlaytime();

Expect best practices not being followed and edits needing to be made, I punched way above my weight with this PR.
Also, I didn't create a test, but maybe the repro code above will ease that process.

I have adjusted the title to enforce a minor release. Releases are automated and entirely derived from commit messages. Check https://github.com/semantic-release/semantic-release and https://commitizen-tools.github.io/commitizen/tutorials/writing_commits/

Hi! This PR has exceeded 30 days of inactivity. I will be closing it as a courtesy. Please feel free to re-open it at any time you would like to revisit this work.


Dealing with ZLIBNG_VERNUM format change in downstream code

I see that the develop branch uses a 32-bit encoding for ZLIBNG_VERNUM while the 2.0.x releases use a 16-bit encoding
This in develop
#define ZLIBNG_VERNUM 0x02010000L   /* MMNNRRMS: major minor revision status modified */

and this in 2.0.6
#define ZLIBNG_VERNUM 0x2060

I can understand the flexibility that this change adds, but it is a possible source of confusion for downstream code that needs to check ZLIBNG_VERNUM.
A  candidate for a short paragraph in the wiki perhaps?

Yes, that change would be a good idea to add to the wiki.

I have documented it here: https://github.com/zlib-ng/zlib-ng/wiki#versioning. Any edits are welcome.


Showing the user what tables & corresponding columns got changed while loading into the database.

DLT will change any column and table name to lower case and will also replace spaces with '_'. As to not confuse the user of the pipeline, it might be helpful to keep track and show these changes after the pipeline has finished loading data into the database. This could be done in load info (showing just how many name changes occurred), logging info every time there is a name change and this information should also be in the schema after the pipeline has finished loading.
Additionally if a column only has NULL values for all rows, it will be dropped. I think this should also be shown to a user. Basically any change of this sort where a user expects thing A to be loaded and thing B (or nothing) gets loaded should be shown to the user.

@redicane

Additionally if a column only has NULL values for all rows, it will be dropped
then it is a bug... I will create a ticket
this information should also be in the schema after the pipeline has finished loading
I'm planning to add lineage information to the schema. I'll attach this ticket to it


feat : README.md updated

PR Description 📜
updated readme.md
Fixes #29

Mark the task you have completed ✅

[X] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[X] I have performed a self-review of my own code or work.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] My changes generates no new warnings.
[X] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE

Thank you soo much for contributing to our repository 💗

@roy7077 please add gssoc and level labels and also assign me in this PR . Thanks


[BEAM-7427] Refactor JmsCheckpointMark to use SerializableCoder

This refactoring uses JmsCheckpointMark coder compatible.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[X] Choose reviewer(s) and mention them in a comment (R: @username).
[X] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[X] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Just to be clear: I strongly think my original implementation is largely better and the only change to do is to remove/transient the messages in the checkpoint mark.
Let me explain. In JMS, the messages stay in destination (topic or queue) up to the ack.
So, when we finalize the checkpoint mark, we ack the pending messages (stored in the checkpoint mark). If, for any reason, the checkpoint mark moves to another executor, and we don't have the pending messages, it's not a big deal as the messages are still in the destination, and will be consumed again.

Hello,
Thank you for this PR.
However, could you please clarify this point? In particular:

When does the finalizeCheckpoint method is used?
Could modifying the messages list to transient cause message loss?

My understanding is the following:
We have a Consumer on a node A. It hasn't acknowledged the message 1 and is stopped by the host. The finalizeCheckpoint method is called. As the Consumer hasn't switched to another node, the messages list still contains this message and it's acknowledged.
If unfortunately the Consumer has switched to a node B for any reason when the stop is asked, the message 1 is not transferred as the Checkpoint messages list is transient, so this consumer doesn't have this message anymore BUT this message is still in the message broker as it hasn't been acked.
The message 1 will be consumed again by the consumer and isn't lost.
Can you confirm it, please?
Does the only impact of changing messages to transient in the CheckpointMark is that they will be consumed twice, or can they be lost?
Thanks a lot for your answer,
Arthur


ats2: 0.3.9 -> 0.3.11

Motivation for this change
Updating ATS2
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build ats2

Success on x86_64-linux (full log)
Attempted: ats2
Partial log (click to expand)
shrinking /nix/store/9gqd0xr8y53rwa61n0p79q96kqdkr4ad-ats2-0.3.11/lib/ats2-postiats-0.3.11/bin/patscc
shrinking /nix/store/9gqd0xr8y53rwa61n0p79q96kqdkr4ad-ats2-0.3.11/lib/ats2-postiats-0.3.11/bin/myatscc
shrinking /nix/store/9gqd0xr8y53rwa61n0p79q96kqdkr4ad-ats2-0.3.11/lib/ats2-postiats-0.3.11/bin/patsopt
strip is /nix/store/21ymadblbmsbb2bk4q7gl4kjasp8zmgd-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/9gqd0xr8y53rwa61n0p79q96kqdkr4ad-ats2-0.3.11/lib  /nix/store/9gqd0xr8y53rwa61n0p79q96kqdkr4ad-ats2-0.3.11/bin
patching script interpreter paths in /nix/store/9gqd0xr8y53rwa61n0p79q96kqdkr4ad-ats2-0.3.11
/nix/store/9gqd0xr8y53rwa61n0p79q96kqdkr4ad-ats2-0.3.11/bin/patscc: interpreter directive changed from "/bin/sh" to "/nix/store/qckzjk3406va7h6s40cy9s75z2w715rq-bash-4.4-p19/bin/sh"
/nix/store/9gqd0xr8y53rwa61n0p79q96kqdkr4ad-ats2-0.3.11/bin/myatscc: interpreter directive changed from "/bin/sh" to "/nix/store/qckzjk3406va7h6s40cy9s75z2w715rq-bash-4.4-p19/bin/sh"
/nix/store/9gqd0xr8y53rwa61n0p79q96kqdkr4ad-ats2-0.3.11/bin/patsopt: interpreter directive changed from "/bin/sh" to "/nix/store/qckzjk3406va7h6s40cy9s75z2w715rq-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/9gqd0xr8y53rwa61n0p79q96kqdkr4ad-ats2-0.3.11...


Standalone client build i.promise.then is not a function

Hi, I tried making a standalone client build but I get this error

I tried using the client build included in the repository and it does work, but not with a new build

I have faced the same issue when attempting to load jpeg via the http(s) image loader. If you try to load the simpleJpeg.json example which contains a few jpegs it's going to fail. I investigated it a bit and I think it's because the /client/compatibility/cornerstoneWebImageLoader.js is old.
To fix that I've added the cornerstone-web-image-loader as a npm dependency to ohif-cornerstone package and loaded it in the main.js like the following:
cornerstoneWebImageLoader.external.cornerstone = cornerstone
I don't know if it is the best solution but I works for me. I hope it works for you too :)

Thanks @LucasAntoniassi!

This is fixed in PR #188, thank you @LucasAntoniassi for the information.
@yehudahkay, I see you submitted a PR  #185 to fix this (Thanks!) but seems you have a different approach and the diff is significant. Will you please describe more on the PR what changes you had to do?

Hi, I had simply updated cornerstonewebimageloader.js to the latest version in the project. I don't know this project very well but your approach seems better.

Looks like this was fixed in #188. Thanks for the report!


Tests: override HOME

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you successfully run brew tests with your changes locally?

This is an alternate fix for #2197.
This prevents tools like git and svn from reading their configuration in the user's home directory. This is useful, for example, if the user has git configured to GPG sign every commit, which would cause every place we git commit in the tests to fail.
I chose HOMEBREW_CACHE over TEST_TMPDIR in order to avoid introducing file leakage from newly-written dotfiles. We clear out HOMEBREW_CACHE before checking for leakage.
cc @alyssais, @woodruffw

Hm, looks like changing the home directory interferes with the Linux tests.

Weird. Only two failures in the brew search tests, and the last line they trace back to in our code is where we restore ENV ( ./test/spec_helper.rb:99). Not sure what to make of that, but userland programs on Linux frequently find the user's "home" in all sorts of incorrect ways (/home/$USER, /etc/passwd, etc). Maybe this is an instance of that?

We might want to try setting this just once in cmd/tests.rb? It doesn't need to be set and restored for every test, and that's where all the other test environment setup happens. Might help with whatever weirdness @woodruffw is describing.

We might want to try setting this just once in cmd/tests.rb?

👍 if it works.

Another thought: I think File.expand_path doesn‘t use HOME to expand ~, which might break stuff.

expand_path respects ENV["HOME"] for me, on Linux:
>> ENV["HOME"] = '/tmp' #=> "/tmp"
>> File.expand_path '~' #=> "/tmp"

That's 2.4.1, though, so maybe 2.3.x and below have different behavior?

I can‘t check right now, I just remember it from the Cask merger, where we had to use HOME instead of expand_path because it wouldn‘t respect Jenkins‘ HOME directory.

I can‘t check right now, I just remember it from the Cask merger, where we had to use HOME instead of expand_path because it wouldn‘t respect Jenkins‘ HOME directory.

Yes, correct 👍


Mismatch between the diagram and the Dataflow summary

[Enter feedback here]
There is a mismatch between Diagram and Dataflow bullet points.
Point no. 4 says red arrow line shows the source NAT but that line is missing in the picture.
Thanks

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

ID: d2c472ec-9be4-6e0e-eac6-0236e2e1044a
Version Independent ID: d2c472ec-9be4-6e0e-eac6-0236e2e1044a
Content: Network-hardened web app - Azure Example Scenarios
Content Source: docs/example-scenario/security/hardened-web-app.yml
Service: architecture-center
Sub-service: example-scenario
GitHub Login: @timwarner-msft
Microsoft Alias: timwarner

@pjtalkstech  - Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can update the document accordingly.

PR submitted. Thanks, @pjtalkstech!

#please-close


Updating word

Category

[x] Content fix

What's in this Pull Request?
Updating word ".Net" to ".NET"

thx @gauravgoyal5, appreciated.


Implement smart pointers and signals2 tracking

This is an experimental branch to solve some of the issues we've been facing, described in #57 and #86
Basically, I've added the ability to gain access to clients from CBs as std::weak_ptrs rather than raw pointers. Since they were saved as std::shared_ptrs already within the state machine, it was a pretty straight-forward change.
The main change is using boost::signals2 tracking facility to automatically monitor and control a slot's lifetime based on when the owning CB goes out of scope. Early tests show that this might prevents the issue we've been facing where member variables are being access from a CB callback after the callback has been deallocated upon state transitions.
Both these changes have been integrated in a small part within the sm_ferrari example's CB if you wanted to see the intended usage
This is very much WIP and a lot needs to be refined. Opening up this PR for visibility and comments
┆Issue is synchronized with this Jira Task by Unito

Hey @pabloinigoblasco. I understand your point that it shouldn't be possible for callbacks to trigger while states transition, but my investigations make me believe otherwise (see my last comments on #86 where I post a couple of examples showing member variables seemingly being deallocated while a callback is active).

Using client references should not be in general a problem since their lifetime is the whole application.

This is true in principle. However, you have to remember that the pointer variables pointing to the clients can go out of scope and become invalid, even while the clients themselves are perfectly valid. This is the underlying problem.

Client behaviors signal callbacks are disconnected before they are disposed in memory so no error should happen.

Could you point me to the code where this happens? When I look at  this, I see that client behaviours are deallocated and destroyed before the slots are disconnected from the signals. Does this not mean that the callbacks can still be triggered after a transition has been triggered? This seems to be the case here and would explain segfaults from referencing out-of-scope memory
The idea I'm exploring in this PR is to use boost::signals2's tracking capabilities (see here under the 'Automatic Connection Management' section) to syncronise the lifetimes of the callback (i.e. a slot attached for a SmaccSignal) and its owning object (client behaviour) and automatically manage their lifetimes. signals2 does this by attaching a slot to a shared_ptr reference to the owning object, hence the shared_from_this() call within the client behaviour. To reiterate, the client doesn't store any references to the client behaviour (that would be counter to SMACC' design!) - however, the SmaccSignal, i.e. a signals2::signal stores a reference to the client behaviour when it creates a slot, which it then uses to track the lifetime of the CB and deallocate the slot when the referenced client behaviour is destroyed during state transitions. This is an important point and one where I think there's some misunderstanding from your first point.

Hey @pabloinigoblasco. I understand your point that it shouldn't be possible for callbacks to trigger while states transition, but my investigations make me believe otherwise (see my last comments on #86 where I post a couple of examples showing member variables seemingly being deallocated while a callback is active).

I didnt notice the updates of that thread. I have to read it thoroughly, I only had a quick read. My primary goal is to determine if the issue at hand is a bug or if you are utilizing SMACC in a manner that is prohibited by design.
Nonetheless. I read in that bug the following:

"Components and clients are long-lived objects and I can see that their memory is still valid. However, the CB's member variables, including the pointers to long-lived objects, seem to become invalid or change after/during transitions"

That is correct by design. During the transition the state variables and client behavior variables are invalid.
I insist with the same question: who is trying to use those variables? Who is trying to access to those variables? And why?

Using client references should not be in general a problem since their lifetime is the whole application.

This is true in principle. However, you have to remember that the pointer variables pointing to the clients can go out of scope and become invalid, even while the clients themselves are perfectly valid. This is the underlying problem.

When states and client behaviors are being disposed internal variables become invalid. Totally agree with that: "we are in the process of removing the object". What is that callback? is that using SmaccSignals properly?

Client behaviors signal callbacks are disconnected before they are disposed in memory so no error should happen.

Could you point me to the code where this happens? When I look at this, I see that client behaviours are deallocated and destroyed before the slots are disconnected from the signals. Does this not mean that the callbacks can still be triggered after a transition has been triggered? This seems to be the case here and would explain segfaults from referencing out-of-scope memory

Signals must be disconnected before deallocating client behaviors. If that is not like that in the code, there is a bug we have to fix. I will have a look to that now.

I have been having a look to the code, it is right there is a race condition for multithreaded application that could cause a callback on a destroyed object. Hence, that is a bug that it is being shown in your application.
This is the first solution I propose:
https://github.com/robosoft-ai/SMACC/pull/91


The extention crashes when loading VSCode

I noticed that the only working feature seemed to be the language support. I could not find a way to debug or create launch tasks. The Extension Host printed an error during certain tasks like opening and saving a .asm file
Error log: https://pastebin.com/7i7wSPFe
System information:
OS / OS-Version: Ubuntu 20.04
VSCode Version: 1.47.2 (.deb Package not the snapcraft image)

Hi @feldim2425, thanks for reaching and pointing out this. Im still fixing some bugs , also in the next major release (ETA 2 weeks) there will be basic support search for the definition, and references for labels.
I'll review the pr and if everything is ok merge it and publish a new version.
Thanks again for the interest, if you have any other suggestions of features let me know and Ill take them in consideration.


What version of Tekton?

What version of Tekton is this inline with?  Using the latest Tekton Operator (0.67) all pipelines work, with or without a PVC.  When using the extension we get this error:
"message": "Failure executing: POST at: [https://<IP_ADDRESS>:443/apis/tekton.dev/v1beta1/namespaces/pipeline/pipelineruns](https://<IP_ADDRESS>/apis/tekton.dev/v1beta1/namespaces/pipeline/pipelineruns). Message: admission webhook \"webhook.pipeline.tekton.dev\" denied the request: mutation failed: cannot decode incoming new object: json: unknown field \"claims\". Received status: Status(apiVersion=v1, code=400, details=null, kind=Status, message=admission webhook \"webhook.pipeline.tekton.dev\" denied the request: mutation failed: cannot decode incoming new object: json: unknown field \"claims\", metadata=ListMeta(_continue=null, remainingItemCount=null, resourceVersion=null, selfLink=null, additionalProperties={}), reason=BadRequest, status=Failure, additionalProperties={})."
We have tried to downgrade Tekton to 0.66 but the error still persists when a PVC is being used.

For reference, we think we worked it out...
https://groups.google.com/g/quarkus-dev/c/VCM0zxVYkJI


How to generate episode datasets for exploration task after changing the height and radius of agent?

Since I have changed the height and radius of the agent, some points in the original episode dataset may not be navigable. How could I re-generate episodes?

We're currently using PointNav episodes for exploration as well. So you can use this generation script and adapt it to the agent size.
https://github.com/facebookresearch/habitat-lab/blob/master/habitat/datasets/pointnav/pointnav_generator.py

Got it, thanks a lot.


OAI: add a setDescription element to the ListSets response that indicates whether the stream is the current default

It might also be worth adding the date ranges when the stream was the default or the start date if it is currently the default.
Should the organization's slug or name be included in the setDescription?
See notes on "Stream-specific sets" in this story: https://github.com/pod4lib/aggregator/issues/417

Yes, it would also be helpful to have the setDescription include a code for the institution that created the stream.  (It would also be good for the stream name to include the organization name, but a specific code in the setDescription may be easier to automatically process.)
I'm not sure whether the slug or the LC MARC code is better to use as the code.  Either can be changed in the UI by an org owner at present as far as I know, but at least the MARC code is supposed to match an externally assigned persistent identifier.  Though if we're already using the slug in other consumption contexts (such as ResourceSync consumption) it might be good to use it here too, and advise org owners not to change the slug without good reason.

We use the slug throughout the application, the slug is generally easier to understand at a glance than the MARC org code, and I'm not sure all data providers have added a MARC code, so I think the slug is preferable in this case.


Support a new "scope" that includes current "compile" and "runtime" scopes together

Hi,
First of all thanks for this great plugin!
The current released version supports the following scopes as described in the plugin documentation

compile: Shows compile, provided, and system dependencies
provided: Shows provided dependencies
runtime: Shows compile and runtime dependencies
system: Shows system dependencies
test (default): Shows all dependencies

Would it be possible to support a new scope that consolidates the current supported compile and runtime scopes? So that the effective dependency graph would include all declared scopes except the test.
Feature #107 would actually cover this use case but as I understood it will only be released in the 4.x version. Maybe this new scope could be introduced and released sooner in version 3.4.x

Hi,
Does anything prevent you from using Version 4 when it is released. I plan to release the new version soon.

Hi,
Thanks for the quick response!
At first, I believe nothing will prevent using version 4. I would have to double-check but as long the structure of output in .json format does not change it wouldn't have problems.
Would you happen to have an estimated date for releasing version 4? or maybe an indication in terms of weeks or months?

Version 4.0.0 will be on Maven Central soon.


Need a quick way to find out what version we are using

We need to add an easy way to get the version number of the library for debugging purposes.
something like
omniture.version

Looking into this a bit further to do this it should look like this.
import omniture
omniture.__version__

To do this I would need to add the __version__ variable to the init.py

See answer by sorin on how to do this so you garuntee the version number is the same in your setup.py
https://stackoverflow.com/questions/458550/standard-way-to-embed-version-into-python-package/16084844#16084844

Add in #40


Add some C# qps worker features

Add support for multiple qpsworker features in C#:
-- support Poisson process load
-- support client_channels parameter
-- support outstanding_rpcs_per_channel parameter
Also a tiny fix for performance test build script.

CC @vjpai

LGTM

CLA is fine and ObjC tests haven't run (travis didn't pick up this change), but this PR is only touching C# code, so merging.


Getting DOTX061W when chunking map with keydefs setting hrefs to topic files

Expected Behavior
The publishing process should run without warnings.
Actual Behavior
When publishing a ditamap with <keydef> elements referencing topics via the @href attribute and setting @chunk=to-content on the <map> element warnings of type
[DOTX061W][WARN] ID '%1' was used in topicref tag but did not reference a topic element. The href attribute on a topicref element should only reference topic level elements.
are shown in the command line.
The output looks okay, but the warning is wrong. There are topics referenced. See code example at Steps to Reproduce
Steps to Reproduce

example.zip
Remove the @chunk attribute in the <map> in root.ditamap and everything runs fine without warnings

Copy of the error message, log file or stack trace

Environment

DITA-OT version: Tested with dita-ot-3.2.1 and dita-ot-2.5.4 and dita-ot-3.3.0-SNAPSHOT+07cf66a

Operating system and version:
Windows 10

How did you run DITA-OT?
Warnings occur:
dita-ot-3.3.0\bin\dita --input=root.ditamap --format=html5 (SNAPSHOT+07cf66a)
dita-ot-3.2.1\bin\dita --input=root.ditamap --format=html5
dita-ot-2.5.4\bin\dita --input=root.ditamap --format=html5
dita-ot-2.5.4\bin\dita --input=root.ditamap --format=pdf2
No warnings occur:
dita-ot-3.3.0\bin\dita --input=root.ditamap --format=pdf2 (SNAPSHOT+07cf66a)
dita-ot-3.2.1\bin\dita --input=root.ditamap --format=pdf2

Transformation type:
HTML5
PDF

I no longer get any errors for PDF for this; for HTML I don't get the `DOTX061W1 anymore. I get the following messages:
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:6:55: [DOTX061W][WARN]: ID 'id5' was used in topicref tag but did not reference a topic element. The href attribute on a topicref element should only reference topic level elements.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:6:55: [DOTX023W][WARN]: Unable to retrieve navtitle from target: 'root.dita#id5'.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:6:55: [DOTX027W][WARN]: Unable to retrieve linktext from target: 'root.dita#id5'.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:11:59: [DOTX061W][WARN]: ID 'id1' was used in topicref tag but did not reference a topic element. The href attribute on a topicref element should only reference topic level elements.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:11:59: [DOTX023W][WARN]: Unable to retrieve navtitle from target: 'root.dita#id1'.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:11:59: [DOTX027W][WARN]: Unable to retrieve linktext from target: 'root.dita#id1'.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:12:57: [DOTX061W][WARN]: ID 'id2' was used in topicref tag but did not reference a topic element. The href attribute on a topicref element should only reference topic level elements.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:12:57: [DOTX023W][WARN]: Unable to retrieve navtitle from target: 'root.dita#id2'.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:12:57: [DOTX027W][WARN]: Unable to retrieve linktext from target: 'root.dita#id2'.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:13:57: [DOTX061W][WARN]: ID 'id3' was used in topicref tag but did not reference a topic element. The href attribute on a topicref element should only reference topic level elements.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:13:57: [DOTX023W][WARN]: Unable to retrieve navtitle from target: 'root.dita#id3'.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:13:57: [DOTX027W][WARN]: Unable to retrieve linktext from target: 'root.dita#id3'.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:14:57: [DOTX061W][WARN]: ID 'id4' was used in topicref tag but did not reference a topic element. The href attribute on a topicref element should only reference topic level elements.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:14:57: [DOTX023W][WARN]: Unable to retrieve navtitle from target: 'root.dita#id4'.
[move-meta] file:/C:/dita-ot/TESTGITHUB/3203/root.ditamap:14:57: [DOTX027W][WARN]: Unable to retrieve linktext from target: 'root.dita#id4'.

I'm not sure how exactly to resolve that but it's clearly not a good result. We have key definitions for topics that are (or sort of are) merged into one merged doc, so we rewrite them, but they aren't actually merged because they are just key definitions so they should not be there. As long as keys are resolved before chunking it might be best if key definitions are ignored for this step, provided that the key resolution has already moved the appropriate metadata from the definition.


Remove expected flag and add show-excluded flag to maestro docs

Co-authored-by: Zayd Zori<EMAIL_ADDRESS>

Thanks @miafryling
We noticed that you deleted the maestro regenerate heading. We assume that was a slip-of-the-mouse and will add it back.


Shortcut Request - Cmd/Ctrl + K to clear logs

 Issue by ericclemmons
Tuesday Nov 01, 2016 at 19:22 GMT
Originally opened as https://github.com/storybooks/storybook-addon-actions/issues/25

This is a common shortcut in terminals (and even in the Chrome DevTools!) to clear the output.
This would be very handy.
I can submit this request upstream in https://github.com/kadirahq/react-storybook/issues/new as well, if you'd prefer.
Thanks!

 Comment by arunoda
Saturday Nov 05, 2016 at 05:39 GMT

We would to add a feature to support for addons to register keyboard shortcuts.
With that we could do this.
I add a task here: https://github.com/orgs/storybooks/projects/1#card-698087

Adding an api for registering shortcuts will be part of: #865 #1209
Addon-actions would also have to be modified to take advantage of this and clear itself when the called.


Implement profanity filter

-profanity filter for reviews/names
Originally posted by @kappanjoe in https://github.com/gagataiga/Phone-Review-App/issues/8#issuecomment-1497248242

This package might be useful: https://www.npmjs.com/package/profanity-check


Install problems

I run this as root:
curl -sL http://git.io/vtf5N | bash -s arkserver
Then with the user arkserver this:
arkmanager install
And get this:
[  ERROR  ]     Your SteamCMD root seems not valid.
[  ERROR  ]     Your SteamCMD exec could not be found.
Running command 'install' for instance 'main'
[  ERROR  ]     Your SteamCMD root seems not valid.
[  ERROR  ]     Your SteamCMD exec could not be found.
[  WARN  ]      Your ARK server exec could not be found.
mkdir: cannot create directory ‘/home/steam’: Permission denied
[  ERROR  ]     The ARK SavedArks directory is not writable, and saveworld will fail
Creating the ARK server root directory (/home/steam/ARK)
mkdir: cannot create directory ‘/home/steam’: Permission denied
/usr/local/bin/arkmanager: line 1130: cd: /home/arkserver/steamcmd: No such file or directory
Installing ARK server ... -/usr/local/bin/arkmanager: line 440: /home/arkserver/steamcmd/steamcmd.sh: No such file or directory

There are just 2 lines, and i did it although wrong?

When I did mine, I left it with steam as the username.  I also installed the steam stuff in the steam users acct.  I'm guessing you have a different account for steam.  I would think you could just install steamcmd in the arkserver users folder if you want.  I followed the steam guide found at:  https://developer.valvesoftware.com/wiki/SteamCMD#Linux   I found it easy to leave it all as the same account.  I am no expert with this stuff.  I'm brand new with this.  Good luck!

It sounds like you first ran it using steam as the user argument, and then with arkserver as the user argument.  Either that, or the install script somehow failed to modify the config.
Please open /etc/arkmanager/arkmanager.cfg with a text editor, and change the following settings:

Change steamcmd_user from steam to arkserver
Change arkserverroot from /home/steam/ARK to /home/arkserver/ARK
Change steamcmdroot from /home/steam/steamcmd to /home/arkserver/steamcmd
Change steamcmd_appinfocache from /home/steam/Steam/appcache/appinfo.vdf to /home/arkserver/Steam/appcache/appinfo.vdf

I missed the steamcmd install thought it would do that too xD

v1.6.04 will now install steamcmd if it's not already installed (and if the steamcmdroot setting is set correctly).


Breakpoints can not be set

Browser Version:
Edge for Business 131.0.2903.86
Problem Description:
When using the developer tools in Edge, I encounter an issue while trying to set breakpoints in the source tab. Although the source code is displayed, attempting to add a breakpoint by clicking on a line does not reflect in the list of breakpoints. Additionally, the red dot, which indicates an active breakpoint, does not appear.
Impact:
This issue is urgent as it disrupts our web development process. Our corporate policy mandates the use of this specific version of Edge, and no alternative browsers or versions are permitted.
Steps to Reproduce:

Open Microsoft Edge for Business (version 131.0.2903.86).
Navigate to a webpage and open the Developer Tools .
Go to the 'Sources' tab.
Attempt to set a breakpoint by clicking on a line of code.

Expected Behavior:
The breakpoint should appear in the list of breakpoints, and a red dot should be visible next to the line number.
Actual Behavior:
The breakpoint does not appear in the list, and the red dot is absent.
Additional Information:
The problem persists even after clearing cache and cookies.
Disabling all extensions did not resolve the issue.
The issue occurs in normal and InPrivate browsing modes.
I appreciate your prompt attention to this matter, as it significantly impacts our development workflow.

Thank you for reporting this. It's a known issue that has already been fixed. Tracked here: https://github.com/MicrosoftEdge/DevTools/issues/293


Mirror rendering is broken

Is it a mod compatibility issue?
No
If it's a mod compatibility issue, please provide the link(s) to the incompatible mod(s)
No response
Minecraft version
1.20.2
Describe the issue
Mirrors appear to lack any draw distance and are strangely rendering the blocks they're made from. See video for more information: https://cdn.discordapp.com/attachments/190944839183892481/1185933335009165322/mirrorbroke.mp4

Does this issue occur with the mirrors that are created via flint-and-steel naturally? (In the video the mirror seems modified by commands.)

I initially created it naturally using flint and steel, I only modified the destination to provide more information about the issue.

Additionally, the same thing happens with portals that have been turned into 'fake enterable mirrors,' it happens with all mirrors no matter how they're created or edited, but it does not happen with normal portals that have simply been turned around 180 degrees. https://cdn.discordapp.com/attachments/190944839183892481/1186118722562297926/allmirrorbroke.mp4

Does this happen with no other mods installed? (Such as Figura and other mods like it)

Yes.

I can reproduce the issue. Going to fix.

Should be fixed in 4.1.3


Add celsius to fahrenheit functionality

Name
Add celsius to fahrenheit functionality
Describe your Issue
Add celsius to fahrenheit functionality

Hello sir, Can you assign this issue to me?

@roshan-acharya Go ahead


Downgrade CMake 3.17 -> 3.16 for CLion IDE users to build the project and enjoy intelligent code completion

Hi thanks for this interesting assignment! There are many CLion IDE users. The latest IDE currently only supports CMake 3.16, and it seems that this project does not need those latest features introduced in CMake 3.17. Therefore, I am making this PR to allow CLion IDE users to build the project and enjoy intelligent code completion.

Sounds good, thanks!


[1.18.2] Resourceful Furnace import bug

There seems to be a defect in the automatic loading function of Resourceful Furnace.
If the carry-in and carry-out match in the GUI for specifying the carry-in slot, the material will be carried out as it is.

Oh, I found another problem when I was investigating the problem of delivery
It seems that firing will stop if you return to the title and reconnect while using Resourceful Furnace.
If you take out the items inside and bring them back in, it will work properly.


Allow customizing messages in cmd/serv.go

Feature Description
Allow customizing responses sent to clients from cmd/serv.go, especially messages from this switch block:
func runServ(c *cli.Context) error {
	// ...
	switch key.Type {
	case asymkey_model.KeyTypeDeploy:
		println("Hi there! You've successfully authenticated with the deploy key named " + key.Name + ", but Gitea does not provide shell access.")
	case asymkey_model.KeyTypePrincipal:
		println("Hi there! You've successfully authenticated with the principal " + key.Content + ", but Gitea does not provide shell access.")
	default:
		println("Hi there, " + user.Name + "! You've successfully authenticated with the key named " + key.Name + ", but Gitea does not provide shell access.")
	}
	// ...
}

This would be useful for both localization as well as adding custom branding.
Screenshots
$ ssh -T git@GIT_SERVER
Hi there, USER_NAME! You've successfully authenticated with the key named KEY_NAME, but Gitea does not provide shell access.
If this is unexpected, please log in with password and setup Gitea under another user.

It's a small change. Maybe you can hold them in your fork. I don't think it's necessary to have a configuration for that.

I would argue that It'd still be useful for rebranding a custom Gitea instance. It'd be great if it took APP_NAME from the custom/conf/app.ini or something like that.

If you would like to rebrand, you would need to change a lot, but not only these strings. Theoretically you have to maintain your own changes.
APP_NAME is for the whole instance's name, it isn't for rebranding.


Stop refresh on Trip ID Error

Checklist

[x] Appropriate branch selected
[x] Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description
This PR seeks to address an issue where users who were saving a trip id previously held by a different trip were getting an error that asked them to reload the page and lose their changes.

Returns an error message that can be closed instead of prompting refresh
Stop state refresh on error
Fixes a small error where field could not be reset to its original value after error

Thanks everyone!


improvement use dynamic segment size to cache remote file block

Proposed changes
CachedRemoteFileReader has used fixed segment size(file_cache_max_file_segment_size=4M) to cache remote file blocks. However, the column size in a rowgroup/strip maybe smaller than 10K if a parquet/orc file has many columns, resulting in particularly serious read amplification. For example:
Q1 in clickbench: select count(*) from hits
-  FileCache:  0ns
  -  IOHitCacheNum:  552
  -  IOTotalNum:  835
  -  ReadFromFileCacheBytes:  19.98  MB
  -  ReadFromWriteCacheBytes:  0.00
  -  ReadTotalBytes:  29.52  MB
  -  SkipCacheBytes:  0.00
  -  WriteInFileCacheBytes:  915.77  MB
  -  WriteInFileCacheNum:  283

Only 30MB of data is needed, but 900MB+ of data is read from hdfs. The query time of Q1(single scan thread) increased from 5.17s to 24.45s when enable file cache.
Therefore, this PR introduce dynamic segment size which is based on the read_size of the data. In order to prevent too small or too large IO, the segment size is limited in [4096, file_cache_max_file_segment_size].
Q1 in clickbench is 5.66s when enable file cache. The performance is almost the same as if the cache is disabled, and the data size read from hdfs is reduced to 45MB.
-  FileCache:  0ns
    -  IOHitCacheNum:  297
    -  IOTotalNum:  835
    -  ReadFromFileCacheBytes:  8.73  MB
    -  ReadFromWriteCacheBytes:  0.00
    -  ReadTotalBytes:  29.52  MB
    -  SkipCacheBytes:  0.00
    -  WriteInFileCacheBytes:  45.66  MB
    -  WriteInFileCacheNum:  544

Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[x] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[x] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.24 seconds
load time: 467 seconds
storage size:<PHONE_NUMBER>0 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230207155046_clickbench_pr_91534.html


Migrate from eth_utils.compose to toolz.functoolz.compose

What is wrong?
The library currently uses the compose function from the ethereum-utils library.  This function has been deprecated as of ethereum-utils>=0.3.0
How can it be fixed?

Add the toolz library to the web3.py dependencies in setup.py
Change all places in web3.py that use eth_utils.compose to instead use toolz.functoolz.compose.

Note that these two functions do not behave the same.  The eth_utils.compose function applies the functions from left to right.  The toolz.functoolz.compose function applies the functions from right to left.  When swapping these out, care must be taken to reverse the order of the function composition to preserve the intended ordering.

Picking this up. going to knock this one out before 203

done in #205


[FLINK-29791] Redirect gc log from stdout to file

Redirct gc log from stdout to file

Hi @LadyForest I want to fix this bug, can you help to review this PR? THX

Hi @LadyForest I want to fix this bug, can you help to review this PR? THX

Thx for the contribution, will take a look soon


State Save : Reopen last file

Description
File's state before it was closed is not maintained when it is re-opened. Cursor is always at the beginning of the page no matter where you were last time.
Steps to Reproduce

Open any file.
Do some work.
Close it
Reopen it with Ctrl + Shift + T

Expected behavior: It should take the cursor to the place where it was last when file was closed. Preferably, word.line selection, if any, should also be maintained.
Actual behavior: Keep the cursor at the beginning of the page
Versions
Atom    : 1.7.4
Electron: 0.36.8
Chrome  : 47.0.2526.110
Node    : 5.1.1

@gauravmak https://github.com/tmunro/remember-file-positions should do what you want

https://github.com/atom/atom/issues/9437?

@Ben3eeE Good call!
Closing as duplicate of #9437.


Add option to visualize raster in detection visualization

This adds the following functionality

Utility functions for downsampling, reprojecting, and visualizing rasters
An option to show a raster behind the detections in RegionDetections and RegionDetectionsSet
Examples of using this visualization in the example prediction notebook

Note that this relies #49


Question : Non blocking request

Hi,
I have a question regarding non blocking request. I wanted to send the request, but I don't care to wait for the response. Is there a way to achieve it ?
Say calling a url : http://example.com/post/hello?w=222 . Just a dummy one.
Thank you.

Hi harikt,
see the example in the readme:
https://github.com/amphp/artax#simple-uri-get
as long as you dont call $response = Amp\wait($promise); it doesnt block.

Thank you.
I will test it.

@harikt: Can we close this issue?

Basically the documentation seems wrong and I don't remember where I ended up.


[Autocomplete] Tags with white space is getting listed as separate tags

Expected behavior
One tag with white space should get listed as one tag
Actual behavior
Tag with white space is getting listed as different tags
Steps to replicate

add a tag

tags: one, two with space, three

autocomplete tags lists two with and space are getting listed as different tags.

Version
Gauge version: 0.9.4

Plugins
-------
csharp (0.10.2.nightly-2017-10-26)
html-report (4.0.2)
java (0.6.6.nightly-2017-09-14)
js (2.0.3.nightly-2017-11-07)
ruby (0.4.3.nightly-2017-10-17)
screenshot (0.0.1)
spectacle (0.1.2)
xml-report (0.2.0)

Details included in the story


How to make pull request for templates?

Hi,
There is a bug in test cases of default template for Nodejs8.10. I'm wondering how can I make a pull request and suggest an edit? Can you please tell me where are the templates files on git?

Hello @azarboon - Thank you for opening up an issue to notify us!
What issue have you found? There was a PR that recently fixed the test case and should be in the next release -- https://github.com/awslabs/aws-sam-cli/pull/662
FWIW, you can find the latest test case for NodeJS here with the test case fixed: https://github.com/awslabs/aws-sam-cli/blob/develop/samcli/local/init/templates/cookiecutter-aws-sam-hello-nodejs/{{cookiecutter.project_name}}/hello_world/tests/unit/test_handler.js
And this is the test case available in the current SAM CLI (0.6) release:
https://github.com/awslabs/aws-sam-cli/blob/master/samcli/local/init/templates/cookiecutter-aws-sam-hello-nodejs/{{cookiecutter.project_name}}/hello_world/tests/unit/test_handler.js
We're considering moving all default sam init templates to a dedicated repository to make this process easier and with better automated tests.

@azarboon Is the test failure you are experiencing different than what @heitorlessa pointed out?

Currently, init creates a test function against app.lambda_handler function which doesn't exist. It should be app.lambdaHandler. As I can see, the new test case (proposed one?) will fix the problem.

The update from app.lambda_handler to app.lambdaHandler in the test case has already been merged and waiting for a release. Closing this as it was already been fixed.


Fix reference links to .from_io methods

Ref: read_bytes
Notice the "See ..." section

Comparison to write_bytes

That's incorrect, since they're class methods.

@Sija Oh, you're right, #to_io is different to .from_io :sweat_smile: , Any idea why this link wasn't generated here?


ENTESB-19495: Skips problematic 1.6.8 release

For new release of 1.6.9, define a skip rather than replace to allow
users to workaround the probelmatic 1.6.8 release

@balejosg @squakez @VaishnaviR18
I have set this to draft for the moment as there may be other changes to be made for bumping the version to 1.6.9. However, at the moment this represented the minimum I needed to create a 1.6.9 operator and bundle.


Revanced Patch?

Hi, really love your extension and would love to have its features on the twitter app since as of yesterday even my own API key I was using on a 3rd party app finally died. Not sure if you are aware of what Revanced is but if you are I would love for this extension to be also created as Revanced Patch pls, since there aren't many patches available atm for twitter. Thanks.
Links:
https://revanced.app/patches/com.twitter.android
https://revanced.app/docs

https://revanced.app/patches/com.twitter.android

"This page received a cease and desist letter from a multi-billion dollar tech company"
Count me out of that :)

Well that was not the case last night LOL, still using Twitter Revanced and also YouTube Revanced as we speak...


REST API is here

Hey there,
just wanted to let you guys know, we managed to release bookmarks v0.10 which comes with the brand new v2 API. Happy hacking :)

Yes thx :)

@marcelklehr keep it open I'm not the dev xD


[Feature] Add Operation to query the internal Prometheus instance

Problem Statement
I want to be able to assert that a certain operation within the cluster is completed within x seconds, or is completed with less than x errors. This information is not available to Chainsaw or is an aggregate over x period of time and so is hard or impossible for Chainsaw to gather on its own.
Solution Description
Have a prometheus running within the cluster scraping pods and aggregating this information over time. Then have a operation in Chainsaw that can run some PromQL against that Prometheus and assert if a threshold has been reached.
Alternatives
Currently I used a script like this to achieve the solution - this assumes your Chainsaw tests are running out side the Cluster:
- script:
        timeout: 10s
        env:
          - name: query
            value: "histogram_quantile%280.999%2C+sum%28rate%28sidecar_injection_time_seconds_bucket%5B5m%5D%29%29+by+%28le%29%29"
        content: |
          kubectl -n monitoring port-forward svc/prometheus-server 8091:80 > /dev/null &
          sleep 0.5
          wget -O - "http://localhost:8091/api/v1/query?query=${query}"
          kill %1
        check:
          (json_parse($stdout).status): success
          (json_parse($stdout).data.resultType): vector
          (length(json_parse($stdout).data.result)): 1
          (to_number(json_parse($stdout).data.result[0].value[1]) < `1.0`): true

This works, but is verbose and error prone, so be great to have a specific operation for this.
Additional Context
No response
Slack discussion
No response
Research

[X] I have searched other issues in this repository and mine is not recorded.

I added a proxy helper that should do the trick:
    - proxy:
        apiVersion: v1
        kind: Service
        namespace: monitoring
        name: prometheus-serve
        port: '80'
        path: (join('=', ['/api/v1/query?query', $query]))

great thanks @eddycharly - this seems like a great solution that will be useful for more than just prom use cases!


Plan an Cassandra implementation of the IndexedStorage trait

Examine the interface IndexedStorage and figure out the exact Cassandra queries necessary for each function of the trait.
This is a preparation step before actually writing the implementation. Document the plan as a comment of this issue. Make sure to also plan the Cassandra statements required to initialize the database (create tables but only if does not exists, etc). The trait implementation will have to run these if needed so can be transparently used in place of the other implementations.

IndexedStorage in Cassandra
Table definition

Indexed storage tables

CREATE TABLE index_storage (         namespace text,         key text,         id bigint,  -- Numeric identifier for each entry          value blob,  -- Arbitrary binary payload         PRIMARY KEY ((namespace, key), index)  ) WITH CLUSTERING ORDER BY (index ASC);  CREATE INDEX key_idx ON index_storage (key);

Queries by function

trait function
statement

number_of_replicas
SELECT replication FROM system_schema.keyspaces WHERE keyspace_name = ?

wait_for_replicas
Not implemented in Cassandra, we can control the replication acknowledgment behavior through consistency levels, but this applies on a write operation only.

exists
SELECT 1 FROM index_storage WHERE namespace = ? AND key = ?;

scan
SELECT key FROM index_storage WHERE namespace = ? AND key LIKE ? AND token(key) > ? LIMIT ?

append
INSERT INTO inedx_storage (namespace, key, id, value) VALUES (?,?,?,?);

length
SELECT COUNT(*) FROM index_storage WHERE namespace = ? AND key = ?;

delete
DELETE FROM index_storage WHERE namespace = ? AND key = ?;

read
SELECT id, value FROM index_storage WHERE namespace = ? AND key = ? AND id >= ? AND id <= ?;

first
SELECT id, value FROM index_storage WHERE namespace = ? AND key = ? ORDER BY id ASC LIMIT 1;

last
SELECT id, value FROM index_storage WHERE namespace = ? AND key = ? ORDER BY id DESC LIMIT 1;

closest
SELECT id, value FROM index_storage WHERE namespace = ? AND key = ? AND id <= ? ORDER BY id DESC LIMIT 1;

drop_prefix
DELETE FROM index_storage WHERE namespace = ? AND key = ? AND id = ( SELECT id FROM index_storage WHERE namespace = ? AND key = ? AND id < ? ORDER BY id DESC LIMIT 1 );


Error installing (Store 2)

Hello,
I have a problem installing via composer (store 2).
I took the stept described in the README.md.
When I run the command "composer update fruitcakestudio/omnipay-sisow", it gives me the following error:
Problem 1
    - The requested package expressodev/laravel-codeigniter-db == <IP_ADDRESS> could not be found.

I looked in the composer.json file, and I found this;
"require": {
        "expressodev/laravel-codeigniter-db": "dev-master"

How can I solve this problem?

Now I got the message:
Package nettob/omnipay-sisow is abandoned, you should avoid using it. Use fruitcakestudio/omnipay-sisow instead

But I used the fruitcakestudio version.. At least it working now...


Implement soma repository manifest deserialization

All soma repositories must setup soma.toml on their root directory.
It will initially support binary executable only.
For example, if you want to setup the soma repository which

name of the problem is disposable
position of the executable is /home/disposable
executable is available to download for challengers
position of the flag is /home/flag
flag is a read-only file
service runs on ubuntu:16.04 and not self-hosted

then the content of soma.toml should be written as following:
name = "disposable"

[[executable]]
path = "deploy/home/disposable"
public = true

[[readonly]]
path = "deploy/home/flag"

[binary]
os = "ubuntu:16.04"
entry = "./disposable"

Merged on #10


Add timeunit and delay options

preview: https://kimikage.github.io/ProfileSVG.jl/previews/PR60/other-options/#timeunit-and-delay
Closes #59

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3246b53). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master       #60   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         2
  Lines             ?       189
  Branches          ?         0
==========================================
  Hits              ?       189
  Misses            ?         0
  Partials          ?         0

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 3246b53...2217f81. Read the comment docs.

wow thank you!

Looks like it can be merged and tagged?


HattCI segfault

Hello,
Thanks for publishing this software, it looks really cool! 😁
I'm getting a strange error about HattCI using the example data you uploaded in #3:
Python 2.7.15 | packaged by conda-forge | (default, Mar  5 2020, 14:56:06)
[GCC 7.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import migfinder as mf
>>> mf.main("mgm4570<IP_ADDRESS>.fasta")
sh: line 1:  5816 Segmentation fault      (core dumped) hattci.out -b -s 1000 -t 6 mgm4570<IP_ADDRESS>.fasta mgm4570<IP_ADDRESS>.tmp > mgm4570<IP_ADDRESS>.hmmlog
mv: cannot stat ‘hmmresults/outHattCI.fasta’: No such file or directory
HattCI done!
Failed to open sequence file ../hmmresults/mgm4570<IP_ADDRESS>_hattci.fasta for reading

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/users/bwingfield/repos/migfinder/migfinder/migfinder.py", line 890, in main
    infernal(fastafile="../hmmresults/"+out+"_hattci.fasta", out=out)
  File "/users/bwingfield/repos/migfinder/migfinder/migfinder.py", line 76, in infernal
    filein = open(out+".tmp")
IOError: [Errno 2] No such file or directory: 'mgm4570<IP_ADDRESS>.tmp'

hattci.out segfaults and then the rest of the script errors from there. However, if I run the same command outside of migfinder it's fine:
$ hattci.out -b -s 1000 -t 6 mgm4570<IP_ADDRESS>.fasta mgm4570<IP_ADDRESS>.tmp > mgm4570<IP_ADDRESS>.hmm.log
$ cat mgm4570<IP_ADDRESS>.hmm.log
Mbases processed: 29.916034
Sequences read: 23682
Sequences processed: 23682
Hits (top strand): 635
Hits (bottom strand): 651

Any help would be appreciated.
Cheers,
Ben

Hi Ben,
thanks a lot for your interesting in migfinder and the detailed report of the error!
I believe the error to be generated by the relative paths in the code. I will have a look at it tonight and get back to you as soon as possible.
Best wishes,
Mariana

Hi again, Ben!
I believe you need to provide the full path to your fasta even if it is in the directory where you are working from. Can you try that and let me know if it works, please?
Sorry for the inconvenience.
Mariana

Hi Mariana,
Thanks for your help :grin: I'm getting a different error about a missing model when I give a full path, but I think it's related to paths again (the file definitely exists).
(mig) [bwingfield@login2 [kelvin2] migfinder-master]$ pwd
/users/bwingfield/repos/migfinder-master
(mig) [bwingfield@login2 [kelvin2] migfinder-master]$ python
Python 2.7.15 | packaged by conda-forge | (default, Mar  5 2020, 14:56:06)
[GCC 7.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import migfinder as mf
m>>> mf.main("/users/bwingfield/repos/migfinder-master/test/mgm4570<IP_ADDRESS>.fasta")
HattCI done!

Error: CM file migfinder/cm_model/selection109_oriR.cm not found (nor an .i1m binary of it)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "migfinder/migfinder.py", line 890, in main
  File "migfinder/migfinder.py", line 76, in infernal
IOError: [Errno 2] No such file or directory: 'mgm4570<IP_ADDRESS>.tmp'

I had some trouble installing migfinder, and I think this could be related to that. When I try to install migfinder with pip (python 2.7):
(mig) [bwingfield@login2 [kelvin2] migfinder-master]$ pip install --user -e migfinder
DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 is no longer maintained. pip 21.0 will drop support for Python 2.7 in January 2021. More details about Python 2 support in pip, can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support
Obtaining file:///users/bwingfield/repos/migfinder-master/migfinder
ERROR: Could not find a version that satisfies the requirement Bio (from migfinder==1.0) (from versions: none)
ERROR: No matching distribution found for Bio (from migfinder==1.0)

I can install OK using python3 -m pip install --user -e migfinder (suggested in #1) but this installs the package into my python3 site-packages, where python2 can't read from. If I try to run migfinder in python3:
(mig) [bwingfield@login2 [kelvin2] repos]$ python3
Python 3.6.5 |Anaconda, Inc.| (default, Apr 29 2018, 16:14:56)
[GCC 7.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import migfinder as mf
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/users/bwingfield/repos/migfinder-master/migfinder/migfinder.py", line 286
    print "attC hits from pipeline: "+str(len(table))
                                    ^
SyntaxError: invalid syntax

So I think that migfinder requires python 2.7 to run. I've just been importing migfinder successfully by running python in the /users/bwingfield/repos/migfinder-master directory. Can I double check with you the best process to reinstall migfinder?
Thanks,
Ben

Hi again Ben @nebfield ,
Indeed this is probably related to the installation. I have started installation from scratch on my computer and found an issue with Bio. I have updated the setup.py. Can you remove migfinder and install it again, using python 2.7 and the files here on GitHub, please?
I am really sorry for the inconvenience!
Best wishes,
Mariana

I can install with python 2.7 now! 🎉 Thanks for the update
I have found a workaround for the missing model error (Error: CM file migfinder/cm_model/selection109_oriR.cm not found (nor an .i1m binary of it)). You have to run migfinder in the top-level directory that the git repo is cloned to, for example:
(mig) [bwingfield@login2 [kelvin2] repos]$ pwd
/users/bwingfield/repos
(mig) [bwingfield@login2 [kelvin2] repos]$ python
Python 2.7.15 | packaged by conda-forge | (default, Mar  5 2020, 14:56:06)
[GCC 7.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import migfinder as mf
>>> mf.main("/users/bwingfield/repos/migfinder/exampleData/test.fasta")
HattCI done!
attC hits from pipeline: 7
Infernal done!
Prodigal done! Staring pos-processing...
Pos-processing done!

Runs OK, but running outside of this specific directory fails:
(mig) [bwingfield@login2 [kelvin2] exampleData]$ pwd
/users/bwingfield/repos/migfinder/exampleData
(mig) [bwingfield@login2 [kelvin2] migfinder]$ python
Python 2.7.15 | packaged by conda-forge | (default, Mar  5 2020, 14:56:06)
[GCC 7.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import migfinder as mf
>>> mf.main("/users/bwingfield/repos/migfinder/exampleData/test.fasta")
HattCI done!

Error: CM file migfinder/cm_model/selection109_oriR.cm not found (nor an .i1m binary of it)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "migfinder/migfinder.py", line 890, in main
    infernal(fastafile="../hmmresults/"+out+"_hattci.fasta", out=out)
  File "migfinder/migfinder.py", line 76, in infernal
    filein = open(out+".tmp")
IOError: [Errno 2] No such file or directory: 'test.tmp'

I'm not a Python expert (I prefer R) but I think a fix would be to get the model path from the absolute install directory path. For example:
(mig) [bwingfield@login2 [kelvin2] repos]$ pip list --user
DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 is no longer maintained. pip 21.0 will drop support for Python 2.7 in January 2021. More details about Python 2 support in pip, can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support
Package   Version Location
--------- ------- -------------------------------------------
migfinder 1.0     /users/bwingfield/repos/migfinder/migfinder
(mig) [bwingfield@login2 [kelvin2] repos]$ python
Python 2.7.15 | packaged by conda-forge | (default, Mar  5 2020, 14:56:06)
[GCC 7.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import migfinder as mf
>>> mf.__file__
'/users/bwingfield/repos/migfinder/migfinder/migfinder.pyc'

But I could be totally wrong! 😅
Also, I'm not sure how I would run migfinder on multiple samples, since some output files have the same name for every run (e.g. hmmresults). Can I suggest an additional argument for the main function to include specifying an output directory?
Apologies for this gigantic wall of text and thanks again for all of your help!

Hi Ben!
Very nice exploratory work with the path! What I am confused about is that I can successfully run  migfinder on any path on my computer :/ In fact, even when I use another computer, I get the correct path if I do:
python
import pkg_resources
pkg_resources.resource_filename('migfinder', 'cm_model/selection109_oriR.cm')

However, it does seem another user had the same problem. (I also prefer R!)
Can you report what does the above spill out for you, please?
And a very ad-hoc solution is for you to edit line 22 to give your path, i.e. cm_model = '/users/bwingfield/repos/migfinder/migfinder/cm_model/selection109_oriR.cm'

Re how to run it for multiple samples, I have done so by creating one dir per fastafile and running migfinder inside each one of them. If the path for cm is fixed these files could be anywhere. However, you can also run them from the same dir,  each output will have one name, the common names are folder which separate each step of the pipeline, i.e. hattci, infernal and prodigal. The resulting files will be fastaname.results.
Let me know how it goes, very happy to help :)

That's weird, I seem to get the correct path:
(mig) [bwingfield@login1(kelvin2) ~]$ python
Python 2.7.15 | packaged by conda-forge | (default, Mar  5 2020, 14:56:06)
[GCC 7.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import pkg_resources
>>> pkg_resources.resource_filename('migfinder', 'cm_model/selection109_oriR.cm')
'/users/bwingfield/repos/migfinder/migfinder/cm_model/selection109_oriR.cm'

Maybe pkgutil.get_data would be a good approach?
I'll try the latest changes you've made 😁

Perfect, with your most recent changes I can run migfinder anywhere:
(mig) [bwingfield@login1(kelvin2) test]$ pwd
/users/bwingfield/repos/test
(mig) [bwingfield@login1(kelvin2) test]$ python
Python 2.7.15 | packaged by conda-forge | (default, Mar  5 2020, 14:56:06)
[GCC 7.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import migfinder as mf
>>> mf.main("/users/bwingfield/repos/test/test.fasta")
HattCI done!
attC hits from pipeline: 7
Infernal done!
Prodigal done! Staring pos-processing...
Pos-processing done!
>>>

Thanks for your help 😁

Happy it worked! Let me know if you have questions on how to interpret the results :)


CheckDefaultPage failed in 9.4.0 RC

Description of bug
CheckDefaultPage failed in 9.4.0 RC
Steps to reproduce
List the steps to reproduce the behavior:

install Dnn Platform 9.4.0 RC and login as host;
navigate to Setting > Security > Security Analyzer in Persona bar.

Current result
the CheckDefaultPage step failed.
Expected result
all steps should pass the check.
Screenshots

Affected version

[ *] 9.4.0 release candidate

This is expected, we do not yet have updated checksums

I am closing this issues since we already have one to handle this in AdminExperience


Can duktape parse an object?

The documentation say that "Object is supported in declarations, but the type of the actual value passed must be one of the above(boolean, int, double, string and void) or null."
Is there any way to send an Object to js?

No, but you can use an interface to expose its data to JS similar to https://github.com/square/duktape-android/#example

Or serialize it as JSON and send it as a string and deserialize on the other side.

These solutions will increase a lot the complexity on Android app and on Java script side. If I have for example a class Person(firstName:String, lastName:String) it would be great to have a possibility to send it as it is to js and js to know to interpret it.

If you want to send data then serializing as JSON is likely your best bet.


winwave: add support for FLOAT sample format

This PR adds support for FLOAT sample format.
See here for a list of audio drivers and sample formats:
https://github.com/alfredh/baresip/wiki/Audio-sample-formats

hi @nicotyze do you want to review/test this patch ?
Alfred

Hi @alfredh,
The patch looks good.
I've tested from win10 with following config:

ausrc_format		float		# s16, float, ..
auplay_format		float		# s16, float, ..
auenc_format		s16		# s16, float, ..
audec_format		s16		# s16, float, ..

/auloop => OK, float format seems to be well supported
My baresip version does not support opus or mpa so I can not test if it is ok with these codecs.
But when I do a call test with g711,  FLOAT  -->  S16LE and S16LE  -->  FLOAT conversions seem to work.
Nicolas

thanks for helping @nicotyze  !


simplify snapshotting repositories

The changes here are in preparation for the e2e test. The new -snapshot flag allows us to snapshot repository contents (the same contents always output the same YAML, because we sort the hashmap items). This ability to snapshot repositories gives us a "fingerprint" of a GCR, so that we can use predefined fingerprints to check against a live repository (the test-only GCRs) during tests.
Also update dependencies.

/assign @justinsb

/cc @amy

Looks good - one question about the format, but I'm going to lgtm & hold so as not to slow this down any further.  (Feel free to unhold if you disagree / would rather address in a subsequent PR)
/lgtm
/hold

Thanks for looking at the feedback!
/lgtm

/hold cancel


feat(compilerOption): add instantiationDepthLimit and instantiationCountLimit

Fixes #34933.
I am not familiar with this project structure. This PR might be incomplete, so I hope someone contributes to complete it.
I only edited 3 files, and didn't touch Diagnostics though I referred to new fields from it.
Please let users be able to adjust instantiationDepthLimit and instantiationCountLimit by their free decisions and responsibilities, instead of hard-coded values (50 and 5000000 for each).
Currently, the limitation thresholds are fixed to 50 and 5000000.
src/compiler/checker.ts
if (
  instantiationDepth === 50 ||
  instantiationCount >= 5000000
) {
  // We have reached 50 recursive type instantiations and there is a very high likelyhood we're dealing
  // with a combination of infinite generic types that perpetually generate new type identities. We stop
  // the recursion here by yielding the error type.
  error(
    currentNode,
    Diagnostics.Type_instantiation_is_excessively_deep_and_possibly_infinite
  );
  return errorType;
}

I think compilerOptions would be great solution for these to be configurable.
{
  "compilerOptions": {
    "instantiationDepthLimit": 100,
    "instantiationCountLimit": 10000000
  }
}

Thanks :)

See also https://github.com/microsoft/TypeScript/pull/29602

@sandersn
Though it can be viewed as an implementation detail, there're high demands for configuration(See thumbs).
What's more, There're still cases the increased limit can't cover.
Thus, how about experimental configuration?
P.S. Actually I left a comment about experimental configuration on #45025, but nobody replied to this idea.

I can't stress enough that we don't think this is appropriate to expose as configuration. Today instantiation is a process with a particular memory and runtime cost that occurs during certain operations, but every aspect of that is an implementation detail that could change tomorrow, at which point the numeric limits specified at any given point in time do not translate to the same behavior any more.
This isn't a fix for the linked issue and, again, we do not consider this to be a "tunable" parameter.

Ok, so you don't consider it, but then I guess more and more "solution" will appear.
Since I am not allowed to use a patched tsc, to compile my code I had to use // @ts-ignore (see https://stackoverflow.com/a/64688342/146334) to have my complex mongodb schema built.
I don't see why it is better to mute errors (with known side effects) instead of allowing parameters tuning for users. The default value can stay in your ownership, so 99.999% of users will use that. If even in a patch version, the behavior changes, we still can adapt to it using the same, or new experimental configuration.

I have the same problem while running typeof someJsonObject where the json object has something around 3k properties.
Does anyone know at least where is the limit? Or should I go and test recursively until I find it?

The depth fix mentioned in this article resolved my problem, even when sticking to an initial depth of 1
https://www.angularfix.com/2022/01/why-am-i-getting-instantiation-is.html
I would love to know why 😄

I can't stress enough that we don't think this is appropriate to expose as configuration. Today instantiation is a process with a particular memory and runtime cost that occurs during certain operations, but every aspect of that is an implementation detail that could change tomorrow, at which point the numeric limits specified at any given point in time do not translate to the same behavior any more.
This isn't a fix for the linked issue and, again, we do not consider this to be a "tunable" parameter.

Sorry for commenting on an old post, but changing the parameters to something larger temporarily would allow the developer to determine whether their type is actually recursive or infinite


Cocoapods - Build fails with several errors in the Swift-2.0 branch

Hello folks,
I'm using Xcode 7.0 beta 3, and here's the relevant section of my Podfile:
pod 'Alamofire', :git => 'https://github.com/Alamofire/Alamofire.git', :branch => 'swift-2.0'

The build fails with several errors:
/Users/Teja/Projects/Work/CareNarrative/Pods/Alamofire/Source/ServerTrustPolicy.swift:125:30: Cannot invoke 'append' with an argument list of type '(Unmanaged<SecCertificate>)'
/Users/Teja/Projects/Work/CareNarrative/Pods/Alamofire/Source/ServerTrustPolicy.swift:167:13: Cannot invoke 'SecTrustSetPolicies' with an argument list of type '(SecTrust, [Unmanaged<SecPolicy>!])'
/Users/Teja/Projects/Work/CareNarrative/Pods/Alamofire/Source/ServerTrustPolicy.swift:173:17: Cannot invoke 'SecTrustSetPolicies' with an argument list of type '(SecTrust, [Unmanaged<SecPolicy>!])'
/Users/Teja/Projects/Work/CareNarrative/Pods/Alamofire/Source/ServerTrustPolicy.swift:204:17: Cannot invoke 'SecTrustSetPolicies' with an argument list of type '(SecTrust, [Unmanaged<SecPolicy>!])'
/Users/Teja/Projects/Work/CareNarrative/Pods/Alamofire/Source/ServerTrustPolicy.swift:253:30: Cannot invoke 'append' with an argument list of type '(Unmanaged<SecCertificate>)'
/Users/Teja/Projects/Work/CareNarrative/Pods/Alamofire/Source/ServerTrustPolicy.swift:261:62: 'Unmanaged<CFData>!' is not convertible to 'NSData'
/Users/Teja/Projects/Work/CareNarrative/Pods/Alamofire/Source/ServerTrustPolicy.swift:272:29: Cannot invoke 'publicKeyForCertificate' with an argument list of type '(Unmanaged<SecCertificate>)'
/Users/Teja/Projects/Work/CareNarrative/Pods/Alamofire/Source/ServerTrustPolicy.swift:286:35: Cannot invoke 'SecTrustCreateWithCertificates' with an argument list of type '(SecCertificate, Unmanaged<SecPolicy>!, inout SecTrust?)'
/Users/Teja/Projects/Work/CareNarrative/Pods/Alamofire/Source/ServerTrustPolicy.swift:289:25: Cannot assign a value of type 'Unmanaged<SecKey>!' to a value of type 'SecKey?'

Any thoughts?

Actually, never mind, I found that this issue has already been discussed and responded to here


Use global event to control process substitution from another process

AnyBuild client that is driving a build, decides when it is appropriate to enable or disable process substitution globally for a machine. Then substitution is disabled this allows avoiding spending time on launching the shim process only to fall back to building locally.

@narasamdya FYI


[Bug Report] Horizontal menu error.

Element UI version
2.15.6
OS/Browsers version
Microsoft Edge Version 94.0.992.50 (Official build) (64-bit)
Vue version
2.6.14
Reproduction Link
https://codepen.io/edric-li/pen/vYJKOag
Steps to reproduce
Open the console
The following error will appear
Uncaught RangeError: Maximum call stack size exceeded.
What is Expected?
No error is expected.
What is actually happening?
Actually produced an error.

Temporary solution:

this bug still work on version 2.15.7


Error when parsing variable names with spaces

Describe the bug
If variable names include spaces, CYLC cannot parse this into the workflow (spaces are not permitted).
How to reproduce
Etc. for variable "Turbulent mixing height after boundary layer", produces the following error during validation:
GraphParseError: Illegal graph node: collate_deterministic_domain_surface_histogram_series_.Turbulent mixing height after boundary layer
Expected behaviour
Could add another parser that replaces spaces with an underscore.
Environment

Version: [e.g. from cset --version]
Browser (if UI issue): [e.g. Chrome, Firefox]

This is fixed in #765, as I've added a proper task name sanitiser.

Nice! Upcoming fix in #765 so closing this.


Fix package.json properties

According to the plugin guidelines, in order to improve error messages and make the plugin discoverable:

netlify and netlify-plugin should be added to the keywords in package.json
the repository and bugs properties should be added to package.json

Closed by #44.


cmd+a doesn't work in search bar

I wanted to select all text to clear it but it wouldn't work. Perhaps b/c preventDefault is called? Idk haha. Anyway: figured feedback might be useful (:

thanks, yup need to fix this
the search bar is quite hacky with caching and state etc - ultimately I'd like to get it cleaned up to something less complex

should be fixed in v1.0.6 https://github.com/codeforscience/sciencefair/releases/tag/v1.0.6


The plugin doesn't work on Android

I have posted the full problem on stack overflow here: https://stackoverflow.com/questions/31743868/cordova-google-analytics-plugin-works-on-ios-but-breaks-android

This plugin is outdated so doesn't work anymore.
Changing to this new plugin worked for me:
https://github.com/danwilson/google-analytics-plugin