Add an & feature

Not sure if this is already implemented but i just thought it would be nice to be able to search for more than one category. Say if I wanted to to make a random encounter involving both Kobolds and Dragon Wyrms, I could search for "Kobolds & Wyrms". Then I could simply press "Random Encounter Hard" instead of having to continually search wyrm then kobold while trying to get the balance right.
You might be able to do this by adding something like this in the function SortAndFilterMonsters(monsterLib):
requested = input.split('&');
for (u = 0; u < requested.length; u++){
	new = requested[u];
	for ( i = 0; i < new.length; i++ ) {
		if ( monsterLib.checkMonster(input[i], filters) ) {
			output.push(input[i]);
		}
	}
}

I've added the ability to use regular expressions, which will allow you to do what you want (and way, way, way more).
To do a regex search, start your search with a / (otherwise it will be treated as a normal search).
To do multiple finds, just separate your terms with pipes (|). For example, to find kobolds and wryms, you could search for /kobold|wyrm
This is a full JavaScript regex, though, so there's a lot more you can do. One simple example is using \b for word boundaries, which lets you find dogs and cats without catoblepas and dogmoles also showing up: /\bcat\b|\bdog\b

This is pretty useful, thanks. It should probably be put in the FAQ so that others can find it as well.

Yeah... I posted on reddit about it in /r/asmor, /r/dnd, and /r/dndnext. I probably will add it to the FAQ at some point, but I'd prefer to have a better way of informing people and I'm not really sure what that looks like yet.

The best way to inform people, in my opinion, is to build it into the UI
somehow. A button that shows help in a dialog is the simplest. A drop-down
or dialog that assists in building the regex is more ambitious. Example of
this is Gmail's search drop down that gives a ton of power.
On Sun, Jun 3, 2018, 2:48 PM Ian Toltz<EMAIL_ADDRESS>wrote:

Yeah... I posted on reddit about it in /r/asmor, /r/dnd, and /r/dndnext. I
probably will add it to the FAQ at some point, but I'd prefer to have a
better way of informing people and I'm not really sure what that looks like
yet.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Asmor/5e-monsters/issues/127#issuecomment-394186468,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADdyRlPCVh65BD-1kwFlCN8l_YPUCMOMks5t5D2egaJpZM4USY3V
.


build-support.ocaml.dune: check for minimalOCamlVersion typo

build-support.ocaml.dune:

accept both minimalOcamlVersion and minimumOCamlVersion silently.

IMO this is the wrong fix. buildDunePackage should (quietly) accept both spellings.

IMO this is the wrong fix. buildDunePackage should (quietly) accept both spellings.

What would be the correct fix?

The good fix for minimalOCamlVersion would be to accept it in buildDunePackage.
You mean, I should replace minimumOCamlVersion with minimalOCamlVersion and then block minimumOCamlVersion?
Just to be clear. I'm not doing the right thing out of laziness but of misunderstanding what you mean.

Good! The minimalOCamlVersion attribute makes no sense to stdenv.mkDerivation. Can you please remove it from the args set before it is passed to mkDerivation? (There is a primitive removeAttrs that can be used for this purpose.)

Done. Removed only minimalOCamlVersion.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/minimumocamlversion-and-minimalocamlversion/17186/2

Good! The minimalOCamlVersion attribute makes no sense to stdenv.mkDerivation. Can you please remove it from the args set before it is passed to mkDerivation? (There is a primitive removeAttrs that can be used for this purpose.)

We should also removeAttrs for minimumOCamlVersion  ?


Add SSM Rename Test To Match Internal Pipeline Test

Description of the issue
Need to test all the functionality that was in the internal pipeline tests
Add ssm tests
Description of changes
Add ssm tests
License
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Tests
Ran in the private repo gha

Failure for metric value benchmark for rhel8 and ol9 only same as the current private repo. It passes that test on other os same as private repo.


Appends binDir to CNI_PATH

Appends binDir to CNI_PATH so that invoke.DelegateAdd/Del can find CNI plugins in alternate paths
@s1061123 can you take a look at this? I feel like this is a ham-fisted (that is, not elegant) way of doing this by explicitly setting CNI_PATH. Feels like I am doing something wrong. Do you think there's another way to approach this?
Currently, if you set binDir in the configuration, like:
{ "name": "multus-cni-network", "type": "multus", "binDir": "/opt/multus/bin/", "logLevel": "debug", "logFile": "/var/log/multus.log", "kubeconfig": "/etc/cni/net.d/multus.d/multus.kubeconfig", "delegates": [ { "name": "cbr0", "plugins": [ { "type": "flannel", "delegate": { "hairpinMode": true, "isDefaultGateway": true } }, { "type": "portmap", "capabilities": { "portMappings": true } } ] } ] }

And add plugins in that directory CNI library will complain with something like:
2019-09-17T19:37:01Z [error] Multus: error in invoke Delegate add - "foo": failed to find plugin "foo" in path [/opt/cni/bin]

Where the binDir is not noted in the final [] at the end of the Delegate Add call. This error message originates here: https://github.com/containernetworking/cni/blob/master/pkg/invoke/find.go#L42
Additionally, you can see in https://github.com/containernetworking/cni/blame/master/pkg/invoke/delegate.go#L30 that the paths that get used for the FindInPath function are directly from the CNI_PATH.

Pull Request Test Coverage Report for Build 877

9 of 9 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 73.701%

Totals

Change from base Build 876:
0.2%

Covered Lines:
950

Relevant Lines:
1289

💛  - Coveralls


Update prometheus to v2.52.0

Description of changes:
This PR updates

Update prometheus to v2.52.0
Update prometheus to v1.8.1

Upstream source of truth:
Upstream GIT_TAG: https://github.com/prometheus/prometheus/releases/tag/v2.52.0
https://github.com/prometheus/node_exporter/releases/tag/v1.8.1
Upstream HELM_GIT_TAG: https://github.com/prometheus-community/helm-charts/blob/prometheus-25.21.0/charts/prometheus/Chart.yaml
Explanations of changed patches

Fixed the merge conflicts in prometheus
removed old patch from node-exporter

Test:
Tested on vsphere cluster using
helm install prometheus oci://public.ecr.aws/g0y6o3v0/prometheus/charts/prometheus --version 2.52.0-1ce89733ecc3f0160b6b90e7c00cc3885918e1a5 --set sourceRegistry=public.ecr.aws/g0y6o3v0 --namespace observability

verified the logs and checked prometheus UI was up
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/approve


WEEK with mode selector

As week numbers starts different from country to country - mysql have added a "mode" selector to week.
https://dev.mysql.com/doc/refman/5.5/en/date-and-time-functions.html#function_week
You have hardcoded to 3 in both mysql and postgresql - would it be possible to change this value, so I could write WEEK(xxx, 1) in my dql - Right now I just get a "[Syntax Error] line 0, col 65: Error: Expected Doctrine\\ORM\\Query\\Lexer::T_CLOSE_PARENTHESIS, got ','"

Hi, @lsv
doctrine-extensions provide additional functions and compatibility layer for MySQL and PostgreSQL. Same functions return same results on different platforms.
According to ISO

The number of the ISO 8601 week-numbering week of the year. By definition, ISO weeks start on Mondays and the first week of a year contains January 4 of that year. In other words, the first Thursday of a year is in week 1 of that year.

In the ISO week-numbering system, it is possible for early-January dates to be part of the 52nd or 53rd week of the previous year, and for late-December dates to be part of the first week of the next year. For example, 2005-01-01 is part of the 53rd week of year 2004, and 2006-01-01 is part of the 52nd week of year 2005, while 2012-12-31 is part of the first week of 2013. It's recommended to use the isoyear field together with week to get consistent results.


Apply updates to factorio, and get version using --version

There's also some compatibility shim so this can run on python3.

And I only checked issues after I wrote the code. Oh well.

That's excellent, thank you -- I just updated a 0.15.10 to .13 using nothing but a path to the executable and the --experimental flag. Very nice!
If you don't mind, I'll probably edit the argument names a bit and update the README, but the patch is excellent. Look for it to be merged in the next half-hour or so.

Ok, so slightly more than a half-hour. What can I say, documentation is hard (or maybe I'm just too nitpicky about it).


remove student method for confidence_interval

Closes #49

I believe your notebook is still using the student method: https://github.com/dsaxton/resample/blob/master/notebooks/confidence_intervals.ipynb

Thanks @HDembinski

Thank you for fixing the notebook.


Fixes #106

Short Description:
We assume item_data['count'] is present but it is missing as a field if no inventory is present
Changes:
Check for item_id and count fields before trying to access them
@OpenPoGo/maintainers
Check for  count field before using

Nice catch! Do you think you could modify the title to say "Closes #106" or "Fixes #106"? Thanks!
:+1:

@MaxLeiter  Done.

:+1:


Fix ADCE to not eliminate top level DebugInfo instructions

Specifically, DebugSourceContinued, DebugCompilationUnit, and
DebugEntryPoint. These instructions are top-level instructions
which do not or may not have a user except for the tool and so
should not be eliminated.

@Keenuts I think I addressed all of your concerns. Let me know if you have any others.


Add kirkstone compatibility

Signed-off-by: Zoltán Böszörményi<EMAIL_ADDRESS>

Did you need this extra in your build?
diff --git a/recipes-devtools/python/python3-scipy_1.5.3.bb b/recipes-devtools/python/python3-scipy_1.5.3.bb
index 259ea84..44fb876 100644
--- a/recipes-devtools/python/python3-scipy_1.5.3.bb
+++ b/recipes-devtools/python/python3-scipy_1.5.3.bb
@@ -18,6 +18,7 @@ export LAPACK = "${STAGING_LIBDIR}"
 export BLAS = "${STAGING_LIBDIR}"

 export F90 = "${TARGET_PREFIX}gfortran"
+export F77 = "${TARGET_PREFIX}gfortran"
 export FARCH = "${TUNE_CCARGS}"
 # Numpy expects the LDSHARED env variable to point to a single
 # executable, but OE sets it to include some flags as well. So we split

I ran a build recently in working to upgrade our build and found it was required.

Did you need this extra in your build?
diff --git a/recipes-devtools/python/python3-scipy_1.5.3.bb b/recipes-devtools/python/python3-scipy_1.5.3.bb
index 259ea84..44fb876 100644
--- a/recipes-devtools/python/python3-scipy_1.5.3.bb
+++ b/recipes-devtools/python/python3-scipy_1.5.3.bb
@@ -18,6 +18,7 @@ export LAPACK = "${STAGING_LIBDIR}"
 export BLAS = "${STAGING_LIBDIR}"

 export F90 = "${TARGET_PREFIX}gfortran"
+export F77 = "${TARGET_PREFIX}gfortran"
 export FARCH = "${TUNE_CCARGS}"
 # Numpy expects the LDSHARED env variable to point to a single
 # executable, but OE sets it to include some flags as well. So we split

I ran a build recently in working to upgrade our build and found it was required. I think it was around the work RP did with FORTRAN on oe-core, maybe it's not in the kirkstone release but just on master.

It may still be needed, for now I haven't built the image that actually had python-scipy on it.
I use the same manifest for multiple images, though, and I just noticed that current master needs kirkstone compatibility now.

Ok thanks, I've pushed a couple of updates I had hanging around locally so let me know if you hit any further issues.


Auto-exports cannot be updated

Debug log ID
4GBBPBMR-refs-euc/6.7.143-7
What happened?
After downloading the latest BibTex, Zotero opens in a safe mode (toolkit.startup.recent_crashes count goes up 1 with every Zotero launch, so you need to reset it to launch in a normal mode). Also, when I change "My library" in Better BibTex settings, it immediately changes back after closing the settings. These issues are probably unrelated to the main problem.

Change apa7 to apa. I previously didn't use the style properly (apa7 isn't actually a style Zotero knows) but the error used to come during the refresh, now it comes during the compile. Sorry for the disruption.

