RangeError: Invalid status code: 1

This piece of code
app.use(sassMiddleware({
  src: path.join(__dirname, 'public'),
  dest: path.join(__dirname, 'public'),
  indentedSyntax: true, // true = .sass and false = .scss
  sourceMap: true
}));

is causing the following error:
RangeError: Invalid status code: 1
    at ServerResponse.writeHead (_http_server.js:195:11)
    at ServerResponse.writeHead (/Users/raigovind93/LPN-Fitness-App/node_modules/on-headers/index.js:55:19)
    at ServerResponse.writeHead (/Users/raigovind93/LPN-Fitness-App/node_modules/on-headers/index.js:55:19)
    at ServerResponse._implicitHeader (_http_server.js:159:8)
    at ServerResponse.end (_http_outgoing.js:621:10)
    at ServerResponse.end (/Users/raigovind93/LPN-Fitness-App/node_modules/express-session/index.js:354:19)
    at ServerResponse.send (/Users/raigovind93/LPN-Fitness-App/node_modules/express/lib/response.js:211:10)
    at done (/Users/raigovind93/LPN-Fitness-App/node_modules/express/lib/response.js:967:10)
    at Object.exports.renderFile (/Users/raigovind93/LPN-Fitness-App/node_modules/pug/lib/index.js:422:12)
    at View.exports.__express [as engine] (/Users/raigovind93/LPN-Fitness-App/node_modules/pug/lib/index.js:465:11)
    at View.render (/Users/raigovind93/LPN-Fitness-App/node_modules/express/lib/view.js:128:8)
    at tryRender (/Users/raigovind93/LPN-Fitness-App/node_modules/express/lib/application.js:640:10)
    at Function.render (/Users/raigovind93/LPN-Fitness-App/node_modules/express/lib/application.js:592:3)
    at ServerResponse.render (/Users/raigovind93/LPN-Fitness-App/node_modules/express/lib/response.js:971:7)
    at /Users/raigovind93/LPN-Fitness-App/app.js:97:7
    at Layer.handle_error (/Users/raigovind93/LPN-Fitness-App/node_modules/express/lib/router/layer.js:71:5)

Could this be because the build is failing?
Here is my dependency in my package.json:
...
    "node-sass-middleware": "^0.9.8"
...

Anyone?

You mention this happens when you add node-sass, but nothing in the log information indicates the error is bubbling from there.
Can you confirm what version of Node and node-sass you're running?

Hi Nick, I finally figured it out... it was because I had a SASS syntax error, and debug: true helped me out. Still I feel like the non-compilation should throw a more helpful message..

@govindrai I didn't get how you figured it out I am struggling with the same issue.your suggestion will be helpful

i have a similar problem . but its when i created a PUT request with knex i always retured " Unhandled rejection RangeError 1 " please any solution  ?


Was passed main parameter '--project' but no main parameter was defined in your arg class

I get this when running M13.2:
Was passed main parameter '--project' but no main parameter was defined in your arg class
Usage: detekt [options]
Options:
--baseline, -b
If a baseline xml file is passed in, only new code smells not in the
baseline are printed in the console.
--config, -c
Path to the config file (path/to/config.yml).
--config-resource, -cr
Path to the config resource on detekt's classpath (path/to/config.yml).
--create-baseline, -cb
Treats current analysis findings as a smell baseline for further detekt
runs.
Default: false
--debug, -d
Debugs given ktFile by printing its elements.
Default: false
--disable-default-rulesets, -dd
Disables default rule sets.
Default: false
--filters, -f
Path filters defined through regex with separator ';' (".test.").
--format
Enables formatting of source code. Cannot be used together with
--config.
Default: false
--generate-config, -gc
Export default config to default-detekt-config.yml.
Default: false
--help, -h
Shows the usage.

--input, -i
Input path to analyze (path/to/project).
--output, -o
Specify the file to output to.
--output-format, -of
Specify the output format.
Default: XML
Possible Values: [PLAIN, XML]
--parallel
Enables parallel compilation of source files. Should only be used if the
analyzing project has more than ~200 kotlin files.
Default: false
--rules, -r
Extra paths to ruleset jars separated by ',' or ';'.
--useTabs
Tells the formatter that indentation with tabs are valid.
Default: false

I feel like an old artifact was uploaded since it still says input instead of project in the output of all command line options.

Rename -p to -i.

The gradle plugin is doing the work for me so I can't rename it.

That sounds not good. I can't reproduce it. If I seach for --project I get no matches.

Yeah already saw it. Could it be that an old artifact was uploaded? Or that there's a mismatch with versions?

Maybe you build detekt local and overwrote some of the jars? try cleaning your m2 and gradle caches for detekt und redownload everything?

Never mind. There's something mixed up on my machine. Got it resolved now.


Channel reset button in nsqadmin

channel reset time pick & button in topic's channel page, nsqadmin

@absolute8511 for your review

#51


Quick reply dialog is still white when using the dark theme

This also goes for the black theme.

Here's a reproduction courtesy of @nvbln :D

I'll try to work on this eventually, perhaps this weekend.


Run the cross-validation for multiple iterations

Splitting the sample into testing and training samples is very useful, but with such a small sample, the N=27 participants in the testing sample could differ considerably from the training sample by chance. One easy solution could be to take multiple testing and training samples (e.g., 1000 or 5000 iterations), and then concatenate the y_pred and y_test results from each iteration.
You could just put your existing code in a loop...
n_iter = 5000
y_preds = []
y_tests = []

# scale before splitting into test and train samples
X_sc = StandardScaler().fit_transform(X)

for i in range(n_iter):
    # take a new testing and training sample
    X_train, X_test, y_train, y_test = train_test_split(X_sc, y, test_size = 0.25, random_state = i)
    y_tests.append(y_test)  # store the y_test sample
    # fit the logistic regression
    classifier = LogisticRegression(random_state = i, solver ='liblinear')
    classifier.fit(X_train, y_train)
    # get the y predictions and store
    y_pred = classifier.predict(X_test)
    y_preds.append(y_pred)

You could then generate the confusion matrix after concatenating the lists of arrays (e.g., np.concatenate()). If you scaled this by the sum of the array (i.e., cm/np.sum(cm)) then you would get probabilities for hits, misses, false positives and true negatives.

Just to add to this, concatenating X_test in the same way will also let you calculate more informative ROC curves, e.g.,

Just to add this, although it's a bit trickier, you could also generate a more informative ROC curve using this method. Here, you could do both the classifier.predict() and classifier.predict_proba() methods inside each iteration of the loop, and save the outputs. You would then use the roc_auc_score() and roc_curve() functions outside of the loop, as you currently do, but using the concatenated test and predictions from each iteration as arguments.
I get something like this with this method

I see you've addressed this with a similar solution - cool! At the moment, your ROC curve is just the result of the last iteration. Don't forget to use the variable name for the results from all iterations when you plot the ROC curve (i.e., y_tests instead of y_test). To do this you'll also need to alter your code for the iterations to also store and concatenate X_test from each iteration (e.g., in a variable called X_tests).

When I try to do this and plot the ROC curve, it says " Found array with dim 3. Estimator expected <= 2." Idk why, since I altered the code accordingly

Sorry Jack! Many things going on right now, I decided to let the ROV curves out since the measures should be enough. Currently working on interpreting the output and writing the discussion, and that should be it.

Okay! Not sure what the problem is, but if you decide to include it and want me to have a look at the code let me know :slightly_smiling_face:


Migrate log4j-osgi-test to JUnit 5

Hello 👋
We are from Neighbourhoodie, the implementation partner of the STF Bug Resilience Program. This work is part of our agreed Milestone 1. Upgrade from JUnit 4 to JUnit 5. This PR migrates the tests located in log4j-osgi-test.
Thank you!
Checklist

Base your changes on 2.x branch if you are targeting Log4j 2; use main otherwise
./mvnw verify succeeds (if it fails due to code formatting issues reported by Spotless, simply run ./mvnw spotless:apply and retry)
Non-trivial changes contain an entry file in the src/changelog/.2.x.x directory
Tests for the changes are provided
Commits are signed (optional, but highly recommended)

We have some questions we would like to ask. We've found that the test suites CoreOsgiTest, JULProviderTest and SLF4JProviderTest are using PaxExam, and the current version does not support JUnit 5. Before attempting an upgrade, we were wondering, does this feature need to be updated? We are missing some context, what is being checked by using this runner? Any help in this regard is greatly appreciated. Thank you!

Before attempting an upgrade, we were wondering, does this feature need to be updated? We are missing some context, what is being checked by using this runner? Any help in this regard is greatly appreciated. Thank you!

Since PAX Exam is basically dormant, it would be better to move those tests to a different framework, but honestly I don't know which. @HannesWell, any suggestions?
Frameworks like Testcontainer for Java or Arquillian are not very OSGi friendly. The "new" osgi-test assumes that the test is already running in an OSGi environment. Maybe we could use something like laeubisoft/osgi-test-framework or the BND Testing Plugin. I will experiment with the latter. We already use BND plugins and they work well. I would like to keep the list of Open Source vendors we rely on minimal.

@vy, should we move log4j-osgi-test to the Samples repository (which is our integration test repo).


Invalid fields are not saved

If a submitter changes a field value to something invalid, for example by removing the proposal's description on page 1, they can submit that page or navigate away without that field actually being saved.
If a field is left blank from the start, this behaviour is not observed.
This can lead to problems if a user intends to leave a field blank. The application does not warn them about the missing information but keeps the old value. That old value may be not at all what the user wanted to submit.

The desired behaviour is already present for simple model fields with blank=True and soft validation, except for Create forms which really do need a value to go to the next step. That is good. I currently believe the bug comes from ConditionalModelForm and check_dependency(), which bypasses soft validation . I'm looking into a neat way of fixing this.

I think this is still happening? I noticed it again today ...


Request: Refactor to provide reusable widgets that can be used in other Urwid projects

I think several widgets would be helpful to have available for reuse elsewhere:

GridListWidget
Simplified widget(s) - given a fixed row count, render a (Pillow) Image using Kitty, iTerm, Sixel, or ANSI.
I have separate widgets that do this in PRs for the toot project: ANSI widget and SIXEL widget

Thanks for the request.
As i mentioned here, it will take one or two releases before this happens.
It's not that I can't release the widget now but for the following reason, I don't want to.
In my TUI, I implemented mechanisms to render images asynchronously (using a mixture of threading and multiprocessing). I plan to implement these mechanisms as classes in the library first.
Without asynchronous rendering, the widget will be of little or no use... rendering images will delay UI drawing, input processing, etc.
What do you think?

Thinking about it again... it wouldn't actually do any harm releasing the widget(s) now and then releasing the asynchronous rendering as an improvement later on.

I forgot to mention that SIXEL support, that isn't available yet... I intend to work on it towards the release after the next.

I just checked back here now and noticed you mentioned the GridListBox widget. Thinking about it now, I have a few concerns:

Such a widget actually seems to be outside the scope of this library :thinking:.
It was a really hacky thing, it basically uses a GridFlow widget and purges out the contents of its underlying pile into a ListBox, using some of the private interface of the widgets. The square-shaped LineBox you see within the grid is another widget LineSquare which monkey-patches the wrapped widget of a LineBox with another widget LineSquareMiddleColumns.
If you read through its code, you'll notice it's very closely tied to other parts of the viewer's codebase. In fact, it works hand-in-hand with a thread that [re-]populates the grid continuously.

You're free to reuse and modify the code in another project if needed but I should note that if I were to re-implement the widget(s) today, I'd do them differently.


Add "DIVERS" to anrede enum

this was thought to be an example for demonstrating the git workflow ;)

Which got workflow? ^^

🤿

we can re-open that later. as i said it was mainly for demo purposes


New feature request: Display air quality parameters based on pincode

Add a button called "pollution" which takes you to a different page where current air pollution data is displayed based on the Pincode entered by the user.
NOTE: Use only the current air pollution data for this issue. And everything should be handled through the Tkinter GUI only
Reference for the API VERY IMPORTANT! https://openweathermap.org/api/air-pollution
You will have 30 mins to work on this issue and report updates to the maintainer. If the maintainer is satisfied with your progress, time may be extended.

can you assign this to me

Go for it! Will check on it in 30 mins :)

hey @charan2308  can i work on this pls?

@Ganther3301 go ahead , need updates in 30 mins

Hi, can I work on this?

hey can i be reassigned

@Ganther3301 go ahead , need updates in 30 mins

okay so i've all my api requests ready. i just have to add it to the tkinter ui

I would like to work on this

