Declare an array proxy property for ordered relationships

This change defines a new readonly property of the form (name)Array for each ordered to-many relationship which returns the ordered set's -array proxy. It also defines ordered insertion and removal methods for (name)Array sufficient to make -mutableArrayValueForKey: work. This combination seems to allow you to bind an ordered relationship to an NSArrayController (though I wouldn't recommend mutating the relationship through one of the other accessors).

We are declaring pull request and issue 0 now that 1.3 is out. If this is still an issue you'd like to see address with 1.30 and going forward, please open a new issue so we can start a fresh discussion. Thank you!


updated pilosa badge to 0.3.2

Coverage remained the same at 100.0% when pulling 3669e49500eeeb8041699f40d32ce00b16d7d29d on yuce:next-v0.3.3 into bc127892c8ecf0067e16c76a67fb5f66b22388fc on pilosa:master.


Go ahead and build the doc.. We are doing so and will go through it

Once we agree, I can create a preSRG release, which uses GitHub tags to store the state of the project at this point in time.

Do you mind posting apdf on Google Drive. I would like to do a quick
comparison just to make sure I am doing the workflow correctly.  pdf is
fine.
On Mon, Feb 8, 2016 at 4:43 PM, cgrandin<EMAIL_ADDRESS>wrote:

Once we agree, I can create a preSRG release, which uses GitHub tags to
store the state of the project at this point in time
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105.

Still a few ** in glossary

We are getting those now...

Here is the bibtex entry for Stewart et al, which is referenced in the
glossary:
@article{Stewart201337,
title = "A comparison of stock assessment uncertainty estimates using
maximum likelihood and Bayesian methods implemented with the same model
framework ",
journal = "Fisheries Research ",
volume = "142",
number = "",
pages = "37 - 46",
year = "2013",
note = "",
issn = "0165-7836",
doi = "http://dx.doi.org/10.1016/j.fishres.2012.07.003",
url = "http://www.sciencedirect.com/science/article/pii/S0165783612002081",
author = "Ian J. Stewart and Allan C. Hicks and Ian G. Taylor and James T.
Thorson and Chantell Wetzel and Sven Kupschus",
keywords = "Maximum-likelihood",
keywords = "Bayesian",
keywords = "Uncertainty",
keywords = "Stock assessment ",
abstract = "Many fisheries stock assessment models are implemented
specifically for likelihood-based estimation or for Bayesian inference (via
full integration of the joint posterior distributions), but not all have
appropriate structure for both statistical approaches. Bias correction of
recruitment deviations, in particular, must be adjusted to achieve
consistency in each case. Fisheries management often uses the two types of
results similarly, setting future catch quotas based on expected values or
posterior medians depending on which is available given time constraints.
Using two recent examples from the U.S. west coast, Pacific hake and
sablefish, both implemented in Stock Synthesis, we find that
likelihood-based estimates of key management quantities, such as spawning
biomass, corresponded well with posterior modes, but tend to be lower (on
an absolute scale) than posterior median values and that the asymptotic
approximation for uncertainty intervals based on the Hessian matrix tends
to overestimate the likelihood of smaller stock sizes and underestimate
that of larger stock sizes. This pattern may be caused by a basic asymmetry
in most fisheries data-sets: the necessity of a minimum stock size to have
generated the observed catch/time-series, but little information regarding
the plausibility among much larger stock sizes. Where only one type of
inference is available, this asymmetry may be important for management
decision-making. Even if management takes explicit account of uncertainty,
in some cases adding a precautionary buffer that scales with the relative
uncertainty in point estimates, the differences in catch advice may turn
out to be important and the relative reductions non-linear. "
}
On Mon, Feb 8, 2016 at 4:45 PM, aaronmberger<EMAIL_ADDRESS>wrote:

Still a few ** in glossary
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181647278
.

i've done all that all be able to on this for now.  Thanks so much for your
efforts getting all latex knitr together!
On Mon, Feb 8, 2016 at 4:43 PM, cgrandin<EMAIL_ADDRESS>wrote:

Once we agree, I can create a preSRG release, which uses GitHub tags to
store the state of the project at this point in time
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105.

Ok, we are almost there. I'll post a PDF as soon as I merge these changes and build. Should be less than 10 mins.

Doing the bibtex. Are all the ** gone inthe main text?

I only see ** in the glossary of the build I just did. So, looking good.
On Mon, Feb 8, 2016 at 4:53 PM, andrew-edwards<EMAIL_ADDRESS>wrote:

Doing the bibtex. Are all the ** gone inthe main text?
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181649143
.

some new additions to sensitivity section seem incorrect ("2016 estimates at 0.8% of unfished biomass for the base model and 1% for the age-1 index").  I

If you can fix it, there's still time

I tried to generalize it. Need to multiply those by 100. Just do that in
the Sexpr
On Mon, Feb 8, 2016 at 4:58 PM, cgrandin<EMAIL_ADDRESS>wrote:

If you can fix it, there's still time
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181649869
.

Good catch Aaron!
On Mon, Feb 8, 2016 at 4:58 PM, Allan Hicks - NOAA Federal <
<EMAIL_ADDRESS>wrote:

I tried to generalize it. Need to multiply those by 100. Just do that in
the Sexpr
On Mon, Feb 8, 2016 at 4:58 PM, cgrandin<EMAIL_ADDRESS>wrote:

If you can fix it, there's still time
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181649869
.

and, use fmt0(val, 1) to make it nice

I'll wait for your commit

just did commit, but only removed %
didn't see this until after
On Mon, Feb 8, 2016 at 5:00 PM, cgrandin<EMAIL_ADDRESS>wrote:

I'll wait for your commit
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181650165
.

--
Aaron M. Berger, Ph.D. Research Mathematical Statistician
Fishery Resource Analysis and Monitoring Division
Northwest Fisheries Science Center
2032 SE OSU Dr. Bldg. 955
Newport, OR 97365-5275
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Committed. And, I should be because I really messed that one up.
On Mon, Feb 8, 2016 at 5:00 PM, cgrandin<EMAIL_ADDRESS>wrote:

I'll wait for your commit
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181650165
.

I thought for a moment there we switched over to Pacific bluefin!

I assume that Master Chris can sort this out.
On Mon, Feb 8, 2016 at 5:02 PM, Allan Hicks - NOAA Federal <
<EMAIL_ADDRESS>wrote:

Committed. And, I should be because I really messed that one up.
On Mon, Feb 8, 2016 at 5:00 PM, cgrandin<EMAIL_ADDRESS>wrote:

I'll wait for your commit
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181650165
.

You are still gun shy, I see.
On Mon, Feb 8, 2016 at 5:03 PM, aaronmberger<EMAIL_ADDRESS>wrote:

I thought for a moment there we switched over to Pacific bluefin!
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181651130
.

Chris, note that I corrected another error besides the %'s.  You should
choose my changes over Aaron's since I found an additional error.
On Mon, Feb 8, 2016 at 5:04 PM, Allan Hicks - NOAA Federal <
<EMAIL_ADDRESS>wrote:

You are still gun shy, I see.
On Mon, Feb 8, 2016 at 5:03 PM, aaronmberger<EMAIL_ADDRESS>wrote:

I thought for a moment there we switched over to Pacific bluefin!
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181651130
.

Yep, I did so. Building now. Will get back to you in a couple minutes

