Some bee specialties are produced ten times too often

If you look back at commit 03068d7ddbac45ee49f7ab067138e5e886c0becf in the code history, you'll see that when MysteriousAges updated the MagicBees code from Forestry 3.4 to 3.6, he updated a lot of the bee-products code to use floating-point numbers instead of integers for the chance of producing a given product. (You might have to click on the "Load Diff" link to see the BeeProductHelper.java diff, since it's a large enough diff that GitHub won't display it by default). Previously, the Forestry API specified production chances as integers that represent a percentage (from 0 to 100), and after the 3.6 chances, production chances were floats between 0.0 and 1.0 instead. However, some of the updated values were incorrect: the integer 8 (representing 8%) in some cases became 0.8 (80%) when it should have been 0.08 (8%), so some specialty products ended up being produced ten times as often as intended. For example, here's the before-and-after part of the diff for the Arcane branch of bees:
// Before:
		ESOTERIC.addProduct(Config.combs.getStackForType(CombType.OCCULT), 18);
		MYSTERIOUS.addProduct(Config.combs.getStackForType(CombType.OCCULT), 20);
		ARCANE.addProduct(Config.combs.getStackForType(CombType.OCCULT), 25)
			.addSpecialty(Config.drops.getStackForType(DropType.ENCHANTED, 1), 9);

// After:
		ESOTERIC.addProduct(Config.combs.getStackForType(CombType.OCCULT), 0.18f);
		MYSTERIOUS.addProduct(Config.combs.getStackForType(CombType.OCCULT), 0.20f);
		ARCANE.addProduct(Config.combs.getStackForType(CombType.OCCULT), 0.25f)
			.addSpecialty(Config.drops.getStackForType(DropType.ENCHANTED, 1), 0.9f);

Notice how Arcane bees now produce Enchanting Drops 90% of the time, instead of 9% of the time as originally intended: that final 0.9 value should have been 0.09.
I've looked back through the Git history, and the incorrect values are still there in the current commits of each of the active branches of the code:

1.11-rewrite branch (permalink)
mc-1.10 branch (permalink)
mc-1.7.10 branch (permalink)

(Note: the first link in each item above just points to the branch, so if code is added to that file in the future, the highlighted line numbers might not be correct any more. The "permalink" in each item goes to the actual commit hash, so its highlighted line number will always be correct).
You can easily figure out all the affected species by looking through the diff for commit 03068d7ddbac45ee49f7ab067138e5e886c0becf, but here's a list of affected species (and products) for your convenience:

Arcane bee: Enchanting drops went from 9% chance to 0.9 (90%) chance
Supernatural bee: Unusual pollen went from 8% chance to 0.8 (80%) chance
Watery bee: Ice blocks went from 2% chance to 0.025 (2.5%) chance - likely not a mistake
Ghastly bee: Furtive comb went from 8% chance to 0.8 (80%) chance
Ghastly bee: Ghast tears went from 2% chance to 0.099 (9.9%) chance - likely not a mistake
Doctoral bee: Jelly babies went from 7% chance to 0.078 (7.8%) chance - likely not a mistake
Diamond bee: Diamond shards went from 6% chance to 0.6 (60%) chance
Emerald bee: Emerald shards went from 4% chance to 0.4 (40%) chance
Transmuting bee: Silky comb went from 5% chance to 0.5 (50%) chance
Transmuting bee: Simmering comb went from 5% chance to 0.5 (50%) chance
Certus bee: Certus quartz went from 8% chance to 0.8 (80%) chance
Fluix bee: Fluix crystal went from 6% chance to 0.6 (60%) chance
Brainy bee: Rotten flesh went from 6% chance to 0.6 (60%) chance
Brainy bee: Zombie brain went from 2% chance to 0.2 (20%) chance
Batty bee: Gunpowder went from 4% chance to 0.4 (40%) chance
Wispy bee: Silk wisp went from 4% chance to 0.4 (40%) chance
Scholarly bee: Lore fragment went from 2% chance to 0.2 (20%) chance
Savant bee: Lore fragment went from 5% chance to 0.5 (50%) chance
Poultry bee: Chicken nuggets went from 9% chance to 0.9 (90%) chance
Beefy bee: Beef nuggets went from 9% chance to 0.9 (90%) chance
Porcine bee: Pork nuggets went from 9% chance to 0.9 (90%) chance
Minium bee: Minium shard went from 6% chance to 0.6 (60%) chance
Quintessential bee: Quintessence went from 5% chance to 0.5 (50%) chance
Terrestric (AM Earth) bee: Earth essence went from 7% chance to 0.7 (70%) chance
Aiolic (AM Air) bee: Air essence went from 7% chance to 0.7 (70%) chance
Igneous (AM Fire) bee: Fire essence went from 7% chance to 0.7 (70%) chance
Aqueous (AM Water) bee: Water essence went from 7% chance to 0.7 (70%) chance
Fulguric (AM Lightning) bee: Lightning essence went from 7% chance to 0.7 (70%) chance
Fruticic (AM Plant) bee: Plant essence went from 7% chance to 0.7 (70%) chance
Glacic (AM Ice) bee: Ice essence went from 7% chance to 0.7 (70%) chance
Blizzy bee: Blizz powder went from 9% chance to 0.9 (90%) chance
Gelid bee: Cryotheum dust went from 9% chance to 0.9 (90%) chance
Dante bee: Blaze powder went from 5% chance to 0.5 (50%) chance
Dante bee: Sulfur went from 9% chance to 0.9 (90%) chance
Pyro bee: Blaze powder went from 5% chance to 0.5 (50%) chance
Pyro bee: Pyrotheum dust went from 9% chance to 0.9 (90%) chance
Destabilized bee: Redstone went from 5% chance to 0.5 (50%) chance
Lux bee: Glowstone went from 5% chance to 0.5 (50%) chance
Winsome bee: Endearing comb went from 5% chance to 0.5 (50%) chance
Winsome bee: Ender pearl went from 5% chance to 0.5 (50%) chance
Winsome bee: Platinum dust went from 9% chance to 0.9 (90%) chance
Endearing bee: Endearing comb went from 5% chance to 0.5 (50%) chance
Endearing bee: Enderium nugget went from 9% chance to 0.9 (90%) chance
Fluxed bee: Flux nugget went from 9% chance to 0.9 (90%) chance
Carbon bee: Carbon comb went from 5% chance to 0.5 (50%) chance
Carbon bee: Coal went from 5% chance to 0.5 (50%) chance
Bloody bee: Sanguine comb went from 8% chance to 0.8 (80%) chance
Bound bee: Sanguine comb went from 8% chance to 0.8 (80%) chance
Botanic bee: Transmuted comb went from 5% chance to 0.5 (50%) chance
Botanic bee: Petals went from 1% chance to 0.1 (10%) chance
Blossom bee: Transmuted comb went from 5% chance to 0.5 (50%) chance
Blossom bee: Petals went from 4% chance to 0.4 (40%) chance
Floral bee: Transmuted comb went from 5% chance to 0.5 (50%) chance
Vazbee bee: Soul comb went from 5% chance to 0.5 (50%) chance
Vazbee bee: Wool went from 2% chance to 0.2 (20%) chance
Vazbee bee: Red flower went from 6% chance to 0.6 (60%) chance
Vazbee bee: Pasture seeds went from 4% chance to 0.4 (40%) chance
Somnolent bee: Watery comb went from 8% chance to 0.8 (80%) chance
Skystone bee: Sky stone went from 2% chance to 0.2 (20%) chance

It's actually pretty easy to find the incorrect values, if your text editor / IDE has a regex search feature. Just look for the pattern 0.#, where # is a single digit (i.e., a \d that is not immediately followed by a second \d), i.e. use a "negative lookahead assertion" to say that the \d should not be followed by a second one. Since different regular-expression syntaxes write negative lookahead assertions differently, I can't give an example that will work in every IDE. But it should be pretty straightforward to write such a search expression.

Thanks for the report!
Would you mind making a PR? You should be able to edit the values in Github directly and submit a PR with your changes.

I'm at work right now, but I'll make a PR when I get home.
Note that it'll really be three PRs, since GitHub PRs only target a single branch and I want to fix this in all the active branches.

Fixed for the 1.10/1.11 version, the 1.10 branch is not active, so no need to make a pull request for that

Oh dear. I have a PR almost ready to push, and it conflicts with yours. I'm just now double-checking everything to make sure I got it right; I'll push it real soon, and we can work out the differences once it's pushed.

Okay, #19 is the one that conflicts. Most of them are trivial conflicts, e.g. where my code has 0.10 and your code has 0.1 (I changed every percentage number to two digits so that nobody would possibly wonder, later, if it was supposed to be 10% or 1%). But some of them are actual conflicts:

Apatite bee, frequency of apatite as a product. Old number was 10% (10 as an int, which became 0.1 correctly), and someone had changed it to 20% (0.2) at some point. I changed it to 0.20 to keep the same values, you changed it to 0.05 to reduce the frequency of apatite. Either one works, but someone who's not me should decide which of these values to keep.

Carbon bee (TE_COAL enum), frequency of carbon combs. Old number from the int-to-float change was 5%, but that had become 0.1 (10%) at some point before my patch. I just changed the number to 0.10 to keep it at 10% since I figured that was a deliberate change at some point in the past, but then the patch you just created has that number at 0.05 (5%) to match the frequency of coal. Either one, 10% or 5%, is fine, but again, someone who's not me needs to decide which of those values to keep.

Endearing bee, frequency of ender pearls. Here you kept the 0.5 (50%) value, whereas I changed it to 0.05 (5%). Looking back at commit 03068d7ddbac45ee49f7ab067138e5e886c0becf, the frequency of ender pearls was 5% (and they were, for some odd reason that I think was a bug, a product of winsome bees instead of endearing bees). The fact that they're now produced by endearing bees is, I think, correct, but here I'm pretty sure that the 0.05 value is correct, and the 0.5 value is wrong.

Botanic bees, frequency of petals. Here your number is 0.10 and mine is 0.01, and I'm pretty sure 0.01 is correct: the old numbers were 1 for botanic bees, 4 for blossom bees, and 16 for floral bees. So the float value for botanic should be 0.01, blossom should be 0.04, and floral should be 0.16 so that there's a proper improvement in the chance as you move down the breeding line.

That's it, I think. All the rest of the differences I found were trivial.

Apatite: 0.10, that sounds like a balanced number to me
Carbon: Keep at 0.05, since that was the initial intended value (5%)
Endearing speciality: I missed that one, whoops, good catch
Botanic: Is already at 0.01 https://github.com/ForestryMC/MagicBees/blob/1.10/1.11-rewrite/src/main/java/magicbees/bees/EnumBeeSpecies.java#L1896
If there are no significant changes (except the wrong value for the Endearing speciality), it might
be easier to not merge it, as there are 14 conflicts