@ad1tya24 I have already assigned this issue to someone who had requested earlier. Hence not able to assign the issue to you :(


Add support for Repository Policy

Issue: https://github.com/aws-controllers-k8s/community/issues/582
Description of changes:

Added a new field named Policy to the Repository CRD (from the
SetRepositoryPolicy API Call.
Add e2e tests for CRUD operations on the Policy field
Refactor setAddtionalFields hook function
Fix some e2e rare error appearing when trying to patch an outdated CR

By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache 2.0 license.

/lgtm


Slide From Bottom

Hello, basically i have a view at the bottom of my view controller that I want to drag from it's position at the bottom to the very top of my view controller. Any suggestions? Similar to twitters post view

What about simply manipulating the view (within one View Controller) or choose an animation you want in a transition(a new View Controller on the previous one)? Sound like what you want doesn't need Dynamic while the twitter's implement is done by segue I guess.


Fix collection drawer render order bug

Summary
I've fixed a bug with the way ReorderableList elements were being drawn in CollectionEditor/GenericPropertyDrawer that should result in each item in the collection being rendered properly in the ReorderableList itself inline with their indexed position rather than directly below the list and in between the developer description. The chief thing to note here is that usage of ReorderableList is akin to a property drawer in that it does not respect GUILayout elements.
Before

After

Testing
I've created some test collection variables in an additional test commit to inspect and if there any custom types that would normally span a single line in height they should also work with this change. One of the results of this fix is that each element should be sized properly for the height of the property drawer rendering it.
Changes
Fixed issue with reorderable list elements

Renamed the existing method DrawPropertyDrawer to DrawPropertyDrawerLayout to indicate it used the native Unity auto-layout controls and created a new method using the older method name that uses the non-layout controls. Added method documentation to both explaining their usage.
Changed BaseVariableEditor to use the new DrawPropertyDrawerLayout method name.
Cleaned up CollectionEditor and removed several redundant calls to modify the rect, redundant callbacks, etc..
Fixed a bug in CollectionEditor where the ReorderableList elements were being drawn by EditorGUILayout methods. These elements act similarly to PropertyDrawers in that they do not respect the auto-layout system methods and need to drawn using the non-layout methods that accept a Rect. I have added usage of callback elementHeightCallback so that the height of the element rect can vary depending on the generic T type element's property drawer and changed the element drawing logic to use a non-layout version of the same method from GenericPropertyDrawer. The end result is that these elements are now drawn inline with the reorderable list element instead of below the entire list.

Merged


Don't show Applies to on tools/armor

Having Applies to on tools/armor seems a bit redundant, since you know it applies to that item.
Maybe adjust the config option to switch Applies to between Yes/No/Only Books?

This has been added as a new config option in 1.3.5 for both 1.16/17


Update readme

@david Flagged you on this as I thought we could pull bits and pieces for the 'fiches' related to ECCC call

@aulemahal @Zeitsperre   first bit of desc for choice of ERA5-land ready for review.  Fancy markdown tricks / suggestions more than welcome.

Just pushed a few esthetic fixes as well as:
Use name "ESPO-R5 v1.0" (with the 5).
Use AHCCD (and not DCCAH).

Ahh, sorry guys. I pulled this branch into my main, so when I pused, it merged automatically.

ok no worries

@huard your adds as well as initial sdba methods should be in update_readme branch


how to write a c++ xgboost predictor

Hi, all
I write a xgboost predictor which build tree from xgboost dump file.
But I can not get same result as the python api.
Train Model
def xgb_train(train_file, save_file, eval_file=None, num_round=10, param=None):
    if param is None:
        param = {
            'silent': 1,
            'objective': 'reg:logistic',
            'nthread': 32,
            'eval_metric': ['auc'],
        }

    dtrain = xgb.DMatrix(train_file, missing=np.nan)
    if eval_file is not None:
        deval = xgb.DMatrix(eval_file, missing=np.nan)
        eval_list = [(dtrain, 'train'), (deval, 'eval')]
    else:
        eval_list = [(dtrain, 'train')]

    bst = xgb.train(param, dtrain, num_round, eval_list)
    bst.save_model(save_file)
    bst.dump_model(save_file + ".raw")
xgb_train("train_data_path", "mode_name")

this code will generate two model file, one binary(mode_name), the other is raw text (model_name.raw).
I use model_name.raw to build tree in c++, and use binary model in python.
Python Way
import xgboost as xgb
click_bst = xgb.Booster(model_file='binary_model_path')
feature_matrix = np.zeros((1, 60))
# here omit some code to generate feature matrix
xgb_matrix = xgb.DMatrix(feature_matrix, missing=np.nan)
click_bst.predict(xgb_matrix)

C++ way
I build tree according to text dump model, then find leaf node for each tree based on feature_matrix, finally add all leaf node value to get the predict score.
But I found the score is quiet different to python result.
Here is my c++ code.
I would like to know is it correct to add all leaf value to get the predict when my objective function is reg:logistic.

i have the same problem, do you know the reason?

@yangl326-Dylan I have find the problem.
You need to set the basic score to 0 if you use reg:logistic.
and Do sigmoid after sum all the score in the leaf.

yes
is the problem solved?


WIP: Add 'androguard' executable script

WIP: Only for androaxml for now
See issue #561
How this PR improves the project:

New users will intuitively try the command androguard. By adding this and the use of --help they can discover all of the functionality of androguard.
Moving code to the androguard directory makes it easier to test and possible to import in other code.
By not removing the scripts I think this is backwards-compatible (I'm not sure how big the userbase of androguard is). I would, however, suggest to deprecate the old top level scripts.

@reox Let me know what you think. If this looks good to you, I would refactor the other commands as well. When you now install the package, you can do:
$ androguard --help
Usage: androguard [OPTIONS] COMMAND [ARGS]...

  Androguard is a full Python tool to play with Android files.

Options:
  --version  Show the version and exit.
  --help     Show this message and exit.

Commands:
  androaxml  Parse the AndroidManifest.xml.

and then
$ androguard androaxml --help
Usage: androguard androaxml [OPTIONS] [FILE_]

  Parse the AndroidManifest.xml.

  Parsing is either direct or from a given APK and prints in XML format or
  saves to file.

  This tool can also be used to process any AXML encoded file, for example
  from the layout directory.

  Example:

      $ androguard androxml AndroidManifest.xml

Options:
  -i, --input TEXT   AndroidManifest.xml or APK to parse (legacy option)
  -o, --output TEXT  filename to save the decoded AndroidManifest.xml to
  --help             Show this message and exit.

nice! this looks very nice! Please go for the refactoring!

Like this PR 👍

I think I've added all commands to the androguard entrypoint.
Open questions for this PR:

androguard auto: What is a good one-line description for this?
androguard odd: Should I rename odd to decompile?

I'm not going to write tests in this PR for code that existed before and had not been tested before.

Very nice work!

Open questions for this PR:
* `androguard auto`: What is a good one-line description for this?

Mhh this tool is quite outdated and I don't even know how it works...
This is what the documentation says: https://androguard.readthedocs.io/en/latest/tools/androauto.html
I think it was ment to run some kind of scripts somehow...

* `androguard odd`: Should I rename `odd` to `decompile`?

The old name was androdd, right? So the actual name of the tool was dd. But decompile is better IMO.

Things open for other PRs:
* Default arguments for functions and for CLI should always be the same: [#561 (comment)](https://github.com/androguard/androguard/issues/561#issuecomment-426590486)

According to the developers of click you should not use the same function again... But they did not say anything for default arguments. Maybe have to open a discussion in the google groups or something and ask for a good advice how to deal with such a situation.

* Currently, a lot of functionality is in `androguard/cli/main.py`. Maybe this can be moved to other modules?

Maybe, for some parts. But we would really carefully look through the code and see which functions are useful in the modules and which are not...

* Testing: I have no idea how to unittest this repository. I would suggest to add `tox` (I'm not sure about nose, I've only used pytest recently)

Okay, so the current tests are in the tests folder. They are basic unittest tests which can be run via nose.

* Documentation: See [mpu](https://github.com/MartinThoma/mpu) and its [pypi page](https://pypi.org/project/mpu/0.15.0/)

Documentation, yes... So we have some here: https://androguard.readthedocs.io/en/latest/tools/tools.html
The problem is, that we can not automatically build it on readthedocs, as I can not set up webhooks. See #423 @adesnos ping

* Style guide: I think having a style which is checked is nice. `androguard` currently gives 1643 errors with `pep8`.

hehe... yeah, there were some attempts to autopep everything but there is so much more code which I did not took any look when I fixed it for python3, so probably autopeping everything again would be a good idea.

Btw: The scripts were tested just for error outputs. So we would run all scripts with common parameters on the APKs in this repo and raise errors if the script crashed. See the travis.yml.

@reox Ok, I removed the changes to "androauto" and changed "odd" to "decompile". From my side, this PR is ready to be merged. What do you think?

Two more ideas:

I guess you are the maintainer of androguard, right? Maybe we should add you to the setup.py
If you (as the maintainer) don't know what androauto.py is good for ... it might be a good idea to deprecate it. (See https://github.com/MartinThoma/mpu/blob/master/mpu/decorators.py#L26 )

Two more ideas:
* I guess you are the maintainer of androguard, right? Maybe we should add you to the `setup.py`

Uhm, at least I can upload it now to pypi :D So yes - maybe that would be a good idea.

* If you (as the maintainer) don't know what `androauto.py` is good for ... it might be a good idea to deprecate it. (See https://github.com/MartinThoma/mpu/blob/master/mpu/decorators.py#L26 )

There is plenty of stuff in androguard, where I have no clue what it does but some people seem to use it.
it looks like that androauto is older than the git repo, thus it is from the time where androguard was still on code.google.com and managed by mercurial... In principle it looks to be a useful tool. Maybe we should get more people involved into androguard who actually use it on a daily basis and can decide better what is deprecated and what is not.
For my case, I use androguard on a regular basis but only use a subset of the functions available. But it would be not wise to deprecate everything that I do not use...

I've added a help page which is similar to the other CLI tool help pages

Thank you very much for the PR!


NullReferenceException if BaseAddress not set on specified HttpClient

I've just been playing with refit in an existing application that already had HttpClient wired up through DI, and initially made a mistake during the implementation where HttpClient.BaseAddress wasn't being set before being passed to RestService.For<T>(HttpClient).
https://github.com/paulcbetts/refit/blob/afacf7061b96581c4761f5484d70defb432a57e4/Refit/RequestBuilderImplementation.cs#L366
This resulted in a NullReferenceException being thrown. I was able to track this down by the line numbers being present in the refit stack trace, so I could look it up in GitHub, but troubleshooting this would have been simpler if this had been validated by the call to RestService.For<T>(HttpClient) with a custom exception message.

The only concern here is that this could be a breaking change. You can get at the HttpClient instance today by putting an HttpClient Client {get;} member on your interface. Then you can set/change the BaseAddress property.
I think it's probably a bad idea to be changing the BaseAddress on an instance of the service, but it is allowed today. Granted, it'd still be allowed, but this would force someone to set a bogus BaseUrl first before setting/changing it later.
Thoughts @paulcbetts @Cheesebaron @bennor?

If you can't require BaseAddress when you call For() (since BaseAddress is mutable, I'd say you can't) then in code you can throw an InvalidOperationException when you try to use BaseAddress and it's null. That's a far better debugging experience then dealing with NullReferenceException.

I can't properly look at the code from on my phone to see how it might break anything, but I'm not opposed to throwing a more useful exception. It sounds like this is something that's already not working and all we need to do is make the problem clearer to users.
If that's all it is then I think it's a great idea. 👍

Fixed


fix: rename less-loader option

Fixes #241

Going to close this in favour of #243
Should have checked for an outstanding PR first 😅


Force convert all the diff content to UTF-8

Force convert all the diff content to UTF-8
What
We are facing exception:
/bundle/ruby/2.6.0/bundler/gems/rspec_junit_formatter-1129bf0d5fb4/lib/rspec_junit_formatter.rb:186:in `sub': invalid byte sequence in UTF-8 (ArgumentError)

Why
One of the diff string contains invalid UTF-8 characters.
E.g. string = "784: unexpected token at '\u001F\x8B\b'"
Then if you run string.sub('xxx', 'xxx') you will see the same exception be raised.
Solution
Force encode string to UTF-8 encoding at the beginning.

Hi @sj26
Could you take a look this patch and let me know if it looks good to you. If not, please let me know what should I do to make it able to be merged.
Thanks!

Hi @agate, sorry for the delay. This should already be handled by escape(...). Can you provide a failing example?

Sorry that I didn't scan all the methods in RSpecJUnitFormatter. Seems escape will also have this problem. Let me show you some examples.
> RSpecJUnitFormatter.new("").send(:escape, "test")
 => "test"
> RSpecJUnitFormatter.new("").send(:escape, "\u001F\x8B\b")
Traceback (most recent call last):
        7: from /Users/foobar/.rvm/rubies/ruby-2.6.3/bin/irb:23:in `<main>'
        6: from /Users/foobar/.rvm/rubies/ruby-2.6.3/bin/irb:23:in `load'
        5: from /Users/foobar/.rvm/rubies/ruby-2.6.3/lib/ruby/gems/2.6.0/gems/irb-1.0.0/exe/irb:11:in `<top (required)>'
        4: from (irb):16
        3: from (irb):16:in `rescue in irb_binding'
        2: from /Users/foobar/.rvm/gems/ruby-2.6.3/gems/rspec_junit_formatter-0.4.1/lib/rspec_junit_formatter.rb:165:in `escape'
        1: from /Users/foobar/.rvm/gems/ruby-2.6.3/gems/rspec_junit_formatter-0.4.1/lib/rspec_junit_formatter.rb:165:in `gsub'

Same idea to strip_diff_colors:
> RSpecJUnitFormatter.new("").send(:strip_diff_colors, "test")
 => "test"
> RSpecJUnitFormatter.new("").send(:strip_diff_colors, "\u001F\x8B\b")
Traceback (most recent call last):
        6: from /Users/foobar/.rvm/rubies/ruby-2.6.3/bin/irb:23:in `<main>'
        5: from /Users/foobar/.rvm/rubies/ruby-2.6.3/bin/irb:23:in `load'
        4: from /Users/foobar/.rvm/rubies/ruby-2.6.3/lib/ruby/gems/2.6.0/gems/irb-1.0.0/exe/irb:11:in `<top (required)>'
        3: from (irb):19
        2: from /Users/foobar/.rvm/gems/ruby-2.6.3/gems/rspec_junit_formatter-0.4.1/lib/rspec_junit_formatter.rb:181:in `strip_diff_colors'
        1: from /Users/foobar/.rvm/gems/ruby-2.6.3/gems/rspec_junit_formatter-0.4.1/lib/rspec_junit_formatter.rb:181:in `sub'
ArgumentError (invalid byte sequence in UTF-8)

So for sure I need to update this PR. But I noticed that strip_diff_colors didn't call escape method. So even if I fix this problem in escape the strip_diff_colors method will still raise exception. Then the question will be do you want me to add encode('UTF-8', 'UTF-8', invalid: :replace) inside of escape method and then call escape inside of strip_diff_colors? Or do the encode for both methods?

strip_diff_colours has been calibrated to strip expected rspec escape codes in an expected place, and where there is no other way to remove the colours. It's output is consumed by a thing which should then call escape on the output. The idea is that we don't want to strip legit codes from example output, like for example if you were working on a library which ansi colourises things.
I can fix it if you can tell me the version of rspec, rspec_junit_formatter you're using, and provide a spec file which causes bad output?

@agate are you able to supply a failing case, here?

Sorry. I wasn't able to find out the output that we use to see. I was hoping to reproduce this in our CI. But after weeks wait. We still not able to reproduce the problem. I suggest we can just close this PR. And when I see the error shows up again I will reopen one.

Sorry @agate, thanks for understanding — please keep an eye out and if you find a reproducible case we can fix it. 🙌


Fix another typo

Just noticed that this would be obviated by #136, so feel free to close in favor of that one.

Yeah, closing in favor of #136, since that will be merged soon. Thank you for the catch, regardless!


Feature Request: Hold CTRL to Measure from Selected Token

A feature of show drag distance was to hold CTRL when selecting a token to get a ruler to appear coming from that selected token, would be amazing if you could add that to this module!

I don't think that this functionality fits well into Drag Ruler, as it's a fundamentally different function from what drag ruler does. For this reason I've put this runctionality into a separate module called Rangefinder
(Duplicate issue #6)


Change SimplyClip version to 4 in manifest

This can help in identifying new versions of the code.

Fixed with 1e2ce32eae4cf84bbb87ad6fd5339921bf901558


Add back the short forms of --mode (-m) and --version (-v).

Eventually we'll want to transition these flags to be subcommands
instead, but for the time being, these short names were accidentally
dropped in the migration to ArgumentParser, so put them back for
now.

No worries, I should have caught it! But I mostly run swift-format in default mode and rarely use the flag. Clearly we need better end-to-end tests 🙂


Definition edit not opening in new buffer, tabe weird behaviour

Describe the bug
When in definition

editing does not open in new buffer, floating window stays open
tabe does open new buffer, but behaves weirdly when executing :q, it looks like duplicate buffers or something

https://user-images.githubusercontent.com/54907476/215755804-30a96009-82ef-439a-b614-e5e030e32364.mp4
Steps to reproduce

open nvim
wait for lsp to load
go to definition
edit / tabe

Expected behavior

when pressing enter it should close the preview window and open in buffer
when doing tabe it should close the preview window and not duplicate buffers

Neovim version (nvim -v)
NVIM v0.9.0-dev
lspsaga commit
ffddb6242cd8e9f26e95c8f625ad4266636d8ef8
Terminal name/version
Windows Terminal (wsl, zsh)

this is peek_definition allow you edit it in float window. it's not preview window. and it's same buffer. if you want open a new buffer use Lspsaga goto_definition

@glepnir Iam sure this was possible before tho. I used to open the floating window (peek_definition) and if I wanted, I then jumped to the definition in new buffer using enter. Is it somehow possible to do that now?

just

Now that I understand how it works its not a dealbreaker tho, I can just close the peek_definition window and then use goto_definition...

fixed. it's bug of keymap not works fine in peek definiton window. it still support you jump from peek

I change double linked list to a stack to store the data. not test it. sorry . but i still think this is not a best way .


Move NetworkServiceManagerStatus.URL field to NetworkServiceManagerSp…

Description

Implementation of #1697
Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

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

Checklist:

@alex-yust Please rebase on master so we can see if it passes CI or not.

@alex-yust Please rebase on master so we can see if it passes CI or not.
@edwarnicke Ok, 2 aws-related test fail now

The thing causing your CI failures should be fixed by: #1740 so if you rebase you should pass CI :)


Cast JSON string to object when calling MeasurementJSON.get() function

Hi,
when updating ooni-wui in NetProbe the iOS app, I had to change the way to inject json from
NSString* MeasurementJSON = [NSString stringWithFormat:@"\n var MeasurementJSON = {  \n"
                             "get: function() {  \n"
                             "return "
                             "'%@';"
                             "   } \n"
                             "}", content];

To
NSString* MeasurementJSON = [NSString stringWithFormat:@"\n var MeasurementJSON = {  \n"
                             "get: function() {  \n"
                             "return "
                             "%@;"
                             "   } \n"
                             "}", content];

to make it work again.
Unfortunately in the Android app something broke with the new wui and I don't have so much control to the injected json, maybe @hellais could help me identify the problem.
Android code:
    WebView wv = (WebView) myScrollView.findViewById(R.id.webview);
    wv.getSettings().setJavaScriptEnabled(true);
    String jsonContent = LogUtils.readLogFile(c, jsonfile);
    wv.addJavascriptInterface(new InjectedJSON(jsonContent), "MeasurementJSON");
    wv.loadUrl("file:///android_asset/html/webui/index.html");

public static class InjectedJSON {
    private String jsonData;

    public InjectedJSON(String json) {
        jsonData = json;
    }

    @JavascriptInterface
    public String get() {
        return jsonData;
    }
}

Oh, yes I see what is the issue here. Basically we expect the get() function to return an object and not a string (https://github.com/TheTorProject/ooni-wui/blob/master/src/app/components/measurement/measurement.factory.js#L7).
In iOS this is possible, but android doesn't allow you to inject an object directly so we need to change the behaviour.
The fix to this would be to call get() and run the result through a JSON.parse() to convert it to an object.
I can't seem to find history of it working otherwise in older versions, but it may be due to the fact that the code was moved from one repo to another.

Alright, I understand, so there will be a fix in this repo soon?
I wont touch anything in the netprobe-android for now.

Pushed a fix for this. You should be good to go now.

@hellais  Could this be the problem why now doesn't work in iOS?

In the iOS version you should return a string in the javascript code.
NSString* MeasurementJSON = [NSString stringWithFormat:@"\n var MeasurementJSON = {  \n"
                             "get: function() {  \n"
                             "return "
                             "'%@';"
                             "   } \n"
                             "}", content];

Is that not what you are doing?

No, now it is
NSString* MeasurementJSON = [NSString stringWithFormat:@"\n var MeasurementJSON = {  \n"
"get: function() {  \n"
"return "
"%@;"
"   } \n"
"}", content];
And it doesn't show any result.
But even with your version I can only see the result page of tcp connect and dns injection tests.  (and ndt) Not Web connectivity and http invalid request line.
Any hint?

@lorenzoPrimi yeah, you need to return a string instead of an object on iOS (this is because in iOS you can inject objects into javascript, but you can't do the same with Android, so the cross platform solution is to always use strings).
It's very strange that you would only see a result page for tcp_connect and dns injection tests. Do you have some way of extracting error messages from the webkit view? If so it would be useful to have the console output of the webkit view.

Here are the .js of the two tests. Try it yourself building the mobile.html page.
Btw, I fixed the iOS app injecting a string instead of object.
web_connectivity.js.txt
http_invalid_request_line.js.txt

For the web_connectivity test the issue is that the ' chars are not being escaped in the JSON document.
You will probably need to do something like:
content = [content stringByReplacingOccurrencesOfString:@"'" withString:@"\'"];

good guess, but still doesn't work
web_connectivity.js.txt

@hellais any other guess or solution to fix this?

So I looked a bit more into this and it seems like the issue is that for multiple entry JSON fields we need to present an intermediate view to select which specific entry (and at what index) you need to present. When that is selected you can then redirect to the page that displays the web connectivity test view.
I will try to write this up in a branch on netprobe-ios.

So further debugging shows that the MeasurementJSON is actually malformed.
If I print it out I get:
2016-09-16 16:48:26.697 NetProbe[41291:2852581]
 var MeasurementJSON = {
get: function() {
return '(null)';   }
};

Protip: If you need to debug the WebKit views you need to do this from Safari.
To enable debugging in Safari, open it go to Preferences -> Advanced -> Show Develop menu in menubar.
Then fire up the app and when the webview is loaded you will see under the Develop tab of the Safari app on your computer a simulator item that contains the name of the app. There you will be able to see the console and the inspector.
In order to trigger console.log messages and such you will have to reload it.
To make the reload work though I had to change the way the page is loaded to:
[self.webView loadHTMLString:htmlData baseURL: [NSURL fileURLWithPath:pathToHtml]];

Ignore my comment  about null being returned. That was due to me having messed up some of the segue's.
I do however see that the JSON returned is malformed and I get some parsing errors with it.
SyntaxError: JSON Parse error: Unterminated string — mobile.bundle.js:36645

Ok, pushed a fix for this on the netprobe-ios branch fix/viewresults.
I believe this issue can now be closed since it's not a problem with ooni-wui.


Resolve ENS names for profiles

Is your feature request related to a problem?
No. It's just a nice to have.
Describe the solution you'd like.
When I got to my profile, the URL is ugly. e.g. proofofhumanity.id/profile/0xstuff.....
I have an ENS name associated with the wallet I used for my PoH profile.
So I'd like to augment the controller that resolves profiles by checking to see if the parameter entered is an ENS name and then resolving the underlying address.
Would be cool to be able to show my identity as https://proofofhumanity.id/profile/cwynar.eth
I'm hoping other people think so as well.
I'm going to take a look at the code and see if can't write a PR for this.
Describe alternatives you've considered.
I have considered none. Resolving ENS names seems.. like a pretty narrowly defined thing.

This would be amazing nicolas, I'd be happy to review the PR

Hi, I tried to do this but I'm stuck with a problem:
I tried to use the @ensdomains/ensjs library, and the package's README gives this as an example:
import ENS, { getEnsAddress } from '@ensdomains/ensjs'

const ens = new ENS({ provider, ensAddress: getEnsAddress('1') })

ens.name('resolver.eth').getAddress() // 0x123

I couldn't understand what the provider is supposed to be. I came accross some 'provider' objects in the PoH project, but I don't understand what they are / do. Maybe if someone can give me an example I can get to it quicker

Provider is a web3 provider. https://docs.ethers.io/v5/api/providers/provider/

Ha. Looks like you all are getting farther with this than I did. I dug around for a couple hours, got confused, and forgot about this.

Provider is a web3 provider. https://docs.ethers.io/v5/api/providers/provider/

Thanks @ferittuncer I will look into it
@RyanCwynar Yep, I also got there heh

Aight, got something working in my fork.
You can try it with git clone --branch feat/ens-name-profiles git@github.com:nicolasgargano/proof-of-humanity-web.git
What works so far:

https://localhost:3000/profile/santi.eth will take you to Santi's profile and keep the url.
An invalid name like https://localhost:3000/profile/santiddd.eth behaves the same as an inexistent wallet address.

However, I'm doing something wrong with the caching of the query because it doesn't load as fast as wallets urls on successive visits. Also, it feels hacky to do it in the relay provider rather than the profile page but I couldn't figure out another way. Both of these issues have to do with me not being familiar with Relay I think, so I'll read up on that and see if I can come up with something better.

that must be because it's making an extra REST call to resolve that ens name into an address every time. 🤔

Done on https://github.com/bilinkis/proof-of-humanity-web and it's live on https://proofofhumanity.org.


identity.email permission is not in OptionalPermission

References:

https://developer.chrome.com/docs/extensions/reference/identity/#type-ProfileUserInfo
https://chromium.googlesource.com/chromium/src/+/refs/heads/main/chrome/common/extensions/api/identity.idl#31

There might be more permissions missing, but I haven't seen others yet

Related: I wonder if these changes should be made to the source of the data instead because the global @types package probably needs the same changes that I suggest here.
https://github.com/jsmnbom/definitelytyped-firefox-webext-browser
I wonder if the efforts could be joined somehow? cc @jsmnbom

Thanks for the report, I will take a look.
I've looked into contributing to mozilla code, but it's a real big mess.
It's not just a matter of creating a simple PR. You have to learn a new, eh old and strange, vcs and other tools before you can even get to the point of contributing. For me the effort is not with it.


Support auto-prefixing of CSS styles set on views

Code like this.$.style.transform = 'foo' should actually be this.$.style.transform = this.$.style.webkitTransform = 'foo'. We should implement something like this.setStyle('transform', 'foo', this.id) for this purpose. A temporary fix for existing code is checked in at 25decb0a26c8d533b81e80c2ed386f15025fedad.

This should be nicely solved in U2.


implement jupyter kernel for tutd interpreter

It might be appealing to newcomers to the relational algebra to try our flavor or TutorialD in a jupyter notebook.

Documentation

Some jupyter users made the following suggestions:

autocompletion support
help/docs section (a la python kernel)
use previous result (a la "_" in python)
"magic" to connect to different database

I punted on:

autocompletion (not even the command line version has it yet)
previous result placeholder- commands can be of different types, so I'm not sure this makes sense

but I implemented:

magic commands which allow users to connect to databases not controlled by the kernel and to find documentation
a sample notebook which runs down the basics of TutorialD


Added option to compact the income vs. expense table so it uses less whitespace.

Explanation of Bugfix/Feature/Enhancement:
The income expense report uses too much damn whitespace.
Recommended Release Notes:
Added option to compact the income vs. expense table so it uses less whitespace.

Thanks!


Not able to add the matter device

Hi,
I am trying to add the matter device, but not able to establish the connection so basically I am not getting the call back in "AppCommissioningService->onCommissioningRequested" method.
Kindly help in this.

Please make sure to provide the information documented in "How-to-investigate-issues-with-GHSAFM-3p-ecosystem" if you'd like help from people in this community.


unable to install ryu in ubuntu 22.04

refer this https://youtu.be/q4wsx4u5juU?si=I9JXkqK4zJJ1AZVi

Thank you so very much
On Mon, Apr 8, 2024, 05:33 Dipesh Yadav @.***> wrote:

refer this https://youtu.be/q4wsx4u5juU?si=I9JXkqK4zJJ1AZVi
—
Reply to this email directly, view it on GitHub
https://github.com/chiragbiradar/DDoS-Attack-Detection-and-Mitigation/issues/19#issuecomment-2041800144,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3774JWLV574G5O2BBJ27HDY4IFYBAVCNFSM6AAAAABFUYJINSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBRHAYDAMJUGQ
.
You are receiving this because you authored the thread.Message ID:
<chiragbiradar/DDoS-Attack-Detection-and-Mitigation/issues/19/2041800144@
github.com>

Hey , If anyone finished the project , let me know I need your help


Disposing activeTerminal causes te renderer to freeze

In an extension, have

import * as vscode from 'vscode';

export function activate(context: vscode.ExtensionContext) {
	const disposable = vscode.commands.registerCommand('extension.helloWorld', () => {
		vscode.window.activeTerminal?.dispose();
	});

	context.subscriptions.push(disposable);
}

Open the terminal
Type in text (this is required, uninteracted terminals don't trigger this)
Run the command
After several seconds, the window will freeze

/duplicate https://github.com/microsoft/vscode/issues/76548

Oh yea, I didn't check but this happens when closing any terminal... pretty sure this is new to recent builds, this would otherwise make the integrated terminal pretty risk to use for me (having to kill all vscode windows if I accidentally close a terminal)

It could have been an OS update, it also seems to repro at different rates for different people, I guess depending on hardware specs and usage habits.

Sounds good. I will rebuild my PC in a week or two in order to attempt a workaround.
(Just waiting on the 3080 to ship 🙂 )

Jealous 🙄


Build dependency on CARGO_ENCODED_RUSTFLAGS causes needless rebuilds

In our CI/CD, we do something along the following lines:
cargo build --all-targets
maturin develop --manifest-path src/python_bindings/Cargo.toml
cargo test

The problem is that portable-atomic has a build dependency on CARGO_ENCODED_RUSTFLAGS, see https://github.com/taiki-e/portable-atomic/blob/0479f5b36c1e506fb61847e79de0dba4dc108bbd/build.rs#L41
For some reason, portable-atomic is built each time (3 times) rather than once. Maturin passes the exact same CARGO_ENCODED_RUSTFLAGS environment variable as Cargo does, but the rebuild is still triggered by Cargo, saying that the aforementioned environment variable has changed. Adding debug printing (with proper non-ASCII escaping) in the portable-atomic build.rs file confirms that the env var is identical on each command.
This could be very a bug in cargo itself, but since portable-atomic is the only crate I've encountered with this build-time dependency, I thought the would be the right place for an initial issue report.

This is probably due to maturin setting as CARGO_ENCODED_RUSTFLAGS what was originally set as RUSTFLAGS.
If so, changing this line to the following would fix the rebuild problem (although that is not a better fix here).
https://github.com/PyO3/maturin/blob/fd85ab4f2af21d6e04abf96de704f6b0a60500c0/src/compile.rs#L362
From:

build_command.env("CARGO_ENCODED_RUSTFLAGS", rustflags.encode()?);

To:

build_command.env("RUSTFLAGS", rustflags.encode_space_separated()?);

I think a better fix here is to add a method like encode_preserve that respects the format of the rustflags source to  in cargo-config2 and change maturin to use that.

I have managed to make a reproducible example: portable-atomic-maturin.zip
Typing:
cargo build --all-targets -vv
maturin develop -vv
cargo test -vv

will show for the last two commands:
       Dirty portable-atomic v1.6.0: the env variable CARGO_ENCODED_RUSTFLAGS changed
   Compiling portable-atomic v1.6.0

This is only reproducible if the following is added in `.cargo/config.toml':
[target.'cfg(target_arch = "x86_64")']
rustflags = "--codegen target-cpu=x86-64-v3"

Ah, that's a tricky issue involving interaction with config.
If cargo fixes a bug (https://github.com/rust-lang/cargo/issues/13003), there would be no need for erun-if-env-changed=*RUSTFLAGSs there in the first place although...

This is probably due to maturin setting as CARGO_ENCODED_RUSTFLAGS what was originally set as RUSTFLAGS.

I'm not sure I fully understand. Printing the content of the env var in portable-atomic build file shows the string to be completely identical between cargo build and maturin develop, including the weird ASCII separator character used by CARGO_ENCODED_RUSTFLAGS. Surely that should be enough for cargo not to rebuild portable-atomic, or is there a weird interaction with RUSTFLAGS and others that I'm not aware of?
(I've not tried your proposal yet, maturin is non trivial to rebuild from source and test)

I'm not sure I fully understand. Printing the content of the env var in portable-atomic build file shows the string to be completely identical between cargo build and maturin develop, including the weird ASCII separator character used by CARGO_ENCODED_RUSTFLAGS. Surely that should be enough for cargo not to rebuild portable-atomic, or is there a weird interaction with RUSTFLAGS and others that I'm not aware of?
(I've not tried your proposal yet, maturin is non trivial to rebuild from source and test)

Thanks to the detailed information provided, this turned out to be an irrelevant issue.

In my understanding, what is happening:

rerun-if-env-changed=CARGO_ENCODED_RUSTFLAGS is not triggered if CARGO_ENCODED_RUSTFLAGS is set by the cargo as described here.
https://github.com/taiki-e/portable-atomic/blob/0479f5b36c1e506fb61847e79de0dba4dc108bbd/build.rs#L38-L41
i.e., CARGO_ENCODED_RUSTFLAGS for rerun-if-env-changed is considered unset.

maturin sets the CARGO_ENCODED_RUSTFLAGS even if it has not changed, so the conditions for rerun-if-env-changed=CARGO_ENCODED_RUSTFLAGS to be triggered are met when building with maturin.
i.e., CARGO_ENCODED_RUSTFLAGS for rerun-if-env-changed is considered set.

There are several possible fixes here:

Fix a cargo bug (https://github.com/rust-lang/cargo/issues/13003) and remove rerun-if-env-changed=*RUSTFLAGS in the version that contains the fix. This is the best thing to do, and a PR to fix the bug has already been submitted, but it will take months to be available in stable even if it is merged right now.

Accept the Cargo bug and remove the rerun-if-env-changed=*RUSTFLAGS. I do not want to do this if possible.

Change maturin to not set CARGO_ENCODED_RUSTFLAGS if rustflags has not been changed. This is probably the most preferred way, because there are no adverse effects whatsoever and the fixes will be available as soon as they are released a new versions.

Change maturin to not set CARGO_ENCODED_RUSTFLAGS if rustflags has not been changed. This is probably the most preferred way, because there are no adverse effects whatsoever and the fixes will be available as soon as they are released a new versions.

Opened https://github.com/PyO3/maturin/pull/1978 with this approach.

https://github.com/PyO3/maturin/pull/1978 has been released in maturin 1.5.0.

 Impressive rollout with a new release of maturin within 24h of raising this initial issue.
We've tested maturin 1.5.0 and can confirm it fixes the rebuild issues. Timings of the CI/CD pipeline are much better.

FYI, the underlying Cargo bug has been fixed in the latest stable (1.79), and in the latest portable-atomic, the workaround in the portable-atomic side is only used for old Cargo.
https://github.com/taiki-e/portable-atomic/blob/6cc21f3701e66baec6172f9d511c0394ce9daa16/build.rs#L65-L72


remove unnecessary Function#call in ThrottleSink#end

Previously of c36db75fb8ddca310014c7e353e3970ba5f720d8 it was also ThrottleSink.prototype.end   = Sink.prototype.end;. Was there a reason why this has been changed?

Thanks @campersau.  I can't see any good reason for the change--seems like it was just a mistake.  Appreciate the fix :)


(feature) builds createATeam controller

functioning createATeam controller, but getting an error in the console (might a problem in the team.server.controller).

@jeffycyang will fix the bug


Updated adamw to use packed data types

Before Runtime
total average iteration time: 38.547570 ms
After Runtime:
total average iteration time: 37.901735 ms
Kernel development file specs:
Barely noticeable with the current test suite:
Before:
time gpu 0.0098 ms
After:
time gpu 0.0097 ms

I think the problem was it can't work with only 1 loop, it was skipping some of the elements for some of the arrays because fo the different f128/x128 sizes, here's my attempt at fixing that in Chris' kernel which seems to work (haven't looked into perf yet):
__global__ void adamw_kernel4(floatX* params_memory, const floatX* grads_memory, float* m_memory, float* v_memory, size_t num_parameters,
                              float learning_rate, float beta1, float beta2, float beta1_correction, float beta2_correction, float eps, float weight_decay,
                              unsigned int seed) {
   int idx = blockIdx.x * blockDim.x + threadIdx.x;
   int idx_offset = idx*x128::size;
   if (idx_offset >= num_parameters) { return; }

   x128 packed_grads_memory = load128(grads_memory + idx_offset);
   x128 packed_params_memory = load128(params_memory + idx_offset);
   for (int n = 0; n < (x128::size / f128::size); n++) {
    int idx_n_offset = idx_offset + n*f128::size;
    f128 packed_m_memory = load128(m_memory + idx_n_offset);
    f128 packed_v_memory = load128(v_memory + idx_n_offset);
    for(int k = 0; k < f128::size; ++k){
        int k_n_offset = k + n*f128::size;
        float grad = (float)packed_grads_memory[k_n_offset];
        float m = packed_m_memory[k];
        float v = packed_v_memory[k];
        // update the first moment (momentum)
        m = lerp(grad, m, beta1);
        packed_m_memory[k] = m;
        // update the second moment (RMSprop)
        v = lerp(grad * grad, v, beta2);
        packed_v_memory[k] = v;
        m /= beta1_correction; // Setting these values explicitly due to compiler error for modifying
        v /= beta2_correction; // packed128 values when using
        // update the parameters (weight/bias)
        float param = (float)packed_params_memory[k_n_offset] - (learning_rate * (m / (sqrtf(v) + eps) + weight_decay * (float)packed_params_memory[k_n_offset]));
        unsigned int random = Get2dNoiseUint(threadIdx.x, blockIdx.x, seed);
        // todo - explain stochastic rounding here
        stochastic_rounding(param, &packed_params_memory[k_n_offset], random);
    }
    store128(m_memory + idx_n_offset, packed_m_memory);
    store128(v_memory + idx_n_offset, packed_v_memory);
   }
   store128(params_memory+idx_offset, packed_params_memory);
}

Updated the PR to show the new kernel, it does have a speedup in the train loop for me of:
total average iteration time: 38.287047 ms
to
total average iteration time: 37.143633 ms


[lang] anchors are not supported

I've seen #172 and the limitations and this prevents processing/installing knative resources with ytt.
https://github.com/knative/serving/blob/6c3eb0433d235d5ba9768ada4e7be90ddeb74458/config/core/resources/configuration.yaml#L26-L55

we do want them to work since they may be used within third party configuration.

minimal reproduction: https://get-ytt.io/#gist:https://gist.github.com/cppforlife/f03ac62b09da71bd8549e372fb921bd6

this open PR to go-yaml fixes issue: https://github.com/go-yaml/yaml/pull/364; however, after the fix cases like knative config:
  - << : *version
    name: v1beta1

present a challenge since they introduce duplicate keys.

this open PR to go-yaml fixes issue: go-yaml/yaml#364; however, after the fix cases like knative config:
  - << : *version
    name: v1beta1

present a challenge since they introduce duplicate keys.

but the last key listed should just override no? I'm guessing there's something in the spec regarding the behaviour

Some additional conversation/context in slack.

Adding to list of issues to review during pre-IPM ==> recommended

any progress on that? I'm attempting to port anchor heavy configurations to ytt and lack support for anchors is making it increasingly complicated

@groundnuty asked

any progress on that? I'm attempting to port anchor heavy configurations to ytt and lack support for anchors is making it increasingly complicated

Unfortunately not, so far.
We've been in a "wait and hope for upstream fix" for some time now. So, while technically "in progress" is correct, we're really blocked on this upstream fix... (or we decide to find a way to land it in our own fork).
It be clear, it's in the Carvel maintainer's backlog 👍🏻 , but it's currently sitting below the OpenAPI Schema export feature 🤷🏻 .

@pivotaljohn maybe it's somehow connected with the memorable moment, that go-yaml stopped supporting flexible/forward anchors? It broke a lot of anchoring features... here is the discussion, where you can find a commit that broke that: https://github.com/go-yaml/yaml/issues/558
Just guessing here/trying to helpful :-)

Appreciate that, @groundnuty. I'm looking into this.

This also stops me from migrating my yamls to ytt.

FWIW, this issue was addressed on the main branch: #550 and will be included in the next release (coming soon).

Released in v0.39.0.

woot! thanks!


Add HOMEBREW_NO_INSTALL_CLEANUP.

This will be more useful when HOMEBREW_INSTALL_CLEANUP becomes the default.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Cleaner implementation rolled into https://github.com/Homebrew/brew/pull/5472.


[en] Missing command jumping into the folder before download the sample

add command to jump into the folder before download the samples
/assign @bradtopol

I mistook when submit the fixing


App should have a swipe up/down to hide/show filters

Add a swipe up / down gesture to show/hide filters.

I've figured out how to remove and add the filters, but a swipe might not be cohesive due to it overlapping with taking a photo. I think a button would work better, and I'm working on creating buttons for the filters and black/white colors.

The buttons don't work at all.  Please fix.

For the show/hide and color buttons, I can confirm that they work on my end, but you have to change to another filter for the changes to take effect. However, I'm trying to find a way to update the current filter, and I'm also considering your end might be due to layout changes.


feat(ast, ast_macros): apply stable repr to all #[ast] items.

closes #4296

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4373  👈
#4371
#4309
#4289
main

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

Join @rzvxa and the rest of your teammates on  Graphite

Aside from comments above, this looks good.
But... I suggest we hold off merging until rest of the stack is complete. With just this PR, the types' memory layouts are not optimal, and you can see it a bit on benchmarks. Let's wait until macro+codegen rearranges the fields so they end up in same order as they were before #[repr(C)] was applied.

@rzvxa Aside from comments above, and resolving conflicts, I think this PR + 3 more above it in this stack are ready to merge.


Setter (semigroupy) composition

Would this be useful as a PR?
I wrote this to compose PSetters. (I initially thought to do it with PTraversal but I think we'd need to widen the Applicative constraint to Monad to get join.)
  implicit class SetterExt[S, A, B](val x: PSetter[S, S, A, B]) extends AnyVal {
    def merge(y: PSetter[S, S, A, B]): PSetter[S, S, A, B] =
      PSetter(f => y.modify(f) compose x.modify(f))
  }

  def compositeSetters[S, A, B](h: PSetter[S, S, A, B], t: PSetter[S, S, A, B]*): PSetter[S, S, A, B] =
    t.foldLeft(h)(_ merge _)

sounds interesting, a few questions:

is it possible to have a PSetter[S, S, A, B] where A != B? if no then lets use Setter[S, A]
how do you think this relate to SemiGroup[Endo]? After all, modify(f) is an Endo[S]

more I think about it, more it looks like this composition forms another Category for optics where the zero is the optics that does does not point to any value. In facts, this can be generalised to Prism (and so Optional, Traversal, Setter and Fold), e.g.:
def zero[S, A]: Prism[S, A] = Prism(_ => Option.empty[A])(identity)

from haskell lens irc, composing Setter horizontally (as suggested here) is unsound in case they overlap, for example:
val s1, s2 = Setter.id[Int]
def inc(n: Int): Int = n + 1
def neg(n: Int): Int = -n

(s1 merge s2).modify(inc . neg)
  == s1.modify(inc . neg) andThen s2.modify(inc . neg)
  == neg(_) + 1 andThen neg(_) + 1
  == neg(neg(_) + 1) + 1
  == identity

(s1 merge s2).modify(inc) . (s1 merge s2).modify(neg)
  == (s1 merge s2).modify(inc)
  == _ + 2

Sorry for my neglect on this one. I did plan to be more useful :) Your conclusion is very reasonable.


Add devcontainer in support for codespaces

This one supersedes #130 - adding devcontainer with proper configuration makes 1) codespaces work, 2) docker creation seamless and 3) environments consistent.
Building the environment takes quite a while, because we don't just install a pretty sizable python image but also install all app dependencies & quarto. Let's see if we can improve this after.

@operdeck @yusufuyanik1 could you give the new codespaces thing a shot? Should be as simple as just hitting the code button on the homepage and then clicking Create codespace on master. Be sure to kill your codespace after using it on https://github.com/codespaces


ng-map and ng-show not showing full map

Great directive very easy to get working, possible problem when using with an ng-show command only the first tile of the google map is displayed, if you use the chrome debugger to try and see what is happening the rest of the map is displayed
I have made a plnkr to show this
http://plnkr.co/edit/i5Nme3VKeFao3Ievluip?p=preview

CDN is changed http://plnkr.co/edit/vLlulW7qtbG9q6Uup5d2?p=preview


ISPN-15556 Document vector search

https://issues.redhat.com/browse/ISPN-15556

Thank you @rigazilla and @marko-bekhta for the reviews.
I really appreciate it.
I tried to apply your ideas here.

Thank you @karesti


Config loading raises exceptions in Jest 27

Hello,
I am running into the following error message when using jest-erb-transformer with Jest 27:
TypeError: Cannot read property 'filter' of undefined

      at loadConfig (../node_modules/jest-erb-transformer/index.js:68:48)
      at Object.process (../node_modules/jest-erb-transformer/index.js:129:20)
      at ScriptTransformer.transformSource (../node_modules/@jest/transform/build/ScriptTransformer.js:612:31)
      at ScriptTransformer._transformAndBuildScript (../node_modules/@jest/transform/build/ScriptTransformer.js:758:40)
      at ScriptTransformer.transform (../node_modules/@jest/transform/build/ScriptTransformer.js:815:19)

I have done some digging in the way configs are loaded, and it seems that config object is not approached correctly. On the line raising the exception:
const erbTransformers = jestConfig.transform.filter(e => e[1] === __filename)

changing it to
const erbTransformers = jestConfig.config.transform.filter(e => e[1] === __filename)

fixes the issue.
When I use Jest 26 instead of Jest 27 in my project, this line does not raise an exception and the transformer works as expected.
I don't have enough knowledge or skill to point out the breaking change in Jest 27 with certainty, but I think somewhere in there the way options are passed to transformers has been changed. I think in https://github.com/facebook/jest/pull/10834?
PR with a hotfix for the issue: https://github.com/cpcwood/jest-erb-transformer/pull/17
Thank you for all of your work, I am very happy with this transformer, and please let me know if I can help in any way!

Hi @rjclaasen,
Yeah looks like they've updated how the config gets passed through in v27, thanks for catching this error and taking the time to make PR 💯
I'll take a look at the PR and update the circleci config to let it pass the CI checks.
Thanks, Chris

@rjclaasen I've merged and released your changes in v1.2.4
Thanks again for spotting this and contributing :)


Fuzzy search labels  when adding them to issues/PR's

Gitea version (or commit ref): *
Git version:
Operating system:
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
Would be nice to have fuzzy search labels when adding these to issues or PR's.
e.g fusejs
Screenshots

I'm looking to implement this, can someone help me get started? @silverwind

Looks like these issue/pull filterable dropdowns are implemented via semantic dropdown. I don't find where these dropdowns are initiailized. Any pointers?
Semantic dropdown supports fuzzy search via fullTextSearch setting , is there a non-obvious reason this isn't enabled?
Other filterable dropdowns (e.g. repo branch/tag selector) support fuzzy searching. They are implemented via initFilterSearchDropdown() or initFilterBranchTagDropdown() or a custom Vue component that (somewhat) supports fuzzy searching. Are there non-obvious obstacles to port the issue-property menus to these components?

I think most dropdowns are initialized via HTML only. I never dealth with these parts but I assume there must be a way to specify option in HTML too. If it's not possible, I guess you'd have to initialize them via JS.

this was implemented somewhere around 1.15


Add survey criteria

Loads all survey data for real time filtering.
Improves question/answer name handling, though indexing changes will
be needed to handle global search correctly.
Multi-select plus instance level data still to be dealt with.
Config changes will come separately since they'll break stored data.
Copies some filter generation code from entity group criteria but this
allows the config and selection data to be distinct from entity group
criteria.  There is already some divergence and this allows them to
evolve separately. This wasn't originally planned but handling
backwards compatability while switching to a new
plugin/config/selection data seemed pretty sketchy.
Fixes a bug where entity group criteria with modifiers but no
selection weren't handled correctly.
Testing is weird because we don't currently have survey data. The
basics are the same as entity group criteria so using the same data
for now but TBD how we want to handle this.

RE survey tests: Now that we have the AoU CT test data indexed in verily-tanagra-dev, we could run tests against that, as long as the VUMC folks are okay with that. I imagine they will be, since we got the okay to run tests against the 2019 test data -- but that was several years ago now, so probably good to double check.

RE survey tests: Now that we have the AoU CT test data indexed in verily-tanagra-dev, we could run tests against that, as long as the VUMC folks are okay with that. I imagine they will be, since we got the okay to run tests against the 2019 test data -- but that was several years ago now, so probably good to double check.

Sounds good. I'll check with them on Wednesday.


ToggleClass option

Hey I have code for a menu that opens in javascript like this:
    let hamburger = document.getElementById("hamburgerbtn");

    let mobileMenu = document.getElementById("mobileMenu");

    hamburger.addEventListener("click", function () {
      mobileMenu.classList.toggle("active");
    });

and I am trying to make this work in Java like this
    public void openHamburger( ) throws ClassNotFoundException {
        Element mobileMenu = (Element) Window.current().getDocument().getElementById("mobileMenu");
        mobileMenu.getClass().getClassLoader().loadClass("active");
    }

Either my code is wrong or is there no way of toggling the class ?

@lucas-goldner https://github.com/konsoletyper/teavm/pull/553 this PR fixes issue you mentioned. After it's merged and new dev teavm version released, you can use it as follows:
mobileMenu.getClassList().toggle("active");

@lucas-goldner https://github.com/konsoletyper/teavm/pull/553 this PR fixes issue you mentioned. After it's merged and new dev teavm version released, you can use it as follows:
mobileMenu.getClassList().toggle("active");


Help with Testing

I'm trying to follow along with the testing section in the documentation, but can't seem to get it working properly. Does anyone have any example apps with tests written for redux loop?
I'm calling an action that will kick off an Effects.promise in the reducer and I'm not seeing the state change at all. When I do a deepEqual it gives me the exact same state with this appended to the end:
{
  "args": [
   {
    "mediaType": "movie",
    "mediaId": 14
   }
  ],
  "type": "PROMISE"
 }

Any detailed documentation or examples would be greatly appreciated, thanks!

Can you share your reducer? It'll be easier to tell what might be going wrong

Closing until we get some more information
https://github.com/redux-loop/redux-loop/blob/master/CONTRIBUTING.md


Request to publish SIG Issue Triage Guide

I'd like to publish the below SIG Issue Triage Guide, which outlines and standardizes the process SIGs should use to triage GitHub issues into their backlogs. This guide was initially published internally at Amazon with an open source perspective, and all SIGs are currently following the process outlined. I'd like to move this guide to an accessible location for all contributors to follow.
SIG Issue Triage Guide.pdf

This feels like something that should be part of the community repro. All other SIGs can then reference it as required.
As there was no public guide, we ended up writing our own. See:

https://github.com/o3de/sig-network/blob/main/TRIAGE_GUIDE.md
https://github.com/o3de/sig-content/blob/main/TRIAGE_GUIDE.md
etc.

Would be good to get these aligned with the common guide.

Next Steps

Community repo is the appropriate location
We should standardize label usage across all sigs
@forhalle will create a PR and have it submitted to the tsc mailing list (work with ninepoints)

@lmbr-pip submitted a PR to publish the guide to o3de/community: https://github.com/o3de/community/pull/146#pullrequestreview-1095354941
He will message the SIGs to advertise the guide, and will close this issue.

Delivered to community repro, this can be closed out.


Adding guidance for 10.6 Xcode users

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

This may not be wanted but it threw me for a while before I worked out what to do so thought that there may be other people who still use Xcode on SnowLeopard (for ruby on rails development and learning OpenGL) because don't want iOS features of later MacOS's that will slow the system down on their old hardware (I'm still happily using a MacBook 5.1 from Oct 2008)

This shouldn't be needed. Can you explain the issue you're seeing? Thanks!

Hi,
I am still running Xcode 3.2.6 on Snow Leopard so don't have the Command
Line Tools installed and https://github.com/kennethreitz/osx-gcc-installer
advised against installing if Xcode is installed.  That meant that 'brew
install ...' just stopped after updating the formulae and just printed:
Error: Your Command Line Tools are too outdated.
By reverting to the 1.0.9 tag and stopping the auto update then it allowed
me to continue using Homebrew on my outdated system.
Cheers,
Doug.
On 12 December 2016 at 13:26, Mike McQuaid<EMAIL_ADDRESS>wrote:

This shouldn't be needed. Can you explain the issue you're seeing? Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Homebrew/brew/pull/1651#issuecomment-266430483, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABe-E8fdQiM1L5OkOYjogRc_-WTKSRF-ks5rHUujgaJpZM4LKiWM
.

Ok. This should not be triggering for OS X 10.6. The logic to be changed should be in here: https://github.com/Homebrew/brew/blob/666463ca2be08261cf103e4870b267f5b461a7f3/Library/Homebrew/os/mac/xcode.rb#L183-L271. If that baffles you a bit I'll try to take a look at some point (about to jump on a flight).

I added 'MacOS::CLT.version_tag_instructions' underneath 'MacOS::CLT.update_instructions' and then called it in 'Homebrew::Diagnostic::Checks.check_clt_minimum_version' so that the instructions were only displayed once rather than after 'Homebrew::Diagnostic::Checks.check_clt_up_to_date' as well (which is needed for later macOS versions that just need their CLT updated).

@doughazell Can I see you brew doctor and brew config output? Thanks!

It's not really an update more of a hack to keep on old system running that still works.  I submitted the Pull-Request since there must be other people in my situation who want to keep using Homebrew (that is brilliant BTW...)
$ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

If you want to keep using Xcode on your old, unsupported OS/HW sys for a bit longer (with a bit of alchemy), then:
cd brew --repository; git reset --hard 1.0.9
export HOMEBREW_NO_AUTO_UPDATE=1
Warning: A newer Command Line Tools release is available.
Warning: The system curl on 10.8 and below is often incapable of supporting
modern secure connections & will fail on fetching formulae.
We recommend you:
brew install curl
Warning: Unbrewed .pc files were found in /usr/local/lib/pkgconfig.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.
Unexpected .pc files:
/usr/local/lib/pkgconfig/fuse-ext2.pc
Warning: You are using macOS 10.6.
We (and Apple) do not provide support for this old version.
You may encounter build failures or other breakages.
Please create pull-requests instead of filing issues.
Warning: You have uncommitted modifications to Homebrew
If this is a surprise to you, then you should stash these modifications.
Stashing returns Homebrew to a pristine state but can be undone
should you later need to do so for some reason.
cd /usr/local/Homebrew/Library && git stash && git clean -d -f
Warning: Broken symlinks were found. Remove them with brew prune:
/usr/local/share/man/man1/brew-cask.1.cpy
/usr/local/share/man/man1/brew.1.cpy
$ brew config
HOMEBREW_VERSION: 1.1.2-76-g163bbb675
ORIGIN: https://github.com/Homebrew/brew.git
HEAD: 163bbb675a9553fec1cfaa16985dab9b0556af74
Last commit: 73 minutes ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: a4d50c465d7398710f04022fafea080c7da83347
Core tap last commit: 3 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_REPOSITORY: /usr/local/Homebrew
HOMEBREW_CELLAR: /usr/local/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://homebrew.bintray.com
CPU: dual-core 64-bit penryn
Homebrew Ruby: 2.0.0-p648 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.0.0-p648/bin/ruby
GCC-4.0: build 5494
GCC-4.2: build 5666
Clang: 1.7 build 77
Git: 2.11.0 => /usr/local/bin/git
Perl: /usr/bin/perl
Python: /usr/local/bin/python => /usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/bin/python2.7
Ruby: /Users/doug/.rvm/rubies/ruby-2.2.2/bin/ruby
Java: 1.6.0_65-b14-462
macOS: 10.6.8-x86_64
Xcode: 3.2.6
CLT: N/A
X11: 2.3.6 => /usr/X11

Thanks. I'm going to close this out but have opened https://github.com/Homebrew/brew/pull/1652 which should addressed your issues when merged.

Cool thanks. Worth doing, result...:)


Error 456

new error, you have to log into the webapp and it makes you calculate a maths question.
after i calculated it and went back into autobuyer. the error appeared again.
calculated it again, stopped using autobuyer. Got into a bidding war in the webapp and it logged me out and made me do another calculation! lost the auction too

screenshot

probably because they suspect you of being a bot

no its happening to normal users on reddit too, even futbin can't update fast enough due to this

Yeah i've read on reddit that anyone using a rpm of 3 or higher will get this because ea is using this to get rid off people using bots as it takes way too long to quit the bot, log onto the web app do the "maths problem" and use the bot again. This is very inefficient so most botters will stop as they will be put off by this. The only way to not get the captcha is to use a very low rpm like 1-3rpm.

nvm haha its fixed it was probably a bug from ea that was fixed lol

really? nice! so what RPM should we use?
thanks for keeping me updated

i always use 6 rpm usually just because i use the bot to bid on items rather than BIN just never use an rpm higher than 10 in my opinion.


Separate collection list and toylist

Make two separate visible lists detailing toys you own and toys you want to own. Tapping an item in the wishlist could move it to the collection list.

I am going to claim this issue!

@404thecomputer don't forget to close the issue when the pull request is merged!


Create "service" dependency injector for test cases

User story
As a developer, I want a way to inject the service dependencies into my unit tests so I can instantiate the controller classes.
Acceptance criteria

A helper class which creates and injects the service dependencies into unit tests

Needed for #74


Catch nulls in updatePosition and adjustForCaptureGestures

On some calls to updatePosition this.drawer and this.main were null and
to adjustForCaptureGestures this.mainOverlay and this.drawerOverlay
were null.
This occurred on Android API Level 22 but not on 23 or earlier.
Signed-off-by: Ian Hilt<EMAIL_ADDRESS>

Should have looked at that closer :) thanks. I will test tonight, but please report back if there are any issues or errors.


Add 16bit flags

Please check if the PR fulfills these requirements

[X] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature change

What is the current behavior? (You can also link to an open issue here)
Flag events only support up to 8 different flags.

What is the new behavior (if this is a feature change)?
Flag events now support up to 16 flags.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No

Other information:
Thanks to pautomas and toxa for their help!

Works great thanks @Rebusmind


Table of contents doesn't print accents in uppercase letters at the title

Renderer
v3
Browser
other
Operating System
Windows
What happened?
When creating the table of contets and trying to change its title to "Índice" (Index in spanish), the first "Í" with accent was printed without the accent.
Code
No response

It would appear that the font MrsEavesRemake needs a glyph tweak.

I wouldn't call a glyph tweak to adding all latin suplement uppercase characters, also have in mind, #116 is our oldest issue.

Hey! I was trying to fix this as my first issue, I think I fixed it but still not working... Using a online visualizer of the font @ericscheid said I saw the problem:

So I decided to modify the font so it works as it should and this is the result:

So I replaced the font file here: /homebrewery/themes/fonts/e5/Mr Eaves Small Caps.woff2 and here /homebrewery/themes/fonts/e5 legacy/Mr Eaves Small Caps.woff2, rebuild and run. Not workin :(
Any idea? Building is enough to see the changes, right? Did I modified the correct file?

Hey! I was trying to fix this as my first issue, I think I fixed it but still not working... Using a online visualizer of the font @ericscheid said I saw the problem:

So I decided to modify the font so it works as it should and this is the result:

So I replaced the font file here: /homebrewery/themes/fonts/e5/Mr Eaves Small Caps.woff2 and here /homebrewery/themes/fonts/e5 legacy/Mr Eaves Small Caps.woff2, rebuild and run. Not workin :(
Any idea? Building is enough to see the changes, right? Did I modified the correct file?

/homebrewery/themes/fonts/e5/Mr Eaves Small Caps.woff2 or
homebrewery/themes/fonts/5e/Mr Eaves Small Caps.woff2

Do we want to modify the Legacy font? we can do it, but @MiniX16 for the future, you don't need to update anything in the legacy version. I believe there was consensus on leaving that as it, with bugs and all. I do think a font change this small won't break anything.
This looks good.


Fix older lein builds

While trying to address #132 I noticed that leiningen 2.9.6- versions weren't building because of the author's recent public key change. Fixing that led me to implement a couple things I've been meaning to improve anyway. 2.9.7 is out, but I want to get this and #126 in first before releasing that.

@Quantisan Do you have a few minutes to review this and #126? Please and thank you!

@cap10morgan I'm on medical leave these days. I'm afraid that you'll have to proceed without me for the next couple weeks. Thank you for all the work!

@Quantisan OK, will do! Hope everything is OK!


How to get the path of raw file?

In zotero, when you right click an item, you can select 'show file' to pop the directory of raw file up. How may I access the path in the python code?

I put raw files under directory of dropbox and link them to the Zotero storage path

Local file paths aren't supported by the remote API.


Add configurable parser to verticle configuration files.

@ddimensia when you get a chance I would love your input on this. I could not figure out an easy way to replace the root configuration parser, but I'm willing to live with that (at least for now). Open to suggestions and feedback. Thanks!


Cuboids

Hello there,
This past week I have been trying to tackle a certain problem : the computation of polycubes
that fits inside a 3x3x3 cuboid. My work brought me to improve the Python code to make the computation faster
but it also gave me an idea for the computation of polycubes of size n.
I consider as valid polycubes inside a cuboid of size (k1,k2,k3),
those which span across the full extent of the cuboid in the 3 directions.
I also put the constraint of k1>=k2>=k3 which does not change the final result because the problem is rotation invariant.
The direct consequence is that the number of cubes (let me call that the rank)
n of a valid polycube is bounded like so:
k1+k2+k3-2 <= n <= k1k2k3. For the (3,3,3) cuboid, we have 7 <= n <= 27.
The generation of polycubes of each increasing rank inside a cuboid of a given shape is done starting from the rank 1 polycube
at all possible positions inside the cuboid. The expansion function is modified since the output shape is the same as the input shape.
Restricting the polycubes that fits inside the cuboid makes for a few optimizations in the computation:
First, the shape is fixed so it does not need to be added to the packing of the cube.
With Python, I have used the tobytes() function for an even faster packing but that does take 8 times as much space as needed.
Second, the number of rotations in a cuboid is limited to 8 when k1=/=k2=/=k3 which is the most occuring general situation.
In the case of the (3,3,3) cuboid, this is still 24 though.
Third, the shape being fixed, a rotation is an index permutation operation and is made much faster (at least with numpy).
As a result, the number of polycubes of each rank tends to increase exponentially from 1 till n=k1k2k3/2 and then decreases exponentially to 1.
The screening of invalid polycubes removes around 10% of the total after the expansion is over.
By the way, the result for cuboids with k3>=2 and k1k2k3<=28 are available on my online viewer.
https://asliceofcuriosity.fr/assets/polycube-viewer/index.html
I'll make a PR soon with these improvements.
As for my contribution to the computation of polycubes of rank n, here is my idea.
When you have all the polycubes in a cuboid of a certain shape (k1,k2,k3), you can compute part of the polycubes
of the cuboids of shape (k1+1,k2,k3), (k1,k2+1,k3), (k1,k2,k3+1).
Starting with the (1,1,1) cuboid, this makes for a graph of dependency where each cuboid
can be computed by the expansion in different directions of three previous cuboids
and can be expanded into three cuboids.
So if one wants to generate and enumerate all of the polycubes of rank n,
it needs to gather the polycubes of rank n from cuboids that satisfy the bounding equation
and all of the cuboids before them in the graph up to a certain rank and so on.
This dependency of cuboids and ranks is "easy" to compute and would split up the computation
into several smaller chunks which would reduce file sizes, RAM usage and maybe computation time too.
I think that grouping together polycubes with the same shape would allow for many optimizations in terms of storage and computation.
I have seen the idea mentioned in different ways across the posts from different people on different topics.
I am in the process of implementing this idea in Python.
The graph starts like this, with of all cuboids of the same depth in the graph on the same line and with the shape, the minimum and maximum rank for each cuboid:
(1, 1, 1) 1 1
(2, 1, 1) 2 2
(3, 1, 1) 3 3 / (2, 2, 1) 3 4
(3, 2, 1) 4 6 / (4, 1, 1) 4 4 / (2, 2, 2) 4 8
(5, 1, 1) 5 5 / (3, 3, 1) 5 9 / (3, 2, 2) 5 12 / (4, 2, 1) 5 8
(6, 1, 1) 6 6 / (5, 2, 1) 6 10 / (4, 3, 1) 6 12 / (3, 3, 2) 6 18 / (4, 2, 2) 6 16
(4, 3, 2) 7 24 / (5, 2, 2) 7 20 / (7, 1, 1) 7 7 / (4, 4, 1) 7 16 / (3, 3, 3) 7 27 / (6, 2, 1) 7 12 / (5, 3, 1) 7 15

Hello again,
I have not followed up with my idea in terms of code but I have computed the numbers for the first cuboids thanks to @snowmanam2 very fast code.
The partial results are available on this webpage.
I have observed that the evolution of the number of polycubes with each count in a given cuboid, in a log scale, follows a polynomial equation of degree 2. I have no idea why though.


How to use it with nodejs?

I've been reading a lot about deepstream lately and wanted to use this in our system. We have a python/django based system and found that there's no official client for python yet. The next option for us was nodejs and microservice architecture. I found a few things in the documentation but there was nothing related to emitting events, records or rpc from node. Can someone explain ( or even point me to a resource ) on how to do this?

hey @arvindnrbt, the way deepstream works is that all logic is carried out by the client. This is quite different from most other realtime solutions that require logic to be run within the actual server (such as featherJS, meteor and socketIO).
Heres a link to getting started: https://deepstreamhub.com/tutorials/getting-started/node/

Thanks @yasserf , that link was a good start for me. But what about valve and permissions? How do I set permissions, authentication with the node api? The existing documentation lacks information about these things for the open source version.

Permissions look like they are configured via permissionHandler in the node API. Permissions look like they can be set at start time in the config:
  type: config
  options:
    path: ./permissions.yml

At run time, you can modify the permissions with this: https://deepstreamhub.com/tutorials/guides/valve-realtime/
Full disclosure I have yet to actually start using Deepstream

The example in the link seems to be focusing on DSHub. I am also looking for info on how to use Valve and control dynamic permissions from API (in the node, programatically).
For example, the document says

Valve rules can be edited in the permissions section of the deepstreamHub dashboard.

It does not explain which exact API does the hub then uses to change the permissions in the backend/DB.
If we have to provision similar permissions, and change them at runtime, how to achieve it through API?
@yasserf  any help / pointers are greatly appreciated.

Yes, a simple tutorial on realtime permissions would be great.

Hello! Still a work in progress (finishing touches) but we have been really busy with getting a new version out. Link to documentation is here:
http://deepstream.io-v3.s3-website-us-east-1.amazonaws.com/
I know it has been a year since this was raised, things were put on hold for a short while but we are back in full steam again with a few new releases planned over the next few months.

https://deepstream.io/tutorials/core/permission/valve-dynamic/


Fix examples

Most of the issues were caused by outdated JSON response type (PoV < 4.0.0) or invalid JSON issues. There were also minor issues in the code.
All examples should work now.

Looks like we have CI issues.
I don't have access to take a look, but I guess things might be out of date.

I don't have access to take a look, but I guess things might be out of date.

I gave the entire Android team Developer access (I think this makes sense), and you should also have Admin access specifically to manage workflows.

There is an issue with connecting the project to Github or SSH keys
Build fails with error:
Warning: Permanently added 'github.com,<IP_ADDRESS>' (ECDSA) to the list of known hosts.
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

In settings:

Since I am not sure how Github integration with Bitrise is configured I think it is best for LEs to check it.

@antunflas Please check if ssh key is included into Infinum-fastlane account.

CI has been fixed.

@antunflas I will just merge this as no "real" changes are present and PR is open for some time


Node replacement and graph isomorphism

Add myia.anf_ir_utils.replace(node1, node2) to replace a node by another in a graph.
Add myia.debug.utils.isomorphic(graph1, graph2) to check whether two graphs are structurally equivalent. This is very handy to test replace and optimizations.

The PR also includes the following minor changes:

Add cls argument to is_constant, so that e.g. one can write is_constant(node, int).
Create myia.debug.utils to hold some debugging utilities that may be generally useful.
Move myia.debug.label.Index to myia.debug.utils.GraphIndex.
The tests for replace use GraphIndex and isomorphic, and these are likely to be useful for many other tests, so I added tests for both in tests/debug/test_utils.py.

I made isomorphic non-recursive. I'm not sure if I should move it to anf_ir_utils just yet, since it's only used in tests. It could also be moved if/when part of the code requires it.

I would prefer to move it to utils right now so that it is where someone would expect it when they need it.


remove all compressed log viewing code

Goes with #1380
/cc @tpetr

this is in stable now, merging it into the tailer branch


[QUESTION] Which specific LGPL is in use for this project?

Is the LGPL that has been in use since Nov 2011 updated? opensource.org/licenses only shows 2.0, 2.1 and 3.0 versions of the LGPL. Which text is in use here?

We switched to MIT


Migration: Support dependencies inside buildSrc/src/main/kotlin/*.gradle.kts

The plugin "de.fayard.refreshVersions" doesn't appear to work for buildSrc plugins inside a multi-module project, but it would be nice if it could.
Steps to produce scenario:

With Gradle 7.2, created a multi-module project with gradle init, choosing options 2-3-2-2 (application, Java, yes, Kotlin), and choosing "demo" as the project name and source package.

Add the following to ./settings.gradle.kts and ./buildSrc/settings.gradle.kts
pluginManagement {
    plugins {
        id("de.fayard.refreshVersions") version "0.20.0"
    }
}

plugins {
    id("de.fayard.refreshVersions")
}

Run the command gradle refreshVersionsMigrate

Open the file ./buildSrc/src/demo/kotlin/demo.java-common-conventions.gradle.kts. Notice that these versions have not been replaced. Replacing the versions manually with _ doesn't work either.
dependencies {
    constraints {
        implementation("org.apache.commons:commons-text:1.9")
    }
    testImplementation("org.junit.jupiter:junit-jupiter:5.7.2")
}

@LouisCAD I'm taking this up.

Keen to know if there are any updates here (or workarounds in the meantime) - thanks!


Toolchains: adjust environment variable generation on Windows

Avoid injecting a _EXE into environment variables for tool
replacements on Windows.  This allows us to use the same variable across
all the platforms.

@swift-ci please test


Stop appending cache

We use Jobs._run_local for remote inference - so I think the cache might be appended in another part of the code as well?


many warning when change the timestamp_resolution and --auto_timestamp_resolution set to true

Hi,
For the given sample video (tennis.mov) I have use the following command to generate events. Here i set the timestamp_resolution to 5ms instead and set the auto_timestamp_resolution to true. (in the example command given in the read me time_stamp_resolution was 3ms and auto_timestamp_resolution was set to False. ). In that case there are many warning as shown. So, for our own video how do we decide the exact timestamp resolution to generate events without these kind of warnings ?(because 3ms timestamp these warning did not come). Thank you.
Command:
python v2e.py -i inputs/tennis.mov --overwrite --slomo_model inputs/SuperSloMo39.ckpt --timestamp_resolution=.005 --auto_timestamp_resolution=True --dvs_exposure duration 0.005 --output_folder=output/tennis --overwrite --pos_thres=.15 --neg_thres=.15 --sigma_thres=0.03 --dvs_aedat2 tennis.aedat --output_width=346 --output_height=260 --stop_time=5 --cutoff_hz=15
Warnings:
WARNING:v2ecore.emulator:no events generated for frame, generating any sampled temporal noise for this frame with 1 iteration
WARNING:v2ecore.emulator_utils:IIR lowpass filter update has large maximum update eps=0.39 from delta_time/tau=0.00417/0.0106
WARNING:v2ecore.emulator_utils:IIR lowpass filter update has large maximum update eps=0.39 from delta_time/tau=0.00417/0.0106

It means your timestamp resolution is too coarse (your sample rate is
too low) to support the IIR lowpass filter cutoff you chose. You need to
either decrease the cutoff frequency or increase the upsampling
resolution (i.e., decrease the timestamp resolution).
This restriction is actually quite severe. For instance, if you want a
reasonable (but still quite slow) 100Hz photoreceptor cutoff frequency,
then you will need to use a timestamp resolution that allows accurate
IIR lowpass updates. So if
f3dB=100Hz
then the tau would be
tau=1/(f3dB2Pi)=1/(1002Pi)=1.6ms.
Therefore the sample interval should be at least 3 times higher for
accurate IIR lowpass filter updates, which use eps=dt/tau. We need to
make sure eps is much smaller than 1. Therefore, in this case, we could
use for eps=0.3 a value of dt=1.6ms/3=500us.
Unfortunately, unless we are missing something about how IIR filters
work, this expensive upsampling is not avoidable for accurate
photoreceptor filtering modeling.
PS if you don't care about modeling DVS photoreceptor lowpass filtering,
then just disable it. But then you may get unrealistically-fast DVS
pixel responses that do not model the motion blur that DVS pixels really
do have (despite the many false claims in the computer vision literature).
We recommend reading at least the pixel behavior section of the v2e
paper to better understand these real effects.
Hope this helps.
Tobi