Change apa7 to apa. I previously didn't use the style properly (apa7 isn't actually a style Zotero knows) but the error used to come during the refresh, now it comes during the compile. Sorry for the disruption.

Thanks, I had no idea it's so easy! However, that issue with recent_craches remains. It's not convenient resetting it every fourth launch.

when I change "My library" in Better BibTex settings, it immediately changes back after closing the settings.

That's to do with auto-export I take it? Can you install 5537 and send a new log after editing the auto-export again?
BTW I prefer to have one github issue per problem. People do use the issues as a knowledge base so it's helpful to have the title match the problem, which I also need to keep clear what issues I'm working on.

when I change "My library" in Better BibTex settings, it immediately changes back after closing the settings.

That's to do with auto-export I take it? Can you install 5537 and send a new log after editing the auto-export again?
BTW I prefer to have one github issue per problem. People do use the issues as a knowledge base so it's helpful to have the title match the problem, which I also need to keep clear what issues I'm working on.

Sorry, toolkit.startup.recent_crashes count has probably nothing to do with Better BibTex.
New log with 5537: JPDP3542-euc/<IP_ADDRESS>41.5537-7

Is the edit still failing? I see no errors in the log.

Yes, I choose the second file, close the settings, the first is still on.

And you did:

install 5537
make edit in auto-export
send a debug log?

because I'm not seeing errors but I'm also not seeing edits in JPDP3542-euc/<IP_ADDRESS>41.5537-7

And you did:
* install 5537

* make edit in auto-export

* send a debug log?

because I'm not seeing errors but I'm also not seeing edits in JPDP3542-euc/<IP_ADDRESS>41.5537-7

That's right

Oh wait, it's not the case that just the one you select is "on". All auto-exports are on, depending on the "on change" dropdown above it. Selecting an auto-export there just allows you to edit that particular auto-export, picking an auto-export from that list does nothing on its own.

If that is correct, we can move to the final sub-issue. Individual enable/disable per auto-export is coming but not right now.

Although I'm not entirely clear on what you mean by

Also, when I change "My library" in Better BibTex settings, it immediately changes back after closing the settings

There isn't a "My Library" to change there. You have two auto-exports of the same collection ("Antipsychotic Gut") for export paths (one to your home dir, one into an obsidian vault). BBT does not remember which auto-export you were viewing when you close the prefs, that is true. But I don't think you are actually making any changes there.

For the safe-mode startup problem, can you start Zotero with
zotero -ZoteroDebugText 2>&1 > ~/zotero.txt

and attach the resulting ~/zotero.txt here?

closing for inactivity.


Add multiline argument to MatchNext method?

I noticed that Match, Test, Replace, and others have a multiline input argument but MatchNext does not. So in your Example 3, to specify multiline, I need to do the following:
Dim matcherState As MatcherStateTy

matcherState.multiline = True '<-- add this line

Do While StaticRegex.MatchNext(matcherState, regex, exampleString)
   Debug.Print StaticRegex.GetCapture(matcherState, exampleString)
   Debug.Print StaticRegex.GetCaptureByName(matcherState, regex, exampleString, "year")
Loop

Is there a reason why MatchNext does not have a multiline argument?
And taking a step back, why wouldn't multiline (and perhaps localMatch) be specified once in  StaticRegex.InitializeRegex along with caseInsensitive argument?
Thanks!

Dear GCuser99,
Thank you for your suggestions and questions.

Why doesn't MatchNext have a multiline parameter? Firstly, MatchNext is intended to be used for successive matches (as in your example). The user should not be required to specify multiline (and other flags) each time MatchNext is called. Of course, we could add a ternary parameter (multiline = TRUE or FALSE or unspecified), where unspecified would mean "use the value from matcherState", but I think that would be confusing. Second, the idea behind MatchNext is to perform successive matches with the same set of flags. Adding flag parameters would encourage changing flags from call to call, which should work, but is not a use case I had in mind. The engine will soon support unbounded modifiers like (?m) (the dev branch already does), which may mitigate the problem.

Why is multiline not specified along with caseInsensitive? The information about whether the match should be caseInsensitive is needed at compile time; a case-insensitive regex has a different bytecode than a case-sensitive one. That's why caseInsentive has to be specified from the beginning.

Best regards,
sihlfall


Reference the bindata container ID as const

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
This makes the overall readability of the code more clear.
Which issue(s) this PR fixes:

None
Does this PR have test?
Refers to https://github.com/kubernetes-sigs/security-profiles-operator/pull/618#discussion_r739704268

None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

Codecov Report

Merging #634 (efa7e66) into master (b9563ca) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #634      +/-   ##
==========================================
- Coverage   43.82%   43.76%   -0.06%
==========================================
  Files          29       29
  Lines        1563     1565       +2
==========================================
  Hits          685      685
- Misses        832      834       +2
  Partials       46       46

/test pull-security-profiles-operator-test-e2e


Universal Storage II compatibility data

According to the forum thread, the current version (<IP_ADDRESS>) is listed as being compatible with KSP 1.4.x upwards, whereas CKAN only has 1.6.0 compatibility noted. The same might also be true for the intermediate 1.5.x.y mod versions.

Is there a bug tracker for this mod? The version file which we would otherwise use for this is out of date and would produce bad data in CKAN (note that VERSION says <IP_ADDRESS> in the download for <IP_ADDRESS>):
{
"NAME":"Universal Storage 2","URL":"http://ksp-avc.cybutek.net/version.php?id=591",
"DOWNLOAD":"https://spacedock.info/mod/1933/Universal%20Storage%20II/download/<IP_ADDRESS>",
"CHANGE_LOG_URL":"https://spacedock.info/mod/1933/Universal%20Storage%20II",
"VERSION":{"MAJOR":1,"MINOR":5,"PATCH":1,"BUILD":9},
"KSP_VERSION":{"MAJOR":1,"MINOR":6,"PATCH":0},
"KSP_VERSION_MIN":{"MAJOR":1,"MINOR":4,"PATCH":5},
"KSP_VERSION_MAX":{"MAJOR":1,"MINOR":6,"PATCH":99}
}

If the author can get that fixed, we could start using this data.

Hmm, trying it out in #7005, the metadata actually turns out OK, but it's still not super safe to knowingly use partially wrong data. Not sure how to proceed.

Thank you.

Thanks for letting us know about this!


Only one person should be in "control"

Currently, if anyone clicks anything, it changes the view for everyone and loses current state (such as votes)
Only one person should be in control of a session, such as determining what story is being estimated, and clicking reveal, and everyone else is just allowed to vote.

Hi as @hangy said, turn Security on and it will work.


Update Gilbert's Portfolio

Codecov Report

Merging #260 (07195fe) into master (c6a50c1) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #260   +/-   ##
=========================================
  Coverage     73.80%   73.80%
  Complexity      669      669
=========================================
  Files            96       96
  Lines          2111     2111
  Branches        260      260
=========================================
  Hits           1558     1558
  Misses          473      473
  Partials         80       80

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 c6a50c1...07195fe. Read the comment docs.


Units on displayed body data

Displayed values should include units
radius/position: meters (m)
mass: kilograms (kg)
velocity: meters/second (m/s)

Notes from Aaron:

Distance unit options: m, km, AU (astronomical units)
Mass unit options: kg, Mg (megagrams)
Time unit options: seconds, days, years


Export number in CSV in wrong format

I display my number with a comma as the thousand separator but when I download my dashboard in CSV, I got a dot instead.
Expected results
When I download my dashboard in CSV I got comma as thousand separator.
Actual results
In my csv I dot a dot as thousand separator
Screenshots
If applicable, add screenshots to help explain your problem.

Hi @aehanno, can  you please provide a screenshot of where you are setting the thousand separator? It would help me to debug and identify faster

Hi @aehanno, can you please provide a screenshot of where you are setting the thousand separator? It would help me to debug and identify faster

Hi @Govindarajan-D , there is screenshot of our setting

H @aehanno
I tried the below and I get a dot only at the end. Is this the one you described?

Hi @Govindarajan-D
Yes, it's an issue for us because we can´t exploit this information as is a string and not in the same format
That's a bug for you ?

Yes its a bug. Let me check what I can do for this.

Hi @Govindarajan-D, thank you and I come back to you to know if you have find a solution to the probleme.
And if it will be possible to have a correction ?

Hi @aehanno, sorry for the delay in reply. I couldn't figure out the issue. Can you try to assign someone on this?

@Govindarajan-D thanks for the help, do you know How I can find maybe someone who can help on that ?

Is anyone here still planning to work on this? I'm tempted to close it as stale. The initial report also predates supported versions of Superset, so please let us know if you're still facing this in Superset 3.1/4.0


policy sync may require reboot

under certain conditions that impact some users, the messaging and meeting policies will not sync, and thus after every policy change, they should advise impacted users to reboot
impacts all tenants
fix ETA maybe February 2022
see ICM https://portal.microsofticm.com/imp/v3/incidents/details/270279237/home

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

ID: 813bcae6-1c45-1d12-6652-de7d0986edfa
Version Independent ID: fe0b1991-f210-bf08-850a-487f4c9b1b8e
Content: Manage meeting policies in Microsoft Teams - Microsoft Teams
Content Source: Teams/meeting-policies-overview.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

Hi @DanStevensonMSFT, than you for your feedback.
If you have a Microsoft alias, you can link your accounts in GitHub so that a blue "Microsoft submitter" label appears to any PR's or issue you submit. Guidance on it here: https://docs.opensource.microsoft.com/tools/github/index.html
Here is the site you can use to see the mappings between GitHub and Microsoft aliases: https://repos.opensource.microsoft.com/people?q=DanStevensonMSFT

@CarolynRowe, I do not have access to the link. Could you kindly help me to confirm the information there? Thank you!

