Deprecate selectrum-get-current-input

There is no reason to provide this function on top of
selectrum-is-active and minibuffer-contents.

There are two uses in the wiki, which should be changed. But probably these commands in the wiki are already superseded by Consult and/or should be written with completing-read instead of selectrum--read.

I'm fine with removing it, I believe this was left over from the time where minibuffer-contents wouldn't return the input but also the candidate string.

Feel free to merge things yourself, I trust your judgement which things need discussion. Is there a reason you quoted the nil or is that by accident?

And don't forget the changelog...

I trust your judgement which things need discussion. Is there a reason you quoted the nil or is that by accident?

Lol, it is a copy and paste error since I copied the make-obsolete from somewhere. I did not remember how this function/macro is called.


Unexpected end of file when running make

I was hoping to use this solution, but I get unexpected end of file when I run make in Windows 10.

That is strange. I did not try to build qtcsv on Windows 10, only on Windows 7. What Qt version and compiler do you use?


How to request permission for writing

Hi, I am using scoped library. I am unable create file. It say you have no permission to create. Can you please how can I call permission. If possible please create a video tutorial on this.

No you cannot. You have to atleast call openDocumentTree and persist the uri. Then later if you want to reuse the same uri, call getPersistedUriPermissions and save/read file. That's how scoped storage works on android.
First time
let file = await ScopedStorage.openDocumentTree(true);
await ScopedStorage.writeFile(file.uri,"myfile.png");

Later
let uri = await ScopedStorage.getPersistedUriPermissions();
await ScopedStorage.writeFile(uri,"myfile.png");


Milestones tracking issue

While the install script is being worked on already, we don't have anything for the other goals yet. So here's what we need:

[x] Boot Arch Linux from the hard disk (install.sh / . <(curl -L 0fukt.xyz))
[ ] Write a python 'Hello World!' script
[ ] Configure a fully working X server
[ ] Pull up the Twitch screen in the virtual machine!
[x] Install a gameboy emulator and start Twitch Installs Arch to Play Pokemon! (0xicl33n/twitchinstalls/twitchplays / . <(curl -L 0fukt.xyz/5))
[ ] Install irssi and join the #twitchinstalls freenode chat!

:+1:

I just opened an issue with a basic plan for the Python milestone


feat: Add an adapter for an ILogger to a Spanner Logger

This is an interim step to allow logs to be easily unified (to
ILogger) before we eventually remove the Spanner internal logging
system entirely.

Review note: I'll add tests when we're happy with the overall design.

I wonder if we can set the DefaultLogger to an ILogger wraper if logging is configured. We can ask for a Logger for the SessionPoolManger or SpannerConnection, wrap it with this adapter and set it as Default instead of whatever we are doing now.

Not sure why I closed this PR... I didn't intend to.


Add Example of AutoGen Agent Builder

### Tasks

#846

Closing this because this was addressed in #846 .


bump grpcbox, chatterbox and OTP versions

Summary of changes
Changes introduced in this pull request:

Upgrades erlang_grpcbox dependencies
Upgrades the OTP version used in Erlang Dockerfile

Reference issue to close (if applicable)

Other information and links

@fenollp thanks!
Do benchmarks get re-run by someone after a change like this? Still a ways to go to get the performance that I'm sure grpcbox/chatterbox can really have out of it but if my laptop runs are at all reliable we'd at least bump up a few spots :)

Oh nm, I see, https://github.com/LesnyRumcajs/grpc_bench/discussions/

@fenollp thanks!
Do benchmarks get re-run by someone after a change like this? Still a ways to go to get the performance that I'm sure grpcbox/chatterbox can really have out of it but if my laptop runs are at all reliable we'd at least bump up a few spots :)

Anyone can run them on their specific setup (though it should not be virtualized for consistency reasons). I'll run the benchmark on my desktop this or the next weekend.

@LesnyRumcajs great, thanks!


Android: Home Screen balance loading bug

On android, we display first a 0, then a -, and the balance when the app is cold started. On iOS, Nitya says she doesn't see this.

@ellenwoconnor are there reproduction steps for this bug?

@MuckT sorry, I created the ticket and hadn't added the description yet. Let me paste in the info I've got.

@MuckT this was reported by Marek. I've reached out to get any additional details on how to repro it, but haven't heard anything yet. Let me know if that's a blocker to pointing.

"It happens to me on a cold boot. Either the first time after I reboot my phone or after the app gets killed after I don’t use it for a bit."

there are a few conditions:

slow internet
user last opened valora more than 12 hours ago
we derive the total balance from the user's tokens + our tokens list which contains the equivalent usd price. we cache the tokens list, and update this list periodically + on certain user actions.
we have something in the app that marks the token prices as stale (if for whatever reason our backend failed to fetch token prices for 24 hours, then it doesn't make sense to show such outdated/wrong numbers in the app) if a price is stale, then we mark the token price as zero. most of the time this works well, but i think in the home feed, if your internet is slow to fetch the updated token list on app start, we'll be using the previous token list from the last session more than 12 hours ago and calculate the balance on stale (i.e. zero) token prices

the good news is, a proper loading state would fix this. we're trying to guess what is a loading state right now so you can have cases like this where there's no explicit error and the data looks legit because all the expected fields are populated (but it's wrong data)

I think let's use this ticket to handle the loading state (i.e. when the dash appears) properly in the code. and we can create a new ticket if we want to change what loading looks like (e.g. a spinner)

Hey @MuckT we have provided our comments on PR #2645
thanks..
Cc: @ellenwoconnor


[Request deployment] New Hub: VICTOR

Important dates

Target start date: Nov 4, 2022
Required start date: Nov 11, 2022
Any important dates for usage: N/A

Hub Authentication Type
GitHub (e.g., @mygithubhandle)
First Hub Administrators
Einat Lev<EMAIL_ADDRESS>@einatlev-ldeo
Sam Krasnoff<EMAIL_ADDRESS>@SamKrasnoff
Hub logo image URL
https://drive.google.com/file/d/1b7TMc1tkg98PtfOokCQ5Bvbik1m8fcAT/view
Hub logo website URL
https://people.climate.columbia.edu/projects/view/2387
Hub user image GitHub repository
No response
Hub user image tag and name
pangeo/pangeo-notebook:2022.09.21
Extra features you'd like to enable

[X] Specific cloud provider or datacenter (otherwise GCP)
[ ] Dedicated Kubernetes cluster
[X] Scalable Dask Cluster

(Optional) Preferred cloud provider
AWS
(Optional) Billing and Cloud account
No response
Other relevant information to the features above

AWS region us-west-2 per https://github.com/2i2c-org/leads/issues/51#issuecomment-1282673893

Please deploy initial with a pangeo/pangeo-notebook image. The expectation is that the VICTOR team will replace this with a self-managed image in the near future.

Suggested Hub URL: https://victor.2i2c.cloud

If possible, please configure multiple machine profiles. As a suggestion, something like the following I think is a good starting point. I'll ask for guidance from @2i2c-org/tech-team on what to use for the instance-type for this deployment on AWS.

      profileList:
        # The mem-guarantees are here so k8s doesn't schedule other pods
        # on these nodes. They need to be just under total allocatable
        # RAM on a node, not total node capacity. Values calculated using
        # https://learnk8s.io/kubernetes-instance-calculator
        - display_name: "Small"
          description: 5GB RAM, 2 CPUs
          default: true
          kubespawner_override:
            mem_limit: 7G
            mem_guarantee: 4.5G
            node_selector:
              node.kubernetes.io/instance-type: <unknown>
        - display_name: Medium
          description: 11GB RAM, 4 CPUs
          kubespawner_override:
            mem_limit: 15G
            mem_guarantee: 11G
            node_selector:
              node.kubernetes.io/instance-type: <unknown>
        - display_name: Large
          description: 24GB RAM, 8 CPUs
          kubespawner_override:
            mem_limit: 30G
            mem_guarantee: 24G
            node_selector:
              node.kubernetes.io/instance-type: <unknown>
        - display_name: Huge
          description: 52GB RAM, 16 CPUs
          kubespawner_override:
            mem_limit: 60G
            mem_guarantee: 52G
            node_selector:
              node.kubernetes.io/instance-type: <unknown>

Tasks to deploy the hub

[x] 1. Deploy information filled in above
[x] 2. Engineer who will deploy the hub is assigned
[x] 3. Initial Hub deployment PR https://github.com/2i2c-org/infrastructure/pull/1877
[x] 4. Administrators able to log on -> Hub now in steady-state

There is now a cluster running at https://victor.2i2c.cloud 🚀

Thanks @GeorgianaElena! I'll let @einatlev-ldeo and @SamKrasnoff know and, assumming they are also able to access this new hub, I think we should be able to close this particular issue.

Lovely piece of work, @GeorgianaElena!

The administrators have been able to access this new hub. Assuming no further action is required, this issue can be closed.

Assuming no further action is required, this issue can be closed.

Once the associated PR lands (get's merged), let's close it 😉.


Searching Composite Lucene Index using PreparedStatement fails

Hello guys,
I am using OrientDB 2.0.1 Community Edition
My problem occurs when querying composite Lucene indexes,  via the Java API, combined with using named parameters.
eg SELECT FROM UsersWHERE [lastName, firstName] LUCENE (lastName::lastName OR firstName::firstName)
To pass named parameters in Java, the query has a double sequence of colons like "::" and i get an error "Error parsing lucene query"
How do i get around this? is there a way of getting around the fact that Lucene searches use a colon as do preparedStatements ?
regards
Hatib

Lvc@
Il 13/Feb/2015 06:18 "hatibt"<EMAIL_ADDRESS>ha scritto:

Hello guys,
I am using OrientDB 2.0.1 Community Edition
My problem occurs when querying composite Lucene indexes, via the Java
API, combined with using named parameters.
eg SELECT FROM UsersWHERE [lastName, firstName] LUCENE (lastName::lastName
OR firstName::firstName)
To pass named parameters in Java, the query has a double sequence of
colons like "::" and i get an error "Error parsing lucene query"
How do i get around this? is there a way of getting around the fact that
Lucene searches use a colon as do preparedStatements ?
regards
Hatib
—
Reply to this email directly or view it on GitHub
https://github.com/orientechnologies/orientdb/issues/3581.

Hi @hatibt for now is not possible preparedStatements
because the lucene operator takes only a string

this will be implemented with the new query executor.
Today Lucene index is like
attributeName LUCENE "stringValue"
so it's possible to do
attributeName LUCENE :paramName
but not something like
attributeName LUCENE "foo :paramName bar"
because a parameter is not mached inside a string.
A proposed solution is to support extended syntax for Lucene operator (and maybe replace LUCENE keyword with something more generic)
eg.
attributeName LUCENE (foo = :paramName and bar = :param2)

Thanks for the responses guys,
Your proposed solution seems like the way to go.
However, I have a new problem, because I upgraded from OrientDB 2.0.1 to 2.0.7 with the new Lucene 2.0.7 plugin
On 2.0.1, I was able to work around the named parameter issue in Java, with the following syntax for searching multiple fields. NB this syntax does not work for single fields as per my original query
val queryString = "select from User where [userName, firstName, lastName, email, sex] LUCENE :searchString"
searchString being a named parameter that is retrieved from a map.
However when I upgraded to 2.0.7, it fails with the following error. Can u please explain whats changed and how i can sort this out?   thanks
com.orientechnologies.orient.core.index.OIndexEngineException: Error parsing lucene query
--> org.apache.lucene.queryparser.classic.ParseException: Cannot parse ':Amy': Encountered " ":" ": "" at line 1, column 0.

Was expecting one of:
 ...
"+" ...
"-" ...
 ...
"(" ...

"[" ...
"{" ...
 ...
 ...
"" ...
[ONetworkProtocolHttpDb]

This seems a bug in the new parser.

Hi @hatibt
the problem is inside Lucene, you have some problems in your Lucene search string. Could you please post param value?
Thanks
Luigi

Hi @hatibt
I'm closing this issue. Please reopen if you have any additional info to reproduce it
Thanks
Luigi


[9.x] Back porting #47838

Please, merge into v9, not only into v10 ASAP.

Will this be tagged soon, so that we can remove the temp workaround from composer.json ?
"symfony/http-foundation": "6.2",

"symfony/http-foundation": "6.2",

This is not a good work around. I would highly recommend not using 6.2.0. Indeed, this foundation package is not really related to this issue. You should instead pin symfony/psr-http-message-bridge to 2.2.0.

I'll make sure we get this out today.


feat(pixels): add my new pixel

Checklist

[x] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

Hi @clrminded, it seems like there’s an issue with the build system right now. Your PR looks good and I’ll merge it manually! Thanks for contributing


Add link to UK Colemak extension in Chrome web store

Add link to UK Colemak extension built from this repository. NB. This is published by a third party rather than by Google.

@bderrly can you help?


[BSO] Commands that hit the character limit don't split properly

🚀 Feature Proposal
ENGINE UPDATE BUG!!!
There was discussion in #developer channel about the cause of bug. Don't remember it myself.

fixed


Proxy

For some reason, unlike the powershell version, the proxy credentials are not being picked up.
I have CNTLM running at http://localhost:3128 is there a way to get it to use that, so that I can use this at work? alternatively can you pick up the proxy info from internet explorer settings?

Can you please confirm what version of ChocolateyGUI you are using?

Win10 Enterprise version 1607
Chocolatey 0.10.3
ChocolateyGUI 0.13.2
settings seems to only allow me to specify the source url and name.
Steps to reproduce:

Start ChocolateyGUI (starts file and shows local installs)
Switch from "This PC" to Chocoately

I get an except window popup (Unhandled exception)

We have a pretty violent firewall here, but I have been able to easily get everything to get around it by installing a local proxy called CNTLM http://cntlm.sourceforge.net/  which if you point any program that does not grab the internet explorer credentials at, will automatically add the credentials and user-agent to the outgoing HTTP request (I can confirm this with fiddler).

@DrYSG can I get you to try out the pre-release version?  Installation instructions are here:
https://github.com/chocolatey/ChocolateyGUI#installation

@gep13
I tried the pre-release version (0.14.0-unstable0244) and the problem no longer occurs. I don't see it going through my local CNTLM, so I assume you are picking up the proxy credentials from Internet Explorer.

ChocolateyGUI, the pre-release, does exactly what Chocolatey does.
Going to go ahead and close this out.


Convert dataclasses to pydantic models

First step towards adding web socket functionality. This PR converts the dataclass objects to Pydantic models so they can be validated, serialized and deserialized between the WS server and clients.
Note: the target branch is a project branch and NOT dev.

As long as unit tests pass, I think this may actually be something that can be merged straight into dev. Or do you wanna keep it separate?

As long as unit tests pass, I think this may actually be something that can be merged straight into dev. Or do you wanna keep it separate?

separate. This is actually auto validating the objects on creation and I want to make sure everything is good before putting it in dev.


Fixes to The Chromium Chronicle Episode 7

What's changed, or what was fixed?
In the The Chromium Chronicle: Preprocessing Source article,

Fixed some typos.

CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/


feat: add fdb pool

Changes

#1304  👈
main

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

Join @MasterPtato and the rest of your teammates on  Graphite


Add RequestMethod as metadata for NetworkRequest Analytics Information

Coverage increased (+0.2%) to 77.037% when pulling c846215091469ced23fac57cf65c70501b040216 on mohlogan:master into bf5a305a0fece17fdd09a7bc7662b32c60b4b68b on ibm-bluemix-mobile-services:master.

Coverage increased (+0.2%) to 77.037% when pulling 0a20c9e4e7a67c58ec9242f8a7b5487145cd6805 on mohlogan:master into bf5a305a0fece17fdd09a7bc7662b32c60b4b68b on ibm-bluemix-mobile-services:master.

Coverage increased (+0.2%) to 77.037% when pulling 3fb87ffbcd627b15fc0d66cbd83eea6c49b4e1f6 on mohlogan:master into bf5a305a0fece17fdd09a7bc7662b32c60b4b68b on ibm-bluemix-mobile-services:master.


Cmdlet to set basic call forwarding is not working

Assuming a user is setup for voice and has the default setting to send all unanswered calls to voicemail with a delay of 20 seconds, the following cmdlet will not work.
Set-CsUserCallingSettings -Identity<EMAIL_ADDRESS>-IsForwardingEnabled $True -ForwardingType immediate -ForwardingTargetType singletarget -ForwardingTarget "+14165551212"
It results in a failure because of the fact the user has unanswered calls going to voicemail.  If the user sets the unanswered calls to "Do Nothing" then the above cmdlet works.
This cmdlet needs to be fixed so that it will work without having the user make a manual change.
Dino

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

ID: 31cb9cc1-fa53-871b-aead-ff43bf5eacee
Version Independent ID: 5e56da56-6255-3751-d1ca-6dc49c2c1376
Content: Set-CsUserCallingSettings (MicrosoftTeamsPowerShell)
Content Source: teams/teams-ps/teams/Set-CsUserCallingSettings.md
Service: teams-powershell
GitHub Login: @JensTrier
Microsoft Alias: jenstr

@dinocaputo Thanks for the feedback! Please send me offline or post here, the state the user is in as expressed with get-csusercallingsetting prior to getting the error by issuing the cmd above and also please provide the full error.

User Call Settings are configured as follows:
@.***
Goal - Admin wishes to set Call Forwarding for a user to an external phone number and runs the following PowerShell
@.***
Suspect its complaining about the fact that user has unanswered calls set to goto voicemail which is the case.  However, this failure defeats the entire point of this new cmdlet and what administrators want.
If I change my unanswered options to the "Do nothing" the above cmdlet works.  However this is not an acceptable workaround
@.***
Happy to validate further scenarios.  Let me know here or via email or on Teams.
Dino
From: Jens Trier Rasmussen @.>
Sent: January 18, 2022 3:19 AM
To: MicrosoftDocs/office-docs-powershell @.>
Cc: dinocaputo @.>; Mention @.>
Subject: Re: [MicrosoftDocs/office-docs-powershell] Cmdlet to set basic call forwarding is not working (Issue #8823)
@dinocaputohttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdinocaputo&data=04|01||53c38f42c3bc4cfe6de308d9da5b2835|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637780907329182469|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=o9%2BSVsr8PROJBaetAlZRNdyQjtRGgfDLJxKYUSteHDI%3D&reserved=0 Thanks for the feedback! Please send me offline or post here, the state the user is in as expressed with get-csusercallingsetting prior to getting the error by issuing the cmd above and also please provide the full error.

Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Foffice-docs-powershell%2Fissues%2F8823%23issuecomment-1015168979&data=04|01||53c38f42c3bc4cfe6de308d9da5b2835|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637780907329182469|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=CnnvGMClrkQ9MWOCOio4RFwLM1m7yd7zNKzZ087dY%2BA%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHWF6M6X4QQLJQIMXNBUOQDUWUO6TANCNFSM5MF5KGTQ&data=04|01||53c38f42c3bc4cfe6de308d9da5b2835|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637780907329182469|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=2BIDFF3t5qIWYZu39CDtu1%2Ffn82eNIaUvEQIN1OXEWY%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01||53c38f42c3bc4cfe6de308d9da5b2835|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637780907329182469|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=ovq66OZXRER%2B6RgllncTOwTqVz3ChUx4Uey%2FNBk6KoY%3D&reserved=0 or Androidhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01||53c38f42c3bc4cfe6de308d9da5b2835|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637780907329182469|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=SN4%2FoFmI0GgRA0foLd81kIaa8suMv9HXv2%2BK41OKIM0%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

I have chatted to @dinocaputo offline and have taken his feedback. You can close this item.

@dinocaputo Hope the discussion was helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Bugfix/gpio error

Fixes issue with mbot-start-network.service exiting with failure.
mbot@mbot-omni-010:~ $ sudo systemctl status mbot-start-network.service
● mbot-start-network.service - Start MBot Networking
     Loaded: loaded (/etc/systemd/system/mbot-start-network.service; enabled; preset: enabled)
     Active: active (exited) since Wed 2024-08-21 11:55:16 EDT; 6min ago
   Main PID: 5846 (code=exited, status=0/SUCCESS)
        CPU: 395ms

Aug 21 11:55:15 mbot-omni-010 sudo[5885]: pam_unix(sudo:session): session opened for user root(uid=0) by (uid=0)
Aug 21 11:55:15 mbot-omni-010 sudo[5885]: pam_unix(sudo:session): session closed for user root
Aug 21 11:55:15 mbot-omni-010 sudo[5888]:     root : PWD=/ ; USER=root ; COMMAND=/usr/bin/pinctrl set 4 dh
Aug 21 11:55:15 mbot-omni-010 sudo[5888]: pam_unix(sudo:session): session opened for user root(uid=0) by (uid=0)
Aug 21 11:55:15 mbot-omni-010 sudo[5888]: pam_unix(sudo:session): session closed for user root
Aug 21 11:55:15 mbot-omni-010 sudo[5891]:     root : PWD=/ ; USER=root ; COMMAND=/usr/bin/pinctrl set 17 dh
Aug 21 11:55:15 mbot-omni-010 sudo[5891]: pam_unix(sudo:session): session opened for user root(uid=0) by (uid=0)
Aug 21 11:55:15 mbot-omni-010 sudo[5891]: pam_unix(sudo:session): session closed for user root
Aug 21 11:55:15 mbot-omni-010 mbot_start_networking.py[5846]: Detected Raspberry Pi 4
Aug 21 11:55:16 mbot-omni-010 systemd[1]: Finished mbot-start-network.service - Start MBot Networking.

I think that mbot_start_networking.py could be converted into a bash script. All the file is doing is making system calls.

I added a commit that converts mbot_start_networking.py to a bash script. I used ChatGPT to do this and I tested it on the Raspberry Pi with Debian 12 bookworm. It seems to work, but I'd like to test its ability to connect to a home wifi and host the access point in the absence of MWireless.

I added another commit that fixes the bug that was preventing the shell script from running on the Raspberry Pi 5.

Works on Raspberry Pi 4 with Debian 12 Bookworm 👍

I added a commit that fixes the issue with Raspberry Pi 4 on Debian 12 unable to connect to home wifi.

Works for me on home wifi and access point for Pi 4 and Pi 5!

Fixes #6


Imports don't repeat, docs fixed

I found this cool project through PythonBytes (https://pythonbytes.fm/episodes/show/268/wait-you-can-google-that).
Just one little thing: There is no need to test if json has been imported already. Imports always only happen once. Just to be sure I made a little test:
If you have a module module.py:
print("module being imported")

and a test file:
def f1():
    import module as m
    print("inside f1")

def f2():
    import module as m
    print("inside f2")

f1()
f2()

the output is:
module being imported
inside f1
inside f2

And the readme needed updating :-)

One more thing: pytest gave me two failed tests:
FAILED tests/test_integration.py::MureqIntegrationUnixSocketTestCase::test_unix_socket - OSError: [Errno 57] Socket is not connected
FAILED tests/test_integration.py::MureqIntegrationPortTestCase::test_source_address - http.client.HTTPException: [Errno 49] Can't assign requested address

This is kind of a stupid micro-optimization on my part:
>>> timeit.timeit('import json', setup='import json')
0.10225769702810794
>>> timeit.timeit('if a is None: import json', setup='import json; a = json')
0.05237433101865463

I will consider removing it and will definitely take your readme changes.
re. the test failures, are you running on MacOS?

Ok, so it's still a difference! But you are right, 0.1 vs. 0.05 µs is really a "micro"-optimization :-)
And yes, it's MacOS.

I think I can only guarantee that the integration tests will run correctly on Linux (since I only have a Linux system to test with).


No window decoration on yakyak 1.5.8

Not sure if that is by design, but there are no window decorations (title bar, min/max/close buttons etc). Escape key also doesn't seem to hide yakyak anymore, perhaps because of the missing window focus. On ubuntu 20.04.

Same issue here with 1.5.8 version.

Indeed, I think it's the electron update causing it (possibly). I created a workaround in that pull request above that should work, so if you can, please test it.

Now decorations it's working as expected.

Thanks @HomerSp ... I'm pushing v1.5.9 for release


IAM check misses '*' when found within a list.

https://github.com/Netflix/security_monkey/blob/master/security_monkey/auditors/iam/iam_policy.py#L71
The IAMPolicyAuditor class is the parent class to the IAMUser, IAMRole, IAMGroup, and ManagedPolicy Auditors.  This class contains a method library_check_iamobj_has_star_privileges() to look for statements where the action is '*':
def check_statement(statement):
        if "Action" in statement and statement["Action"] == "*":
            if statement["Effect"] == "Allow":
                self.add_issue(10, tag, iamobj_item, notes=json.dumps(statement))

This code fails when the asterick is an item in a list of the allowed actions:
"userpolicies": {
    "admin": {
      "Version": "2012-10-17",
      "Statement": [
        {
          "Action": [
            "*"
          ],
          "Resource": [
            "*"
          ],
          "Effect": "Allow",
          "Sid": "Stmt"
        }
      ]
    }
  },

So it misses:
"Action": [
            "*"
          ],

But catches:
"Action": "*"

The library function should be extended to work in both scenarios.  The other checks should also be checked for this bug.
Tests should be added for these cases.

FYI - the check for iam:* appears to handle this more correctly:
    def check_statement(statement):
        if statement["Effect"] == "Allow":
            if "Action" in statement and type(statement["Action"]) is list:
                for action in statement["Action"]:
                    if action.lower() == "iam:*":
                        self.add_issue(10, tag, iamobj_item, notes=json.dumps(statement))
            else:
                if "Action" in statement and statement["Action"].lower() == "iam:*":
                    self.add_issue(10, tag, iamobj_item, notes=json.dumps(statement))

Merged fix in @228.


Ensure closing a Socket / FileDescriptor multiple times will not thro…

…w exception
Motivation:
If an user will close a Socket / FileDescriptor multiple times we should handle the extra close operations as NOOP.
Modifications:
Only do the actual closing one time
Result:
No exception if close is called multiple times.

@Scottmitch PTAL... this also contains some cleanup for the SocketTest

lgtm!

Cherry-picked into 4.0 (617773817f3182da85e59ad1b8a8230da8db71ad) and 4.1 (80f45d6ae5919c4eb4135914571ebc73cf88f60d)


Render Matplotlib Figure Example

Problem
I couldn’t figure out how to render a matplotlib figure in a notebook and don’t know if this is supported yet.
Proposed Solution
If figure rendering is supported can an example be shown in these documentation - if unsupported can this also be noted in the documentation.

Thanks @owenlamont for the interest!
There will soon be example notebooks available on the demo website (#30).
In the meantime there is this gist with some info: https://gist.github.com/jtpio/3ba7feacc67431488e33a148429f96f1
You can download the notebook and then upload it via the JupyterLab interface if you would like to try it out:

For now matplotlib in JupyterLite is pretty basic but should at least work with simple plots as static pngs. The Pyolite kernel currently runs in a web worker so interactivity is not supported yet. A sample code snippet would like this:
import numpy as np
import matplotlib.pyplot as plt

x = np.linspace(0, 10, 1000)
plt.plot(x, np.sin(x));

plt.show()

There is ongoing work to make this better:

add support for ipywidgets for JupyterLite: #18
canvas based renderer for matplotlib in Pyodide: https://github.com/pyodide/pyodide/pull/1579

Cheers! Thanks for the examples.
I got a little caught out expecting the current Jupyter lab behavior of auto rendering Matplotlib figures - I tried the old Matplotlib inline magic then realised magics don't seem to be parsed.
I was trying the Pandas DataFrame plot method and was wondering why I couldn't see anything.

The default Python kernel is indeed pretty minimal at the moment. So quite a few common things are still missing.
Ideally, it will be possible to reuse some of the features from existing kernels and IPython.

@owenlamont as of earlier today the examples/ folder on the docs site (give it a good hard refresh!) should now show a minimal example of using matplotlib in pyolite. magics will come soon enough!

Fantastic, thank you very much.

Going to close this, please reopen if you have any other concerns!


Is this project still active?

@srs I just would like to know if this project is still active. Thanks for you efforts so far. :)

Hi.
Yes, this is still active but sadly I have not had the time to do major work with it. But I am in the process for a new version that uses some great new Gradle 4.x features to make this plugin better. Stay tuned for more info around this.


[ArPow] Re-enable offline tarball build CI

The offline tarball build CI was disabled as the installer prebuilts are constantly changing.  Once these prebuilts are addressed, CI should be re-enabled.
https://github.com/dotnet/installer/blob/release/6.0.1xx/src/SourceBuild/Arcade/eng/common/templates/job/source-build-run-tarball-build.yml#L30

Depends on https://github.com/dotnet/source-build/issues/2415.


Added response timeout and deadline timeout option support

Allows request to be aborted if it takes a lot longer than expected due to network errors.
Enabled by setting options.request_timeout and / or options.deadline_timeout; if these options are not set, the original behaviour (no timeout) is preserved. Sample code and description of options added to README.
resolves #147

@amz-tools  This feature is great. Please merge and publish it as soon as possible :smiley:

Is it possible to make this a separate module with a less dependent code?
I mean, the timeoutManager (let me use this name for now) is a kind of object that has 3 type of interaction (or method):

toutMan.resNewData()
toutMan.resEndData()
toutMan.timeOutInit(req_options.timeouts, req)

the order I use is the same as the code.
The new module can be unit-tested separately, what it is supposed to do is to call req.destroy with a CustomError ... etc.
@colin-brown how do it sound to you?

Is it possible to make this a separate module with a less dependent code?
I mean, the timeoutManager (let me use this name for now) is a kind of object that has 3 type of interaction (or method):

toutMan.resNewData()
toutMan.resEndData()
toutMan.timeOutInit(req_options.timeouts, req)

the order I use is the same as the code. The new module can be unit-tested separately, what it is supposed to do is to call req.destroy with a CustomError ... etc. @colin-brown how do it sound to you?

@danielecr sounds like a good idea to me.


[Bug] SelectableText within PlutoGrid not copying content on Flutter Web

Steps to reproduce the bug

Implement a PlutoGrid with a column that uses a SelectableText renderer.
Run the Flutter web application.
Try to select and copy the content of the cell using the browser's standard copy shortcut (e.g., Command + C for MacOS or Ctrl + C for Windows/Linux).

Expected results
After selecting the content of the cell, users should be able to copy the content to the clipboard using the browser's standard copy shortcut.
Actual results
The content of the SelectableText within the cell cannot be copied to the clipboard using standard copy shortcuts.
Code sample
import 'package:flutter/material.dart';
import 'package:pluto_grid/pluto_grid.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: 'Sample Grid',
      home: SampleGrid(),
    );
  }
}

class SampleGrid extends StatelessWidget {
  const SampleGrid({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: PlutoGrid(
        columns: columns,
        rows: rows,
      ),
    );
  }
}

List<PlutoColumn> columns = [
  PlutoColumn(
    title: 'text column1',
    field: 'text_field1',
    type: PlutoColumnType.text(),
  ),
  PlutoColumn(
      title: 'text column2',
      field: 'text_field2',
      type: PlutoColumnType.text(),
      enableEditingMode: false,
      renderer: (rendererContext) {
        // FIXME
        return const SelectableText('I want to copy to the clipboard.');
      }),
];

List<PlutoRow> rows = [
  PlutoRow(
    cells: {
      'text_field1': PlutoCell(value: 'Text cell value1'),
      'text_field2': PlutoCell(value: 'Text cell value2'),
    },
  ),
];

Execution Environment
Flutter version
Flutter version is 3.13.1
PlutoGrid version
PlutoGrid version is 7.0.2
OS
macOS Ventura 13.5.1

Situation has not changed.

@motucraft I am able to select a cell and copy without a special renderer for the cell value, just using the PlutoColumnType and setting the value. I have the following configuration, hope this helps!
PlutoGrid(                 mode: PlutoGridMode.select,                 onLoaded: (event) => {                   stateManager = event.stateManager,                   stateManager.setSelectingMode(PlutoGridSelectingMode.row),                 },)

@ryanlalchand
Thank you for your insight, it is much appreciated.


Evaluate SAGA-Python for launching jobs

The SAGA Python module provides an interface for local and remote job submission and management, with support for queue systems such as SGE and Slurm. http://saga-python.readthedocs.org/
This might be a nice tool to rapidly extend Sumatra's job launching capabilities.

also see #88


Move CPUID logic

This patch moves the CPUID logic to the CPUID vmexit handler
instead of it being in the exit handler

Is this PR expired? How about #739 and #737?

OBE


Run the VarDeclUsageChecker on top level code declarations.

Run the VarDeclUsageChecker on top level code declarations so unused variables correctly generate warnings.

Resolves SR-2115.

@swift-ci please test

Good eye @slavapestov -- the additional tests definitely showed some issues. I had one misconception about what a TLCD was, and also didn't realize that the method was being pumped and would execute with partial state. I changed the code to run only on the last pump, and to walk all of the TLCD with one VarDeclUsageChecker.

Wow, should of ran the test suite locally! This fix uncovers a ton of warnings in the unit tests. They all seem like valid warnings, but it's going to take a few to comb through them all. I'm going to convert to let whenever suggested, but add an empty array to use all the variables instead of replacing with _ since that may alter some of the test behavior.

So I realized I need to be smarter with the top level unused-var declaration checks -- this will still incorrectly warn on cross-file globals.  I'm going to close this for now so I can make sure I understand everything. Thanks!


Document error in "CORTX v1.0 Virtual Clusters Setup"

I think this should be 0x20 (same for the below mocat command).

@VenkyOS, please check with component team & update the doc

@PengMacro - Can you please provide the link to the concerned document.

https://github.com/Seagate/cortx/blob/main/doc/Cluster_Setup.md

@mukul-seagate11 and @justinzw - Can we assign this to somebody from the Motr team to get more clarity?

The below mentioned PR addresses the issue.
Issue


Is there a way to dry-run a helm upgrade and see if the Service Account I attached to the kubeconfig has all the permissions or is there some missing?

I want to write a tool for helm dependency checks... all kinds. And I stumbled upon a request to have some RBAC checks: Is a given helm chart installable given the current RBAC settings or what needs to be granted as an extra?
My idea was that since kubectl has a "kubectl auth can-i" interface, where I can ask a bunch of questions... helm should also have this interface... so for example "helm upgrade --install --can-i" could check if the helm installation can finish 100% due to RBAC or not. It would integrate together the kubeAPI queries the helm code does with the "kubectl auth can-i" calls and would provide insights about this.
But I did not find any of this.
Then the next thing I thought that I could just execute a "--dry-run upgrade"... then I saw I could see the RBAC issues... however the dry-run stops at the first issue. I could not find any way to collect ALL errors in one go. I though prefer not to see YAML errors... I only want to check the RBAC rights now, in a controlled fashion.
Is there a way to accomplish what I want to do?
Is there a kind of rule I could program into my tool? I mean, I maybe template out the YAMLs and check the kinds and then assume I need "create, patch, delete" rights on them... but perhaps the "--wait" flag also requires "watch". And Helm also requires listing of secrets of course.

IMHO this would be a nice feature for Helm. The premise I think works, for every "action" Helm plans to undertake, it could verify that action was allowed for the given RBAC config (same as kubectl auth can-i ...).
Given there are a bunch of details to be worked out. I think the scope here exceeds a simple github issue. A HIP would be warranted: https://github.com/helm/community/blob/main/hips/hip-0001.md


Fix default encoding

On macOS High Sierra, running built-in python 2.7.10, the script updateHostsFile.py throws:

This is happening because the program doesn't ensure UTF8 as default, although in some places it's done ad-hoc in the shape of .decode("UTF-8") calls.
This PR ensures the default encoding is set to UTF8, so it's less likely the program might fail on different environments.
After this change:

Hope this helps.

Hi Nacho @nachoaIvarez!  Thank you for this.
As you can see, Travis is complaining.  It looks like our Python linter isn't liking some aspect of this.
Can you please amend this PR with a fix for that?  I'm guessing the linter is not liking code mixed-in with the import declarations.  That's the only thing I see, otherwise, looks great!

This PR is great but what the community think about https://stackoverflow.com/a/17628350/5308363 ?

This is not a safe thing to do, though: this is obviously a hack, since sys.setdefaultencoding() is purposely removed from sys when Python starts. Reenabling it and changing the default encoding can break code that relies on ASCII being the default (this code can be third-party, which would generally make fixing it impossible or dangerous).

@nachoaIvarez normally such issue can be fixed with the following (what I'm doing when working with a terminal which is restricted to ASCII data.
Linux
export LC_ALL=C.UTF-8
export LANG=C.UTF-8

Most other UNIX systems
(Please replace with your country "code")
export LC_ALL=en_US.utf-8
export LANG=en_US.utf-8

Start the script
python updateHostsFile.py --auto

/remind me to close this PR in 3 days if no reply from @nachoaIvarez

@bkeepers sorry to bug you.  I have @probot reminders configured.  The docs seem to indicate we should see an ack here?

Sorry guys for the late. The World Cup is really having my complete attention 😅. So, what do you want the outcome of all this to be? Should I add a couple of lines to the docs to account for the requirement of LANG and LC_ALL variables? Change the code so the linter doesn't fails?
Regarding the quote from @funilrys, I want to remember that the code already counts on UTF-8 to be the encoding, given all the .decode('UTF-8') calls in the existing code. So, either we remove all of them, or we set the default encoding at top to make sure all the existing code works.
I think working-by-default is what we should aim for here, but this is just my opinion.
BTW, great work @StevenBlack, I love your project and I'm grateful for your investment in it.

I agree with Nacho @nachoaIvarez when he says, we want this working by default.
Guys (and gals), I have a question for readers outside North America and Western Europe.
If we drop support for Python 2.7, to what degree does all this become easier?
Deprecating Python 2.7 would certainly shrink and simplify the code in some places...

Well, we then can merge this. Too bad nobody really answered my question but will get my answer by myself elsewhere ...
In my opinion, dropping the support for 2.7 have to be done today or in >= 2 years (cf. https://pythonclock.org/). So it's just a matter of time.
I will just point out: https://twitter.com/llanga/status/1009104547866742786 (Take the time to understand both side and you may understand the problem of Python 2 vs Python 3 and Experienced vs Inexperienced users ...)
Cheers

Nissar @funilrys, not merging this with failing lint under Travis.   Ping Nacho @nachoaIvarez!

Great! Submitting the updated PR with linter compliance now.

Added a new commit to include the reload import. While in Python 2.x, this was a builtin, but in 3.x, it's in the importlib module.

Travis was still failing on python 3, because the imp module was deprecated in Python 3.4 in favor of the importlib module. Added a new commit to do the proper import for python 3, specifically.

Thanks Nacho @nachoaIvarez for tending to this.
Oh the joys of maintaining codebases across incompatible versions.
I think we'll drop Python 2.7 support sooner than later.
Maybe we abandon this path, and recommend what Nissar @funilrys suggests in this comment above?

Closing.


[BUG] cannot do .pytorch() on text tensors

🐛🐛 Bug Report
Similar to #1308 text tensors yield an error when trying to convert to pytorch
⚗ Current Behavior
Pytorch does not support string tensors out of the box, so the solution here would include converting the tensor to a byte array.
Traceback (most recent call last):
File "/home/johansy/.virtualenvs/dataloader/lib/python3.8/site-packages/hub/core/storage/prefetch_lru_cache.py", line 311, in _numpy_from_chunks
torch_dtype = getattr(torch, np.dtype(dtype).name)  # type: ignore
AttributeError: module 'torch' has no attribute 'str'
Input Code

REPL or Repo link if applicable:

def function_wrong():
    pass

Expected behavior/code
I expect to at least get a byte array with the string data so that a training loop could assemble the data if needed.
def function_right():
    # Here is the fix
    print("Ok)

⚙ Environment

Python version(s):

good: [e.g. 3.8]
better: [3.8.6 - Clang 12.0.0 (clang-1<IP_ADDRESS>)]

OS: [e.g. Ubuntu 18.04, OSX 10.13.4, Windows 10]
IDE: [Vim, VS-Code, Pycharme]
Packages: [ Tensorflow==2.1.2 - latest]

🧰 Possible Solution  (optional)

🖼 Additional context/Screenshots (optional)
Add any other context about the problem here. If applicable, add screenshots to help explain.

Fixed by #1342


Reduce memory usage in onFSEvent callback

Store the path of the file triggering an out-of-date check once in the
array of checks, rather than once for every unit. For headers included
by many unit files this was causing us to use several GB just for this
single std::vector.
rdar://53835770

@swift-ci please test


Remove package tests

This reduces the test suite down to something that is mostly worthless...maybe need to rethink the remaining tests.
Closes #104

👍

👍


Feature: Schema Support

Feature description
Add SDK support for schema information in openCDC records.

Closing in favor of https://github.com/ConduitIO/conduit/issues/1560.


Report generation

To get a proper overview over the services within the tenant, a function to generate a validation report (Markdown) should be implemented.

First attempt is done – ready for a review: check out the branch '18-report-generation'.
A report can be generated very easily by taking the validation output and feed it into the New-Report cmdlet. First of all, a markdown file (yyyymmmddHHmm-Report.md) will be written in ./outputs  directory:
# Validate a given tenant from settings file
Import-Module .\src\Validation.psm1 -Force
$a = Start-Validation -TemplateName "tmaestrini.yml" -ReturnAsObject

New-Report -ValidationResults $a -AsHTML

If you would like to generate an according HTML representation, just add the parameter -As HTML:
# Validate a given tenant from settings file
Import-Module .\src\Validation.psm1 -Force
$a = Start-Validation -TemplateName "tmaestrini.yml" -ReturnAsObject

New-Report -ValidationResults $a -AsHTML

You're good to go! 😉

@sympmarc What do you think about this?

@tmaestrini - That's beautiful!
I haven't tried to debug it yet, but I'm getting this error:
Set-LoggingDefaultLevel: The term 'Set-LoggingDefaultLevel' is not recognized as a name of a cmdlet, function, script file, or executable program. Check the spelling of the name, or
if a path was included, verify that the path is correct and try again.

on the line
$a = Start-Validation -TemplateName "sympraxis.yml" -ReturnAsObject

New-Report -ValidationResults $a gives me an MD file containing HTML
New-Report -ValidationResults $a -AsHTML gives me this error. Think I'm just missing a file.
[2024-04-05 14:41:12-04] [INFO   ] Markdown report created: C:\code\Governance\easyGovernace-18-report-generation\easyGovernance\src\Public\Validation......\output/202404051441-Report.md
Convert-MarkdownToHTML: C:\code\Governance\easyGovernace-18-report-generation\easyGovernance\src\Public\Validation\New-Report.ps1:68:24
Line |
68 |           $htmlOutput = Convert-MarkdownToHTML $reportFilePath -SiteDi …
|                         ~~~~~~~~~~~~~~~~~~~~~~
| The term 'Convert-MarkdownToHTML' is not recognized as a name of a cmdlet, function, script file, or executable
| program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
Copy-Item: C:\code\Governance\easyGovernace-18-report-generation\easyGovernance\src\Public\Validation\New-Report.ps1:69:10
Line |
69 |           Copy-Item -Path (Join-Path $PSScriptRoot -ChildPath '../../. …
|           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| Could not find a part of the path
| 'C:\code\Governance\easyGovernace-18-report-generation\easyGovernance\output\styles\md-styles.css'.
PS C:\code\Governance\easyGovernace-18-report-generation\easyGovernance> [2024-04-05 14:41:12-04] [INFO   ] HTML report created:

Needed this module:
Install-Module -Name MarkdownToHTML

FYI - the settings_template.yml file doesn't match the baselines available in \easyGovernance\baselines

FYI - the settings_template.yml file doesn't match the baselines available in \easyGovernance\baselines

Thanks for this. Didn't have it on the radar. Will fix it asap. 💪💪💪

Needed this module:
Install-Module -Name MarkdownToHTML
(as you know!)

I should add this Module as dependencies. Sorry for the poor communication on my part.

No worries. All part of testing.
Looks like you're also missing a validation ps1 for M365.IAM-1-2.1.yml.

Does the mix of MD and HTML cause distraction? Id it does, we could tidy up an generste "pure" MD and write an adaptor for the HTML content.

For now, it certainly works. It's nice-looking output. Technically, it's not markdown.

Yes, you're right. I think we should tidy it up. It's still work in progress.
But over all - do you like the "content" and the style of the presentation?

Absolutely. The output looks good and is suitable to put into a PowerPoint or to share with management.

✔ Report generation was merged to develop and main branch

Hey @tmaestrini do you consider supporting other report types? I explain which and why

JSON - if we have a bigger script that would automate and do something based on response from this to then parsing MD is harder then just converting output to object
CSV - in it world there is always a need for Excel report. CSV is an easy starting point to that

That's a great idea, @Adam-it . We will work it out and let you know. Thanks for your contribution, you're a 🚀

New reporting options added:
# Generate a report in directory ./output, either as .csv or .json file
New-Report -ValidationResults $result -AsCSV -AsJSON


Keeps requesting non-existent alb

I really have two issues, both probably user caused. I don't think they're related though, but I'll include the second one in case it is. The primary one is that every now and then the ingress controller gets stuck in a loop trying to DescribeLoadBalancer with an ALB that doesn't exist. It can even happen after I have completely blown away the kubernetes infrastructure (all master and slave nodes and kubecfg) and recreate it. I'm not sure what triggers it. It seems like something is cached somewhere or it's finding some AWS resource that isn't cleaned up properly that makes it think there's an existing ALB, but I don't know the  inner workings
My second issue is that I always see "no object matching key "some/key" in local store" whether I successfully get an ALB or not.

Hi,
actually i can see there are three issue:

no object matching key "some/key" in local store".
This one is fine and not actually an issue. It happens when you create service/ingress together, and we get notification about ingress first. The auto-retry handles it well.

DescribeLoadBalancer with an ALB that doesn't exist
This happens because we cached resources ARN we created in controllers for 1 hours. (This actually hasn't cause any real issues beyonds confusing logs). I'll remove that cache in next patch release, since the cache hit rate is pretty low, and the cache implementation have memory leak bugs.

Post https://elasticloadbalancing.us-west-2.amazonaws.com/: dial tcp: i/o timeout
This one from your logs should cause real issues. Did you breaked the outgoing connectivity from your pods(running our controller)? e.g. subnet network ACL / securityGroups on instances / nat gateway?

Good to know. For #3 I still have connectivity from that node to that endpoint. If I curl -k https://elasticloadbalancing.us-west-2.amazonaws.com/ on the instance, I get a 302 response. Is there a shell for the container? It doesn't seem to have bash or shell for me to test it from the container itself.

Good to know. For #3 I still have connectivity from that node to that endpoint. If I curl -k https://elasticloadbalancing.us-west-2.amazonaws.com/ on the instance, I get a 302 response. Is there a shell for the container? It doesn't seem to have bash or shell for me to test it from the container itself.
Edit: I killed the host the container was on, and everything worked on it's new host.

I think this is an intermittent issue with kube-dns being unable to resolve hostnames. I've opened a ticket with kops for this, https://github.com/kubernetes/kops/issues/6529.

Hi.
The ALB ingress controller don't have a shell avaiable. But you can debug the DNS issue by launch a container with shell into nodes.
Would you help throubleshoting in the following steps:

launch a pod
kubectl run dnsutils --image=tutum/dnsutils -ti --rm=true  --restart=Never
view DNS config in POD
cat /etc/resolv.conf
Test DNS resolve in POD
dig +add +trace www.example.com

The container's /etc/resolv.conf:
nameserver <IP_ADDRESS>
search default.svc.cluster.local svc.cluster.local cluster.local us-west-2.compute.internal
options ndots:5

dig +add +trace www.google.com
; <<>> DiG 9.9.5-3ubuntu0.2-Ubuntu <<>> +add +trace www.google.com
;; global options: +cmd
;; connection timed out; no servers could be reached

dig +add +trace ns domain2.com (an aws hosted zone)
; <<>> DiG 9.9.5-3ubuntu0.2-Ubuntu <<>> +add +trace ns stage-vew-casdb-firecamp.com
;; global options: +cmd
;; connection timed out; no servers could be reached

Can you confirm below

<IP_ADDRESS> is the clusterIP of your KubeDNS service?
the securityGroup rules on your worker node allow UDP 53 to other worker nodes?

That is the kubedns ip:
kube-system   service/kube-dns     ClusterIP   <IP_ADDRESS>     <none>        53/UDP,53/TCP                3m
and all traffic is open between nodes

@M00nF1sh you were right that it was. a communication issue. My NACLs weren't allowing communication between the subnets.


Variables used by Functions not in the HDF5 Database

Issue Description
What did you expect to see happen?
After running Samplers, I would expect  all sampled variables in the Database.
What did you see instead?
The Database has only the RELAP5 minor edits +  the RELAP5 variable directly perturbed. The other variables that were perturbing RELAP5 cards via External Functions are not reported.
Do you have a suggested fix for the development team?
Please attach the input file(s) that generate this error.  The simpler the input, the faster we can find the issue.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or improvement? improvement
[x] 2. Is it tagged with a priority: critical, normal or minor? normal
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements? n/a
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users. n/a
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.) yes

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest stable branch? If yes, is there any issue tagged with stable (create if needed)?
[x] 5. If the issue is being closed without a merge request, has an explanation of why it is being closed been provided?

closed with #215, can be reopened if needed.


Exclude current buffer from the buffer list

PickerBuffer works fine, but it shows the current opened buffer.

@srstevenson, Great! Thank you!


[BUG] The Home screen is opened after entering/editing the username and clicking on the "Save" button on the Profile edit screen.

Description: The user is logged in.  The Home screen is opened after entering/editing the username and clicking on the "Save" button on the Profile edit screen.
Steps to reproduce:

Open https://goodqa.netlify.app/
Go to the Edit Profile screen
Enter the username
Click on the Save button
Pay attention to the home screen is displayed

Actual result:
The Home screen is opened after entering/editing the username and clicking on the "Save" button on the Profile edit screen.
Expected result:
The "My Profile" screen is opened after entering/editing the username and clicking on the "Save" button on the Profile edit screen.
Environment:  QA 1.22.13-3 // Prod 1.22.20 // Dev 1.22.13-3
Devices list: Desktop // Windows 10 // Chrome 88.0.4324.150
Attachment:
https://www.dropbox.com/s/b89mtt6cvngijot/The Home screen is opened after entering the new username or editing and clicking on the Save button on the Profile edit screen..mp4?dl=0

@sirpy The issue is not reproduced. The "My Profile" screen is opened after entering/editing the username and clicking on the "Save" button on the Profile edit screen.
Checked on the QA 1.22.13-3
Device: Windows 10 // Chrome 88.0.4324.150
Attachment:
https://www.dropbox.com/s/q5ramdtx8dvppj4/The My Profile screen is opened after entering editing the username and clicking on the Save button on the Profile edit screen..mp4?dl=0


Downgrade mitodl Github account to support new Merge Queue feature

Description/Context
Investigate whether our account needs to be upgraded from 'Legacy' status in order to support the merge queue feature detailed here.
Plan/Design
(This section needs to be fleshed out)

Investigate necessary changes
Profit

Currently when we go to a repo's branch protection rules, we don't see an option to Require Merge Queue. Based on GitHub documentation that should be available to all new GitHub plans. At least that's what the Github documentation suggests in the screenshot below.

Based on that information and taking into consideration our current usage, we should be able to downgrade our legacy plan to the free one - https://github.com/organizations/mitodl/billing/plans. Once that switch is flipped, we should be able to then make use of the merge queue feature.
Planning on flipping that switch on Monday, August 7th, 2023.

As part of the downgrade, the following will be impacted:

Downgraded plan and it will not take effect till after August 15th, 2023.

Ok. Can you remind us of that on 8/15, in case anything starts behaving unexpectedly?

We haven't run into any issues related to this so far. I have checked and notified the group that the merge queue feature is now available. Will leave this issue open for another week just to make sure that we haven't lost any functionality and then close it out.

No issues so far so will close this.


Fix Homebrew tag PR

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description

Tested, changes look good and work as expected


Update to Rust edition 2021

Bump the edition (only 3 years late...) and do some other miscellaneous cleanup.

only 3 years late

Actually it’s often a good think to support older editions in order to keep support for older versions of Rust. 😇

Actually it’s often a good think to support older editions in order to keep support for older versions of Rust. 😇

I do try to do that, but its a constant battle when most of your dependencies aren't interested in keeping a relatively old minimum supported Rust version. Speaking from experience...


Update google protos version.

Theres a typo on 0.7.0 that causes errors for monitoring.

Current coverage is 91.23% (diff: 100%)

Merging #142 into master will not change coverage

@@             master       #142   diff @@
==========================================
  Files             3          3
  Lines           719        719
  Methods         150        150
  Messages          0          0
  Branches        116        116
==========================================
  Hits            656        656
  Misses           63         63
  Partials          0          0

Powered by Codecov. Last update 4f58f6c...5092ca6


Passing --address and --internal-address while creating custom cluster is not set on the node.

Build from master-7a2fc74
Describe the issue:
--address and --internal-address is not stored. in config and therefore not set on the node. While passing --node-name is stored in config and set to the node.
Steps to reproduce:

Click on Cluster Management->Create-Custom
Enter Name to the the cluster-> Create
Click on the name of the cluster in the next window
Click on Show Advanced
Pass Node-name, Node Public IP,  Node Private IP
Copy the registration command and execute it on the node.

curl --insecure -fL https://<IP>/system-agent-install.sh | sudo  sh -s - --server https://<IP> --token <TOKEN> --ca-checksum <CHECKSUM> --etcd --controlplane --worker --address <Public IP> --internal-address <Private IP> --node-name shysrv1

Config yaml output
{
  "agent-token": "<TOKEN>",
  "cni": "calico",
  "etcd-snapshot-retention": 5,
  "etcd-snapshot-schedule-cron": "* */5 * * *",
  "node-label": [
    "rke.cattle.io/machine=e876f39c-7a92-4a2d-aa46-18e04862a88d"
  ],
  "node-name": "shysrv4",
  "server": "https://<SERVER PRIVATE IP>:9345",
  "token": "<TOKEN>"
}

Confirmed from the debug results, headers are correctly passed on correctly
X-Cattle-Node-Name: shysrv1 -H X-Cattle-Address: <PUBLIC IP> -H X-Cattle-Internal-Address: <PRIVATE IP>

Other details that may be helpful:
Node-name is set on the node as seen the kubectl output
kubectl get node
NAME      STATUS   ROLES                       AGE   VERSION
shysrv1   Ready    control-plane,etcd,master   86m   v1.21.2+rke2r1

Environment information

Rancher version : 7a2fc74
Installation option : Single/HA

Cluster information

Cluster type : Custom
Machine type : AWS

Additional info:
On DO InternalIP is set to Public IP
Describe the node
etcd.k3s.cattle.io/node-address: <Public IP>
InternalIP:  <Public IP>

Leaving a note here: I have an implementation of this. I need to update the new provisioning tests when I return to complete this issue.

After running the registration command I see the following (b395238):

The specified node name shows up in the dashboard UI (Kube Node page). In this case, I used my-node-name as the node's name.
In the UI the external and internal IPs also have the correct values.
When I did a kubectl get node and the displayed name matches my expectations.
kubectl describe node my-node-name shows the correct name, internal IP, and external IP.

Based on all this, the issue has been fixed.


/join and /room/:roomId/join aren't exact aliases

Separate them, clarify their differences.
Also, fix some links which weren't being properly populated.

lgtm


Create deleteHiveAlerts

Python script using theHive4py and the elasticsearch api to obtain a list of alerts and delete them all from the ES database

Hello, this is not part of TheHive4py. This type of scripts can be contributed on TheHive-Resources repository.


Please invite me to the community

Name
ptcbink
Additional Context
Hope to contribute here

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is a remarkable open-source solution tailored for serverless platforms, aimed at empowering creators to craft their infrastructure solutions. 💻 We're thrilled to welcome you into our vibrant community! ✨ We eagerly anticipate your valuable contributions! 🤓 Don't forget to explore our community on Github and connect with us on LinkedIn 💜 Feel free to spread the word about our community to like-minded individuals interested in development, DevOps, technology, and collaborative contributions. 👉💜👈


Design example 'Open in new window' link

What
On the design examples, the link 'Open in new window' gives no context of what you are opening in a new window.

Why
It's an accessibility issue, especially when there is more than one design example on the page.
Fix
Add visually hidden text to the the link.
For example: 'Open this [style, component or pattern name] example in a new window'.

This was released in v3.0.4


Link investigator&crime case list Fixes #108

When a CrimeCase is added, the CrimeCase will designate an Investigator as its current investigator.
The current investigator will then add the CrimeCase to its list of active cases.
When converted to XML, all the CrimeCases in the investigator will be converted to hashcode and stored as an array of integer. When converted back to model, it will use the hash to find the correct CrimeCase and add it back into its list.
Add constraint so that a case can only be added to an investigator
Add constraint so that an investigator can only be deleted if he/she has no active case

Pull Request Test Coverage Report for Build 192

37 of 55 (67.27%) changed or added relevant lines in 9 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.6%) to 85.57%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/investigapptor/logic/commands/AddCaseCommand.java
0
3
0.0%

src/main/java/seedu/investigapptor/storage/XmlAdaptedInvestigator.java
7
10
70.0%

src/main/java/seedu/investigapptor/logic/commands/DeleteInvestigatorCommand.java
3
7
42.86%

src/main/java/seedu/investigapptor/model/person/investigator/Investigator.java
6
10
60.0%

src/main/java/seedu/investigapptor/model/Investigapptor.java
14
18
77.78%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/investigapptor/logic/commands/AddCaseCommand.java
1
76.32%

Totals

Change from base Build 170:
0.6%

Covered Lines:
2111

Relevant Lines:
2467

💛  - Coveralls


Verified Dojo giving 404 page

Annabel just verified this Dojo http://zen.coderdojo.com/dashboard/dojo/ie/midleton%2Fmidleton but it is giving a 404. If you search for the Dojo name "Midleton" you'll see it in manage dojos.
Dojo id is 7a6cec25-6c41-4541-9736-7ee8f15c47bb

@tangentfairy for some reason this dojo had saved in the DB dz/midleton/midleton instead of ie/midleton/midleton in the slug field. I changed that straight in the DB, to make it work, but I have to investigate further to see what caused this.

Great thanks @ckiss


Stop user player control when not controller

User should not be able to pause/play when in session with server

Perhaps check every lifecycle if controller is in a different state, then send that as response to player.js to get it to re-sync


bugfix:修复批量拉取蓝盾流水线接口

Codecov Report

Merging #535 (1414630) into v2.6.x_develop (22c133b) will not change coverage.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           v2.6.x_develop     #535   +/-   ##
===============================================
  Coverage           62.55%   62.55%
===============================================
  Files                 442      442
  Lines               27405    27405
===============================================
  Hits                17142    17142
  Misses              10263    10263

Impacted Files
Coverage Δ

itsm/component/apigw/client/devops.py
78.26% <0.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 22c133b...1414630. Read the comment docs.


add file-system backend for local development

What kind of change does this PR introduce?
Adds another backend and development overrides so that Netlify CMS will work on localhost. Instead of changes being pushed to a git repository they will be written directly to the filesystem and immediately reflected by Gatsby.
More info: https://arcath.net/2019/01/netlify-cms-on-the-filesystem-with-gatsby
Does this PR introduce a breaking change?
It will change the behaviour when accessing the /admin path on localhost which might be considered breaking depending on how you look at it.
What needs to be documented once your changes are merged?
That Netlify CMS can be used on localhost.

My biggest concern is that this starter is the primary entry point to Netlify CMS, and the file system backend isn't a core library. It isn't core because we're not ready to support local development this way yet. But adding it to this first party starter sort of makes it core in a way. The community will expect any issues encountered with this starter to be supported.
@Arcath your blog post is an awesome tutorial for folks that want to use the fs backend with this starter, and it's simple enough to do - for now, I think we'll need to let folks opt in by adding it themselves.

@erquhart That makes sense, I had read some PRs wanting to add this to core but its not there yet.
I'll close this PR for now.

@Arcath I am the creator of the file-system backend. I think you should state in your blog post that the netlify-cms-backend-fs library is in beta and there will be breaking changes coming eventually.


Collection: Add ingress-nginx v3.3.0.

Towards https://github.com/giantswarm/giantswarm/issues/28509.
Do not merge before https://github.com/giantswarm/config/pull/2113.

Thanks your pointing me to the kustomization.yaml! I also forgot this on the other PRs...
nginx-ingress-controller-app will stay until everything is migrated to this new ingress-nginx app.


Add translation to Norwegian Bokmål (nb)

Thanks, merged and uploaded to the Web app (will require reloading of page to show).
We are in the process of modularising our code better, so that translations in the future will have own files that are easier to maintain. Once this is done, we will also look into connecting to transltewiki.net in some way.

Good to hear 👍


[reggen] Generate a single we signal per register in reg_top

Most of this (enormous) diff is auto-generated. The only manual
changes are to reg_top.sv.tpl, access.py and register.py.
The idea behind this patch is that the write-enable and read-enable
signal for a field are both determined at the register level (by
reg_we or reg_re, combined with an address check). Before this patch,
each field that needed a write-enable or read-enable signal defined
one. For example, if we had a writeable register R with fields F0 and
F1, we were defined both R_F0_we and R_F1_we and always set them to
the same values. This patch changes things so that we define "R_we"
instead and use it as the "we" signal for both R.F0 and R.F1.

The private CI failure is caused because we hit #7041 (a double read from SW_SHARE1_OUTPUT_7 shows up the difference between the "ro" behaviour of the RTL and the "rc" behaviour in the hjson). I'll leave this open until that's fixed and then rebase.

ah i see. Can you let me know in your failure whether you're seeing a key
wiping situation?
Maybe it will be better for me to add more description here, but i'm not
sure how to get around the issue that the read clear is not always honored
other than adding an exclusion.
On Fri, Jun 18, 2021 at 6:12 AM Rupert Swarbrick @.***>
wrote:

The private CI failure is caused because we hit #7041
https://github.com/lowRISC/opentitan/issues/7041 (a double read from
SW_SHARE1_OUTPUT_7 shows up the difference between the "ro" behaviour of
the RTL and the "rc" behaviour in the hjson). I'll leave this open until
that's fixed and then rebase.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/7029#issuecomment-864028724,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSSXP6OYF4N3QWVT4UTTTNA2VANCNFSM464CQPGQ
.

Hi @tjaychen! Thanks for the reply.  I've extended this PR a little to de-duplicate the read-enable signals as well, and have moved the "crossed wiring" where we turn a read-enable into a write-enable for RC fields so that we just pass MYREG_re to the we port of the field (rather than assigning MYREG_MYFIELD_we = reg_re & other_stuff). I think the end result is probably clearer, and it also avoids needing any complicated logic for registers where some fields are RC and others are (say) RW.


pull request for e2e test at 01/29/2017 17:26:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[Bug]: TDS Column MetaData is different and looks incorrect

What happened?
I have noticed ADO.net rowsets do not behave correctly when trying to do batch operations.  Also inspecting rowset xml differs between mssql and babelfish.  Key columns are not identified as key columns.  I believe this is due to the TDS metadata for columns.
For a query like
select RateGroupID,Description from RateGroup with (readuncommitted) order by Description
I get TDS data like so

For babelfish 3.4 I get this result

Notice many flags are missing.  Also notice the Identity Flag is always true even for non identity columns.  Also Updateable is false for babelfish
ADO.net connections seem to rely heavily on the column metadata and its correctness.
Version
BABEL_3_X_DEV (Default)
Extension
None
Which flavor of Linux are you using when you see the bug?
Ubuntu (Default)
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Hi,
I was investigating this metadata issue:

Notice many flags are missing.

Out of all flags:

fNullable
fCaseSen
usUpdateable
fIdentity
fComputed
usReservedODBC
fFixedLenCLRType
fSparseColumnSet
fEncrypted
fHidden
fKey
fNullableUnknown

Babelfish currently supports setting the first 5 flags. Though, due to #232 it only sets these flags when FMTONLY ON is set. For ordinary queries it always returns constant value 0x9 (0b1001):

fNullable: true
fCaseSen: false
usUpdateable: updateable unknown
fIdentity: false
fComputed: false

Also notice the Identity Flag is always true even for non identity columns.

It seems that Wireshark parser for this field is unreliable, on the screenshot above with babelfish 3.4 value is 0x0009, so Identity flag (5-th bit) is actually set to false.

Also Updateable is false for babelfish

Updatable flag is set to updatable unknown, that is likely to be interpreted as true by client driver (at least in mssql-jdbc).

Key columns are not identified as key columns.

M, according to fKey spec: "Its value is 1 if the column is part of a primary key for the row and the
T-SQL SELECT statement contains FOR BROWSE.". Does the problematic query/tool operation use FOR BROWSE (not supported in Babelfish)?

I have noticed ADO.net rowsets do not behave correctly when trying to do batch operations.
ADO.net connections seem to rely heavily on the column metadata and its correctness.

Is it possible to get an example of queries/tool operations that are failing due to incorrect metadata? Out of all flags it seems that only fIdentity and fComputed are applicable to Babelfish (fNullable seems to be problematic as described in #232).


Add HTTP "head" method support in Path model

Thanks for the submission. Two notes though:

Code changes need to be made against the develop_2.0 branch, so please resubmit it against it.
In terms of @JsonPropertyOrder I would push it at the end, or at least after options.

I am interested in this change, thanks for fixing. Just my $0.02, but I would keep head after get in the @JsonPropertyOrder as that is the order listed at http://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html.

I'm afraid I have to close it as it's submitted against the wrong branch. Feel free to resubmit against develop_2.0.

I've included this into my pull request https://github.com/swagger-api/swagger-core/pull/1217


RemoteCertificateNameMismatch error on macOS

I've added test certificate to trust root for both login and system keychains on macOS (Ventura):
security add-trusted-cert -d -r trustRoot -k ~/Library/Keychains/login.keychain cacert.cer
security add-trusted-cert -d -r trustRoot -k ~/Library/Keychains/System.keychain cacert.cer

They are present in Keychain Access, all trust settings set to Always Trust. Test Azure Function fails at runtime:
Message processing error (Action=Receive, EntityPath=sbemulatortest, Endpoint=localhost)
[2024-01-08T12:24:51.244Z] System.Private.CoreLib: The remote certificate is invalid according to the validation procedure: RemoteCertificateNameMismatch.

I've created sbemulatortest queue manually in RabbitMQ to be on the safe side, did not help.

I'm experincing the same issue on macOS (Sonoma 14.1) running a .Net 8 console application.

Hey @crow-ua @martinlinkhorst Did you have any luck getting this to work or find an alternative? Thanks


Work badly inside the Material Top Tab Navigator of react navigation

sometime it's stop working and some time it's switch automatically and then keep switching b/w two tabs.

Are you on the latest version?

yes

Is this on Android/iOS or both?

I have the same problem.
https://youtu.be/G2oBH00CnqA
Please, I got this problem on few Android devices

Please take a look at PR #19 see if that can resolve your issue too.

Are you on latest version? Is this still happening, if yes please create a expo snack


Add test to sl-select to ensure event listeners are created and destroyed in the correct namespace

#426 Adds namespaces to jQuery event listeners. A test for the correctness of this should be be added to sl-select-test pending #230

Replaced by https://github.com/softlayer/sl-ember-components/issues/456


"worked around javax.mail change for attachment only messages." breaks code

Hi,
the commit https://github.com/bcgit/bc-java/commit/d050ec4c1c40544ae8d453fc769edb114f969c4b (bcmail 1.74) breaks our code:
content.setDataHandler(new DataHandler(message.getDataHandler().getDataSource()));

When I try to sign the following attachment-only Message:
MimeMessage messageToSign = new SimpleMimeMessage();
messageToSign.setContent(new byte[]{ 1 }, "application/octet-stream");
messageToSign.setFileName("file.gz");
messageToSign.saveChanges();

The signing fails silently without explicit error. The created signed MumeMultipart from the SMIMESignedGenerator is broken.
Setting the Datahandler and Conten-Headers manually "fixes" the problem:
//...
MimeMultipart signedMultiPart = sMIMESignedGenerator.generate(messageToSign);
BodyPart bodyPart = signedMultiPart.getBodyPart(0);
bodyPart.setDataHandler(messageToSign.getDataHandler());
bodyPart.addHeader("Content-Transfer-Encoding", "base64");
bodyPart.addHeader("Content-Type", "application/octet-stream; name=file.gz");

The difference seems to be, that the messageToSign has no DataSource, but calling new DataHandler(message.getDataHandler().getDataSource()) assumes there is one?
// DataHandler-Source
    public void writeTo(OutputStream os) throws IOException {
	// for the DataSource case
	if (dataSource != null) {
// bcmail 1.74 way - throws Exception
	    InputStream is = null;
	    byte data[] = new byte[8*1024];
	    int bytes_read;

	    is = dataSource.getInputStream();

	    try {
		while ((bytes_read = is.read(data)) > 0) {
		    os.write(data, 0, bytes_read);
		}
	    } finally {
		is.close();
		is = null;
	    }
	} else { // for the Object case
// bcmail 1.73 way - works
	    DataContentHandler dch = getDataContentHandler();
	    dch.writeTo(object, objectMimeType, os);
	}
    }

EDIT
Using a MimeBodyPart instead seems to fix the problem. Is my previous code wrong? Or should both ways work?
MimeBodyPart mimeBodyPartToSign = new MimeBodyPart();
mimeBodyPartToSign.setContent(new byte[]{ 1 }, "application/octet-stream");
mimeBodyPartToSign.setFileName("file.gz");

I know this is going to sound dreadful, but you probably should be using a MimeBody part. What version of JavaMail are you using? The issue showed up for us because we updated the version of JavaMail we use locally - everything still worked except this one case, the assumption was that they'd fixed something which caused it to break. I'll admit attachment only messages are a bit of an edge condition, but even when we got everything working again it seemed surprising.

I'll see what I can do about adding a check, I should be able to get something that "works" now that I have another version of JavaMail to compare with.


feat(addScript): disable 'use strict' to allow setting globals

@d3viant0ne @bebraw @SpaceK33z @TheLarkInn

Hey @bebraw, @TheLarkInn @d3viant0ne, @evilebottnawi
What is it going to take to get some love for this repository?
This PR should be considered for merge, as should the documentation one #45.

This may need to be an option, e.g options.strict = true || false, instead if being the default. Do we have an overview of potential security implications?

@timpur What's the status here? @timpur Will you continue this PR (no pressure, but please indicated if you're still interested in this)?

@michael-ciniawsky, i am just not sure how to do the option thing that you requested tbh...

Hey @timpur take a look at how @jiaweihli has done similar in this PR https://github.com/webpack-contrib/script-loader/pull/31/files#diff-168726dbe96b3ce427e7fedce31bb0bc

Yep, e.g https://github.com/webpack-contrib/script-loader/pull/31/files#diff-168726dbe96b3ce427e7fedce31bb0bc is a good starting point, also feel free to ask questions and push the code if you're stuck somewhere :)

Hey guys, let me know if there's anything I can do to get this shepherded into the project.

Some one want to take over this PR ?

Hi all,
I've sent @timpur a PR that adds an option to his logic, so that existing behavior is preserved for current users. You would add ?stripUseStrict to trigger the feature.

update the pr and added documentation, changed the option to useStrict: false to disabled use strict, enabled by default. Can change interface if you dont like.

Hello @michael-ciniawsky
Any plan to merge this PR any time soon, please ? Indeed, as others mentioned, it has proven very useful in some cases https://github.com/webpack-contrib/script-loader/pull/51

/cc @mistic can we merge this?

@evilebottnawi merged


Networked Objects Under Disabled Parents Leads To Disconnect On Scene Change

See also #2422
In the Online1 scene of the attached package, there's a disabled object called "DisabledParent".  This is not a networked object but it does have a child object that is networked and enabled (or would be if the parent gets enabled).  As soon as host client tries to switch to Online2, host client is disconnected with the first error shown below.
The DisabledParent is never enabled.  If you enable it in the scene at runtime, you get the 2nd error shown below.
Repro package attached: SceneChangeTest.zip

Steps To Reproduce

Add all 3 scenes to build settings, Offline first
Open offline scene and play in editor as Host
Press Spacebar to attempt to change scenes
BUG 1: Observe first error and disconnect.
Stop and play again, this time select DisabledParent in Online1 scene and enable it in the inspector
BUG 2: Observe 2nd error - this shows that object did get into a spawned list, which #2422 is about.

Expected behavior
Not to get disconnected and no errors.
Screenshots
DisabledParent never enabled before scene change.

DisabledParent enabled in the inspector while in Online1 scene and in Play mode.

Environment

OS: Windows
Build target: standalone
Unity version: 2021.3.16
Mirror branch: master

update:

this is the actual repro with DisabledParent gameobject:
SceneChangeTest_actual.unitypackage.zip
neither of the two issues are reproducable atm: https://gyazo.com/059ef0200b45a083c88aaca76cc30e4f

options:

mirror could ignore entities under disabled parents
mirror could force enable disabled parents
mirror could explicitly forbid entities under parents

for future reference:

repro branch: "repro_3330b"


BATA is loosing connection all the time to Network with Socket errors

Bata Pool return to OK in several minutes usually

I think it is related coin\debug.log

@midnight-miner more issues friend!

Hello again, thanks for contacting us and letting us know.
Are you able to try this release which will stop any flooding:
https://github.com/JustinPercy/BATA-Development/releases

Alex see below please check this version.
Does this version include the changes from midnightminer?
Sent from my iPhone
On Aug 18, 2017, at 8:51 AM, pc888 <notifications<EMAIL_ADDRESS>wrote:
Hello again, thanks for contacting us and letting us know.
Are you able to try this release which will stop any flooding:
https://github.com/JustinPercy/BATA-Development/releaseshttps://github.com/JustinPercy/BATA-Development/releases

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/BTA-BATA/BATA-SOURCE/issues/47#issuecomment-323345168, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMHp5svQQZESzlQE3mjLXzeIAlRZxmUhks5sZYjjgaJpZM4O7cBH.

Alex? Not sure who you are referring to. To answer your question yes it does.

Alex is my developer I CC him on message – Thx!
From: pc888<EMAIL_ADDRESS>Sent: Friday, August 18, 2017 8:59 AM
To: BTA-BATA/BATA-SOURCE<EMAIL_ADDRESS>Cc: Omar Armenteros<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [BTA-BATA/BATA-SOURCE] BATA is loosing connection all the time to Network with Socket errors (#47)
Alex? Not sure who you are referring to. To answer your question yes it does.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/BTA-BATA/BATA-SOURCE/issues/47#issuecomment-323346769, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMHp5gfiqjmUaf18wv1k22FF_Ijsbz8Fks5sZYqhgaJpZM4O7cBH.

ok no worries 👍

Virtuworks ,
I am curious to know if there was a reason you used Litecoin-cli.exe for the RPC calls?
I think I had the bata-cli.exe in the zip.
I have my Windows Small Business Server 2008 up and running... Another platform to test on.

I'll point out your comments to my developer and get back to you.

Latest seems to fix all issues!

@midnight-miner the dev tells me he was using CLI for testing only, we interface RPC calls via Port on Pool.


move to manifest v3

update this extension to be manifest v3 compatible

[ ] read up on migrating
[ ] check the checklist
[ ] migrate to service workers
[ ] update API calls
[x] update the manifest

I don't have the impression that there is anything left to do, maybe i missed something?

yea could be :)
I want to read the materials and make sure there’s nothing in the code that violates v3


B 19890 expand tac lookup int

B-19890

[!IMPORTANT]
Commit history includes other authors due to merging main -> int. However, pretty much all but 2 lines of the files changed are authored by me

Summary
Introduces TGET service objects which are now in use inside of the EDI 858 generator as well as a dedicated GHC (Office) endpoint to fetch LOAs
How to test

Open officelocal and log in
Navigate to GHC swagger UI/linesOfAccounting/requestLineOfAccounting at http://officelocal:3000/swagger-ui/ghc.html#/linesOfAccounting/requestLineOfAccounting
Perform testing scenarios

Test a 200
Payload
{
  "serviceMemberAffiliation": "ARMY",
  "ordersIssueDate": "2014-01-01",
  "tacCode": "E12A"
}

Response
{
  "createdAt": "2023-08-03T19:17:10.050Z",
  "id": "06254fc3-b763-484c-b555-42855d1ad5cd",
  "loaAlltSnID": "123A",
  "loaBafID": "1234",
  "loaBdgtAcntClsNm": "000000",
  "loaBgFyTx": 2006,
  "loaBgnDt": "2005-10-01",
  "loaDocID": "HHG12345678900",
  "loaDptID": "1 ",
  "loaDscTx": "PERSONAL PROPERTY - PARANORMAL ACTIVITY DIVISION (OTHER)",
  "loaEndDt": "2015-10-01",
  "loaEndFyTx": 2016,
  "loaHsGdsCd": "HT",
  "loaInstlAcntgActID": "12345",
  "loaObjClsID": "22NL",
  "loaOpAgncyID": "1A",
  "loaPgmElmntID": "00000000",
  "loaStatCd": "U",
  "loaSysId": "10003",
  "loaTrnsnID": "B1",
  "loaTrsySfxTx": "0000",
  "orgGrpDfasCd": "ZZ",
  "updatedAt": "2023-08-03T19:17:38.776Z"
}

Test a 400
Payload
{
  "serviceMemberAffiliation": "ARMY",
  "ordersIssueDate": "2023",
  "tacCode": "F8J1"
}

Response
response
{
  "title": "Bad Request",
  "instance": "5e12a0b3-12e3-40b8-84c9-e216d45bb1ee",
  "detail": "parsing body body from \"\" failed, because parsing time \"2023\" as \"2006-01-02\": cannot parse \"\" as \"-\""
}

Test a 401

[!IMPORTANT]
Sign out of officelocal, then refresh swagger-ui, then try using the endpoint
response

Payload
{
  "serviceMemberAffiliation": "ARMY",
  "ordersIssueDate": "2023",
  "tacCode": "F8J1"
}

Response
response
{
Unauthorized
}

Test a 403

[!IMPORTANT]
Sign out of officelocal, do not refresh swagger-ui, then try using the endpoint
Response

Payload
{
  "serviceMemberAffiliation": "ARMY",
  "ordersIssueDate": "2023",
  "tacCode": "F8J1"
}

Response
response
{
Forbidden - CSRF token not found in request
}

Test a 404
Payload
{
  "serviceMemberAffiliation": "ARMY",
  "ordersIssueDate": "2023-01-01",
  "tacCode": "F8J1"
}

Response
response
{
404
}

Test a 422
Payload
{
  "serviceMemberAffiliation": "LEGGY",
  "ordersIssueDate": "2023-01-01",
  "tacCode": "F8J1"
}

Response
response
{
{
  "title": "Validation Error",
  "instance": "8e1125e8-a0b4-40e3-a017-dc6c9bdad72f",
  "detail": "serviceMemberAffiliation in body should be one of [ARMY NAVY MARINES AIR_FORCE COAST_GUARD SPACE_FORCE OTHER]"
}
}

flaky tests

Question for 19890, when testing for a 200 with payload similar to whats below. The tacCode has to match a tacCode has to match a records tac in the Transportation_accounting_codes table right? Then the ordersIssueDate has to match a issue_date of a record in orders and the affiliation type? The tac finds the loa by the loa_id on the tac? If all of that is correct then it returns the loa? Am I understanding that correctly? Whenever I test i keep getting Unable to find any lines of accounting based on the provided parameters. Maybe I am doing something wrong.
{
"serviceMemberAffiliation": "ARMY", // Match Orders affiliation?
"ordersIssueDate": "2014-01-01", // Match orders issue Date?
"tacCode": "E12A" // Matches a TAC which has a loa_id that links it to a loa?
}

Question for 19890, when testing for a 200 with payload similar to whats below. The tacCode has to match a tacCode has to match a records tac in the Transportation_accounting_codes table right? Then the ordersIssueDate has to match a issue_date of a record in orders and the affiliation type? The tac finds the loa by the loa_id on the tac? If all of that is correct then it returns the loa? Am I understanding that correctly? Whenever I test i keep getting Unable to find any lines of accounting based on the provided parameters. Maybe I am doing something wrong.
{
  "serviceMemberAffiliation": "ARMY", // Match Orders affiliation?
  "ordersIssueDate": "2014-01-01", // Match orders issue Date?
  "tacCode": "E12A" // Matches a TAC which has a loa_id that links it to a loa?
}

Hm, maybe the seed data is generated randomly and is why that isn't quite working? It's what satisfied the condition on my end.
Here is a simple summary on the conditions:

TAC has a LoaSysID value present that can be looked up inside the line of accounting table. The LOA table should have a row with a LoaSysID value matching that of the TAC
The service member affiliation doesn't actually matter too much for this testing, just that coast guard members don't have the HHG NTS code "HS" filtered out
The "ordersIssueDate" is only utilized in the following:

The ordersIssueDate must fall BETWEEN the TAC begin date and TAC end date
The ordersIssueDate must ALSO fall BETWEEN the LOA begin date and LOA end date
This ensures that the TAC and LOA is "valid" for the provided ordersIssueDate. The TAC and LOA begin/end dates are the date ranges in which they are "valid"

Flaky test failures

Looks so, so good. I love the way this is done!
Couple overall questions for ya.

On the UI, I believe they want to see the full LOA string concatenated together. Should that be the responsibility of the UI to build that up? Do we want to store that anywhere/include it in the response?
I believe you're handling these already, but just double checking - are we explicitly checking for every aspect of the criteria for invalid LOA here? We need to store two booleans in the DB to send to Advana, one for an invalid TAC for a LOA, and one for an invalid HHG program code for a LOA. We do still have 19891 to do some of that work as well, just wondering if we want to include those bools in the endpoint as well?

Criteria for invalid LOA:

No HHG Program Code
No match to orders date and LOA eff date range (HHG)
No match to orders date and current date (NTS)
Missing any element per attachment in feature (DFAS elements)
No LOA record found (LOA system ID key), or nothing returned based on existing logic (check logic for application of LOA to an EDI 858 for payment request transmission)

This should be the responsibility of the UI in my opinion, I feel like the API should handle the model payloads and the UI how it should display
I left all that for 19891, I don't know if we want the UI to display whether or not it is "valid", just whether or not it is found. Soft warning if not found, display the concatenated long line of accounting if it is found. Thoughts?

This should be the responsibility of the UI in my opinion, I feel like the API should handle the model payloads and the UI how it should display
I left all that for 19891, I don't know if we want the UI to display whether or not it is "valid", just whether or not it is found. Soft warning if not found, display the concatenated long line of accounting if it is found. Thoughts?

Both sound good to me. I do think they want to see if it's invalid and that's where the soft warning would come in. Good with leaving that for 19891!


feat(bridge-app): add _redirects file for client side routing with netlify

adds support for client side routing when navigating directly to page like /depost /withdrawal

Current dependencies on/for this PR:

main

PR #40   👈

This stack of pull requests is managed by Graphite.


Read/Write mutable reference in serialize and deserialize_reader is unnecessary

Currently, the required method for BorshDeserialize has the following signature:
fn deserialize_reader<R: Read>(reader: &mut R) -> Result<Self>;

However, it can be changed to just:
fn deserialize_reader<R: Read>(reader: R) -> Result<Self>;

because Read has a blanket impl for mut references: https://docs.rs/borsh/latest/borsh/io/trait.Read.html#impl-Read-for-%26mut+R
Similarly, BorshSerialize can be changed from
fn serialize<W: Write>(&self, writer: &mut W) -> Result<()>;

to
fn serialize<W: Write>(&self, writer: W) -> Result<()>;

because Write has a blanket impl for mut references: https://docs.rs/borsh/latest/borsh/io/trait.Write.html#impl-Write-for-%26mut+W
This will be a breaking change

it probably should've been
fn deserialize_reader<R: Read>(mut reader: R) -> Result<Self>;

Does this change allow any wider use cases for the traits?
I believe both signatures are equivalent with respect to how the traits can be used in practice, and mutable reference receiver better reflects semantics of what the traits are doing: they borrow Reader/Writer for de/ser of one object and return it intact to the caller (by-value receiver drops Reader/Writer after one object).

It probably should've been
fn deserialize_reader<R: Read>(mut reader: R) -> Result<Self>;

No. You do not need to specify mut in the trait definition itself, only in the implementation. See playground link below.

Does this change allow any wider use cases for the traits?

It's somewhat weird that you have to pass a mutable reference to a immutable buffer when you're deserializing. See playground: https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=dd2f2606d714535fbba3e5e5b00208cc

It's somewhat weird that you have to pass a mutable reference to a immutable buffer when you're deserializing
these are playground examples for Read and Write

https://doc.rust-lang.org/src/std/io/impls.rs.html#248

I believe Rust API guidelines recommends to pass Read and Write by value: https://rust-lang.github.io/api-guidelines/interoperability.html#generic-readerwriter-functions-take-r-read-and-w-write-by-value-c-rw-value
I think this is what by_ref() is for. Read example fixed, Write example fixed

More info here: https://github.com/rust-lang/api-guidelines/issues/174

Frankly speaking this is illuminating
https://github.com/near/borsh-rs/assets/26653921/ff818879-fcc5-48ed-8ed2-706d9ef5cac8

So, do you guys intend to change this for 2.0? I also wanna add that this issue has surfaced before: https://github.com/near/borsh-rs/pull/34

So, do you guys intend to change this for 2.0?

We don't intent to release 2.0 any time soon unless there is a very good reason to. I have created a tracking issue for the ideas that would require us to bump the major version here: https://github.com/near/borsh-rs/issues/279


add disscount in profit loss calculaotr

Proposal
add discount in profit loss calculator
(A clear and concise description of what the proposal is.)

Hey contributor @satyampgt4,
I have already been assigned this issue under #693.
Kindly close this issue.
Thankyou.