No worries. :-) Now that there's been a decision, I'll rebase my commit on top of the current branch and push the rebased commit to the PR branch, and the PR will still be mergeable. (I write C# for my day job and I'm very familiar with Git, so that's not a problem at all for me).
I do think it would be a good idea to merge the PR eventually, since I've made a few more improvements like putting 0.10 / 0.20 / etc everywhere where the values 0.1 / 0.2 / etc were correct. That way there will never again be confusion over whether the 0.1 was supposed to be 10% or 1%. But I'm running out of time for non-work stuff today, so I'll have to wait until tomorrow to rebase this particular PR.

I've pushed a rebased version of #19 now, and it's mergeable. Its only remaining changes are the apatite, carbon and endearing chances, but I also kept in all the places where I changed 0.1 to 0.10, 0.2 to 0.20, and so on. That will allow anyone who reads the code later on to know that those were the intended values.
#19 should be ready to be merged now.


how to prepare data for mxnet.io.NDArrayIter

Hi, there,
I am a little confused about the usage of mxnet.io.NDArrayIter. I have read the documentation here.http://mxnet.readthedocs.org/en/latest/python/io.html#io-api-reference. It suggested to call it in this way:
mxnet.io.NDArrayIter(data, label=data_label, batch_size=1, shuffle=False, last_batch_handle='pad')
How should we store the training data in an array data and its label in an array data_label.
Specifically, I want to train a Lenet for MNIST. So, I get a numpy array of size (60000,28,28) and another one of size (60000,1) for labels. How can I feed them to a mxnet model for training?

reshape to (60000, 784) and (60000, 1)

thanks for your reply @piiswrong ! However, it shows this error after reshipped to (60000,784) and (60000,1)
Check failed: (dshape.ndim()) == (4) Input data should be 4D in batch-num_filter-y-x
Should we prepare a 4D array for the training data?

if you are feeding to conv layers, reshape to (60000, 1, 28, 28)

Thanks for your suggestion! @piiswrong
I tried to reshape the data array to (60000,1,28,28) and label array to (60000,1). And called this command to do the data load:
train=mx.io.NDArrayIter(data,label=data_label,batch_size=100)
And called this command to start the training:
model.fit(X=train)
But it returns this error:
Shape inconsistent, Provided = (100,1), Inferred shape=(100)
Also, I tried to
Do you have any idea about how to solve this? Thanks!

reshape label to (60000,)

I tried that too. But it didn't work. It reports:
mxnet.base.MXNetError: [09:43:02] include/mxnet/ndarray.h:230: Check failed: (shape_[0]) >= (end) Slice end index out of range
I also tried to reshape the label matrix to (60000,10). It reported a similar error as the previous one:
Shape inconsistent, Provided =(100, 10), inferred shape=(100)

I'll soon submit a fix for this. For now, try in softmax_output-inl.h:148, replace Shape1(dshape[0]) with Shape2(dshape[0], 1) and use label shape (60000, 1)

actually, this code works for me without any change:
data = mx.symbol.Variable('data')
fc1 = mx.symbol.FullyConnected(data = data, name='fc1', num_hidden=128)
act1 = mx.symbol.Activation(data = fc1, name='relu1', act_type="relu")
fc2 = mx.symbol.FullyConnected(data = act1, name = 'fc2', num_hidden = 64)
act2 = mx.symbol.Activation(data = fc2, name='relu2', act_type="relu")
fc3 = mx.symbol.FullyConnected(data = act2, name='fc3', num_hidden=10)
mlp = mx.symbol.SoftmaxOutput(data = fc3, name = 'softmax')
#mysoftmax = NumpySoftmax()
#mlp = mysoftmax(data=fc3, name = 'softmax')

# data

#train, val = mnist_iterator(batch_size=100, input_shape = (784,))
train=mx.io.NDArrayIter(np.zeros((60000, 784)),label=np.zeros((60000,)),batch_size=100)

# train

logging.basicConfig(level=logging.DEBUG)

model = mx.model.FeedForward(
    ctx = mx.cpu(), symbol = mlp, num_epoch = 20,
    learning_rate = 0.1, momentum = 0.9, wd = 0.00001)

model.fit(X=train)

I tried a MLP model and it works fine. But problems arise in conv models. After replacing Shape1(dshape[0]) with Shape2(dshape[0], 1) in softmax_output-inl.h:148, do I need to re-compile mxnet to make it work?

You need to recompile. That's weird though. What's the full message for this error:
mxnet.base.MXNetError: [09:43:02] include/mxnet/ndarray.h:230: Check failed: (shape_[0]) >= (end) Slice end index out of range

The full error message is:
[17:20:15] ./dmlc-core/include/dmlc/logging.h:215: [17:20:15] include/mxnet/ndarray.h:230: Check failed: (shape_[0]) >= (end) Slice end index out of range
Traceback (most recent call last):
File "", line 1, in
File "mnist.py", line 87, in
batch_end_callback=mx.callback.Speedometer(batch_size))
File "/auto/rcf-proj2/qh/qianguih/mxnet/mxnet_multi/mxnet/python/mxnet/model.py", line 656, in fit
logger=logger, work_load_list=work_load_list, monitor=monitor)
File "/auto/rcf-proj2/qh/qianguih/mxnet/mxnet_multi/mxnet/python/mxnet/model.py", line 276, in _train_multi_device
executor_manager.load_data_batch(eval_batch)
File "/auto/rcf-proj2/qh/qianguih/mxnet/mxnet_multi/mxnet/python/mxnet/executor.py", line 424, in load_data_batch
_load_data(data_batch, self.data_arrays)
File "/auto/rcf-proj2/qh/qianguih/mxnet/mxnet_multi/mxnet/python/mxnet/executor.py", line 305, in _load_data
_load_general(batch.data, targets)
File "/auto/rcf-proj2/qh/qianguih/mxnet/mxnet_multi/mxnet/python/mxnet/executor.py", line 301, in _load_general
d_src[slice_idx].copyto(d_dst)
File "/auto/rcf-proj2/qh/qianguih/mxnet/mxnet_multi/mxnet/python/mxnet/ndarray.py", line 236, in getitem
return self._slice(in_slice.start, in_slice.stop)
File "/auto/rcf-proj2/qh/qianguih/mxnet/mxnet_multi/mxnet/python/mxnet/ndarray.py", line 276, in _slice
self.handle, start, stop, ctypes.byref(handle)))
File "/auto/rcf-proj2/qh/qianguih/mxnet/mxnet_multi/mxnet/python/mxnet/base.py", line 77, in check_call
raise MXNetError(py_str(LIB.MXGetLastError()))
mxnet.base.MXNetError: [17:20:15] include/mxnet/ndarray.h:230: Check failed: (shape[0]) >= (end) Slice end index out of range
BTW, here is my codes.
mnist.txt

Hi, @piiswrong , is it possible to feed 2d label array for mxnet in training? Basically, I get a data array with size (80000,224,224) and a label array with size (80000,1000). The thing is we can not reshape label array to (80000,1) because there may be multi "1" in each row, such as (0,1,0,0,1,0). Can we just feed the (80000,1000) label array to mxnet model using mxnet.io.NDArrayIter ?

I also am confused about prepping data for MXNet.... (although this is more general)
Even the method you recommend  is only talked about in theory and has no practical demonstration as to how one might convert say a CSV file to your format.
Also while you do have CSVIter it seems way to complex. Here are two lines from the parameters:
- data_csv (string, required) – The input CSV file or a directory path.
- label_csv (string, optional, default='NULL') – The input CSV file or a directory path. If NULL, all labels will be returned as 0.

What is the difference between data_csv and label_csv besides one being optional? If I define both does the latter override the former? Why doesn't this have a delim parameter so you could just as easily handle TSV files? What if the file has a header?
=


Incorrect synopsis for char[] options

I have some password options defined like the following:
@Option(names = {"--password", "-p"}, interactive = true, echo = false, arity = "0..1", required = true)
@Getter private char[] password;

Likely due to this option being defined as an array, the command synopsis in help output and man page show this as a repeatable option:
-p[=<password>] [-p[=<password>]]...

This is confusing for users, as it makes them think they can specify multiple passwords, whereas in reality they can only specify a single password consisting of multiple characters.

I'll look into providing a PR with fix and unit test either myself or by my colleague.

Looking at this more closely, I think https://github.com/remkop/picocli/issues/1846 (treating char[] as single value type) is a better solution. I will make the changes.


Cleanup unused 3rd-party build system variables

It looks like original pion was designed for re-purposing the whole project for your custom webserver. To the contrary I am trying to use it like a 3rd-party dependency. As such, I am building pion outside of my project, then pull it in.
Now, basing a webserver on a complete webserver stack will always lead to a strong dependency on the specific framework. But at least my own code shouldn't be entangled. It makes developing pion and pulling in new stuff much easier.
That's why I propose to remove all build-related variables (macros) of 3rd-party libraries (for include/link paths) not used by pion itself - they should be pulled in by the custom projects themselves, especially now that there are C++ library/package managers available, vcpkg or conan, just to mention a few.
List of libraries:

libxml2

iconv
icu

bzip2
yajl
python
sqlite
(possibly, though required for spdy) zlib

I myself can only speak confidently about the Visual C++/MSBuild build system. It seems that other build system scripts don't "suffer" from this mess to this extent - I have only seen bzip2.
So, e.g. I would remove from build/third_party_libs_x64.props:
<LIBXML_INC>$(LIB_ROOT)\libxml2-2.9.0\include</LIBXML_INC>
<LIBXML_LIB>$(LIB_ROOT)\libxml2-2.9.0\lib\x64</LIBXML_LIB>
<LIBXML_BIN>$(LIB_ROOT)\libxml2-2.9.0\bin\x64</LIBXML_BIN>

Hi @trueqbit,
You think this should be done before making windows travis successful and publishing as is for one pass?
So people have it as they are familiar.
And then clean up?
I have an old windows box I'm trying to stand up again to better go after the travis windows build

I have seen your effort! If you intend a release it's probably good to wait with my changes (unless targeting windows with travis builds is a never-ending story :) ).

yea:-)  I got windows working on another project
https://github.com/rimmartin/saxon-node
It can be done; but the cmake stuff in this project I'm not as familiar with


Overlapping of Social Media Icons with Scroll to Top Button

Description:
The social media icons in the footer are currently overlapping with the "scroll to top" button. This overlap makes it difficult for users to access or click on the social icons when scrolling to the bottom of the page.
Expected Behavior:
The social media icons should be fully visible and clickable, without being obstructed by other elements on the page.
Proposed Solution:
The issue can be resolved by adjusting the position of icons. By making this change, the icons will remain fully accessible, and users won’t encounter any problems interacting with them.
I've added a video so you can better visualize the issue.
screen-capture.webm
If you find this issue/proposed solution suitable, please assign this to me under the labels of GSSoC and Hacktoberfest and appropriate level.

@PriyaGhosal any update on this?
would be great if you can review it once

@snehhhcodes Thanks for creating the issue in BuddyTrail!!! The issue has been assigned to you.
Before you start working on your PR, please make sure to:

⭐ Star the repository if you haven't already.
Pull the latest changes to avoid any merge conflicts.
Attach before & after screenshots in your PR for clarity.
Include the issue number in your PR description for better tracking.

Contributors will start at Level 1, and their level will increase based on the quality and quantity of their contributions. The more effort and value you bring, the faster you'll progress. Put forth your best effort to unlock new levels and opportunities!
I'm excited to see your contribution as part of GSSOC EXTENDED 2024! 🎉
Don't forget to follow me on GitHub for more updates.
Happy open-source contributing! 🚀

@PriyaGhosal hey Priya just saw the live preview of the project and your latest merged PR solved the issue i was assigned for so  i dont think so i should work on it anymore as the changes are done or should i do it ?

@PriyaGhosal  please assign it to me, I want to work on it

@PriyaGhosal please assign it to me, I want to work on it

working on it!!


Added detection for webapp-runer to support heroku-cli-deploy

This fixed a problem where the Heroku Deploy CLI plugin required the buildpacks to be in the reverse order (heroku/jvm,heroku/metrics). The resulting bad behavior was:

The heroku-metrics-agent.jar was not copied into the slug because none of the detection criteria in the bin/compile passed.
The profile.d script tried to put the heroku-metrics-agent.jar on the agent path even though it didn't exist because the detection passed.
The app crashed.

The reverse order of buildpacks fixed the problem (because it installed the JDK first) but this is the opposite of what we require for Git based deploys. That's bad.
See:

https://support.heroku.com/tickets/523050
https://support.heroku.com/tickets/522311

Let's make sure we merge https://github.com/heroku/heroku-buildpack-metrics/pull/26 first so we can test this.

Should we root all the paths in $HOME so that the implied execution directory of the script isn't a factor?

@freeformz yes, in another PR


GOBIN: undefined
toolsGopath: undefined
gopath: undefined
GOROOT:
PATH: /usr/bin:/bin:/usr/sbin:/sbin

unable to locate "go" binary in GOROOT () or PATH (/usr/bin:/bin:/usr/sbin:/sbin)
unable to locate "go" binary in GOROOT () or PATH (/usr/bin:/bin:/usr/sbin:/sbin)

When I launch VS Code by running code from a terminal session the error message does not occur and the Go: Locate Configured Go Tools command correctly finds the Go tools.
Checking configured tools....
GOBIN: undefined
toolsGopath:
gopath: /Users/dbourque/dev/golang
GOROOT: /usr/local/Cellar/go/1.15.2/libexec
PATH: [redacted]

   gocode: /Users/dbourque/dev/golang/bin/gocode installed
   gopkgs: /Users/dbourque/dev/golang/bin/gopkgs installed
   go-outline: /Users/dbourque/dev/golang/bin/go-outline installed
   go-symbols: /Users/dbourque/dev/golang/bin/go-symbols installed
   guru: /Users/dbourque/dev/golang/bin/guru installed
   gorename: /Users/dbourque/dev/golang/bin/gorename installed
   gotests: /Users/dbourque/dev/golang/bin/gotests installed
   gomodifytags: /Users/dbourque/dev/golang/bin/gomodifytags installed
   impl: /Users/dbourque/dev/golang/bin/impl installed
   fillstruct: /Users/dbourque/dev/golang/bin/fillstruct installed
   goplay: /Users/dbourque/dev/golang/bin/goplay installed
   godoctor: /Users/dbourque/dev/golang/bin/godoctor installed
   dlv: /Users/dbourque/dev/golang/bin/dlv installed
   gocode-gomod: /Users/dbourque/dev/golang/bin/gocode-gomod installed
   godef: /Users/dbourque/dev/golang/bin/godef installed
   goimports: /Users/dbourque/dev/golang/bin/goimports installed
   golangci-lint: /Users/dbourque/dev/golang/bin/golangci-lint installed
   gopls: /Users/dbourque/dev/golang/bin/gopls installed

go env
Workspace Folder (tryitout): /Users/dbourque/dev/tryitout
	GO111MODULE=""
	GOARCH="amd64"
	GOBIN=""
	GOCACHE="/Users/dbourque/Library/Caches/go-build"
	GOENV="/Users/dbourque/Library/Application Support/go/env"
	GOEXE=""
	GOFLAGS=""
	GOHOSTARCH="amd64"
	GOHOSTOS="darwin"
	GOINSECURE=""
	GOMODCACHE="/Users/dbourque/dev/golang/pkg/mod"
	GONOPROXY=""
	GONOSUMDB=""
	GOOS="darwin"
	GOPATH="/Users/dbourque/dev/golang"
	GOPRIVATE=""
	GOPROXY=""
	GOROOT="/usr/local/Cellar/go/1.15.2/libexec"
	GOSUMDB="off"
	GOTMPDIR=""
	GOTOOLDIR="/usr/local/Cellar/go/1.15.2/libexec/pkg/tool/darwin_amd64"
	GCCGO="gccgo"
	AR="ar"
	CC="clang"
	CXX="clang++"
	CGO_ENABLED="1"
	GOMOD="/Users/dbourque/dev/tryitout/go.mod"
	CGO_CFLAGS=""
	CGO_CPPFLAGS=""
	CGO_CXXFLAGS="-g -O2"
	CGO_FFLAGS="-g -O2"
	CGO_LDFLAGS="-g -O2"
	PKG_CONFIG="/usr/local/bin/pkg-config"
	GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/3t/4048hqr5459dw83ksmkqxn7r0000gn/T/go-build186798659=/tmp/go-build -gno-record-gcc-switches -fno-common"

Does this issue occur when all extensions are disabled?: No
This issue for the Go Extension generated the same error message but this fix for it does not correct the behavior for me.

@dylan-bourque As I explained in the slack, your issue is different from #713 - the error message is different: users who were affected by #713 get crashes; you got a warning popup.
I hope the error message is clear enough that your Go is installed in /usr/local/bin but that's not in the PATH.  (FYI #713 does extra search in /usr/local/go/bin)
The value is what the vscode gave to the extension. You can verify what PATH value vscode sees by

disabling the go extension,
opening Developer: Toggle Developer Tools and
querying process.env['PATH'] from the developer tool console.

What does it say?

This fixed it for me

follow the instructions here to set GOPATH, GOROOT & PATH. NOTE: I didn't use the install.sh script!
Close & open VSCode
Disable & re-enable vscode-go plugin
Hit Update on the notification popping up talking about a changed GOROOT

@hyangah process.env['PATH'] returns:
"/Users/dbourque/.pyenv/shims:/usr/local/opt/gnu-sed/libexec/gnubin:/Users/dbourque/.nvm/versions/node/v14.4.0/bin:/usr/local/opt/make/libexec/gnubin:/Users/dbourque/dev/golang/bin:/Users/dbourque/bin:/Users/dbourque/.cargo/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Users/dbourque/Library/Python/2.7/bin"

I see here that /usr/local/bin is actually included and that is where my go binary is installed (by Brew).  A bit strangely, $GOPATH is set to $HOME/dev/golang and $GOPATH/bin is added to $PATH in ~/.bashrc and that is included, even though one suggestion was that the problem is that the editor only reads .profile, .bash_profile and .zprofile on a Mac.
@CanRau I already have my environment configuration in ~/.bashrc like that link suggests.
As I've said, none of my "profile" files (~/.profile, ~/.bash_profile, ~/.zprofile, ~/.bashrc, ~/.zshrc) have changed at all.  My environment (both login and interactive shells) is the same as it was 2 months ago when I was not seeing this error.

I am still not convinced that this is a VS Code issue

From my standpoint, $GOROOT has never been set and $PATH has never included my Go installation based on just my .profile, .bash_profile and .zprofile environment.  That environment configuration has been added in my .bashrc since I've had this machine (roughly a year) and this error message at startup is new (within the last few weeks).  If I launch the editor from an open terminal (by running code) the error does not occur.
It appears that the Go extension is getting a system-default $PATH (only /usr/bin:/bin:/usr/sbin:/sbin) despite process.env['PATH'] containing additional directories (including /usr/local/bin which is where Go is installed).

It's really strange that vscode-go sees a different process.env['PATH'] - the following is where the process.env['PATH'] is picked up in the go extension https://github.com/golang/vscode-go/blob/70f7a71e6bb85c987eefaca069185624f4c7168a/src/utils/pathUtils.ts#L20)
Note that the value is cached and it's initialized only once.
One possibility is that  the process.env['PATH'] was changed after the go extension was initialized I think.
@dylan-bourque how about trying to downgrade 1) vscode (1.46 or 1.47?) and 2) vscode-go (0.15.2 maybe) and to narrow down the issue?

I can try that later today or tomorrow.

Strange, yeah interestingly the extension worked for me initially without any env/path changes only after the error wouldn't go away did I follow the instructions I linked before which luckily resolved my issue

The only environment configuration I'm missing is explicitly setting $GOROOT.  I've always relied on the fallback/default value of the Go install dir.

@dylan-bourque you shouldn't need to set $GOROOT. Actually, that's not recommended.

That was my understanding as well, but I noticed that it's included in the link that @CanRau posted.

Yes I've set it as shown and it resolved all problems 🤷🏻♂

@CanRau Of course that's because the extension is also checking GOROOT/bin in addition to PATH etc. But what @dylan-bourque is getting at is about why the go extension isn't seeing the process.env['PATH'] it should see. I suspect a race between go extension's initialization and environment variable setup for extension host. Can we focus on the issue?

FYI just encountered another issue that talks about PATH issue on Mac that started recently
https://github.com/microsoft/vscode/issues/107169

cc @joaomoreno

Is there any message in the DevTools like this?
Unable to resolve shell environment in a reasonable time

I don't know if there was some specific change related to this, but I'm no longer seeing this behavior after updating VS Code to v1.50.0 and running the Go: Install/Update Tools command from the Go extension.

I'm getting this error, v1.50 hasn't fixed it. I'm seeing Unable to resolve shell environment in a reasonable time in my devtools.

Same here, usually I have to restart VSCode, second start works as expected 🤷♂

Running Developer: reload window fixed it for me. Seems they reduced the timeout for loading the environment from 10s to 3s.

Uh good alternative, more work with multiple windows open tho 😅

@beamery-tomht interestingly since yesterday quiting&re-opening VSCode won't work anymore, luckily your Developer: reload window trick works, even though I have to do it for every single window failing 😅 better than "not being able to work" I guess 🤷♂

No worries @CanRau , I stole the idea from the depths of another github issue thread and have to do the same for all of my windows. Luckily not all of them need the full environment!


eslint blocks legitimate errors in HTML context

I have an .php file that is mostly HTML at this point. Inside I have tried <script></script> and <script src="..."></script>.
This resulted in an error along the lines of Unexpected token '<' from eslint (verified using sublimelinter debug mode).
So then I went into my project's root folder and tried to do eslint --init, answered all the questions and ended up with the following as my .eslintrc file.
rules:
  indent:
    - 2
    - tab
  quotes:
    - 2
    - single
  linebreak-style:
    - 2
    - unix
  semi:
    - 2
    - always
env:
  browser: true
extends: 'eslint:recommended'
ecmaFeatures:
  jsx: true
  experimentalObjectRestSpread: true
plugins:
  - react

Opening the file again, eslint now gives me: Error - missing semicolon. (semi)
These errors are false positives, and block me being able to see errors from other linters. I don't plan to have much (if any) inline js code in my html/php files, so a good workaround for me would be to disable eslint in html contexts if that's possible. Of course I'd like it best if the solution made it work for all applicable contexts.
To me the error seems to be the selectors (html selector being the only one) set in linter.py. The selector (at least as far as I can tell) doesn't strip the <script></script> tags and eslint isn't expecting to start in an HTML context.
Please let me know if I can be of further assistance in troubleshooting/fixing this. Also it's entirely possible I'm just dumb and am missing some configuration flag.
EDIT: minor formatting edit and english, and clarity

Hello
Did you check this doc http://www.sublimelinter.com/en/latest/usage.html?highlight=script#customized-syntax-definitions ? Probably there is a solution.

@roadhump thanks for the quick reply!
I looked through that document, but I'm missing it's relevance to my issue... Are you suggesting that I modify the HTML syntax definition files? I wonder if there is no better solution.. I can't imagine I'm the only one that has come across this issue.
Maybe I'm just being thick and missing something?

This doc means that your HTML syntax file will be patched after SublimeLinter is installed and it will make linters work with code inside <script> tags. I suggest to restart ST if you are not yet. Also, is it possible that you use any custom, non-default HTML-syntax plugin?

Just restarted again, no dice.
I don't think I'm using anything special.. Maybe PHP syntax definitions? But I think those are built into ST3, I don't recall downloading anything like that.
Scanning through my loaded packages I found PHP CodeBeautifier in the list. I disabled it, restarted again, and still getting the same errors.
Is there some way I can check which HTML and HTML (Rails) syntax definitions are loaded?

Oh wait. Maybe I need to make whatever fix was done to the HTML and HTML (Rails) languages to the PHP language definition?

Looking at PHP.tmlanguage it looks like it relies on the underlying HTML language definitions instead. I'm not seeing any reference to <script or <script So I'm not seeing the issue.
As a test I also renamed the file to be index.html, reloaded ST3, and I'm still getting the same issue.

It is strange, I've tested html and php syntax and it seems work. Probably reinstall SublimeLinter will help.

Just reinstalled..
For now I'm using PackageResourceViewer, opening SublimeLinter-contrib-eslint\linter.py and removing 'html' from the list to stop it from linting using eslint in this context. But I am interested in continuing to troubleshoot.
I recently (within the past week) refreshed my PC (completely clean reinstall of windows). At that point I installed the latest version of sublime and started installing linters (I had never used linters in sublime prior to this).
Maybe fresh installs of sublime, sublimelinter or sublimelinter-contrib-eslint, cause some kind of issue? Maybe I just screwed up the configuration in some way?
I'd be happy to continue troubleshooting this, if you need me to do some debugging on my end to determine what is different between your environment and mine.

Try another SublimeLinter js plugins (jshint or jscs). If they work the same way in HTML files, I think you should post this issue to SublimeLinter itself.


CI

Continuous Integration is a great way of providing quick feedback on incoming PRs. Weave should use travis or some other free CI tool.

We have a weave CI now, thanks to @tomwilkie. -> closing


improvementChange docker shell DorisImage version

Proposed changes
Version：1.2.1 -> 2.0.0

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

run buildall


BHBC-2251: Dataset details update

Overview
Links to Jira tickets
BHBC-2251
Description of relevant changes

Modified dataset page for Survey/ Project specific fields
Updated CSS to be more inline with tickets UX pin
Details section only

PR Checklist
A list of items that are good to consider when making any changes.
Note: this list is not exhaustive, and not all items are always applicable.
Code

[ ] New files/classes/functions have appropriately descriptive names and comment blocks to describe their use/behaviour
[ ] I have avoided duplicating code when possible, moving re-usable pieces into functions
[ ] I have avoided hard-coding values where possible and moved any re-usable constants to a constants file
[ ] My code is as flat as possible (avoids deeply nested if/else blocks, promise chains, etc)
[ ] My code changes account for null/undefined values and handle errors appropriately
[ ] My code uses types/interfaces to help describe values/parameters/etc, help ensure type safety, and improve readability

Style

[ ] My code follows the established style conventions
[ ] My code uses native material-ui components/icons/conventions when possible

Documentation

[ ] I have commented my code sufficiently, such that an unfamiliar developer could understand my code
[ ] I have added/updated README's and related documentation, as needed

Tests

[ ] I have added/updated unit tests for any code I've added/updated
[ ] I have added/updated the Postman requests/tests to account for any API endpoints I've added/updated

Linting/Formatting

[ ] I have run the linter and fixed any issues, as needed
See the lint commands in package.json
[ ] I have run the formatter and fixed any issues, as needed
See the format commands in package.json

SonarCloud

[ ] I have addressed all SonarCloud Bugs, Vulnerabilities, Security Hotspots, and Code Smells

Code looks good!

Tested again locally, works as expected.


PSR Includes are out of date

There's been a change that makes a link to PSR-15 not working any more (it's been renamed from middleware to handlers PSR). The relevant PR: https://github.com/php-fig/fig-standards/pull/954
I located this to be in the _includes/ directory, where it's pointing to a revision in the fig-standards repo, however I never came across this before and I'm not sure how to update the revision.

I have attempted to fix this in the referenced PR. I never worked with git submodules, so double check it please ;]


update network.json to include Orai Network

Thanks for this - can you confirm Orai works completely with REStake? Adding to networks.json will mark it as 'full support' in the UI


experiment with switching out bootstrap for carbon on the homepage only

Feature description
See how feasible it is to switch out bootstrap for Carbon
This issue isn't for actually replacing it, but seeing how much work it will be to do.

talked with Kidus and it's feasible. Its def worth it to start this now rather than later


AttributeError: Can't get attribute 'DSW' on <module 'model' from '/home/user/TDAN-VSR/model.py'>

Hello,
I'm facing this error while using pytorch 0.3.1 installed via pip.
I am on a RTX 2080Ti, CUDA 10, Python 3.6, Ubuntu 18.04.
Traceback (most recent call last):
File "eval.py", line 41, in
model = torch.load(model_path)
File "/home/user/anaconda3/envs/enlighten/lib/python3.6/site-packages/torch/serialization.py", line 267, in load
return _load(f, map_location, pickle_module)
File "/home/user/anaconda3/envs/enlighten/lib/python3.6/site-packages/torch/serialization.py", line 420, in _load
result = unpickler.load()
AttributeError: Can't get attribute 'DSW' on <module 'model' from '/home/user/TDAN-VSR/model.py'>
I face this error when I run: python eval.py -t test_example/
Also when I run bash make.sh there is no output it simply exits.
Would be grateful for the help!
Thank you,
Sree Harsha

I updated the code before but have not added the new pre-trained model with much better performance.
Could you try the new code with new eval.py and model for bicubic? When you do testing, you should first prepare your LR frames in your test set directory. With Vid4 dataset as an example, you will have "data/Vid4/city/LR_bicubic",  "data/Vid4/foliage/LR_bicubic"...
If you have any questions or issues when running the code, please let me know.

Same problem. Following your kind advice, I update the codes with the newest version which you just pushed, however another problem occurs:
Traceback (most recent call last):   File "eval.py", line 117, in <module>     output, _ = model(lr)   File "/data/software/anaconda3/envs/py36_torch031/lib/python3.6/site-packages/torch/nn/modules/module.py", line 477, in __call__     result = self.forward(*input, **kwargs)   File "/data/software/anaconda3/envs/py36_torch031/lib/python3.6/site-packages/torch/nn/parallel/data_parallel.py", line 123, in forward     outputs = self.parallel_apply(replicas, inputs, kwargs)   File "/data/software/anaconda3/envs/py36_torch031/lib/python3.6/site-packages/torch/nn/parallel/data_parallel.py", line 133, in parallel_apply     return parallel_apply(replicas, inputs, kwargs, self.device_ids[:len(replicas)])   File "/data/software/anaconda3/envs/py36_torch031/lib/python3.6/site-packages/torch/nn/parallel/parallel_apply.py", line 77, in parallel_apply     raise output   File "/data/software/anaconda3/envs/py36_torch031/lib/python3.6/site-packages/torch/nn/parallel/parallel_apply.py", line 53, in _worker     output = module(*input, **kwargs)   File "/data/software/anaconda3/envs/py36_torch031/lib/python3.6/site-packages/torch/nn/modules/module.py", line 477, in __call__     result = self.forward(*input, **kwargs)   File "/data/guotaian/vsr/TDAN-VSR-master/model.py", line 231, in forward     lrs = self.align(out, x_center) # motion alignments   File "/data/guotaian/vsr/TDAN-VSR-master/model.py", line 199, in align     fea = (self.dconv_1(fea, offset1))   File "/data/software/anaconda3/envs/py36_torch031/lib/python3.6/site-packages/torch/nn/modules/module.py", line 477, in __call__     result = self.forward(*input, **kwargs)   File "/data/guotaian/vsr/TDAN-VSR-master/modules/deform_conv.py", line 43, in forward     self.num_deformable_groups)   File "/data/guotaian/vsr/TDAN-VSR-master/functions/deform_conv.py", line 23, in conv_offset2d     return f(input, offset, weight)   File "/data/guotaian/vsr/TDAN-VSR-master/functions/deform_conv.py", line 54, in forward     self.dilation[0], self.deformable_groups)   File "/data/software/anaconda3/envs/py36_torch031/lib/python3.6/site-packages/torch/utils/ffi/__init__.py", line 202, in safe_call     result = torch._C._safe_call(*args, **kwargs) torch.FatalError: invalid argument 5: 4D weight tensor (nOutputPlane,nInputPlane,kH,kW) expected, but got: (null) at /mnt/ssd0/project/ytian21/videoSR/DCN/src/deform_conv_cuda.c:15
So how could this problem be solved? Thank you so much!

So I got it to run! It SR-ed the input images by 4x.
I believe adding these steps would be useful for people working forward

PyTorch installed via conda using this command is the only one that worked
conda install pytorch==0.3.1 torchvision cuda90 -c pytorch