Hi @dariomws:  Are you asking me to check the links?  Yes, both worked for me.  Although the second did not find a search result for danStevenson...

Hi @CarolynRowe,
I'm asking for the following link:
https://portal.microsofticm.com/imp/v3/incidents/details/270279237/home
I want to confirm this information and maybe we should (I can do it) add a note to this article with the information that Dan is sharing.
Thanks.

@DanStevensonMSFT
Hi, Dan,
Thank you for your submission.
Because you are an internal Microsoft contributor, we recommend finishing your GitHub setup:

Set up your accounts

And then you can submit a PR in the private repo to add the information:

Quickly update an article by using GitHub.com

As mentioned at the end of the article, @ mention the article author so that she can approve your changes.
Thanks!


Bound methods cause surprising JIT cache misses

Description
Reproducer:
import jax

class Foo:

  def __init__(self):
    self.f_traced = False

  def f(self):
    print('f')
    assert not self.f_traced
    self.f_traced = True

foo = Foo()
jax.jit(foo.f).lower()
jax.jit(foo.f).lower()  # Fails with assertion error.

I put some more details in this colab notebook.
The root cause of the issue is that the bound method instance changes between invocations of foo.f, and JAX's jit cache relies on object identity.
It would be useful if jit could special case types.MethodType and treat this as a weakref pair of the unbound method and the instance.
For now I would recommend the following workaround for users:
class Foo:
  def __init__(self):
    self.f_traced = False
    self.f = self.f  # Ensure `jit(self.f)` sees the same bound method instance.

  def f(self):
    print('f')
    assert not self.f_traced
    self.f_traced = True

foo = Foo()
jax.jit(foo.f).lower()
jax.jit(foo.f).lower()  # Does not fail.

What jax/jaxlib version are you using?
HEAD
Which accelerator(s) are you using?
CPU
Additional system info?
No response
NVIDIA GPU info
No response

Thanks for the report Tom – this is a known issue, and is due to a quirk of how Python implements bound method calls. See e.g. #9672 and #15338 for previous discussions.
The best work-around would be to avoid jit-compilation of bound methods.

Jake, we discussed this with Matt yesterday and decided that it's worth sketching a fix for this. My gut feeling is that it isn't very invasive from the JAX impl side, nor surprising ot users. We could emit a good error message if self is not hashable. Wdyt?


Vendor org.graalvm.collections

Motivation:
Package org.graalvm.collections is used throughout the Pkl codebase. It is most heavily used in the Truffle interpreter, which requires putting most collection methods behind @TruffleBoundary. At the moment, this is done by wrapping collection methods with static methods declared in classes EconomicMaps and EconomicSets. However, static wrapper methods are inconvenient to use, decrease code readability, add some overhead, and are easy to forget about.
Changes:

vendor package org.graalvm.collections into org.pkl.core.collection

org.graalvm.collections is licensed under: The Universal Permissive License (UPL), Version 1.0
vendored version: https://github.com/oracle/graal/tree/graal-23.0.3/sdk/src/org.graalvm.collections/src/org/graalvm/collections
add package-info.java with original license
annotate most public methods with @TruffleBoundary
add @Nullable annotations

switching to JSpecify will enable more accurate nullability checks

remove prefix tree classes (not used in Pkl)
make no other code changes to simplify review/updates

inline and remove EconomicMaps/Sets wrapper methods
replace usages of EconomicMaps.equals/hashCode with EconomicMapUtil.equals/hashCode
fix ProjectDeps.hashCode()

Result:
Cleaner, safer, and more efficient code.

I am not sure we want to do this. This makes it hard to upgrade the collections library, and I'm not sure how much of a performance gain we get out of this.

and are easy to forget about

Thankfully, our CI builds (on main) will catch these errors if they are forgotten.
Curious about @holzensp and @stackoverflow's thoughts

Owning the collection library(s) used by a Truffle language implementation makes a lot of sense:

it is desirable for performance reasons (can optimize for partial evaluation, etc.)
it avoids ugly wrapper code
EconomicMap is very mature and hasn't had significant changes in years. Vendoring it is the logical choice.

I just stumbled upon the following conversation in the GraalVM Slack, where a Truffle committer argues for owning the collection library used in the Pkl interpreter. He's talking about Paguro, but it's the same thing, except that Paguro, or its future replacement, is way more challenging to own than EconomicMap(Impl).

Christian Humer Apr 5th at 7:36 AM
Looking at the Pkl types a bit. I think RrbTree could be a good candidate to make partial evaluatable. Atm it looks as if there all just boundaries, which hinders optimizations. Do you have any experience trying this?

yeah you need to manually profile and specialize for recursions and things like that. that typically involves owning the library you are using.

I also see the benefits (and the low risk, due to the stability of this library). It would require a further update to NOTICE.txt, btw.

It would require a further update to NOTICE.txt, btw.

done


How to know if a semaphore is signaled?

I am doing a port of LevelDB and I need to redesign the function WaitForSingleObject(sem1_, INFINITE);
The only thing I need is to know when a ThreadX semaphore is signaled in order to continue the execution and stop waiting.

Something like this: https://docs.microsoft.com/en-us/azure/rtos/threadx/chapter4#tx_semaphore_put_notify


Plugin for tracking burner times in Player Owned Houses

Looking to get a new feature for runelite. I currently have to use a web timer with a loop feature to keep track of my burners in the player owned houses. At the end, it makes a sound. Someone suggested that I suggest this to help with hosting houses and guilded altars.
Anyway, it's based on firemaking level. At 30, its 2 minute burn time, and 99 is 4 minutes.
Not sure if theres a way to track the current status or from the lantern be able to see its about to go out, but if there was a way to determine it, it would be a great plugin. I wouldn't care if there wasn't a sound, even just a clock.
Thank you

Duplicate #4679
Also there's a PR open for this #5179


Fix for timeout upon no results, highlighting with adjacent punctuation

Fix for https://github.com/swirlai/swirl-search/issues/77, DS-240

Note: this fixes timeout in RequestsGet only, due to mishandling of an empty response ([])


merge from konsum-frontend-svelte,template-arlac77-github,template-svelte-app

package.json

chore(deps): add ^10.12.2 remove ^10.12.1 (devDependencies.npm-pkgbuild)

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

npm package (@latest dist-tag)
v1.32.0
GitHub release

Your semantic-release bot :package::rocket:


chore: update driver adapter READMEs

Closes https://github.com/prisma/team-orm/issues/473

[x] PlanetScale
[x] Neon
[x] Turso

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

General question: this might get out of sync with docs regularly, keeping the READMEs tiny and pointing to docs for instructions would work better?

I think the PlanetScale and Neon adapter READMEs should be fine. We can trim down the content on the libSQL adapter README. WDYT?  🤔

General question: this might get out of sync with docs regularly, keeping the READMEs tiny and pointing to docs for instructions would work better?

I think the PlanetScale and Neon adapter READMEs should be fine. We can trim down the content on the libSQL adapter README. WDYT? 🤔

Just asking, I thought we were more on a single source of truth approach in the past to avoid extra work to keep things in sync.


require() of ES Module

i got new issue when start project.
Nestjs 9
"@squareboat/nest-console": "^1.1.0",
node_modules/@squareboat/nest-console/dist/inquirer.js:16
const inquirer_1 = __importDefault(require("inquirer"));
                                   ^
Error [ERR_REQUIRE_ESM]: require() of ES Module<EMAIL_ADDRESS>

*Update: it working on version 1.0.9
maybe that is a problem of new version

Hello @tidusvn05, due to nestjs not giving out complete support for ESM Module, I have downgraded the inquirer and ora package to their previous commonjs versions. Please feel free to use v1.1.1 from npm.


can not run properly after build

apps run very well on dev mode, but can not serve as production
cause me 404 error on http://localhost:3000/subapp1/ which port 3000 is my main app
这个代码build之后不能成功运行，会有404错误。如果你那边可以运行，麻烦在package.json中添加script脚本防止其他人运行错误

Resolved


On Windows 10 not generating pages for Templates on build or develop mode