We agreed on line numbers on yes? It all works fine (as long you latex three times after bibtex, if you delete the .bbl each time [which you shouldn't really]). Works fine on mine.

For the sensitivity section, we can manually enter numbers if needed. I
just got an error. Let me know if you want the numbers.
On Mon, Feb 8, 2016 at 5:09 PM, andrew-edwards<EMAIL_ADDRESS>wrote:

We agreed on line numbers on yes? It all works fine (as long you latex
three times after bibtex, if you delete the .bbl each time [which you
shouldn't really]). Works fine on mine.
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181653952
.

Or, I thik the 100* needs to be within fmt0.  I was using round earlier
On Mon, Feb 8, 2016 at 5:14 PM, Allan Hicks - NOAA Federal <
<EMAIL_ADDRESS>wrote:

For the sensitivity section, we can manually enter numbers if needed. I
just got an error. Let me know if you want the numbers.
On Mon, Feb 8, 2016 at 5:09 PM, andrew-edwards<EMAIL_ADDRESS>wrote:

We agreed on line numbers on yes? It all works fine (as long you latex
three times after bibtex, if you delete the .bbl each time [which you
shouldn't really]). Works fine on mine.
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181653952
.

Sounds like it. And that's what's freaking Chris out right now.
Think he's found it.

I jsut made a commit
On Mon, Feb 8, 2016 at 5:15 PM, Allan Hicks - NOAA Federal <
<EMAIL_ADDRESS>wrote:

Or, I thik the 100* needs to be within fmt0.  I was using round earlier
On Mon, Feb 8, 2016 at 5:14 PM, Allan Hicks - NOAA Federal <
<EMAIL_ADDRESS>wrote:

For the sensitivity section, we can manually enter numbers if needed. I
just got an error. Let me know if you want the numbers.
On Mon, Feb 8, 2016 at 5:09 PM, andrew-edwards<EMAIL_ADDRESS>wrote:

We agreed on line numbers on yes? It all works fine (as long you latex
three times after bibtex, if you delete the .bbl each time [which you
shouldn't really]). Works fine on mine.
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181653952
.

My commit seems to be working (knitting)
On Mon, Feb 8, 2016 at 5:16 PM, andrew-edwards<EMAIL_ADDRESS>wrote:

Sounds like it. And that's what's freaking Chris out right now.
Think he's found it.
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181655847
.

My build with my recent commit looks OK
On Mon, Feb 8, 2016 at 5:18 PM, Allan Hicks - NOAA Federal <
<EMAIL_ADDRESS>wrote:

My commit seems to be working (knitting)
On Mon, Feb 8, 2016 at 5:16 PM, andrew-edwards<EMAIL_ADDRESS>wrote:

Sounds like it. And that's what's freaking Chris out right now.
Think he's found it.
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181655847
.

Ok, I put the build on the google drive... Check it out, What should we do now? I can create a release on github if we are happy with it.

I'll have a 2-minute look through. Then, email it to Miako and she will
distribute.
On Mon, Feb 8, 2016 at 5:30 PM, cgrandin<EMAIL_ADDRESS>wrote:

Ok, I put the build on the google drive... Check it out, What should we do
now? I can create a release on github if we are happy with it.
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181660002
.

Looks good to me!
Great job. It is always tough on the last day, especially when you have no
internet.
Thanks.
Allan
On Mon, Feb 8, 2016 at 5:31 PM, Allan Hicks - NOAA Federal <
<EMAIL_ADDRESS>wrote:

I'll have a 2-minute look through. Then, email it to Miako and she will
distribute.
On Mon, Feb 8, 2016 at 5:30 PM, cgrandin<EMAIL_ADDRESS>wrote:

Ok, I put the build on the google drive... Check it out, What should we
do now? I can create a release on github if we are happy with it.
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181660002
.

Ok, Andy has the line numbering better than mine, so he will post his on the Google drive, so please use that one. Thanks all for the hard work!

Great. And, can you email a copy to Miako? She doesn't have access to our
Google Drive. Or, I can download it and email it to her if you have any
issues.
On Mon, Feb 8, 2016 at 5:37 PM, cgrandin<EMAIL_ADDRESS>wrote:

Ok, Andy has the line numbering better than mine, so he will post his on
the Google drive, so please use that one. Thanks all for the hard work!
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181661257
.

We don't have email, so if you could do that we'd appreciate it. Thanks!

Mine and Chris's had identical pages (I checked each page), except his line numbers got out of whack. So use mine.....

Nice work all!
On Monday, February 8, 2016, Allan C Hicks<EMAIL_ADDRESS>wrote:

Great. And, can you email a copy to Miako? She doesn't have access to our
Google Drive. Or, I can download it and email it to her if you have any
issues.
On Mon, Feb 8, 2016 at 5:37 PM, cgrandin<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Ok, Andy has the line numbering better than mine, so he will post his on
the Google drive, so please use that one. Thanks all for the hard work!
—
Reply to this email directly or view it on GitHub
<
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181661257
.

—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181661823
.

--
Aaron M. Berger, Ph.D. Research Mathematical Statistician
Fishery Resource Analysis and Monitoring Division
Northwest Fisheries Science Center
2032 SE OSU Dr. Bldg. 955
Newport, OR 97365-5275
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Excellent. Bit hectic at the end, and sorry we were out of touch for a while, but we got there. I think we had enough eyes on everything that things are good. Our roles kind of evolved.... Chris is finishing his beer and then I think we're done.

So, Allan, you'll submit yes? No more changes.
If something really needs changing then do an issue and we'll check in at 8:30pm. Else, we're done.
Cheers....

Nice.  It is always hectic at the end. One year, Ian Stewart and I thought
we were done at 3pm.  It wasn't until 5:30 that we sent the document due to
pdf conversion issues.    The good thing is that we have a really good document, and
we can mess with the details over the next few weeks.
Get some rest, and we'll talk about the SRG soon.
Allan
On Mon, Feb 8, 2016 at 5:46 PM, andrew-edwards<EMAIL_ADDRESS>wrote:

Excellent. Bit hectic at the end, and sorry we were out of touch for a
while, but we got there. I think we had enough eyes on everything that
things are good. Our roles kind of evolved.... Chris is finishing his beer
and then I think we're done.
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181665115
.

Great. Just let us know when you've sent the email.

Already sent to Miako, who is waiting for it.  I cc'ed your DFO accounts,
so that you don't have to worry about it! Out of sight, out of mind!
On Mon, Feb 8, 2016 at 5:49 PM, andrew-edwards<EMAIL_ADDRESS>wrote:

So, Allan, you'll submit yes? No more changes.
If something really needs changing then do an issue and we'll check in at
8:30pm. Else, we're done.
Cheers....
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181665810
.

Excellent. We are done.

She has it and has uploaded it to the SRG google Drive folder:
https://drive.google.com/a/noaa.gov/folderview?id=0B8vORV3L9mJJNDFCeGs2MVlHaWc&usp=drive_web#
Have a good night,
Allan
On Mon, Feb 8, 2016 at 5:51 PM, andrew-edwards<EMAIL_ADDRESS>wrote:

Excellent. We are done.
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/105#issuecomment-181666147
.

Cheers. We even got the email! No need to check in tonight - have a good one.


etcdctlv3: del command from-key feature added

TODO: added from-key flag for del command.

@sinsharat Could you add e2e test cases for this flag?

@gyuho yes was planning to add it. do you want me to add it in this same pull request?

@sinsharat Yes, let's put them in this PR. Thanks!

@gyuho sure will add it.
Thank you!

@gyuho have added the test case.
Thanks!

@gyuho Can you suggest why the del test cases are failing on semaphore as when i am testing all the del cases are passing. Proof attached:

@xiang90 Request you to review this PR.
Thanks!

@sinsharat
test failed with
	ctl_v3_kv_test.go:210: delTest #3: ctlV3Del error (read /dev/ptmx: input/output error (expected "3", got ["4\r\n"]))

It seems that there is a bug at server side.

@xiang90 yes i guess earlier test case is somewhere putting key and not removing it. What i will do is change the key for putting and deleting in the example also will use the prefix from starting key to avoid such issues.
Thanks!

@sinsharat Oh. Yea. We created the first key []kv{{"this", "value"}} without actually deleting it.

@xiang90 but i wonder why that will cause the issue. Since when i am running those test cases locally all are passing :(

@xiang90 any how will try with a different key lets see what happens.
Thanks!

Lgtm

@xiang90 Thanks for the suggestion of using zoo, it worked :-)


Update CondorUtils.java

Fix JAVA_OPTS so as not to overwrite nersc/shifter configs
Create a log directory on the njs_wrapper container for the logs so they can be be generated per job. You still need to retrieve them with "condor_transfer_data" though.
Add JOB_DIR for container reaper
Add default concurrency limit to be the account group. This requires a corresponding change in the condor config (https://github.com/kbase/condor/pull/18)
7 day max job retirement time for condor_drain (MaxJobRetirementTime = 604800)


The accuracy on training dataset

What's your accuracy on training dataset.
After 20 epochs, I got the following accuracy:
17:49:33 Te=19 Loss=2.0825 | AccT=93.3490% (424355/454590) 2.6694 5.00 35525

Does this accuracy is  right?  How to improve the accuracy on training dataset?

Run python lfw_eval.py to eval your model. If it is not as good as you need, please  fine-tune it.


don't compile all parsed modules as entry modules in prover

Unfortunately the modules in the spec file could be part of a specification and therefore contain unbound variables on the right hand side and other features not supported in axioms, which means that it is not in general safe to compile them as part of the definition being proved. Fortunately, we don't actually need to do this, so this PR removes that functionality.

Can one of the admins verify this patch?

#57 fixes the issue with checkstyle.

@dwightguth does this work with the smart-contracts-verification repo? The only reason I could think that it doesn't is because extra modules are added after the fact for those proofs (the lemmas modules).

I tested it with the master branch of the evm repo and all the proofs pass. I'm pretty sure that's because the modules that are actually used are part of the dependency graph of the main module and don't actually need to be added separately.


Allow lenient matches

In theory, this would allow similar monitor configurations to match and a layout to get applied even if the exact head configuration doesn't match.
Looks like I should probably match on make+model+serial. I also have been capturing physical size, but I think that's probably unnecessary.
This mainly solves issues around hotplugging. I use USB-C with daisy chaining. Looks like this results in random display names (e.g., DP-2) that also seem to increment over time.

Fixed in ab6fbd6. I've only implemented exact matching (previous behaviour), or matching on the make+model+serial number (and I also disable this additional matching if the monitor is missing make/model). In theory I should have different "levels". For example, a make+model+serial number is a better match than make+model. However I think this will probably be enough for me.


KeyUpEvent Bug

Mac Only

A KeyUpEvent is dispatched, but the state shows that the physical key is not pressed. If this occurs in real application, please report this bug to Flutter. If this occurs in unit tests, please ensure that simulated events follow Flutter's event model as documented in HardwareKeyboard. This was the event: KeyUpEvent#6408a(physicalKey: PhysicalKeyboardKey#700e5(usbHidUsage: "0x000700e5", debugName: "Shift Right"), logicalKey: LogicalKeyboardKey#00103(keyId: "0x200000103", keyLabel: "Shift Right", debugName: "Shift Right"), character: null, timeStamp: 0:42:49.897992)
'package:flutter/src/services/hardware_keyboard.dart':
Failed assertion: line 432 pos 16: '_pressedKeys.containsKey(event.physicalKey)'

me too but it is mine on window（flutter run -d windows）
══╡ EXCEPTION CAUGHT BY SERVICES LIBRARY ╞══════════════════════════════════════════════════════════
The following assertion was thrown during a platform message callback:
A KeyUpEvent is dispatched, but the state shows that the physical key is not pressed. If this occurs
in real application, please report this bug to Flutter. If this occurs in unit tests, please ensure
that simulated events follow Flutter's event model as documented in HardwareKeyboard. This was the
event: KeyUpEvent#81540(physicalKey: PhysicalKeyboardKey#00036(usbHidUsage: "0x1600000036",
debugName: "Key with ID 0x1600000036"), logicalKey: LogicalKeyboardKey#00103(keyId: "0x200000103",
keyLabel: "Shift Right", debugName: "Shift Right"), character: null, timeStamp: 2:21:23.964957)
'package:flutter/src/services/hardware_keyboard.dart':
Failed assertion: line 432 pos 16: '_pressedKeys.containsKey(event.physicalKey)'
Either the assertion indicates an error in the framework itself, or we should provide substantially
more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
https://github.com/flutter/flutter/issues/new?template=2_bug.md
When the exception was thrown, this was the stack:
#2      HardwareKeyboard._assertEventIsRegular. (package:flutter/src/services/hardware_keyboard.dart:432:16)
#3      HardwareKeyboard._assertEventIsRegular (package:flutter/src/services/hardware_keyboard.dart:443:6)
#4      HardwareKeyboard.handleKeyEvent (package:flutter/src/services/hardware_keyboard.dart:535:5)
#5      KeyEventManager.handleRawKeyMessage (package:flutter/src/services/hardware_keyboard.dart:881:37)
#6      BasicMessageChannel.setMessageHandler. (package:flutter/src/services/platform_channel.dart:197:49)
#7      _DefaultBinaryMessenger.setMessageHandler. (package:flutter/src/services/binding.dart:387:35)
#8      _invoke2 (dart:ui/hooks.dart:186:13)
#9      _ChannelCallbackRecord.invoke (dart:ui/channel_buffers.dart:42:5)
#10     _Channel.push (dart:ui/channel_buffers.dart:132:31)
#11     ChannelBuffers.push (dart:ui/channel_buffers.dart:329:17)
#12     PlatformDispatcher._dispatchPlatformMessage (dart:ui/platform_dispatcher.dart:599:22)
#13     _dispatchPlatformMessage (dart:ui/hooks.dart:89:31)
(elided 2 frames from class _AssertionError)


pip-audit: 2.3.1 -> 2.3.2

Description of changes
https://github.com/trailofbits/pip-audit/blob/main/CHANGELOG.md#232---2022-05-14

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 177767 run on x86_64-linux 1

1 package built:

pip-audit


Arbitrarily-specified types in semantic templates unused when building dynamic library

def combine_signatures_or_rename_preds(unused, exprs, preferred_sig=None)

@kmineshima I will be solving this next week, and then we can merge the theorem branch.

Unittests did not signal a failure of this function. Extend those tests.

I fixed this and merged to master.


Make sheetMode public

I would like to do something like:
var flags xlsx.SheetMode
if !save {
  flags = xlsx.SheetModeStream
}
...
[perhaps elsewhere...]
...
sheet := xl.Sheet(0, flags)

But:

sheetMode can't be referenced
nice to have a name for the "default mode" (SheetModeUnknown?) available.

Of course there are other ways to do this, but it would seem a trivial change to the package that would allow more idiomatic code.

Well, don't forget that we have Sheet() to open and AddSheet() to create. But probably need to make sheetMode public, not sheetModeUnknown.

Check v1.0.4 - made SheetMode type public


Added low-level mechanism to run weaver apps.

This PR introduces a new low-level mechanism, run.Run, to run a weavelet when the topology of the application (e.g., the address of every listener and component) is static and predetermined.
To draw an analogy, git distinguishes between plumbing and porcelain. Plumbing commands are complicated low-level commands. Porcelain commands are simple, high-level commands that are built on top of plumbing commands. Most users use porcelain commands most (or all) of the time, but plumbing commands exist to make more complex git operations possible (though not necessarily convenient).
For Service Weaver, weaver single, weaver multi, weaver ssh, weaver kube, and weaver gke are porcelain commands. We don't offer any plumbing commands, besides writing a deployer completely from scratch. This PR aims to introduce a generally useful plumbing command.
Motivation
Here's our vision for how this PR will be useful. The details are not final, but I want to share something concrete to motivate the PR. We'll add a user-facing plumbing command on top of the run package, let's say weaver run. weaver run will take a topology file describing the topology of the application, say topo.toml:
deployment_id = "v1"
topology = {
    "Main": ["localhost:9000"],
    "Odd": ["localhost:9001"],
    "Even": ["localhost:9002"],
}
groups = [
    {
        name: "main",
        components: ["Main"],
        listeners: {"collatz": "localhost:8000"},
        address: "localhost:9000",
    },
    {
        name: "odd",
        components: ["Odd"],
        address: "localhost:9001",
    },
    {
        name: "even",
        components: ["Even"],
        address: "localhost:9002",
    },
]

Then, you can run an application manually like this:
$ weaver run weaver.toml topo.toml odd  # run "odd" group
$ weaver run weaver.toml topo.toml even # run "even" group
$ weaver run weaver.toml topo.toml main # run "main" group

This command will be generally useful, but our motivating use case is to enable people to run Service Weaver applications on Kubernetes with a level of control not possible with weaver kube.

This work has been deprioritized.


Documentation improvements

Connection with issue(s)

Close #???

Connected to #???
Solution description
Edited the description of the package on README to explain that the package is not only used with Flutter Form Builder package since this package can be used on any FormField widget - even without Flutter Form Builder
Also made grammatical improvements to README
Screenshots or Videos

To Do

[x] Read contributing guide
[x] Check the original issue to confirm it is fully satisfied
[x] Add solution description to help guide reviewers
[x] Add unit test to verify new or fixed behaviour
[x] If apply, add documentation to code properties and package readme

Perfect!


Update dependencies and project tooling

This PR updates dependencies and project tooling to align with other python projects in Aries.

I'm not 100% sure of the priority order for format checks; is the code quality check failing because this PR hasn't yet been merged?

@mepeltier checks are now fixed.


Can we have support for GGML as triton backend

I think this is something interesting and no one has done it before

Indeed llama.cpp/GGML is famous (about 30k star), but the code is really hard to maintain.
InferLLM may be a better choice.

wow what is it can you elaborate a bit more, to what i understand it's just that you need to implement the ggml itself into the triton backend and later on you can reuse the triton backend and have the benefit of ggml update after every iteration

will not be re-using fastertransformer i guess but more about how the ggml itself can be integrated with tritonbackend

will not be re-using fastertransformer i guess but more about how the ggml itself can be integrated with tritonbackend

Here is my opinion, when you integrate a inference backend, you have to make sure of service quality.
Let's take a assumption, after integration, if ggml has a bug, lmdeploy has responsibility to locate it or fix it.
The greater cost of software is maintenance, so we need to consider the code complexity of ggml.

About InferLLM  https://github.com/MegEngine/InferLLM

but having ggml also can attract more contributors, also i'm curious whether inferLLM support cuda, doesn't see that on the page

saw enable gpu on the cmake file still

but having ggml also can attract more contributors, also i'm curious whether inferLLM support cuda, doesn't see that on the page

yes, attracting more contributors is a good reason to integrate ggml.

but having ggml also can attract more contributors, also i'm curious whether inferLLM support cuda, doesn't see that on the page

InferLLM cuda part is WIP  https://github.com/MegEngine/InferLLM/pull/27

InferLLM will have many language barriers for non-chinese speakers contributors also, which is a very large audience, for most of the open source implementation it depends on how many contributors you have.

InferLLM will have many language barriers for non-chinese speakers contributors also, which is a very large audience, for most of the open source implementation it depends on how many contributors you have.

Already passed on to InferLLM team .. QvQ

Close since no more activity over two weeks. Feel free to reopen it if it is still an issue


Update success.Rmd

Which package could be the real world one? Why not add "for instance "

Maybe agromet?

If it survives rOpenSci's peer review...

If it survives rOpenSci's peer review...

Hey we're not known for murdering packages :rofl:


Access-Control-Allow-Headers shows as two separate headers which makes IE not process a cors response.

Moving the bug from https://github.com/aspnet/Home/issues/640
@esyorcho
I've been trying to "tie" both headers together (authentication and content-type), so that they show up in one line like:
Access-Control-Allow-Headers: content-type, authorization
But no matter what I did, they were being separated in 2 lines. I removed the authorization header from my first call (the login) since I didn't need it anyways, and then it went through (because it only had the content-type header). Somehow post calls within the application that need the authorization header still have the same problem (content-type missing, because it's in the second line and IE can't read it). Could you please tell me if there's a way that you know to make both headers show up in one line? At the moment we're adding the Authorization header this way in js:
            // add authentication header:
            var authHeader = "";//"basic ";
            authHeader += this.get("token") ? JSON.stringify(this.get("token")) : null;
            return {
                headers: _.extend(headers, { Authorization: authHeader })
            };

and we do our post calls with restangular this way:
RestangularFactory.all("search/getFields").post(criteria, "");
In our server api we have all post and get calls with:
[Produces("application/json")]
[HttpPost]
public string GetFields([FromBody]CriteriaInputData criteria)
{
    return ...
}

If there is any other information that you need please let me know
Thank you very much

@esyorcho can you please post your policy again? Are you sure you are not adding the Access-Control-Allow-Headers yourself? This is a response header and not a request header.
Have you added a Access-Control-Allow-Headers in the WithHeaders policy?

https://github.com/aspnet/CORS/blob/dev/src/Microsoft.AspNet.Cors.Core/CorsService.cs#L147-L154
@harshgMSFT The headers are being added as a list of array values. This results in them being sent as multiple headers. If you need one header they need to be added as a single array element "value1, value2"

ah I see .. yeah makes sense...

Hi @harshgMSFT and @Tratcher ,
I've been trying to add them as 1 header but I couldn't.
We have in startup application:
app.UseCors(policy => policy.WithOrigins("http://domain1.com", "http://localhost:2025").AllowAnyMethod().AllowAnyHeader().AllowCredentials());
Then we add the authorization header this way:
var authHeader = "";//"basic ";
authHeader += veropathStorageFactory.get("token") ? JSON.stringify(veropathStorageFactory.get("token")) : null;
return {
headers: _.extend(headers, { Authorization: authHeader })
};
Here I've tried this like:
                return {
                    headers: { Authorization: authHeader }
                };

But there's still a "content-type" header added as a different line besides Authorization.
We have in the controller of the rest api:
[Produces("application/json")]
But I've also tried removing it and still adds the extra header:
Access-Control-Allow-Headers: content-type
that is not read by IE and blocks our calls.
Do you know what could be adding this extra content-type header?
Thank you

@esyorcho yes we know and there needs to be a fix in the lines pointed out by @Tratcher . Well be sending out a fix soon.

Thank you very much guys

@kichalla I'm donating this bug to @harshgMSFT

b1ade8f2cf2e0fec1e878eb251f8e626f2306861


Fix Drag-and-drop error

Fixes
Issue Number: closes #1548, closes #1047

Oops, forgot it was already submitted.  Sorry for the duplicate ticket.


WPF inner tmp_proj build doesn't generate version information

I had to add this XML to a WPF project before it would recognize the version of the project, even after working around https://github.com/NuGet/Home/issues/5894
  <Target Name="WpfPackageReferenceNBGVWorkaround" AfterTargets="PrepareForBuild" DependsOnTargets="GenerateAssemblyVersionInfo" />

No repro. I think I was confused by the nuget bug.


Graphics layer does not render items offset outside of it

This seems to be intentional (since normally the graphics layer's contents are confined within the layer). However, there is a use case for why one might want a graphics layer the renders regardless of it's position/size:
For my application I am using the layer as a sort of cartesian graph where the user can pan/zoom/etc, and all of the contents can exist at any (X,Y) value in space. Each of those elements is offset with .offset { their x, y pos }. The problem is that when you pan up above the origin and drag an element above the origin, it disappears because it is no longer within the graphics layer, which is set to only be the max size of it's container box.
The ability to be able to render items outside of the bounds of the graphics layer is what I would need to get around this - it might require a new type of graphics layer that does not do bounds checking when deciding whether to render or not - since the entire "graph" is an infinite cartesian plane, items should be able to be rendered anywhere, even outside of the bounds of the graphics layer.
I have tried another method to get around this limitation, which is by choosing not to pan the graphics layer at all and place all elements at .offset { their x, y pos + graph pan x, y }, but this causes an enormous amount of lag when panning and there are a lot of elements in the graph, because whenever the pan amount changes it will cause every single element in the graph to update their position at once. The more performant option is just to stuff all elements inside of the graphics layer that gets panned, eliminating the cascading updates occurring to each element, but that's impossible if panning the graph makes it all disappear when the main graph goes offscreen.
I'm not sure if there are any methods that I am unaware of that would solve this problem. Thanks!

I've run into the same issue. This issue manifests even with high-level composables, such as Modifier.offset.
Example:
fun main() = application {
    Window(onCloseRequest = ::exitApplication) {
        var offset by remember { mutableStateOf(IntOffset.Zero) }

        LaunchedEffect(Unit) {
            repeat(200) {
                delay(10)
                offset -= IntOffset(10, 0)
            }
        }

        Box(
            modifier = Modifier
                .fillMaxSize()
                .background(Color.Red)
                .offset { offset }
                .background(Color.Blue)
        ) {
            repeat(20) {
                Text(
                    text = "$it",
                    modifier = Modifier
                        .size(50.dp)
                        .offset(100.dp * it, 0.dp)
                        .background(Color.Green),
                )
            }
        }
    }
}

We add enough Text composables so that some of them extend beyond the boundaries of the parent Box. Then, if we .offset { offset } the Box enough that the blue background is no longer visible, it will also hide all Texts, although some of them should still be visible.
Not sure what to do about this.

Oh wow. This happens only with the pixel-based offset. Once replaced with the DP-based offset, the issue disappears.
fun main() = application {
    Window(onCloseRequest = ::exitApplication) {
        var offset by remember { mutableStateOf(DpOffset.Zero) }

        LaunchedEffect(Unit) {
            repeat(200) {
                delay(10)
                offset -= DpOffset(10.dp, 0.dp)
            }
        }

        Box(
            modifier = Modifier
                .fillMaxSize()
                .background(Color.Red)
                .offset(offset.x, offset.y)
                .background(Color.Blue)
        ) {
            repeat(20) {
                Text(
                    text = "$it",
                    modifier = Modifier
                        .size(50.dp)
                        .offset(100.dp * it, 0.dp)
                        .background(Color.Green),
                )
            }
        }
    }
}

I just checked this out again in 2023, and it is still present a year later in Jetbrains Compose 1.4.3 on Desktop.

Seems to also be related to #2817 (and by extension perhaps #1559)

The core issue is the same as in https://github.com/JetBrains/compose-multiplatform/issues/2807, let's keep only 2807.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Categories reordering

Q
A

Bug?
no

New Feature?
yes

Sulu Version
1.5.3

Is there any way to reorder categories?

Currently, there is no plan for ordering categories but there is a PR which makes categories movable https://github.com/sulu/sulu/pull/3212

Could you please explain me how to implement this feature to my sulu-minimal v1.5.3? I need to change the ordering just once but it is very important for me now.

you could extend the Category entity (see http://docs.sulu.io/en/latest/cookbook/extend-entities.html) for categories use the configuration sulu_category.objects.{model, repository}. there you can add a column order which you can set over the database and then use this column to write SQL statements in you controller. if you have any deeper question you can contact us directly over slack (register on http://sulu.io/en#question)

@ilyasavitski if you just want to reorder it once you could write a symfony command to reorder the categories and use the moveUp and moveDown function from the repository. The categories use the Gedmo Tree extension. Which also have a simple reorder funciton in the repository. The categories repository service you get with sulu.repository.category.


Adds support for BigDecimal tagged types

Fixes #382

@juanpedromoreno
I'm guessing since the older decimal serializer doesn't fulfill the avro specs, we can make the tagged type the default behavior for the idl gen. Let me know if that makes sense.


[FIX] ruff: ignore printf-string-formatting of rule up031

When not using Ruff, pre-commit prevents pyupgrade from replacing printf with str.format() (https://github.com/OCA/oca-addons-repo-template/pull/49).
Using Ruff this rule is not being ignored. This commit modifies the Ruff configuration adding an ignore rule for UP031.

In which cases ruff is not used?

In which cases ruff is not used?

Ruff is not being used in version < 14 or when use_ruff is set to False in Copier by the user.

I think we should not disable UP031.
If my memory is correct, using .format is only a risk for translatable strings, and that is caught by pylint Odoo.
And in Odoo the best pattern for translatable strings is _("aaa %s bbb %s", 1, 2), which is not handled by UP031.
In other cases, that rule is useful, because f-strings are generally more readable that printf style.

If my memory is correct, using .format is only a risk for translatable strings, and that is caught by pylint Odoo. And in Odoo the best pattern for translatable strings is _("aaa %s bbb %s", 1, 2), which is not handled by UP031.

I have just tested this on the pre-commit config, confirming that this is the current working behaviour. So since formatting conventions for translatable strings are upheld, I will close the PR. Thanks for your feedback.


Welcome to the CIDO issue tracker website

This is the CIDO issue tracker website. You are welcome to submit your issues related to the CIDO ontology. Thanks.

I will close this first welcome message. Thanks, all.


Accessing the hash in a template

I'm using a Sass mixin to include all webfonts on my site. That mixin requires a few parameters: font-family, font filename slug, font weight, font style and a cache busting string (hash).  I want to use that mixin to include my icons in the grunt-webfont template too. The only thing that I'm unable to reference in the template is the generated font file's hash. Is it available in the template?

Is it still possible to append the hash to the filename? Or somehow get it in Grunt and append it manually?

@danielguillan Nope. You can do something hackish like store hash in CSS comment, then read it and rename files.

That's what I though. Thanks @sapegin !


Update interactive chat to v1.1

Adds support for split chat messages and fixed game client layout
Fixes:
https://github.com/richardcane/interactive-chat/issues/2
https://github.com/richardcane/interactive-chat/issues/6

It looks like your mouse listener is never unregistered if the plugin is disabled, please fix that--register and unregister it in your plugin on startup and shutdown instead of your overlay on instantiation, for example.
Does this line trigger an assertion with -ea? I'm not sure what thread MouseEvent is fired on.

It looks like your mouse listener is never unregistered if the plugin is disabled, please fix that--register and unregister it in your plugin on startup and shutdown instead of your overlay on instantiation, for example.
Does this line trigger an assertion with -ea? I'm not sure what thread MouseEvent is fired on.

As mentioned in discord, will fix the registration issue.
This line does not trigger any assertion with -ea but I will look into forcing it onto the client thread for clear intent if nothing else.


Increase the timeout ~2m for stop operation

Right now during the graceful shutdown the VM is taking more than 90 sec which is the default time limit so we need to increase it around ~120 sec and also handle the error better way on crc side.

resolved by #13


Cookies.txt file not being found

Keeps saying the cookies.txt isn’t there and I’ve placed it in every folder labeled gamdl. Is there a specific folder or place where I’m supposed to place it that I don’t know of?

The same thing happens when a run on android termux terminal.

You have not place the cookies on the current directory. Do it like so or specify the location using the cookies path option.


Update policy-csp-bitlocker.md

issue #2145,
added footnote

added link to bitlocker CSP

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@MaratMussabekov - please merge the commit on the copy/edit review made by JohanFreelancer9 / nenonix  or make the necessary changes on the PR following the suggestions made. Thank you.


Send a notification for events

Since there is code in place to add permission to join the SHOWS voice channel, we should also notify the interested users that an event is starting, would need a slight modification to the on_scheduled_event_update method.

Marking as closed, added a list that tracks all interested users, after the for loop we send a notification using channel.send to send the notification to the interested users.
mention_string = ' '.join(mentioned_users) - allows for the @user in the channel.send
Final changes:


Fix for bef thunk custom call test

Thanks for your contribution! Unfortunately, tensorflow/runtime is currently not
accepting contributions. Please see the
Contribution Guidelines for
more information.
/cc @chsigg

This is wrong so closing.


Planned functions

[x] Download bibjson metadata from /articles route
[x] Download summary stats - e.g., number of articles per journal, journals per publishers etc.
[ ] Plot the age distribution of documents per journal/publisher
[ ] Return the docid_term tuples for a given dictionary
[ ] NNP Cluster function
[ ] Simple terms, doc, sentid, sentence function
[ ] Find linked words using dep_paths/parent information
[ ] Chop off references/literature cited

@aazaff  have you revisited this

Updated

The references chopping should be handled by @iross on his end.


[NES-1051] [NES-1052] Add options for sip header

I am not sure that they want to use the same template. I will check. Other point, perhaps they prefer another SIP header. I take the points.

I am not sure that they want to use the same template. I will check. Other point, perhaps they prefer another SIP header. I take the points.

I am not sure that they want to use the same template. I will check. Other point, perhaps they prefer another SIP header. I take the points.

Why? For me this template is for the same use case for them.

I am not sure that they want to use the same template. I will check. Other point, perhaps they prefer another SIP header. I take the points.

Why? For me this template is for the same use case for them.


attributes are returned in lowercase (at least in chrome) which breaks conditionals

Hi,
In chrome, and possible others, the domElement.atrributes are all returned in lowercase.
This breaks the cf-conditional checks if the user has different casing for element names.
Changing the code at FlowManager.prototype.areConditionsInFlowFullfilled from:
...
var tagCondition = tagConditions[j];
if ("cf-conditional-" + tag.name === tagCondition.key) {
...

to:
...
var tagName = tag.name;
if (tagName) {
	if ("cf-conditional-" + tagName.toLowerCase() === tagCondition.key) {
	...
}

fixes the issue. Probably also documentation about conditionals should be updated.
mzZzl,
JamBo

Hi Felix,
I would like to suggest using a little more defensive implementation in this case.
When the tag is invalid, par example an empty option tag, the current implementation will break as the tag name will be null.
With defensive programming the tag would be ignored.
...
var tagName = tag.name;
var tagCondition = tagCondition.key;
if (tagName && tagCondition ) {
	if ("cf-conditional-" + tagName.toLowerCase() === tagCondition.toLowerCase()) {
	...
}

mzZzl,
JamBo


NBA url ?

Checklist

[X] I'm asking a question
[X] I've looked through the README and FAQ for similar questions
[X] I've searched the bugtracker for similar questions including closed ones

Question

Hi, i check all issues about NBA and i don't find who i find (😅) the url for game match ?
thanks for the help

If you don't know the URL that plays some video, yt-dl won't find it for you. On the other hand, if you have a page (and its URL) with a video that you'd like to capture, and yt-dl doesn't do that, please open an issue with the details: a site support request if the URL falls back to the Generic Extractor; a broken site support request if an exiting extractor for that site fails.
For info on how to watch (I assume) basketball online, try a forum dedicated to the sport.


Unifying UX/UI and software engineering work

To provide some context to #57,
@tgeorgeux and I have been brainstorming how to synchronize UI/UX and developer work more effectively on Github. This is particularly challenging since UI/UX collaboration doesn't usually happen on Github. It happens in something like Figma.
In short, designers live on Figma and developers live on Github. This can lead to a disconnect between designers and developers working on open-source projects.
To bridge this gap, we thought it would be helpful to explore ways to bring @tgeorgeux's Figma work to Github. Right now, he's planning to post GIFs of his Figma designs as separate Github issue threads demonstrating user stories that will be accomplished by this extension (we'll label these issues with "Design and UX" tag). The hope is that this will spur discussion between designers (i.e. Tim) and developers that inform the direction of the project.
Thoughts/ideas/feedback?

That sounds like a good idea!


'destroyContact' API request failed

Hi,
I am running the following scenario:

Call to the IVR.
Call routed to the agent CCP
Call automatically accepted using contact.accept()
After 10 seconds I am trying to disconnect call using contact.destroy() and getting the following error:

[2017-07-27T20:49:11.939Z] [ERROR]: 'destroyContact' API request failed: Error
ERROR @ lily_ccp_Prod_8998283….gz.js:22343
Logger.addLogEntry @ lily_ccp_Prod_8998283….gz.js:22570
(anonymous) @ lily_ccp_Prod_8998283….gz.js:25513
(anonymous) @ lily_ccp_Prod_8998283….gz.js:23446
EventBus.trigger @ lily_ccp_Prod_8998283….gz.js:23444
Conduit._dispatchEvent @ lily_ccp_Prod_8998283….gz.js:23759
(anonymous) @ lily_ccp_Prod_8998283….gz.js:22770
16:49:12.733
Any ideas?
Thank you.
Yan

Hi yann140573,
Can you reproduce the issue, then execute "connect.getLog().download()", or click the "Download Logs" button in the CCP to gather logs and attach them to this issue?
Thanks!
Lain

agent-log.txt
Done!

Hi yann140573,
Thanks for the logs.  I believe the destroyContact method was deprecated at some point and was not removed from this API.  You should use destroy() on the initial connection of the contact instead:
contact.getInitialConnection().destroy()

If you are managing a three way call, you will want to destroy the third party connection as well:
var initialConnection = contact.getInitialConnection();
var thirdParty = contact.getSingleActiveThirdPartyConnection();
if (initialConnection) {
    initialConnection.destroy();
}
if (thirdParty) {
    thirdParty.destroy();
}

The contact is destroyed when its constituent connections are destroyed (with the exception of the agent connection).
Sorry for the confusion!  We will remove this deprecated Contact.destroy( ) method in a future iteration.
Lain

Another point of notice: you can destroy the agent connection in a three way call scenario.  This will leave the third party and initial connections together while the contact is destroyed.  This scenario is sometimes referred to as a "warm transfer" since the agent is initially connected and then leaves the contact.


Add Makefile

In my opinion our own tooling and custom commits (that are not ment to go upstream) should be commited to the ssb branch, such that main can remain a clean fork of the original project. What do you think?


Hello, congratulations on creating such awesome thing. I would like you to add: Seeing or accessing it through lock screen/screen saver. Is this possible? Please reply. THANKS <3

Prerequisites
Please check that your issue isn't already filed: https://github.com/kmikiy/SpotMenu/issues?utf8=✓&q=
Description
[Description of the issue]
Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

Expected behavior: [What you expect to happen]
Actual behavior: [What actually happens]
Reproduces how often: [What percentage of the time does it reproduce?]
Versions
SpotMenu version: [Version of SpotMenu]
You can get this information by highlighting SpotMenu.app (most likely in your Applications folder) in finder and pressing space.
MacOS version: [Version of MacOS]
You can get this information by clicking the  logo on the top left of your menu bar and selecting About This Mac.
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

Unfortunately, I think this is not possible. It looks like MacOS doesn't allow this kind of behaviour 😕


set-output is deprecated

Getting the following warning for both the github/codeql-action/init@v2 and github/codeql-action/analyze@v2 actions.
Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Link in warning: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Thank you for posting this. We are aware of the new deprecation of set-ouput, and are working on a fix.

I figured you guys where (being Github and all, lol), but I did not see an issue or a fix, so figured it needed to be explicitly stated. Helps others track as well.

Fix is here: https://github.com/github/codeql-action/pull/1301

Will that be added to the v2 version? Or do you mean a v3 version? (the former means there is nothing I need to do, the later means I need to keep an eye out for updates)

It will be in v2. There is nothing you need to do.


Can't run Spatial Transformation example

It seems like the keras' build function of the layers has changed its prototype and it now takes the input_shape as parameter, so this fails (see keras documentation).
Regardless, once I fix this, I get an error once the SpatialTransform layer tries to set the self.input attribute from locnet's input. This is what I get when I try to run the example:
AttributeError                            Traceback (most recent call last)
<ipython-input-6-8756311b2bac> in <module>()
      2
      3 model.add(SpatialTransformer(localization_net=locnet,
----> 4                              downsample_factor=3, input_shape=input_shape))
      5
      6 model.add(Convolution2D(32, 3, 3, border_mode='same'))

/home/onieto/local/anaconda3/lib/python3.5/site-packages/keras/models.py in add(self, layer)
    278                 else:
    279                     input_dtype = None
--> 280                 layer.create_input_layer(batch_input_shape, input_dtype)
    281
    282             if len(layer.inbound_nodes) != 1:

/home/onieto/local/anaconda3/lib/python3.5/site-packages/keras/engine/topology.py in create_input_layer(self, batch_input_shape, input_dtype, name)
    368         # and create the node connecting the current layer
    369         # to the input layer we just created.
--> 370         self(x)
    371
    372     def assert_input_compatibility(self, input):

/home/onieto/local/anaconda3/lib/python3.5/site-packages/keras/engine/topology.py in __call__(self, x, mask)
    485                                     '`layer.build(batch_input_shape)`')
    486             if len(input_shapes) == 1:
--> 487                 self.build(input_shapes[0])
    488             else:
    489                 self.build(input_shapes)

/home/onieto/local/seya/seya/layers/attention.py in build(self, input_shape)
     48         self.regularizers = self.locnet.regularizers
     49         self.constraints = self.locnet.constraints
---> 50         self.input = self.locnet.input  # This must be T.tensor4()
     51
     52     @property

AttributeError: can't set attribute

Any idea how to fix this? Thanks!
UPDATE: I'm on keras 1.1.1, using Python 3.5.2 on Anaconda 4.2.0
UPDATE 2: After trying with several keras versions, it seems like the jupyter example only works with keras 0.3.2

yeah, seya is based on keras 0.3
but a few layers were adapted in the branch called keras1. STN is one of them, try that one for latest versions of keras

Thanks for your reply, I will try out the keras1 branch.

I fixed the build function issue by updating the attention.py file to the keras1 version: https://github.com/EderSantana/seya/blob/keras1/seya/layers/attention.py
But now I have this issue, any ideas what the fan_in and fan_out are used for?
File "/home/gabi/PycharmProjects/fish/fish_detection/spatial_trafo/mnist.py", line 91, in <module>     model.add(Convolution2D(32, 3, 3, border_mode='same'))   File "/usr/local/lib/python2.7/dist-packages/keras/models.py", line 324, in add     output_tensor = layer(self.outputs[0])   File "/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py", line 491, in __call__     self.build(input_shapes[0])   File "/usr/local/lib/python2.7/dist-packages/keras/layers/convolutional.py", line 409, in build     self.W = self.init(self.W_shape, name='{}_W'.format(self.name))   File "/usr/local/lib/python2.7/dist-packages/keras/initializations.py", line 59, in glorot_uniform     s = np.sqrt(6. / (fan_in + fan_out)) ZeroDivisionError: float division by zero

that error is weird. fan_in and fan_out are the sizes of input and output layers. and I'm sure you didn't declare a layer with zero connections...

Yeah. I couldn't figure it out. I'm just running it with keras 0.3.3 now and no issues.

I couldn't get the SpatialTransformer to work in my normal environment. So I set up a virtualnev with Keras 0.3.3 and installed Seya to that.
Trying to run the ST example ipynb for cluttered mnist, but I'm getting a problem. The net is learning nothing, loss stays exactly the same.

@9thDimension  @redsphinx @urinieto  here https://github.com/oarriaga/spatial_transformer_networks I have a working ipython notebook with the very same example using keras 1.1.1 with tensorflow 0.12.0 as backend.
@EderSantana if you want, I can make a pull request with the tensorflow implementation, just tell me where should I put the file containing the layer.

Hi @oarriaga thanks for all the help!
I think you can write the it all to the attention.py file.
Check if the user uses Theano or TF and use the appropriate code.
Have other ideas?
Also, if your code is keras1 compatible make sure you PR to that branch.

Great, thanks!
On Thu, Feb 16, 2017 at 8:20 PM, Eder Santana<EMAIL_ADDRESS>wrote:

Hi @oarriaga https://github.com/oarriaga thanks for all the help!
I think you can write the it all to the attention.py file.
Check if the user uses Theano or TF and use the appropriate code.
Have other ideas?
Also, if your code is keras1 compatible make sure you PR to that branch.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EderSantana/seya/issues/48#issuecomment-280547453,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADhisdsFCul84TwOCRvwZ6C_WwmXz1qqks5rdSALgaJpZM4K5uRH
.

Hi everyone,
I got the MNIST example running with the help of @oarriaga Notebook. I tried to load a custom image directory/dataset from disk for running Spatial Transformation, but got stuck with an issue.
Specifically after some amount of inspecting the code, there seems to be some issue with the datatype of the numpy image array.
When the images are -
uint8 - image display working fine , training not working(images are displayed blank during training) (Loss is becoming zero with blank images displayed). Similair to @9thDimension Comment before
float32 - image getting deformed(pixellated, random noise). But training seems to be working with noisy images. The noisy image looks like this

Any idea how to fix this? Thanks!


fix(project): Bump dependency version for vulnerability

Signed-off-by: James Hart<EMAIL_ADDRESS>PR summary

Updates a golang dependency. Believe the versioning tool should automatically bump the package-lock.json contents as part of creating the release. If not , we'll investigate that finding separately.
Fixes: Dependabot finding(s)
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[x] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

:tada: This PR is included in version 0.0.16 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Suggestion: import & export * as name from module in one line

TypeScript Version:  2.7.0-dev.201xxxxx

Search Terms:
Code
hope can export * as name from module
without write
when we wanna use path module too
import * as path from 'path';
export { path };

Expected behavior:
export * as path from 'path';

path.join(__dirname, '123');

Actual behavior:
import * as path from 'path';
export { path };

path.join(__dirname, '123');

Playground Link:
Related Issues:

This would fall out of scope of the design goals since it would change the behavior of ES6 modules.

This was a proposal https://github.com/tc39/proposal-export-ns-from, but I don't see it on the active proposals page. Marking this as a duplicate of #4813.


fix: use ssr transformRequest for css crawling for server

#554  👈
main

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

Join @hi-ogawa and the rest of your teammates on  Graphite


DBZ-2113 OpenShift tests for SQL Server Connector

https://issues.redhat.com/browse/DBZ-2113

@jpechane, could you check this one out?

@jcechace, can we wrap up this one?

@gunnarmorling  Will need some refactoring to play nicely with Mongo. In order to avoid cherry-picks and rebases I'm closing this sone and will send a new PR on Monday containing both -- Mongo and SqlServer together. Is that ok?

Do as you prefer. In generally, multiple smaller PRs are easier to handle.


About the code of diffusion function in SAFDNet.

Why the max pooling contains bg_group?
Following the fig.4 of SAFDNet, the bg voxels should not be diffused.
If I'm not mistaken,  max pooling is to diffuse voxels as described in AFD.
Looking forward your reply!

I post a example as following, which occur in the initial stage of training:
medium:5-->360 large:17-->1149 small:0 bg:9964-->18701  total:18701
Those number are number of voxels, and the medium, large, small are defined by here, bg means background voxel which below the fg_thr.
Although the threshold will be more precise as training and the number of background voxels will decrease, I can't deny the code conflicts with the paper if I didn't make mistake.
Looking forward your reply!

Why the max pooling contains bg_group? Following the fig.4 of SAFDNet, the bg voxels should not be diffused. If I'm not mistaken, max pooling is to diffuse voxels as described in AFD.
Looking forward your reply!

Fig.4 is an example, and it does not indicate that 'the bg voxels should not be diffused'. I admit this looks like a trick in our writing process (to highlight the sparse feature maps), but we have presented the implementation details in the Appendix.
It's worth noting that the SAFDNet is proposed to improve the model efficiency on long-range perception scenarios. On the Argoverse2 dataset with a long perception range, setting the diffusion range of background voxels to zero yields results similar to those obtained with diffusion (the released one). On the Waymo Open and nuScenes datasets, which have a shorter perception range, setting the diffusion range of background voxels to zero slightly degrades performance.

Got it. It's great work!


Toggle automatic line creation on creating filler on / off

Which area of IMT is your request about?  (ex:  Lines, Fillers, UI)
Lines / Fillers
Describe your idea clearly and concisely.
I would like to be able to turn off the automatic line creation feature when creating a filler which was introduced in 1.7.
Why would this idea be a good addition to IMT?
In order to create kerb extensions at junctions, I need the pavement fillers to clip beneath the pavement attached to the road. Since the release of the automatic line creation the edge of the filler follows the pavement edge and creates a gap.

Fillers cant be without lines.


Added new translations and modified existing ones

Thank you, @SuperPranx! Nice to see you again :+1:


add sm4 gcm encrypt mode

add sm4  gcm encrypt mode，this is a very good algorithm.

fix bug

fix bug


vendor: update vsock package version

The vsock package needs go >= 1.11. The latest version of the package adds a
compile time check on the go version.
Fixes: #542
Signed-off-by: Marco Vedovati<EMAIL_ADDRESS>

/test

done, thanks @jodh-intel

/retest

/retest

/test

@marcov We only require at least 1.10.4 (https://github.com/kata-containers/runtime/blob/master/versions.yaml#L267). Please update the runtime repo for the least required version.

tag dnm until versions.yaml is updated in the runtime repo.

@bergwolf sure I can do that
When building the rootfs, the agent's Go version is read from a file in osbuilder, so that should also be updated to use versions.yaml in the runtime repo. Actually I am not sure if the switch may break something.

@marcov - which switch do you mean - changing osbuilder to look at the runtime's versions.yaml?
/cc @jcvenegas.

@jodh-intel correct, it would require checking the versions.yaml file from the runtime repo instead of a local file in osbuilder. And I am not sure if having osbuilder depend on runtime repo (or on something in that repo), may pose a problem.

I don't think it will cause problems, but then if not why did we create the separate versions.txt? Since osbuilder only needs that single file, we can probably do what the packaging repo does:

https://github.com/kata-containers/packaging/blob/master/Makefile#L41

In effect the golang binary is downloaded using curl, so that means that both curl and network access should be available.

kata-containers/runtime#1693 is merged


b2d7621645d956d29a934b50dbba97319b33820a
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
b2d7621645d956d29a934b50dbba97319b33820a
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
722234730372f1f1737d34c74888fbcdb98e300a
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
0eb31c3e12163b597d3fceeae43df81faf6499eb
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
0eb31c3e12163b597d3fceeae43df81faf6499eb
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
cca65bb2fa2fd7cf6ec46751687e9c0187923b1d
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
cca65bb2fa2fd7cf6ec46751687e9c0187923b1d
link
/test integ-multicluster-k8s-tests_istio

lint_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
cca65bb2fa2fd7cf6ec46751687e9c0187923b1d
link
/test integ-multicluster-k8s-tests_istio

lint_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test lint_istio

gencheck_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
cca65bb2fa2fd7cf6ec46751687e9c0187923b1d
link
/test integ-multicluster-k8s-tests_istio

lint_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test lint_istio

gencheck_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test gencheck_istio

unit-tests_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test lint_istio

gencheck_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test gencheck_istio

unit-tests_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test gencheck_istio

unit-tests_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test integ-multicluster-k8s-tests_istio

lint_istio
122846c3c3d3fa27b6f6bb956db3c2786ddf3838
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
a4fe3f0e3cc36091429c311010cc16de0fcff653
link
/test integ-multicluster-k8s-tests_istio

lint_istio
122846c3c3d3fa27b6f6bb956db3c2786ddf3838
link
/test lint_istio

gencheck_istio
122846c3c3d3fa27b6f6bb956db3c2786ddf3838
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
122846c3c3d3fa27b6f6bb956db3c2786ddf3838
link
/test lint_istio

gencheck_istio
122846c3c3d3fa27b6f6bb956db3c2786ddf3838
link
/test gencheck_istio

integ-multicluster-k8s-tests_istio
122846c3c3d3fa27b6f6bb956db3c2786ddf3838
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
215b12056287002e090c2de66e3a9c0f2d81a029
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


optimize processor layout (2d)

at the same time, look at profiling and scaling

PR #316 partially addressed some of this

2-d layout has been implemented (after removing bottleneck of allowing processors to have no tiles, #316).
Runtime has not yet been improved noticeably using the 2d layout.
Closing this issue.


Add Metrics.Net IMetricsClient

Adds Metrics.Net support (http://www.nuget.org/packages/Metrics.NET/0.2.16). I
find it more feature complete and it supports multiple metrics backends.
Note: It needs to be configured on startup
// Metric.Config
// .WithHttpEndpoint("http://localhost:1234/metrics/")
// .WithAllCounters()
// .WithInternalMetrics()
// .WithReporting(config => config
//     .WithConsoleReport(TimeSpan.FromSeconds(30))
// .WithCSVReports(@"c:\temp\reports", TimeSpan.FromSeconds(10))
// .WithTextFileReport(@"C:\temp\reports\metrics.txt",
TimeSpan.FromSeconds(10))
// .WithGraphite(new Uri("net.udp://localhost:2003"),
TimeSpan.FromSeconds(1))
// .WithInfluxDb("<IP_ADDRESS>", 8086, "admin", "admin", "metrics",
TimeSpan.FromSeconds(1))
// .WithElasticSearch("<IP_ADDRESS>", 9200, "metrics",
TimeSpan.FromSeconds(1))
// );

Thanks for your changes. I just noticed it was in the root folder and just did a commit to move it into it's own folder.

I actually just noticed that myself. Was gonna mention it.
Welcome. Glad to help. Very useful library.

I just renamed the metrics client with the proper casing :). It will be available here: https://www.nuget.org/packages/Foundatio.MetricsNET/


No feedback on construct query errors

There are cases where the query is malformed but there is no error reporting.
Example:

result = engine.construct(query="constru ct * where { service <x-sparql-anything:> { ?s ?p ?o } }")
print(result)

Shows an empty rdflib graph.

Probably the solution to #27 would also solve this issue.


Adding @@ignore to a model that references an enum fails to generate enhanced client

Description and expected behavior
Ignoring a model with @@ignore causes the enhanced client generation to fail if said model references an enum.
For example, this schema.zmodel:
datasource db {
  provider          = "postgresql"
  url               = env("DATABASE_URL")
}

generator js {
  provider = "prisma-client-js"
  output   = "../node_modules/@prisma-app/client"
}

enum Enum {
    SOME_VALUE
    ANOTHER_VALUE
}

model Model {
    id String @id @default(cuid())
    value Enum
    @@ignore
}

model AnotherModel {
    id String @id @default(cuid())
}

Causes the following error when running zenstack generate:
⌛ ZenStack CLI v2.8.0, running plugins
✔ Generating Prisma schema
✔ Generating PrismaClient enhancer
Error compiling generated code:
node_modules/.zenstack/policy.ts:5:10 - error TS2305: Module '"../@prisma-app/client"' has no exported member 'Enum'.

5 import { Enum } from "../@prisma-app/client";
           ~~~~

: Error compiling generated code
error: "zenstack" exited with code 1

Screenshots
N/A.
Environment (please complete the following information):

ZenStack version: 2.8.0
Prisma version: 5.18.0
Database type: PostgreSQL

Additional context
We're currently investigating a better process for dropping tables/columns with no downtime. Part of this process uses @ignore/@@ignore to first remove the table/column reference from the Prisma client, and then, in a subsquent work, we actually drop the table/column. While doing that, we stumbled upon this issue where, if we ignore a model with an enum, we can't generate the enhanced client due to the compilation error sent up there.
Prisma doesn't seem to support @ignore for enums (https://www.prisma.io/docs/orm/prisma-schema/data-model/models#defining-enums), so we weren't able to explore that.

Hi @wmartins , the cause is Prisma's ts code generation doesn't generate an Enum type if it's not referenced by any model. I think a temp workaround could be to create a temp model to keep the reference (but exclude it from migration), and remove it afterwards.
I'll see how we can detect this situation in zenstack to avoid the error.

Hi @ymc9 thanks for looking into it and taking the time to respond!
I think the temp workaround could work, I'll try that next time we need it. Hopefully we can get it sorted in a new version.

Fixed in 2.9.0


Discuss changing interview_metadata to al_metadata

The path dependencies on this one just..don't seem worth it at this point. Closing for now.


[adobepass] Add MSOs

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

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

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

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

Description of your issue, suggested solution and other information
I have a question before I submit any pull requests. I have some proposed changes for adobepass, but I wondered if it was better to add one MSO at a time and wait for testing, than to add several MSOs  at once. In other words, should I submit separate pull requests as they're approved, even though it's one file being updated? I think I've found a way to make all MSOs work as long as they are in the dict, but have no way of testing more than one.

That depends. The general rule is to have a separate PR for a separate functionality or a feature.


Filtering values based on null /empty field from a JSON

Is it possible to search based on a value for being null or empty ?
in other words when a value is not being set all and want to filter out null values
for example  I would like to get all the people(first name) who do not have a last name
like this
name := gjson.Get(json, programmers.#(lastName="").firstName)
tried null.nil , ""  does not work
Thanks
Pt

Try programmers.#(lastName==).firstName (must be in `)

Seems null not handled at all.

@anair-stack did you find any workaround?

I'm struggling to query an empty array. Any idea how this could work?

I'm trying to query an empty array.
This seems to work:

This doesn't:

I would expect [["Empty"]]

This works: #.friends.#(nets.# == [])#.first

Another way is to use the ~ operator, which converts a value to a boolean before comparison. There's some information on how it works at the bottom of the section about queries
friends.#(nets.#==~true)#.first

nets.# gets the number of values in the  nets array.
==~true auto converts that value to either true or false and does the comparison.
What's nice about this approach is that the ~ handle non-existent and null values as false.
Take this JSON for example:
{
  "name": {"first": "Tom", "last": "Anderson"},
  "age":37,
  "children": ["Sara","Alex","Jack"],
  "fav.movie": "Deer Hunter",
  "friends": [
    {"first": "Dale", "last": "Murphy", "age": 44, "nets": ["ig", "fb", "tw"]},
    {"first": "Roger", "last": "Craig", "age": 68, "nets": ["fb", "tw"]},
    {"first": "Empty", "last": "Craig", "age": 68, "nets": []},
    {"first": "Missing", "last": "Craig", "age": 68},
    {"first": "Jane", "last": "Murphy", "age": 47, "nets": ["ig", "tw"]}
  ]
}

With the tilde:
friends.#(nets.#==~true)#.first     =>  ["Dale","Roger","Jane"]
friends.#(nets.#==~false)#.first    =>  ["Empty","Missing"]

Without the tilde, the nets must be an actual array. Otherwise it can be compared to anything because net.# is considered non-existent.
friends.#(nets.#==[])#.first        =>  ["Empty"]
friends.#(nets.#==0)#.first         =>  ["Empty"]
friends.#(nets.#>0)#.first          =>  ["Dale","Roger","Jane"]

But if what you want it only return those that explicitly define nets as an array:
friends.#(nets.#==0)#.first         =>  ["Empty"]
friends.#(nets.#!=0)#.first         =>  ["Dale","Roger","Jane"]


Update package.json

@maritome, @MohamadKh75
Please, look at this PR.
The ms-teams-webhook version under 2.0.x is not compatible with the new Microsoft Teams Incoming Webhooks.

Would be nice to have this merged!


Add WireGuard tunnels for Multi-cluster traffic

Add WireGuard tunnels mode for Multi-cluster traffic. WireGuard tunnels will be established between member clusters and responsible for all cross-cluster traffic in this mode.
Implementation:

When a Gateway is enqueued, the controller will initialize the WireGuard interface and route on the host. We use the first IP  address of the Service CIDR  of the cluster as the WireGuard interface's IP. For example, if the ServiceCIDR is <IP_ADDRESS>/24, then the WireGuard interface's IP address is <IP_ADDRESS>.
When a ClusterInfoImport is enqueued, the controller will try to update the WireGuard peer based on the ClusterInfoImport.

Second what @hongliangl said. Please add description of how traffic is routed, and why we must do policy based routing. @hjiajing @luolanzone

/test-multicluster-e2e

Hi Jianjun @jianjuns. On Friday Quan, Lan, and Hongliang had discuss the IP rule and firewall mark. Quan recommended me to use the first IP address of the Node Pod CIDR as the WireGuard interface IP address so that we could avoid using PKT_MARK in open flows.  So I updated the code. What's more, in order to avoid the potential race condition @luolanzone met before with two RateLimitQueue, I use one RateLimitQueue and distinguish ClusterInfoImport and Gateway with the key.  Could you please help to review it? Thanks.

/test-multicluster-e2e

/test-multicluster-e2e

/test-multicluster-e2e

/test-multicluster-e2e

@jianjuns

I do not mean to add the func to the netlink interface, but as a standalone func in netlink_linux.go

The LinkExists func is based on LinkByName, which is defined in netlink_linux.go. So maybe we cannot define another function using LinkByName in  netlink_linux.go.  Another way is defining the LinkExists in pkg/agent/route, but that package is all about route not link. I'm sure if it's reasonable to add a new link method.

I do not mean to add the func to the netlink interface, but as a standalone func in netlink_linux.go

The LinkExists func is based on LinkByName, which is defined in netlink_linux.go. So maybe we cannot define another function using LinkByName in netlink_linux.go. Another way is defining the LinkExists in pkg/agent/route, but that package is all about route not link. I'm not sure if it's reasonable to add a new link method.

@hjiajing LinkByName in netlink_linux.go is used for testing only? If you add the LinkExists func, it should call  LinkByName from github.com/vishvananda/netlink.

I do not mean to add the func to the netlink interface, but as a standalone func in netlink_linux.go

The LinkExists func is based on LinkByName, which is defined in netlink_linux.go. So maybe we cannot define another function using LinkByName in netlink_linux.go. Another way is defining the LinkExists in pkg/agent/route, but that package is all about route not link. I'm not sure if it's reasonable to add a new link method.

@hjiajing LinkByName in netlink_linux.go is used for testing only? If you add the LinkExists func, it should call LinkByName from github.com/vishvananda/netlink.

Done. Add a new function LinkExists using  netlink.LinkByName in netlink_linux.go now.

@hjiajing the dependent PR has been merged, this PR can rebase on main now.

@tnqn Thanks for the reminder, rebased

@hjiajing I think you didn't re-base it correctly, I still see two commits.

@luolanzone Fixed, there's only one commit now.

@hjiajing don't forget to update the commit message in next push with some helpful details for others to understand the commit in the future.

@jianjuns Move Currently encryption only works with encap mode. to the last of the sentence now.

I guess it returns "invalid argument" because you didn't set link.Index. And client already caches the interface index, I think we can pass it? Then the pre-check is not needed.

Hi Quan. I tried to pass the link.Index to the LinkDel function, it returned no such device. Now I'm not sure which one is better, just using LinkDel and check the error message with strings.Contain or using LinkByName first and checking the error type (The netlink package wraps "not found" error when in LinkByName, but not in Linkdel).

@hjiajing could you re-base and resolve the conflicts?

@luolanzone Sure, I rebased in local and doing some verification now. I will push the latest code later.

Hi @jianjuns @luolanzone , I have rebased to the main branch and used Gateway's ServiceCIDR instead of serviceCIDRProvider, could you please help to review when you're free? Thanks.

@hjiajing please ignore me approval, not sure how it's selected.

@hjiajing please resolve the code conflicts.

@jianjuns @tnqn do you have any further comments?

/test-all

@hjiajing could you check if possible to improve the patch test coverage?

@hjiajing please resolve the file conflict.

/test-all

/test-e2e
/test-multicluster-e2e
/test-networkpolicy

@tnqns Hi Quan. I have addressed the comments you left, but after I pushed the latest code, some comments disappeared both in the conversation and Files changed, so I didn't reply every comment you left. Could you please take a look? Thanks.

@jianjuns @luolanzone do you have other comments for this one?

/test-multicluster-e2e
/test-all

/test-multicluster-e2e


error while testing vue with mochapack

Laravel Mix Version: 5.0.1
Node Version: 12.18.2
NPM Version : 6.14.6
OS: Windows 8.1

Description:
Hello, I am trying to test vue components, but I got this error:
node_modules\laravel-mix\src\Manifest.js:76
File.find(this.path())
^
TypeError: File.find is not a function
at Manifest.refresh
Steps To Reproduce:
npm test
this is my package.json

{
    "private": true,
    "scripts": {
        "dev": "npm run development",
        "development": "cross-env NODE_ENV=development node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
        "watch": "npm run development -- --watch",
        "watch-poll": "npm run watch -- --watch-poll",
        "hot": "cross-env NODE_ENV=development node_modules/webpack-dev-server/bin/webpack-dev-server.js --inline --hot --disable-host-check --config=node_modules/laravel-mix/setup/webpack.config.js",
        "prod": "npm run production",
        "production": "cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --no-progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
        "test": "mochapack --webpack-config=node_modules/laravel-mix/setup/webpack.config.js --require tests/js/setup.js tests/js/\\*\\*/\\*.spec.js"
    },
    "devDependencies": {
        "@vue/test-utils": "^1.1.1",
        "axios": "^0.19",
        "bootstrap": "^4.0.0",
        "cross-env": "^7.0",
        "expect": "^26.6.2",
        "jquery": "^3.3.1",
        "jsdom": "^16.4.0",
        "jsdom-global": "^3.0.2",
        "laravel-mix": "^5.0.1",
        "lodash": "^4.17.19",
        "mocha": "^8.2.1",
        "mochapack": "^2.0.6",
        "popper.js": "^1.12",
        "resolve-url-loader": "^2.3.1",
        "sass": "^1.20.1",
        "sass-loader": "^8.0.0",
        "vue": "^2.5.17",
        "vue-template-compiler": "^2.6.10",
        "vuex": "^3.3.0"
    }
}

What can I do? thank you.

@lupanvi I had the same problem too. I think the jsdom-global is setting a global File object, which doesn't have a find method. I've temporally solved by adding this to my setup.js file
const file = require('laravel-mix/src/File')

require('jsdom-global')();
global.File = file;

I think the proper solution would be adding a local variable File in the Manifest.js file. I was looking into the codebase, and it is modified in the v6.0.0-beta.14 tag. I'll create a PR later to fix this issue.

Confirmed, and @lukzgois fix solved the issue.

It works, thank you.


token search modal

on the token search modal, some tokens have long name that are not supported and are currently cut.
please handle in this way:

increase supported token name length to 25 characters
IF token name is GREATER than 25,

hide the last 3 characters of the token name
replace them with ...
implement text overflow ellipsis marquee
(https://stackoverflow.com/questions/41987383/how-to-make-marquee-text-only-when-its-overflowing)

This is a limitation of the token list, we can not increase it to 25 because the token list returns names with max 19 characters.
I can add the ... to any token that has a length of 19 characters instead.


Draft website content

What needs to be done
Draft content for website
Why it needs to be done
Website needs to have basic information

Pages configured and finished following the draft guide created in figma.
Pending revision of details
https://web3partners.community/

https://bogota.web3partners.community/


Introns by transcript overwrites gene_id with transcript_id

Currently gr.features.introns(by="transcript") results in the gene_id column being overwritten by transcript_id

Thanks for reporting! I rely on user feedback to make pyranges better. Will try to fix tomorrow 👍🏻

Should be fixed in 0.0.79. Thanks for reporting :)


Navigation slow on development

It seems like the navigation is a bit slow when we're in development mode.
Why is it?
How to reproduce

Clone this repo
yarn dev
Try click one of the post. It will take a few seconds before new page loaded

This is expected in dev mode and inherent to the way Next.js works. When you watch your console output, you'll see a message wait - compiling..., so the site is re-generating. This allows for hot-reloading which is what most people want in dev mode. In addition to the initial compile step, every static page is also re-rendered on every request in development. Your actual slowness, however, also depends on the capabilities of your development environment (2 seconds is normal on fast machines for the initial request). Most important: Sites are blazing fast in production.

Closed as answered. As always, this issue can be re-opened on request.


Support customized tags from request header for tracing

Describe the feature request
Envoy v2 API supports request_headers_for_tags, which can be used to create tags for the active span from request headers, but looks look like Istio currently doesn't support to generate such envoy configuration.
However, this feature is useful if the operators expect to differentiate tracing data from different application.
Describe alternatives you've considered
Expect Istio to support new API or enhance current tracing API to support envoy request_headers_for_tags configuration.
/cc @linsun @gyliu513 @clyang82

@douglas-reid This is another area to consider for the tracing API - as it would tend to be service specific.

@douglas-reid Any thought?

+1
I've found just a similar request for this issue within Kiali.
From Kiali, I want to create a matching rule based in a header that might be or not be part of a span.
(As developer could not add it).
But as it's used inside a VirtualService I'd like to see it in Jaeger, to also correlate it in Kiali with other metrics.
So, yeah, something where headers can be added to a span via configuration would be a great feature.

Is there any possible we have this in istio 1.1.x?

@morvencao Can this feature meet your requirements? https://istio.io/docs/reference/config/networking/v1alpha3/virtual-service/#Headers
Here is example vs configuration for productpage:
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  creationTimestamp: 2019-04-24T12:56:37Z
  generation: 1
  name: bookinfo
  namespace: default
  resourceVersion: "8291608"
  selfLink: /apis/networking.istio.io/v1alpha3/namespaces/default/virtualservices/bookinfo
  uid: 65639705-6690-11e9-9063-00163e019bbe
spec:
  gateways:
  - bookinfo-gateway
  hosts:
  - '*'
  http:
  - match:
    - uri:
        exact: /productpage
    - uri:
        exact: /login
    - uri:
        exact: /logout
    - uri:
        prefix: /api/v1/products
    route:
    - destination:
        host: productpage
        port:
          number: 9080
      headers:
        request:
          add:
            x-team: develop

from the envoy log, you can see 'x-team', 'develop' was added.
[2019-04-26 03:40:54.556][24][debug][router] [external/envoy/source/common/router/router.cc:381] [C2][S17715858181031491984] router decoding headers:
':authority', 'cyangadd1.fyre.ibm.com:31380'
':path', '/productpage'
':method', 'GET'
':scheme', 'http'
'cache-control', 'max-age=0'
'upgrade-insecure-requests', '1'
'user-agent', 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/71.0.3578.98 Safari/537.36'
'accept', 'text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8'
'accept-encoding', 'gzip, deflate'
'accept-language', 'en-US,en;q=0.9'
'x-forwarded-for', '<IP_ADDRESS>'
'x-forwarded-proto', 'http'
'x-request-id', '4d7f5c28-75b9-4c4d-971b-7bd02c1d0f1d'
'x-team', 'develop'
'content-length', '0'
'x-envoy-internal', 'true'
'x-b3-traceid', '850e57ea05403b2af0130b2de960ff30'
'x-b3-spanid', 'fd59bf17855398c3'
'x-b3-parentspanid', 'f0130b2de960ff30'
'x-b3-sampled', '0'

@clyang82 Header manipulation rules help us to manipulate headers in http request/response to/from destination service.
We need something beyond that, more specifically, we need to customize jaeger tags from headers in http request/response to/from destination service.
We need to update the Istio api to implement this feature.

There is already a proposal by @douglas-reid to add tracing control(including tagsFromHeaders) in Sidecar API, hopefully we will have this feature in next release.

Work is currently underway in Envoy to enable custom tags to be created from a variety of sources, including from request headers (which was supported before) but now having the ability to use a different name for the tag. Once this lands, it can be used via the EnvoyFilter initially, and then eventually through an appropriate Istio API.

Not stale.

Related to https://github.com/istio/istio/issues/22457
@morvencao for the time being you can use the envoy capability until we have a better way of supporting it in istio (see the link above).

thanks @jacob-delgado

Verified