Docs: Fungible Token for AI conversation

Updating the docs in examples/fungibletoken/README.md with a step by step instruction for the flow of starting an AI conversation, tracking token usage, and refunding unspent tokens.
See a brief demo video here: https://youtu.be/-R_zok--L_A?si=AkBl4IBjk6dbXDiw
The screenshot below is from the ongoing Pull Request in https://github.com/petersalomonsen/near-openai/pull/11

@race-of-sloths include

@petersalomonsen Thank you for your contribution! Your pull request is now a part of the Race of Sloths!

Current status: waiting for scoring
We're waiting for maintainer to score this pull request with @race-of-sloths score [0,1,2,3,5,8,13] command. Alternatively, autoscoring [1,2] will be applied for this pull request
What is the Race of Sloths

Race of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow
For contributors:

Tag @race-of-sloths inside your pull requests
Wait for the maintainer to review and score your pull request
Check out your position in the Leaderboard
Keep weekly and monthly streaks to reach higher positions
Boast your contributions with a dynamic picture of your Profile

For maintainers:

Score pull requests that participate in the Race of Sloths
Engage contributors with fair scoring and fast responses so they keep their streaks
Promote the Race to the point where the Race starts promoting you
Grow the community of your contributors

Feel free to check our website for additional details!

Bot commands

For contributors

Include a PR: @race-of-sloths include to enter the Race with your PR

For maintainers:

Invite contributor @race-of-sloths invite to invite the contributor to participate in a race or include it, if it's already a runner.
Assign points: @race-of-sloths score [1/2/3/5/8/13] to award points based on your assessment.
Reject this PR: @race-of-sloths exclude to send this PR back to the drawing board.
Exclude repo: @race-of-sloths pause to stop bot activity in this repo until @race-of-sloths unpause command is called

@race-of-sloths score 3


this is the basic html server with an empty homepage.

prickly-pufferfish folder

There are some Linter errors for unused files, i kept them as placeholders if we need them eventually.


Add option to redirect agent's logs to stdout/stderr to be captured in container's logs

What does this PR do?
This PR adds the option to send the logs of the agent running in a container to the logs of this container instead of in addition to the agent's log files. It allows for a more direct access of the logs.
This is done by setting the environment variable LOGS_STDOUT to 'yes' when running the docker container with -e LOGS_STDOUT=yes.
Motivation
See this issue
Additional Notes
Right now, I only managed to test on my local environment by building my own docker image and running the container with the environment variable set to 'yes'. Following the logs of the container, I saw as expected the logs from the forwarder inside.
I haven't tested if these logs were indeed appearing inside of the logs of kubernetes as requested in the issue but I checked there and it should be ok.
Tested and it works.

Looks better, but now i get these in docker logs and /var/log/datadog/supervisord.log:
2017-02-20 14:50:42,174 CRIT uncaptured python exception, closing channel <POutputDispatcher at<PHONE_NUMBER>81544 for <Subprocess at<PHONE_NUMBER>00712 with name forwarder in state RUNNING> (stderr)> (<type 'exceptions.IOError'>:[Errno 29] Illegal seek [/opt/datadog-agent/embedded/lib/python2.7/site-packages/supervisor/supervisord.py|runforever|227] [/opt/datadog-agent/embedded/lib/python2.7/site-packages/supervisor/dispatchers.py|handle_read_event|232] [/opt/datadog-agent/embedded/lib/python2.7/site-packages/supervisor/dispatchers.py|record_output|166] [/opt/datadog-agent/embedded/lib/python2.7/site-packages/supervisor/dispatchers.py|_log|142] [/opt/datadog-agent/embedded/lib/python2.7/site-packages/supervisor/loggers.py|info|275] [/opt/datadog-agent/embedded/lib/python2.7/site-packages/supervisor/loggers.py|log|293] [/opt/datadog-agent/embedded/lib/python2.7/site-packages/supervisor/loggers.py|emit|186] [/opt/datadog-agent/embedded/lib/python2.7/site-packages/supervisor/loggers.py|doRollover|211])

Fixed it.


Select directive error and valid states

Are there plans for an errorvans valid state for the Select directive?

We plan to have disabled, error and valid states, help and error messages for all inputs.
We'll provide it in a close future!


[Event Request] OnBeforeItemAttributeValueDelete in table Item Attribute Value Mapping

Hello, please consider adding the following event to table 7505 "Item Attribute Value Mapping":
    [IntegrationEvent(false, false)]
    local procedure OnBeforeItemAttributeValueDelete(var Rec: Record "Item Attribute Value Mapping"; var IsHandled: Boolean)
    begin
    end;

and raise it in the OnDelete trigger, like so:
    trigger OnDelete()
    var
        [...]
        IsHandled: Boolean;
    begin
        [...]

        ItemAttributeValueMapping := Rec;
        if ItemAttributeValueMapping.Find then begin
            OnBeforeItemAttributeValueDelete(Rec, IsHandled);
            if not IsHandled then
                ItemAttributeValue.Delete();
        end;
    end;

Thank you.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Log / info methods don't respect metadata overrides

Please tell us about your environment:

winston version?

[ ] winston@2
[x] winston@3

node -v outputs: v10.14.2
Operating System? (Windows, macOS, or Linux) Windows and Linux
Language? all

What is the problem?

I also noticed that this behavior is broken even on the root logger. If you use defaultMeta, it will overwrite data in your info object.
What do you expect to happen instead?
I expect the child behave consistently when calling .log and .info
Other information
There seem to be a few problems here.
First Problem:
When you create a logger with default metadata, it is stored as defaultMeta. When you call log(info), the default metaData is applied onto info, effectively overwriting whatever you passed.
logger.log({ level: 'debug', message: 'root log', component: 'ROOT2' });
// => [debug][ROOT]: root log

_addDefaultMeta(msg) {
    if (this.defaultMeta) {
      Object.assign(msg, this.defaultMeta); // msg gets overwritten with defaultMeta
    }
  }
}

this function should probably assign in reverse order to a blank object and return the value..
Second Problem:
When calling .info(), the same happens:
  logger.info({ message: 'root info', component: 'ROOT2' });
 // => [info][ROOT]: root info

Third Problem:
When creating a child logger which overrides component, if you call .info on the child, the metadata from the root is used instead. This is because the child() method only overrides  write()
Please let me know if you need a full reproduction. I can try to provide one.

I have created a PR that I believe will address your issue, as well as mine @kbirger. Please test it and let me know if it does not function as expected.

I'm currently out of the country and my laptop has bit the dust. I'll have a look early next week, after I settle back into real life 🤣. Had a (really) brief look and the concept looks correct to me though

@kbirger any updates? No worries if you haven't had a chance as of yet.

@Maverick1872 thanks for the reminder. Just had a look and commented on your PR. I recommended two small changes, but other than that looks good, thanks!

This bug is affecting me as well. The problem is, that I cannot overwrite defaultMeta.

Hey all, finally working through this again. Reviewed the comments on the PR and have decided I'll likely do this in phases. Updated branches my master branch, going back checking test coverage for logger.js first. Will be updating coverage where applicable before picking up the resolution for this issue to ensure I don't break anything that @DABH  was worried about in regards to breaking functionality.
@mpetrunic I will be referencing your PR as well.

@mpetrunic @kbirger Feel free to try out my PR branch if y'all want. Think I've addressed everything although there is a couple things I want to talk through with the maintainer to ensure backwards compatibility. Something to be aware of is that the .log(level, message) function signature does not apply any metadata as of right now so don't expect to see any logger configured metadata be appended to those log statements for the time being.

@Maverick1872 Missed this comment, PR works for me

@mpetrunic Awesome! Thanks so much for checking that! I'm basically just waiting to hear feedback about it from @DABH. In the interim i should likely add more tests to ensure the SPLAT functionality is still functioning as expected. I'm just not super familiar with it myself.

Is there any plan to merge a fix for this into winston? Ran into this bug myself, and would be great if
It seems that this is the latest PR that passes added tests and resolves this issue is it OK to merge?
https://github.com/winstonjs/winston/issues?q=child+override+is%3Aopen

@kkacquah Per the discussion on my PR here I've moved on personally. It appears as if the maintainer has abandoned the project. I toyed with the idea of forking and taking it over myself but I can't reasonably maintain the project as a whole on my own.

Some new maintainers have been brought on board for the time being — @wbt and @fearphage can hopefully take a look soon.  This is one of the hairier issues, though, so might take a bit for them to get up to speed.

@DABH Hey!!! Nice to see you around again. I'd be glad to help out in some capacity; along with re-opening my PR for the described issue if desired.

Hey hey :) The more help the merrier — I think the problem is when this gets down to one maintainer it’s not really sustainable (especially when that maintainer gets buried by work/life for a year - 2020 was interesting 😬).  If you are interested in being a maintainer please email me, my email is $(npm info yadeep maintainers.email).
For this issue, if you’d like to open your PR and maybe re-state/summarize the issue and the fix, @wbt @fearphage and I can take a look?  They have been doing more Node and Winston stuff than I have recently so they should be able to help offer useful comments.  I do think that this is one of maybe 1-4 major issues with Winston currently, and they’ve already solved 1 or 2 of those issues, so we are well on our way to getting Winston 3.x to where it needs to be (there are always minor improvements but I’m not so concerned about those…)

Closing as this issue is being consolidated into #2029

Hi is the above issue addressed cause. I am still facing the same issue.
_addDefaultMeta(msg) {     if (this.defaultMeta) {       Object.assign(msg, this.defaultMeta); // msg gets overwritten with defaultMeta     }   } }
Does implementing this approach is a good solution?.
     _addDefaultMeta(msg) {         if (this.defaultMeta) {           const msgClone = JSON.parse((0, stringify_1.default)(msg));           Object.assign(msg,this.defaultMeta,msgClone);         }       }


⚠ ClamAV has degraded performance

In b41fd30, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 211 ms

Resolved: ClamAV performance has improved in 39171e8 after 8 minutes.


POST URLs are incorrect

[Enter feedback here]
The list of POST URLS near the top of the page are not correct.  You cannot create an upload session without specifying the file name.  So, for example:
POST /sites/{siteId}/drive/items/{itemId}/createUploadSession
should be changed to:
POST /sites/{siteId}/drive/items/{itemId}:/{fileName}:/createUploadSession
along with all of the others.

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

ID: c42d74d1-8c6f-5282-9fba-30fdd6449362
Version Independent ID: 88bd8f18-730d-6970-8a02-641346661404
Content: driveItem: createUploadSession - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/driveitem-createuploadsession.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @JeremyKelley
Microsoft Alias: MSGraphDocsVteam

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


Resolving reserved identities in cilium bpf policy get doesn't work

If you don't specify -n to cilium bpf policy get, ie to avoid resolving identities, then it prints nothing:
$ sudo cilium bpf policy get 16996
LABELS (source:key[=value])   PORT/PROTO   ACTION   BYTES   PACKETS
$ sudo cilium bpf policy get 16996  -n
IDENTITY   PORT/PROTO   ACTION    BYTES   PACKETS
1          ANY          allowed   0       0

Possible regression since the kvstore rework.

Same issue as #2857  .

Closing as duplicate.


Add Adam baseline

Adam Baseline

Implementation: Optax's AdamW
Tunable hyperparameters: learning_rate, b1, b2, weight_decay
Fixed hyperparameters: epsilon=1e8, eps_root=0.0

I assume the LR schedule is the same as for the target-setting runs?
Should label_smoothing, grad_clip, and warmup_steps be tunable hyperparameters? And what about the same hyperparamters for the other baselines (for the ones with polynomial LR schedule also end_factor and decay_steps_factor)?

Addressed by #306.


Subclass of FlutterViewController, overriding canBecomeFirstResponder causes an error.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Objective-c

create subclass of FlutterViewController

MyViewController.h

@interface MyViewController : FlutterViewController

@end

MyViewController.m

@interface MyViewController ()

@end

@implementation MyViewController

- (void)viewDidLoad {
    [super viewDidLoad];
    // Do any additional setup after loading the view.
}
-(BOOL)canBecomeFirstResponder{
    return YES;
}

Replace the FlutterViewController in Main.storyboard with MyViewController
lanuch app, then error occurred

2023-10-09 14:32:22.014393+0800 Runner[11115:412577] void * _Nullable NSMapGet(NSMapTable * _Nonnull, const void * _Nullable): map table argument is NULL
2023-10-09 14:32:22.114439+0800 Runner[11115:412922] flutter: The Dart VM service is listening on http://<IP_ADDRESS>:50688/ZNUf_kKSZZs=/

Expected results
no error occurred.
Actual results
error occurred
2023-10-09 14:32:22.014393+0800 Runner[11115:412577] void * _Nullable NSMapGet(NSMapTable * _Nonnull, const void * _Nullable): map table argument is NULL
2023-10-09 14:32:22.114439+0800 Runner[11115:412922] flutter: The Dart VM service is listening on http://<IP_ADDRESS>:50688/ZNUf_kKSZZs=/

Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.6, on macOS 13.4.1 22F770820d darwin-x64, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 14.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2020.2.3)
[✓] IntelliJ IDEA Community Edition (version 2023.2.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2018.3.5)
[✓] VS Code (version 1.75.0)
[!] Proxy Configuration
    ! NO_PROXY is not set
[✓] Connected device (3 available)
[✓] Network resources

! Doctor found issues in 1 category.

Hi @GordonHuangYong. Thanks for filing this. I see a similar log, can you check out https://developer.apple.com/forums/thread/718179 to see if the possible solution provides more information. If not, can you share a sample repo that we could run to reproduce this?


Replace "eg" with "like" for report a problem

https://insidegovuk.blog.gov.uk/2016/07/20/changes-to-the-style-guide-no-more-eg-and-ie-etc/

Terms like eg, ie and etc, while common, make reading difficult for some.

Thanks @alexmuller 👍  🎉


[Suggestion] Set bot presence/game as spoiler syntax

In the config allow for the end user to supply a "game" for discord to play. This will allow it to show underneath the Name in the member's list of the server.
For Example:
Spoiler Bot
Playing [TOPIC]:spoiler:[TEXT]

Sounds like a great idea, will implement it sometime soon.
For the time being you could use an external bot instance and specify the game manually.


[Nodejs] group chat

tell me how to get messages  in skype from all users in group?
regardless, they are turning to the bot or not

I think the bot can only get messages IF it is directly addressed to it. It cannot listen to the group's conversation in any way as far as I'm concerned.

@divinemanwah
about the treatment directly to the bot, I know
but the question was not about that

you can't, by policy of the skype team


How to create file from base64 data returned from response and upload to server that supports multipart/form only

"react-native": "0.61.5",
"react-native-image-picker": "2.3.3",
Both Platform
const data = new FormData();
data.append('file', {
uri: response.data,
type: response.type,
name: 'photo.jpeg',
});
and upload the data to server but i get error Request part file not found,Please help to convert response to file format,atob and btoa is not wroking

"react-native": "0.61.5",
"react-native-image-picker": "2.3.3",
Both Platform
const data = new FormData();
data.append('file', {
uri: response.data,
type: response.type,
name: 'photo.jpeg',
});
and upload the data to server but i get error Request part file not found,Please help to convert response to file format,atob and btoa is not wroking

Hi, did you find a solution to this?

change uri: response.data to response.uri

change uri: response.data to response.uri

worked thanks 😁

change uri: response.data to response.uri

worked thanks 😁


YouTube videos do not play in the Brave browser

There appears to be some kind of issue loading YouTube videos with MediaElement on the Brave browser (based on Chrome).
https://brave.com/
I'm not sure if this is a Brave issue or something to do with MediaElement.
Steps to reproduce

Download + open the Brave browser (I'm using MacOS Version 1.1.20 Chromium: 79.0.3945.74 (Official Build) (64-bit))
Go to the MediaElement homepage (https://www.mediaelementjs.com/)
Select YouTube as the video source
Click play

Expected behaviour
The video should play
Actual behaviour
The video does not play and the following error can be seen in the developer console
Access to XMLHttpRequest at 'https://googleads.g.doubleclick.net/pagead/id' from origin 'https://www.youtube.com' has been blocked by CORS policy: The value of the 'Access-Control-Allow-Origin' header in the response must not be the wildcard '*' when the request's credentials mode is 'include'. The credentials mode of requests initiated by the XMLHttpRequest is controlled by the withCredentials attribute.

Other notes
Normal YouTube embedding appears to work fine in Brave

Please allow autoplay from MediaElements Website and the function works fine.  Due method how the player handles external render like Youtube. Brave thinks is an auto play and this is not allowed. When Enableling auto play it works fine
The error is most likely caused by privacy features of Brave browser. Sadly enough we can't control this.


[Portuguese] Localize 'Imutable Infrastructure'

Localize https://glossary.cncf.io/immutable-infrastructure/

Hi, I would like to contribute, could you please assign this issue to me. Thanks

Sure! done :)

Duplicated issue.


Introduce torus template command

The torus template command allows a user to inject secrets stored in
torus as they derive a configuration file from a template.
The current templating engine is the raymond
handlebars engine for Go
which supports v3 of the handlebars spec.
Example Usage
Simple variable replacement example with a json file:
$ cat test.json.template
{
    "message": "{{ message }}",
    "port": {{ port }}
}
$ ./torus template test.json.template
{
    "message": "this value was set using ag",
    "port": 34234
}

Outputting to a file instead of stdout using the optional output parameter:
$ ./torus template test.json.template config.json
$ cat config.json
{
    "message": "this value was set using ag",
    "port": 34234
}

Closes manifoldco/torus-cli#51

This works great for the most basic use-case, however, to iterate over the exposed configuration items and inject them into a json file isn't exactly ideal:
$ cat test.json.template
{
    "message": "{{ message }}",
    "port": {{ port }},
    "config": {
    {{#each _config}}
        "{{ name }}": "{{ value }}"{{#unless @last}},{{/unless}}
    {{/each}}
    }
}
$ ./torus template test.json.template
{
    "message": "this value was set using ag",
    "port": 4432,
    "config": {
        "message": "this value was set using ag",
        "port": "4432",
        "test_db": "testdburl"
    }
}

A few problems:

Your template has to produce valid json -- so you'll have to do that yourself
We don't offer a good helper to do the type outputting properly
We don't offer a helper for converting an entire slice or map to JSON inside the template itself

These will be problems despite the engine we use for templating -- as the engine will never be aware enough to offer nice solutions for deriving JSON, YAML, or . To make this a nice experience, we'd have to offer helpers.
I'm not super happy with the more advanced use cases, however, it seems to me that as you got more advanced you'd probably be using something like ansible or chef to derive your configuration files. At which point, you wouldn't want to shell out, you'd want a native plugin.
Thoughts?

Thoughts?

For the use cases this feature supports (I have a config file for a service, I want to put secrets inside it) the feature as coded works well. The challenges I noted above, only comes into effect if you are storing complex data (which we don't support today) or you want to combine multiple service configs into one config file (which this feature does not support).
Therefore, for the purposes of enabling this workflow, I think we can go ahead and merge this as implemented. Then we can begin discussing how we'd want to enable those other use cases (which I'd advocate offering language libraries and plugins to things like ansible, chef or puppet).

Values that are found in the template but not actually set in the environment come out as "".
Should they not be null?

I guess the problem there is that they're likely wrapped in quotes in the template...

Yeah -- ideally we'd error but I'm not sure if the template engine we're using provides a hook for catching non-provided template variables which would be ideal.

Going to look for a better package before we move ahead with this change.


Fault injection fix

whenever routeToLocation is being called, it will clear the locationEndpointToRoute. but FaultInjection is using locationEndpointToRoute for condition checking when region is configured. So in this PR, the locationEndpointToRoute will be recorded in faultInjectionContext as well

API change check
API changes are not detected in this pull request.


Engague the OhShitStore based on the f3Base, not the head

The certstore stores the base power table, not "future" power tables. So we need to keep EC power tables around until we've passed the instance lookback.
Also, fix an issue in the certstore where we were storing power tables one epoch after we should have.

(consider merging by rebase)


router: Workaround currently broken test in Go 1.4.3

Go 1.4.3 reduced some new request smuggling protection that seems to be running amok with non-GET/HEAD requests. For now we will disable checking for connection liveliness here and track it in #2358 to re-enable it when migrating to Go 1.5 and also notify/work with upstream to get it resolved on the 1.4.X branch.

LGTM


Left pane in Global Options window not visible?

Hi,
Thanks for the great theme!
There is one main issue that I encounter once applying it, though: when you open the settings menu (that allows, among other things, to apply different themes), the text on the left pane that lists the configuration options is barely visible; please see attached snapshot.
Is there a way to address this?
Snapshot:

Thanks!

I have the same issue, I'm running the preview RStudio Version 1.3.911

Hi, I actually fixed it, I think.
Looking at the source, I realized that you can specify and override the properties any class in RStudio source.
This realization by itself was priceless for me, for different reasons: please see my post https://community.rstudio.com/t/capturing-configuration-applied-via-rstudio-devtool-in-themes-or-in-a-project-file/55433
Thank you, Patrick (is that your name?) for sharing this source which made me realize that.  It helped me fixed the issue mentioned in the post above and saves me lots of time.
As for fixing the bug in this profile: I make the following simple changes to the rstheme file to set the background to black:
*** Mojave Dark (Static).orig.rstheme	2020-04-20 13:27:50.000000000 -0400
--- Mojave Dark (Static).rstheme	2020-03-05 13:51:16.000000000 -0500
***************
*** 162,179 ****
--- 162,184 ----
  /*button labels in Git commit page*/
  .GD15MCFCMU {
    color: white;
  }

+ .GGBOEFPDGK {
+   background: black;
+ }
+
  .dialogMiddleCenterInner button.gwt-Button.GD15MCFCH1 {
    filter: invert(90%)
  }

  .rstudio-themes-flat .gwt-TabLayoutPanelTabs table.rstheme_tabLayoutCenter, .rstudio-themes-flat .rstheme_minimizedWindowObject table.rstheme_tabLayoutCenter {
    border-radius: 0px;
  }

How do you figure out the class name?
Use the "inspect element" when right clicking the place in the window that is closest to what you want to change; that would open the DevTool window, and you can interactively point to class names, and it will show you on the RStudio window which elements of the window they corresponds to.  It sometimes takes a while to figure out exactly the subclass you want to change, as there are lots of subcomponent and you can configure the style of  almost anything, but you can play with that, change values, in DevTool, etc, and you immediately see the results.
Once you recognize what you want to change, add it to a one of the themes.
Hope that helps.

Can confirm this also occurs on the stable 1.3 release (RStudio 1.3.959).

Was just coming to say the same.
This is Mojave Dark in the stable release:

Doesn't happen with the in-built dark themes, but that's because they don't change the skin.

Jus pointing out my follow up post above, showing how to fix it by adding 3 lines to the scheme file. It's easy and the problem goes away.  See post/comment above https://github.com/patrickshox/Mojave-Dark-RStudio-Theme/issues/17#issuecomment-616749543

@PrivacyMatter
Related to the issue, currently the background of the pane "New R Markdown" is white and this makes the pane severely unreadable.  What and in which line should I add .GGBOEFPDGK {background: black;}-like command on the rstheme file?

Yep, that's how I found what needed to be changed also; Inspect element and waling the class tree there.  Note that whatever you are changing there become visible on the fly so you can experiment with it, etc.
I don't think that the location at the rstheme file matters much as long as it is not succeeded by another statement that modifies another class that you inherit from.  The whole system is hierarchical, so make sure you keep the changes near the leafs
Cheers

This issue has been resolved.

This issue has been resolved.

Thanks! This issue has been resolved, but the update panel is still have the same problem.

That's been fixed now. Thanks for bringing this to my attention.


chirp: 20180512 -> 20180519

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/chirp-daily/versions.
These checks were done:

built on NixOS
Warning: no invocation of /nix/store/s2g23ds5fsnl0bg7dyn9j5y3drvxs3ar-chirp-daily-20180519/bin/.chirpw-wrapped had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/s2g23ds5fsnl0bg7dyn9j5y3drvxs3ar-chirp-daily-20180519/bin/chirpw had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 20180519 with grep in /nix/store/s2g23ds5fsnl0bg7dyn9j5y3drvxs3ar-chirp-daily-20180519
directory tree listing: https://gist.github.com/a05ef55580a6032a5b40aabb92e6dfd7
du listing: https://gist.github.com/13517a738616514609c44b95f30fc5a1

meta.description for chirp is: '"A free, open-source tool for programming your amateur radio"'.
cc @the-kenny for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: chirp
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: chirp
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/s2g23ds5fsnl0bg7dyn9j5y3drvxs3ar-chirp-daily-20180519
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/s2g23ds5fsnl0bg7dyn9j5y3drvxs3ar-chirp-daily-20180519/bin
patching script interpreter paths in /nix/store/s2g23ds5fsnl0bg7dyn9j5y3drvxs3ar-chirp-daily-20180519
/nix/store/s2g23ds5fsnl0bg7dyn9j5y3drvxs3ar-chirp-daily-20180519/share/chirp/chirpw: interpreter directive changed from "/usr/bin/env python" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python"
/nix/store/s2g23ds5fsnl0bg7dyn9j5y3drvxs3ar-chirp-daily-20180519/share/chirp/rpttool: interpreter directive changed from "/usr/bin/env python" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python"
checking for references to /build in /nix/store/s2g23ds5fsnl0bg7dyn9j5y3drvxs3ar-chirp-daily-20180519...
/nix/store/s2g23ds5fsnl0bg7dyn9j5y3drvxs3ar-chirp-daily-20180519

Success on aarch64-linux (full log)
Attempted: chirp
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/q14nwvlmk6py82nxrc0bsyrmzjim46ia-chirp-daily-20180519
strip is /nix/store/jk6j4lh9v5mvjdbdc35sj0zffhhf6s56-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/q14nwvlmk6py82nxrc0bsyrmzjim46ia-chirp-daily-20180519/bin
patching script interpreter paths in /nix/store/q14nwvlmk6py82nxrc0bsyrmzjim46ia-chirp-daily-20180519
/nix/store/q14nwvlmk6py82nxrc0bsyrmzjim46ia-chirp-daily-20180519/share/chirp/chirpw: interpreter directive changed from "/usr/bin/env python" to "/nix/store/vhz9s1dzzbndz032dysnixj8mxyfl30s-python-2.7.14/bin/python"
/nix/store/q14nwvlmk6py82nxrc0bsyrmzjim46ia-chirp-daily-20180519/share/chirp/rpttool: interpreter directive changed from "/usr/bin/env python" to "/nix/store/vhz9s1dzzbndz032dysnixj8mxyfl30s-python-2.7.14/bin/python"
checking for references to /build in /nix/store/q14nwvlmk6py82nxrc0bsyrmzjim46ia-chirp-daily-20180519...
/nix/store/q14nwvlmk6py82nxrc0bsyrmzjim46ia-chirp-daily-20180519


ChainingPrincipalResolver changes to allow overriding the Principal ID

This pull request is the result of my June 11, 2018 posting to the cas community Google group.
The main impetus behind this change is to make the cas.personDirectory.principalAttribute in cas.properties work properly.  While principal resolvers can currently return an attribute to be used as the Principal ID, some code in ChainingPrincipalResolver is preventing this from working correctly as the value returned by all principal resolvers have to be the same.  Allowing principal resolvers to properly override the Principal ID via cas.personDirectory.principalAttribute is tremendously useful in cases such as delegated authentication: when a user logs in via Google, the Google-returned Principal ID in the authentication phase can be overridden by the local Principal ID (if the Google account has previously been linked to a local account) via an attribute during the Principal resolution phase.
Brief description of changes applied

Modified ChainingPrincipalResolver so that each principal resolver is allowed to resolve into a Principal with a different ID.  The ID of the final Principal is taken from the last resolved Principal.
Modified CasCoreAuthenticationPrincipalConfiguration so that EchoingPrincipalResolver, which is usually the Principal resolved during authentication, be the first instead of last in the chain, so that other resolvers may override the resolved Principal ID.

Any documentation on how to configure, test
A unit test verifyChainingResolverOverwritePrincipal() is added to PersonDirectoryPrincipalResolverTests to test Principal ID overriding.
To test overriding the Principal ID:

Configure any number of principal resolvers
Specify cas.personDirectory.principalAttribute in cas.properties
Ensure that attribute resolution in the configured principal resolvers return different values for the specified principalAttribute
the returned value from the last principal resolver in the chain should become the final Principal ID

My original post has a stub attribute repository example.
Any possible limitations, side effects, etc
There may be existing code that rely on the first resolved principal ID.  This is only documented in code inside ChainingPrincipalResolver, and not anywhere in the CAS documentation.  Also because of the unique Principal ID restrictions, it was not possible for Principal resolvers to produce multiple Principal IDs.  This should make the changes backward compatible.
Reference any other pull requests that might be related
None

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

I have resolved the conflict.  Thank you for your consideration!

Thanks very much. Continue to fix issues by monitoring the Travis CI build status for this pull request.

I will need some help to fix the latest Travis CI error.  Thanks!

> Task :core:cas-server-core-authentication:checkstyleTest FAILED
[ant:checkstyle] [ERROR] /home/travis/build/apereo/cas/core/cas-server-core-authentication/src/test/java/org/apereo/cas/authentication/principal/PersonDirectoryPrincipalResolverTests.java:106: Line is longer than 190 characters (found 218). [LineLength]

Travis CI seems to be hanging at the Docs stage, even though there had been no errors.


core dump error on import

Hi,
I'm trying to import a bcf file that was generated by first converting a GATK vcf to bcf with bcftools. I'm getting the following error:
Program:   tomahawk-beta-0.7.1 (Tools for computing, querying and storing LD data)
Libraries: tomahawk-0.7.0; ZSTD-1.4.0; htslib 1.9
Contact: Marcus D. R. Klarqvist<EMAIL_ADDRESS>Documentation: https://github.com/mklarqvist/tomahawk
License: MIT
----------
[2019-05-20 16:53:15,426][LOG] Calling import...
[2019-05-20 16:53:15,426][LOG][READER] Opening snp.bcf...
[2019-05-20 16:53:15,433][LOG][VCF] Constructing lookup table for 608 contigs...
[2019-05-20 16:53:15,434][LOG][VCF] Samples: 56...
[2019-05-20 16:53:15,434][LOG][WRITER] Opening snp.twk...
00000000
00001010
tomahawk: lib/core.cpp:117: void tomahawk::twk1_t::calculateHardyWeinberg(): Assertion `ref == 0 || ref == 1 || ref == 4 || ref == 5' failed.
Aborted (core dumped)

The SNPs seem to meet the expectations of the program. I'm not entirely sure what's going wrong here. Please let me know if additional info would be useful.

Thanks for reporting this @peterdfields . The problem appears to an assertion I've place in the computation of Hardy-Weinberg equilibrium. For some reason the offending line has an allele that is not biallelic (encodings 0, 1, 4, or 5 in my internal format). This should not happen as non-biallelic, non-diploid sites should be filtered out. Must be some edge case I have not covered.
Could you find the offending line and report it to me? By email if it is not public.

Hi @mklarqvist. Given that an allele remains non-biallelic I have to assume it has somehow made it past gatk selectvariants and vcftools filtering for biallelic snps. Is there a way to force tomahawk to output the line that has the error? I tried with version of the program built with make DEBUG=true but that doesn't change the import stdout info.

@peterdfields I'll update the error message to reflect the offending variant line number and offending allele encoding. This is something I should've done in the first place.

@mklarqvist would there be an alternative method to localize the problem line?

@peterdfields A crude way would be like a manual binary search:

Input first half of file and check (easiest way is to pipe the data in from bcftools | head -n | tomahawk import)
Input second half and check
Keep splitting the half that fails. You should be able to deduce pretty quickly what the offending line is

I'm digging through the code to find the problem.

@mklarqvist Okay, I followed your advice about doing the manual binary search. The line from the vcf that is causing the error is as follows:
000011F|quiver  1151    .       A       T       1216.54 .       .       GT:AD:DP:GQ:PL  1/1:0,1:1:3:40,3,0      1/1:0,3:3:9:109,9,0     1/1:0,3:3:9:118,9,0     1/1:0,5:5:15:155,15,0   1/1:0,2:2:6:68,6,0      1/1:0,2:2:6:69,6,0      1/1:0,4:4:12:158,12,0   1/1:0,2:2:6:86,6,0      1/1:0,3:3:9:124,9,0     1/1:0,3:3:9:116,9,0     1/1:0,1:1:3:43,3,0      0/0:4,0:4:9:0,9,135     ./.:0,0:0:.:0,0,0       1/1:0,3:3:9:125,9,0     1/1:0,5:5:15:202,15,0

@peterdfields Thanks for helping me getting to the bottom of this. Very helpful! I am investigating this.

@mklarqvist no worries! I'm looking forward to exploring tomahawk.

Hi @mklarqvist. Any news about this issue? Thank you again for your help.

Hello @peterdfields . Sorry for the delay in resolving this. I returned today from a trip abroad. Will pick up were I left of. Thanks for your patience!

Hi @mklarqvist. Okay, great. Thank you again for your assistance!

Hi @mklarqvist. Any luck on tracking down this issue?

Hey @mklarqvist,
I got the same issue, I think the problem is related to missing data, or at least with './.' in the GT field. Replacing missing data with random genotypes or removing loci with any missing data solves the problem with import in my case.

I have the same problem too. Yes, removing sites with ANY missing data will resolve the situation, but this is not really a practical approach for my dataset.
Thanks

Same problem here ... with command line:
tomahawk import -i allsamples_All.bcf -o snp m 0.2 h 0.01
And, besides that,
tomahawk import -i allsamples_All.bcf -o snp -m 0.2 -h 0.01
tomahawk: invalid option -- 'm'
[2019-10-30 10:07:30,955][ERROR] Unrecognized option: ?

And the examples I found are wll with the dash as -m 0.xx -h 0.001 etc.
Isn't the dash needed for the options?

Hi. Is there an update for this problem? I have the same problem as well. Thanks

I also get this
tomahawk: lib/core.cpp:117: void tomahawk::twk1_t::calculateHardyWeinberg(): Assertion ref == 0 || ref == 1 || ref == 4 || ref == 5' failed. Aborted (core dumped)` error. I don't know what is not working.


Trying to access flag --sc2_run_config before flags were parsed

I get this error when running any of the scripts. I cant seem to get this fixed. Maybe its a problem with pysc2 itself?

Same here...

In meanwhile you can edit site-packages/pysc2/run_configrs/init.py
after FLAGS = flags.FLAGS
insert
import sys
FLAGS(sys.argv)

Okay, I got it. This repo uses gflags library, where pysc2 uses absl flags library. Inf your program You just have to set:
if name == 'main':
import sys
from absl import flags
FLAGS = flags.FLAGS
FLAGS(sys.argv)
main()

I had the same issue, when trying to execute one of the pysc2-examples. Changing the gflags library to the absl one fixed this for me. So what I did was:
Replace the line
import gflags as flags
with
import absl as flags
I also got a "All arguments must be passed as keyword arguments." error. This was fixed by changing:
  with sc2_env.SC2Env(
      "CollectMineralShards",
      step_mul=step_mul,
      visualize=True,
      game_steps_per_episode=steps * step_mul) as env:

to
  with sc2_env.SC2Env(
      map_name="CollectMineralShards",
      step_mul=step_mul,
      visualize=True,
      game_steps_per_episode=steps * step_mul) as env:

Anyone got a clue why this is? Couple months ago it did work out of the box for me. This time I had to change the example code. Please clarify if you know :)

I believe that I'm still having this issue, even with the fix.
$ /c/Python36/python train_mineral_shards.py --algorithm=acktr
algorithm : acktr
timesteps : 2000000
exploration_fraction : 0.5
prioritized : True
dueling : True
num_cpu : 4
lr : 0.0005
Process Process-2:
Traceback (most recent call last):
  File "C:\Python36\lib\multiprocessing\process.py", line 249, in _bootstrap
    self.run()
  File "C:\Python36\lib\multiprocessing\process.py", line 93, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Users\GeorgeNychis\Documents\src\sc2\pysc2-examples\common\vec_env\subproc_vec_env.py", line 14, in worker
    map_name=map_name
  File "c:\users\georgenychis\documents\src\sc2\pysc2\pysc2\env\sc2_env.py", line 132, in __init__
    self._setup((agent_race, bot_race, difficulty), **kwargs)
  File "c:\users\georgenychis\documents\src\sc2\pysc2\pysc2\env\sc2_env.py", line 173, in _setup
    self._run_config = run_configs.get()
  File "c:\users\georgenychis\documents\src\sc2\pysc2\pysc2\run_configs\__init__.py", line 38, in get
    if FLAGS.sc2_run_config is None:  # Find the highest priority as default.
  File "C:\Python36\lib\site-packages\absl\flags\_flagvalues.py", line 509, in __getattr__
    raise _exceptions.UnparsedFlagAccessError(error_message)
absl.flags._exceptions.UnparsedFlagAccessError: Trying to access flag --sc2_run_config before flags were parsed.

Environment: Windows 10
Python: 3.62.
pysc2 @ 7a04e74effc88d3e2fe0e4562c99a18d06a099b2
pysc2-examples @ f188d18024aa5c3b13c93400f17a956722280a17
absl-py @ 0.1.4

@gnychis Ran into the same issue, even with the recent changes.
Rectified it by doing the following:
Edit the following in pysc2\run_configs_init_.py (N.B: I'm using Anaconda, so you will need to find the location of pysc2)
From this (Line 25 to 27):
flags.DEFINE_string("sc2_run_config", None,                     "Which run_config to use to spawn the binary.") FLAGS = flags.FLAGS
To this:
`flags.DEFINE_string("sc2_run_config", None,
"Which run_config to use to spawn the binary.")
FLAGS = flags.FLAGS
import sys
FLAGS(sys.argv)`
A bit of a hack I know, however I've only just became acquainted with the project so I'm trying to find the root cause. Hopefully of some help.

@AGreatPigeon works for me! Thank you!

Works, thank you!

@AGreatPigeon Works! thanks


Add (exit) convenience function to shell

@dry Btw, this is a nice addition -- thanks :-)
I've been updating docs to mention it ...


Increment Version

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/49494
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.2 -m "<description of version>" 689a25a504d138992429a0ca113a267898bad6cd
git push origin v0.1.2

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/82939
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.3 -m "<description of version>" e17bc739862bec91259fda8d8d3f2f76fdff186b
git push origin v0.1.3

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/97430
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.4 -m "<description of version>" c4fefc37661f748fe4821ee366e5e6734f60c80e
git push origin v0.1.4

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/97435
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.5 -m "<description of version>" f0657d125338007bf354bbfeb945d9e53af559c9
git push origin v0.1.5


[Bug] newsflash feed - lazy load fetch the same items

Describe the bug
newsflash feed - lazy load fetch the same items
To Reproduce
Steps to reproduce the behavior:
Scenario 1:

open network tab and use  filter /api/news-flash
Go to Homepage
See initial count is 10
click on ynet - see request does not contain count param
click on all - see request does not contain count param

Scenario 1:

open network tab and use  filter /api/news-flash
Go to Homepage
See initial count is 10
scroll newsflash feed down to the bottom- see nest request count is 5
scroll newsflash feed down to the bottom again - each time count add 5 to it's value

Expected behavior

Lazy loading batch (count) should be  constant (value of 100 is a good start)
Pagination should be added to the URL (something like "start_item_index" param). should be discussed with @atalyaalon

Environment
Browser: Chrome
Environment:  Staging
Additional context
See API doc

Opened an issue in BE

hi @atalyaalon  @yuvalbl @almogbhl,
I think an easy API for both BE and FE here will be to add two optional parameters to the query string:
- page, which will be 1 by default
- count/per_page_count, which will be 100 by default as Yuval suggested.
let me know if you agree so I can start implementing it on from BE side

hi @atalyaalon  @yuvalbl @almogbhl,
I think an easy API for both BE and FE here will be to add two optional parameters to the query string:
- page, which will be 1 by default
- count/per_page_count, which will be 100 by default as Yuval suggested.
let me know if you agree so I can start implementing it on from BE side

@idosavion
Fine by me

@idosavion
Fine by me

@idosavion In addition to what @yuvalbl wrote, I suggest following best practices if there are ones. Besides that, seems good to me, you can start implementing.

@idosavion In addition to what @yuvalbl wrote, I suggest following best practices if there are ones. Besides that, seems good to me, you can start implementing.

@yuvalbl, np, but let call the second param limit instead of count? will indicate that the query might include less items if offset is big enough

@yuvalbl, np, but let call the second param limit instead of count? will indicate that the query might include less items if offset is big enough

https://github.com/hasadna/anyway/pull/1676

https://github.com/hasadna/anyway/pull/1676

@yuvalbl @almogbhl
API is in production - you can use it :) - @idosavion  thank you so much!
Note - it's unrelated to this issue - but there is a small bag you encountered yesterday - of News Flashes that don't have road_segment but still appear in the API - we're working to solve this - see this issue.

@yuvalbl @almogbhl
API is in production - you can use it :) - @idosavion  thank you so much!
Note - it's unrelated to this issue - but there is a small bag you encountered yesterday - of News Flashes that don't have road_segment but still appear in the API - we're working to solve this - see this issue.


Update ubuntu version to 22

Codecov Report

Merging #141 (056ec23) into master (011e5fc) will decrease coverage by 0.96%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #141      +/-   ##
==========================================
- Coverage   86.27%   85.30%   -0.97%
==========================================
  Files          17       17
  Lines        3110     3110
==========================================
- Hits         2683     2653      -30
- Misses        427      457      +30

Impacted Files
Coverage Δ

geofileops/util/_general_util.py
66.94% <0.00%> (-17.36%)
:arrow_down:

geofileops/util/_io_util.py
75.00% <0.00%> (-6.67%)
:arrow_down:

geofileops/util/_geoops_gpd.py
86.98% <0.00%> (-0.72%)
:arrow_down:

geofileops/util/_ogr_util.py
81.90% <0.00%> (-0.48%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 011e5fc...056ec23. Read the comment docs.


code

Thank you for your wonderful work! May I ask if the code will be released? If it is released at what point in time roughly? Thanks again!

Thank you for your attention, the code has been uploaded, please feel free to contact us if you have any questions.


Mask non-opaque proxy tiles with stencil buffer

This PR eliminates a long-standing, nasty artifact that occurs with features rendered with non-opaque blend modes when zooming -- as one or more levels of proxy tiles are rendered on top of each other (and/or with the new tile that is loading to replace them), the compounding alpha causes a flickering effect.
This PR avoids these artifacts using the stencil buffer: proxy tiles are drawn from front to back, with the stencil buffer adjusted for each level (e.g. zoom ancestor or descendant level) such that proxies will only draw pixels not previously covered by a previous level. Note, it's important that pixels from proxies are allowed to overwrite other pixels from the same proxy level, for overlapping geometries to render properly; we just want to mask out pixels from other, previously draw proxy tile levels.
(The translucent blend mode already avoided this problem, also by using the stencil buffer, though tailored for the particular "x-ray" effect of that blend mode, where backfaces are eliminated from translucent geometry.)
Before 😭

After 😌

Note: this PR builds on the commits of #703 for convenience (changed similar code). The specific changes for this feature are https://github.com/tangrams/tangram/compare/f7a10c1...8bfc5d0.

\o/

There's an issue when applying this technique to points/text features: unlike polygons/lines, which represent "fixed" geometry in space (thus the proxy tile and foreground tile geometries tend to "line up" and represent the same objects), points and text labels have purely transparent pixels (e.g. for transparent pixels in sprites) and/or partial transparency that unintentionally obscure geometries from other proxy (or foreground) levels. This causes a "cut out" or "x-ray" effect (also typical when mixing depth and alpha). For example:

The original problem this PR solves is much more acute for polygon geometries, especially because of the large tile-shaped chunks which cause an obvious "pop" It is much less of an issue for point features (at least in current use cases). So for now, this PR disables the proxy stencil technique for points/text (keeping it for polygons/lines). Further solutions for points can be researched for the future.


Multi-Discrete Action Space

I am attempting to adapt this agent to work with a custom gym/universe environment I'm building. The action space of my environment uses a MultiDiscrete space. This starter agent explicitly references env.action_space.n from the Discrete space which doesn't exist on a MultiDiscrete object. I am able to change those references to env.action_space.shape instead, but that breaks for Discrete spaces, and, more significantly in my case, it results in the agent choosing a single action from the MultiDiscrete space to 'max out' with these calls:
value = tf.squeeze(tf.multinomial(logits - tf.reduce_max(logits, [1], keep_dims=True), 1), [1])
tf.one_hot(value, d)
[...]
env.step(action.argmax())

My action space looks like this:
self.action_space = spaces.MultiDiscrete([[-80, 80],
                                          [-80, 80],
                                          [0, 1],
                                          [0, 1],
                                          [0, 1]])

The agent chooses an action like [0. 0. 0. 0. 1.] or [0. 1. 0. 0. 0.]. I would like it to choose an action from within the action space such as [74. -52. 0. 1. 0.].
Does anyone have any suggestions as to how I can accommodate this? Ultimately, I think it would be best to build support for all the types of Space since it seems the idea is to abstract away environment details from the agent anyway. I'm new to Gym, Universe and Tensorflow and really ML in general, so any help you can offer would be much appreciated. Thanks.

@bzier any updates on this plz? I am working on your mupen64plus env and I'm having the same issue with the actions spaces

I'm sure there's a more 'correct' way to do it, but for my application, I've just given up entirely on using both multi-discrete or tuple action spaces.  I instead just use a normal discrete space with tons of discrete actions, each representing the combination of what would have been my multi-discrete variables.  It's a pain, but it works at least.  None of the libraries I've found online really work with multi-discrete action spaces for the same reason bzier pointed out.  I recommend that if you actually want to get something working sooner rather than later, you try just translating those combined action states into a single action vector.  Mine has over 1500 as a results, which is annoying, but workable.  Also, if you're using a large action space set, check out the Wolpertinger algorithm developed by DeepMind.  jimkon has built a repo that gets like 95% the way to implementing it... just needs some work for it to work with discrete spaces.  But it's supposed to work much, much better for large discrete spaces than anything else out there. I hope more implementations for it come out soon.

@M00NSH0T this exactly the way I'm implementing it right now, and the least I can say it is not the most "correct" way to do it. Thanks for pointing me to the Wolpertinger algorithm, I will take a look on that.

@aymen-mouelhi yeah far from the most elegant / ideal way to do it.    If you find a better solution, could you post it here?
Also, one other thing I did that seems to have helped a lot in my application, which was inspired by something I read in the paper about the Wolp algo, was action filtering.  This too is something that isn't really built into OpenAI the way I think it needs to be (unless I'm just missing it... I am still learning). I added a function to my own custom openai environment that basically spits out a vector of ones and zeros each timestep to say whether a particular action is valid or not.  I then use that to zero out the Q values for invalid actions before my agent makes a choice, but obviously it would be more computationally efficient if there was a more embedded way of doing this that just prevented those actions from having q values calculated for them in the first place.  I may go back and try to get implemented, but it would require a lot more effort than my quick and dirty solution there. Still, it saves a ton of time in training large action spaces because at least you don't waste resources exploring invalid actions.  My previous approach was to penalize invalid actions, but that only partially worked. Constraining my system to only explore valid actions in the first place has made a huge difference.
Anyway, good luck.

@aymen-mouelhi I never worked out the multi-discrete action space issues. Like @M00NSH0T, I ended up implementing a discrete space, but in my case I only used a dozen of so actions, rather than coming up with all the possible permutations.
See here for the implementation: https://github.com/bzier/gym-mupen64plus/blob/3551409db59ccba1912c004c1612b2b1aa2ecc8d/gym_mupen64plus/envs/MarioKart64/discrete_envs.py#L5
I have been able to successfully train an RL agent using an adapted version of this code, along with some changes to my environment. I still need to clean up some things, write up some documentation, and merge the code. But I can at least point you to what I have at the moment (just be aware it's not done yet).
Environment changes are here: https://github.com/bzier/gym-mupen64plus/pull/38
Agent tweaks are here:
https://github.com/bzier/universe-starter-agent/tree/mario-kart-agent
Be aware that the agent changes are also based on changes to run everything in docker containers. You can find those changes in the 'dockerize' branch here:
https://github.com/bzier/gym-mupen64plus/tree/dockerize
Like I mentioned, I need to do some work still to pull this all together and make setup easier and clearer. If you have any questions, or bump into any problems related to that, feel free to open an issue on that repo. It will make the conversation a little easier to track and reduce clutter here.
If anyone ends up figuring out the true multi-discrete problem, absolutely comment here.

@bzier Thank you very much for your very detailed answer (and links) ! I will try your implementation and try to check how it works and then hopefully some pull requests will start coming :)

Similar problem:
This code:
self.action_space = spaces.Tuple(             [spaces.Discrete(3), spaces.Discrete(10)]             )
gave this error:
num_actions=env.action_space.n, AttributeError: 'Tuple' object has no attribute 'n'
and this:
self.action_space = spaces.MultiDiscrete([3,10])
gave this error:
  self.low = np.array([x[0] for x in array_of_param_array]) TypeError: 'int' object is not subscriptable


Travis-CI

Could Travis-CI be implemented on this repository as to run tests on Pull Requests made?

https://github.com/BitGo/prova/pull/27

Travis-CI can now be enabled on this repository.

Enabled and activated with https://github.com/BitGo/prova/pull/31


Revert "feat: add webhook types"

Reverts prismicio/prismic-types#17

Codecov Report

Merging #20 (d34084b) into master (d895ccf) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #20   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         3    -1
  Lines           17        12    -5
=========================================
- Hits            17        12    -5

Impacted Files
Coverage Δ

src/index.ts
100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d895ccf...d34084b. Read the comment docs.


Update to CRA/scripts 3.1.1

Title tells it all!
Thx

This would be amazing ^^ +1

CRA 3.3.0 is currently up for review.


condlane

one of the variables needs for gradient computation has been modified

one of the variables needs for gradient computation has been modified

Hello，did you resolve it? Please help me.


Search functionality broken

[ ] When I type a number of an NFT and press enter or click to the search icon It should show the one with that number.
[ ] If the number length is less then 4 It should add leading zeros to make the length equal to 4. ie. If I typed; 3 or 0003 It should focus to the parcel 0003

Search will only get the exact number.
No auto complete feature or multiple search results.


Lesser update for moloch_update_geo.sh

Lite version of https://github.com/aol/moloch/pull/1516
Clearly describe the problem and solution
Fix escapable environmental variables
Relevant issue number(s) if applicable
#976661
Did you run npm run lint from the viewer or parliament directory (whichever you are making changes to) and correct any errors
No
Did you Ensure that all tests still pass by navigating to the tests directory and running ./tests.pl --viewer
No
License
I confirm that this contribution is made under an Apache 2.0 license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Unfortunately this breaks one of the most important goals of the script, if the download fail don't break a running moloch. Also don't like the mkdir being there. What is actually the goal here? What is issue 976661?

Unfortunately this breaks one of the most important goals of the script, if the download fail don't break a running moloch. Also don't like the mkdir being there. What is actually the goal here? What is issue 976661?

Understood, I reverted the changes, just quoted the appropriate variables. Issue is via hackerone https://hackerone.com/reports/976661


some data download questions

hi,friend,your work is really charming,but i can not download your V2 version datas,such as customShapeNet and ShapeNetRendering,can you upgrades these datasets download links,thank you very much

Hi pxing6,
Thanks for your interest!
Have you tried all these solutions ? https://github.com/ThibaultGROUEIX/AtlasNet#a-note-on-data
Cheers
-Thibault

thanks for your reply.Yes,i have tried all these solutions,these links are vaild,and i have downloaded your AtlasnetV1 project and datasets(ShapeNetV1PointCloud,ShapeNetV1Renderings).But i can not download your AtlasnetV2 datasets such as customshapenet(https://cloud.enpc.fr/s/j2ECcKleA1IKNzk) and shapenetrendering(https://cloud.enpc.fr/s/S6TCx1QJzviNHq0),these download links are invalid.I really need them,can you help me.thank you very much.

two links are invalid,and i really need these two datasets,can you help me,thanks.

Hi,
is this from a previous version of the repo?
Best
-Thibault

hi,Thibault
yes, they are repo from previous version and i need them(include customShapenet and ShapeNetRendering),could you help me? thank you very much.

HI,
I don't support this version anymore.
You can try these links
renderings : https://cloud.enpc.fr/s/axiC3FHS374FgEf
points : https://cloud.enpc.fr/s/JNf3NAxGbQoQsKY
Cheers,
Thibault

OK,i will try to download these link.thanks for your reply

OK,i will try to download these links.thanks for your reply.

Closing this since the issue is stale, feel free to reopen. Thanks a lot for for playing around with AtlasNet.
Cheers,
Thibault


Fix #13873

Summary
https://github.com/apache/nuttx/pull/13755 break C++ build with GCC libstdc++. This PR re-enable the macro while takin care of MSVC build.
Impact
Fix #13873 - build on G++ while keeping MSVC happy.
Testing
Only with GCC, I do not have tools to test on MSVC.

ping @xuxin930 Can you test this PR with your MSVC setup to ensure no regression on your side?

@leducp could you switch one defconfig to link toolchain's libcxx?

Yes I am planning to do so, but it will not be sufficient: we need a user app that use a few tools from the standard C++ library to catch theses kind of errors.

it's enough to enable apps/testing/cxxtest/

I can add something but I do not know how to tell the CI to use it. Any clue?

it simple to add/modify one defconfig under boards/, ci will build your defconfig automatically.

@leducp could you switch one defconfig to link toolchain's libcxx?

Yes I am planning to do so, but it will not be sufficient: we need a user app that use a few tools from the standard C++ library to catch theses kind of errors.

it's enough to enable apps/testing/cxxtest/

I can add something but I do not know how to tell the CI to use it. Any clue?

it simple to add/modify one defconfig under boards/, ci will build your defconfig automatically.

I enabled the cxxtest app and the C++ toolchain with all options (RTTI, exceptions, etc) on the xmc4800-relax. Let me know if this is OK

looks fine.


cursor focus

When I click on the auto complete box, the cursor focus does not show the normal |
instead it is blank, and it is confusing to the user what textbox the user is in. For example, we have 20 autocomplete boxes on a form.
<input auto-complete
  class="form-control"
  [(ngModel)]="selectedColumn"
  [source]="columnsNotInDv"
  [min-chars]="1"
  [max-num-list]="10"
  placeholder="enter column name"
  display-property-name="DisplayName">

I see a cursor. Is it environmental issue?

@allenhwkim , I am testing in the latest version of google chrome. May I ask, what browser are you using? I will test in IE as well.

It must be something wrong with my css somewhere. cheers.

for future reference: removing the form-control class fixes


library "libsqlitejdbc.so" not found on Android

Describe the bug
Run on the android platform will issue this bug
To Reproduce
Connect SQLite DB with sqlite-jdbc from the Android platform.
Logs
2022-10-12 11:27:01.122 16877-16877/com.demo.sqlite E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.demo.sqlite, PID: 16877
java.lang.UnsatisfiedLinkError: dlopen failed: library "libsqlitejdbc.so" not found
at java.lang.Runtime.loadLibrary0(Runtime.java:1077)
at java.lang.Runtime.loadLibrary0(Runtime.java:998)
at java.lang.System.loadLibrary(System.java:1656)
at org.sqlite.core.NativeDB.(NativeDB.java:40)
at org.sqlite.core.NativeDB.load(NativeDB.java:60)
at org.sqlite.SQLiteConnection.open(SQLiteConnection.java:281)
at org.sqlite.SQLiteConnection.(SQLiteConnection.java:68)
at org.sqlite.jdbc3.JDBC3Connection.(JDBC3Connection.java:28)
at org.sqlite.jdbc4.JDBC4Connection.(JDBC4Connection.java:19)
at org.sqlite.JDBC.createConnection(JDBC.java:104)
at org.sqlite.SQLiteConfig.createConnection(SQLiteConfig.java:108)
Environment (please complete the following information):
OS: [Android 12]
CPU architecture: [aarch64]
sqlite-jdbc version [<IP_ADDRESS>]
Additional context
About Android ABI description:
https://developer.android.com/ndk/guides/abis

The android native library directory name needs to be corrected.
/org/sqlite/native/Linux-Android
aarch64 -> arm64-v8a
arm -> armeabi-v7a
x86
x86_64
I have tested it, It's working now on Android.

The android native library directory name needs to be corrected. /org/sqlite/native/Linux-Android aarch64 -> arm64-v8a arm -> armeabi-v7a x86 x86_64
I have tested it, It's working now on Android.

Can you clarify, i don't understand

About android native library https://developer.android.com/studio/projects/add-native-code

need rename directory aarch64 to arm64-v8a
and
rename directory arm to armeabi-v7a
Then android loads the native library from arm64-v8a if the android MCU is aarch64 type.
The android system will not load the native library from aarch64 directory.
The source from sqlite-jdbc GitHub android native directory as follow:
jrjohn@Chiende-MacBook-Pro-3 Linux-Android % pwd
/Users/jrjohn/Downloads/org/sqlite/native/Linux-Android
jrjohn@Chiende-MacBook-Pro-3 Linux-Android % ls
aarch64	arm	x86	x86_64
jrjohn@Chiende-MacBook-Pro-3 Linux-Android %

The link you provided doesn't explain anything about the directories.
The directory names in sqlite-jdbc are used to perform a lookup depending on the OS/arch detected, then the corresponding native library is copied to a temp directory.
I don't see how changing the directories where the native lib is stored would change anything.

https://developer.android.com/ndk/guides/abis#gradle
ABI management on the Android platform
This section provides details about how the Android platform manages native code in APKs.
Native code in app packages
Both the Play Store and Package Manager expect to find NDK-generated libraries on filepaths inside the APK matching the following pattern:
/lib//lib.so.  <====== this is reason
Here,  is one of the ABI names listed under Supported ABIs, and  is the name of the library as you defined it for the LOCAL_MODULE variable in the Android.mk file. Since APK files are just zip files, it is trivial to open them and confirm that the shared native libraries are where they belong.

Current ABI name-only support:
armeabi-v7a
arm64-v8a
x86
x86_64

I don't think the issue Android UnsatisfiedLinkError #248 is done.

In that case the problem is more about how the lib is loaded for Android than just the directory name.
The articles you provide describe native libs that are within the project, but not really about native libs that are within a dependency though.

So currently if i use
dependencies {
implementation 'androidx.appcompat:appcompat:1.4.2'
implementation 'com.google.android.material:material:1.6.1'
implementation 'androidx.constraintlayout:constraintlayout:2.1.4'
implementation 'androidx.navigation:navigation-fragment:2.3.5'
implementation 'androidx.navigation:navigation-ui:2.3.5'
testImplementation 'junit:junit:4.+'
androidTestImplementation 'androidx.test.ext:junit:1.1.2'
androidTestImplementation 'androidx.test.espresso:espresso-core:3.4.0'
implementation 'org.xerial:sqlite-jdbc:<IP_ADDRESS>'

I need to add jniLibs to myapp source:
app
src
main
jniLibs
armeabi-v7a
libsqlitejdbc.so
arm64-v8a
libsqlitejdbc.so
x86
libsqlitejdbc.so
x86_64
libsqlitejdbc.so
I think this should be included in the jar file.

When i opened the APK generated by Android Studio, i can see the Mac and Windows native libraries in /org/sqlite/native, but not the other libraries for Linux.
I have a feeling that they are excluded by Gradle automatically because they are .so.

I couldn't fathom how the library loading worked, and i just found out that specific bit of code for Android.
When Android is detected, it uses System.loadLibrary instead of the SQLiteJDBCLoader.
My understanding is that

Gradle dependencies cannot have .so files in it. If they do, those are stripped in the resulting APK (as i found out in the previous comment).
System.loadLibrary will find the libraries without having to provide the full path, but that requires the libraries to be added to the jniLibs directory in Android Studio

I feel like this is not an issue, but should be properly documented.
There's no way the jar can provide the android libraries in the right folder AFAIK. If you can find a way for a third-party dependency to provide native libs for Android, we can evaluate a change.

I also found this: https://developer.android.com/studio/releases/gradle-plugin#extractNativeLibs
I haven't found how to keep the .so files in the APK, but it seems it's not the best way to do for Android.
Ideally what we should have is a way to automatically:

extract the 4 android libraries from the jar during build
strip the other libraries of the jar in order to optimize the apk size

That would be however outside of the scope of this project, and should probably be handled by a dedicated Gradle plugin for Android.

I'll try how can finish read .so file from the jar file and report it here. Thanks


Still active?

Is this project still maintained?
If so, what differences are expected from electron-panel-window?

@lacymorrow As you can see, unfortunately, it's currently not maintained. You need to find the alternative. Good luck 😀

@lacymorrow As you can see, unfortunately, it's currently not maintained. You need to find the alternative. Good luck 😀

Thanks for the response, cheers!

Thanks for the response, cheers!


flynn-host instances expire due to discoverd stall

@temujin9 has been running into this one a lot.
It looks like some heartbeat writes are stalling for >8s, and so instance expiry gets triggered:
https://gist.github.com/temujin9/af84cece2583c34b6846#file-ip1010046237-ea6f86be-dd2d-47ce-9aa2-e2081f47cd1a-log-L232-L248
https://gist.github.com/temujin9/0567e5a35d89a4bc7530#file-ip1010036205-ceb1eb9d-645d-4219-b73d-f1fb910f8397-L73-L80
So far it's unclear why this is happening.
/cc @benbjohnson
Debug logs:
https://gist.github.com/dd3949c2cb3a3cb3d229
https://gist.github.com/a99e75e796cefdc10bf0

@temujin9 What EC2 instance type/configuration and region are you using? I think it's worth us trying to reproduce this outside of your system.

The current clusters are single node r3.larges in us-east-1. I'm not designating AZ; the one that's still working is in us-east-1c at the moment.

@titanous The only thing I can think of is latency with the underlying raft log storage although that seems really high even for AWS.

I just reproduced this on AWS with our flynn install tool.

@benbjohnson I can now reliably reproduce this, so if you put together more debug logging we can get moving on fixing this.

@titanous I added raft timings for commit (time to get consensus), lock (time to obtain the Store.mu once applying), and exec (time to actually run the command):
https://github.com/benbjohnson/flynn/tree/apply-log
I wasn't sure if you wanted that PR'd to master or if you wanted to simply work off that commit.

K, great. I'll do a run with that on.

Here's the log: https://gist.github.com/anonymous/c73d3fe340f2d11092ea#file-discoverd-app-ip1001150-9ebda4e8-7516-497b-94a8-f946ab303ebe-log-L252-L266
I guess we need more granular logging in the raft library?

@titanous Is it possible to do a cpu or blocking profile on the discoverd instance?

@benbjohnson Yeah, I can do that later today. The iterations are pretty slow since I need to build an EC2 image, do you have anything else you want me to do in the same run?

@titanous I cherry picked https://github.com/benbjohnson/flynn/commit/ae140dd33ea2703ceba3d53c71c4ae0ef56dfcb3 over to the apply-log branch so we can get granular logging inside raft.

Here's the blocking profile: https://cl.ly/1V2s1p3N331z
And the log: https://gist.github.com/titanous/1210422abdef64bbdb88

I'm closer to this now:

dispatchLogs: store=3.938028492s start=2.666µs set=301ns notify=2.197µs

This line is taking multiple seconds sometimes. Digging further in.

A layer deeper and we find:

StoreLogs: n=1 begin=7.276µs put=32.342µs commit=2.201457741s

That's right, the boltdb commit is taking >2s.

Hmm, can you grab a cpu profile? 2s commit seems strange.  What kind of IOPS are available on the AWS box?

I'm pretty sure this is the problem:

New EBS volumes receive their maximum performance the moment that they are available and do not require initialization (formerly known as pre-warming). However, storage blocks on volumes that were restored from snapshots must be initialized (pulled down from Amazon S3 and written to the volume) before you can access the block. This preliminary action takes time and can cause a significant increase in the latency of an I/O operation the first time each block is accessed. For most applications, amortizing this cost over the lifetime of the volume is acceptable. Performance is restored after the data is accessed once.

https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/ebs-initialize.html
The solution is to create a separate data EBS volume when starting the EC2 instances, this way we don't have the "initialization" cost of the boot volume that loaded from the AMI (aka snapshot). It's not the IOPS limit because the burst credit bucket is pre-loaded with enough credits for 30 minutes of writes at 3k IOPS.

Good find. That seems nuts that it would take that long though.

For all we know it's doing an HTTP request to S3 behind the scenes. :facepunch:


[QUESTION] Having trouble with connecting to external css file?

Question
For some reason my CSS file isn't loading and I'm not sure what do with this type of error. There is a reference to the stylesheet once the app is built: <link rel="stylesheet" href="/assets/dashycsstweaks.css">, but when I click it, there is the following error message: Cannot GET /assets/dashycsstweaks.css. It looks like it's returning a 404, but my background in the same folder shows up fine. I tried the solutions in this SO link (that I understood) to no avail (note: link is a cache link as SO is currently down for maintenance).
CoolConsole.js:6
Dashy  V2.0.4 🚀
<IP_ADDRESS>/:1 Refused to apply style from 'http://<IP_ADDRESS>:4000/assets/dashycsstweaks.css' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.
<IP_ADDRESS>/:1 Uncaught (in promise) EventisTrusted: truebubbles: falsecancelBubble: falsecancelable: falsecomposed: falsecurrentTarget: nulldefaultPrevented: falseeventPhase: 0path: (5) [link, head, html, document, Window]returnValue: truesrcElement: linktarget: linktimeStamp: 3239.600000023842type: "error"[[Prototype]]: Event
<IP_ADDRESS>/:1 Refused to apply style from 'http://<IP_ADDRESS>:4000/assets/dashycsstweaks.css' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.

Response Headers
HTTP/1.1 404 Not Found
X-Powered-By: Express
Content-Security-Policy: default-src 'none'
X-Content-Type-Options: nosniff
Content-Type: text/html; charset=utf-8
Content-Length: 164
Date: Sat, 12 Mar 2022 20:14:35 GMT
Connection: keep-alive
Keep-Alive: timeout=5

Category
Setup and Deployment
Please tick the boxes

[X] You are using a supported version of Dashy (check the first two digits of the version number)
[X] You've checked that this question hasn't already been raised
[X] You've checked the docs and troubleshooting guide
[X] You agree to the code of conduct

I have the same issue.  I am running Dashy using the latest Docker image: ghcr.io/lissy93/dashy
I did not find anywhere in the container filesystem a file called dashycsstweaks.css

dashycsstweaks.css

That's my user-defined CSS file.

In the meantime, you can put your CSS file in /public (or a sub dir within), and that should work.

Oh ok. Weird. It used to work when I had /volume1/docker/dashy/assets/dashycsstweaks.css:/app/src/styles/user-defined-themes.scss, but the error goes away with /volume1/docker/dashy/assets/dashycsstweaks.css:/app/public/assets/dashycsstweaks.css. Thanks!

Ah yeah this would have been caused by the changes in how the config is loaded in the past version, since the app source was removed from the Docker image. This is being added back in, in #559 - so once merged, that should fix your issue


Fix Container Image Condition deleting

Bug-url: https://bugzilla.redhat.com/show_bug.cgi?id=1340165
with the fix:

before:

@simon3z @moolitayer Please review

@miq-bot add_label providers/containers, bug, ui

LGTM :+1:

@moolitayer can you review this? (just to make sure everything is in order... are there any other left?)

Work for me! :+1: LGTM

@simon3z I can't find any OpenSCAP related issues

LGTM :+1:
@miq-bot add_label darga/yes
@miq-bot assign chessbyte

@h-kataria PTAL

looks good


Change to access to User.Read.All in MS Graph

Recently, I can no longer select Azure AD Graph as a service principal.
Therefore, you cannot grant the service principal permission to do things like az ad user list.
I've confirmed that with the latest version, but it seems that the API of Azure AD Graph is accessed by the Azure CLI, which is very troublesome.
Please adapt the az ad user command of Azure CLI as soon as possible to refer to User.Read.All of MS Graph.

Recently, I can no longer select Azure AD Graph as a service principal.

This is explained in the pinned issue https://github.com/Azure/azure-cli/issues/19818.

Please adapt the az ad user command of Azure CLI as soon as possible to refer to User.Read.All of MS Graph.

This is tracked by https://github.com/Azure/azure-cli/issues/12946.


KafkaSource.pull is slow due to immutable.List.append

We've observed that pulling from kafka is quite slow. In the current implementation, KafkaConsumer  running in a separate thread will pull in messages from kafka and put on a in memory queue (LinkedBlockingQueue), and KafkaSource.pull will continuously poll the head of the queue.  I found that KafkaConsumer pulls at high speed while consume blocks at KafkaSource.pull (below).
  override def pull(number: Int): List[Message] = {
    @annotation.tailrec
    def pullHelper(count: Int, msgList: List[Message]): List[Message] = {
      if (count >= number) {
        msgList
      } else {
        val optMsg: Option[Message] = fetchThread.poll.flatMap { kafkaMsg =>
          val msg = messageDecoder.fromBytes(kafkaMsg.msg)
          offsetManagers(kafkaMsg.topicAndPartition).filter(msg -> kafkaMsg.offset)
        }
        optMsg match {
          case Some(msg) => pullHelper(count + 1, msgList :+ msg)
          case None      => pullHelper(count + 1, msgList)
        }
      }
    }
    pullHelper(0, List.empty[Message])
  }

As per http://docs.scala-lang.org/overviews/collections/performance-characteristics.html, immutable.List.append is O(log). Also, I've written a microbenchmark which showed that even using prepand O(1) and then reverse the list would be 7x faster.

For the note, List append operation need to traverse the whole list.


Car Mapping Method by Name/Number only miscalculation if name has multiple numbers and vice versa

This issue was raised in #26. If Name/Number mapping method is chosen and a name/number competes under multiple numbers/names, NRatings averages the averages of name/number combination. For example, Jeff Burton in 2004 drove 22 races in # 99 and 14 races in # 30 for an overall season average finish of 18.7 and average start of 23.7 (https://www.racing-reference.info/driver-season-stats/burtoje01/2004/W/). However NRatings suggests that the average finish is 18.1 average start 23.6. Nratings is taking an average of an average to determine these values. In other words, in the # 99 car, Burton had an average finish/start of 18.9/23.8, and in the # 30 car 15.4/22.9. Averaging those two values results in an average finish/start of 18.1 and 23.6 - the same values NRatings uses for the driver Jeff Burton when the mapping method Name Only is selected.
Following the logic to determine where NRatings is making this error is difficult to do without real life values, but it appears the issue is occurring in the real data processor (https://github.com/64Soft/NRatings.Client/blob/main/src/NRatings.Client/Business/RealDataProcessor.cs) during the creation of the stats list, approximately line 500 for the driver only method. The code for generating that list is different from the above code for mapping by both name and number above, which seems to work correctly. The mapping by number only below follows the same logic as mapping by name only, and therefore should experience this same issue.

@ai-line-mod thanks for your investigation. I unfortunately don't have much time at the moment to look into this, so my question to you is would you feel comfortable to set up a develop environment on your side to try and fix the issue ? I can set you up with the connection to the data server to connect it with your dev environment.

@64Soft Yes most definitely! I just set up the environment on my end. What do you need in order to connect it to the data server?

@ai-line-mod drop me an email at<EMAIL_ADDRESS>so I can send you the details by email


White Color Value

If I am understanding the code correctly in G35 White is defined as (channel max, channel max, channel max. ), which I take to be 15,15,15. There was some discussion on the orginal deepdarc post about only using 13,13,13 for white because the stock controller only used these values. It was speculated that the LEDs may have overheating issues if left at 15,15,15 for long periods. Any thoughts on this?

My experience is that 15, 15, 15 is inconsistently pinkish rather than white, and borderline ugly. I have a feeling it's exceeding current limits so that the blue LED (highest voltage drop) is getting a bit starved. Though now that I think about it, that should lead to a yellowish tint, not pink (RGB - B = RG = yellow). At any rate, yeah, 15/15/15 is at best aesthetically annoying, and as you say possibly damaging.
My lights are down for the season. Can you let me know whether 13, 13, 13 looks better? And if so, I'd be happy to take your patch.

So I was playing with the basic code and uploading a lights.fill_color to make string all one color. Red, Blue Green all look fine but when I upload white or warmwhite the color is all over the place. Some look light pink, others very pink and some look just fine. Also as I understand it the first variable of the function is the bulb at which the fill starts. This works fine of value 0 to 14 but when I use 15 to higher bulbs 0  and on start comming on again.
Any ideas what I am doing wrong here

Answering my own question the issue is somewhere in how I set up the power supply. When i splice in the old controller and adapter to it all the colors look just fine.
The setup I made has another 5v 3a adapter plugged into the Arduino and the V+, Data , and V- plugged into the 5v power pin, pin 13 data pin and ground power pin respectivly.
Any suggestions on a better setup?

The pink you're describing sounds identical to what I mentioned in my January 5 comment. How is what you're describing different from what I described?
Your original suggestion when you filed this issue was to limit white to 0xC, 0xC, 0xC. How has that worked out?

If when you desribed it as "inconsistently pinkish" you mean that when the whole string is filled with one color it looks more like a jumble of 3 or 4 colors than its the same issue. Setting to 0xC, 0xC, 0xC resulted in a inconsistantly yellowish light. I have been resistant to cutting the pigtail which goes from the stock controller to the stock power supply but I may go ahead and try using that and just connecting the grounds of the controller and power supply together and not run the power though the controller.

Sounds like the same as my experience. Let us know if you figure it out. Personally, I don't run all-white because it looks boring, especially for hardware that's RGB-capable.
Note that there are a few problems with your power setup.

The Arduino is speced to require 6-12V on the power jack. This is because there's a significant voltage drop in the 5V regulator. So if you're supplying only 5V to the Arduino, it's almost certainly squeaking by on something like 4V. The ATmega will run mostly fine on that voltage, but the lights most definitely want the full 5V -- particularly the blue lights.

Even if you supplied 6V or more, I suspect the voltage regulator on the Arduino isn't going to be happy with more than maybe 300ma being drawn from it, and these lights will try to suck out 1.6A on full white. My personal observation is that the USB plug on the Arduino is more of a pass-through, so if you have a beefy USB power supply, then your current wiring setup will work.

Keep your existing wiring, but run another wire from the power supply's positive lead directly to V+ on the lights. It'll be happier.
Finally, let's try to keep this issue on-topic. If you have a support request like "why aren't my lights working right?" please enter it separately.

I was running a 12V 1A power supply but the voltage regulator heated up so much and so quickly the board would malfunction after a few minutes. I will let you know if I can produce a consistant white.

Please enter a new issue if you'd like to keep discussing stuff other than the proper values for displaying white. See problem #2 on my last comment why the regulator is overheating.


1 | Two Sum | Java

Title -
What will change -
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code
[x] Improving Code
[x] Improving Documentation
[x] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[x] Python
[x] C++
[x] Java
[x] SQL

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

class Solution {
public int[] twoSum(int[] nums, int target) {
Hashtable<Integer, Integer> hashTable = new Hashtable<Integer, Integer>();
int i = 0;
while ((i < nums.length) && (hashTable.get(nums[i]) == null)) {
hashTable.put(target - nums[i], i);
i++;
}
if (i < nums.length) {
return new int[]{hashTable.get(nums[i]),i};
}
return null;
}
}

please assign to me.


Fix documentation of variable networkmanager_conf

README referenced networkmanager_config, but that should be networkmanager_conf.

conf is just the file extension chosen by the NetworkManager team. When choosing the variables in Ansible it comes to personal preference and I chose config, which I use across my various roles. I don't want to change it, therefore closing this ticket.

I'm sorry, I probably should have explained my intent more clearly. I think you misunderstood my intentions. I am not referring to a file extension and even agree with you that using config looks nice.
What I am intending with this PR is fixing a bug in your role. Variable networkmanager_config, as mentioned twice in the README, does not work. It's never used by the actual code and therefor any setting you put in there is ignored. The snippets of code I am referring to are:
Variable networkmanager_conf is defined in defaults/main.yml as an empty dictionary:
https://github.com/aisbergg/ansible-role-networkmanager/blob/d4939c1ef45bfd34728336839e9e62e6a629cdb7/defaults/main.yml#L19
And tasks/configure.yml is set up to combine the content of networkmanager_conf with defaults from __networkmanager_config_defaults defined in vars/main.yml:
https://github.com/aisbergg/ansible-role-networkmanager/blob/d4939c1ef45bfd34728336839e9e62e6a629cdb7/tasks/configure.yml#L13
All valid and working. If I, as a user of your nice role, define some personal in networkmanager_config it doesn't work. I had a look at the code to figure out why and noticed networkmanager_config is not used by the code but networkmanager_conf is. After which I figured this may have been an oversight in the documentation and set out to fix the issue.
If you prefer it the other way around, that's perfectly OK. I will update my PR accordingly to modify the code to match the documentation instead.


Improve amjadoudeh behavior

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

the last step I did for the lab4


Updated core team members(2024-2025)

Updated core team members for the tenure 2024-2025

will open and merge new PR with the changes.!


Cool stuff page creation

Creation of the "Cool Stuff' page that will contain articles/blog posts/starter kits/OSS projects related to game development with Azure.

created the "other samples" page which includes this stuff.


Document ways to communicate weekly priorities - replace weekly kickoff references

This PR replaces handbook references to the weekly kickoff.

@therebbie Feel free to merge since you and one other have approved.


Get started button icon position on mobile breakpoint

The icon on the get started button is cut off on the mobile breakpoint.
Although there is a specific background-position for this breakpoint, it is is set to 110%.
I would suggest changing to something more like 104% for it to be centred inside the right hand padding.

@dashouse @nickcolley see https://github.com/alphagov/frontend/pull/1402 this PR has been merged. Will be verifying this works on integration this afternoon.

https://www-origin.integration.publishing.service.gov.uk/vehicle-tax
vs.
https://www.gov.uk/vehicle-tax
The button styling is different when using the component, it's simpler and may well solve this issue.

This seems to have been fixed by using the magic of components 🎉


Grpc.Core server rejects gRPC requests forwarded with YARP with "Missing header"

Issue moved from microsoft/reverse-proxy#1191

Please respond to @mzabsky.

From @mzabsky on Wednesday, August 18, 2021 4:43:54 PM
Describe the bug
I have an insecure C# Grpc.Core server and want to forward requests to it via an YARP reverse proxy. The proxy receives the request and then the subsequent HTTP request to the gRPC server fails with an exception:
  System.Net.Http.HttpRequestException: An error occurred while sending the request.
   ---> System.IO.IOException: The request was aborted.
   ---> System.Net.Http.Http2StreamException: The HTTP/2 server reset the stream. HTTP/2 error code 'INTERNAL_ERROR' (0x2).
     --- End of inner exception stack trace ---
     at System.Net.Http.Http2Connection.ThrowRequestAborted(Exception innerException)
     at System.Net.Http.Http2Connection.Http2Stream.CheckResponseBodyState()
     at System.Net.Http.Http2Connection.Http2Stream.TryEnsureHeaders()
     at System.Net.Http.Http2Connection.Http2Stream.ReadResponseHeadersAsync(CancellationToken cancellationToken)
     at System.Net.Http.Http2Connection.SendAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
     --- End of inner exception stack trace ---
     at System.Net.Http.Http2Connection.SendAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
     at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean async, Boolean doRequestAuth, CancellationToken cancellationToken)
     at Yarp.ReverseProxy.Forwarder.HttpForwarder.SendAsync(HttpContext context, String destinationPrefix, HttpMessageInvoker httpClient, ForwarderRequestConfig requestConfig, HttpTransformer transformer)

I have followed documentation, namely the gRPC page. The most interesting gRPC server trace prints appear to be these "Missing header" and "Missing :authority or :path":
I0818 18:27:38.889382 0 ..\..\..\src\core\lib\channel\channel_stack.cc:239: OP[connected:000001FC81FC0CB0]:  CANCEL:{"created":"@1629304058.888000000","description":"Missing :authority or :path","file":"..\..\..\src\core\lib\surface\server.cc","file_line":1421,"referenced_errors":[{"created":"@1629304058.888000000","description":"Failed processing incoming headers","file":"..\..\..\src\core\ext\filters\http\server\http_server_filter.cc","file_line":123,"referenced_errors":[{"created":"@1629304058.888000000","description":"Missing header","file":"..\..\..\src\core\ext\filters\http\server\http_server_filter.cc","file_line":204,"key":"te"}]}]}
I0818 18:27:38.889556 0 ..\..\..\src\core\ext\transport\chttp2\transport\chttp2_transport.cc:1646: perform_stream_op[s=000001FC81FC1A70]:  CANCEL:{"created":"@1629304058.888000000","description":"Missing :authority or :path","file":"..\..\..\src\core\lib\surface\server.cc","file_line":1421,"referenced_errors":[{"created":"@1629304058.888000000","description":"Failed processing incoming headers","file":"..\..\..\src\core\ext\filters\http\server\http_server_filter.cc","file_line":123,"referenced_errors":[{"created":"@1629304058.888000000","description":"Missing header","file":"..\..\..\src\core\ext\filters\http\server\http_server_filter.cc","file_line":204,"key":"te"}]}]}

Full log
The call is served properly when I call the gRPC server from the client directly.
To Reproduce
I have prepared a reproduction repo. Just clone the repo, open in VS and F5.
I have also tried the code-first approach outlined on the Direct Forwarding page as well, with same result (having adjusted VersionPolicy in RequestProxyOptions).
Further technical details

Windows 10
VS 16.9.3
.Net 5.0
Grpc.Core 2.39.1
Yarp.ReverseProxy 1.0.0-preview.12.21328.2

Issue moved from microsoft/reverse-proxy#1191

Please respond to @Tratcher.

From @Tratcher on Wednesday, August 18, 2021 5:04:00 PM
Next steps:

Get a network capture (Wireshark) between YARP and the destination to see the wire format for the request and if one of these request headers really is missing.
Getting the debug level logs from YARP would also be useful. https://docs.microsoft.com/en-us/aspnet/core/fundamentals/logging/?view=aspnetcore-5.0#configure-logging

@mzabsky if you're able to capture those it would help, otherwise we'll get to it.

Issue moved from microsoft/reverse-proxy#1191

Please respond to @mzabsky.

From @mzabsky on Thursday, August 19, 2021 7:14:39 AM
Trace logs for the proxy host
Wireshark capture for the destination port
The headers:

Issue moved from microsoft/reverse-proxy#1191

Please respond to @Tratcher.

From @Tratcher on Thursday, August 19, 2021 4:22:08 PM
Thanks. The Wireshark and the YARP logs are consistent which implies a compat issue at the destination. @JamesNK who's a good person to follow up on the gRPC server end of this?

Issue moved from microsoft/reverse-proxy#1191

Please respond to @JamesNK.

From @JamesNK on Thursday, August 19, 2021 7:35:38 PM
Create an issue at https://github.com/grpc/grpc

I followed up a bit more and did a Wireshark capture of a direct call without forwarding. Here are the headers:

The only header missing in the forwarded request posted above appears to be "te: trailers". I modified the proxy to use HttpForwarder and added the header manually:
        private class CustomTransformer : HttpTransformer
        {
            public override async ValueTask TransformRequestAsync(HttpContext httpContext, HttpRequestMessage proxyRequest, string destinationPrefix)
            {
                // Copy headers normally and then remove the original host.
                // Use the destination host from proxyRequest.RequestUri instead.
                await base.TransformRequestAsync(httpContext, proxyRequest, destinationPrefix);
                proxyRequest.Headers.TE.Add(new TransferCodingWithQualityHeaderValue("trailers"));
                proxyRequest.Headers.Host = null;
            }
        }

After this modification, the forwarded call suddenly works and the client receives the reply. I have pushed a functional version of the reproduction code with this workaround to a branch.
However, I am not very familiar with HTTP/2 or details of the gRPC protocol so I can't tell if the server should be able to serve the request with the "te" header missing.