Description
Using Gridsome on Windows 10 with the Sanity source https://gridsome.org/plugins/gridsome-source-sanity and their demo Gridsome project https://github.com/sanity-io/sanity-template-gridsome-blog
Routes/pages are not being generated for the values of the Template's Component on Windows 10.
Appears to be due to issue with comparison of mixed direction /
At https://github.com/gridsome/gridsome/blob/91225f1616dbdb349b6b2a1b45e6357aa508332e/gridsome/lib/plugins/TemplatesPlugin.js#L284 the comparison is failing since component is of form /sanity-gridsome-blog/web/src/templates/SanityPost.vue while byComponent has paths in the form sanity-gridsome-blog\web\src\templates\SanityPost.vue.
Build works and pages / routes are created on deploy to Netlify.
Steps to reproduce
Run gridsome develop or gridsome build no paths are created for the values of the Collection for the Template SanityPost.vue (see https://github.com/sanity-io/sanity-template-gridsome-blog/tree/master/template/web/src/templates)
Expected result
It should generate a route for each of the posts based on their slug e.g. http://localhost:8080/my-first-post
Actual result
It is not generating routes for each post resulting in a 404 when request a route for an existing post e.g. http://localhost:8080/my-first-post
Environment
System:
OS: Windows 10 10.0.18362
CPU: (4) x64 Intel(R) Core(TM) i5-6500 CPU @ 3.20GHz
Binaries:
Node: 10.16.0 - C:\Program Files\nodejs\node.EXE
npm: 6.9.0 - C:\Program Files\nodejs\npm.CMD

I think this is related #753

@spocock Any luck on this? Were you able to figure this out?

@CraigHead Sorry, not worked on this for some time but looking back I don't see that I worked around this, and then I ended up (for other reasons) moving to using a different implementation.


loading python 3 kernel failed when ADS restarts

Open a large notebook (e.g. our DRI report notebook)
Restart ADS

Expected
the notebook is loaded successfully.
Actual

No, that was a fix for a different issue I found while testing to find this bug. Working on the fix for this issue now.

Gotcha, thanks! Just double checking 😄


Endereum - ClimateOrb Project

项目名：
Endereum x ClimateOrb - Blockchain architecture to create and track climate action initiatives engaging various participants
项目仓库：
Canada
使用到的开发工具或资源 :
Blockchain, IoT, Ethereum, Web3, platon-truffle, IPFS
计划完成时间：
2022 March
预计交付内容：
Hardware:  Prida - Air Quality Monitoring Device to Monitor indoor air quality and environmental indicators to take control of what we breathe and how to make it better
Blockchain Platform: Platform aims to create control and data level for the IoT device distributed storage system for storing climate data
联系方式：
Hellen Ma +1<PHONE_NUMBER> (Canada)
LAT地址（PlatON主网LAT账户地址，用于接收奖励）：
lat10zj87c54e2qpyy9qh4g3ew69nnprehkqfvv7mk

@Shachindra
How is the project progressing?
Reply to Issue to submit progress and submit PR deliverables


Can't trigger on click, getting [object Object] for content

Hi, hope someone can help me please. I can get the hover working, but I can't get this to work using multiple options with an object to enable trigger: 'click'
I am trying to combine v-tooltip with v-clipboard:copy - which could make for a great integration of packages btw. When the user clicks a button, the text on :copy is copied to the keyboard, and the user should get a tooltip.
In my app.js file,
import VueClipboard from 'vue-clipboard2';
Vue.use(VueClipboard); // this works.

import VTooltip from 'v-tooltip'
Vue.directive('v-tooltip', VTooltip); // default hover works for me but breaks down when I pass an object into v-tooltip instead of =" 'Text' "

// component
<button v-clipboard:copy="item.url"
             v-tooltip="{ content: 'Copied', trigger: 'click' }"
>Copy URL</button>

I am also importing all of the styling in the .vue file where I am implementing this action.
With this approach I get the following error

Click does not work
Text still shows on hover
[object Object] is shown instead of the value of content: 'Copied'

What am I doing wrong?  How can an object be passed and triggered on click?
Many thanks.

Can you replicate in a jsfiddle or something? I tried in a project of mine but everything works fine

I cannot reproduce.

Hi, hope someone can help me please. I can get the hover working, but I can't get this to work using multiple options with an object to enable trigger: 'click'
I am trying to combine v-tooltip with v-clipboard:copy - which could make for a great integration of packages btw. When the user clicks a button, the text on :copy is copied to the keyboard, and the user should get a tooltip.
In my app.js file,
import VueClipboard from 'vue-clipboard2';
Vue.use(VueClipboard); // this works.

import VTooltip from 'v-tooltip'
Vue.directive('v-tooltip', VTooltip); // default hover works for me but breaks down when I pass an object into v-tooltip instead of =" 'Text' "

// component
<button v-clipboard:copy="item.url"
             v-tooltip="{ content: 'Copied', trigger: 'click' }"
>Copy URL</button>

I am also importing all of the styling in the .vue file where I am implementing this action.
With this approach I get the following error

Click does not work
Text still shows on hover
[object Object] is shown instead of the value of content: 'Copied'

What am I doing wrong? How can an object be passed and triggered on click?
Many thanks.

How did you fix it?


kernelCTF: add CVE-2023-0461_mitigation

Hey!
Sorry for the late response. We created a Github Actions job to verify the submission PRs.
You can view the result of the check here: https://github.com/google/security-research/actions/runs/5962545386
It failed with two issues:

It could not build the exploit because libkeyutils-dev is missing from the system. You can resolve this by adding a prerequisites target into your Makefile:

prerequisites:
	sudo apt-get install libkeyutils-dev

The exploit reproduction is also failed. Most of the cases, the exploit crashed (segfault). The reproduction system is a bit different than the live one (it runs the exploit directly from /init and there is no nsjail).
Could you please take a look why does it fail and if possible then fix it or give feedback how should we modify our reproduction procedure.

Feel free to modify the PR and the Github Action will run again and you will see the new verification results.
Thank you for your submission and participating in kernelCTF!

Hey!
Looking at logs, nine times out of ten the rop-chain worked, but the exploit segfaulted after returning to user space.
segfault at df2cff8 ip<PHONE_NUMBER>4021ca sp 000000000df2d000 error 6 in exploit[401000+9d000] likely on CPU 0 (core 0, socket 0)

It turned out that it was crashing at the beginning of the getroot() function (0x4021ca) trying to push rbp on the mmapped stack (0xdf2d000).
gef➤  x/100i 0x4021ca
   0x4021ca:    push   rbp # <--
   0x4021cb:    mov    rbp,rsp
   0x4021ce:    sub    rsp,0x20
   0x4021d2:    lea    rax,[rip+0x9bf33]        # 0x49e10c
   0x4021d9:    mov    QWORD PTR [rbp-0x20],rax
   0x4021dd:    lea    rax,[rip+0x9bf32]        # 0x49e116
   ...

Initializing the stack memory before hijacking control flow solved the problem (tested locally using the rootfs_repro_v1.img.gz image).
This is weird because it never happened on the remote instance or in my local environment.
I updated the exploit source code, please let me know if I need to update the exploit binary as well.


BUILD files in client-go repository

I am wondering if Bazel BUILD files in this repository serve any particular reason (e.g. does the CI etc build/test the repository and uses Bazel for it)?
If not, having BUILD files might actually be confusing. Especially in examples/*/ directories, users probably will click to something they don't necessarily understand. I personally don’t know Bazel and seeing that doesn’t help me with anything.
I would recommend removing BUILD files while/after mirroring if possible.

Anyone has opinions about this? tl;dr I think we should not have BUILD files in this repository, especially in examples/*

I think these BUILD files actually work if someone is use Bazel to build client-go. Since the kubernetes main repo is using bazel as its build system, so i think it's not a bad idea to enable bazel for client-go. It could use more documentation for sure.

@caesarxuchao do you anticipate people running examples/ to use these BUILD files? I think they're just causing confusion. examples/ is not being vendored by most people and does not compile as people do not take dependency on examples/... packages. So Bazel doesn't apply to examples I think.

Yeah, we can remove BUILD files from examples/. Not sure where/how to do it elegantly. It would be most reliable we can remove the BUILD files from kubernetes/staging/client-go/examples.

We can probably add a flag to https://github.com/kubernetes/test-infra/blob/master/mungegithub/mungers/publish_scripts/publish_template.sh or https://github.com/kubernetes/test-infra/blob/master/mungegithub/mungers/publish_scripts/publish_client_go.sh.
I'm not familiar with these, I need to look into them to see if they directly take commits, or do any sort of git commit. If so, we might be able to delete BUILD files in examples/ and amend the commit.

It's too risky to change the robot. The robot runs git filter-branch and git cherry-pick to pick up upstream changes, removing contents there might break the cherry-picks. Let's try to remove BUILD files from the source.

Understood. I am not familiar enough with Bazel to answer if we can remove these BUILD files directly on the kubernetes/kubernetes repo. Let's keep this issue open until we have a better answer.

Ok. I hope we can develop in client-go soon. The publishing process puts many restrictions on client-go.

Are the BUILD files causing any actual problem?
On Mon, Jun 5, 2017 at 11:22 AM, Chao Xu<EMAIL_ADDRESS>wrote:

Ok. I hope we can develop in client-go soon. The publishing process puts
many restrictions on client-go.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/client-go/issues/189#issuecomment-306264512,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAnglmLZCfNSgSXWySGoV0lv0Pzyi_L9ks5sBEdVgaJpZM4NZh7W
.

No, just confusion point for beginners.

Please keep them. They are needed for those who use Bazel.

/remove-lifecycle rotten

This is not a big dela.
/close


Add a new command to run preflight in an unshare namespace

Introduces a new command: 'preflight check run'. This command will inspect
the environment for specific values, and exit if they are not found.
Values are:
PREFLIGHT_EXEC_CHECK - The name of a check as given by check.Name()
PREFLIGHT_EXEC_IMAGE - The image under test
What this gives us is the ability to run a test within a podman
namespace, via the unshare command.
This will allow a container to be created, mounted, have the filesystem
directly operated upon, then unmounted and removed, all within the context
of podman unshare.
HasNoProhibitedPackages has been modified to utilize this behavior, in order
to operator on the RPM db directly, without the need for the 'rpm' executable,
which happens to not be available in either a scratch or microubi-based
image.
Fixes #104
Signed-off-by: Brad P. Crochet<EMAIL_ADDRESS>

/hold

I think I understand the problem we are seeking to solve.  It feels like the preflight check run subcommand is  a bit clunky.  I think an optional flag might be better if we need to the user to tell us.   Is this something that could be detected automatically without the user needing to tell us anything?  Is ubi-micro the only base container that we need to solve for.  Initial thoughts...still looking this over.

I think I understand the problem we are seeking to solve. It feels like the preflight check run subcommand is a bit clunky. I think an optional flag might be better if we need to the user to tell us. Is this something that could be detected automatically without the user needing to tell us anything? Is ubi-micro the only base container that we need to solve for. Initial thoughts...still looking this over.

This is a completely hidden command. This is not meant for user consumption. This code does detect when it should be run. While the first use case is for ubi-micro, there are others where having access to the mounted filesystem will benefit. Like in #124 . And for generating the bundle checksum.

/hold cancel

/lgtm
While I prefer not to use the hidden command chain, I do understand that it solves the problem of access to the container filesystem without having to muck around with running and copying things out.
All that said, this appears to work as I expect. From that perspective, this LGTM.

I am sympathetic to the aversion of the hidden command chain, in the short-term it solve a problem until we can rely on native golang libraries. Thanks for all the reviews.


Removing Carthage support

Removing Carthage support, because it was half-broken anyway and nobody is maintaining it.
Reaction to #94.

Result of Integration 1
Duration: 54 seconds
Result: All 69 tests passed, but please fix 1 warning.
Test Coverage: 57%.

Result of Integration 2
Duration: 46 seconds
Result: All 69 tests passed, but please fix 1 warning.
Test Coverage: 57%.


Add comment to code for better understanding

Contributed for Hacktoberfest 2021

Great 🎉 @Drock2001
Do checkout my other repositories, if you like them then show some ❤ by starring the repositories.

Sure Umesh
Thanks for merging my request
On Sat, Oct 2, 2021, 11:43 AM Umesh Singh @.***> wrote:

Great 🎉 @Drock2001 https://github.com/Drock2001
Do checkout my other repositories, if you like them then show some ❤ by
starring the repositories.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Umesh-01/Python_Basics/pull/4#issuecomment-932691179,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOGHFL5F5GZRVEUK25CIRV3UE2PIJANCNFSM5FFJYEGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Uncaught db connection exception

Hi. Can't catch db connection error.
//I use this code in index.php
try {
	// Run the application!
	Zend\Mvc\Application::init(require 'config/application.config.php')->run();
}
catch(\Exception $exception) {
	exit($exception->getMessage());
}

//When the mysql server is not responding, it shows this error
Error message:
Connect Error: SQLSTATE[HY000] [2002].

Fatal error: Zend\Db\Adapter\Exception\RuntimeException:
Connect Error: SQLSTATE[HY000] [2002]. in
C:\xampp\htdocs\zf2\vendor\zendframework\zendframework\library\Zend\View\Helper\Navigation\AbstractHelper.php on line 170

Did you try to catch PDOException ?

It seems weird to me that the exception is not being caught. Did you try catching near where it is being raised?

@mpalourdio  Yes I tried to catch a pdoexception, but failed

@Ocramius I also tried
'service_manager' => array(
	'factories' => array(
		'Zend\Db\Adapter\Adapter' => function ($serviceManager) {
			try {
				$adapterFactory = new Zend\Db\Adapter\AdapterServiceFactory();
				$adapter = $adapterFactory->createService($serviceManager);
				\Zend\Db\TableGateway\Feature\GlobalAdapterFeature::setStaticAdapter($adapter);

				return $adapter;
			}
			catch(\Exception $exception) {
				exit($exception->getMessage());
			}
		}
	),
),

I found the cause of the error.
I call in the layout ViewHelper getName ()
public function getViewHelperConfig()
{
	return array(
		'invokables' => array(
			'name' => 'Account\View\Helper\Name',
		),
	);
}

class Name extends AbstractHelper implements ServiceLocatorAwareInterface
{
	/.../
    public function __invoke()
    {
        $helperPluginManager = $this->getServiceLocator();
        $serviceManager = $helperPluginManager->getServiceLocator();

        $entityManager = $serviceManager->get('Account\Entity\AccountMapper');
        $account = $entityManager->getName();

        if (!$account) {
            return null;
        }

        $output = sprintf("%s %s", $account->getFirstName(), $account->getLastName());
        return $output;
    }
}

Fatal error: Zend\Db\Adapter\Exception\RuntimeException: Connect Error: SQLSTATE[HY000] [2002] ... in C:\xampp\htdocs\zf2\vendor\zendframework\zendframework\library\Zend\View\Helper\Navigation\AbstractHelper.php on line 170

ZF2 not catch this error.
How to catch?

See this article from @samsonasik: https://samsonasik.wordpress.com/2013/06/02/zend-framework-2-handling-db-connection-error/

Thanks. Close this issue


Library list filtered by author

Clicking on the author in a collection item view will open a filtered library page of that author's books.

@kevinglover I've made the author names into links.  Do you want to tweak the link style?

@rlskoeser Nice! I'll work on this real quick

@rlskoeser This won't preserve the orderBy param. Is that ok?

@kevinglover yes, I think that's fine

rollover status of author names: add the same underline as book title

the underline style doesn't seem to be live on the current test site.


NUTCH-2314 indexer-elastic2 plugin is used for javadoc and eclipse targets

Can you also add it to https://github.com/apache/nutch/blob/2.x/default.properties#L160
Thanks

@lewismc I've updated the PR.


can't find my env var on VM after deployment

deployed a VM with env variable:
env_vars {
      key = "TEST_VAR"
      value = "this value for test"
    }

and try to ssh VM and printenv or echo $TEST_VAR but not found
main.tf: https://gist.github.com/mhost39/b692447d30f731a0af569de55c74fe75

This looks like a dropbear thing. The image you are using is using dropbear, a lightweight ssh server, instead of openssh. The environment variables are stored correctly in /etc/environment but dropbear doesn't pick them up,
I tried with a different image which uses openssh and the environment variables appeared in the ssh session.

it worked fine with anther image uses openssh

@muhamadazmy this means our official base isn't working okay


[box plot] rows pill shows aggregated number; row limit can be hit silently

Explore's pill indicator of the number of result rows is showing wrong numbers for Box Plots.
Instead of the number of rows returned from the DB query it displays the number of aggregated rows.
This way, users have no clue if the row limit kicked in and the box plot is based on incomplete data.
Furthermore, the row limit is...

not adjustable for this plot
not communicated to the user
applied without imposing any sort order, thus resulting in a potentially arbitrary sample of rows being evaluated by the box plot

How to reproduce the bug

Use a dataset that has more rows than the configured ROW_LIMIT
Create a box plot on a numeric column, distribute across a column with a high cardinality, e.g., the primary key of the dataset.
Run
Observe the number of rows displayed in the indicator pill

Expected results
Minimal:

The actual number of rows returned by the DB query is shown in the indicator.
If the ROW_LIMIT is hit, the indicator turns red, like it does with all the other visualizations.

Ideal, additionally:

The row limit is configurable, maybe even disabled, like for the histogram, iirc
If a row limit is applied, some sensible ORDER BY should be applied in order to at least yield deterministic, if incomplete, results.

Actual results

Only the number of aggregated rows are shown, equalling the number of series displayed in the chart.
This number is consistent with the Data table below the chart, which also only shows one row per series (box).

Screenshots/Screencasts
The row count pill shows only 7 result rows:

The Data table lists these aggregated rows of the 7 distinct series:

The query applies the configured ROW_LIMIT of 100000:

A Big Number that calculates the distinct count of entities that the box plot was distributed across proves that the ccardinality is much higher than the ROW_LIMIT and therefore the boxplot was based on incomplete data witghout the row count pill turning red:

Environment

browser type and version: Chrome 93.0.4577.63
superset version: 1.3.1, installed via pip
python version: 3.8.11
node.js version: v4.6.1
feature flags active:

    "THUMBNAILS": True,
    "ALERT_REPORTS": True,
    "ALERTS_ATTACH_REPORTS": True,
    "SQLLAB_BACKEND_PERSISTENCE": True,
    "ENABLE_TEMPLATE_PROCESSING": True,
    "DASHBOARD_NATIVE_FILTERS": True,
    "DASHBOARD_CROSS_FILTERS": True,
    "DASHBOARD_NATIVE_FILTERS_SET": True,
    "ENABLE_EXPLORE_DRAG_AND_DROP": True,
    "DASHBOARD_CACHE": True

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ x] I have reproduced the issue with at least the latest released version of superset.
[ x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
´pip freeze´
aiohttp==3.7.4.post0
alembic==1.7.3
amqp==2.6.1
apache-superset==1.3.1
apispec==3.3.2
asn1crypto==1.4.0
async-timeout==3.0.1
attrs==21.2.0
azure-common==1.1.27
azure-core==1.18.0
azure-storage-blob==12.9.0
Babel==2.9.1
backoff==1.11.1
billiard==<IP_ADDRESS>
bleach==3.3.1
boto3==1.18.51
botocore==1.21.51
Brotli==1.0.9
cachelib==0.1.1
cachetools==4.2.4
celery==4.4.7
certifi==2021.5.30
cffi==1.14.6
chardet==4.0.0
charset-normalizer==2.0.6
click==7.1.2
cmdstanpy==0.9.68
colorama==0.4.4
convertdate==2.3.2
cron-descriptor==1.2.24
croniter==1.0.15
cryptography==3.4.8
cx-Oracle==8.2.1
cycler==0.10.0
Cython==0.29.24
defusedxml==0.7.1
deprecation==2.1.0
dnspython==2.1.0
elasticsearch==7.13.4
elasticsearch-dbapi==0.2.6
email-validator==1.1.3
ephem==4.1
et-xmlfile==1.1.0
Flask==1.1.4
Flask-AppBuilder==3.3.3
Flask-Babel==1.0.0
Flask-Caching==1.10.1
Flask-Compress==1.10.1
Flask-JWT-Extended==3.25.1
Flask-Login==0.4.1
Flask-Migrate==3.1.0
Flask-OpenID==1.3.0
Flask-SQLAlchemy==2.5.1
flask-talisman==0.8.1
Flask-WTF==0.14.3
future==0.18.2
geographiclib==1.52
geopy==2.2.0
gevent==21.8.0
google-api-core==2.0.1
google-auth==2.2.1
google-cloud-bigquery==2.27.1
google-cloud-core==2.0.0
google-crc32c==1.2.0
google-resumable-media==2.0.3
googleapis-common-protos==1.53.0
graphlib-backport==1.0.3
greenlet==1.1.2
grpcio==1.41.0
gunicorn==20.0.4
hdbcli==2.10.13
holidays==0.10.3
humanize==3.11.0
idna==3.2
importlib-resources==5.2.2
isodate==0.6.0
itsdangerous==1.1.0
Jinja2==2.11.3
jmespath==0.10.0
jsonschema==3.2.0
kiwisolver==1.3.2
kombu==4.6.11
korean-lunar-calendar==0.2.1
LunarCalendar==0.0.9
Mako==1.1.5
Markdown==3.3.4
MarkupSafe==2.0.1
marshmallow==3.13.0
marshmallow-enum==1.5.1
marshmallow-sqlalchemy==0.23.1
matplotlib==3.4.3
msgpack==1.0.2
msrest==0.6.21
multidict==5.1.0
numpy==1.21.2
oauthlib==3.1.1
openpyxl==3.0.9
oscrypto==1.2.1
packaging==21.0
pandas==1.2.5
parsedatetime==2.6
pgsanity==0.2.9
Pillow==8.3.2
polyline==1.4.0
prison==0.2.1
prophet==1.0
proto-plus==1.19.2
protobuf==3.18.0
psycopg2-binary==2.8.6
pyarrow==4.0.1
pyasn1==0.4.8
pyasn1-modules==0.2.8
pybigquery==0.10.2
pycparser==2.20
pycryptodomex==3.10.4
pyhdb==0.3.4
PyJWT==1.7.1
PyMeeus==0.5.11
pyOpenSSL==20.0.1
pyparsing==2.4.7
pyrsistent==0.18.0
pystan==<IP_ADDRESS>
python-dateutil==2.8.2
python-dotenv==0.19.0
python-geohash==0.8.5
python-ldap==3.3.1
python3-openid==3.2.0
pytz==2021.1
PyYAML==5.4.1
redis==3.5.3
requests==2.26.0
requests-oauthlib==1.3.0
rsa==4.7.2
s3transfer==0.5.0
selenium==3.141.0
setuptools-git==1.2
simplejson==3.17.5
six==1.16.0
slackclient==2.5.0
snowflake-connector-python==2.6.2
snowflake-sqlalchemy==1.2.4
SQLAlchemy==1.3.24
sqlalchemy-hana==0.5.0
SQLAlchemy-Utils==0.36.8
sqlparse==0.3.0
tabulate==0.8.9
tqdm==4.62.3
typing-extensions==<IP_ADDRESS>
ujson==4.2.0
urllib3==1.26.7
vine==1.3.0
webencodings==0.5.1
Werkzeug==1.0.1
WTForms==2.3.3
WTForms-JSON==0.3.3
xlrd==2.0.1
yarl==1.6.3
zipp==3.6.0
zope.event==4.5.0
zope.interface==5.4.0

This issue should not be closed by the stale bot, in my eyes.

bump

bump

Sorry, I should have rephrased it to "please don't close, as the issue is still existing in the most recent release and I feel that it's important enough to remain open for better traceability/visibility".
I'm sorry that I don't have the resources to resolve the issue myself. All I can do is raise awareness of the criticality of it.

Actually, we can also close it.
My recommendation for users who want to visualize the distribution of datasets that exceed the implicit or explicit row limits is to rather use box plot charts instead and delegate the aggregation to the database.
The same recommendation applies to histogram and CDF plots.

hi  @Rumbin, I have the same issue as you, that the plot's statistic is different because the row limit is exceeded, can you share how you resolve this? thank you.

hi @rumbin, I too don't understand your recommendation. Can you please clarify?


snow which could resolve this conflict

error: failed to select a version for snow.
... required by package libp2p-noise v0.30.0
... which is depended on by libp2p v0.37.1
... which is depended on by sc-cli v0.10.0-dev (https://github.com/paritytech/substrate.git?tag=monthly-2021-08#4d28ebeb)
... which is depended on by frame-benchmarking-cli v4.0.0-dev (https://github.com/paritytech/substrate.git?tag=monthly-2021-08#4d28ebeb)
... which is depended on by node-template v3.0.0-monthly-2021-08 (/home/abel/subworkspace/substrate-node-template/node)
versions that meet the requirements ^0.7.1 are: 0.7.2, 0.7.1
all possible versions conflict with previously selected packages.
previously selected package snow v0.7.1
... which is depended on by libp2p-noise v0.30.0
... which is depended on by libp2p v0.37.1
... which is depended on by sc-cli v0.10.0-dev (https://github.com/paritytech/substrate.git?tag=monthly-2021-08#4d28ebeb)
... which is depended on by frame-benchmarking-cli v4.0.0-dev (https://github.com/paritytech/substrate.git?tag=monthly-2021-08#4d28ebeb)
... which is depended on by node-template v3.0.0-monthly-2021-08 (/home/abel/subworkspace/substrate-node-template/node)

https://github.com/paritytech/substrate/pull/9391 bumps the libp2p version which will also bump snow, might be worth a try to cherry-pick.

Hmm I can see that if I try and build Bit.Country master for example that this gets hit by this issue. I presume others will also hit this.
The prost 0.8 patch should probably be backported also as that seems to hit a fair few projects.

build Bit.Country master

Build what?
These monthly releases are absolutely random anyway and don't follow any real versioning.

sc-cli will not build anymore.  I believe this is due to the one of the nested dependencies aes-soft ^0.5.0 being yanked.  See below to recreate.
❯ cargo new hello_world --bin
     Created binary (application) `hello_world` package
❯ cd hello_world
❯ echo "sc-cli = { version = \"0.9.0\" }" >> Cargo.toml
❯ cat Cargo.toml
[package]
name = "hello_world"
version = "0.1.0"
edition = "2018"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
sc-cli = { version = "0.9.0" }
❯ cargo build
    Updating crates.io index
error: failed to select a version for `snow`.
    ... required by package `libp2p-noise v0.29.0`
    ... which is depended on by `libp2p v0.34.0`
    ... which is depended on by `sc-cli v0.9.0`
    ... which is depended on by `hello_world v0.1.0 (PycharmProjects/reef/hello_world)`
versions that meet the requirements `^0.7.1` are: 0.7.2, 0.7.1

all possible versions conflict with previously selected packages.

  previously selected package `snow v0.7.1`
    ... which is depended on by `libp2p-noise v0.29.0`
    ... which is depended on by `libp2p v0.34.0`
    ... which is depended on by `sc-cli v0.9.0`
    ... which is depended on by `hello_world v0.1.0 (PycharmProjects/reef/hello_world)`

failed to select a version for `snow` which could resolve this conflict


Add Ubuntu 14.04 Support

Hi
I have forked your repo and made changes. Created PR. Awaiting reviews and comments. Thanks for all your explanations and support :)

Thanks for all your explanations and support :)

Glad to help, thank you for your work to contribute to WCBench.
Without getting into details about the changes proposed in this PR just yet, there's a pretty fundamental issue - it breaks huge chunks of WCBench's functionally. Totally removes the Vagrantfile, totally removes most of the docs, totally breaks Fedora compatibility.
I'm sorry if I was unclear, but the goal is to add Ubuntu support in a backwards-compatible way. WCBench is a fairly widely consumed project, so it's a huge no-no to cause API backwards-incompatible changes.
What I had in mind was to add conditional logic, perhaps passed as an additional flag, to do the correct OS-spicific stuff for Fedora/Ubuntu. So for example, we might do a yum install <package> if the flag is set to fedora and apt-get install <package> if it's set to ubuntu. This maintains all of our current functionally while adding (not substituting) Ubuntu support, which is the goal.

Thanks for all your explanations and support :)

Just for the sake of learning/feedback, you might find these resources about writing good commit messages to be helpful. This one is exhaustive and very important (used by OpenDaylight and OpenStack). This other one is a bit shorter and maybe a little more readable, but still covers good info.

Totally removes the Vagrantfile,
I dont know how to write a vagrantfile yet.

totally removes most of the docs,
I think "readme" is enough which describes when and how to use this.

totally breaks Fedora compatibility.
This maintains all of our current functionally while adding (not substituting) Ubuntu support, which is the goal.
I didnot know that the changes will affect the main "wcbench.sh" (the one with fedora). I was under the impression that this will create 2 packages, one to use when you have fedora and second when you have ubuntu. I didnot know that ubuntu logic should be added to the original wcbench.sh file.

What I had in mind was to add conditional logic, perhaps passed as an additional flag, to do the correct OS-spicific stuff for Fedora/Ubuntu
Yes, this would be efficient to do.

Made changes...Can you check now?

Sorry for the lag on my part.
This is much better! I'll try to dedicated some solid time to sit down and add detailed in-line comments later today or this weekend.
Thanks again for contributing to WCBench!

Added quite a few inline comments, as you can see. As I said, looks way better, clearly on the right path. Thanks! :+1:

Thabks for reviewing it...Agreed that it is too much of code duplication..Had the same thought while uploading the script. I don,t know how to check flags inside a function. Not strong in shell scripting :(...Will learn and update this weekend.:)

Thabks for reviewing it

It's my pleasure, thank you for submitting the PR. :)

Agreed that it is too much of code duplication..Had the same thought while uploading the script.

Cool, +1

I don,t know how to check flags inside a function. Not strong in shell scripting

Example:
#!/usr/bin/env bash

OS="fedora"

if [ $OS = "fedora" ]; then
  echo "You're using Fedora"
elif [ $OS = "ubuntu" ]; then
  echo "You're using Ubuntu"
else
  # This will go to stderr, FYI
  echo "ERROR: Don't know how to handle OS=$OS" >&2
fi

[~]$ chmod +x bash_tut.sh
[~]$ ./bash_tut.sh
You're using Fedora

sudo wherever added is because for me it runs as root..I will remove that..

Made discussed changes...Added feature to loop N times too..

There have been a few changes, mainly #67, since you forked the repo and started making your changes. That was mostly my fault for taking a while to get this merged (I've been traveling all week for the ODL Hackfest).
There are also a few pedantic changes I'd like to make before I merge your changes into master. Again, mostly my fault for being pedantic.
So, my plan is to merge this into a feature branch, make some changes myself and then merge it into master.
I hope to get that done on my red eye home (which I'm on right now). If I don't, I'll definitly have it merged Tuesday (I'm taking PTO on Monday to recover from the red eye and spend time with a visiting out-of-town friend).
Thanks again for the contributions, sorry for the lag getting them merged.

Added feature to loop N times too..

Just for the sake of learning, it's generally perfered to make logically distinct changes in differnet PRs. So ideally we'd make that change in a new PR, letitng this one stay focused on Ubuntu 14.04 support.

I merged this into the ubuntu branch. Now need to find time to make my pedantic clean-ups and merge it into master.

@Jaanki - Just spent a ton of time trying to clean up those commits using a series of git rebase -is and I still haven't gotten it to a reasonable state.
Since you're doing this (I think) as a learning experience for your Masters, I'm going to re-punt to you to handle this work.
Please submit one PR that contains a single, good commit to make the Ubuntu support change. You can do that by editing this PR or submitting a new one.
Please submit a second PR that adds the "loop N times" feature to loop_wcbench.sh. That's a logically distinct change that should be handled in a totally different PR/commit.
Here are some useful docs I wrote recently.

The loop-n feature was added in #72.

@Jaanki is doing this in #75.


FMU import, nothing happens.

FMU import, nothing happens. No errors raised.
To Reproduce

Press FMU import
Info: In FMIWrapper;  Loading FMU from C:/Users/OMISTAJA/Desktop/FMU/WaveBreaker.fmu...
Info: In FMIWrapper;  Successfully instantiated FMU
But no FMU appears on the model canvas

Expected behavior
*FMU should appear on the model canvas
Hopsan version (please complete the following information):
Version information is shown in the message widget when HopsanGUI is started

OS: Windows 10
Version: HopsanCore 64-bit, Version: <IP_ADDRESS>40527.1228
HopsanGUI 64-bit, Version: <IP_ADDRESS>40527.1228
Build: Official

The FMU component does actually appear on the canvas, but outside of the viewport. You can find it close to the bottom right corner. This is because of a bug with the canvas size. It will be fixed in the next version.

The "easy" solution for now is to add an FMIWrapper component manually from the Connectivity library, double-click on it and set the "path".


remove flask mail dependency from worker

remove flask mail dependency from worker by switching to an alternative smtp client

implemented in 6cf9bc70ed0781588bec2fcc571d2f4b42caed58


.NET8 AND MQTTNET ?

Just wanted to request if MQTTnet library is compliant with .NET8 framework ?

I guess, you mean .Net8? (There is only NetFramework for versions up to 4.x, Version 3 was called .NetCore and since version 5 it's only .Net). But yes, it's compatibel. I use it in several projects.

Ok thanks just wanted make sure before starting a .Net8 app project....
Regards
Joshua Lawson
Owner and Problem Solver
Assembly-Tek Pty Ltd
www.assembly-tek.com.auhttp://www.assembly-tek.com.au
Mobile :+61 456786606
Email: @.***

From: SeppPenner @.>
Sent: Thursday, December 26, 2024 10:37:55 PM
To: dotnet/MQTTnet @.>
Cc: Joshua Lawson @.>; Author @.>
Subject: Re: [dotnet/MQTTnet] .NET8 AND MQTTNET ? (Issue #2133)
I guess, you mean .Net8? (There is only NetFramework for versions up to 4.x, Version 3 was called .NetCore and since version 5 it's only .Net). But yes, it's compatibel. I use it in several projects.
—
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/MQTTnet/issues/2133#issuecomment-2562525156, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BMTF3IDNLG7JUT2ERES42VL2HPTBHAVCNFSM6AAAAABUD5YE2GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRSGUZDKMJVGY.
You are receiving this because you authored the thread.Message ID: @.***>


Adjust cleaners and special riddens order

Correct order in which are shown cleaners (progressions + stats graphs)
Correct order of riddens are displayed on stats graph (ridden killed)

@junior291492659, thanks for fixing the mutation-kills order. It's so nice to see the PvP ones in the PvP order now!


Cbnt prov/new generation

Add support for SPR-SP CBnT implementation.

Verified to work on Archer City CRB. ACM establishes SRTM.


Linear performance across edit distnace?

Hey,
First of all, I would like to say this software is a cool idea.
I am trying to benchmark SWOffinder against my software and I get the same execution time across increasing edit distance. I supply the same edit distance from 0 to 4 across all the parameters in your program: maxE maxM maxMB maxB. If this is true then there is very unique and interesting property of the algorithm., but seems to make no sense to me when searching for example distance 0.
I wonder whether you think this is reasonable given how the algorithm works, or you think something else must be broken in how I benchmark?
Best,
Kornel

Hi Kornel,
I assume you are doing it right. The algorithm is comprised of two steps. The runtime of the first step, which is the modified Smith-Waterman algorithm, depends only on the genome and sgRNA lengths. Only the second step runtime depends on the edit distance and the different thresholds for mismatches and bulges.
For the practical edit distance and mismatch and bulge thresholds, the runtime of the second step is neglectable compared to the first step. Therefore, you should see almost constant runtime for small edit distances like the one you used.
I can see Figure 4B in our manuscript, where we show that the runtime of the second step is neglectable compared to the first step and function of the thresholds, and you are getting almost constant runtime for practical edit distances for CRISPR off-target sites.
I hope this answers your question. Feel free to let me know if you need more information.

Hi Kornel,
Can you please confirm that this answers your question so that I can close the issue?

Yes, thank you for your answer. It is indeed very interesting. I have couple other questions related to your latest manuscript, how can I get in touch with you?

Thank you.
You can send me an email to<EMAIL_ADDRESS>


"ng serve" and "ng build" fails after updating to Angular5

Updated to Angular5 / CLI v. 1.5.0 /  Node 8.9.0 / NPM 5.5.1 / Typescript 2.5.3 yesterday.
Getting following error when running "ng serve" or "ng build":
ERROR in ./node_modules/angular-font-awesome/angular-font-awesome.ts
Module build failed: Error: C:\Users\rj\Documents\Workspace\SeaBase-WEB-UI\node_modules\angular-font-awesome\angular-font-awesome.ts is not part of the compilation output. Please check the other error messages for details.
    at AngularCompilerPlugin.getCompiledFile (C:\Users\rj\Documents\Workspace\SeaBase-WEB-UI\node_modules\@ngtools\webpack\src\angular_compiler_plugin.js:629:23)
    at plugin.done.then (C:\Users\rj\Documents\Workspace\SeaBase-WEB-UI\node_modules\@ngtools\webpack\src\loader.js:467:39)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)
 @ ./src/app/app.module.ts 147:0-85
 @ ./src/main.ts
 @ multi ./src/main.ts

It does not fail, however, when doing production builds ("ng build --prod").
Anyone having similar issues?
There's already an issue about this in the correct repository, you can find it here.


Cannot use Regex Special Characters

What happened?
Any special assertion characters such as caret(^) or dollar($) do not appear to work in DQL queries. The example code below should match all tags which start with characters a through m but it matches nothing.
DQL
TABLE WITHOUT ID Tag, map(rows, (r) => link(r.file.link, r.title)) AS Notes
FROM "Material/Comedy"
WHERE contains(file.tags, "year-2024")
FLATTEN file.tags as Tag
WHERE regexmatch("^[a-m].*", Tag)
GROUP BY Tag

JS
No response
Dataview Version
0.5.64
Obsidian Version
1.5.12
Device
Samsung Galaxy Fold 5
OS
One UI 6.1

Isn't the first character of your tags #?

My apologies I realized that file.tags has the # in front whereas just tags does not have the #. Closing the issue.


More consistent empty set signs

Typst uses nothing sign that in default math font produces thin sign (like crossed zero). It is used rarer than "wide" sign (eg. see Wiki, it is primzry used on most languages I looked). Moreover, nothing.rev produces reversed wide sign.
I moved old nothing to nothing.alt, and replaced default nothing with wide sign, so now it looks like:

It is breaking change, but seems to touch not that much people as no one hadn't reported that before.
Notice: these symbols behave different on different fonts, but I think we should look at default math font first of all.

This change makes very little sense. 2205 and 29b0 are explicitly empty set and reverse empty set, while 2300 is diameter (in LaTeX' unicode-math as well).
We should follow unicode even if new computer modern math has a terrible empty set symbol.

I see, thanks for the insight. I should have done some research first indeed. :)
It still seems to me semantically wrong to write "diameter" when I mean "empty set" in math, it makes it feel wrong. I know that probably let or show rules can help it, but that just feels bad.
I think it should be mentioned somewhere anyway as it is really unintuitive. Maybe, there could be "comments on symbols" in documentation one time.

@sitandr  Might help. https://github.com/johanvx/typst-undergradmath/issues/10

Looking at https://ctan.uib.no/fonts/newcomputermodern/doc/newcm-doc.pdf (section 13.3) the correct way of dealing with this is using the cv01 font feature, as the font provides an alternative version of the symbol. Though I do not believe this feature is currently accessible in Typst.


[ibex/dv/rtl] Updates to run Cadence Xcelium

This PR makes changes to support Cadence Xcelium 20.09.001
Compile error 1: Fixed the following rtl compile error in ibex_cs_registers.sv
assign tselect_rdata = {'b0, tselect_q};
|
xmvlog: *E,NONOWD (/proj/riscv_ibex/users/paulok/lowRISC/ibex_1/rtl/ibex_cs_registers.sv,1288|30): Illegal use of a constant without an explicit width specification [4.1.14(IEEE)].
Compile error 2: Fixed the following DV compile error in ibex_mem_intf.sv
.data_req_o (data_mem_vif.request ),
|
xmelab: *E,ICDCBA (./tb/core_ibex_tb_top.sv,89|14): Illegal combination of driver and output clockvar to variable 'request' detected (output clockvar found in clocking block at line 25 in file ./common/ibex_mem_intf_agent/ibex_mem_intf.sv).
Simulation probes: Fixed issue with probing in yaml for xrun and added licqueue.
Signed-off-by: Paul OKeeffe<EMAIL_ADDRESS>

@PaulOKCrevinn after you've addressed @rswarbrick's comment, you can do a git add -u; git commit --amend; git push -f to update this pull request. The git push -f updates the branch in your fork with the new changes, and this PR will update automatically with it.


Tx's inconsistently not showing in Recent Transactions when completing trades

Overview
Noticing randomly sometimes the tx's are not showing in the Recent Transactions when completing trades
References and additional details

Log into app.shapeshift and connect to wallet of choice
Navigate to trade page and perform a trade
Note that sometimes the tx's show in recent transactions, other times the model will update to Trade Complete but there are no new transactions in the history. In screenshot below, I completed a cowswap usdc to usdt ... but the last tx is a eth receive.

**have not figured out if its one swapper in particular but have seen it happen on a 0x trade as well as a cowswap trade. however, this doesnt mean it happens every time

Acceptance Criteria
All tx's should show in Recent Transactions as they are completed
Need By Date
No response
Screenshots/Mockups
No response
Ownership

[X] If my bounty needs engineering or needs product I have added the respective labels on the right
[X] As the sponsor of this bounty I will review the changes in a preview environment (ops/product) or review the PR (engineering)

Estimated effort
No response
Sponsor / Stakeholder
No response
Bounty Hunters

Join our discord
Include an expected timeline for you to complete work in the work plan when you apply for this bounty!
Please refer to this link for some basic info
Please do not start work on this issue until you are approved in Gitcoin.

still experiencing this - 0x trade fox to eth on mm

0x trade dai on polygon to usdc on polygon on MM txs not showing in recent transactions


add support for SSR projects

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #95, #99, #107
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
*Add support for SSR
Type of change:
Please delete options that are not relevant.

[x] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Drop support for Python 2?

Any objection to dropping support for Python 2 in pybids? What prompted this question is some failing tests due to usage of inspect.getfullargspec, which doesn't exist in 2. This isn't a big deal as it's easy to workaround, but this kind of thing has already come up a few times and will probably continue to. Do we need to support 2 going forward? @chrisfilo @jbpoline @yarikoptic

I'm ok with dropping python 2 support.
On Sun, Aug 20, 2017 at 12:55 PM, Tal Yarkoni<EMAIL_ADDRESS>wrote:

Any objection to dropping support for Python 2 in pybids? What prompted
this question is some failing tests due to usage of inspect.getfullargspec,
which doesn't exist in 2. This isn't a big deal as it's easy to workaround,
but this kind of thing has already come up a few times and will probably
continue to. Do we need to support 2 going forward? @chrisfilo
https://github.com/chrisfilo @jbpoline https://github.com/jbpoline
@yarikoptic https://github.com/yarikoptic
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/INCF/pybids/issues/57, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkpzriNUFzDA-r-pDgM941s_d4raR_ks5sZ7xZgaJpZM4O8hHm
.

Okay. Assuming nobody else has objections (will give it a couple of days), I'll remove Python 2 from the test environment matrix.

Not that I am a very active user yet, but I would object. Even though we (ecosystem) are almost there,  many systems have python 2 as the default etc. I would appreciate if PYTHON 2 was still supported for a bit more

Support for Python 2 was removed in the latest round of PRs, so I'm closing this.

@adelavega - we should update tests/checks for BIDSGrabber on nipype to only run under python 3 and to raise an Exception if someone tries to import the BIDSGrabber in python 2.

@satra makes sense. I wonder how likely it is that BIDSDataGrabber will fail using python 2. If it's not very likely, but it is "unsupported" then perhaps a warning is a better option?

will pybids be installable under python 2? from the sounds of it, no.

I believe it should install, but the tests currently fail on Python 2 (only because of the minor change introduced in #71, which would be easy to workaround if anyone cares enough to patch it).
I don't think we should actively discourage Python 2 support or anything. I think if someone wants to open PRs to fix incompatibilities with 2, we should accept them. But I don't see a need at this point to actively support and test on 2.
@yarikoptic, sorry, I didn't actually see your comment earlier. I guess I wouldn't mind supporting 2 if there was some critical package that depended on pybids and only worked under 2. But the fact that the ecosystem is almost there strikes me as a reason to actually move to 3 quicker--if more packages did that, people would have more incentive to make the switch.


cache_hash can give the wrong hash code for deserialized objects

I just realized a bug in the code I wrote for hash code caching (#426).  Because the hash code cache field gets serialized and deserialized by Pickle, when you deserialize a cache_hash=True attrs object, the hashcode will be the hashcode the object had at serialization-time. However, if your object has fields with hash codes which are not deterministic between interpreter runs, then on a new interpreter run your deserialized object will have a hash code which differs from a newly created identical object.
We can fix this for pickle by recomputing the hash code in __setstate__.  Other serialization libraries which don't respect __setstate__ will still have a problem, but I don't think we can do anything about that.  If the __setstate__ solution sounds acceptable I will implement it next week.

I feel slightly better that I am not the only one to screw this up. :-)


Add cast.any

There should be a cast.any which just silently accepts.

https://github.com/leafpetersen/cast/commit/ac72a687e62e3208912525121fc3ada62a4969b8


Add Parsing and Serialization test-cases for new Opaque Value Instructions

Small part of radar rdar://problem/30080769
Adds Parsing and Serialization test-cases for init_existential_opaque, deinit_existential_opaque, open_existential_opaque and unconditional_checked_cast_opaque
Also fixes small bugs that preventing proper parsing / serialization of these types

@swift-ci Please smoke test

Hi, Joe. When you change Serialization, please remember to bump the version number in ModuleFormat.h; otherwise we get crashes when old compilers try to load the new modules or vice versa. Thanks!


Category names that are numbers

Avoid error when category names are numbers
If a category name is 2016 in the XML pulldom will preserve it as an integer type. As we check the length of the name it needs to be a string.
Issue: https://github.com/torchbox/wagtail-wordpress-import/issues/148

This adds a new category to test in the XML file for one of the items and passes the name through str() to convert the category name before the length is tested.

Testing

[ ] CI passes
[x] If necessary, tests are added for new or fixed behaviour
[x] These changes do not reduce test coverage

Documentation.

[x] Documentation changes are not necessary because: no changes are made to the package operation.

Thanks!


Sync dev with main on new tag

Is your feature request related to a problem? Please describe.

Describe the solution you'd like
Dev sould sync with main on new tag (e.g. v0.0.1)
Dev should sync with beta if the tag ends with an b (e.g. v0.0.1b)
Describe alternatives you've considered
None
Additional context
None

Stable (main) (pseudo)
on:
 push:
  tags:
   - 'v*'

beta (pseudo)
on:
 push:
  tags:
   - 'v*b'


Might be better to use something other than .f3d & .f3z file formats

With the .f3d and .f3z file formats - from memory of when I used to use Fusion 360 - they're undocumented and change over time:

https://forums.autodesk.com/t5/fusion-360-design-validate/file-format-specification-for-f3d/td-p/5804681

It might be better to use a file format like (say) STEP or IGES which are standardised and widely supported. eg can be opened by not just Fusion 360, but many other commercial and OSS packages too
Both of those generally contain rich levels of object data, and at least STEP (not sure about IGES) can contain model construction history so timeline/construction info is preserved.

@justinclift that's a good point :+1:
I haven't opened Fusion 360 for about 2 months so my knowledge could be rusty and forgive me (and please correct me) if I'm wrong but as far as I know it's not possible to export distributed designs (.f3z) in STEP or IGES formats.
Although we are able to publish .f3d files in STEP or IGES formats, they won't be functional inside TheHelmet.f3z file. So what's the point to having files in STEP or IGES formats inside the repository? :blush:
Also we don't track history of any CAD model in this repository. Because it's too expensive for GitHub's policies (What is my disk quota?) Instead we use Git LFS and don't track the history of these files.
P.S. I should clarify that GitHub's policies are completely fair and reasonable.

Hmmm, when you say "distributed designs" are you meaning an assembly made up of multiple objects/files?
STEP definitely can contain multiple objects.  I don't remember if it allows for calling out to other files though. :smile:
It might be a case of needing to saving the high level assembly as a STEP instead of the individual parts.
My point with .f3d/.f3z is also that since the file format changes over time (and doesn't appear to contain a version number) that files you save here now may not be usable by future versions of Fusion 360.  As lousy as that sounds. 😦

@justinclift Distributed design = a file with .f3z extension. Fusion 360 does not allow me to export distributed designs other then .f3z format.
The files are also stored in our Autodesk cloud, so we will not lose any data. This repository's purpose is not version control for CAD files. The primary purpose of this repository is publishing the CAD files. The secondary purpose of this repository is tracking history of any data(small), plan and program files.

No worries. :smile:

@justinclift I decided that you were right. So I migrated from .f3d to .stp and .stl. Could please you verify the repository's state by importing into Fusion 360? Thanks for these suggestions. :+1: :blush:

Cool.  Personally I can't verify this, as I don't use Fusion 360 any more.
Someone else should probably be interested enough to try it out though. :smile:

@justinclift OK then I'm closing this issue. Thank you so much! :blush:


Clean up JS tests

I think the JS unit tests need to be cleaned up. I've been working on it a little bit, I'll submit a pull request if I get positive feedback. Subsections should probably be split by a describe clause. Assertion style should probably also be standardized for readability. Comments don't make much sense inside the tests unless describing some obscure bug (unit tests should be relatively self explanatory). Thoughts?

@kfichter yes agree, and this would be very much appreciated.
If you're going in there, I also very much dislike the pattern of initiating a new contract for each test.
In fact, I would like it if the test suite could be also be use to test a contract already deployed at any address with some small tweaks.

@maurelian awesome, I'll go ahead and clean some more things up and submit pull reqs when I'm done!


[API Bug] Model.fit()

Model.fit([tr_pairs[:, 0], tr_pairs[:, 1]], tr_y,
batch_size=128,
steps_per_epoch=2,
validation_data=([te_pairs[:, 0], te_pairs[:, 1]], te_y))

complains:
Traceback (most recent call last):
  File "examples/mnist_siamese.py", line 133, in <module>
    validation_data=([te_pairs[:, 0], te_pairs[:, 1]], te_y))
  File "build/bdist.linux-x86_64/egg/keras/engine/training.py", line 1603, in fit
  File "build/bdist.linux-x86_64/egg/keras/engine/training.py", line 1093, in _fit_loop
  File "build/bdist.linux-x86_64/egg/keras/engine/training.py", line 1031, in _check_num_samples
ValueError: If steps_per_epoch is set, the `batch_size` must be None.

but with
Model.fit([tr_pairs[:, 0], tr_pairs[:, 1]], tr_y,
batch_size=None,
steps_per_epoch=2,
validation_data=([te_pairs[:, 0], te_pairs[:, 1]], te_y))

complains :
Traceback (most recent call last):
  File "examples/mnist_siamese.py", line 133, in <module>
    validation_data=([te_pairs[:, 0], te_pairs[:, 1]], te_y))
  File "build/bdist.linux-x86_64/egg/keras/engine/training.py", line 1603, in fit
  File "build/bdist.linux-x86_64/egg/keras/engine/training.py", line 1155, in _fit_loop
  File "build/bdist.linux-x86_64/egg/keras/engine/training.py", line 1332, in _test_loop
  File "build/bdist.linux-x86_64/egg/keras/engine/training.py", line 374, in _make_batches
TypeError: float() argument must be a string or a number

the reason is that it calls _test_loop(batch_size=None)
Calling Model.fit() with Passing validation_steps has no effect on this.
This seems like an API bug to me. Possible solutions:

allow batch_size when steps_per_epoch is set, ignore for training, use for validation.
Introduce a separate validation_batch_size, defaulting to batch_size when steps_per_epoch=None
Use _test_loop(batch_size=1) (slow). Maybe only when validation_steps is set?

I also don't understand the reason for prohibiting simultaneous setting of steps_per_epoch and batch_size. I actually expected to simply train with 2 minibatches per epoch, with different epochs going either 0,1,2,3,4... or 0,1,0,1,0,1... This is Model.fit(), not Model.fit_generator().

Same problem, don't understand why steps_per_epoch and batch_size cannot be set. batch_size determines the number of samples per batch and steps_per_epoch should determine how many such batches must be considered in an epoch.


[Ready for Review] Add license picker to projects

Purpose
Users need a way to license their work. Associate proposal here: https://docs.google.com/document/d/1a4hvALTN2AjoaaXy56CCnL5BnCNGvpQMgpPl3miuTfU/edit#
Changes

add node_license dictionary field to None

add license proprty method to None

add licensePicker knockout component and associated template

add list-of-licenses to package.json

update to make search page Projects/Components/Registrations filterable by license type

Side effects

Components inherit the license of the closest ancestor with configured license. Test that this works as expected.
By default forked projects adopt the forked from license
Registration's adopt the registered from license

First pass complete - no bugs found. :star:

Ping @sloria .... could be an easy review! :smiling_imp:

CR2 complete :+1: -- lets point that requirement to a commit and then we can let Steve know everything is ready for his final pass/merge @samchrisinger.

Casing is inconsistent. Use sentence-case.
Inconsistent language. Use "No license".
Inconsistent iconography. There is no icon for the other editable fields. Remove? Ping @erinmayhood for feedback.

We probably shouldn't show "None Selected" for non-contributors viewing a public project:

Why not just hide the "License" field if it isn't set?

@sloria and @samanehsan:

Yes to sentence case
We decided against no license because technically that is inaccurate and we didn't want any confusion from a legal standpoint. I am more comfortable with None selected.
We can remove the icon (putting this on a to do for myself to fix this issue soon)

Seems like strange placement for the submit button; it deviates from the rest of the site.
@erinmayhood

@samchrisinger I agree with @sloria that we can not show the License field  for non-contributors viewing a public project  if nothing is selected.

The border around the license text is unnecessary:

Also, there is no indication of the toggle state of the license text. Perhaps change the color of the icon when it is active/inactive. @erinmayhood

This spacing of this modal is awkward because there is no title.

Also pinging @erinmayhood for language review. This text sounds rather demanding of the user ("It is your responsibility...").

Need to head out now, so leaving my review off here.
Please implement the changes Erin M suggested.
I will continue reviewing the code later.

@sloria , @erinmayhood

to clarify: I'm removing the pencil icon. Hovering over the license text will indicate that it is clickable (if admin). For non-admins we will still show the info button.

The license should have the same styling as the other editable elements to indicate that it is clickable:

This close icon is inconsistent with close buttons in other panels/modals.

This is what it should look like:

This icon should have the pointer cursor.

There's quite a long delay before the license appears in the header.

@samchrisinger Can you investigate why it is taking so long to load? Also, should there be a loading indicator? Perhaps a textual indicator, e.g. "Loading license..." in muted text? @erinmayhood

Pass finished.

Also awaiting @erinmayhood 's feedback on https://github.com/CenterForOpenScience/osf.io/pull/4027#issuecomment-140541605

https://cloud.githubusercontent.com/assets/2379650/9890123/13ac6cec-5bcc-11e5-9516-7e31d7dbd270.png:
@samchrisinger Sure we can soften the language here:
Suggestion:
Your Own License (title if needed for formatting):
You have opted to use your own license. Please upload a license file name license.txt into the OSF Storage for this project.

https://github.com/CenterForOpenScience/osf.io/pull/4027#issuecomment-141167400 - can we just have a spinner if the load time can't be addressed?

@erinmayhood Which spinner? The logo spinner? Probably too large and grabs too much attention. Or something else?

@sloria, @erinmayhood  this loading issue is resolved.
Steve-- as a note, putting markup inside of the component tag works perfectly.

No load time = no loading indicator. Sweeet.

For consistency with the rest of the site, capitalize "OK".

Pass finished.

The validation currently disallows multiple copyright years (e.g. ranges):

We may want to remove the validation for this reason, but it behaves according to spec, so we can defer this.

Looks good. :octopus: