Support for Yamaha TSR-700?

Not sure whether this is an issue or an expected outcome but figured I'd share.
I have a fairly new Yamaha TSR-700 that supports YNCA. I am able to configure the custom integration using the static IP of the receiver and the default port of 50000. Once configured, there doesn't seem to be much in the way of controls available. Even the power button shown below doesn't actually do anything.
I dug around inside of the Setup interface and couldn't find any configuration settings pertinent to YNCA other than the port configuration. Is there something I am missing? Or is my receiver simply unsupported?

If it supports YNCA it should work, however I can only test with my own device, so things might not always work as hoped.
At least it is possible to have some communication with the device as the model and firmware versions are showing and the integration would not add a device if it would not work at all. Not sure what could cause the On button to fail.
To further debug this I would need some logging. You can enable it by adding/extending the logger section in the Home Assistant configuration.yaml with the info below.
Then (re)start Home Assistant, after it started press the On button.
Now the log contains a bunch of debug logging that I am interested in (the lines with ynca).
After that you can undo the changes to the configuration.yaml to cleanup the additional logging
logger:
  default: info
  logs:
    ynca: debug
    custom_components.yamaha_ynca: debug

Just wondering, is there a special reason why you are trying to use this integration instead of the Yamaha Musiccast or the other Yamaha integration included with Home Assistant?
Also you mentioned you didn't see much controls. I am not sure what you are expecting, but this is how mine looks.
There are button entities to select scenes, like the button on the receiver (yours also seem to have those on the front) and per zone there is a media_player (2nd zone is disabled as I don't use it).

The media_player allows you to select source, soundprogram, control volume that kind of stuff. Possiblities depend a bit on the currently selected input.

Thanks for the feedback. Here's what I was able to extract from the logs after enabling logger for the component.
2022-07-15 16:55:15 DEBUG (Thread-4 (_send_handler)) [ynca.connection] < @SYS:MODELNAME=?
2022-07-15 16:55:15 DEBUG (Thread-3) [ynca.connection] > @SYS:MODELNAME=TSR-700
2022-07-15 16:55:46 DEBUG (Thread-4 (_send_handler)) [ynca.connection] < @SYS:MODELNAME=?
2022-07-15 16:55:46 DEBUG (Thread-3) [ynca.connection] > @SYS:MODELNAME=TSR-700
2022-07-15 16:56:16 DEBUG (Thread-4 (_send_handler)) [ynca.connection] < @SYS:MODELNAME=?
2022-07-15 16:56:16 DEBUG (Thread-3) [ynca.connection] > @SYS:MODELNAME=TSR-700
2022-07-15 16:56:46 DEBUG (Thread-4 (_send_handler)) [ynca.connection] < @SYS:MODELNAME=?
2022-07-15 16:56:46 DEBUG (Thread-3) [ynca.connection] > @SYS:MODELNAME=TSR-700
2022-07-15 16:57:16 DEBUG (Thread-4 (_send_handler)) [ynca.connection] < @SYS:MODELNAME=?
2022-07-15 16:57:16 DEBUG (Thread-3) [ynca.connection] > @SYS:MODELNAME=TSR-700
2022-07-15 16:57:46 DEBUG (Thread-4 (_send_handler)) [ynca.connection] < @SYS:MODELNAME=?
2022-07-15 16:57:46 DEBUG (Thread-3) [ynca.connection] > @SYS:MODELNAME=TSR-700
2022-07-15 16:57:52 DEBUG (Thread-4 (_send_handler)) [ynca.connection] < @ZONE2:PWR=On
2022-07-15 16:57:52 DEBUG (Thread-3) [ynca.connection] > @ZONE2:PWR=On
2022-07-15 16:57:52 DEBUG (Thread-3) [ynca.connection] > @ZONE2:AVAIL=Ready
2022-07-15 16:57:52 DEBUG (Thread-3) [ynca.connection] > @ZONE2:INP=AUDIO1
2022-07-15 16:57:52 DEBUG (Thread-3) [ynca.connection] > @ZONE2:MUTE=Off
2022-07-15 16:57:52 DEBUG (Thread-3) [ynca.connection] > @ZONE2:VOL=-40.0
  File "/config/custom_components/yamaha_ynca/media_player.py", line 181, in source
    return ynca.get_all_zone_inputs(self._ynca)[self._zone.input]
  File "/config/custom_components/yamaha_ynca/media_player.py", line 181, in source
    return ynca.get_all_zone_inputs(self._ynca)[self._zone.input]
  File "/config/custom_components/yamaha_ynca/media_player.py", line 181, in source
    return ynca.get_all_zone_inputs(self._ynca)[self._zone.input]
  File "/config/custom_components/yamaha_ynca/media_player.py", line 181, in source
    return ynca.get_all_zone_inputs(self._ynca)[self._zone.input]
  File "/config/custom_components/yamaha_ynca/media_player.py", line 181, in source
    return ynca.get_all_zone_inputs(self._ynca)[self._zone.input]

The first few lines appear every so often on their own. The rest of the excerpt shows what was returned when I tested the power on option of the single control that I currently have available.
Just wondering, is there a special reason why you are trying to use this integration instead of the Yamaha Musiccast or the other Yamaha integration included with Home Assistant?
Also you mentioned you didn't see much controls. I am not sure what you are expecting, but this is how mine looks.
Overall, I am merely curious to see what sort of controls I can leverage within home assistant.  Ideally, I'd like to be able to send commands that toggle between the AV inputs (source selection), or perhaps adjust the volume of the receiver via automations or voice commands. The Musiccast integration only offers a limited number of audio specific controls, which is fine, but not really what I am seeking.   That said, I'm mostly tinkering to learn what is possible.

Quick follow up - I decided to delete and reconfigure the integration with debug logging enabled. I figured the logs may reveal more about the setup process and perhaps give you additional insights into the setup process and how it interacts with my specific receiver.  I should have mentioned I needed the setup logs as those are most useful because during setup it detects the features that the receiver supports.
Right now I see 2 issues.
The reported soundmode is "All-Ch Stereo" which is currently unknown by the library. I need to find a better way to handle differnt supported soundmodes as they seem to differ more than expected (was also the issue with #7 )
Second issue is that some of the commands I assumed that would always be there are not supported by this receiver.
For example I request the list of input names and normally a list comes back with the names of the inputs like HDMI1=DVD, HDMI2=Chromecast etc.., but your receiver does not give a response. It even says the command is @UNDEFINED which means it is an unknown command. Same for ZONENAME and SCENENAME which are supposed to return the names of zone/scene are also not available on your receiver.
I need to make the code more robust to handle this.

I just released v4.3.0 which should work with the TSR-700.
However there are a few caveats:

Since the receiver does not provide zone names you will get generid labels like "MAIN" or "ZONE2", however you can rename the entities in Home Assistant
Since the receiver does not report names for the physical inputs I am providing a generic list with all know inputs, you can hide undesired inputs in the integration options which are accessible by the Configure button on the integration
Since the receiver does not report the scene names I don't know how many scenes are supported for each zone so no scene buttons are generated.
I don't know the full list of supported soundmodes for the TSR-700 so you get the full list of known soundmodes of which probably not all apply to your device

The list seems kinda big, but it should allow you to use all current functionality as mentioned in the readme except for activating scenes.
I want to extend a bit more like allowing to hide entries of the soundmode list, easier way to get diagnostics and maybe a way to do manual configuration of the amount of scenes. However that will take a bit more time.
Please let me know how well this works for you. If there are other issues popping up please attach logging and I can have a look.

Ok, here's the latest logs after upgrading to 4.3.0 and removing/re-adding my receiver. There's a new entity called Zone2, and I now have the ability to control the receiver's volume via a UI slider.  So definitely steps in the right direction.

Also of note, still no input sources are being picked up. At first I suspected this may be due to the fact that I've renamed some of my HDMI inputs but that's a flat out guess.
I noticed you are posting screenshots from the devices section, over there you only get very basic controls, if you want to control more you need to add the entites to a dashboard. Then you can use the stacked-triple-dot icon to get a popup with (among others) the input and soundmode selection.
It looks like this on my dev Home Assistant

Note that you will get a list of all inputs specified in the YNCA, because the receiver does not report them (really wondering why they removed that from the protocol). So you won't ahve your customized names and some will not work as they are not supported by your receiver. You can filter the input list by using the configure option in the integrations screen. If there are inputs missing in the full list enable the logging and select the missing inputs, so I can see the YNCA ids from the logs.

Hmmm, just noticed the screenshots are in Dutch, sorry for that

Success! It took me a while to figure out why I couldn't see the input sources.  Even after adding a media player card to a dashboard, I was still unable to see a Source list. It wasn't until I powered on the receiver that the Source pull down menu appeared.
Powered Off

Powered On

It's an interesting dynamic as I'm accustomed to Home Assistant remembering device specific information without the need to pull data from a powered on device. Not sure if there's a way to create Attributes which are saved with the Device during the setup process.
Regardless, I now have a way to directly call each Source Input by their default name. This is exactly what I needed since the remote that comes with my receiver does not have dedicated buttons for every input.  While doing so requires a bit of effort, it works.  That's a win for me.

Thank you so much for looking into this!

Good to hear it worked out.
Since it seems to work now I am closing this issue


Gate landing on MSVC

r? @metajack or @aneeshusa
See: http://build.servo.org/builders/windows-msvc-dev/builds/0
Woo-hoo!

This change is

@bors-servo r+

:pushpin: Commit 50075a0 has been approved by aneeshusa

:hourglass: Testing commit 50075a0 with merge 1244f38...

:sunny: Test successful - status-travis


Reversing xAxes ticks doesn't seem to work.

Hi, I'm able to reverse the yAxes, but not the xAxes. Are the specs different for the xAxes?
See sample code: https://jsfiddle.net/marineboudeau/4awme3fp/3.
And the part of the code I'm talking about:
options: {
        scales: {
            yAxes: [{
                ticks: {
                    reverse: true, // this works
                    beginAtZero:true
                }
            }],
            xAxes: [{
                ticks: {
                    reverse: true, // this doesn't seem to work
                    beginAtZero: true
                }
            }]
        }
    }

@marineb i can't remember if the reverse setting only works for numeric axes or not. This needs some investigation before triaging as eithe a bug or enhancement.

Thanks @etimberg! I created a new fiddle with numeric axes and it still doesn't seem to work. https://jsfiddle.net/marineboudeau/4awme3fp/6.
I'll reverse them programmatically in the meantime. Thanks!

Hi! I have the same issue here. Reverse works on the y axes but not on x. Is there any news on this or a workaround? Thank you!

Also having the same problem here, would appreciate any help or update on the issue!

Any updates on this ?
I see that if the Y-Axis consists of numbers then setting the reversed flags works. But if the Y-Axis consists of letters than the reversed does not work.


Added 'helm repo update' after 'helm repo add'

added helm repo update after repo add

@rayoef,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Was my CLA signing successful?

@asudbring - Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

Thank you.

#sign-off


Need some help setting up local environment- can't create boards

Hi, I'm just getting started with open source and would appreciate help in getting started.
I've installed Node JS, Gulp, and set up the Firebase, but when I go to my http://localhost:4000 and attempt to create a new board it just loads forever....
Could anyone assist me in fixing this? I feel as if I'm missing something.

@Acrylami Do you see any errors in your console when this happens? I had this same problem when I didn't have my firebase authentication set up correctly. If that were the case, you'd see an error related to your API key.

@Acrylami did you add your js/vendor/firebaseInitialization.js with your firebase configuration?

Honestly, the documentation is not very clear around where you need to go in the firebase console to get started for new contributors.
For example, you have to create a new app and then go to project settings to find YOUR_PROJECT_ID and YOUR_API_KEY

and then YOUR_MESSAGE_ID is referred to as Sender ID under the Cloud Messaging tab in the settings panel.


Model overview: improve placeholder, put confirm/cancel buttons at the bottom, and hide toggle when needed

Description
This PR includes a series of small improvements based on our recent bug bash:

removed ActionButton and replaced it with a proper placeholder for the missing table

moved confirm/cancel buttons to the bottom of flyouts

don't show toggle unless needed

show toggle when there are multiple cohorts

don't show toggle with only 1 cohort

don't show toggle without features selected

show toggle with features selected

fix "placolder" typo in the code (no visible changes)

Checklist

[x] I have added screenshots above for all UI changes.
[ ] Documentation was updated if it was needed.
[ ] New tests were added or changes were manually verified.

Codecov Report

Merging #1414 (f93db18) into main (9441f9b) will decrease coverage by 10.37%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1414       +/-   ##
===========================================
- Coverage   85.95%   75.58%   -10.38%
===========================================
  Files         100       10       -90
  Lines        4798      258     -4540
===========================================
- Hits         4124      195     -3929
+ Misses        674       63      -611

Flag
Coverage Δ

unittests
75.58% <ø> (-10.38%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

raiwidgets/raiwidgets/error_handling.py

erroranalysis/erroranalysis/_internal/utils.py

...iwidgets/raiwidgets/fairness_metric_calculation.py

...ibleai/_tools/shared/state_directory_management.py

...iwidgets/raiwidgets/model_performance_dashboard.py

raiutils/raiutils/common/__init__.py

...ai/responsibleai/_tools/causal/causal_constants.py

raiwidgets/raiwidgets/interfaces.py

...bleai/responsibleai/_managers/explainer_manager.py

responsibleai/responsibleai/_config/__init__.py

... and 80 more

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 9441f9b...f93db18. Read the comment docs.

Can we center align the placeholder text for the table when no features are selected?

Can we center align the placeholder text for the table when no features are selected?

Voila!

closing since it targeted the main branch. #1433 is the new PR


chore:Correct typo in storageSpec

Correct typo in storageSpec name condition in starrocksCnSpec of Helm chart

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


Modifying an Instance Shader Parameter seem to affect Instance Shader Params of Next Pass materials

Godot version
v4.1.1
System information
Godot v4.1.1.stable.mono - Windows 10.0.22621 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Ti (NVIDIA; <IP_ADDRESS>12) - AMD Ryzen 7 5800X3D 8-Core Processor (16 Threads)
Issue description
It seems that setting an instance shader parameter from code changes instance params on the "next pass" shaders as well.
What I would expect to happen is that these shaders and their instance uniforms are completely decoupled and independent from one another. Below I describe the set up I have and there is a gif showing the effect. project file attached.
In the attached project, I have 3 simple custom shaders.

SHADER 1 - has an exposed color instance uniform, that albedo is set to.

SHADER 2 - (this is the "next pass" on SHADER 1) - has an exposed color instance uniform that albedo is set to, it is multiplied by sin(TIME). this shader is blend_mult and unshaded so it should just be multiplied on the existing SHADER 1 color.

SHADER 3 - ( this is the "next pass" on SHADER 2) - has an expose instance uniform float that controls the size of the model in the vertex function. again blend_mult and unshaded.

I then have a sphere with a collider that I call set_instance_shader_param from on mouse enter and mouse exit, changing ONLY SHADER 1's color parameter. I observe differences in all "next pass" shaders down the stack.

It seems to lose the yellow color immediately from SHADER 2, this instance param is not set anywhere but on the shader initialization
and the size from SHADER 3 seems to get reset every time the mouse is either on or off the sphere
Ideally mousing over would just change the base color on shader 1 between blue and red,
the yellow multiply color from shader 2 would always stay consistent
the size and presence of shader 3 would stay consistent.
I'm not sure exactly how the next pass is supposed to work, please correct me if I'm wrong but I wasn't expecting that changing the color of one pass shouldn't affect seemingly unrelated factors of other passes.
Steps to reproduce

make a shader with an instance uniform
make a second shader with an instance uniform
make a material for shader 1
make a material for shader 2
set shader 1's 'next pass' field to shader 2
change shader 1 instance parameter via script
observe shader 2's instance parameter either being changes or improperly rendering

Minimal reproduction project
SetParamOnMouseOver.zip
this project is the ball project shown above, uses same recreation steps but with 3 shaders.

Instance shader parameters are changed for all shaders used by the instance, not per-instance-per-shader.
Additionally, instance uniforms can clash with each other, which is why you can manually assign indices to them with the instance_index hint.
The documentation has more detail

All of my subsequent instance params are on completely different material resources, are each different base shader scripts, and each have different instance parameter names, so there should be no need to us the instance_index per the documentation correct? It is not that the same shader instance param is being synced across multiple meshes, I understand that is how resources like a material are intended to work, the OP is 3 unrelated instance parameters, on 3 unique material resources, each with a unique shader script seemingly affected by one line of code that shouldn't affect any of them. The only thing connecting the instances are that the materials are nested in a stack of next passes, am I misunderstanding the next pass functionality?
it seems to me that none of the potential clashing situations the documentation is warning about apply here

thanks for the reply!

So setting the hint for the instance_index fixed the behavior I was seeing and the same one line of code now only affects the one parameter.
this is the code in the OP gif (didn't realize it was so crushed)

I guess I figured that with different names and types it wouldn't be effected, but I still needed the instance_index hint on each making them 1,2,3 and that was enough to fix it. Thanks!
here is how it is reacting now (this is what I originally intended)


Could not change any ModelID for /amazon-bedrock-workshop/01_Generation/

from langchain.llms.bedrock import Bedrock
inference_modifier = {'max_tokens_to_sample':4096,
"temperature":0.5,
"top_k":250,
"top_p":1,
"stop_sequences": ["\n\nHuman"]
}
textgen_llm = Bedrock(model_id = "ai21.j2-grande-instruct",
client = boto3_bedrock,
model_kwargs = inference_modifier
)
ValueError: Error raised by bedrock service: An error occurred (ValidationException) when calling the InvokeModel operation: The provided inference configurations are invalid

Hi @liyunrui you are using the Anthropic configuration parameters for the AI21Labs model, see details here: https://catalog.workshops.aws/building-with-amazon-bedrock/en-US/foundation/bedrock-inference-parameters. In addition, please make sure you use the new modelID name as of last week (Bedrock Generally Available): ai21.j2-mid-v1 or a21.j2-ultra-v1.


SSTU Solid Rocket Boosters Produce Light Before Ignited

Stock engines work fine, SSTU liquid engines work fine (those I tested), both solids show the issue.
KSP 1.7.1 Vanilla
SSTU-<IP_ADDRESS> with dependencies
output_log.txt

Does this happen when you remove the textures unlimited?

Well, it was very fun to try and put on an invisible part but yes, it still happens.

VAB looks like this without TU.  SSTU parts don't show up.

I saw it, but have no idea, have posted in the SSTU thread for  help

Thanks for the update LGG, looks the lighting is still there but it does look dimmer.


Mark as Compatible with 1.20.1

The latest version seems to work fine on 1.20.1, but It's not marked as such on Modrinth.

Fixed


add license?

Hi, I would like to use this as base image for a project of mine but would need to know if it is available under a free license and under which. Otherwise I cannot use it, I fear.

@jlesage sorry for the ping but would you be able to add a license to the repo?

A license was added with https://github.com/jlesage/docker-baseimage-gui/commit/7bee7fab3daa4f6425c36b53697736df9370d64b :)


Text shadow is not working on visualMap

Version
5.0.1
Reproduction link
https://jsfiddle.net/x4Lf09tm/
Steps to reproduce
Set the text shadow on visualMap:
visualMap: {
  textStyle: {
    color: '#000',
    textShadowColor: '#000',
    textShadowBlur: 2,
  }
}

What is expected?
The text has shadow on it.
What is actually happening?
No shadow on the text.

Text shadow is also missing when setting it for axis labels.
Version 5.3.1


Nostalgia emulators not supported

Hi. Someone already mentioned Nostalgia.gg (apparently it's now supported?) but Nostalgia.nes, Nostalgia.gba and Nostalgia.gbc still aren't detected.
Otherwise, thank you all, Daijisho is incredible. ❤
https://github.com/TapiocaFox/Daijishou/issues/266

When I went through the platforms to create the wiki, Nostalgia apps was one I couldn't find suitable launch arguments. Hence why there is no Nostalgia players available.
You can try creating a player . If you find a launch argument that launches a game or if you find an example of it working another frontend I'll gladly add them.


Clarify prefetch instruction permission

In CMO's spec, it claims
"A cache-block prefetch instruction is permitted to access the specified cache block whenever a load instruction,
store instruction, or instruction fetch is permitted to access the corresponding physical addresses."
does that mean

prefetch.i is permitted to access the specified cache block if instruction fetch is permitted to access the corresponding physical addresses. prefetch.r is permitted to access the specified cache block if a load is permitted to access the corresponding physical addresses. prefetch.w is permitted to access the specified cache block if a store is permitted to access the corresponding physical addresses.
or
Any prefetch instruction (prefetch.i / prefetch.r / prefetch.w) is permitted to access the specified cache block whenever a load instruction,
store instruction, or instruction fetch is permitted to access the corresponding physical addresses."

The intent was the latter (number 2). Since the prefetch instructions are encoded as hints, we don't do any permissions checks, even though the standard address translation applies. The different suffixes really are hints to the memory system for allocation into specific caches (e.g. instruction cache or data cache), with specific cache state permissions (e.g. shared, i.e. read-only, or unique, i.e. read-write), rather than indicating an actual memory operation.


To implement importing and registration of managed clusters

@cdoan1 this implements what I showd in demo to Jeff Brent and @tpouyer

Addeded umbrealla issue for backup-n-nrestore https://github.com/open-cluster-management/sre-tools/issues/10


importing pl_bolts makes a Jupyter xeus-python kernel crash

🐛 Bug
If you run a xeus-python kernel in JupyterLab and then just execute import pl_bolts, the kernel will crash. Everything is fine with a normal IPython kernel. I have also posted the bug in the xeus-python repository since I don't know if the bug is in pl_bolts or in xeus-python (or in both).
What is xeus-python?
xeus-python is a replacement of IPython which enables valuable features in JupyterLab, like a debugger! It may became the new default choice for Jupyter users (to make a parallel, as mamba is replacing conda), so I think it is worth supporting it.
To Reproduce
Steps to reproduce the behavior:

Create a new conda environment. With conda, or mamba, install xeus-python, jupyterlab, torch, torchvision, pytorch-lightning.
With pip install pytorch-lightning-bolts.
Run a new JupyterLab instance with jupyter-lab
Set the kernel to XPython
run import pl_bolts in a cell
the kernel will crash

Expected behavior
Everything should work fine, as it happens with a standard IPython kernel.
Environment

PyTorch Version (e.g., 1.0): 1.8.0
OS (e.g., Linux): centOS
How you installed PyTorch (conda, pip, source): mamba
Build command you used (if compiling from source):
Python version: 3.9
CUDA/cuDNN version: 10.2
GPU models and configuration:
Any other relevant information:

@LucaMarconato Thank you for reporting the issue. Unfortunately, I'm not familiar with xeus-python, but people in the community might be able to help you debug if there's any error log or other information. One thing I'd try is to use Python {3.6,3.7,3.8} because there are still few packages that officially support 3.9, including pytorch-lightning.

The problem disappears when installing pytorch-lightning from conda instead than from pip, as it has been pointed out here.
I close the issue as there is a workaround and probably it is not worth the effort to investigate more into it.


problem with the crawler as the tutorial said

With this code:
CrawlConfiguration crawlConfig = new CrawlConfiguration();
crawlConfig.CrawlTimeoutSeconds = 100;
crawlConfig.MaxConcurrentThreads = 10;
crawlConfig.MaxPagesToCrawl = 1000;
crawlConfig.UserAgentString = "abot v1.0 http://code.google.com/p/abot";
//crawlConfig.ConfigurationExtensions.Add("SomeCustomConfigValue1", "1111");
//crawlConfig.ConfigurationExtensions.Add("SomeCustomConfigValue2", "2222");
PoliteWebCrawler crawler = new PoliteWebCrawler(crawlConfig, null, null, null, null, null, null, null);
This is the error:
Error	1	The best overloaded method match for 'Abot.Crawler.PoliteWebCrawler.PoliteWebCrawler(Abot.Poco.CrawlConfiguration, Abot.Core.ICrawlDecisionMaker, Abot.Util.IThreadManager, Abot.Core.IScheduler, Abot.Core.IPageRequester, Abot.Core.IHyperLinkParser, Abot.Util.IMemoryManager, Abot.Core.IDomainRateLimiter, Abot.Core.IRobotsDotTextFinder)' has some invalid arguments

Thanks, updated the documentation for the extra null that is needed.


question

I don't believe this is an issue on your end but if I do this code for United States I still get the results for the .getAll()
const all = await covid.getAll();
const state = await covid.getState({ state: args.splice(1).join(` `) });
const country = await covid.getCountry({ country: args.splice(1).join(` `) });
if (!args[1]) {
    console.log(all);
} else if (args[0] === `state`) {
    console.log(state);
} else if (args[0] === `country`) {
    console.log(country);
}

Could you please show example output?

I’ll have to get a screenshot after work
Sam Williams

no args = https://prnt.sc/t1jmi0
state args = https://prnt.sc/t1jmxx
country args = https://prnt.sc/t1jnir
@puf17640

Uhm, are you sure you are using this wrapper? There is neither a getAll, getState or getCountry method in the package.

On Fri, Jun 19, 2020 at 2:50 AM puf17640<EMAIL_ADDRESS>wrote:

Uhm, are you sure you are using this wrapper? There is neither a getAll,
getState or getCountry method in the package.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NovelCOVID/node-api/issues/48#issuecomment-646468981,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANAAQ7ORLPMEHAZTOE5PW2LRXMDDZANCNFSM4N5JYUCQ
.

Then how’s it getting any info? Because it worked at first until I gave it
the option for args
Sam Williams

How am I supposed to know what you are using? I can just tell you that this is not how you use THIS library, please check your package.json file.

Fuck. I clicked the wrong one lol sorry man
Sam Williams

@sambeano7 can this be closed?

Yeah I’m really sorry 😅
Sam Williams


[$500] Split scan - "Transaction is missing fields" only shows up after two taps on Split button

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.32-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Create a split scan with two users with an invalid receipt.
Wait for the scanning to fail.
When the scanning fails, go to the group chat.
Tap on the split preview.
Tap on the Split button.
Tap on the Split button again.

Expected Result:
First tap on the Split button will bring up the error "Transaction is missing fields".
Actual Result:
It takes two taps on the Split button for the error "Transaction is missing fields" to show up.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/8bc9d221-a24b-4c49-b646-6a8d46cd999a
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0144be74fa1e6724ba
Upwork Job ID:<PHONE_NUMBER>209477632
Last Price Increase: 2024-01-25

We think that this bug might be related to #vip-split-p2p-chat-groups
CC @gabrielessner

Proposal
Please re-state the problem that we are trying to solve in this issue.
The transaction missing fields error only shows after pressing the split button twice on a failed scan split bill.
What is the root cause of that problem?
We have a useEffect that handles which error message to show.
https://github.com/Expensify/App/blob/f6d2f61503f2ac9dd5d38a25ccf48e748ebad530/src/components/MoneyRequestConfirmationList.js#L288-L299
Currently, it prioritizes failed scan error messages. When we press the split button for the first time, we set the didConfirmSplit state to true, but because we prioritize failed scan error message, the missing field error message doesn't show. The second time we press it, didConfirmSplit state is still true, so the above useEffect isn't triggered and the reason we are able to see the missing field error is because we manually set the error message when pressing the button.
https://github.com/Expensify/App/blob/f6d2f61503f2ac9dd5d38a25ccf48e748ebad530/src/components/MoneyRequestConfirmationList.js#L490-L492
What changes do you think we should make in order to solve the problem?

We can take field missing error message priority higher by switching the if position here.
https://github.com/Expensify/App/blob/f6d2f61503f2ac9dd5d38a25ccf48e748ebad530/src/components/MoneyRequestConfirmationList.js#L288-L296

Same

Same

Missed initial notifications for this. I'll review today.

@bernhardoj's proposal demonstrates a better understanding of the root cause and why the didConfirmSplit flag is being used, so we can move forward with this.
🎀👀🎀 C+ reviewed.

Waiting for @Beamanator sign off on proposal

PR is ready
cc: @akinwale

@akinwale could you please complete checklist so we can prep to issue payment next week?

[x]  [@akinwale] The PR that introduced the bug has been identified. Link to the PR:
[x]  [@akinwale] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[x]  [@akinwale] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

Not a regression. The error message priority was not taken into consideration when checking certain conditions in the implementation.

[x]  [@akinwale] Determine if we should create a regression test for this bug.
[x]  [@akinwale] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Steps

Launch the Expensify app
Create a new split bill on a group chat with an invalid receipt image.
After the scan fails, verify that you see the Scan failed error message.
Click the Split preview to open the details.
Press the Split button.
Verify that the missing fields error message is displayed.

Great. TY! I'll handle that today.

All set to issue payment!

@bernhardoj $500 sent and contract ended!

@akinwale $500 sent and contract ended!

Upwork job closed.


Switch middleware to avoid cookie packing, and use case mangling instead

Description
By mangling the case of the 'Set-Cookie' header, API Gateway will retain them all and include them as 'set-cookie' headers. This replaces the cookie packing which had issues with strict cookie parsers.

Coverage decreased (-1.2%) to 79.974% when pulling 67792f61cd5740e8bea92f49f811a5ea9bf64b3a on bbangert:master into c99f4b2abbde8ef543b996dff8fedc253971f2dc on Miserlou:master.

Extremely excited about this feature! Out with the old hacks, in with the new ones.
Any chance you can repeal and replace the tests for this? Should actually increase overall coverage. It'd be great if you could also throw in some docstrings to the functions explaining what's happening here, and refer back to that pretty code you found on SO for proper attribution as well. 😸

I can confirm that it's working. Getting multiple 'set-cookie' headers back. Whohoo!

BTW, used a django app to test this.

Just tested it with my django project, works perfectly :D


Missing mobile prefix for Faroe Islands

298-78 ( Føroya Tele) is missing. I will share documentation offline.

@alecx
Thanks for reporting this issue.
This is fixed in release 7.2.8, which is made live yesterday.
https://libphonenumber.appspot.com/phonenumberparser?number=781234&country=FO
We will update carrier name in an upcoming release.

Closing this issue as carrier name is also updated.


shuffle, repeat button icon

Icon dosen't change according to the state of those buttons.
Need to find appropriate icon images.

Fixed in 53cebfd


Bug: [RN-Android] The pop-up window on the map doesn't disappear when user goes to another page

Tested areas:

Android 10 (Honor 20)

Description:
The pop-up window with the object name (on the map) doesn't close when an user click the "back" arrow ("back" arrow at the bottom of the Android device's screen) and goes to another page. An user returns to the main page but the pop-up window is still displayed. It's closed only after the app restarting.
Test steps:

Run Radzima app.
Go to the map.
Zoom in the map until the object icons are displayed.
Click the object icon.
Make sure the pop-up window with the object name is displayed.
Click the "back" arrow at the bottom of the screen (this the "back" arrow belongs to the Android device - not to the app).

Expected result:
The pop-up window is closed after clicking the "back" arrow.
Actual result:
The pop-up window is displayed after clicking the "back" arrow. An user needs to restart the app to close the pop-up window.
Screenshots are attached: https://photos.app.goo.gl/aysaZ9qEYF2dHbHv9

@YanaFrolenkova Fixed - see release 140

Verified - closed.


Add logo to readme

#270 created a logo for the package, this PR adds it to the readme.

[x] Check logo link
[x] Add Nele as contributor
[x] check failing test

We could consider uploading a higher quality logo, but this is not default behaviour in usethis. Probably to keep the page loading time down.

I'm not sure why tests are failing

Made a mockup of a version with a red background.

Checked with https://www.color-blindness.com/coblis-color-blindness-simulator/

There was a tiny issue on the right lower corner of the hexagon, where a part of the background was bleeding into the black border. I filled it in with black:

I've also included the sRGB colour profile, and stored it as an interlaced PNG for quicker loading. Not sure that those changes will matter at all because usethis converts the image with imagemagick anyway.

Love the logo @PietrH, for me this is fine. Feels very Jurassic to me

Ready for review, should be able to merge if we agree on the logo.


Content hard to read

As a person who's language is not native English, this article is quite difficult to read. Especially "Log search alert rule - firing and state" took me a long time to read and understand. Would be very beneficial to skip the ";" and  "-" and instead write shorter sentences.

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

ID: ab89353c-6acc-9944-40e0-d8ab067e7d41
Version Independent ID: b8c18210-d14f-4b7c-aee9-545cafd20582
Content: Log alerts in Azure Monitor
Content Source: articles/azure-monitor/platform/alerts-unified-log.md
Service: azure-monitor
GitHub Login: @msvijayn
Microsoft Alias: vinagara

@TheRealKnurft, Thanks for posting the question on this! Your feedback has been shared with the content owner for further review.

@TheRealKnurft Thank you for your feedback. We'll look to rewrite the content in shorter sentences in the next doc refresh in Apr 2019.

@TheRealKnurft, We will close this out with doc refresh pending, but if you feel you need more information please just let us know.


Errors when upgrading a project to version 1

I'm trying to update the project to vituum v1, for this a vituum-v1 branch was created. I ran into some issues with the transfer.
Changing the pages.dir parameter to ./src/views works correctly with npm run dev, but npm run build ignores the contents of ./src/views, building html only works if you remove the parameter pages.dir and move the contents of ./src/views back to ./src/pages.
{
  pages: {
    dir: "./src/views"
  }
}

Running npm run dev creates +.js and +.css in the subfolders ./src/scripts and ./src/styles.
Running npm run build creates 404.twig.js and index2.js in ./dist/assets/js, although only one is imported into 404.twig and index.twig js file ./src/scripts/index.js
Tried importing only pages separately (to disable the vituum imports plugin) as shown in the documentation:
import pages from 'vituum/plugins/pages.js'

In this scenario, npm run dev displays pages in the browser correctly, but if you run npm run build, the build will stop with an error:
Could not resolve entry module "index.html".

Also, when importing pages like this, vscode swears that it cannot find the module at the given path. Tried changing the import to the following path:
import pages from "vituum/src/plugins/pages.js";

But then another error appears on startup:
✘ [ERROR] Missing "./src/plugins/pages" specifier in "vituum" package [plugin externalize-deps]

Is it possible to disable the script and style collector, or do I need to go some other way to do this?

When you change pages directory to something else, you need to also change build.rollupOptions.input - https://vituum.dev/config/main-options.html#input (some goes if you use only pages plugin)
You can turn off imports by setting following in vituum config
imports: {
    paths: []
}

vituum/plugins/pages.js should be exported, but I can see the export is not working correctly, will look into that

Many thanks for your help!
Added vituum and rollup options, this helped to solve the problem of +.js and +.css appearing, html is now also assembled correctly, but is now created when building dist/assets/js/404.twig.js and dist/assets/css/404.twig.css , although dist/assets/js/index.js and dist/assets/css/index.css would be expected.
Can you suggest how to fix this?

You also need to include css and js into rollupOptions.input, otherwise rollup takes the name from the html file. So the paths should look like this
[
'./src/views//*.{json,twig,html}',
'!./src/views//.twig.json',
'./src/styles/.{css,pcss,scss,sass,less,styl,stylus}',
'./src/scripts/*.{js,ts,mjs}'
]

something went wrong again)))

Try adding modulePreload: false to build, that should remove that extra 404.twig.js. But I'm not sure why the js entry is renamed to index2.js. Happens only when you have build.rollupOptions.output and css and js has same name, that is some vite / rollup thing.

yes, the modulePreload option helped get rid of 404.twig.js, but the output is still index2.js
dist/assets/css/index.css
dist/assets/js/index2.js

interestingly, there was no such problem with vituum version 0.x.
I decided to experiment if I rename index.js in main.js, then there will be no two at the end of the file name. very strange...
dist/assets/css/index.css
dist/assets/js/main.js

maybe it has something to do with index.twig? it feels like when building it considers that the name index.js is already taken and adds 2 at the end.

no, it's not related to index.twig. now I tried to change the name index.twig to home.twig, also created index2.js


0.10.0rc0: tf.pack uses the wrong negative axis

When supplied a negative axis, tf.pack() actually operates on (axis - 1). For example, providing axis=-1 operates on the second-to-last axis, not the last axis.
This code illustrates the issue:
import tensorflow as tf
import numpy as np

# 10 tensors with shape 2, 3, 4, 5
x = [tf.convert_to_tensor(np.zeros((2, 3, 4, 5)))] * 10

# pack along last axis explicitly
tf.pack(x, 4).get_shape().as_list() # [2, 3, 4, 5, 10]

# pack along last axis with negative axis, doesn't work
tf.pack(x, -1).get_shape().as_list() # [2, 3, 4, 10, 5]

# this doesn't match the behavior of unpack(), so packing/unpacking
# along the same negative axis fails
len(tf.unpack(tf.pack(x, axis=-1), axis=-1)) == 10 # False

Environment info
Operating System:
macOS 10.11
Installed version of CUDA and cuDNN:
(please attach the output of ls -l /path/to/cuda/lib/libcud*):
N/A
If installed from binary pip package, provide:

Which pip package you installed.
https://storage.googleapis.com/tensorflow/mac/cpu/tensorflow-0.10.0rc0-py3-none-any.whl
The output from python -c "import tensorflow; print(tensorflow.__version__)".
0.10.0rc0

This looks like a bug in shape inference, actually.  What happens if you evaluate the tensors?

@girving Yep, I think you're right:
In [163]: tf.pack(x, 4).eval().shape
Out[163]: (2, 3, 4, 5, 10)

In [164]: tf.pack(x, -1).eval().shape
Out[164]: (2, 3, 4, 5, 10)

In [165]: tf.pack(x, -1)
Out[165]: <tf.Tensor 'pack_105:0' shape=(2, 3, 4, 10, 5) dtype=float64>

And indeed, trying to eval the unpack/pack combination at the end of my snippet raises a shape mismatch  error.

Shape inference there
https://github.com/tensorflow/tensorflow/blob/2a10e469d70827ede21c0bb5d0a46aac97404d4c/tensorflow/python/ops/array_ops.py#L772
doesn't have special logic for negative axis
input_shape.insert(op.get_attr("axis"), len(op.inputs))
So if axis is "-1", it just does "shapelist.insert(-1, stuff)"
It should probably mirror the logic from the C++ implementation
GetAxisForPackAndUnpack
https://github.com/tensorflow/tensorflow/blob/2a10e469d70827ede21c0bb5d0a46aac97404d4c/tensorflow/core/ops/array_ops.cc#L30
On Wed, Aug 10, 2016 at 12:19 PM, Jeremiah Lowin<EMAIL_ADDRESS>wrote:

@girving https://github.com/girving Yep, I think you're right:
In [163]: tf.pack(x, 4).eval().shape
Out[163]: (2, 3, 4, 5, 10)
In [164]: tf.pack(x, -1).eval().shape
Out[164]: (2, 3, 4, 5, 10)
In [165]: tf.pack(x, -1)
Out[165]: <tf.Tensor 'pack_105:0' shape=(2, 3, 4, 10, 5) dtype=float64>
And indeed, trying to eval the unpack/pack combination at the end of my
snippet raises a shape mismatch error.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/3731#issuecomment-238974490,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABaHJt2-3N-yWkNjx9a7qr_LRmkQV4Yks5qeiQygaJpZM4JhRBe
.

I'm in the process of locating the Github account of the axis attr's author.
On the other hand, @cwhipkey: Are we far enough along that Python shape inference routines can go away?

On Wed, Aug 10, 2016 at 1:12 PM, Geoffrey Irving<EMAIL_ADDRESS>wrote:

I'm in the process of locating the Github account of the axis attr's
author.
On the other hand, @cwhipkey https://github.com/cwhipkey: Are we far
enough along that Python shape inference routines can go away?

I think we are still several weeks away from removing the python shape
inference ones.
(@vrv fyi)

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/3731#issuecomment-238988438,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQw4wby3EsDCcv9qgvIfMsLkmKp-g7wFks5qejCYgaJpZM4JhRBe
.

Looks like the author is on vacation, so I'll fix it.

Fixed internally, should push out soon.

@jlowin Thank you for reporting!

@girving thanks for fixing!


Added Windows Installers using WiX - x86 and x64

This change adds two new project to the msvs folder -- a WiX project that will build a Windows installer (.msi). This installer does the following:

Installs files to Program Files (x86)\Redis-Windows or Program Files\Redis-Windows
Installs Redis as a Windows service that runs under the NetworkService account
Adds program to "Programs and Features" for easy removal
Installer can be run from the command-line with the syntax: msiexec /i Redis-Windows-x64.msi LOGLEVEL=verbose CONFIG=redis.windows.conf
Installer can be run by double-clicking the built MSI
Defaults to loglevel "verbose" and config "redis.windows.conf"
Caveats
Currently, there is no GUI for the installer, but one could be added later

Not sure why candle.exe is failing on AppVeyor. If the file is there, it should run okay. I haven't used AppVeyor before. Anyone know more?

Are you planning to deliver the msi installers with the binary releases? So we don't have to build it ourselves.

I'm a bit new to github. Two blockers currently are: (1) this Appveyor builder seems to not like calling the candle.exe executable, which is the WiX compiler. I've included candle.exe in the source files, but maybe there's a security rule against it. It does build locally. (2) I'd like to add a rule to the installer to make an exception in the Windows firewall for redis. It's easy to do and I plan to do it soon.
I'm not sure how to add the MSIs to the binaries. I think it would be useful though! I'm just trying to install redis with Chef, since I'm learning Chef, and an MSI seemed like the better way to go.
Currently, I closed the pull request.

I'm not very familiar with Git neither. I actually didn't know until 20 minutes ago it was capable of automated build and release management.
We've included Redis to our software and we programmatically do the installation steps (copy binaries, declare as a service, add exception...) but it would be nice to have an MSI doing that.
We could build the whole project in our VS Online environment but I don't want to add the compilation of redis + MSI's to our build process that would slow it down a lot.
Anyway if you could add it to the release that'd be great, otherwise I guess i'll manually update it from time to time.

Okay,  might not be much to it.  I will try to find out.


[VL] Untangle code of TransformPreOverrides

Break TransformPreOverrides into 3. The rules are still in TransformPreOverrides as its sub rules.

FilterTransformRule()
RegularTransformRule()
AggregationTransformRule()

RegularTransformRule: This rule implements a single node as Gluten's transformer without checking it's children. While the other two rules conditionally implements filter and aggregate by checking its children. In future we'd put corresponding logics to validation phase, to gain some flexibility during implementation to choose whether to transform one single node or not.

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_4888_time.csv
log/native_master_03_13_2024_d7ed0844e_time.csv
difference
percentage

q1
38.57
38.81
0.244
100.63%

q2
22.10
24.06
1.961
108.87%

q3
37.19
38.18
0.987
102.65%

q4
37.74
38.49
0.745
101.97%

q5
70.03
69.71
-0.330
99.53%

q6
7.33
7.45
0.122
101.66%

q7
85.51
82.46
-3.055
96.43%

q8
84.69
83.21
-1.484
98.25%

q9
123.59
121.83
-1.754
98.58%

q10
44.07
44.36
0.289
100.66%

q11
20.12
20.90
0.776
103.86%

q12
27.87
28.06
0.192
100.69%

q13
49.11
46.88
-2.238
95.44%

q14
23.29
21.98
-1.310
94.38%

q15
28.93
33.08
4.148
114.34%

q16
13.99
13.84
-0.154
98.90%

q17
102.42
101.80
-0.623
99.39%

q18
142.93
141.05
-1.878
98.69%

q19
14.95
15.07
0.120
100.80%

q20
29.15
27.03
-2.129
92.70%

q21
230.08
229.52
-0.552
99.76%

q22
14.27
13.86
-0.408
97.14%

total
1247.95
1241.63
-6.326
99.49%


Parse the names of children of TabContainers on POT generation

TabContainer makes use of its children's names to title its tabs, so those should be translated as well. If a child has auto_translate disabled, it will be skipped, as well as all children if the TabContainer itself disabled it.

This can generate false positives if set_tab_title() is used.

@KoBeWi The the user can toggle auto_translate off in the respective node (or the container) itself, as explained above.

Thanks!


chore: gts fix paginator and operation

Codecov Report

Merging #105 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #105   +/-   ##
=======================================
  Coverage   98.29%   98.29%
=======================================
  Files          15       15
  Lines        2814     2814
  Branches      166      165    -1
=======================================
  Hits         2766     2766
  Misses         25       25
  Partials       23       23

Impacted Files
Coverage Δ

src/paginator.ts
100% <100%> (ø)
:arrow_up:

src/operation.ts
97.82% <100%> (ø)
:arrow_up:

test/paginator.ts
100% <100%> (ø)
:arrow_up:

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 33a65a1...5be325e. Read the comment docs.


DeviceSize returns UnknownSize with landscape left screen

I am coding with Swift language and I have notice when I rotate the iOS simulator for any iPhone model SDiPhoneVersion.deviceSize() returns UnknownSize. That only happen when I rotate to landscape left. I have tried my code with one physical iPhone and I have obtained the same mistake

Same here.

change deviceSize() SDiPhoneVersion.m to:
+(DeviceSize)deviceSize {
CGFloat screenHeight = 0;

if (iOSVersionGreaterThan(@"8")) {

    UIInterfaceOrientation orientationInterface = [[UIApplication sharedApplication]statusBarOrientation];

    if (orientationInterface ==  UIInterfaceOrientationPortrait)
        screenHeight = [[UIScreen mainScreen] bounds].size.height;

    else if ((orientationInterface == UIInterfaceOrientationLandscapeRight) || (orientationInterface == UIInterfaceOrientationLandscapeLeft))
        screenHeight = [[UIScreen mainScreen] bounds].size.width;

} else
    screenHeight = [[UIScreen mainScreen] bounds].size.height;

if (screenHeight == 480)
    return iPhone35inch;
else if(screenHeight == 568)
    return iPhone4inch;
else if(screenHeight == 667)
    return  iPhone47inch;
else if(screenHeight == 736)
    return iPhone55inch;
else
    return UnknownSize;

}

Fixed in 5689723c26d0122230116daafc104f9727956e9f.


/usr/lib/libldap.a cannot be loaded when building functional OpenJDK tests

When trying to build the openjdk functional tests on paix831.rtp.raleigh.ibm.com (using these instructions), the following error occurs when running make compile:
     [exec] check for CryptoTest. If found, the file will be removed.
     [exec] git clone --depth 1 -q https://github.com/rh-openjdk/CryptoTest.git -b master CryptoTest
     [exec] exec(): 0509-036 Cannot load program /opt/freeware/libexec64/git-core/git-remote-https because of the following errors:
     [exec]     0509-150   Dependent module /usr/lib/libldap.a(libldap-2.4.so.2) could not be loaded.
     [exec]     0509-153   File /usr/lib/libldap.a is not an archive or
     [exec]                the file could not be read properly.
     [exec]     0509-026 System error: Cannot run a file that does not have a valid format.

BUILD FAILED
/home/midronij/aqa-tests/TKG/scripts/build_test.xml:95: The following error occurred while executing this line:
/home/midronij/aqa-tests/functional/build.xml:52: The following error occurred while executing this line:
/home/midronij/aqa-tests/functional/security/build.xml:30: The following error occurred while executing this line:
/home/midronij/aqa-tests/functional/security/Crypto/build.xml:67: The following error occurred while executing this line:
/home/midronij/aqa-tests/functional/security/Crypto/build.xml:45: The following error occurred while executing this line:
/home/midronij/aqa-tests/TKG/scripts/build_test.xml:128: exec returned: 1

Total time: 107 minutes 26 seconds
1
make[1]: *** [compile.mk:45: compile] Error 1
make[1]: Leaving directory '/home/midronij/aqa-tests/TKG'
make: *** [makefile:87: compile] Error 2

This happens even when using the most recent versions of OpenJDK, OpenJ9, and OMR with no changes of my own. As well, since it seems that this may be a problem with curl, this system is using version 4.8.0:
bash-5.0# curl --version
curl 8.4.0 (powerpc-ibm-aix<IP_ADDRESS>) libcurl/8.4.0 OpenSSL/1.1.1v zlib/1.2.13 libssh2/1.10.0 nghttp2/1.55.1 OpenLDAP/2.5.16
Release-Date: 2023-10-11
Protocols: dict file ftp ftps gopher gophers http https imap imaps ldap ldaps mqtt pop3 pop3s rtsp scp sftp smb smbs smtp smtps telnet tftp
Features: alt-svc AsynchDNS GSS-API HSTS HTTP2 HTTPS-proxy IPv6 Kerberos Largefile libz NTLM SPNEGO SSL threadsafe UnixSockets

git clone --depth 1 -q https://github.com/rh-openjdk/CryptoTest.git -b master CryptoTest is a stardard cmd. This is machine issue.
I think you need to open the issue at runtimes/infrastructure repo.


Bulk replace html code bock to markdown format

Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Internal previews

📄 File
🔗 Preview link

docs/fsharp/language-reference/attributes.md
Attributes (F#)

docs/fsharp/language-reference/query-expressions.md
Query Expressions

Hi @Saisang
There's more work that needs to take place before this is ready to merge.
The F# Attributes article doesn't render correctly. It looks like the triple-backslash MD tags aren't respected in the HTML table. That's also causing warnings because of the < and > characters being interpreted as XML tags, not inline code.

It looks like " was incorrectly replaced with = instead of ". Also, can you make sure there are blank links before and after code blocks (```). It would also be great if multi-line code blocks were indented properly, as I showed in one of my suggestions.

@gewarren @BillWagner
I was following the requirements in the ticket https://dev.azure.com/msft-skilling/Content/_workitems/edit/168764 and it says replace " with =. Sorry about that. It's hard to fix in the current branch so I'm creating a new PR https://github.com/dotnet/docs/pull/37537 I'm going to close this one.


Censoring product reviewer name

In product page when a user write his review his name is showing fully and for privacy most censor name characters like A**** G**** or user can select name view type.

@aghaeian
I have checked and confirmed that the setting has been added to the configurations, and it is now working fine. Please refer to the attached screenshot for reference. You can also review the PR for further details.
I hope this helps with your concern. We truly appreciate your efforts and attention towards Bagisto.
Screenshot
https://prnt.sc/U39pb04P2g99
https://prnt.sc/yoyfCa9cTJdJ
https://prnt.sc/ClaFdiG7CW2F

I have reviewed the PR again and in Network Review -> There also we are able to see the issue has been fixed.
Video
https://github.com/user-attachments/assets/56dcf9bf-9071-480f-a4d8-6d7887a10b2f


Game crashes with latest build

Describe the Bug
The game crashes with the latest build in
Reproduction Steps

extract the latest build-basins fixed
add to mods folder
Run game
...

Expected Result
Game to run with the mods installed
Screenshots and Videos

Crash Report or Log
https://pastebin.com/26VvdmaV
Operating System
Windows 10
Mod Version
0.4.0f
Minecraft Version
1.18.1
Other Mods
No response
Additional Context
Doesnt seem like the proper version isnt selectable above
Using the latest build i believe
create-mc1.18.2_v0.4.1+486.jar
create-mc1.18.2_v0.4.1+486-sources.jar
with the latest fabric
Thank you for the work your doing, love the mod an love that your trying your best to get it to fabric

duplicate of #231


Resolve bpf issue 515

What type of PR is this?
/kind documentation
Any specific area of the project related to this PR?
/area driver-bpf
/area driver-modern-bpf
/area tests
Does this PR require a change in the driver versions?
I am not sure.
What this PR does / why we need it:
Fixes bpf param 1 from int64_t to int.
Which issue(s) this PR fixes:
Params inconsistencies in our drivers #515 bpf

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ecbadeaux
Once this PR has been reviewed and has the lgtm label, please assign lucaguerra for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @ecbadeaux! It looks like this is your first PR to falcosecurity/libs 🎉

Welcome @ecbadeaux! Thanks for this PR 🚀. The plan is to knock out issues in #515 as time allows.
Tagging @incertum @FedeDP @Andreagit97 for visibility.

@leogr @Andreagit97 and @FedeDP here we are again. We need to decide on a path forward to fix remaining issues in 515 for once. While we fixed few things without needing a new event type for example this change would need a new event.
Is there no way we can introduce versioning of scap files and now would mark breaking backwards compatibility?

ei @ecbadeaux thank you for this and this https://github.com/falcosecurity/libs/pull/1382! Unfortunately, as @incertum has correctly highlighted these patches are more complex than they seem... without making it too long, we cannot change the schema of current events (remove/add/change size of params) otherwise we would no longer able to read old scap-file (libs is able to read all old events if dumped to a file).
The only thing we can do is to add new events with a new format... at least right now... Probably is my fault because in issue https://github.com/falcosecurity/libs/issues/515 I've not highlighted this downside, I will update it ASAP splitting issues that don't require a new event from ones that can be easily addressable. For what concerns the ones that require a new event, as @incertum correctly said, we would like to address them all in one shot when the scap-file management is solved. Sorry again for the inconvenience!

I will update it ASAP splitting issues that don't require a new event from ones that can be easily addressable.

Done -> https://github.com/falcosecurity/libs/issues/515#issuecomment-1746755735


Allow copy precompiled files

Close #54. In this version we just allow to copy arbitrary files and directories specified in the configuration. Copying should be quite fast and it's more straightforward than linking across releases, which is probably language-dependent solution (i.e. we know that we want to link .stack-work, but with other languages maybe we would want to copy just binary file). In any case the solution implemented here seems to be more general to me.

@mrkkrp LGTM


Is this project dead?

There is no new activity on this project since 7 years, there are 14 pending pull request and many unanswered issues.
I would like to know if the maintainer plans to work on this project again. If not, could this project be put as archived, or transferred to someone else interested to maintain it?
Also, is there an interesting fork to follow, or other alternative tools? I'm trying Pandoc but it's heavy compared to ronn.

Dupe of https://github.com/rtomayko/ronn/issues/103, but the de facto ronn for 2020 is Ronn-NG: apjanke/ronn-ng.


error on building

this error appears randomly here is everything
https://pastebin.com/HgTmBvyn

Might be fixed by removing nokogumbo as explicit dependency (see here). I did not test it myself.

remove nokogumbo

Please try again, as I've updated the dependencies


timeline3 offline content

Timeline 3 was added, however it requires Internet access as implemented. Conforming to the CDN pattern used for all other CSS and JS files the content has been installed and a setting must be set in order to use CDN directly from the Internet Source. This content comes from https://timeline.knightlab.com/
The entire content, not just JS and CSS files are needed for multi-language support and if any non-default fonts are used (according to the documentation).
License information: TimelineJS is released under the Mozilla Public License (MPL), version 2.0. That means that TimelineJS is free to "use, reproduce, make available, modify, display, perform, distribute" or otherwise employ. You don't need our permission to publish stories with TimelineJS and you don't need to pay us any fees or arrange any further license beyond the MPL. To read more about what you can do with TimelineJS, read our license page.

Codecov Report

Merging #906 (040a0e8) into master (cad87b9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #906   +/-   ##
=======================================
  Coverage   74.10%   74.10%
=======================================
  Files          52       52
  Lines        5079     5079
  Branches      601      601
=======================================
  Hits         3764     3764
  Misses       1140     1140
  Partials      175      175

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 cad87b9...040a0e8. Read the comment docs.

Thanks great idea! For future reference, I found the zip distribution of TimelineJS at this link: http://timeline.knightlab.com/docs/instantiate-a-timeline.html.


[BUG] generic leave hook not being called

all over the script there's calls to_callLeave
but there's no call to the generic leave

Sorry, what you mean by generic leave.

leaving any route

A generic leave is not supported at the moment.

This is now supported into the new 8.0.0 version. As of today it is still not released. Soon this will happen. In the meantime if anyone is interested (3+years after this issue is lastly commented) use the navigo@beta version.

Here's the commit that covers this functionality with a unit test https://github.com/krasimir/navigo/commit/75f1a89e6865275a8bfedb21898e2eb4c071fbbb


Removed the cat command

Removed the below command. Public CI doesn't use a  proxy, so the proxy file doesn't exist for Public CI.
RUN cat /etc/apt/apt.conf.d/proxy.conf
Signed-off-by: Susmith Poochali<EMAIL_ADDRESS>

good fix from @vejmarie .. thanks for stepping in !


docs(samples): fix translate with glossary sample

Translation results from glossaries are in a different collection
(glossary_translations rather than translations)
The test wasn't checking that the result matched the translation term from the glossary (fixed test, updated code to pass)
Glossary CSV for this test: https://storage.googleapis.com/cloud-samples-data/translation/glossary.csv

When this lands - please make sure to update the commit message to docs(samples): fix translate with glossary sample so this gets properly handled by our release scripts and stuff :)

@beccasaurus looks good to me as soon as samples tests pass :+1:

Looks like the API behavior has changed and there's a better translation of hello world in Croation, causing the test to fail :) I'll submit a separate PR that we should merge in before this one


Fix spelling error in README

Fix minor nit spelling error in README

@googlebot I signed it!

Awesome!  Thanks for the fix.


listening for reservations at wrong address

Hello,
I am using tenserflowonspark inside containers on multi-host. I have a problem when starting a spark job. I keep getting waiting for 1 reservations problem and Connection timeout errors.
the problem is that the reservation listening address is wrong : I have this inside my stdout log
2018-03-22 16:26:56,485 INFO (MainThread-1769) worker node range range(1, 2), ps node range range(0, 1)
2018-03-22 16:26:56,501 INFO (MainThread-1769) listening for reservations at ('<IP_ADDRESS>', 38871)
2018-03-22 16:26:56,502 INFO (MainThread-1769) Starting TensorFlow on executors
2018-03-22 16:26:56,648 INFO (MainThread-1769) Waiting for TFSparkNodes to start
2018-03-22 16:26:56,648 INFO (MainThread-1769) waiting for 2 reservations
2018-03-22 16:26:57,650 INFO (MainThread-1769) waiting for 1 reservations
2018-03-22 16:26:58,651 INFO (MainThread-1769) waiting for 1 reservations

this adress is wrong normally the communication between cluster (hadoop-master and slaves) should use <IP_ADDRESS>
now I will explain the problem and the architecture.
I am using the same image that I have create in my 2 hosts:
I have 2 container (slave2 and master) in the same host
I have 1 container (slave1) in different host
when using docker in multi-hosts I should use overlay network this produce 2 interfaces inside my containers:

the local container interface with the network <IP_ADDRESS>
and another interface is created to communicate with the host machine using <IP_ADDRESS>

hadoop-master: has address <IP_ADDRESS>  and <IP_ADDRESS>
hadoop-slave2: has address <IP_ADDRESS>  and <IP_ADDRESS>
hadoop-slave1: has address <IP_ADDRESS>  and <IP_ADDRESS> (because not the same host as the master)
the hdoop cluster communicate with each other using host-names created by docker and this uses <IP_ADDRESS> interface.
So the reservation listening address <IP_ADDRESS> should instead be 10.0.0.xxx address
I think there is a problem with my spark or yarn configuration but I am not sure. So i was asking what are you using in order to get the interface where to listen for reservation.
Note1: when creating the cluster inside the same host everything is working fine (this is because they can all see the <IP_ADDRESS> network interface).
Note 2: I think that the node manager is listening  on all interfaces <IP_ADDRESS> but not sure
this is my yarn-site.xml
<configuration>
<property>
<name>yarn.nodemanager.pmem-check-enabled</name>
<value>false</value>
</property>
<property>
<name>yarn.nodemanager.vmem-check-enabled</name>
<value>false</value>
</property>
<property>
<name>yarn.nodemanager.aux-services</name>
<value>mapreduce_shuffle</value>
</property>
<property>
<name>
yarn.nodemanager.aux-services.mapreduce_shuffle.class
</name>
<value>org.apache.hadoop.mapred.ShuffleHandler</value>
</property>
<property>
<name>yarn.resourcemanager.hostname</name>
<value>hadoop-master</value>
</property>
</configuration>

Thank you

If you take a look at the executor logs and search for "Cluster spec", you should see which IP addresses are being registered/used for each node.
That said, we've only just started looking at running TFoS within docker containers, and there may still be issues.  (e.g. in your case, the issue is related to finding the right interface to bind to).

How to look in to the executor logs ?
Is there a solution to my problem
Shouldn't I change one of this parameters https://hadoop.apache.org/docs/r2.4.1/hadoop-yarn/hadoop-yarn-common/yarn-default.xml

I have this log
Showing 4096 bytes. Click here for full log

il.SignalUtils: Registered signal handler for TERM
18/03/22 20:12:34 INFO util.SignalUtils: Registered signal handler for HUP
18/03/22 20:12:34 INFO util.SignalUtils: Registered signal handler for INT
18/03/22 20:12:34 INFO spark.SecurityManager: Changing view acls to: root
18/03/22 20:12:34 INFO spark.SecurityManager: Changing modify acls to: root
18/03/22 20:12:34 INFO spark.SecurityManager: Changing view acls groups to:
18/03/22 20:12:34 INFO spark.SecurityManager: Changing modify acls groups to:
18/03/22 20:12:34 INFO spark.SecurityManager: SecurityManager: authentication disabled; ui acls disabled; users  with view permissions: Set(root); groups with view permissions: Set(); users  with modify permissions: Set(root); groups with modify permissions: Set()
18/03/22 20:12:34 INFO client.TransportClientFactory: Successfully created connection to /<IP_ADDRESS>:37303 after 48 ms (0 ms spent in bootstraps)
18/03/22 20:12:34 INFO spark.SecurityManager: Changing view acls to: root
18/03/22 20:12:34 INFO spark.SecurityManager: Changing modify acls to: root
18/03/22 20:12:34 INFO spark.SecurityManager: Changing view acls groups to:
18/03/22 20:12:34 INFO spark.SecurityManager: Changing modify acls groups to:
18/03/22 20:12:34 INFO spark.SecurityManager: SecurityManager: authentication disabled; ui acls disabled; users  with view permissions: Set(root); groups with view permissions: Set(); users  with modify permissions: Set(root); groups with modify permissions: Set()
18/03/22 20:12:34 INFO client.TransportClientFactory: Successfully created connection to /<IP_ADDRESS>:37303 after 1 ms (0 ms spent in bootstraps)
18/03/22 20:12:34 INFO storage.DiskBlockManager: Created local directory at /tmp/hadoop-root/nm-local-dir/usercache/root/appcache/application_1521747295330_0001/blockmgr-e631b764-9929-4075-b17e-184d643c5574
18/03/22 20:12:34 INFO memory.MemoryStore: MemoryStore started with capacity 366.3 MB
18/03/22 20:12:35 INFO executor.CoarseGrainedExecutorBackend: Connecting to driver: spark://CoarseGrainedScheduler@<IP_ADDRESS>:37303
18/03/22 20:12:35 INFO executor.CoarseGrainedExecutorBackend: Successfully registered with driver
18/03/22 20:12:35 INFO executor.Executor: Starting executor ID 2 on host hadoop-slave1
18/03/22 20:12:35 INFO util.Utils: Successfully started service 'org.apache.spark.network.netty.NettyBlockTransferService' on port 46680.
18/03/22 20:12:35 INFO netty.NettyBlockTransferService: Server created on hadoop-slave1:46680
18/03/22 20:12:35 INFO storage.BlockManager: Using org.apache.spark.storage.RandomBlockReplicationPolicy for block replication policy
18/03/22 20:12:35 INFO storage.BlockManagerMaster: Registering BlockManager BlockManagerId(2, hadoop-slave1, 46680, None)
18/03/22 20:12:35 INFO storage.BlockManagerMaster: Registered BlockManager BlockManagerId(2, hadoop-slave1, 46680, None)
18/03/22 20:12:35 INFO storage.BlockManager: Initialized BlockManager: BlockManagerId(2, hadoop-slave1, 46680, None)
18/03/22 20:12:37 INFO executor.CoarseGrainedExecutorBackend: Got assigned task 0
18/03/22 20:12:37 INFO executor.Executor: Running task 0.0 in stage 0.0 (TID 0)
18/03/22 20:12:37 INFO broadcast.TorrentBroadcast: Started reading broadcast variable 0
18/03/22 20:12:37 INFO client.TransportClientF
```actory: Successfully created connection to /<IP_ADDRESS>:37624 after 1 ms (0 ms spent in bootstraps)
18/03/22 20:12:37 INFO memory.MemoryStore: Block broadcast_0_piece0 stored as bytes in memory (estimated size 9.9 KB, free 366.3 MB)
18/03/22 20:12:37 INFO broadcast.TorrentBroadcast: Reading broadcast variable 0 took 70 ms
18/03/22 20:12:37 INFO memory.MemoryStore: Block broadcast_0 stored as values in memory (estimated size 14.1 KB, free 366.3 MB)
2018-03-22 20:12:38,326 INFO (MainThread-664) connected to server at ('<IP_ADDRESS>', 43889)
2018-03-22 20:12:38,327 INFO (MainThread-664) TFSparkNode.reserve: {'job_name': 'ps', 'addr': ('<IP_ADDRESS>', 38601), 'tb_pid': 0, 'worker_num': 0, 'task_index': 0, 'host': '<IP_ADDRESS>', 'authkey': b'\x89\xb3\x1d\x9c\xa4\xcdJF\xa7\x8d\x0f\x8ef\xd88\x9c', 'port': 36061, 'tb_port': 0, 'ppid': 654}

normally the address <IP_ADDRESS> should not be used.

Can you try this pyspark script?  It's a simple script (w/o TFoS or TF) that just tries to see what IP address each executor binds to...
# pyspark --master yarn

def getIP(iter):
  import socket
  s = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
  s.connect(("<IP_ADDRESS>", 80))
  return [s.getsockname()[0]]

rdd = sc.parallelize(range(3),3)
hosts = rdd.mapPartitions(getIP).collect()
for h in hosts:
  print(h)

this what I get
SparkSession available as 'spark'.
>>> from pyspark import SparkContext
>>> sc =SparkContext()
Traceback (most recent call last):
  File "/usr/local/spark/python/pyspark/context.py", line 237, in signal_handler
    raise KeyboardInterrupt()
KeyboardInterrupt
>>> rdd = sc.parallelize(range(3),3)
>>> hosts = rdd.mapPartitions(getIP).collect()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'getIP' is not defined
>>> def getIP(iter):
...   import socket
...   s = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
...   s.connect(("<IP_ADDRESS>", 80))
...   return [s.getsockname()[0]]
...
>>> hosts = rdd.mapPartitions(getIP).collect()
>>> for h in hosts:
...   print(h)
...
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

Normally there should not be two <IP_ADDRESS> but two <IP_ADDRESS> and
when I have started pyspark it was taking too long and I got this
To adjust logging level use sc.setLogLevel(newLevel). For SparkR, use setLogLevel(newLevel).
18/03/22 21:57:52 WARN yarn.Client: Neither spark.yarn.jars nor spark.yarn.archive is set, falling back to uploading libraries under SPARK_HOME.
18/03/22 21:59:25 WARN metastore.ObjectStore: Version information not found in metastore. hive.metastore.schema.verification is not enabled so recording the schema version 1.2.0
18/03/22 21:59:25 WARN metastore.ObjectStore: Failed to get database default, returning NoSuchObjectException
18/03/22 21:59:30 WARN metastore.ObjectStore: Failed to get database global_temp, returning NoSuchObjectException

I know that with this my issues should be moved to docker or hadoop/spark forum because the problem is not related to the TenserflowOnSpark project but I will appreciate any help you can give me.
Thank you very much.

I am confirming now the problem: I think that it is related to this function :
def getIP(iter):
  import socket
  s = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
  s.connect(("<IP_ADDRESS>", 80))
  return [s.getsockname()[0]]

We must use another method in order to find the correct interface

@MohamedAmineOuali can you try:
def getPI(iter):
  import socket
  return socket.gethostname()

This was the original code before PR #109

Closing due to inactivity.  Feel free to reopen if still an issue.

Hello,
We are having some troubles with reservation, this is our setup:
In our cluster each worker have 3 network interfaces (3 diferent ip's):

10.0.2.X
192.168.1.X
192.168.201.X

But just with one (192.168.201.X) they can communicate with each other.
When we execute tensorflowonspark version 1.3.1 reservation process uses the ip in range 10.0.2.x this produces an error.
To solve the problem we have modified get_ip_address function inside util.py to
def get_ip_address():     """Simple utility to get host IP address."""     return socket.gethostname()
moving 1.3.1 code to old version (@leewyang https://github.com/yahoo/TensorFlowOnSpark/issues/251#issuecomment-376337722)
with this little modification all works fine.
But this modification will not work if the nodes can't resolve the name of the workers (lack of DNS or no /etc/hosts configuration)
So. Is there a way to enhance get_ip_address to find the correct ip when the workers have more than one IP?
Thanks in advance.

@MohamedAmineOuali can you try:
def getIP(iter):
  import socket
  return socket.gethostname()

This was the original code before PR #109

thanks very much ! It works.
I use the vagrant to manage the vms,  and i provide a host-only ip for each one, but each vm also has a nat network ip which can not connect to others.


iOS-507 Handle remote notification

handle remote notification
fix some warning
update cocoapods version to 1.2.0 Beta 1
remove SSL encryption (Dev scheme only)
modify version string, add build version.

LGTM


Add LAYERDEPEND on intel layer.

I guess the right fix is to use dynamic layers but that's a bigger effort than this for the time being.

LGTM!


Orbstack 1.6.1 bind mounts have become buggy

Describe the bug
In my normal workflow I use a docker container for my build environment but edit my source files from the Mac side using iTerm and vim.  I have found that with Orbstack 1.6.1 when I do a "git rebase" from the macOS side and then later do a git operation from within the docker container on the bind-mounted repository that it will often complain about not being in a git repository.  If I run the git operation a second time from within the docker container, it works.
I understand that it is a bit odd to be doing git operations from both the Mac side and the docker side, but as part of our build process the build needs to do some git operations to do things like record the SHA1 of the commit we are building off of.
I have been able to reproduce this with a minimal test script and have attached instructions below.  Running the same test in Orbstack 1.5.1 works fine, so I have reverted to 1.5.1 for now.
To Reproduce
In a terminal on the macOS side:
Create a directory for the test:
% mkdir orbstack_git_test
% cd orbstack_git_test

Create the Dockerfile:
% cat > Dockerfile <<END
FROM ubuntu:22.04
ENV LANG C.UTF-8
RUN yes | unminimize
RUN apt-get update -y \
 && DEBIAN_FRONTEND=noninteractive apt-get install -y \
    git
RUN mkdir -p /mnt/bind-root
WORKDIR /mnt/bind-root
END

Build the orbstack-git-test container:
% docker build -t orbstack-git-test .

Create a small git repository here:
% git init
% git add Dockerfile
% git commit -m "Initial commit."
% touch foo
% git add foo
% git commit -m "Second commit."
% touch bar
% git add bar
% git commit -m "Third commit."
% touch baz
% git add baz
% git commit -m "Fourth commit."

Use interactive git rebase: go to the second commit and tell it to do "edit" instead of "pick" so that we stop there.  Do this twice:
% git rebase -i --root
% git rebase --continue
% git rebase -i --root

Finally, run a 'git status' command from within the docker container, and note the following error output:
% docker run -ti --rm --mount type=bind,source="$(pwd)",target=/mnt/bind-root orbstack-git-test git status
fatal: not a git repository (or any parent up to mount point /mnt)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).

Next commands to do (2 remaining commands):
   pick 4e7d92c Third commit.
   pick d048f02 Fourth commit.
  (use "git rebase --edit-todo" to view and edit)
You are currently editing a commit while rebasing branch 'main' on 'd553c6d'.
  (use "git commit --amend" to amend the current commit)
  (use "git rebase --continue" once you are satisfied with your changes)

nothing to commit, working tree clean

Expected behavior
The git operation from within the docker container should work, as it does correctly in Orbstack 1.5.1.
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.6.1
Commit: bfaddc6839de8b00b7aff767dbd673ac6ad4259e (v1.6.1)
System info:
macOS: 14.3.1 (23D60)
CPU: arm64, 10 cores
CPU model: Apple M1 Max
Model: MacBookPro18,4
Memory: 32 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-06-01T04-56-33.363470Z.zip

Screenshots and additional context (optional)
No response

Looks like it's same issue as I reported a few days ago: https://github.com/orbstack/orbstack/issues/1234
I also found there is a big lag on mounted volumes (up to several seconds even on non-busy systems) and had to go back to 1.5.1 unfortunately.

I've also gone back to 1.5.1, but when I was working with 1.6.1 I had felt that it wasn't time-dependent.  Like, my feeling was that at the time I would have done a rebase and not touched my git repo for awhile while editing a file for an extended period then went to do a build and still got the errors.  I haven't tested it again to prove that feeling though and since 1.5.1 is working for me I will just stick with that for now.

Yeah this can be not time-dependent as well. you're right. Anyway, files are missing starting 1.6 :-(

I'm having this same issue, oddly if I go into the container and run ls in the problematic directory the affected files resolve, but in a multi-directory complex application it is impossible to ls into EVERY directory in order to straighten out permission issues.

We are also encountering a similar issue. Our use case involves making changes on host outside of a container and using Air (https://github.com/air-verse/air) to live reload a Go application running inside the container. When saving files, the live reload does not trigger as expected. Inspecting inside the container shows the content updating, but the file metadata, such as the modified date, does not change. Downgrading to Orbstack 1.5.1 fixes the issue.

I've been experiencing intermittent issues where terraform running inside a docker container which mounts directories from the host. I haven't been able to reproduce the issue reliably, but here is an example of the error terraform reports:
Error: Failed to read file

The file "providers.tf" could not be read.

In this case providers.tf is a symlink, but I have seen similar cases where orbstack complains about not being able to find physical files.
Restarting orbstack seems to fix the issues - these issues are unique to 1.6.x - if I roll back to v1.5.1, the issues simply never occur or occur so infrequently as to render the distinction irrelevant.

@levisegal Can you share a reproducer? That sounds like a different issue.

#1234 has been fixed for the next version. The git reproducer here wasn't working for me, so I'm not sure whether it's the same issue.
@jonseymour @oranges13 @tgreeniaus @pchalamet I think you were all running into #1234. Can you try this build? https://cdn-updates.orbstack.dev/exp/OrbStack_v1.6.1-56-g09248bb11_17066_arm64.dmg
I don't think it's a new issue, but it seems to happen far more often in v1.6.x.
@levisegal Yours is probably a different issue that's also fixed in this build, so it'd be great if you could also give it a try.
There's a new fix for #1148 as well.
Let me know if it's not fixed and I'll reopen. Thanks!

In my opinion, it does not depend on time. Random files are not present after git checkout at all. I'm getting Unable to read file '<random file>'. Failed to open stream: No such file or directory error message in case of a PHP app.

@burithetech Can you try the test build above?

@burithetech Can you try the test build above?

It seems the app is working again. But just a quick check. I'll use it all day and I'll give you a feedback here. Thanks. 🙂

@kdrag0n Still running on x64 on my side on primary machine. Can you generate an x64 build for testing ?

@kdrag0n I'm also still on an intel mac as well. Can you generate an x64 build? Happy to test today.

x86 build: https://cdn-updates.orbstack.dev/exp/OrbStack_v1.6.1-57-geada42fb5_17067_amd64.dmg

Great thanks, I will test today and let you know.

Looks good to me - at least from volume mount this looks good. I will comment on https://github.com/orbstack/orbstack/issues/1234 because there is a weird output just for info.

@kdrag0n this issue looks good to me. My primary issue was when traversing between git branches or editing files in an external IDE, those files would then become inaccessible, and that is no longer occurring with this build, so I consider the issue fixed.

Great! Thanks for all the testing.

@burithetech Can you try the test build above?

It seems the app is working again. But it was just a quick check. I'll use it all day and I'll give you a feedback here. Thanks. 🙂

I used it all day, I did a few rebases, a lot of checkouts, and so on and I had no issues at all. 🙂 Thanks. (arm64 build)

As a quick test I did a few git operations and the arm version seems to be working now!  I will run with it all day today and see if the issue pops up again but looks good so far.  Thank you for the quick update!

Released in v1.6.2.

@kdrag0n Our team ran the latest build and everything seems to be working correctly again - Air is picking up updates and we can't reproduce the issue.
Thanks!

I've also been working with Orbstack 1.6.2 and the problem seems to have been resolved.  Thanks for the fix and I really enjoy using Orbstack so thanks for making an awesome product!

#1234 has been fixed for the next version. The git reproducer here wasn't working for me, so I'm not sure whether it's the same issue.
@jonseymour @oranges13 @tgreeniaus @pchalamet I think you were all running into #1234. Can you try this build? https://cdn-updates.orbstack.dev/exp/OrbStack_v1.6.1-56-g09248bb11_17066_arm64.dmg

Thanks, the issues I was having with a terraform image appear to have been solved with v1.6.2.
I did later experience some aberrant FS-related behaviour with another set of docker images, but haven't had the time to workout whether orbstack was actually a factor in these issues or not. If I see any solid evidence of an issue with orbstack, I will update accordingly.

I still have synchronization issues.


Static vs. nonstatic method conflicts are properly reported

fixes #2255

Failure seems related:
+++ Conformance\DeclarationElements\MemberDefinitions\MethodsAndProperties (E_UseInstMethodThroughStatic.fs) +++
Microsoft (R) F# Compiler version 4.1
Copyright (c) Microsoft Corporation. All Rights Reserved.
(10,14): error FS3214: Method or object constructor 'DoStuff' is not static

*** The following necessary lines were never matched:
***	\(10,14\):.+error FS0509:.+Method or object constructor 'DoStuff' is not static$

*** The following necessary lines were incorrectly matched:

Unexpected Compiler Output
FAIL

fixed


seems to fire incorrectly on

If I open a <select> and immediately close it, it doesn't fire.
If I open a <select> and start moving through the options to choose one, it fires
note; there's no long-press involved, it's a quick tap/click

Hi @FirefighterBlu3, please can you include jsfiddle or similar so I can see this issue in action?


Update the URL used to point to help resources.

ping @ldko, @madhulika95b, this is ready for review.  Just needed to add the /metadata/fields/ path to the URL, as that's now the base location for the help documentation we're targeting.


Readthedocs build should fail on warnings

In order to do this, we should set fail_on_warning: true in .readthedocs.yaml.
Before doing so, however, we need to investigate and resolve the numerous autodoc warnings from the build.

There is a related setting SPHINXOPTS      = -W  # fail on warnings in the Makefile, as seen here: https://github.com/openedx/open-edx-proposals/blob/8dfc826fc83870c247f22f4c55a3dada0f334c32/Makefile#L5.

@robrap That's what I was using to test locally

Cool. And if we implement this ticket, noting that both changes should happen in parallel.


ImportError: cannot import name legacy_seq2seq

Hi!
I just downloaded the code and tried to run it with the train.py with tf-v1.0. I think it should train the Shakespeare example. Readme says default parameters. However it fails
Traceback (most recent call last):
  File "train.py", line 11, in <module>
    from model import Model
  File "/opt/gpu-project/char-rnn-tensorflow/model.py", line 3, in <module>
    from tensorflow.contrib import legacy_seq2seq
ImportError: cannot import name legacy_seq2seq

Hmm, my fault. I ran it with python and not with python3.

This should work with Python 2.7, just make sure you have Tensorflow 1.0 (and not, say, the latest Tensorflow 1.2).
pip install tensorflow==1.0


Update docs to use tokio's recv instead of next

Update the docs tokio's recv() instead of next(). https://tokio-rs.github.io/tokio/doc/tokio/sync/mpsc/struct.UnboundedReceiver.html

Unless I'm missing something, next() is the same as recv(), except that for next() you have to have StreamExt in scope, correct?
In that case, I agree, the documentation will be updated to use recv.

I believe that is correct yes. I found when following the examples to the letter, with tokio v 0.2.22 due to other dependencies in my project it refused to compile. switching to recv() was the simplest fix in my case.


Commit node wrapper package.json, updated dependencies

Signed-off-by: Patrik Stas<EMAIL_ADDRESS>

Codecov Report

Merging #282 (4ba49b9) into master (d068d02) will decrease coverage by 4.29%.
The diff coverage is n/a.

:exclamation: Current head 4ba49b9 differs from pull request most recent head 5d7045e. Consider uploading reports for the commit 5d7045e to get more accurate results

@@            Coverage Diff             @@
##           master     #282      +/-   ##
==========================================
- Coverage   49.99%   45.69%   -4.30%
==========================================
  Files         150      149       -1
  Lines       19205    18312     -893
  Branches     6049     5727     -322
==========================================
- Hits         9601     8368    -1233
- Misses       4898     5897     +999
+ Partials     4706     4047     -659

Flag
Coverage Δ

integration
?

unittests
45.69% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

libvcx/src/libindy/utils/pool.rs
14.58% <0.00%> (-52.42%)
:arrow_down:

libvcx/src/libindy/utils/anoncreds.rs
18.04% <0.00%> (-46.50%)
:arrow_down:

libvcx/src/utils/plugins.rs
0.00% <0.00%> (-38.10%)
:arrow_down:

libvcx/src/libindy/utils/ledger.rs
12.00% <0.00%> (-28.00%)
:arrow_down:

libvcx/src/utils/random.rs
75.00% <0.00%> (-25.00%)
:arrow_down:

libvcx/src/utils/devsetup.rs
40.08% <0.00%> (-23.97%)
:arrow_down:

libvcx/src/utils/devsetup_agent.rs
0.00% <0.00%> (-21.17%)
:arrow_down:

libvcx/src/init.rs
25.80% <0.00%> (-20.97%)
:arrow_down:

libvcx/src/credential_def.rs
36.72% <0.00%> (-14.74%)
:arrow_down:

libvcx/src/libindy/utils/payments.rs
39.25% <0.00%> (-12.57%)
:arrow_down:

... and 27 more

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 d068d02...5d7045e. Read the comment docs.


zip file structure is broken

Describe the bug
while sending folders as zip files the structure of zip files is broken
say for example i'm sending a folder Downloads/folder then the zip file structure while extracting is folder/folder/*.*
To Reproduce
Steps to reproduce the behavior:
oneshot -a zip path to any folder
Expected behavior
the file structure while extracting should be like folder/*.* instead of folder/folder/*.*
Desktop:

OS: arch linux
Version: 1.3.1

I can't seem to reproduce this error. I tried sending the internal folder of this repo as a zip archive: oneshot -a zip ./internal.
Here's what I get when I unzip the archive:
$ unzip ~/Downloads/internal.zip
Archive:  /home/rr/Downloads/internal.zip
  inflating: internal/file/reader.go
  inflating: internal/file/tar.go
  inflating: internal/file/writer.go
  inflating: internal/file/zip.go
  inflating: internal/handlers/authenticate.go
  inflating: internal/handlers/bots.go
  inflating: internal/handlers/cgi.go
  inflating: internal/handlers/download.go
  inflating: internal/handlers/redirect.go
  inflating: internal/handlers/upload.go
  inflating: internal/server/add-route.go
  inflating: internal/server/route.go
  inflating: internal/server/server.go

If I'm understanding this correctly, unzip is inflating to internal/internal/file/reader.go... etc. for you?

@raphaelreyna yeah that's correct. maybe I'll try it again later and let you know?

@raphaelreyna it's works fine if i extract the file on my pc. i generally use this program to send files b/w my pc and my phone. while extracting the zip files on my phone this problem occurs. strange idk what's wrong with my file manager. i'll close this issue now as it's not related to the program.


Fix logging assertion

What?

Turns out I was not on latest master before
Fixes logging assertion

cc/ @KnVerey

:shipit:


[not always reproducible] RKE2 Nodes are not being removed from backend/provider when deleting infrastructure-provider clusters through rancher

Rancher Server Setup

Rancher version:v2.6.0-rc6
Installation option (Docker install/Helm Chart):docker

If Helm Chart, Kubernetes Info:
Cluster Type (RKE1, RKE2, k3s, EKS, etc):
Node Setup:
Version:

Proxy/Cert Details:self signed

Information about the Cluster

Kubernetes version:v1.21.4/1.21.3 (RKE2)
Cluster Type (Local/Downstream): downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): Infrastructure provider - Azure, DO

Describe the bug

Sometimes, nodes are not being deleted from the provider when the cluster is removed through rancher.
To Reproduce

provision a cluster using an infrastructure provider (previously observed with azure, digital ocean)

wait for the cluster to come to active

delete the cluster through rancher

wait for rancher to mark the cluster as deleted

through the provider you chose, ensure your node(s) from the cluster have been deleted

Result
sometimes, the nodes are not removed from the provider
Expected Result

all nodes should be removed when deleting an infrastructure-provider cluster through rancher
Screenshots

Additional context

this has been observed on Azure and Digital Ocean with RKE2
this has only been observed so far with multi-node clusters

A couple of notes on this:

If you create a cluster then delete it before the nodes finish provisioning, then the cluster will be deleted from Rancher and the nodes leftover in the cloud provider. (This will be fixed and should be tested as part of the validation of the fix).
As far as the "not always reproducible" status of the issue: previously if the deletion job (to remove the machine from the cloud provider) failed, it would be ignored and never retried. Now the job will at least retry and this issue should be resolved or at least occur much less often.

Suggesting to add improvements to cluster deletion to 2.6.1 release notes. Possibly as one bullet point for all issues mentioned on the PR linked here.

My checks PASSED
Reproduction Environment:

Rancher version: v2.6.0
Rancher cluster type: single-node docker install
Docker version: 20.10
Downstream cluster type: RKE2
Downstream K8s version: v1.21.4+rke2r2
Reproduction steps:

In order to try to reproduce, create a multi-pool cluster. I used the EC2 node driver. Each pool having 2 or more nodes. I had to have more than a few or I just couldn't repro - went with etcd 5, cp 5, worker 5. I created the cluster and waited a bit for the ec2 instances to just BARELY provision (show up in aws) but didn't let the cluster finish creating I then deleted it. (e.g. wait for Creating server (Amazonec2Machine) in infrastructure provider status to almost go away but not quite, check aws and see ec2 instances there - don't wait longer or the cleanup will work; ec2 instances cannot get to Running state before attempting to delete the cluster -- or at least this was my experience).
I waited a bit for the cluster to be removed in rancher - I checked and I was indeed able to reproduce -- but even with 15 nodes usually only 1 or 2 get left behind -- good enough for me. Note, I also waited a bit just in case Rancher was not done cleaning them up (deleting them) still nothing so I indeed did reproduce the issue :)
I manually deleted the ec2 instances left behind.

Validation Environment:

Rancher version: v2.6-head a9e61c5 pulled at 9/24/21 ~2:20pm Pacific
Rancher cluster type: single-node docker install
Docker version: 20.10
Downstream cluster type: RKE2
Downstream K8s version: v1.21.4+rke2r2
Validation steps:

Perform the same steps as are outlined above in the reproduction section, except ensure that all ec2 nodes always get cleaned up. This required multiple attempts to reproduce the problem because the first time I reproduced it took four tries. I attempted this a total of six times to be mostly sure the fix works. I indeed could no longer reproduce, thus most likely the issue is fixed with the code change.

Here we can see from all six attempts to reproduce (validate the issue is fixed) none of the times did Rancher leave any ec2 instances behind:


update openttd to 1.5.2

Thank you for the contribution. It was merged directly as f89127a923abacf9507530ada74f5ecd6473cce3 to keep commit history cleaner. Your contribution is still credited to you.


More JS stream simplifications

More simplification of the JS-backed streams...
Previously, the code was needlessly jumping through a few weakref hoops to try to avoid a hard circular ref. These can be handled just fine via tracing since v8 now more optimistically gc'ing wrapper objects. Long overdue change.

There's a relevant internal ASAN test failure to investigate before this can land