Verify that torch.version.cuda returns 9.0 (As PT 0.3.1 doesn't support CUDA10 etc., and there is a hard requirement of devices to have CUDA9+)

Subsequent to this -- the folder directory to be maintained should be
test_folder/input/LR_bicubic/
At line 111: frames_lr[idx, :, :, :] = io.imread(os.path.join(LR, ims[k]))
I got this error
ValueError: Could not find a format to read the specified file in single-image mode

This was fixed by manually typing out the path in place of "LR". Just wanted to make a note of this.

Another query was, I'm unsure of the bottleneck resolution (which in the case of Zooming Slo-Mo I deduced to be 704x528) the resolution I got to work on a Tesla V100 with 16 GB VRAM was 200x160. The others were always OOM (The resolution I am trying to work at is generally hovers around 700x? resolution. Any suggestions of how I could go about this?

Thank you so much for the prompt response.
Regards,
Sree Harsha

I am happy to learn that you can run the code successfully. The TDAN uses much fewer memory than our ZoomingSlomo. But I have not done a test to find the exact upper bound for it. But you can always use frame splitting to divide frames into smaller patches and do testing for large inputs. Please check chop_forward() function in solver.py.

Thank you for the prompt replies, I am currently trying out chop_forward and so far its not OOM. I probably should bring this up in Zooming Slo-Mo repo, but just in case its okay, would it possible to adapt the same strategy for it as well?

Yes, it can be applied to Zooming Slo-Mo as well.


PIDatePicker's value doesn't change.

Hi.  Firstly, thank you PIDatePicker. It is very useful for me.
But there is a litte bit problem.
I cannot change the value of the datepicker. If I set value of datepicker , like this:
let dateStringFormatter = NSDateFormatter()         dateStringFormatter.dateFormat = "yyyy-MM-dd"         let convertedStartDate = dateStringFormatter.dateFromString("2013-05-15")         debugPrint(convertedStartDate)         datePicker.setDate(convertedStartDate!, animated: true)        datePicker.reloadAllComponents()
DatePicker preserves the value that is modified.
Thank you for your answer in advance.

Hi @pointphoton, can you double check that this is still an issue for you? Just tested your code and the date picker is updating properly.

Hi @atecle. Unfortunately, I am not working on that project any more.


Fix a few typos

I fixed a few typos in Japanese document.
Thank you for publishing the great tutorial.

Thanks!


feature - easier time tracking

Requires some thought and design. Below is a bit of a brainstorm.
First syntax support:

$after(20ms) - expands to ($tis > $scale_time(20, ms))
$after(20 ms)
$after(20 * (2 + 6), ms) - comma optional and maybe useful to ease parsing.
$after(20, time) - time is the underlying units provided by user. Could be 1/64 of a second. We don't need to know.
$after(some_var, ms) - some_var could be an expansion or variable.
$time_in_state - expands to ($time - $state_entered_time)
$tis - time in state (user's time units)
$tis(ms) - time in state in milliseconds.
$state_timer_reset()
$state_entered_time - expands to this.vars._timer_level<X>_start
$time - expands to whatever user specified or cached time var this.vars._cached_now.
$scale_time(X, units) - expands to (X * units_scaler)

Later syntax support:

after 20 time - time is the underlying units provided by user. Could be 1/64 of a second. We don't need to know.
after 500ms
after 500 ms
after some_var ms - some_var could be an expansion or variable.

For this to work StateSmith will need:

a way to get the current time. Provided via an expansion.
a way to know the scaling of time provided by the above. Does the above return milliseconds? Seconds? Microseconds? Consider 1/64th of a second and non base 10 fractions.
whether it should cache now() in a variable rather than always get current time.

[ ] Depth first search tree. Find each state that uses auto time tracking.
[ ] Give them a time layer number and add to list.
[ ] Allocate time vars for each layer.
[ ] Upon entering state, store current time in bar.
[ ] When handling events, calc time in state inside function from now - entered.
[ ] Map expansions

Later support

[ ] allow user to define timers that can be shared among states. Basically just like what we do now with expansions, but without needing the expansions.

being able to support periodic behaviors with something like every 10 seconds would also be cool

Auto-off:
stateDiagram

off
on

off  --> on: BUTTON
on --> off: after 30 seconds

Blinky:
stateDiagram

off
on

off --> on: every 1 second
on --> off: after 200ms

As I write this, I'm a little unclear on the semantics of every. I know that the previous tool I used supported it, but in this example it's not clear how Blinky would be different if I just replaced every with after.


Missing a Health 2.2 release plan

Missing in release list. Please create a release plan. You can look at others (metrics, fault tolerance, etc) to get an idea of what to enter. "Create a release" is in the committer tools list.

So sorry @jclingan I didn't get to it sooner. Thanks for the remainder.
Here it is - https://projects.eclipse.org/projects/technology.microprofile/releases/health-2.2


Unable to pipe output from curl command.

Can you add a flag to suppress the HTTP output from the curl command. This will enable users to pipe the output to a tool for formatting and processing JSON, such as jq.

@danhigham The curl response body should be coming back on stdout, and the response headers should be on stderr. You should be able to use om curl | jq just fine. Can you provide an example of something that isn't working for you?

Ah, right. I stand corrected. I will close this issue :-)


Building wrong query

I've just installed Speckcommerce via GitClone->composerInstall->SpeckInstaller, and on home page getting this error.
In file '\speck\vendor\speckcommerce\speck-catalog\src\SpeckCatalog\Mapper\Category.php' on line 27:
System replaces content of $joinString ('catalog_category_website.category_id = catalog_category.category_id') in join statement, so final query is exactly:
SELECT "catalog_category"., "catalog_category_website". FROM "catalog_category" INNER JOIN "catalog_category_website" ON "catalog_categ"or"y_website"."categ"or"y_id" = "catalog_categ"or"y"."categ"or"y_id" WHERE "website_id" = '1' AND "parent_category_id" IS NULL
Using new \Zend\Db\Sql\Expression($joinString) is fixing the issue but why system strangely replacing 'or'?

same happens to me, any ideas ?


Added ST_SnapToGrid support

Coverage increased (+0.09%) to 98.599% when pulling a35f9c14810c4caaa2863ff09c135d9a774e8d74 on dragosprotung:stsnaptogrid into 065686e8d3fb5c90379d98bb6b63057524d06e04 on creof:master.

Coverage increased (+0.09%) to 98.599% when pulling a35f9c14810c4caaa2863ff09c135d9a774e8d74 on dragosprotung:stsnaptogrid into 065686e8d3fb5c90379d98bb6b63057524d06e04 on creof:master.

Coverage increased (+0.09%) to 98.599% when pulling a35f9c14810c4caaa2863ff09c135d9a774e8d74 on dragosprotung:stsnaptogrid into 065686e8d3fb5c90379d98bb6b63057524d06e04 on creof:master.

This was my last PR for now. Any chance to create a new release ?


Parser throws an exception on empty input

Hello,
I've recently tested some stuff and found that fsJsonParser throws an exception if empty string was passed to it. I think this is not best behaviour, and we must have fsResult.Fail for that case instead of exception.

Okay, a proper error is now given. I've also fixed a scenario where the parser would crash on
{
    "hello":

Cool, thanks!


ObservableForProperty can't notify when beforeChange = true

I'm having a problem with a ObservableForProperty when beforeChange = true.
I have a TestViewModel, TestViewModel has a SubViewModel property, and SubViewModel has a SubSubViewModel property.
                   var viewModel = new TestViewModel();             viewModel.SubViewModel = new TestSubViewModel(1);             viewModel.SubViewModel.SubViewModel = new TestSubSubViewModel(1);             viewModel.SubViewModel.SubViewModel = null;
In TestViewModel's construtor:
        public TestViewModel()         {             this.ObservableForProperty(x => x.SubViewModel.SubViewModel, true, false) 		.Select(x => x.Value) 		.Subscribe(model =>  			Console.WriteLine("SubViewModel.SubViewModel changing model " + model));         }
Unfortunately, the log message can't print...
Is this a bug for ReactiveUI or I am wrong?

I write a simple test which can reappear it issue.
MyTest.txt


Reload data on all chains

Need to run the following mutations on all balancer chains to ensure consistent data:
  poolLoadOnChainDataForAllPools
  userInitWalletBalancesForAllPools
  userInitStakedBalances(stakingTypes:[GAUGE])

Test:

[x] Mainnet
[x] Arb
[ ] Avax
[ ] Polygon
[x] Base
[x] Gnosis
[x] Zkevm

Prod:

[x] Mainnet
[x] Arb
[ ] Avax
[ ] Polygon
[x] Base
[ ] Gnosis
[ ] Zkevm

founds bugs #99 #98


How to create my server with "Server.js"

Hi ! Thx for your work, it works very well on p5editor, but now I'm trying do that in my local network and I have some problems !
I correctly npm install express, cors, websocket and socket.io. --save
Then I found on the net 'star_itp_io.key' and 'star_itp_io.pem' files.
But now I have this error :

Could you help me ? Sorry, but I'm nioub in js...
Thx in advance

Hi Romain,
The Socket.io library changed a bit since this was released.  You could
either use an older version or update the code that is giving an error to
something like this:
const { Server } = require('socket.io');
const io = new Server(httpServer, {});
Also, those certs you found are not likely to be valid.  You should
probably generate a self-signed cert for use on your own network.
Hope this helps
On Thu, Sep 29, 2022 at 4:49 PM Romain Al @.***> wrote:

Hi ! Thx for your work, its work very well on p5editor, but now I'm trying
do that in my local network and I have some problems !
I correctly install express, cors, websocket and socket.io.
Then I found on the net 'star_itp_io.key' and 'star_itp_io.pem' files.
But now I have this error :
[image: Capture d’écran 2022-09-29 à 22 44 00]
https://user-images.githubusercontent.com/60849348/193138934-e56c8e69-c6d1-4726-9537-c1722ab687d4.png
Could you help me ? Sorry, but I'm nioub in js...
Thx in advance
—
Reply to this email directly, view it on GitHub
https://github.com/vanevery/p5LiveMedia/issues/18, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADJNHUG5D4WSV6NFSQB46LWAX6ERANCNFSM6AAAAAAQZEUOMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yes ! Thx ! It work well now with this code :
`const { createServer } = require("https");
const { readFileSync } = require("fs");
const { Server } = require("socket.io");
const options = {
key: readFileSync('key.pem'),
cert: readFileSync('cert.pem')
};
const httpServer = createServer(options, (req, res) => {
res.writeHead(200);
res.end('This is my WebRTC signaling server\n');
}).listen(1337, () => {console.log("Server is running")});
const io = new Server(httpServer, {
cors: {
origin: "*"
}
});`
With my own cert/key using "mkcert" !
Thank you very much !
Last question : how do you do to host your signaling server on github ?


failed to parse manifest at error in Ubuntu 16.04

$ cat /etc/os-release
NAME="Ubuntu"
VERSION="16.04.1 LTS (Xenial Xerus)"

$ rustc --version
rustc 1.7.0

$ cargo --version
cargo 0.8.0 (built 2016-03-22)

Before running any cargo commands, inside .bashrc:
export CARGO_HOME=$HOME/.local/cargo

Error message:
$ cargo install --verbose ripgrep
    Updating registry `https://github.com/rust-lang/crates.io-index`
 Downloading ripgrep v0.3.2
failed to parse manifest at `/home/$USER/.local/cargo/registry/src/github.com-88ac128001ac3a9a/ripgrep-0.3.2/Cargo.toml`

Caused by:
could not parse input as TOML
.local/cargo/registry/src/github.com-88ac128001ac3a9a/ripgrep-0.3.2/Cargo.toml:43:9 expected a key but found an empty string
.local/cargo/registry/src/github.com-87ac128001ac3a9a/ripgrep-0.3.2/Cargo.toml:43:9-43:10 expected `.`, but found `'`

I believe this is complaining about this line which uses single-quotes in key names rather than double quotes. This was added as a feature relatively recently in Cargo, so I think what you'll need to do here is upgrade the version of Cargo that you're using.
If that doesn't help though just let me know!

Ok, thank You for the info.

Thank you, update cargo from source http://doc.crates.io/ and in new version all work fine.


Line endings problem

I tried loading an .svg file with Windows line endings on OS X and SvgDocument.Open hanged. Converting the line endings to Unix style fixed it.

Follow up: Loading a file with Unix line endings on Windows works fine. This simplifies the workaround for me, all svg files will be kept with Unix line endings.

I don't have a Mac, so I can't test this - is this still an issue? I'm also not sure if this an SVG bug or a Mono problem. Has oneone more information about this?

Could not reproduce it.
It was probably a Mono problem, Mono has changed a lot for the past 3 years. And we never upgraded Svg.NET, we're still at 2.0.0 (I see there's 2.3.0 at NuGet).


{:error, "Table 'requests' could not be found"}

Hey, great library.
So. Works in dev, deployed to fly.io and it produces an error. No events are logged, but the page opens.
2024-11-01T12:31:23Z app[2871ed6b664758] arn [info]** (MatchError) no match of right hand side value: {:error, "Table 'requests' could not be found"}
2024-11-01T12:31:23Z app[2871ed6b664758] arn [info]    (phoenix_analytics 0.2.1) lib/phoenix_analytics/repo.ex:215: PhoenixAnalytics.Repo.insert_many/1
2024-11-01T12:31:23Z app[2871ed6b664758] arn [info]    (phoenix_analytics 0.2.1) lib/phoenix_analytics/services/batcher.ex:101: PhoenixAnalytics.Services.Batcher.handle_info/2

and then some lines later it's
2024-11-01T12:31:38Z app[2871ed6b664758] arn [info]** (MatchError) no match of right hand side value: {:error, "Catalog Error: Table with name requests does not exist!\nDid you mean \"pg_sequences\"?\nLINE 1: ...urce': path, 'visits': COUNT(*) } FROM requests\n                                                  ^"}
2024-11-01T12:31:38Z app[2871ed6b664758] arn [info]    (phoenix_analytics 0.2.1) lib/phoenix_analytics/repo.ex:258: PhoenixAnalytics.Repo.execute_fetch/1
2024-11-01T12:31:38Z app[2871ed6b664758] arn [info]    (phoenix_analytics 0.2.1) lib/phoenix_analytics/web/live/components/charts/popular_chart.ex:54: PhoenixAnalytics.Web.Live.Components.PopularChart.fetch_data/3
2024-11-01T12:31:38Z app[2871ed6b664758] arn [info]    (phoenix_analytics 0.2.1) lib/phoenix_analytics/services/cache.ex:92: PhoenixAnalytics.Services.Cache.fetch/2
2024-11-01T12:31:38Z app[2871ed6b664758] arn [info]    (phoenix_analytics 0.2.1) lib/phoenix_analytics/web/live/components/charts/popular_chart.ex:41: PhoenixAnalytics.Web.Live.Components.PopularChart.chart_data/2
2024-11-01T12:31:38Z app[2871ed6b664758] arn [info]    (phoenix_analytics 0.2.1) lib/phoenix_analytics/web/live/components/charts/popular_chart.ex:34: anonymous fn/2 in PhoenixAnalytics.Web.Live.Components.PopularChart.update/2
2024-11-01T12:31:38Z app[2871ed6b664758] arn [info]    (phoenix_live_view 1.0.0-rc.7) lib/phoenix_live_view/async.ex:148: anonymous fn/2 in Phoenix.LiveView.Async.assign_async/4
2024-11-01T12:31:38Z app[2871ed6b664758] arn [info]    (phoenix_live_view 1.0.0-rc.7) lib/phoenix_live_view/async.ex:220: Phoenix.LiveView.Async.do_async/5
2024-11-01T12:31:38Z app[2871ed6b664758] arn [info]    (elixir 1.17.2) lib/task/supervised.ex:101: Task.Supervised.invoke_mfa/2
2024-11-01T12:31:38Z app[2871ed6b664758] arn [info]Function: #Function<7.88405273/0 in Phoenix.LiveView.Async.run_async_task/5>
2024-11-01T12:31:38Z app[2871ed6b664758] arn [info]    Args: []

I would suspect that the migrations haven't been run, but as you know fly.io handles that and I haven't had any issues with migrations before.
Just to be clear I do have this migration.
defmodule MyApp.Repo.Migrations.AddPhoenixAnalytics do
  use Ecto.Migration

  def up, do: PhoenixAnalytics.Migration.up()
  def down, do: PhoenixAnalytics.Migration.down()
end

I checked manually and select * from schema_migrations; confirms the migrations have run.
and my config.exs is
config :phoenix_analytics,
  duckdb_path: System.get_env("DUCKDB_PATH") || "analytics.duckdb",
  app_domain: System.get_env("PHX_HOST") || "example.com"

I have not set the DUCKDB_PATH, as I understand there is no need? Prod does have HOST configured.

I'm experiencing the same issue.
It seems in PhoenixAnalytics.Queries.Table, @db_alias is being appended directly to the table name, and attempt to create a table "postgres_db.requests" rather than just "requests".
defmodule PhoenixAnalytics.Queries.Table do
  @moduledoc false
  alias PhoenixAnalytics.Services.Utility

  @db_alias "postgres_db"
  @requests if Utility.mode() == :duck_postgres, do<EMAIL_ADDRESS>else: "requests"

  def name() do
    @requests
  end

  def create_requests do
    query = """
    CREATE TABLE IF NOT EXISTS #{@requests} (
      request_id UUID PRIMARY KEY,
      method VARCHAR NOT NULL,
      ...

docs to the Utility,.mode() are here: https://hexdocs.pm/phoenix_analytics/PhoenixAnalytics.Services.Utility.html#mode/0
so basically if both duck and postgres are configured that db_alias is used. in my case indeed both are used. question is what's different from my dev to prod.

Hi @KristerV ,
Thank you for raising issue, I never worked with fly io. Could you please provide example how connection to db looks there?
As I can add simple parameter "ignore_alias" or something similar.

oh, right, i thought i read that you use fly.io actually.
i'll provide verbose info, who knows what will explain the issue.
First of all fly.io servers are shown as such:

The only secrets the app has are DATABASE_URL and SECRET_KEY_BASE.
Postgres server info:

Other than that there's not much more i can talk about the Postgres server. Don't think there's anything special about it.
App server
Is it possible that since DuckDB tries to make a local file that the app server is important to discuss? Fly creates a docker image and deploys that. Here's the Dockerfile:
# Find eligible builder and runner images on Docker Hub. We use Ubuntu/Debian
# instead of Alpine to avoid DNS resolution issues in production.
#
# https://hub.docker.com/r/hexpm/elixir/tags?page=1&name=ubuntu
# https://hub.docker.com/_/ubuntu?tab=tags
#
# This file is based on these images:
#
#   - https://hub.docker.com/r/hexpm/elixir/tags - for the build image
#   - https://hub.docker.com/_/debian?tab=tags&page=1&name=bullseye-20240904-slim - for the release image
#   - https://pkgs.org/ - resource for finding needed packages
#   - Ex: hexpm/elixir:1.17.2-erlang-27.0.1-debian-bullseye-20240904-slim
#
ARG ELIXIR_VERSION=1.17.2
ARG OTP_VERSION=27.0.1
ARG DEBIAN_VERSION=bullseye-20240904-slim

ARG BUILDER_IMAGE="hexpm/elixir:${ELIXIR_VERSION}-erlang-${OTP_VERSION}-debian-${DEBIAN_VERSION}"
ARG RUNNER_IMAGE="debian:${DEBIAN_VERSION}"

FROM ${BUILDER_IMAGE} as builder

# install build dependencies
RUN apt-get update -y && apt-get install -y build-essential git \
    && apt-get clean && rm -f /var/lib/apt/lists/*_*

# prepare build dir
WORKDIR /app

# install hex + rebar
RUN mix local.hex --force && \
    mix local.rebar --force

# set build ENV
ENV MIX_ENV="prod"

# install mix dependencies
COPY mix.exs mix.lock ./
RUN mix deps.get --only $MIX_ENV
RUN mkdir config

# copy compile-time config files before we compile dependencies
# to ensure any relevant config change will trigger the dependencies
# to be re-compiled.
COPY config/config.exs config/${MIX_ENV}.exs config/
RUN mix deps.compile

COPY priv priv

COPY lib lib

COPY assets assets

# compile assets
RUN mix assets.deploy

# Compile the release
RUN mix compile

# Changes to config/runtime.exs don't require recompiling the code
COPY config/runtime.exs config/

COPY rel rel
RUN mix release

# start a new build stage so that the final image will only contain
# the compiled release and other runtime necessities
FROM ${RUNNER_IMAGE}

RUN apt-get update -y && \
  apt-get install -y libstdc++6 openssl libncurses5 locales ca-certificates \
  && apt-get clean && rm -f /var/lib/apt/lists/*_*

# Set the locale
RUN sed -i '/en_US.UTF-8/s/^# //g' /etc/locale.gen && locale-gen

ENV LANG en_US.UTF-8
ENV LANGUAGE en_US:en
ENV LC_ALL en_US.UTF-8

WORKDIR "/app"
RUN chown nobody /app

# set runner ENV
ENV MIX_ENV="prod"

# Only copy the final release from the build stage
COPY --from=builder --chown=nobody:root /app/_build/${MIX_ENV}/rel/kids ./

USER nobody

# If using an environment that doesn't automatically reap zombie processes, it is
# advised to add an init process such as tini via `apt-get install`
# above and adding an entrypoint. See https://github.com/krallin/tini for details
# ENTRYPOINT ["/tini", "--"]

CMD ["/app/bin/server"]

Then anything interesting for the deployment itself should be in fly.toml:
# fly.toml app configuration file generated for kids on 2024-10-28T16:19:07+02:00
#
# See https://fly.io/docs/reference/configuration/ for information about how to use this file.
#

app = 'kids'
primary_region = 'arn'
kill_signal = 'SIGTERM'

[build]

[deploy]
  release_command = '/app/bin/migrate'

[env]
  PHX_HOST = 'example.com'
  PORT = '8080'

[http_service]
  internal_port = 8080
  force_https = true
  auto_stop_machines = true
  auto_start_machines = true
  min_machines_running = 1
  processes = ['app']

  [http_service.concurrency]
    type = 'connections'
    hard_limit = 1000
    soft_limit = 1000

[[vm]]
  memory = '1gb'
  cpu_kind = 'shared'
  cpus = 1

Anything else that could help understand the issue?

if I configure like this:
config :phoenix_analytics,
  app_domain: System.get_env("PHX_HOST") || "example.com",
  postgres_conn: "dbname=phoenixanalytics user=postgres password=postgres host=localhost",
  cache_ttl: 120

Then I just get an error:
14:20:50.835 [info] == Running<PHONE_NUMBER>2212 MyApp.Repo.Migrations.AddPhoenixAnalytics.up/0 forward
** (FunctionClauseError) no function clause matching in Duckdbex.open/1

    The following arguments were given to Duckdbex.open/1:

        # 1
        nil

    Attempted function clauses (showing 2 out of 2):

        def open(path) when is_binary(path)
        def open(%Duckdbex.Config{} = config)

    (duckdbex 0.3.6) Duckdbex.open/1
    (phoenix_analytics 0.2.1) lib/phoenix_analytics/migration.ex:11: PhoenixAnalytics.Migration.up/0
    (ecto_sql 3.12.1) lib/ecto/migration/runner.ex:310: Ecto.Migration.Runner.perform_operation/3
    (stdlib 6.0.1) timer.erl:590: :timer.tc/2
    (ecto_sql 3.12.1) lib/ecto/migration/runner.ex:25: Ecto.Migration.Runner.run/8
    (ecto_sql 3.12.1) lib/ecto/migrator.ex:365: Ecto.Migrator.attempt/8
    (ecto_sql 3.12.1) lib/ecto/migrator.ex:282: anonymous fn/5 in Ecto.Migrator.do_up/5
    (ecto_sql 3.12.1) lib/ecto/migrator.ex:337: anonymous fn/6 in Ecto.Migrator.async_migrate_maybe_in_transaction/7

So this means that PostgreSQL alone isn't enough. It must have DuckDB configured also, in which case I still get the error in the above post. I'm full of frustration so I guess I'm going to have to go with a separate service after all. Hope to come back to this some day.

Hi @KristerV
Could you add as well duck db path into your config?
In case you use Postgres as backend, duck db file will be empty, but it's still needed for duck db to operate properly.

okay, so
config.exs
config :phoenix_analytics,
  duckdb_path: System.get_env("DUCKDB_PATH") || "analytics.duckdb",
  app_domain: System.get_env("PHX_HOST") || "example.com",
  postgres_conn: "dbname=phoenixanalytics user=postgres password=postgres host=localhost",
  cache_ttl: 120

runtime.exs
  config :phoenix_analytics,
    app_domain: host,
    postgres_conn: database_url,
    cache_ttl: 120

so both postgres and duck have configs. but still same error:
[info]** (MatchError) no match of right hand side value: {:error, "Catalog Error: Table with name requests does not exist!\nDid you mean \"analytics.pg_catalog.pg_sequences\"?"}

what else can we try?

Following up on that issue, I made a PR where I introduced an option not to touch disk at all.
I need to run performance tests and if it's gonna be ok, i will add it in next release.


Feature Request : Add ChatGPT (Web)

I hope that talking about another open source extension is not against the rules
The ChatGPTBox extension has an option called ChatGPT (Web), which is enough to keep chat.openai.com open in a tab so that you can use its translation ability on different sites (that is, there is no need to api, etc.)
I wrote the name of this extension because I don't know how it does this
Please add this item so that those who don't have an open ai subscription (Users who have a free subscription to OpenAI) can use gpt 3.5 for free in the direct translation of sites.

I do not know how GPT can help in translation, but I join the request

@seregadushka
Translations made by artificial intelligence are far better than machine translations
Go to gemini or chat.openai.com and give it a page and tell it to translate for you
Then compare the translation it made with the machine translation
There is a significant difference


Badger DB storage backend for Trie

This provides us a both persistent and non-persistent storage for our Trie. Non-persistent Tries are particularly useful when calculating transaction and event commitments.
Extracted from #483

@IronGauntlets , updated according to your comments.


Better Log abstraction

Consider migrating this unnecessary snippet to something that's more user driven, like SLF4J
https://github.com/team-supercharge/rxsnappy/blob/master/rxsnappy/src/main/java/io/supercharge/rxsnappy/RxSnappyLog.java

Thanks for your feedback!
I think I will remove this class from the project, not really needed.

Fixed on 0.2


Does the "intel mac" support mean "intel MIC" (Arch for intel xeon phi acceleration card)?

No. It means Intel processor on Mac provided by Apply.


Revert "Improve capa checkout in sync workflow"

Seems like something's off here
Reverts fireeye/capa-rules#40

it seems the token is needed. I'll send another one. It is the same problem as in https://github.com/fireeye/capa-testfiles.


Remove unused Pip log scraping.

Fixes #facepalm.

This was un-needed when introduced in #1579. Proof? Well tests still pass, but the log line format does not even exist in the vendored Pip, only in more modern Pip. Clearly I had experimented with Pip in a venv and did not use the right version, added scraping, wondered why it didn't work, went on to enhance runtime patches, then never cleaned up. The runtime patches ended up being needed more widely to support --target-systems and proper --interpreter-constraints handling, but facepalm nonetheless.


Running Jest programmatically in dependent projects

I have a Node project A that has a number of scripts that can be run as tasks via my-scripts <task>. This project has Jest as a dependency and one of the scripts, my-scripts test calls jest.run. This is all handled by a my-scripts.js that is set as bin in the package.json that takes the third argument as the script name and anything after that as arguments.
I have another Node project B that depends on A and is able to run the scripts in the same way. This project does not have Jest as a dependency, since the scripts/tasks live in A.
Running the script my-script test works in both projects. However, the tests in project B fail due to babel-jest not being used. An inspection of the configs via --showConfig shows the following key differences:
Project A
"setupFiles": [
    "/projectA/node_modules/regenerator-runtime/runtime.js"
],
"transform": [
    [
        "^.+\\.jsx?$",
        "/projectA/node_modules/babel-jest/build/index.js"
    ]
],

Project B
"setupFiles": []

So, project A has both setupFiles and transform while B has an empty array for setupFiles and transform is nowhere to be seen.
Is there some piece I am missing to have the script work the same in both projects or is this a bug?
I have the latest version of Node, NPM, and Jest.

I think this isn't a bug but with the limited information you provided that's hard to tell. Jest infers the existence of babel-jest by looking at the closest node_modules and whether it has a babel-jest folder. You can always manually define the above configuration for both projects also.


Add README.md for Japanese

This PR is related to the following issues.
Please confirm this.
#2038

👍 Can't validate the content but thanks for this

Thank you for merging the PullRequest.
However, there was a mistake in the text that I wrote.
Specifically, the "Translated documentation" table is collapsed.
I'm quite sorry. I will send another PullRequest.


feat: add initial support for more advanced game settings

Description

This is the first step towards building a more complex game selection process that will eventually involve timers, a results page, and more fine-tuned options for people studying. This work focuses on allowing users to pick multiple categories to study at once.
To Validate

Make sure all PR Checks have passed
Pull down this branch
Run npm start
Click through the Flash Cards, Short Answer, and Multiple choice quick start links, confirming that you go straight to a question page with "Question 1 of n", where n is less than or equal to 50
The subheading should list the category or categories that the question belongs to, separated by commas
If you click through to the last question, there should be a link that leads to the game settings page for the game type you're playing
Go to the game settings page for Flash Cards, Short Answer, and Multiple Choice, checking the following for each:

By choosing no categories and pressing the "Start Game" button, you start the game with all questions
By choosing one or more categories (other than "All Questions"), you only see questions in those categories
By choosing categories with overlap, like "Assistive Technologies" and "Screen Readers", the number of questions indicated by the "Start Game" button matches the number of unique questions, not the sum of the category totals
When JavaScript is disabled, the "Start Game" button still takes you to a question, but the order won't be randomized, and it will be all questions

Acknowledging I don't have the user testing to back this up, I don't think it would be unclear to the user what toggling "All Questions" would mean for a game even if the action does not change the state of any categories that might have already been chosen. Especially since the question total is repeated in both the "All Questions" label and the "Start Game" button. It does differ from the common pattern of the "all" option clearing them, though.
I'm back and forth on it.
We could replace "All Questions" with "select all" and "clear all" actions but lose the visual on how many questions there are in total unless we place that elsewhere. The default state of the game setup screen would also have fourteen checked checkboxes, which I'm not a fan of.
We could have "All Questions" checked by default - "Start Game (153 Questions)" - and change the result when it is unchecked - "Start Game (0 Questions)." I'm averse to disabling the "Start Game" button in this case, in favor of an error/helper message that says something to the effect of at least one category is required. In this case, checking the "All Questions" would clear any chosen categories, and choosing any single category would uncheck "All Questions."
All that to say, now that I'm typing all that out, that's deserving of a new PR and the behavior we have currently is at a good point for this work.


python310Packages.ghrepo-stats: 0.5.1 -> 0.5.2

Diff: https://github.com/mrbean-bremen/ghrepo-stats/compare/refs/tags/v0.5.1...v0.5.2
Changelog: https://github.com/mrbean-bremen/ghrepo-stats/blob/v0.5.2/CHANGES.md
Description of changes

Things done

Built on platform(s)

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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

[ ] Fits CONTRIBUTING.md.

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

2 packages built:

ghrepo-stats (python310Packages.ghrepo-stats)
python311Packages.ghrepo-stats


Update group creator js to reflect new db

Current student data and preferences are stored in the script.
Script needs to be refactored to pull data from studentcourse for data from courses and students to determine size of Course, take in input from instructor for number of teams.

We have put this off until entire site is complete, maybe indefinitely, will reopen if necessary


Delete Empty Folders: console error "rm returned EISDIR (is a directory)"

I've just installed the plugin and run the commands as listed on the README instructions:

Replace All Wiki Links with Markdown Links
Convert All Embed Paths to Relative
Convert All Link Paths to Relative
Collect All Attachments
Delete Empty Folders

All run without issues until step 5. The command didn't display any messages, so I checked the command log:

I've tried manually deleting this specific folder "C:\Users\asukanec\OneDrive - Grupo Mirgor\Desktop\Notas Mirgor\Test\sub_assets" but the issue remained, now with another empty folder throwing the error.
I'm using Obsidian v1.5.12 on Windows 11 and plugin v1.3.0

@suka97 Should be fixed in https://github.com/dy-sh/obsidian-consistent-attachments-and-links/releases/tag/2.0.0
Feel free to reopen the bug if it is still happening

I've updated the plugin to last v2.0.0 but now the plugin doesn't work. When I move a markdown file with an attachment it does not move the attachment. On the console the following error is displayed:

So I went back to old v1.3.0 which was working for me (besides this issue)

@suka97 can you provide a test vault with the video showing how to reproduce the issue

Here is a video of the new issue appearing on v2.0.0, as well as v1.3.0 working as it should (besides not emptying the _assets folder)
https://github.com/user-attachments/assets/cd91fa0a-d196-4e57-b600-ed2b3d2c4387
https://github.com/user-attachments/assets/8188ad48-dc4d-41e6-8135-03bb75463e58
Here's my sample Obsidian Vault: ObsidianTemplate.zip

Fixed in https://github.com/dy-sh/obsidian-consistent-attachments-and-links/releases/tag/2.1.0

The issue regarding getCacheSafe on v2.0.0 was fixed:

I've updated the plugin to last v2.0.0 but now the plugin doesn't work. When I move a markdown file with an attachment it does not move the attachment. On the console the following error is displayed:

So I went back to old v1.3.0 which was working for me (besides this issue)

But the original error "rm returned EISDIR" remained:

I've just installed the plugin and run the commands as listed on the README instructions:

Replace All Wiki Links with Markdown Links
Convert All Embed Paths to Relative
Convert All Link Paths to Relative
Collect All Attachments
Delete Empty Folders

All run without issues until step 5. The command didn't display any messages, so I checked the command log:
I've tried manually deleting this specific folder "C:\Users\asukanec\OneDrive - Grupo Mirgor\Desktop\Notas Mirgor\Test\sub_assets" but the issue remained, now with another empty folder throwing the error.
I'm using Obsidian v1.5.12 on Windows 11 and plugin v1.3.0

I'm testing this with the sample Obsidian Vault attached above


Don't run unsupported tests on 8.6 and 9.0

https://coreos.slack.com/archives/C0235DZB0DT/p1650536467098679?thread_ts=1650512166.141439&cid=C0235DZB0DT
This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

@atodorov so the tricky part is that these tests are not supported only for the versions that are in RHEL repositories. But the main composer can run them successfully on 8.6 and 9.0. So we should just not run them in the nightly runs which do not use osbuild-composer from the git.


Suggest rewording

Point 6 in the example - "Keep These values are interchangeable checked if the phrases can be used interchangeably." is not useful.  It's like a dictionary entry of "Aardvark - see Aardvark."
Suggest rewording to describe what used interchangeably means in this context.

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

ID: c975bacb-0bee-7bb7-0d22-1296e79364b4
Version Independent ID: 57f07a89-94d6-74ef-1ba3-846e590ac838
Content: Use features to improve LUIS word list - Azure Cognitive Services
Content Source: articles/cognitive-services/LUIS/luis-how-to-add-features.md
Service: cognitive-services
Sub-service: language-understanding
GitHub Login: @aahill
Microsoft Alias: aahi

@davecatt Thanks for the feedback! We will review and update accordingly.

@davecatt thanks for your feedback, the PR will be deployed soon. Thanks.


feat(api.json): add new APIs across multiple categories

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #(issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[ ] Any new tools are named according to the Tool Naming Convention outlined in contributing.md
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Screenshots (if applicable):

All checks were passed and the guidelines were followed

All checks were passed and the guidelines were followed

I accidentally added an old API. Can you remove it?


Add using XLSX in the Database

I want to add using the Excel sheets in the database like to import data from it and export data from json to xlsx or even read the data from xlsx

Done, you can go check the documentation JsonVerse Documentation


Use biosdev NIC names for overcloud node interfaces

https://github.com/openstack-k8s-operators/osp-director-dev-tools/pull/109


Register not logging in users

Describe the bug
When a user registers with the site, they are not logged in after being directed to the profile page.
To Reproduce
Steps to reproduce the behavior:

Go to https://momentum-mentors.netlify.app/
Click on sign up
Enter new user information
You will be redirected to the profile page
The login button will stay logged in and not flip to logout

Expected behavior
The user should be logged in completely after clicking the register button.
Screenshots
N/A
Desktop (please complete the following information):

Browser Chrome

Additional context
N/A

Looks great! Code is merged!


Support CI clis

Colored output seems to be disabled because color package is not detecting the CI system CLIs. It looks like many is setting a CI env variable we can look for and enable colors on the known providers. Also see the reference at https://github.com/chalk/supports-color/blob/master/index.js#L66.

Or provide some other env variable to force color output?

I can add it, need to be sure it's worth adding it (probably is for many).
As @broady said, in gotest main function, you could do it with the following (as an example just for travis):
if os.Getenv("CI") == "Travis" {
	color.NoColor = false
}

^ this looks like a better patch than doing it in color.

Happy to apply the patch instead.

Nice. Closing this. I believe it's better not add anything necessary.


Add ability to print all parameter values to a csv file

Add a helper function to parameter values that allow them to be quickly printed to a csv file.

see print_parameters.py

Ah nice, cheers.

yeah I don’t think there was a reason for it not being a method. makes sense for it to be


Killing Nashorn, adding merger on external process

Adresses  #816.
NOTE: This PR adds the external server (45 MB), in src/test/resources. Until I findd a better approach, I create this pull request to get some initial feedback :)
Implements:

Removing Nashorn from the TypeScript plug-in.
Connecting to an external server that implements merger and beautifier.
Changing tests to fit new functionality

@devonfw/cobigen

Closing because I have opened a clean pull request #862


nbdkit server crashes with "munmap_chunk(): invalid pointer" when nbdcopy starts

Hi,
I'm trying to migrate an Oracle Linux 8 VM from VMware to Openstack.  From what I can see it's successfully creating the snapshot, as I can see that in the vSphere portal.  It starts the nbdkit server successfully, and I can see the pid for the server and the socket within /tmp.  It correctly identifies that this is a first-time copy and a full copy is needed.
But when it comes to launch nbdcopy, the server appears to crash, with an invalid pointer error.
This is the command we run, slightly redacted.  We have mirrored the docker image to our own harbor for simplicity.
docker run -it --rm --privileged -v /etc/hosts:/etc/hosts:ro -v /dev:/dev  -v /usr/lib64/vmware-vix-disklib/:/usr/lib64/vmware-vix-disklib:ro --env-file <(env | grep OS_) migratekit:latest  migrate --vmware-endpoint REDACTED_ENDPOINT  --vmware-username REDACTED --vmware-password REDACTED --vmware-path "/datacentre/vm/REDACTED"
I'm running on Oracle Linux 9.4, Docker 27.2.0.  Our VDDK from VMware is 8.0.3.
I've seen that you've suggested on #9 to use Docker on Ubuntu.  We switched from Podman to Docker but that didn't help.  I don't think I can move from OEL9 to Ubuntu as I'm restricted as to what I can use.
I've tried modifying the golang code to turn on verbose debug, and add additional logging.  Right before the invalid pointer error, I see two warnings...I'm not sure if they are related or not.
Here's the output of nbdkit/vddk immediately after the nbdcopy begins.
Running command: /usr/bin/nbdcopy --progress=3 nbd+unix:///?socket=/tmp/migratekit-3504199861/nbdkit.sock /dev/vdd --destination-is-zero  destination=/dev/vdd source="nbd+unix:///?socket=/tmp/migratekit-3504199861/nbdkit.sock"
nbdkit: debug: accepted connection
nbdkit: vddk[1]: debug: vddk: preconnect
nbdkit: vddk[1]: debug: newstyle negotiation: flags: global 0x3
nbdkit: vddk[1]: debug: newstyle negotiation: client flags: 0x3
nbdkit: vddk[1]: debug: replying to unknown (0xb) with unknown (0x80000001)
nbdkit: vddk[1]: debug: newstyle negotiation: unknown (0x8): client requested structured replies
nbdkit: vddk[1]: debug: replying to unknown (0x8) with unknown (0x1)
nbdkit: vddk[1]: debug: newstyle negotiation: unknown (0xa): client requested export ''
nbdkit: vddk[1]: debug: newstyle negotiation: unknown (0xa): set count: 1
nbdkit: vddk[1]: debug: newstyle negotiation: unknown (0xa): set base:allocation
nbdkit: vddk[1]: debug: newstyle negotiation: unknown (0xa): replying with base:allocation id 1
nbdkit: vddk[1]: debug: replying to unknown (0xa) with unknown (0x1)
nbdkit: vddk[1]: debug: newstyle negotiation: unknown (0xa): reply complete
nbdkit: vddk[1]: debug: newstyle negotiation: unknown (0x7): client requested export ''
nbdkit: vddk[1]: debug: vddk: open readonly=1 exportname="" tls=0
nbdkit: vddk[1]: debug: vddk: default_export readonly=1 tls=0
nbdkit: vddk[1]: debug: VDDK call: VixDiskLib_AllocateConnectParams ()
nbdkit: vddk[1]: debug: VDDK call: VixDiskLib_ConnectEx (h->params, 1, snapshot-76330, NULL, &connection)
nbdkit: vddk[1]: debug: VixDiskLib: VixDiskLib_ConnectEx: Establish connection using (null).
nbdkit: vddk[1]: debug: VixDiskLib: VixDiskLibConnectInt: Establish connection.
nbdkit: vddk[1]: debug: VixDiskLib: Resolve host.
nbdkit: vddk[1]: debug: VDDK call: VixDiskLib_Open (connection, [REDACTED] REDACTED/REDACTED.vmdk, 68, &handle)
nbdkit: vddk[1]: debug: VixDiskLib: VixDiskLib_OpenEx: Open disk '[REDACTED] REDACTED/REDACTED.vmdk'.
nbdkit: vddk[1]: debug: 2024-09-03T11:33:50.865Z warning -[00078] [Originator@6876 sub=DiskEnum] ScanSysfsDir: File path /sys/class/nvme not exist for nvme.
nbdkit: vddk[1]: debug: 2024-09-03T11:33:50.865Z warning -[00078] [Originator@6876 sub=DiskEnum] ScanSysfsDir: File path /sys/class/nvme-subsystem not exist for nvme.
munmap_chunk(): invalid pointer
nbdcopy: nbd+unix:///?socket=/tmp/migratekit-3504199861/nbdkit.sock: nbd_connect_uri: recv: server disconnected unexpectedly

Do you have any suggestions on where to go from here?  Myself and a colleague are comfortable modifying the golang code to provide any additional debug information that might be helpful.  But we are more reluctant to dive into the c code behind nbdkit. :-)

I've ran into this error a few times and it feels like it's an nbdkit issue.  I just pushed up #15 to get more debug logging out of virt-v2v but we can totally re-use that infrastructure for other components like nbdkit.
What flags did you add to get debug out of nbdkit?  I can add that to the --debug.

FTR: https://gitlab.com/nbdkit/nbdkit/-/issues/24

@paulfelton can you try running your commands with -v /sys:/sys -- It may be the lack of a proper /sys mount inside the Docker container.

https://github.com/vexxhost/migratekit/pull/16
Once we land this, I think this will solve your issue.

@paulfelton can you please try pulling down the latest image and see if it works now? :)

@mnaser Thanks for this.  I've retested and I'm no longer getting the invalid pointer.  I'm now getting connectivity problems to the esxhost:
nbdkit: vddk[1]: error: [NFC ERROR]NfcNewAuthdConnectionEx: Failed to connect to peer. Error: Error reading from vmware-authd socket. Reason: Resource temporarily unavailable
nbdkit: vddk[1]: error: [NFC ERROR]NfcEstablishAuthCnxToServer: Failed to create new AuthD connection: Error reading from vmware-authd socket. Reason: Resource temporarily unavailable
nbdkit: vddk[1]: error: [NFC ERROR]Nfc_BindAndEstablishAuthdCnx3: Failed to create a connection with server

@paulfelton https://www.veritas.com/support/en_US/article.100053291 is probably the same issue, I suggest having a look there.  The debug flag for nbdkit might include helpful info.

Yes, I think that's right.
We've moved beyond this now, and are failing later in virt-v2v, so I think we can close this issue - thanks for your help.


[Fix](Short Circuit) fix no project list in OlapScanNode

query like
select a, `__DORIS_DELETE_SIGN__` from  `test_tbl` WHERE k = '1111111'

will not contain project list on top of OlapScanNode, so for short circuit queries, we should use output exprs on top of plan fragment.
+----------------------------------------------------------------------------------------------------------------------------+
| Explain String(Nereids Planner)                                                                                |
+----------------------------------------------------------------------------------------------------------------------------+
| PLAN FRAGMENT 0                                                                                |
|   OUTPUT EXPRS:                                                                                |
|     k1[#0]                                                                                |
|     k2[#1]                                                                                |
|     k3[#2]                                                                                |
|     k4[#3]                                                                                |
|     k5[#4]                                                                                |
|     k6[#5]                                                                                |
|     k7[#6]                                                                                |
|     k8[#7]                                                                                |
|     k9[#8]                                                                                |
|     k10[#9]                                                                                |
|     k11[#10]                                                                                |
|     k12[#11]                                                                                |
|     new_column0[#12]                                                                                |
|     new_column1[#13]                                                                                |
|     __DORIS_DELETE_SIGN__[#14]                                                                                |
|   PARTITION: UNPARTITIONED                                                                                |
|                                                                                |
|   HAS_COLO_PLAN_NODE: false                                                                                |
|                                                                                |
|   VRESULT SINK                                                                                |
|      MYSQL_PROTOCAL                                                                                |
|                                                                                |
|   1:VEXCHANGE                                                                                |
|      offset: 0                                                                                |
|      distribute expr lists: k1[#0], k2[#1], k3[#2]                                                                         |
|                                                                                |
| PLAN FRAGMENT 1                                                                                |
|                                                                                |
|   PARTITION: HASH_PARTITIONED: k1[#0], k2[#1], k3[#2]                                                                      |
|                                                                                |
|   HAS_COLO_PLAN_NODE: false                                                                                |
|                                                                                |
|   STREAM DATA SINK                                                                                |
|     EXCHANGE ID: 01                                                                                |
|     UNPARTITIONED                                                                                |
|                                                                                |
|   0:VOlapScanNode(149)                                                                                |
|      TABLE: regression_test_serving_p0.tbl_point_query0(tbl_point_query0), PREAGGREGATION: ON                              |
|      PREDICATES: (k1[#0] = 1231) AND (k2[#1] =<PHONE_NUMBER>00000) AND (k3[#2] = 'ddd') AND (__DORIS_DELETE_SIGN__[#14] = 0) |
|      partitions=1/1 (tbl_point_query0)                                                                                |
|      tablets=1/1, tabletList=13203                                                                                |
|      cardinality=0, avgRowSize=0.0, numNodes=1                                                                             |
|      pushAggOp=NONE                                                                                |
|      SHORT-CIRCUIT                                                                                |
|                                                                                |
|                                                                                |
| Statistics                                                                                |
|  planed with unknown column statistics                                                                                |
+----------------------------------------------------------------------------------------------------------------------------+

Proposed changes
Issue Number: close #xxx

run buildall


Topics

Thanks for adding topics to your package!
Due to a bug, we were not validating the names on package upload. This is being fixed and will soon be enforced.
The guidelines for choosing topic names is https://dart.dev/tools/pub/pubspec#topics. Concretely we do not allow space.
Could you please update your topics names in the pubspec?
Thanks in advance.

The new version has been modified


Release CLI tool on tag push

Closes #123

@0crat status

@0crat status (here)

@g4s8 This is what I know about this job in CT2E6TK9B, as in §32:

The job #127 is in scope for 9hr
The role is REV
The job is not assigned to anyone
The budget is not set yet
These users are banned and won't be assigned:

@sammers21/z: This user reported the ticket

Job footprint (restricted area)

This pull request #127 is assigned to @g4s8/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @g4s8/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job

Job was finished in 0 hours, bonus for fast delivery is possible (see §36)


fix a trivial-case bug when using Yen

When images are filled with only one value, the histogram becomes trivial.

Fantastic, thank you!